On 09/19/2017 07:58 AM, Tsimbalist, Igor V wrote:
> Here is an updated patch (version #2). Mainly attribute and option  names 
> were changed.
> The test for ICF will be introduced in x86 specific tests (patch 0006-Part-6) 
> as the implementation
> checks if the CF instrumentation is on to adjust a hash based on 
> 'nocf'_check' attribute presence.
> In generic part CF instrumentation is off as no implementation exist.
> 
> The patch for x86 specific tests (patch 0006-Part-6) is being reviewed by 
> Uros.
> 
> gcc/testsuite/
>       * c-c++-common/fcf-protection-1.c: New test.
>       * c-c++-common/fcf-protection-2.c: Likewise.
>       * c-c++-common/fcf-protection-3.c: Likewise.
>       * c-c++-common/fcf-protection-4.c: Likewise.
>       * c-c++-common/fcf-protection-5.c: Likewise.
>       * c-c++-common/attr-nocf-check-1.c: Likewise.
>       * c-c++-common/attr-nocf-check-2.c: Likewise.
>       * c-c++-common/attr-nocf-check-3.c: Likewise.
> 
> Is it ok for trunk?
>
This is OK once the CET changes for the compiler are approved.

Jeff

Reply via email to