[Bug target/116339] New: arm-unknown-elf fails to build

[Bug target/116339] New: arm-unknown-elf fails to build

[dkm at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=116339

            Bug ID: 116339
           Summary: arm-unknown-elf fails to build
           Product: gcc
           Version: 15.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: target
          Assignee: unassigned at gcc dot gnu.org
          Reporter: dkm at gcc dot gnu.org
  Target Milestone: ---

The nightly build of arm-unknown-elf has been failing the last 3 days in
compiler-explorer:

Configured with: ../gcc-14.1.0/configure
--prefix=/opt/compiler-explorer/gcc-build/staging
--enable-libstdcxx-backtrace=yes -
       │ -build=x86_64-linux-gnu --host=x86_64-linux-gnu
--target=x86_64-linux-gnu --disable-bootstrap --enable-multiarch --with-abi=m64
--with-multilib-list=m32,m64,mx32 --e
       │ nable-multilib --enable-clocale=gnu
--enable-languages=c,c++,fortran,ada,objc,obj-c++,go,d,rust,m2 --enable-ld=yes
--enable-gold=yes --enable-libstdcxx-debug --enabl
       │ e-libstdcxx-time=yes --enable-linker-build-id --enable-lto
--enable-plugins --enable-threads=posix
--with-pkgversion=Compiler-Explorer-Build-gcc--binutils-2.42


327511   │ 2024-08-12T00:22:22.0398812Z [ALL  ]   
/opt/.build/arm-unknown-linux-gnueabihf/src/gcc/libgcc/config/arm/lib1funcs.S:
Assembler messages:
327512   │ 2024-08-12T00:22:22.0400251Z [ERROR]   
/opt/.build/arm-unknown-linux-gnueabihf/src/gcc/libgcc/config/arm/lib1funcs.S:1490:
Error: too many positional arguments
327513   │ 2024-08-12T00:22:22.0401288Z [ALL  ]    echo '#include
"soft-fp/divdf3.c"' >> divdf3.c
327514   │ 2024-08-12T00:22:22.0401820Z [ERROR]    make[3]: *** [Makefile:498:
_dvmd_lnx_s.o] Error 1
327515   │ 2024-08-12T00:22:22.0402344Z [ERROR]    make[3]: *** Waiting for
unfinished jobs....

[Bug target/116339] arm-unknown-elf fails to build

[pinskia at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=116339

Andrew Pinski <pinskia at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |MOVED
             Status|UNCONFIRMED                 |RESOLVED

--- Comment #1 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
Binutils  bug:
https://sourceware.org/pipermail/binutils/2024-August/136228.html

[Bug target/116339] arm-unknown-elf fails to build

[pinskia at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=116339

Andrew Pinski <pinskia at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           See Also|                            |https://sourceware.org/bugz
                   |                            |illa/show_bug.cgi?id=32073

--- Comment #2 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
Also https://sourceware.org/bugzilla/show_bug.cgi?id=32073

[Bug target/116339] arm-unknown-linux-gnueabihf fails to build

[dkm at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=116339

Marc Poulhiès <dkm at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |UNCONFIRMED
            Summary|arm-unknown-elf fails to    |arm-unknown-linux-gnueabihf
                   |build                       |fails to build
             Target|                            |arm-unknown-linux-gnueabihf
         Resolution|MOVED                       |---

--- Comment #3 from Marc Poulhiès <dkm at gcc dot gnu.org> ---
Got the wrong configure line sorry.

The actual command for configure is: 

CC_FOR_BUILD='x86_64-build_pc-linux-gnu-gcc' CFLAGS='-O2 -g
-I/opt/.build/arm-unknown-linux-gnueabihf/buildtools/include  '
CFLAGS_FOR_BUILD='-O2 -g
-I/opt/.build/arm-unknown-linux-gnueabihf/buildtools/include  ' CXXFLAGS='-O2
-g -I/opt/.build/arm-unknown-linux-gnueabihf/buildtools/include   '
CXXFLAGS_FOR_BUILD='-O2 -g
-I/opt/.build/arm-unknown-linux-gnueabihf/buildtools/include   '
LDFLAGS='-L/opt/.build/arm-unknown-linux-gnueabihf/buildtools/lib  '
CFLAGS_FOR_TARGET='-g -O2 ' CXXFLAGS_FOR_TARGET='-g -O2 ' LDFLAGS_FOR_TARGET=''
'/usr/bin/bash' '/opt/.build/arm-unknown-linux-gnueabihf/src/gcc/configure'
'--build=x86_64-build_pc-linux-gnu' '--host=x86_64-build_pc-linux-gnu'
'--target=arm-unknown-linux-gnueabihf'
'--prefix=/opt/.build/arm-unknown-linux-gnueabihf/buildtools'
'--exec_prefix=/opt/.build/arm-unknown-linux-gnueabihf/buildtools'
'--with-local-prefix=/opt/compiler-explorer/arm/gcc-trunk-20240812/arm-unknown-linux-gnueabihf/arm-unknown-linux-gnueabihf/sysroot'
'--with-sysroot=/opt/compiler-explorer/arm/gcc-trunk-20240812/arm-unknown-linux-gnueabihf/arm-unknown-linux-gnueabihf/sysroot'
'--with-newlib' '--enable-threads=no' '--disable-shared' '--with-arch=armv7-a'
'--with-fpu=neon' '--with-float=hard' '--enable-__cxa_atexit'
'--disable-libgomp' '--disable-libmudflap' '--disable-libmpx' '--enable-libssp'
'--enable-libquadmath' '--enable-libquadmath-support' '--disable-libstdcxx'
'--with-gmp=/opt/.build/arm-unknown-linux-gnueabihf/buildtools'
'--with-mpfr=/opt/.build/arm-unknown-linux-gnueabihf/buildtools'
'--with-mpc=/opt/.build/arm-unknown-linux-gnueabihf/buildtools'
'--with-isl=/opt/.build/arm-unknown-linux-gnueabihf/buildtools' '--enable-lto'
'--enable-target-optspace' '--with-glibc-version=2.32' '--disable-nls'
'--disable-multilib' '--enable-languages=c' '--with-mode=thumb'
'--enable-checking=release'

[Bug target/116339] arm-unknown-linux-gnueabihf fails to build

[pinskia at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=116339

Andrew Pinski <pinskia at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|---                         |MOVED

--- Comment #4 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
https://sourceware.org/pipermail/binutils/2024-August/136234.html

[Bug target/116339] arm-unknown-linux-gnueabihf fails to build

[pinskia at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=116339

--- Comment #5 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
Note this is not a gcc bug but rather binutils and has already been reported.

[Bug target/116339] arm-unknown-linux-gnueabihf fails to build

[dkm at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=116339

--- Comment #6 from Marc Poulhiès <dkm at gcc dot gnu.org> ---
Thanks Andrew, I'll switch our builds master binutils, sorry for the noise.

[Bug target/116339] arm-unknown-linux-gnueabihf fails to build

[sjames at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=116339

--- Comment #7 from Sam James <sjames at gcc dot gnu.org> ---
I think master is what's broken, unfortunately.

[Bug target/116339] arm-unknown-linux-gnueabihf fails to build

[dkm at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=116339

--- Comment #8 from Marc Poulhiès <dkm at gcc dot gnu.org> ---
Ha, correct, we're already using master. But it's also where the revert has
been applied, so everything should be ok now. I should have waited one more day
and this would have saved you some time...

[Bug target/116339] arm-unknown-linux-gnueabihf fails to build

[sjames at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=116339

--- Comment #9 from Sam James <sjames at gcc dot gnu.org> ---
The revert hasn't landed and I fear it's going to be a bit controversial to get
it to land, even. :(

i.e. I think master is still broken.

[Bug target/116339] arm-unknown-linux-gnueabihf fails to build

[sjames at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=116339

Sam James <sjames at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jbeulich at suse dot com,
                   |                            |sjames at gcc dot gnu.org

--- Comment #10 from Sam James <sjames at gcc dot gnu.org> ---
Jan, can you have a look? I'm happy to report issues for the various breakages
if they look prima facie like real problems and not something necessitating a
change in the gas validation.

[Bug target/116339] arm-unknown-linux-gnueabihf fails to build

[jbeulich at suse dot com]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=116339

--- Comment #11 from jbeulich at suse dot com ---
(In reply to Andrew Pinski from comment #5)
> Note this is not a gcc bug but rather binutils and has already been reported.

Mind me asking what you take this from? See the gas bug that you did already
link to. Just because gas accepted broken code doesn't mean it'll need to
continue to. We may need some transitional mode, yet how that's to look like is
entirely unclear.

(In reply to Sam James from comment #10)
> Jan, can you have a look? I'm happy to report issues for the various
> breakages if they look prima facie like real problems and not something
> necessitating a change in the gas validation.

I already looked at another instance in the same file that was pointed out:

97:     cfi_pop 97b - \unwind, 0xe, 0x0

There are 5 arguments there, not (as gas previously parsed it) just three. Same
for

98:     cfi_push 98b - __aeabi_ldiv0, 0xe, -0x4, 0x8

which I think is the line in question here.

Since macro arguments (and parameters as well) are permitted to be separated by
just blanks, arguments including blanks need parenthesizing or quoting. But
let's continue the discussion on a proper way forward in the binutils bug.