On 08/01/2017 02:56 AM, Tsimbalist, Igor V wrote:
> Part#3. Add tests for -finstrument-control-flow and notrack attribute.
> 
> 
> 0003-Part-3.-Add-tests-for-finstrument-control-flow-and-n.patch
> 
> 
> From 7869de8a0c0ec55c4e9240c2483fefee97bf34c9 Mon Sep 17 00:00:00 2001
> From: Igor Tsimbalist <igor.v.tsimbal...@intel.com>
> Date: Mon, 3 Jul 2017 17:29:08 +0300
> Subject: [PATCH 3/9] Part#3. Add tests for -finstrument-control-flow and
>  notrack attribute.
> 
> gcc/testsuite/
> 
>       * c-c++-common/finstrument-control-flow.c: New test.
>       * c-c++-common/notrack-1.c: Likewise.
>       * c-c++-common/notrack-2.c: Likewise.
No concerns with the existing tests.

We should consider an ICF test as I outlined in an earlier message.

We should also consider tests where we drop/add the notrack attribute as
ISTM we ought to be getting warnings in those cases.

Finally, you should consider tests in gcc.target/i386 that verify we
generate the proper instrumentation for a  few tests.

jeff

Reply via email to