The revised patch is attached. The differences are in what options are defined 
and propagated to Makefile for CET enabling.

Ok for trunk?

Igor


> -----Original Message-----
> From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-
> ow...@gcc.gnu.org] On Behalf Of Jeff Law
> Sent: Wednesday, October 18, 2017 1:34 AM
> To: Tsimbalist, Igor V <igor.v.tsimbal...@intel.com>; gcc-
> patc...@gcc.gnu.org
> Subject: Re: [PATCH 10/22] Enable building libcilkrts with Intel CET
> 
> On 10/12/2017 02:13 PM, Tsimbalist, Igor V wrote:
> > Enable building libcilkrts with CET options.
> >
> > libcilkrts/
> >     * Makefile.am: Add AM_CXXFLAGS and XCXXFLAGS.
> >     * configure.ac: Set CET_FLAGS, update XCFLAGS, XCXXFLAGS.
> >     * Makefile.in: Regenerate.
> >     * aclocal.m4: Likewise.
> >     * configure: Likewise.
> >
> So like the other patches in this space, the inclusion of cet.h seems
> wrong.  I don't see why this should be needed here.
> 
> It's OK with that bit removed and once any prereqs are OK'd.
> 
> jeff

Attachment: 0010-Enable-building-libcilkrts-with-Intel-CET.PATCH
Description: 0010-Enable-building-libcilkrts-with-Intel-CET.PATCH

Reply via email to