[4.8/4.9 Regression] Error: value of 256 too large for field of 1 bytes at 68242

Created attachment 32416
preprocessed source

originall reported as a gas issue:
https://sourceware.org/bugzilla/show_bug.cgi?id=16735

seen when building the ppl 1.1 bindings for SWI Prolog, using binutils from the 2.24 branch on arm-linux-gnueabihf with a compiler defaulting to thumb mode.

seen with current 4.8 branch and at least trunk 20140306.

$ g++ -c -g -O2 ppl_prolog_common.ii
/tmp/ccuRxr3p.s: Assembler messages:
/tmp/ccuRxr3p.s:126688: Error: value of 256 too large for field of 1 bytes at 68242

$ g++ -marm -c -g -O2 ppl_prolog_common.ii
<succeeds>

$ as -v -march=armv7-a -mfloat-abi=hard -mfpu=vfpv3-d16 -meabi=5 -o ppl_prolog_common.o ppl_prolog_common.s
GNU assembler version 2.24 (arm-linux-gnueabihf) using BFD version (GNU Binutils for Debian) 2.24
ppl_prolog_common.s: Assembler messages:
ppl_prolog_common.s:130339: Error: value of 256 too large for field of 1 bytes at 70430

$ gcc -v
Using built-in specs.
COLLECT_GCC=gcc
COLLECT_LTO_WRAPPER=/usr/lib/gcc/arm-linux-gnueabihf/4.8/lto-wrapper
Target: arm-linux-gnueabihf
Configured with: ../src/configure -v --with-pkgversion='Debian 4.8.2-16' --with-bugurl=file:///usr/share/doc/gcc-4.8/README.Bugs --enable-languages=c,c++,java,go,d,fortran,objc,obj-c++ --prefix=/usr --program-suffix=-4.8 --enable-shared --enable-linker-build-id --libexecdir=/usr/lib --without-included-gettext --enable-threads=posix --with-gxx-include-dir=/usr/include/c++/4.8 --libdir=/usr/lib --enable-nls --with-sysroot=/ --enable-clocale=gnu --enable-libstdcxx-debug --enable-libstdcxx-time=yes --enable-gnu-unique-object --disable-libmudflap --disable-libitm --disable-libquadmath --enable-plugin --with-system-zlib --disable-browser-plugin --enable-java-awt=gtk --enable-gtk-cairo --with-java-home=/usr/lib/jvm/java-1.5.0-gcj-4.8-armhf/jre --enable-java-home --with-jvm-root-dir=/usr/lib/jvm/java-1.5.0-gcj-4.8-armhf --with-jvm-jar-dir=/usr/lib/jvm-exports/java-1.5.0-gcj-4.8-armhf --with-arch-directory=arm --with-ecj-jar=/usr/share/java/eclipse-ecj.jar --enable-objc-gc --enable-multiarch --disable-sjlj-exceptions --with-arch=armv7-a --with-fpu=vfpv3-d16 --with-float=hard --with-mode=thumb --enable-checking=release --build=arm-linux-gnueabihf --host=arm-linux-gnueabihf --target=arm-linux-gnueabihf
Thread model: posix
gcc version 4.8.2 (Debian 4.8.2-16)

Confirmed - reducing.

Created attachment 32418
Reduced testcase

Reduced testcase. Looks prima-facie like lengths are messed up somewhere which needs careful digging.

-march=armv7-a -mthumb -c -O2 -mfpu=vfpv3-d16 on trunk.

Ramana

I am working on this

I have proposed a fix for this:

http://gcc.gnu.org/ml/gcc-patches/2014-04/msg00122.html

Author: yroux
Date: Mon Apr 7 15:07:33 2014
New Revision: 209191

URL: http://gcc.gnu.org/viewcvs?rev=209191&root=gcc&view=rev
Log:
2014-04-07 Charles Baylis <email address hidden>

        PR target/60609
        * config/arm/arm.h (ASM_OUTPUT_CASE_END): Remove.
        (LABEL_ALIGN_AFTER_BARRIER): Align barriers which occur after
        ADDR_DIFF_VEC.

2014-04-07 Charles Baylis <email address hidden>

        PR target/60609
        * g++.dg/torture/pr60609.C: New test.

Added:
    trunk/gcc/testsuite/g++.dg/torture/pr60609.C
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/config/arm/arm.h
    trunk/gcc/testsuite/ChangeLog

Fixed on the trunk, but not on the 4.8 branch yet.

Author: cbaylis
Date: Thu May 8 17:06:01 2014
New Revision: 210226

URL: http://gcc.gnu.org/viewcvs?rev=210226&root=gcc&view=rev
Log:
2014-05-08 Charles Baylis <email address hidden>

        Backport from mainline
        2014-04-07 Charles Baylis <email address hidden>

        PR target/60609
        * config/arm/arm.h (ASM_OUTPUT_CASE_END): Remove.
        (LABEL_ALIGN_AFTER_BARRIER): Align barriers which occur after
        ADDR_DIFF_VEC.

Added:
    branches/gcc-4_8-branch/gcc/testsuite/g++.dg/torture/pr60609.C
Modified:
    branches/gcc-4_8-branch/gcc/ChangeLog
    branches/gcc-4_8-branch/gcc/config/arm/arm.h

Author: cbaylis
Date: Thu May 8 17:06:04 2014
New Revision: 210227

URL: http://gcc.gnu.org/viewcvs?rev=210227&root=gcc&view=rev
Log:
2014-05-08 Charles Baylis <email address hidden>

        Backport from mainline
        2014-04-07 Charles Baylis <email address hidden>

        PR target/60609
        * config/arm/arm.h (ASM_OUTPUT_CASE_END): Remove.
        (LABEL_ALIGN_AFTER_BARRIER): Align barriers which occur after
        ADDR_DIFF_VEC.

Added:
    branches/gcc-4_7-branch/gcc/testsuite/g++.dg/torture/pr60609.C
Modified:
    branches/gcc-4_7-branch/gcc/ChangeLog
    branches/gcc-4_7-branch/gcc/config/arm/arm.h