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

[Jeff Law <law@redhat.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