[Patch,ld] Fix check_lto_available
H.J. Lu
hjl.tools@gmail.com
Wed Dec 11 14:09:00 GMT 2013
On Wed, Dec 11, 2013 at 5:57 AM, Vidya Praveen <vidyapraveen@arm.com> wrote:
> Hello,
>
> This patch modifies check_lto_available of ld regression test to
>
> 1. Make use of cflags, ldflags from board description file
>
> 2. Test by building an executable rather than a shared library
> - This is needed for the baremetal targets as shared library makes no sense
> to the baremetal targets. lto.exp which makes use of this check also has
> guard for shared library tests.
>
> Reference:
> https://sourceware.org/ml/binutils/2013-10/msg00214.html
>
> Tested for aarch64-none-elf and arm-none-eabi, arm-none-linux-gnueabihf and
> x86_64-unknown-linux-gnu.
>
> OK for trunk?
>
> Cheers
> VP.
>
> ld/testsuite/ChangeLog:
>
> 2013-12-11 Vidya Praveen <vidyapraveen@arm.com>
>
> * lib/ld-lib.exp (check_lto_available): Support cflags,ldflags and
> test by compiling for an executable rather than shared library.
>
You also changed check_plugin_api_available. Please send
a separate patch for it.
As for check_lto_available, it is not OK unless lto.exp is changed
to handle LTO tests which use -shared.
--
H.J.
More information about the Binutils
mailing list