From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 59053 invoked by alias); 3 Jul 2019 09:09:34 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Received: (qmail 58780 invoked by uid 89); 3 Jul 2019 09:09:33 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-4.9 required=5.0 tests=AWL,BAYES_00 autolearn=ham version=3.3.1 spammy= X-HELO: nikam.ms.mff.cuni.cz Received: from nikam.ms.mff.cuni.cz (HELO nikam.ms.mff.cuni.cz) (195.113.20.16) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Wed, 03 Jul 2019 09:09:32 +0000 Received: by nikam.ms.mff.cuni.cz (Postfix, from userid 16202) id B2B61280838; Wed, 3 Jul 2019 11:09:30 +0200 (CEST) Date: Wed, 03 Jul 2019 09:09:00 -0000 From: Jan Hubicka To: Martin =?iso-8859-2?Q?Li=B9ka?= Cc: Xiong Hu Luo , gcc-patches@gcc.gnu.org, segher@kernel.crashing.org, wschmidt@linux.ibm.com, luoxhu@cn.ibm.com Subject: Re: [PATCH 2/2] Rename SINGE_VALUE to TOPN_VALUES counters. Message-ID: <20190703090930.hxovaaesio4cowxa@kam.mff.cuni.cz> References: <20190618014521.67198-1-luoxhu@linux.ibm.com> <20190620134731.fhjf7ezskq55yz6o@kam.mff.cuni.cz> <92ed1dc9-0ecf-a794-a907-9f5bf9f8017f@suse.cz> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <92ed1dc9-0ecf-a794-a907-9f5bf9f8017f@suse.cz> User-Agent: NeoMutt/20170113 (1.7.2) X-SW-Source: 2019-07/txt/msg00239.txt.bz2 > And the second part is rename so that it reflect reality > that single value can actually track multiple values. > > Patch can bootstrap on x86_64-linux-gnu and survives regression tests. > > Ready to be installed? > Thanks, > Martin > From cc9e93d43941176e92b5821e5a8134a5319a10b4 Mon Sep 17 00:00:00 2001 > From: Martin Liska > Date: Thu, 20 Jun 2019 14:50:23 +0200 > Subject: [PATCH 2/2] Rename SINGE_VALUE to TOPN_VALUES counters. > > gcc/ChangeLog: > > 2019-06-20 Martin Liska > > * gcov-counter.def (GCOV_COUNTER_V_SINGLE): Remove. > (GCOV_COUNTER_V_TOPN): New. > (GCOV_COUNTER_V_INDIR): Use _topn. > * gcov-io.h (GCOV_DISK_SINGLE_VALUES): Remove. > (GCOV_TOPN_VALUES): New. > (GCOV_SINGLE_VALUE_COUNTERS): Remove. > (GCOV_TOPN_VALUES_COUNTERS): New. > * profile.c (instrument_values): Use HIST_TYPE_TOPN_VALUES. > * tree-profile.c: > (gimple_init_gcov_profiler): Rename variables from one_value > to topn_values. > (gimple_gen_one_value_profiler): Remove. > (gimple_gen_topn_values_profiler): New function. > * value-prof.c (dump_histogram_value): Use TOPN_VALUES > names instead of SINGLE_VALUE. > (stream_out_histogram_value): Likewise. > (stream_in_histogram_value): Likewise. > (get_most_common_single_value): Likewise. > (gimple_divmod_fixed_value_transform): Likewise. > (gimple_stringops_transform): Likewise. > (gimple_divmod_values_to_profile): Likewise. > (gimple_stringops_values_to_profile): Likewise. > (gimple_find_values_to_profile): Likewise. > * value-prof.h (enum hist_type): Rename to TOPN. > (gimple_gen_one_value_profiler): Remove. > (gimple_gen_topn_values_profiler): New. > > libgcc/ChangeLog: > > 2019-06-20 Martin Liska > > * Makefile.in: Use topn_values instead of one_value names. > * libgcov-merge.c (__gcov_merge_single): Move to ... > (__gcov_merge_topn): ... this. > (merge_single_value_set): Move to ... > (merge_topn_values_set): ... this. > * libgcov-profiler.c (__gcov_one_value_profiler_body): Move to > ... > (__gcov_topn_values_profiler_body): ... this. > (__gcov_one_value_profiler_v2): Move to ... > (__gcov_topn_values_profiler): ... this. > (__gcov_one_value_profiler_v2_atomic): Move to ... > (__gcov_topn_values_profiler_atomic): ... this. > (__gcov_indirect_call_profiler_v4): Remove. > * libgcov-util.c (__gcov_single_counter_op): Move to ... > (__gcov_topn_counter_op): ... this. > * libgcov.h (L_gcov_merge_single): Remove. > (L_gcov_merge_topn): New. > (__gcov_merge_single): Remove. > (__gcov_merge_topn): New. > (__gcov_one_value_profiler_v2): Move to .. > (__gcov_topn_values_profiler): ... this. > (__gcov_one_value_profiler_v2_atomic): Move to ... > (__gcov_topn_values_profiler_atomic): ... this. OK, I would rename the __gcov_topn_values_profiler to _v2 since we had this function before. Honza