Linaro GCC

Merge lp:~ams-codesourcery/gcc-linaro/thumb-ifcvt2 into lp:gcc-linaro/4.7

  1. thumb-ifcvt2
  2. Merge into 4.7
Proposed by Andrew Stubbs
Status: Superseded
Proposed branch: lp:~ams-codesourcery/gcc-linaro/thumb-ifcvt2
Merge into: lp:gcc-linaro/4.7
To merge this branch: bzr merge lp:~ams-codesourcery/gcc-linaro/thumb-ifcvt2
Reviewer Review Type Date Requested Status
Michael Hope Needs Fixing
Ulrich Weigand Pending
Review via email: mp+93596@code.launchpad.net

This proposal supersedes a proposal from 2012-02-15.

This proposal has been superseded by a proposal from 2012-02-20.

Description of the change

Remove peephole optimizations that impede if-conversion.

UPDATE 20120217: Fixed regressions.

Posted upstream: http://gcc.gnu.org/ml/gcc-patches/2012-02/msg00734.html

To post a comment you must log in.
Revision history for this message
Michael Hope (michaelh1) wrote : Posted in a previous version of this proposal

cbuild has taken a snapshot of this branch at r114966 and queued it for build.

The diff against the ancestor r114965 is available at:
 http://builds.linaro.org/toolchain/snapshots/gcc-linaro-4.7+bzr114966~ams-codesourcery~thumb-ifcvt2.diff

and will be built on the following builders:
 a9-builder armv5-builder i686 x86_64

You can track the build queue at:
 http://ex.seabright.co.nz/helpers/scheduler

cbuild-snapshot: gcc-linaro-4.7+bzr114966~ams-codesourcery~thumb-ifcvt2
cbuild-ancestor: lp:gcc-linaro/4.7+bzr114965
cbuild-state: check

Revision history for this message
Michael Hope (michaelh1) wrote : Posted in a previous version of this proposal

cbuild successfully built this on x86_64-natty-cbuild245-oort1-x86_64r1.

The build results are available at:
 http://ex.seabright.co.nz/build/gcc-linaro-4.7+bzr114966~ams-codesourcery~thumb-ifcvt2/logs/x86_64-natty-cbuild245-oort1-x86_64r1

The testsuite results are the same as the branch point lp:gcc-linaro/4.7+bzr114965

The full diff is at:
 http://ex.seabright.co.nz/build/gcc-linaro-4.7+bzr114966~ams-codesourcery~thumb-ifcvt2/logs/x86_64-natty-cbuild245-oort1-x86_64r1/testsuite-diff.txt

The full testsuite results are at:
 http://ex.seabright.co.nz/build/gcc-linaro-4.7+bzr114966~ams-codesourcery~thumb-ifcvt2/logs/x86_64-natty-cbuild245-oort1-x86_64r1/gcc-testsuite.txt

cbuild-checked: x86_64-natty-cbuild245-oort1-x86_64r1

Revision history for this message
Michael Hope (michaelh1) wrote : Posted in a previous version of this proposal

cbuild successfully built this on armv7l-natty-cbuild245-tcpanda05-armv5r2.

The build results are available at:
 http://ex.seabright.co.nz/build/gcc-linaro-4.7+bzr114966~ams-codesourcery~thumb-ifcvt2/logs/armv7l-natty-cbuild245-tcpanda05-armv5r2

+UNSUPPORTED: gcc.target/arm/thumb-ifcvt.c

The full diff is at:
 http://ex.seabright.co.nz/build/gcc-linaro-4.7+bzr114966~ams-codesourcery~thumb-ifcvt2/logs/armv7l-natty-cbuild245-tcpanda05-armv5r2/testsuite-diff.txt

The full testsuite results are at:
 http://ex.seabright.co.nz/build/gcc-linaro-4.7+bzr114966~ams-codesourcery~thumb-ifcvt2/logs/armv7l-natty-cbuild245-tcpanda05-armv5r2/gcc-testsuite.txt

cbuild-checked: armv7l-natty-cbuild245-tcpanda05-armv5r2

Revision history for this message
Michael Hope (michaelh1) wrote : Posted in a previous version of this proposal

cbuild successfully built this on i686-natty-cbuild245-oort2-i686r1.

The build results are available at:
 http://ex.seabright.co.nz/build/gcc-linaro-4.7+bzr114966~ams-codesourcery~thumb-ifcvt2/logs/i686-natty-cbuild245-oort2-i686r1

The testsuite results are the same as the branch point lp:gcc-linaro/4.7+bzr114965

The full diff is at:
 http://ex.seabright.co.nz/build/gcc-linaro-4.7+bzr114966~ams-codesourcery~thumb-ifcvt2/logs/i686-natty-cbuild245-oort2-i686r1/testsuite-diff.txt

The full testsuite results are at:
 http://ex.seabright.co.nz/build/gcc-linaro-4.7+bzr114966~ams-codesourcery~thumb-ifcvt2/logs/i686-natty-cbuild245-oort2-i686r1/gcc-testsuite.txt

cbuild-checked: i686-natty-cbuild245-oort2-i686r1

Revision history for this message
Michael Hope (michaelh1) wrote : Posted in a previous version of this proposal

cbuild successfully built this on armv7l-natty-cbuild245-tcpanda06-cortexa9r1.

The build results are available at:
 http://ex.seabright.co.nz/build/gcc-linaro-4.7+bzr114966~ams-codesourcery~thumb-ifcvt2/logs/armv7l-natty-cbuild245-tcpanda06-cortexa9r1

-PASS: gcc.target/arm/ivopts-2.c object-size text <= 26
+FAIL: gcc.target/arm/ivopts-2.c object-size text <= 26
-PASS: gcc.target/arm/ivopts-3.c object-size text <= 30
+FAIL: gcc.target/arm/ivopts-3.c object-size text <= 30
-PASS: gcc.target/arm/ivopts-4.c object-size text <= 36
+FAIL: gcc.target/arm/ivopts-4.c object-size text <= 36
-PASS: gcc.target/arm/ivopts-5.c object-size text <= 30
+FAIL: gcc.target/arm/ivopts-5.c object-size text <= 30
-PASS: gcc.target/arm/ivopts.c object-size text <= 20
+FAIL: gcc.target/arm/ivopts.c object-size text <= 20
-PASS: gcc.target/arm/pr43597.c object-size text <= 30
+FAIL: gcc.target/arm/pr43597.c object-size text <= 30
-PASS: gcc.target/arm/pr43920-2.c object-size text <= 54
+FAIL: gcc.target/arm/pr43920-2.c object-size text <= 54
+PASS: gcc.target/arm/thumb-ifcvt.c (test for excess errors)
+PASS: gcc.target/arm/thumb-ifcvt.c scan-assembler asrne
+PASS: gcc.target/arm/thumb-ifcvt.c scan-assembler lslne
-PASS: gcc.target/arm/thumb2-mul-space.c scan-assembler muls
+FAIL: gcc.target/arm/thumb2-mul-space.c scan-assembler muls

The full diff is at:
 http://ex.seabright.co.nz/build/gcc-linaro-4.7+bzr114966~ams-codesourcery~thumb-ifcvt2/logs/armv7l-natty-cbuild245-tcpanda06-cortexa9r1/testsuite-diff.txt

The full testsuite results are at:
 http://ex.seabright.co.nz/build/gcc-linaro-4.7+bzr114966~ams-codesourcery~thumb-ifcvt2/logs/armv7l-natty-cbuild245-tcpanda06-cortexa9r1/gcc-testsuite.txt

cbuild-checked: armv7l-natty-cbuild245-tcpanda06-cortexa9r1

Revision history for this message
Ulrich Weigand (uweigand) wrote : Posted in a previous version of this proposal

Those regressions appear to be real at first glance --- can you have a look?

review: Needs Fixing
Revision history for this message
Michael Hope (michaelh1) wrote :

cbuild has taken a snapshot of this branch at r114968 and queued it for build.

The diff against the ancestor r114965 is available at:
 http://builds.linaro.org/toolchain/snapshots/gcc-linaro-4.7+bzr114968~ams-codesourcery~thumb-ifcvt2.diff

and will be built on the following builders:
 a9-builder armv5-builder i686 x86_64

You can track the build queue at:
 http://ex.seabright.co.nz/helpers/scheduler

cbuild-snapshot: gcc-linaro-4.7+bzr114968~ams-codesourcery~thumb-ifcvt2
cbuild-ancestor: lp:gcc-linaro/4.7+bzr114965
cbuild-state: check

Revision history for this message
Michael Hope (michaelh1) wrote :

cbuild successfully built this on armv7l-natty-cbuild250-tcpanda05-armv5r2.

The build results are available at:
 http://ex.seabright.co.nz/build/gcc-linaro-4.7+bzr114968~ams-codesourcery~thumb-ifcvt2/logs/armv7l-natty-cbuild250-tcpanda05-armv5r2

+UNSUPPORTED: gcc.target/arm/thumb-16bit-ops.c
+UNSUPPORTED: gcc.target/arm/thumb-ifcvt.c

The full diff is at:
 http://ex.seabright.co.nz/build/gcc-linaro-4.7+bzr114968~ams-codesourcery~thumb-ifcvt2/logs/armv7l-natty-cbuild250-tcpanda05-armv5r2/testsuite-diff.txt

The full testsuite results are at:
 http://ex.seabright.co.nz/build/gcc-linaro-4.7+bzr114968~ams-codesourcery~thumb-ifcvt2/logs/armv7l-natty-cbuild250-tcpanda05-armv5r2/gcc-testsuite.txt

cbuild-checked: armv7l-natty-cbuild250-tcpanda05-armv5r2

Revision history for this message
Michael Hope (michaelh1) wrote :

cbuild had trouble building this on armv7l-natty-cbuild250-tcpanda06-cortexa9r1.

See the *failed.txt logs under the build results at:
 http://ex.seabright.co.nz/build/gcc-linaro-4.7+bzr114968~ams-codesourcery~thumb-ifcvt2/logs/armv7l-natty-cbuild250-tcpanda06-cortexa9r1

The test suite results were not checked.

cbuild-checked: armv7l-natty-cbuild250-tcpanda06-cortexa9r1

review: Needs Fixing
Revision history for this message
Michael Hope (michaelh1) wrote :

cbuild successfully built this on i686-natty-cbuild250-oort2-i686r1.

The build results are available at:
 http://ex.seabright.co.nz/build/gcc-linaro-4.7+bzr114968~ams-codesourcery~thumb-ifcvt2/logs/i686-natty-cbuild250-oort2-i686r1

The testsuite results are the same as the branch point lp:gcc-linaro/4.7+bzr114965

The full diff is at:
 http://ex.seabright.co.nz/build/gcc-linaro-4.7+bzr114968~ams-codesourcery~thumb-ifcvt2/logs/i686-natty-cbuild250-oort2-i686r1/testsuite-diff.txt

The full testsuite results are at:
 http://ex.seabright.co.nz/build/gcc-linaro-4.7+bzr114968~ams-codesourcery~thumb-ifcvt2/logs/i686-natty-cbuild250-oort2-i686r1/gcc-testsuite.txt

cbuild-checked: i686-natty-cbuild250-oort2-i686r1

Revision history for this message
Michael Hope (michaelh1) wrote :

cbuild successfully built this on x86_64-natty-cbuild250-oort1-x86_64r1.

The build results are available at:
 http://ex.seabright.co.nz/build/gcc-linaro-4.7+bzr114968~ams-codesourcery~thumb-ifcvt2/logs/x86_64-natty-cbuild250-oort1-x86_64r1

The testsuite results are the same as the branch point lp:gcc-linaro/4.7+bzr114965

The full diff is at:
 http://ex.seabright.co.nz/build/gcc-linaro-4.7+bzr114968~ams-codesourcery~thumb-ifcvt2/logs/x86_64-natty-cbuild250-oort1-x86_64r1/testsuite-diff.txt

The full testsuite results are at:
 http://ex.seabright.co.nz/build/gcc-linaro-4.7+bzr114968~ams-codesourcery~thumb-ifcvt2/logs/x86_64-natty-cbuild250-oort1-x86_64r1/gcc-testsuite.txt

cbuild-checked: x86_64-natty-cbuild250-oort1-x86_64r1

Subscribers

People subscribed via source and target branches