From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 5946 invoked by alias); 10 Jun 2004 14:28:58 -0000 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 Received: (qmail 5902 invoked from network); 10 Jun 2004 14:28:56 -0000 Received: from unknown (HELO dberlin.org) (69.3.5.6) by sourceware.org with SMTP; 10 Jun 2004 14:28:56 -0000 Received: from [192.168.1.7] (account dberlin HELO [192.168.1.7]) by dberlin.org (CommuniGate Pro SMTP 4.1.4) with ESMTP-TLS id 6760850; Thu, 10 Jun 2004 10:28:56 -0400 In-Reply-To: <1086876890.22854.177.camel@localhost.localdomain> References: <1C733CAA-B7D9-11D8-909E-000A95DA505C@dberlin.org> <1086873868.22854.163.camel@localhost.localdomain> <731EAB5E-BAE3-11D8-9651-000A95DA505C@dberlin.org> <1086876890.22854.177.camel@localhost.localdomain> Mime-Version: 1.0 (Apple Message framework v618) Content-Type: text/plain; charset=US-ASCII; format=flowed Message-Id: <8074F2B6-BAEA-11D8-9651-000A95DA505C@dberlin.org> Content-Transfer-Encoding: 7bit Cc: GCC Patches From: Daniel Berlin Subject: Re: [PATCH]: Add annotations for expression and constants Date: Thu, 10 Jun 2004 15:38:00 -0000 To: Diego Novillo X-SW-Source: 2004-06/txt/msg00638.txt.bz2 On Jun 10, 2004, at 10:14 AM, Diego Novillo wrote: > On Thu, 2004-06-10 at 09:38, Daniel Berlin wrote: > >> I'd have to create a new union or void *ify it, since it's currently a >> union tree_ann_d *, not a void *. But i can't easily void * it >> because it's GC'd. >> > OK. Creating the annotations is easier then. BTW, are you also going > to number naked decls to commonize multiple loads from global > variables, > for instance? Yes, as part of load value numbering (which is going to be a followup, so i can get this stuff in as a starting point). > If so, the hash value will also be needed in var_ann_d. > Yup. > I'm thinking if we shouldn't just put the hash value in > tree_common_ann_d directly. > Yeah. I'll do that in the GVN-PRE patch (which actually adds the annotations) > > Diego. >