public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [pph] Removing unused timevars following pph instrumentation removal patch (issue4568053)
@ 2011-06-03 17:29 Gabriel Charette
  2011-06-03 20:38 ` Diego Novillo
  0 siblings, 1 reply; 2+ messages in thread
From: Gabriel Charette @ 2011-06-03 17:29 UTC (permalink / raw)
  To: reply, crowl, dnovillo, gcc-patches

Removed the 3 PPH timevars used in pph instrumentation. Those are not used
anymore following the instrumentation removal patch.

Bootstrap build and regression testing were successful.

2011-06-03  Gabriel Charette  <gchare@google.com>

	* timevar.def (TV_PPH_CACHE_IN): Remove.
	(TV_PPH_CACHE_OUT): Remove.
	(TV_PPH_MANAGE): Remove.

Index: timevar.def
===================================================================
--- timevar.def	(revision 174568)
+++ timevar.def	(working copy)
@@ -69,11 +69,6 @@
 DEFTIMEVAR (TV_PTH_SKIP_TOKENS       , "PTH skip cached tokens")
 DEFTIMEVAR (TV_PTH_INIT              , "PTH initialization")
 
-/* Time spent handling PPH state.  */
-DEFTIMEVAR (TV_PPH_CACHE_IN          , "PPH cache in")
-DEFTIMEVAR (TV_PPH_CACHE_OUT         , "PPH cache out")
-DEFTIMEVAR (TV_PPH_MANAGE            , "PPH bookkeeping")
-
 DEFTIMEVAR (TV_CGRAPH                , "callgraph construction")
 DEFTIMEVAR (TV_CGRAPHOPT             , "callgraph optimization")
 DEFTIMEVAR (TV_VARPOOL               , "varpool construction")

--
This patch is available for review at http://codereview.appspot.com/4568053

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [pph] Removing unused timevars following pph instrumentation removal patch (issue4568053)
  2011-06-03 17:29 [pph] Removing unused timevars following pph instrumentation removal patch (issue4568053) Gabriel Charette
@ 2011-06-03 20:38 ` Diego Novillo
  0 siblings, 0 replies; 2+ messages in thread
From: Diego Novillo @ 2011-06-03 20:38 UTC (permalink / raw)
  To: Gabriel Charette; +Cc: reply, Lawrence Crowl, gcc-patches

On Fri, Jun 3, 2011 at 13:28, Gabriel Charette <gchare@google.com> wrote:

> 2011-06-03  Gabriel Charette  <gchare@google.com>
>
>        * timevar.def (TV_PPH_CACHE_IN): Remove.
>        (TV_PPH_CACHE_OUT): Remove.
>        (TV_PPH_MANAGE): Remove.

OK.  Committed.


Diego.

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2011-06-03 20:38 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-06-03 17:29 [pph] Removing unused timevars following pph instrumentation removal patch (issue4568053) Gabriel Charette
2011-06-03 20:38 ` Diego Novillo

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).