This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, CHKP] Restore transparent alias chains
- From: Ilya Enkovich <enkovich dot gnu at gmail dot com>
- To: gcc-patches <gcc-patches at gcc dot gnu dot org>
- Date: Thu, 2 Apr 2015 17:54:19 +0300
- Subject: Re: [PATCH, CHKP] Restore transparent alias chains
- Authentication-results: sourceware.org; auth=none
- References: <20150320082033 dot GG64546 at msticlxl57 dot ims dot intel dot com>
Ping. This patch doesn't affect not instrumented code.
Thanks,
Ilya
2015-03-20 11:20 GMT+03:00 Ilya Enkovich <enkovich.gnu@gmail.com>:
> Hi,
>
> Identifiers read with input_identifier miss IDENTIFIER_TRANSPARENT_ALIAS bit. We always expect it for instrumentation clones, thus restore it input_cgraph_1. Bootstrapped and tested on x86_64-unknown-linux-gnu. OK for trunk?
>
> Thanks,
> Ilya
> --
> 2015-03-20 Ilya Enkovich <ilya.enkovich@intel.com>
>
> * lto-cgraph.c (input_cgraph_1): Always link instrumented
> assembler name with original one.
>
>
> diff --git a/gcc/lto-cgraph.c b/gcc/lto-cgraph.c
> index c875fed..d782327 100644
> --- a/gcc/lto-cgraph.c
> +++ b/gcc/lto-cgraph.c
> @@ -1613,9 +1613,8 @@ input_cgraph_1 (struct lto_file_decl_data *file_data,
> }
>
> /* Restore decl names reference. */
> - if (IDENTIFIER_TRANSPARENT_ALIAS (DECL_ASSEMBLER_NAME (cnode->decl))
> - && !TREE_CHAIN (DECL_ASSEMBLER_NAME (cnode->decl)))
> - TREE_CHAIN (DECL_ASSEMBLER_NAME (cnode->decl))
> + IDENTIFIER_TRANSPARENT_ALIAS (DECL_ASSEMBLER_NAME (cnode->decl)) = 1;
> + TREE_CHAIN (DECL_ASSEMBLER_NAME (cnode->decl))
> = DECL_ASSEMBLER_NAME (cnode->orig_decl);
> }
> }