Linaro GCC

Merge lp:~christophe-lyon/gcc-linaro/backport_bswap16-v2 into lp:gcc-linaro/4.7

  1. backport_bswap16-v2
  2. Merge into 4.7
Proposed by Christophe Lyon
Status: Merged
Approved by: Yvan Roux
Approved revision: no longer in the source branch.
Merge reported by: Yvan Roux
Merged at revision: not available
Proposed branch: lp:~christophe-lyon/gcc-linaro/backport_bswap16-v2
Merge into: lp:gcc-linaro/4.7
To merge this branch: bzr merge lp:~christophe-lyon/gcc-linaro/backport_bswap16-v2
Reviewer Review Type Date Requested Status
Yvan Roux Approve
Review via email: mp+140612@code.launchpad.net

Description of the change

New attempt, after rebasing on an up-to-date gcc-linaro/4.7 version, to try to avoid false regression reports.

To post a comment you must log in.
Revision history for this message
Linaro Toolchain Builder (cbuild) wrote :

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

The diff against the ancestor r115040 is available at:
 http://cbuild.validation.linaro.org/snapshots/gcc-linaro-4.7+bzr115041~christophe-lyon~backport_bswap16-v2.diff

and will be built on the following builders:
 a9-builder a9hf-builder armv5-builder i686 x86_64 xaarch64 xaarch64_bare xcortexa15hf

You can track the build queue at:
 http://cbuild.validation.linaro.org/helpers/scheduler

cbuild-snapshot: gcc-linaro-4.7+bzr115041~christophe-lyon~backport_bswap16-v2
cbuild-ancestor: lp:gcc-linaro+bzr115040
cbuild-state: check

Revision history for this message
Yvan Roux (yvan-roux) wrote :

obsoleted with the new proposal

review: Needs Resubmitting
Revision history for this message
Yvan Roux (yvan-roux) wrote :

Sorry, misread the proposal version !

review: Needs Information
Revision history for this message
Yvan Roux (yvan-roux) wrote :

* Diff matches upstream with the appropriate modifications for backport.
* Waiting for the ancestor testsuite results to compare

review: Needs Information
Revision history for this message
Christophe Lyon (christophe-lyon) wrote :

Indeed I replaced the test for armv6 by armv7a to avoid the errors we see when validating with a compiler configured with thumb and hard-fp as default (when a test adds the -march=armv6 option, it results in an error "Thumb-1 and VFP not implemented", thus the workaround of using -march=armv7a instead, but it changes the coverage a little bit in other configurations, since the test now generates thumb-2 code, not thumb-1).

Revision history for this message
Linaro Toolchain Builder (cbuild) wrote :

cbuild successfully built this on x86_64-precise-cbuild445-oort3-xcortexa15hfr1.

The build results are available at:
 http://cbuild.validation.linaro.org/build/gcc-linaro-4.7+bzr115041~christophe-lyon~backport_bswap16-v2/logs/x86_64-precise-cbuild445-oort3-xcortexa15hfr1

-PASS: gcc.dg/tree-ssa/forwprop-11.c scan-tree-dump-times forwprop1 "= MEM\\[\\(int \\*\\)a_.. \\+ 4B\\];" 2
+PASS: gcc.dg/tree-ssa/forwprop-11.c scan-tree-dump-times forwprop1 "= MEM\\[\\(int \\*\\)[ap]_.. \\+ 4B\\];" 2
-PASS: gcc.dg/tree-ssa/ssa-fre-1.c scan-tree-dump fre1 "Replaced \\*c_[^\n].*with t_"
+PASS: gcc.dg/tree-ssa/ssa-fre-1.c scan-tree-dump fre1 "Replaced \\*[ac]_[^\n].*with t_"
+PASS: gcc.target/arm/builtin-bswap16-1.c scan-assembler-not orr[ \t]
+PASS: gcc.target/arm/builtin-bswap16-1.c (test for excess errors)
+PASS: gcc.target/arm/builtin-bswap-1.c scan-assembler-not orr[ \t]
+PASS: gcc.target/arm/builtin-bswap-1.c scan-assembler-times rev16\\t 2
+PASS: gcc.target/arm/builtin-bswap-1.c scan-assembler-times revsh\\t 2
+PASS: gcc.target/arm/builtin-bswap-1.c scan-assembler-times rev\\t 4
+PASS: gcc.target/arm/builtin-bswap-1.c (test for excess errors)
-PASS: 30_threads/shared_future/members/wait.cc execution test
+FAIL: 30_threads/shared_future/members/wait.cc execution test
+WARNING: program timed out.
+WARNING: program timed out.
+WARNING: program timed out.

The full diff is at:
 http://cbuild.validation.linaro.org/build/gcc-linaro-4.7+bzr115041~christophe-lyon~backport_bswap16-v2/logs/x86_64-precise-cbuild445-oort3-xcortexa15hfr1/testsuite-diff.txt

The full testsuite results are at:
 http://cbuild.validation.linaro.org/build/gcc-linaro-4.7+bzr115041~christophe-lyon~backport_bswap16-v2/logs/x86_64-precise-cbuild445-oort3-xcortexa15hfr1/gcc-testsuite.txt

cbuild-checked: x86_64-precise-cbuild445-oort3-xcortexa15hfr1

Revision history for this message
Linaro Toolchain Builder (cbuild) wrote :

cbuild successfully built this on x86_64-precise-cbuild445-oort3-x86_64r1.

The build results are available at:
 http://cbuild.validation.linaro.org/build/gcc-linaro-4.7+bzr115041~christophe-lyon~backport_bswap16-v2/logs/x86_64-precise-cbuild445-oort3-x86_64r1

-PASS: gcc.dg/tree-ssa/forwprop-11.c scan-tree-dump-times forwprop1 "= MEM\\[\\(int \\*\\)a_.. \\+ 4B\\];" 2
+PASS: gcc.dg/tree-ssa/forwprop-11.c scan-tree-dump-times forwprop1 "= MEM\\[\\(int \\*\\)[ap]_.. \\+ 4B\\];" 2
-PASS: gcc.dg/tree-ssa/ssa-fre-1.c scan-tree-dump fre1 "Replaced \\*c_[^\n].*with t_"
+PASS: gcc.dg/tree-ssa/ssa-fre-1.c scan-tree-dump fre1 "Replaced \\*[ac]_[^\n].*with t_"
+PASS: gcc.target/i386/builtin-bswap-4.c scan-assembler-not bswap[ \t]
+PASS: gcc.target/i386/builtin-bswap-4.c (test for excess errors)
-PASS: g++.dg/tree-ssa/ivopts-2.C -std=gnu++11 scan-tree-dump-times ivopts "PHI <p" 1
+PASS: g++.dg/tree-ssa/ivopts-2.C -std=gnu++11 scan-tree-dump-times ivopts "PHI <[pb]" 1
-PASS: g++.dg/tree-ssa/ivopts-2.C -std=gnu++98 scan-tree-dump-times ivopts "PHI <p" 1
+PASS: g++.dg/tree-ssa/ivopts-2.C -std=gnu++98 scan-tree-dump-times ivopts "PHI <[pb]" 1

The full diff is at:
 http://cbuild.validation.linaro.org/build/gcc-linaro-4.7+bzr115041~christophe-lyon~backport_bswap16-v2/logs/x86_64-precise-cbuild445-oort3-x86_64r1/testsuite-diff.txt

The full testsuite results are at:
 http://cbuild.validation.linaro.org/build/gcc-linaro-4.7+bzr115041~christophe-lyon~backport_bswap16-v2/logs/x86_64-precise-cbuild445-oort3-x86_64r1/gcc-testsuite.txt

cbuild-checked: x86_64-precise-cbuild445-oort3-x86_64r1

Revision history for this message
Linaro Toolchain Builder (cbuild) wrote :

cbuild successfully built this on x86_64-precise-cbuild445-oort1-xaarch64r1.

The build results are available at:
 http://cbuild.validation.linaro.org/build/gcc-linaro-4.7+bzr115041~christophe-lyon~backport_bswap16-v2/logs/x86_64-precise-cbuild445-oort1-xaarch64r1

-PASS: gcc.dg/tree-ssa/forwprop-11.c scan-tree-dump-times forwprop1 "= MEM\\[\\(int \\*\\)a_.. \\+ 4B\\];" 2
+PASS: gcc.dg/tree-ssa/forwprop-11.c scan-tree-dump-times forwprop1 "= MEM\\[\\(int \\*\\)[ap]_.. \\+ 4B\\];" 2
-PASS: gcc.dg/tree-ssa/ssa-fre-1.c scan-tree-dump fre1 "Replaced \\*c_[^\n].*with t_"
+PASS: gcc.dg/tree-ssa/ssa-fre-1.c scan-tree-dump fre1 "Replaced \\*[ac]_[^\n].*with t_"
-FAIL: gcc.target/aarch64/builtin-bswap-1.c scan-assembler-times rev16\\t 2
+PASS: gcc.target/aarch64/builtin-bswap-1.c scan-assembler-times rev16\\t 2
-FAIL: gcc.target/aarch64/builtin-bswap-2.c scan-assembler-times rev16\\t 2
+PASS: gcc.target/aarch64/builtin-bswap-2.c scan-assembler-times rev16\\t 2

The full diff is at:
 http://cbuild.validation.linaro.org/build/gcc-linaro-4.7+bzr115041~christophe-lyon~backport_bswap16-v2/logs/x86_64-precise-cbuild445-oort1-xaarch64r1/testsuite-diff.txt

The full testsuite results are at:
 http://cbuild.validation.linaro.org/build/gcc-linaro-4.7+bzr115041~christophe-lyon~backport_bswap16-v2/logs/x86_64-precise-cbuild445-oort1-xaarch64r1/gcc-testsuite.txt

cbuild-checked: x86_64-precise-cbuild445-oort1-xaarch64r1

Revision history for this message
Linaro Toolchain Builder (cbuild) wrote :

cbuild successfully built this on x86_64-precise-cbuild445-oort1-xaarch64_barer1.

The build results are available at:
 http://cbuild.validation.linaro.org/build/gcc-linaro-4.7+bzr115041~christophe-lyon~backport_bswap16-v2/logs/x86_64-precise-cbuild445-oort1-xaarch64_barer1

-PASS: gcc.dg/tree-ssa/forwprop-11.c scan-tree-dump-times forwprop1 "= MEM\\[\\(int \\*\\)a_.. \\+ 4B\\];" 2
+PASS: gcc.dg/tree-ssa/forwprop-11.c scan-tree-dump-times forwprop1 "= MEM\\[\\(int \\*\\)[ap]_.. \\+ 4B\\];" 2
-PASS: gcc.dg/tree-ssa/ssa-fre-1.c scan-tree-dump fre1 "Replaced \\*c_[^\n].*with t_"
+PASS: gcc.dg/tree-ssa/ssa-fre-1.c scan-tree-dump fre1 "Replaced \\*[ac]_[^\n].*with t_"
-FAIL: gcc.target/aarch64/builtin-bswap-1.c scan-assembler-times rev16\\t 2
+PASS: gcc.target/aarch64/builtin-bswap-1.c scan-assembler-times rev16\\t 2
-FAIL: gcc.target/aarch64/builtin-bswap-2.c scan-assembler-times rev16\\t 2
+PASS: gcc.target/aarch64/builtin-bswap-2.c scan-assembler-times rev16\\t 2

The full diff is at:
 http://cbuild.validation.linaro.org/build/gcc-linaro-4.7+bzr115041~christophe-lyon~backport_bswap16-v2/logs/x86_64-precise-cbuild445-oort1-xaarch64_barer1/testsuite-diff.txt

The full testsuite results are at:
 http://cbuild.validation.linaro.org/build/gcc-linaro-4.7+bzr115041~christophe-lyon~backport_bswap16-v2/logs/x86_64-precise-cbuild445-oort1-xaarch64_barer1/gcc-testsuite.txt

cbuild-checked: x86_64-precise-cbuild445-oort1-xaarch64_barer1

Revision history for this message
Linaro Toolchain Builder (cbuild) wrote :

cbuild successfully built this on i686-precise-cbuild445-oort2-i686r1.

The build results are available at:
 http://cbuild.validation.linaro.org/build/gcc-linaro-4.7+bzr115041~christophe-lyon~backport_bswap16-v2/logs/i686-precise-cbuild445-oort2-i686r1

-PASS: gcc.dg/simulate-thread/atomic-load-int.c -O2 -g thread simulation test
+UNSUPPORTED: gcc.dg/simulate-thread/atomic-load-int.c -O2 -g thread simulation test
-PASS: gcc.dg/simulate-thread/atomic-other-int.c -O0 -g thread simulation test
+UNSUPPORTED: gcc.dg/simulate-thread/atomic-other-int.c -O0 -g thread simulation test
-PASS: gcc.dg/tree-ssa/forwprop-11.c scan-tree-dump-times forwprop1 "= MEM\\[\\(int \\*\\)a_.. \\+ 4B\\];" 2
+PASS: gcc.dg/tree-ssa/forwprop-11.c scan-tree-dump-times forwprop1 "= MEM\\[\\(int \\*\\)[ap]_.. \\+ 4B\\];" 2
-PASS: gcc.dg/tree-ssa/ssa-fre-1.c scan-tree-dump fre1 "Replaced \\*c_[^\n].*with t_"
+PASS: gcc.dg/tree-ssa/ssa-fre-1.c scan-tree-dump fre1 "Replaced \\*[ac]_[^\n].*with t_"
+PASS: gcc.target/i386/builtin-bswap-4.c scan-assembler-not bswap[ \t]
+PASS: gcc.target/i386/builtin-bswap-4.c (test for excess errors)
-UNRESOLVED: g++.dg/lto/20090112 cp_lto_20090112_0.o-cp_lto_20090112_0.o execute -O0 -flto -flto-partition=none -fuse-linker-plugin
+PASS: g++.dg/lto/20090112 cp_lto_20090112_0.o-cp_lto_20090112_0.o execute -O0 -flto -flto-partition=none -fuse-linker-plugin
-FAIL: g++.dg/lto/20090112 cp_lto_20090112_0.o-cp_lto_20090112_0.o link, -O0 -flto -flto-partition=none -fuse-linker-plugin
+PASS: g++.dg/lto/20090112 cp_lto_20090112_0.o-cp_lto_20090112_0.o link, -O0 -flto -flto-partition=none -fuse-linker-plugin
-PASS: g++.dg/tree-ssa/ivopts-2.C -std=gnu++11 scan-tree-dump-times ivopts "PHI <p" 1
+PASS: g++.dg/tree-ssa/ivopts-2.C -std=gnu++11 scan-tree-dump-times ivopts "PHI <[pb]" 1
-PASS: g++.dg/tree-ssa/ivopts-2.C -std=gnu++98 scan-tree-dump-times ivopts "PHI <p" 1
+PASS: g++.dg/tree-ssa/ivopts-2.C -std=gnu++98 scan-tree-dump-times ivopts "PHI <[pb]" 1

The full diff is at:
 http://cbuild.validation.linaro.org/build/gcc-linaro-4.7+bzr115041~christophe-lyon~backport_bswap16-v2/logs/i686-precise-cbuild445-oort2-i686r1/testsuite-diff.txt

The full testsuite results are at:
 http://cbuild.validation.linaro.org/build/gcc-linaro-4.7+bzr115041~christophe-lyon~backport_bswap16-v2/logs/i686-precise-cbuild445-oort2-i686r1/gcc-testsuite.txt

cbuild-checked: i686-precise-cbuild445-oort2-i686r1

Revision history for this message
Yvan Roux (yvan-roux) wrote :

* Diff matches upstream with the appropriate modifications for backport.
* One failure on cortexa15 in libstdc++ : related to Qemu threading support
* Other targets are clean

review: Approve

Subscribers

People subscribed via source and target branches