Merge lp:~ramana/gcc-linaro/fix-lp-1013209 into lp:gcc-linaro/4.6

Proposed by Ramana Radhakrishnan
Status: Merged
Approved by: Ulrich Weigand
Approved revision: no longer in the source branch.
Merge reported by: Ramana Radhakrishnan
Merged at revision: not available
Proposed branch: lp:~ramana/gcc-linaro/fix-lp-1013209
Merge into: lp:gcc-linaro/4.6
To merge this branch: bzr merge lp:~ramana/gcc-linaro/fix-lp-1013209
Reviewer Review Type Date Requested Status
Ulrich Weigand (community) Approve
Review via email: mp+111453@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Michael Hope (michaelh1) wrote :

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

The diff against the ancestor r106898 is available at:
 http://builds.linaro.org/toolchain/snapshots/gcc-linaro-4.6+bzr106899~ramana~fix-lp-1013209.diff

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

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

cbuild-snapshot: gcc-linaro-4.6+bzr106899~ramana~fix-lp-1013209
cbuild-ancestor: lp:gcc-linaro/4.6+bzr106898
cbuild-state: check

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

cbuild successfully built this on armv7l-precise-cbuild330-ursa4-cortexa9hfr1.

The build results are available at:
 http://ex.seabright.co.nz/build/gcc-linaro-4.6+bzr106899~ramana~fix-lp-1013209/logs/armv7l-precise-cbuild330-ursa4-cortexa9hfr1

+PASS: gcc.target/arm/lp1013209.c (test for excess errors)
-PASS: libmudflap.c/fail68-frag.c (-static) (test for excess errors)
-PASS: libmudflap.c/fail68-frag.c (-static) crash test
-PASS: libmudflap.c/fail68-frag.c (-static) output pattern test
+FAIL: libmudflap.c/fail68-frag.c (-static) (test for excess errors)
+WARNING: libmudflap.c/fail68-frag.c (-static) compilation failed to produce executable
+FAIL: libmudflap.c/pass13-frag.c (-static) (test for excess errors)
-PASS: libmudflap.c/pass13-frag.c (-static) (test for excess errors)
-PASS: libmudflap.c/pass13-frag.c (-static) execution test
+WARNING: libmudflap.c/pass13-frag.c (-static) compilation failed to produce executable

The full diff is at:
 http://ex.seabright.co.nz/build/gcc-linaro-4.6+bzr106899~ramana~fix-lp-1013209/logs/armv7l-precise-cbuild330-ursa4-cortexa9hfr1/testsuite-diff.txt

The full testsuite results are at:
 http://ex.seabright.co.nz/build/gcc-linaro-4.6+bzr106899~ramana~fix-lp-1013209/logs/armv7l-precise-cbuild330-ursa4-cortexa9hfr1/gcc-testsuite.txt

cbuild-checked: armv7l-precise-cbuild330-ursa4-cortexa9hfr1

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

cbuild successfully built this on armv7l-natty-cbuild331-tcpanda03-armv5r2.

The build results are available at:
 http://ex.seabright.co.nz/build/gcc-linaro-4.6+bzr106899~ramana~fix-lp-1013209/logs/armv7l-natty-cbuild331-tcpanda03-armv5r2

+PASS: gcc.target/arm/lp1013209.c (test for excess errors)

The full diff is at:
 http://ex.seabright.co.nz/build/gcc-linaro-4.6+bzr106899~ramana~fix-lp-1013209/logs/armv7l-natty-cbuild331-tcpanda03-armv5r2/testsuite-diff.txt

The full testsuite results are at:
 http://ex.seabright.co.nz/build/gcc-linaro-4.6+bzr106899~ramana~fix-lp-1013209/logs/armv7l-natty-cbuild331-tcpanda03-armv5r2/gcc-testsuite.txt

cbuild-checked: armv7l-natty-cbuild331-tcpanda03-armv5r2

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

cbuild successfully built this on armv7l-natty-cbuild330-tcpanda03-cortexa9r1.

The build results are available at:
 http://ex.seabright.co.nz/build/gcc-linaro-4.6+bzr106899~ramana~fix-lp-1013209/logs/armv7l-natty-cbuild330-tcpanda03-cortexa9r1

+PASS: gcc.target/arm/lp1013209.c (test for excess errors)

The full diff is at:
 http://ex.seabright.co.nz/build/gcc-linaro-4.6+bzr106899~ramana~fix-lp-1013209/logs/armv7l-natty-cbuild330-tcpanda03-cortexa9r1/testsuite-diff.txt

The full testsuite results are at:
 http://ex.seabright.co.nz/build/gcc-linaro-4.6+bzr106899~ramana~fix-lp-1013209/logs/armv7l-natty-cbuild330-tcpanda03-cortexa9r1/gcc-testsuite.txt

cbuild-checked: armv7l-natty-cbuild330-tcpanda03-cortexa9r1

Revision history for this message
Ramana Radhakrishnan (ramana) wrote :
Download full text (4.5 KiB)

> cbuild successfully built this on armv7l-precise-cbuild330-ursa4-cortexa9hfr1.
>
> The build results are available at:
> http://ex.seabright.co.nz/build/gcc-linaro-4.6+bzr106899~ramana~fix-
> lp-1013209/logs/armv7l-precise-cbuild330-ursa4-cortexa9hfr1
>
> +PASS: gcc.target/arm/lp1013209.c (test for excess errors)
> -PASS: libmudflap.c/fail68-frag.c (-static) (test for excess errors)
> -PASS: libmudflap.c/fail68-frag.c (-static) crash test
> -PASS: libmudflap.c/fail68-frag.c (-static) output pattern test
> +FAIL: libmudflap.c/fail68-frag.c (-static) (test for excess errors)
> +WARNING: libmudflap.c/fail68-frag.c (-static) compilation failed to produce
> executable
> +FAIL: libmudflap.c/pass13-frag.c (-static) (test for excess errors)
> -PASS: libmudflap.c/pass13-frag.c (-static) (test for excess errors)
> -PASS: libmudflap.c/pass13-frag.c (-static) execution test
> +WARNING: libmudflap.c/pass13-frag.c (-static) compilation failed to produce
> executable

This does not make sense but looking at the error message it appears to be : I suspect binutils but have to verify this carefully.

Executing on host: /cbuild/slaves/ursa4/gcc-linaro-4.6+bzr106899~ramana~fix-lp-1013209/gcc/default/build/gcc/xgcc -B/cbuild/slaves/ursa4/gcc-linaro-4.6+bzr106899~ramana~fix-lp-1013209/gcc/default/build/gcc/ -ggdb3 -DDEBUG_ASSERT -I/cbuild/slaves/ursa4/gcc-linaro-4.6+bzr106899~ramana~fix-lp-1013209/gcc/gcc-linaro-4.6/libmudflap/testsuite -I/cbuild/slaves/ursa4/gcc-linaro-4.6+bzr106899~ramana~fix-lp-1013209/gcc/gcc-linaro-4.6/libmudflap/testsuite/.. -I.. -L/cbuild/slaves/ursa4/gcc-linaro-4.6+bzr106899~ramana~fix-lp-1013209/gcc/default/build/arm-linux-gnueabi/./libmudflap/.libs /cbuild/slaves/ursa4/gcc-linaro-4.6+bzr106899~ramana~fix-lp-1013209/gcc/gcc-linaro-4.6/libmudflap/testsuite/libmudflap.c/pass13-frag.c -static -fmudflap -lmudflap -L/cbuild/slaves/ursa4/gcc-linaro-4.6+bzr106899~ramana~fix-lp-1013209/gcc/default/build/arm-linux-gnueabi/./libmudflap/testsuite -ldl -lm -o ./pass13-frag.exe (timeout = 300)
spawn /cbuild/slaves/ursa4/gcc-linaro-4.6+bzr106899~ramana~fix-lp-1013209/gcc/default/build/gcc/xgcc -B/cbuild/slaves/ursa4/gcc-linaro-4.6+bzr106899~ramana~fix-lp-1013209/gcc/default/build/gcc/ -ggdb3 -DDEBUG_ASSERT -I/cbuild/slaves/ursa4/gcc-linaro-4.6+bzr106899~ramana~fix-lp-1013209/gcc/gcc-linaro-4.6/libmudflap/testsuite -I/cbuild/slaves/ursa4/gcc-linaro-4.6+bzr106899~ramana~fix-lp-1013209/gcc/gcc-linaro-4.6/libmudflap/testsuite/.. -I.. -L/cbuild/slaves/ursa4/gcc-linaro-4.6+bzr106899~ramana~fix-lp-1013209/gcc/default/build/arm-linux-gnueabi/./libmudflap/.libs /cbuild/slaves/ursa4/gcc-linaro-4.6+bzr106899~ramana~fix-lp-1013209/gcc/gcc-linaro-4.6/libmudflap/testsuite/libmudflap.c/pass13-frag.c -static -fmudflap -lmudflap -L/cbuild/slaves/ursa4/gcc-linaro-4.6+bzr106899~ramana~fix-lp-1013209/gcc/default/build/arm-linux-gnueabi/./libmudflap/testsuite -ldl -lm -o ./pass13-frag.exe
/usr/bin/ld: error: /cbuild/slaves/ursa4/gcc-linaro-4.6+bzr106899~ramana~fix-lp-1013209/gcc/default/build/arm-linux-gnueabi/./libmudflap/.libs/libmudflap.a(mf-runtime.o) contains a reloc (0x00090d34) for section .debug_info that references a non-existent global symbol
/usr/b...

Read more...

Revision history for this message
Ulrich Weigand (uweigand) wrote :

   /* Create a type that describes the full access. */
   upper_bound = build_int_cst (size_type_node, nelems - 1);
+
   array_type = build_array_type (elem_type, build_index_type (upper_bound));
+ /* Dereference EXP using that type. */
+ exp = fold_convert (build_pointer_type (array_type), exp);

It seems the empty line is added in the wrong place?

+/* Test the `vadd_s64' ARM Neon intrinsic. */

That comment seems incorrect.

Otherwise, this patch looks good to me.

review: Approve

Subscribers

People subscribed via source and target branches