Add a debugcounter to IPA-SRA

Message ID	20100728133155.GA5687@virgil.arch.suse.de
State	New
Headers	show Return-Path: <gcc-patches-return-269171-incoming=patchwork.ozlabs.org@gcc.gnu.org> Date: Wed, 28 Jul 2010 15:31:56 +0200 From: Martin Jambor <mjambor@suse.cz> To: GCC Patches <gcc-patches@gcc.gnu.org> Subject: [PATCH] Add a debugcounter to IPA-SRA Message-ID: <20100728133155.GA5687@virgil.arch.suse.de> Mail-Followup-To: GCC Patches <gcc-patches@gcc.gnu.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline User-Agent: Mutt/1.5.20 (2009-06-14) Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk Sender: gcc-patches-owner@gcc.gnu.org

Message ID

20100728133155.GA5687@virgil.arch.suse.de

State

New

Headers

Date: Wed, 28 Jul 2010 15:31:56 +0200
From: Martin Jambor <mjambor@suse.cz>
To: GCC Patches <gcc-patches@gcc.gnu.org>
Subject: [PATCH] Add a debugcounter to IPA-SRA
Message-ID: <20100728133155.GA5687@virgil.arch.suse.de>
Mail-Followup-To: GCC Patches <gcc-patches@gcc.gnu.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
User-Agent: Mutt/1.5.20 (2009-06-14)
Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm
Precedence: bulk
Sender: gcc-patches-owner@gcc.gnu.org

Commit Message

Martin Jambor July 28, 2010, 1:31 p.m. UTC

Hi,

debug counters can often be useful when looking into IPA-SRA related
issues and so the patch below adds one to the gate of the pass.
Bootstrapped and tested on x86_64-linux.  OK for trunk?

Thanks,

Martin


2010-07-26  Martin Jambor  <mjambor@suse.cz>

	* dbgcnt.def (eipa_sra): New counter.
	* tree-sra.c (ipa_early_sra_gate): Also check eipa_sra debug counter.

Comments

Richard Biener July 28, 2010, 1:38 p.m. UTC | #1

On Wed, Jul 28, 2010 at 3:31 PM, Martin Jambor <mjambor@suse.cz> wrote:
> Hi,
>
> debug counters can often be useful when looking into IPA-SRA related
> issues and so the patch below adds one to the gate of the pass.
> Bootstrapped and tested on x86_64-linux.  OK for trunk?

Ok

Thanks,
Richard.

> Thanks,
>
> Martin
>
>
> 2010-07-26  Martin Jambor  <mjambor@suse.cz>
>
>        * dbgcnt.def (eipa_sra): New counter.
>        * tree-sra.c (ipa_early_sra_gate): Also check eipa_sra debug counter.
>
> Index: mine/gcc/dbgcnt.def
> ===================================================================
> --- mine.orig/gcc/dbgcnt.def
> +++ mine/gcc/dbgcnt.def
> @@ -169,6 +169,7 @@ DEBUG_COUNTER (pre)
>  DEBUG_COUNTER (pre_insn)
>  DEBUG_COUNTER (treepre_insert)
>  DEBUG_COUNTER (tree_sra)
> +DEBUG_COUNTER (eipa_sra)
>  DEBUG_COUNTER (sched2_func)
>  DEBUG_COUNTER (sched_block)
>  DEBUG_COUNTER (sched_func)
> Index: mine/gcc/tree-sra.c
> ===================================================================
> --- mine.orig/gcc/tree-sra.c
> +++ mine/gcc/tree-sra.c
> @@ -4451,7 +4451,7 @@ ipa_early_sra (void)
>  static bool
>  ipa_early_sra_gate (void)
>  {
> -  return flag_ipa_sra;
> +  return flag_ipa_sra && dbg_cnt (eipa_sra);
>  }
>
>  struct gimple_opt_pass pass_early_ipa_sra =
>

Index: mine/gcc/dbgcnt.def
===================================================================
--- mine.orig/gcc/dbgcnt.def
+++ mine/gcc/dbgcnt.def
@@ -169,6 +169,7 @@  DEBUG_COUNTER (pre)
 DEBUG_COUNTER (pre_insn)
 DEBUG_COUNTER (treepre_insert)
 DEBUG_COUNTER (tree_sra)
+DEBUG_COUNTER (eipa_sra)
 DEBUG_COUNTER (sched2_func)
 DEBUG_COUNTER (sched_block)
 DEBUG_COUNTER (sched_func)
Index: mine/gcc/tree-sra.c
===================================================================
--- mine.orig/gcc/tree-sra.c
+++ mine/gcc/tree-sra.c
@@ -4451,7 +4451,7 @@  ipa_early_sra (void)
 static bool
 ipa_early_sra_gate (void)
 {
-  return flag_ipa_sra;
+  return flag_ipa_sra && dbg_cnt (eipa_sra);
 }
 
 struct gimple_opt_pass pass_early_ipa_sra =