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