Code review comment for lp:~christophe-lyon/gcc-linaro/backport_bswap16-v2

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