Re: [PATCH, libmpx, i386, PR driver/65444] Pass '-z bndplt' when building dynamic objects with MPX

[Jeff Law <law at redhat dot com>]

On 06/05/2015 08:01 AM, Ilya Enkovich wrote:
> On 04 Jun 15:53, Joseph Myers wrote:
> > On Thu, 4 Jun 2015, Ilya Enkovich wrote:
> >
> > > 2015-06-03 18:59 GMT+03:00 Joseph Myers <joseph@codesourcery.com>:
> > > > You could, for example, have
> > > >
> > > > #if SOMETHING
> > > > #define MSG "%nsome msg"
> > > > #else
> > > > #define MSG ""
> > > > #endif
> > > >
> > > > and have another spec using MSG - that should work.
> > >
> > > In this case I should define SOMETHING in configure of gcc, not in
> > > configure of libmpx, right? But it would mean I check host linker, not
> > > target.
> >
> > gcc/ configure tests on the linker generally test the build-x-target
> > linker, which is required to have the same version and be configured the
> > same as the host-x-target linker.  (The tests mustn't actually try to link
> > anything, but they can e.g. see if a particular option is mentioned in
> > --help output.  In general they also have version number checks for the
> > case of an in-tree linker build.)
> >
> > --
> > Joseph S. Myers
> > joseph@codesourcery.com
>
> Thanks for help!  Here is a new patch version with a check moved into gcc/configure.  Does it look OK?
>
> Thanks,
> Ilya
> --
> gcc/
>
> 2015-06-05  Ilya Enkovich  <enkovich.gnu@gmail.com>
>
> 	* config/i386/linux-common.h (LINK_MPX): New.
> 	(MPX_SPEC): Use LINK_MPX instead of %(link_mpx).
> 	* configure.ac: Add HAVE_LD_BNDPLT_SUPPORT macro
> 	indicating '-z bndplt' support by linker.
> 	* configure: Regenerate.
> 	* config.in: Regenerate.
>
> libmpx/
>
> 2015-06-05  Ilya Enkovich  <enkovich.gnu@gmail.com>
>
> 	* configure.ac: Remove link_mpx.
> 	* libmpx.spec.in: Likewise.
> 	* configure: Regenerate.
OK.  Sorry for the long wait.

jeff