On Tue, Dec 13, 2016 at 09:51:13AM +0100, Martin Liška wrote:
> >From ff843db17ba284d2e99bab567d76d4b779f9f1d6 Mon Sep 17 00:00:00 2001
> From: marxin <mli...@suse.cz>
> Date: Fri, 9 Dec 2016 16:24:16 +0100
> Subject: [PATCH 1/2] Add pretty printer for ASAN_MARK and add a helper fn
>
> gcc/ChangeLog:
>
> 2016-12-09 Martin Liska <mli...@suse.cz>
>
> * asan.c (asan_mark_poison_p): Remove.
> (asan_mark_p): New function.
> (transform_statements): Use the function.
> (asan_expand_mark_ifn): Do not use masked enum.
> * asan.h (enum asan_mark_flags): Declare it via a macro.
> * gimple-pretty-print.c (dump_gimple_call_args): Dump first
> argument of ASAN_MARK.
> * gimplify.c (build_asan_poison_call_expr): Use new enum values.
> (asan_poison_variable): Likewise.
>
> gcc/testsuite/ChangeLog:
>
> 2016-12-12 Martin Liska <mli...@suse.cz>
>
> * gcc.dg/asan/use-after-scope-goto-1.c: Update first argument of
> scanned pattern ASAN_MARK.
> * gcc.dg/asan/use-after-scope-goto-2.c: Likewise.
> * gcc.dg/asan/use-after-scope-switch-1.c: Likewise.
> * gcc.dg/asan/use-after-scope-switch-2.c: Likewise.
> * gcc.dg/asan/use-after-scope-switch-3.c: Likewise.
Ok. But the builtins should be renamed too (incrementally),
BUILT_IN_ASAN_CLOBBER_N, "__asan_poison_stack_memory",
should really be BUILT_IN_ASAN_POISON_STACK_MEMORY etc.
Jakub
