[PATCH 2/2] ld: Don't evaluate unneeded PROVIDE expressions.
Hans-Peter Nilsson
hans-peter.nilsson@axis.com
Tue Jan 20 14:21:00 GMT 2015
> From: Andrew Burgess <andrew.burgess@embecosm.com>
> Date: Mon, 19 Jan 2015 12:06:20 +0100
> * Alan Modra <amodra@gmail.com> [2015-01-12 11:08:53 +1030]:
>
> > > ld/testsuite/ChangeLog:
> > >
> > > * ld-scripts/provide-4.d: New file.
> > > * ld-scripts/provide-4-map.d: New file.
> > > * ld-scripts/provide-4.t: New file.
> > > * ld-scripts/provide.exp: Run the provide-4.d test.
> >
> > This part isn't OK yet. You need to make sure new tests pass on more
> > than just Linux x86.
>
> Alan,
>
> Thanks for taking the time to review my patch, and I'm sorry that the
> change wasn't up to standard.
>
> I've updated the expected results, and tested against the same set of
> builds you suggest, I'm now seeing passes on all but 3 builds, and
> these 3 failures are, I believe, not related to my changes.
I'd guess the testsuite adjustments were not double-checked that
they actually do pass on a 32-bit host, as these FAILs are
introduced by your patches, for targets cris-axis-linux-gnu,
arm-unknown-eabi, cris-axis-elf *on a 32-bit host* (so if
nothing else use 'CC=gcc -m32' on your x86_64-unknown-linux-gnu
host as my autotester does and double-check that you get a
'32-bit bfd_vma'):
Running /tmp/hpautotest-binutils/bsrc/src/ld/testsuite/ld-scripts/provide.exp ...
FAIL: ld-scripts/provide-4 (map file check)
FAIL: ld-scripts/provide-5 (map file check)
(Please fix.)
brgds, H-P
More information about the Binutils
mailing list