[PATCH, MPX runtime 1/2] Integrate MPX runtime library

[Ilya Enkovich]

2014-12-09 22:57 GMT+03:00 Jeff Law <law@redhat.com>:
> On 12/09/14 01:24, Ilya Enkovich wrote:
>>
>> On 24 Nov 17:02, Ilya Enkovich wrote:
>>>
>>>
>>> Right.  This works for both top level and multilib checks because failing
>>> test is used and CC is usually not set when it's called by the top level
>>> configure.  If we configure with CC=... then it may go wrong.  I left only
>>> target check in configure.tgt and inlined test for x32 into libmpx
>>> configure.
>>>
>>>>
>>>> --
>>>> Joseph S. Myers
>>>> joseph@codesourcery.com
>>>
>>>
>>> Here is an updated version.
>>>
>>> Thanks,
>>> Ilya
>>> --
>>
>>
>> Here is an updated version.  I moved linker specs to target.  Currently
>> mpx libraries are built for x86_64-*-linux* | i?86-*-linux*, so I think
>> gcc/config/i386/linux-common.h is a proper place for LIBMPX delcarations.
>>
>> Thanks,
>> Ilya
>> --
>> 2014-12-09  Ilya Enkovich  <ilya.enkovich@intel.com>
>>
>>         * Makefile.def: Add libmpx.
>>         * configure.ac: Add libmpx.
>>         * Makefile.in: Regenerate.
>>         * configure: Regenerate.
>>
>> gcc/
>>
>> 2014-12-09  Ilya Enkovich  <ilya.enkovich@intel.com>
>>
>>         * config/i386/linux-common.h (LIBMPX_LIBS): New.
>>         (LIBMPX_SPEC): New.
>>         (CHKP_SPEC): New.
>>         * gcc.c (CHKP_SPEC): New.
>>         (LINK_COMMAND_SPEC): Add CHKP_SPEC.
>>         * c-family/c.opt (static-libmpx): New.
>>
>> libmpx/
>>
>
>> diff --git a/gcc/c-family/c.opt b/gcc/c-family/c.opt
>> index b9f7c65..65731cc 100644
>> --- a/gcc/c-family/c.opt
>> +++ b/gcc/c-family/c.opt
>> @@ -1020,6 +1020,9 @@ fchkp-instrument-marked-only
>>   C ObjC C++ ObjC++ LTO Report Var(flag_chkp_instrument_marked_only)
>> Init(0)
>>   Instrument only functions marked with bnd_instrument attribute.
>>
>> +static-libmpx
>> +Driver
>> +
>>   fcilkplus
>>   C ObjC C++ ObjC++ LTO Report Var(flag_cilkplus) Init(0)
>>   Enable Cilk Plus
>
> Wasn't this duplicated in the mpx-wrapper patch?

Wrappers patch introduces similar option static-libmpxwrappers.

>
> I think this is OK on the technical side.  I need to do to some vote
> tallying on the steering committee side WRT licensing, ownership of the
> sources, canonical source location, etc, so please don't commit yet.

Great! Let me know when it's time to commit.

Thanks,
Ilya

>
> jeff
>