This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH] PR ld/19645: Properly implement STT_COMMON
- From: Andreas Schwab <schwab at linux-m68k dot org>
- To: "H.J. Lu" <hjl dot tools at gmail dot com>
- Cc: Binutils <binutils at sourceware dot org>
- Date: Tue, 03 Jan 2017 13:21:16 +0100
- Subject: Re: [PATCH] PR ld/19645: Properly implement STT_COMMON
- Authentication-results: sourceware.org; auth=none
- References: <20160222213825.GA8172@intel.com> <CAMe9rOqb-XQft4YgxMwoOz87=DgJ3H5vfj4K7tHQwX3Qc6=wsA@mail.gmail.com>
On Feb 26 2016, "H.J. Lu" <hjl.tools@gmail.com> wrote:
> diff --git a/ld/testsuite/ld-elfcomm/elfcomm.exp b/ld/testsuite/ld-elfcomm/elfcomm.exp
> index 81fd520..926b98c 100644
> --- a/ld/testsuite/ld-elfcomm/elfcomm.exp
> +++ b/ld/testsuite/ld-elfcomm/elfcomm.exp
> @@ -187,7 +170,9 @@ if [istarget nios2*-*-*] {
> # Explicitly use "-fcommon" so that even if $CFLAGS includes
> # "-fno-common", these tests are compiled as expected.
> if { ![ld_compile "$CC $CFLAGS -fcommon" $srcdir/$subdir/common1a.c tmpdir/common1a.o]
> - || ![ld_compile "$CC $CFLAGS -fcommon" $srcdir/$subdir/common1b.c tmpdir/common1b.o] } {
> + || ![ld_compile "$CC $CFLAGS -fcommon" $srcdir/$subdir/common1b.c tmpdir/common1b.o]
> + || ![ld_compile "$CC $CFLAGS -Wa,--elf-stt-common=yes -fcommon" $srcdir/$subdir/common1b.c tmpdir/common1c.o] } {
This is passing --elf-stt-common=yes to the installed assembler, not the
newly built one.
Andreas.
--
Andreas Schwab, schwab@linux-m68k.org
GPG Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5
"And now for something completely different."