Hi, On 2018-10-26 10:25 a.m., Jan Hubicka wrote: >> From aea94273e7a477a03d1ee10a5d9043d6d13b8e8d Mon Sep 17 00:00:00 2001 >> From: Michael Ploujnikov <michael.ploujni...@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.ploujni...@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.ploujni...@oracle.com> >> >> * lto-partition.c (privatize_symbol_name_1): Use >> clone_function_name_numbered. >> >> gcc/testsuite/ChangeLog: >> >> 2018-10-26 Michael Ploujnikov <michael.ploujni...@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 >
Thanks again for the review. This is my first patch and I don't have commit access. What should I do? - Michael
signature.asc
Description: OpenPGP digital signature
