This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH v4] Avoid unnecessarily numbering cloned symbols.


> From aea94273e7a477a03d1ee10a5d9043d6d13b8e8d Mon Sep 17 00:00:00 2001
> From: Michael Ploujnikov <michael.ploujnikov@oracle.com>
> Date: Thu, 25 Oct 2018 13:16:36 -0400
> Subject: [PATCH] Avoid unnecessarily numbering cloned symbols.
> 
> gcc/ChangeLog:
> 
> 2018-10-26  Michael Ploujnikov  <michael.ploujnikov@oracle.com>
> 
> 	* cgraph.h (clone_function_name_1): Replaced by new
> 	  clone_function_name_numbered that takes name as string; for
> 	  privatize_symbol_name_1 use only.
> 	  (clone_function_name): Renamed to
> 	  clone_function_name_numbered to be explicit about numbering.
> 	  (clone_function_name): New two-argument function that does
> 	  not number its output.
> 	  (clone_function_name): New three-argument function that
> 	  takes a number to append to its output.
> 	* cgraphclones.c (duplicate_thunk_for_node):
> 	  (clone_function_name_1): Renamed.
> 	  (clone_function_name_numbered): Two new functions.
> 	  (clone_function_name): Improved documentation.
> 	  (cgraph_node::create_virtual_clone): Use clone_function_name_numbered.
> 	* config/rs6000/rs6000.c (make_resolver_func): Ditto.
> 	* final.c (final_scan_insn_1): Use the new clone_function_name
> 	  without numbering.
> 	* multiple_target.c (create_dispatcher_calls): Ditto.
> 	  (create_target_clone): Ditto.
> 	* omp-expand.c (grid_expand_target_grid_body): Ditto.
> 	* omp-low.c (create_omp_child_function_name): Ditto.
> 	* omp-simd-clone.c (simd_clone_create): Ditto.
> 	* symtab.c (simd_symtab_node::noninterposable_alias): Use the
> 	  new clone_function_name without numbering.
> 
> gcc/lto/ChangeLog:
> 
> 2018-10-26  Michael Ploujnikov  <michael.ploujnikov@oracle.com>
> 
> 	* lto-partition.c (privatize_symbol_name_1): Use
> 	  clone_function_name_numbered.
> 
> gcc/testsuite/ChangeLog:
> 
> 2018-10-26  Michael Ploujnikov  <michael.ploujnikov@oracle.com>
> 
> 	* gcc.dg/tree-prof/cold_partition_label.c: Update for cold
> 	  section names without numbers.
> 	* gcc.dg/tree-prof/section-attr-1.c: Ditto.
> 	* gcc.dg/tree-prof/section-attr-2.c: Ditto.
> 	* gcc.dg/tree-prof/section-attr-3.c: Ditto.

OK,
thanks!
Honza


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]