From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 27664 invoked by alias); 1 Jun 2011 21:12:21 -0000 Received: (qmail 27656 invoked by uid 22791); 1 Jun 2011 21:12:21 -0000 X-SWARE-Spam-Status: No, hits=-0.8 required=5.0 tests=AWL,BAYES_00,RDNS_DYNAMIC,TVD_RCVD_IP X-Spam-Check-By: sourceware.org Received: from 195-14-0-142.nuxit.net (HELO de558.ispfr.net) (195.14.0.142) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Wed, 01 Jun 2011 21:12:07 +0000 Received: from ours.starynkevitch.net ([213.41.244.95] helo=glinka.lesours) by de558.ispfr.net with smtp (Exim 4.72) (envelope-from ) id 1QRshv-0001Nk-K3; Wed, 01 Jun 2011 23:12:03 +0200 Date: Wed, 01 Jun 2011 21:12:00 -0000 From: Basile Starynkevitch To: Xinliang David Li Cc: GCC Patches Subject: Re: Dump before flag Message-Id: <20110601231202.224188ad.basile@starynkevitch.net> In-Reply-To: References: Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit 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/msg00107.txt.bz2 On Wed, 1 Jun 2011 13:26:24 -0700 Xinliang David Li wrote: > Hi, this is a simple patch that support dump_before flag. E.g, > > -fdump-tree-pre-before > > This is useful for diffing the the IR before and after a pass. Perhaps you forgot to actually attach the patch? > Gcc dumping needs more cleanups -- such as allowing IR only dump, > allowing IR dumping for a particular function etc. The exposure of > 'dumpfile' (instead of a dumping_level () function) makes those change > a little messy, but can be done. I don't understand what you mean by a dumping_level () function. What should that hypothetical function do? (I'm wrongly guessing it would return an integer, but IIRC dumpfile is a FILE*) Regards -- Basile STARYNKEVITCH http://starynkevitch.net/Basile/ email: basilestarynkevitchnet mobile: +33 6 8501 2359 8, rue de la Faiencerie, 92340 Bourg La Reine, France *** opinions {are only mine, sont seulement les miennes} ***