From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 31197 invoked by alias); 10 Jan 2006 14:20:28 -0000 Received: (qmail 31190 invoked by uid 22791); 10 Jan 2006 14:20:27 -0000 X-Spam-Check-By: sourceware.org Received: from mx1.redhat.com (HELO mx1.redhat.com) (66.187.233.31) by sourceware.org (qpsmtpd/0.31) with ESMTP; Tue, 10 Jan 2006 14:20:25 +0000 Received: from int-mx1.corp.redhat.com (int-mx1.corp.redhat.com [172.16.52.254]) by mx1.redhat.com (8.12.11/8.12.11) with ESMTP id k0AEKM47003881; Tue, 10 Jan 2006 09:20:22 -0500 Received: from pobox.toronto.redhat.com (pobox.toronto.redhat.com [172.16.14.4]) by int-mx1.corp.redhat.com (8.11.6/8.11.6) with ESMTP id k0AEKM101385; Tue, 10 Jan 2006 09:20:22 -0500 Received: from vpn50-92.rdu.redhat.com (vpn50-92.rdu.redhat.com [172.16.50.92]) by pobox.toronto.redhat.com (8.12.8/8.12.8) with ESMTP id k0AEKLS1013949; Tue, 10 Jan 2006 09:20:21 -0500 From: Diego Novillo To: Zdenek Dvorak Subject: Re: Patch ping Date: Tue, 10 Jan 2006 14:20:00 -0000 User-Agent: KMail/1.9.1 Cc: gcc-patches@gcc.gnu.org References: <20060110140351.GA24831@atrey.karlin.mff.cuni.cz> In-Reply-To: <20060110140351.GA24831@atrey.karlin.mff.cuni.cz> MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Content-Disposition: inline Message-Id: <200601100920.19969.dnovillo@redhat.com> X-IsSubscribed: yes Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org X-SW-Source: 2006-01/txt/msg00530.txt.bz2 On Tuesday 10 January 2006 09:03, Zdenek Dvorak wrote: > Hello, > > http://gcc.gnu.org/ml/gcc-patches/2005-12/msg00670.html > -- a patch to make dumps in several loop optimization passes (ivopts, > vectorizer, ...) easier to read, by having scev & data dependence > analysis dumps disabled by default > Not completely OK. When I specify -details, I want every piece of information produced by the pass. It's fine with me if you want to add a specific -analysis switch that is orthogonal from -details. But with your patch, -details would not print everything in the scev and data dependence dumps. A simple tweak to this patch would give us both things. Make TDF_DETAILS include TDF_ANALYSIS in its bitmask. In fact, we could even have: #define TDF_DETAILS (1<<3) | TDF_ANALYSIS | TDF_STATS > http://gcc.gnu.org/ml/gcc-patches/2005-12/msg01820.html > -- an updated version of one of tree-ssa-operands.c > cleanup patches; it was approved by Diego except for one > suspicious place, which is changed in this version > OK.