traverso version 0.49.2-4 failed to build on armel

[Ricardo Salveti]

traverso version 0.49.2-4 failed to build on armel
Link to failed build: https://launchpad.net/ubuntu/+source/traverso/0.49.2-4/+build/2657149

Direct link to the build log: https://launchpad.net/ubuntu/+source/traverso/0.49.2-4/+build/2657149/+files/buildlog_ubuntu-oneiric-armel.traverso_0.49.2-4_FAILEDTOBUILD.txt.gz

This log snippet might be of interest, since it triggered the matcher 'Purging chroot-autobuild'.
Excerpt 1374 lines into the build log:

cd /build/buildd/traverso-0.49.2/obj-arm-linux-gnueabi/src/engine && /usr/bin/c++ -DQT_GUI_LIB -DQT_CORE_LIB -DSTATIC_BUILD -DQT_NO_DEBUG -DLV2_SUPPORT -DMP3_DECODE_SUPPORT -DMP3_ENCODE_SUPPORT -DHAVE_SYS_VFS_H -DHAVE_SYS_STAT_H -DUSE_MLOCK -DALSA_SUPPORT -DJACK_SUPPORT -DQT_NO_DEBUG -g -O2 -O2 -Wall -fPIC -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -I/build/buildd/traverso-0.49.2/obj-arm-linux-gnueabi/src/engine -I/build/buildd/traverso-0.49.2/src/engine -I/usr/include/qt4 -I/usr/include/qt4/QtGui -I/usr/include/qt4/QtCore -I/usr/include/redland -I/usr/include/rasqal -I/build/buildd/traverso-0.49.2/src/common -I/usr/include/raptor2 -o CMakeFiles/traversoaudiobackend.dir/memops.o -c /build/buildd/traverso-0.49.2/src/engine/memops.cpp
/build/buildd/traverso-0.49.2/src/engine/memops.cpp: In function 'void sample_move_d32u24_sS(char*, audio_sample_t*, long unsigned int, long unsigned int, dither_state_t*)':
/build/buildd/traverso-0.49.2/src/engine/memops.cpp:98:1: internal compiler error: in get_arm_condition_code, at config/arm/arm.c:17488
Please submit a full bug report,
with preprocessed source if appropriate.
See <file:///usr/share/doc/gcc-4.6/README.Bugs> for instructions.
make[3]: *** [src/engine/CMakeFiles/traversoaudiobackend.dir/memops.o] Error 1
make[3]: Leaving directory `/build/buildd/traverso-0.49.2/obj-arm-linux-gnueabi'
make[2]: *** [src/engine/CMakeFiles/traversoaudiobackend.dir/all] Error 2
make[2]: Leaving directory `/build/buildd/traverso-0.49.2/obj-arm-linux-gnueabi'
make[1]: *** [all] Error 2
make[1]: Leaving directory `/build/buildd/traverso-0.49.2/obj-arm-linux-gnueabi'
dh_auto_build: make -j1 returned exit code 2
make: *** [build] Error 2
dpkg-buildpackage: error: debian/rules build gave error exit status 2
******************************************************************************
Build finished at 20110730-1041
FAILED [dpkg-buildpackage died]
Purging chroot-autobuild/build/buildd/traverso-0.49.2

[Rmansfield-t (rmansfield-t)]

Created attachment 24268
preprocessed source

$ ./xgcc -v
Using built-in specs.
COLLECT_GCC=./xgcc
Target: arm-unknown-linux-gnueabi
Configured with: ../configure --target=arm-unknown-linux-gnueabi --prefix=/home/ryan/x-tools/arm-unknown-linux-gnueabi --with-sysroot=/home/ryan/x-tools/arm-unknown-linux-gnueabi/arm-unknown-linux-gnueabi//sys-root --disable-multilib --with-local-prefix=/home/ryan/x-tools/arm-unknown-linux-gnueabi/arm-unknown-linux-gnueabi/sys-root --disable-nls --enable-threads=posix --enable-symvers=gnu --enable-c99 --enable-long-long --enable-target-optspace target_alias=arm-unknown-linux-gnueabi --enable-languages=c++ --disable-shared --disable-libmudflap --disable-libssp
Thread model: posix
gcc version 4.7.0 20110517 (experimental) [trunk revision 173832] (GCC)
$ ./xgcc -B. -O1 ~/ice.i
/home/ryan/ice.i: In function 'bar':
/home/ryan/ice.i:33:1: internal compiler error: in get_arm_condition_code, at config/arm/arm.c:17180
Please submit a full bug report,
with preprocessed source if appropriate.
See <http://gcc.gnu.org/bugs.html> for instructions.

[Ibolton (ibolton)]

I haven't been able to reproduce this. Please can you include the args that are being sent to cc1.

[Rmansfield-t (rmansfield-t)]

With rev173940

$ ./cc1 -fpreprocessed ice.i -quiet -O1
ice.i: In function 'bar':
ice.i:33:1: internal compiler error: in get_arm_condition_code, at config/arm/arm.c:17021
Please submit a full bug report,
with preprocessed source if appropriate.
See <http://gcc.gnu.org/bugs.html> for instructions.

[Ibolton (ibolton)]

I was hoping the compiler command would tell me what -mcpu it was defaulting to.

Anyway, I have managed to reproduce it now, by compiling as follows:

  cc1 -fpreprocessed 49030.i -quiet -O1 -o 49030.s -mcpu=arm7tdmi -marm

Note that it all works with -mthumb and/or -mcpu=cortex-a9.

I have tried this with my old toolchain and it is broken as far back as r164700.

[Rmansfield-t (rmansfield-t)]

(In reply to comment #3)
> I was hoping the compiler command would tell me what -mcpu it was defaulting
> to.

I'm using an arm-unknown-linux-gnueabi configuration which defaults an arm10tdmi.

linux-eabi.h:#define SUBTARGET_CPU_DEFAULT TARGET_CPU_arm10tdmi

[Michael Hope (michaelh1)]

The same assert is seen in https://bugs.launchpad.net/ubuntu/+source/gcc-4.5/+bug/689887 in Thumb-2 mode on trunk r174795.

Chung-Lin posted a patch at http://gcc.gnu.org/ml/gcc-patches/2011-03/msg00790.html but there are bootstrap issues.

Adding -fno-if-conversion works around the problem.

[Mikpe (mikpe)]

It's caused by r161764:

Author: sandra
Date: Sat Jul 3 01:00:37 2010
New Revision: 161764

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=161764
Log:
2010-07-02 Daniel Jacobowitz <email address hidden>
     Julian Brown <email address hidden>
     Sandra Loosemore <email address hidden>

 gcc/
 * config/arm/arm.c (arm_canonicalize_comparison): Canonicalize DImode
 comparisons. Adjust to take both operands.
 (arm_select_cc_mode): Handle DImode comparisons.
 (arm_gen_compare_reg): Generate a scratch register for DImode
 comparisons which require one. Use xor for Thumb equality checks.
 (arm_const_double_by_immediates): New.
 (arm_print_operand): Allow 'Q' and 'R' for constants.
 (get_arm_condition_code): Handle new CC_CZmode and CC_NCVmode.
        ...

[Mikpe (mikpe)]

Created attachment 24503
more reduced test case, ICEs at -O1 for armv5te and armv6

Using this slightly more reduced test case I've found that gcc transforms

(set (reg:CC_NCV ...) (compare:CC_NCV (reg:DI ...) (const_int 1 ...)))
(if_then_else (ge (reg:CC_NCV ...) ...) ...)

to

(set (reg:CC_NCV ...) (compare:CC_NCV (reg:DI ...) (const_int 0 ...)))
(if_then_else (gt (reg:CC_NCV ...) ...)

because ifcvt calls noce_get_condition which calls canonicalize_condition, and the latter has a general rule to transform (GE x const) to (GT x const-1).

So the careful code in e.g. cbranchdi4 to not generate unimplemented comparisons is neutralized by generic code.

Do DImode comparisons really have to be asymmetric? I.e., could CC_NCV be removed? I see that LT is apparently supported so why can't GT be too?

[Mikpe (mikpe)]

I have a preliminary patch which creates a clone of *arm_cmpdi_insn that does a reverse comparison, via a new CC mode for must-be-reversed comparisons, and corresponding updates to arm_select_cc_mode and get_arm_condition_code. It's the same transformation from eg (GT x y) to (LT y x) that cstoredi4 does, only later so it can handle problematic comparisons introduced by the middle-end.

This fixes the ICEs on the two test cases in this PR. I'll do a full bootstrap and regtest as soon as my test machine is done with its current bootstrap and regtest cycle.

I'll send the FSF Yet Another Ping about my employer's copyright disclaimer first thing tomorrow. They've had the papers since January or February...

[Ricardo Salveti (rsalveti)]

Bug doesn't happen at Debian: https://buildd.debian.org/status/fetch.php?pkg=traverso&arch=armel&ver=0.49.2-4&stamp=1312062131

[Marcin Juszkiewicz (hrw)]

14:55 hrw@malenstwo:engine$ gcc -v
Using built-in specs.
COLLECT_GCC=/usr/bin/gcc-4.6.real
COLLECT_LTO_WRAPPER=/usr/lib/arm-linux-gnueabi/gcc/arm-linux-gnueabi/4.6.1/lto-wrapper
Target: arm-linux-gnueabi
Configured with: ../src/configure -v --with-pkgversion='Ubuntu/Linaro 4.6.1-5ubuntu2' --with-bugurl=file:///usr/share/doc/gcc-4.6/README.Bugs --enable-languages=c,c++,fortran,objc,obj-c++ --prefix=/usr --program-suffix=-4.6 --enable-shared --enable-multiarch --with-multiarch-defaults=arm-linux-gnueabi --enable-linker-build-id --with-system-zlib --libexecdir=/usr/lib/arm-linux-gnueabi --without-included-gettext --enable-threads=posix --with-gxx-include-dir=/usr/include/c++/4.6 --libdir=/usr/lib/arm-linux-gnueabi --enable-nls --with-sysroot=/ --enable-clocale=gnu --enable-libstdcxx-debug --enable-libstdcxx-time=yes --enable-plugin --enable-objc-gc --enable-multilib --disable-sjlj-exceptions --with-arch=armv7-a --with-float=softfp --with-fpu=vfpv3-d16 --with-mode=thumb --disable-werror --enable-checking=release --build=arm-linux-gnueabi --host=arm-linux-gnueabi --target=arm-linux-gnueabi
Thread model: posix
gcc version 4.6.1 (Ubuntu/Linaro 4.6.1-5ubuntu2)

[Marcin Juszkiewicz (hrw)]

14:57 hrw@malenstwo:engine$ g++-4.5 -DQT_GUI_LIB -DQT_CORE_LIB -DSTATIC_BUILD -DQT_NO_DEBUG -DLV2_SUPPORT -DMP3_DECODE_SUPPORT -DMP3_ENCODE_SUPPORT -DHAVE_SYS_VFS_H -DHAVE_SYS_STAT_H -DUSE_MLOCK -DALSA_SUPPORT -DJACK_SUPPORT -DQT_NO_DEBUG -g -O2 -O2 -Wall -fPIC -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -I/home/hrw/devel/porting-jam/traverso-0.49.2/obj-arm-linux-gnueabi/src/engine -I/home/hrw/devel/porting-jam/traverso-0.49.2/src/engine -I/usr/include/qt4 -I/usr/include/qt4/QtGui -I/usr/include/qt4/QtCore -I/usr/include/redland -I/usr/include/rasqal -I/home/hrw/devel/porting-jam/traverso-0.49.2/src/common -I/usr/include/raptor2 -o CMakeFiles/traversoaudiobackend.dir/memops.o -c /home/hrw/devel/porting-jam/traverso-0.49.2/src/engine/memops.cpp
/home/hrw/devel/porting-jam/traverso-0.49.2/src/engine/memops.cpp: In function ‘void sample_move_d32u24_sS(char*, audio_sample_t*, long unsigned int, long unsigned int, dither_state_t*)’:
/home/hrw/devel/porting-jam/traverso-0.49.2/src/engine/memops.cpp:98:1: internal compiler error: in get_arm_condition_code, at config/arm/arm.c:17326
Please submit a full bug report,
with preprocessed source if appropriate.
See <file:///usr/share/doc/gcc-4.5/README.Bugs> for instructions.

14:58 hrw@malenstwo:engine$ g++-4.5 -v
Using built-in specs.
COLLECT_GCC=/usr/bin/g++-4.5.real
COLLECT_LTO_WRAPPER=/usr/lib/arm-linux-gnueabi/gcc/arm-linux-gnueabi/4.5.3/lto-wrapper
Target: arm-linux-gnueabi
Configured with: ../src/configure -v --with-pkgversion='Ubuntu/Linaro 4.5.3-5ubuntu1' --with-bugurl=file:///usr/share/doc/gcc-4.5/README.Bugs --enable-languages=c,c++,fortran,objc,obj-c++ --prefix=/usr --program-suffix=-4.5 --enable-shared --enable-multiarch --with-multiarch-defaults=arm-linux-gnueabi --enable-linker-build-id --with-system-zlib --libexecdir=/usr/lib/arm-linux-gnueabi --without-included-gettext --enable-threads=posix --with-gxx-include-dir=/usr/include/c++/4.5 --libdir=/usr/lib/arm-linux-gnueabi --enable-nls --with-sysroot=/ --enable-clocale=gnu --enable-libstdcxx-debug --enable-libstdcxx-time=yes --enable-plugin --enable-gold --enable-ld=default --with-plugin-ld=ld.gold --enable-objc-gc --disable-sjlj-exceptions --with-arch=armv7-a --with-float=softfp --with-fpu=vfpv3-d16 --with-mode=thumb --disable-werror --enable-checking=release --build=arm-linux-gnueabi --host=arm-linux-gnueabi --target=arm-linux-gnueabi
Thread model: posix
gcc version 4.5.3 (Ubuntu/Linaro 4.5.3-5ubuntu1)

[Marcin Juszkiewicz (hrw)]

15:16 hrw@malenstwo:engine$ g++ -DQT_GUI_LIB -DQT_CORE_LIB -DSTATIC_BUILD -DQT_NO_DEBUG -DLV2_SUPPORT -DMP3_DECODE_SUPPORT -DMP3_ENCODE_SUPPORT -DHAVE_SYS_VFS_H -DHAVE_SYS_STAT_H -DUSE_MLOCK -DALSA_SUPPORT -DJACK_SUPPORT -DQT_NO_DEBUG -g -O2 -O2 -Wall -fPIC -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -I/home/hrw/devel/porting-jam/traverso-0.49.2/obj-arm-linux-gnueabi/src/engine -I/home/hrw/devel/porting-jam/traverso-0.49.2/src/engine -I/usr/include/qt4 -I/usr/include/qt4/QtGui -I/usr/include/qt4/QtCore -I/usr/include/redland -I/usr/include/rasqal -I/home/hrw/devel/porting-jam/traverso-0.49.2/src/common -I/usr/include/raptor2 -o CMakeFiles/traversoaudiobackend.dir/memops.o -c /home/hrw/devel/porting-jam/traverso-0.49.2/src/engine/memops.cpp -save-temps
/home/hrw/devel/porting-jam/traverso-0.49.2/src/engine/memops.cpp: In function 'void sample_move_d32u24_sS(char*, audio_sample_t*, long unsigned int, long unsigned int, dither_state_t*)':
/home/hrw/devel/porting-jam/traverso-0.49.2/src/engine/memops.cpp:98:1: internal compiler error: in get_arm_condition_code, at config/arm/arm.c:17605
Please submit a full bug report,
with preprocessed source if appropriate.
See <file:///usr/share/doc/gcc-snapshot/README.Bugs> for instructions.
15:16 hrw@malenstwo:engine$ g++ -v
Using built-in specs.
COLLECT_GCC=g++
COLLECT_LTO_WRAPPER=/usr/lib/gcc-snapshot/libexec/gcc/arm-linux-gnueabi/4.7.0/lto-wrapper
Target: arm-linux-gnueabi
Configured with: ../src/configure -v --with-pkgversion='Ubuntu/Linaro 20110813-1ubuntu1' --with-bugurl=file:///usr/share/doc/gcc-snapshot/README.Bugs --enable-languages=c,ada,c++,java,fortran,objc,obj-c++ --prefix=/usr/lib/gcc-snapshot --enable-shared --enable-multiarch --with-multiarch-defaults=arm-linux-gnueabi --enable-linker-build-id --with-system-zlib --disable-nls --with-sysroot=/ --enable-clocale=gnu --enable-libstdcxx-debug --enable-libstdcxx-time=yes --enable-plugin --disable-browser-plugin --enable-java-awt=gtk --enable-gtk-cairo --with-java-home=/usr/lib/jvm/java-1.5.0-gcj-4.7-snap/jre --enable-java-home --with-jvm-root-dir=/usr/lib/jvm/java-1.5.0-gcj-4.7-snap --with-jvm-jar-dir=/usr/lib/jvm-exports/java-1.5.0-gcj-4.7-snap --with-arch-directory=arm --with-ecj-jar=/usr/share/java/eclipse-ecj.jar --enable-objc-gc --enable-multilib --disable-sjlj-exceptions --with-arch=armv7-a --with-float=softfp --with-fpu=vfpv3-d16 --with-mode=thumb --disable-werror --enable-checking=yes --build=arm-linux-gnueabi --host=arm-linux-gnueabi --target=arm-linux-gnueabi
Thread model: posix
gcc version 4.7.0 20110813 (experimental) [trunk revision 177733] (Ubuntu/Linaro 20110813-1ubuntu1)

[Michael Hope (michaelh1)]

Thank you for the bug report. I've confirmed this with gcc-linaro-4.6-2011.08 on ARM:

michaelh@ursa1:~/linaro/bugs$ /tools/toolchains/arch/armv7l/gcc-linaro-4.6-2011.08-armv7l-natty-cbuild162-ursa1-cortexa9r1/bin/g++ -O2 -S memops.ii
/home/hrw/devel/porting-jam/traverso-0.49.2/src/engine/memops.cpp: In function 'void sample_move_d32u24_sS(char*, audio_sample_t*, long unsigned int, long unsigned int, dither_state_t*)':
/home/hrw/devel/porting-jam/traverso-0.49.2/src/engine/memops.cpp:98:1: internal compiler error: in get_arm_condition_code, at config/arm/arm.c:17488

The fault occurs at -O1, -O2, and -O3. The work-around is to compile with -O0 or -marm. The fault also exists in gcc-4.6.1 and trunk r178025. It does not exist in gcc-4.5.3.

I've set it to medium priority as it is a ftbfs, a work-around exists, and the fault exists upstream.

This is probably the same fault as LP: #809761.

[Michael Hope (michaelh1)]

Related to https://bugs.launchpad.net/ubuntu/+source/oss4/+bug/809761

[Rsandifo-gcc (rsandifo-gcc)]

Author: rsandifo
Date: Wed Sep 7 13:48:03 2011
New Revision: 178636

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=178636
Log:
gcc/
 PR target/49030
 * config/arm/arm-protos.h (maybe_get_arm_condition_code): Declare.
 * config/arm/arm.c (maybe_get_arm_condition_code): New function,
 reusing the old code from get_arm_condition_code. Return ARM_NV
 for invalid comparison codes.
 (get_arm_condition_code): Redefine in terms of
 maybe_get_arm_condition_code.
 * config/arm/predicates.md (arm_comparison_operator): Use
 maybe_get_arm_condition_code.

gcc/testsuite/
 PR target/49030
 * gcc.dg/torture/pr49030.c: New test.

Added:
    trunk/gcc/testsuite/gcc.dg/torture/pr49030.c
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/config/arm/arm-protos.h
    trunk/gcc/config/arm/arm.c
    trunk/gcc/config/arm/predicates.md
    trunk/gcc/testsuite/ChangeLog

[Rmansfield-t (rmansfield-t)]

I am out of the office until September 19th. For emergencies please contact Fred Plante.

[Gjl (gjl)]

Author: gjl
Date: Fri Sep 9 17:00:26 2011
New Revision: 178736

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=178736
Log:
 PR target/49030
 * gcc.dg/torture/pr49030.c: Run only if target int32plus.

Modified:
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/testsuite/gcc.dg/torture/pr49030.c