From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 6840 invoked by alias); 1 Jun 2011 08:51:26 -0000 Received: (qmail 6830 invoked by uid 22791); 1 Jun 2011 08:51:25 -0000 X-SWARE-Spam-Status: No, hits=-2.3 required=5.0 tests=AWL,BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FROM,RCVD_IN_DNSWL_LOW,RFC_ABUSE_POST X-Spam-Check-By: sourceware.org Received: from mail-wy0-f175.google.com (HELO mail-wy0-f175.google.com) (74.125.82.175) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Wed, 01 Jun 2011 08:51:10 +0000 Received: by wye20 with SMTP id 20so4627437wye.20 for ; Wed, 01 Jun 2011 01:51:09 -0700 (PDT) MIME-Version: 1.0 Received: by 10.227.62.207 with SMTP id y15mr6923800wbh.86.1306918269197; Wed, 01 Jun 2011 01:51:09 -0700 (PDT) Received: by 10.227.37.152 with HTTP; Wed, 1 Jun 2011 01:51:09 -0700 (PDT) In-Reply-To: References: Date: Wed, 01 Jun 2011 08:51:00 -0000 Message-ID: Subject: Re: -fdump-passes -fenable-xxx=func_name_list From: Richard Guenther To: Xinliang David Li Cc: GCC Patches Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable X-IsSubscribed: yes 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 X-SW-Source: 2011-06/txt/msg00012.txt.bz2 On Wed, Jun 1, 2011 at 1:34 AM, Xinliang David Li wrot= e: > The following patch implements the a new option that dumps gcc PASS > configuration. The sample output is attached. =A0There is one > limitation: some placeholder passes that are named with '*xxx' are > note registered thus they are not listed. They are not important as > they can not be turned on/off anyway. > > The patch also enhanced -fenable-xxx and -fdisable-xx to allow a list > of function assembler names to be specified. > > Ok for trunk? Please split the patch. I'm not too happy how you dump the pass configuration. Why not simply, at a _single_ place, walk the pass tree? Instead of doing pieces of it at pass execution time when it's not already dumped - that really looks gross. The documentation should also link this option to the -fenable/disable options as obviously the pass names in that dump are those to be used for those flags (and not readily available anywhere else). I also think that it would be way more useful to note in the individual dump files the functions (at the place they would usually appear) that have the pass explicitly enabled/disabled. Richard. > Thanks, > > David >