From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 11022 invoked by alias); 17 Oct 2008 17:00:22 -0000 Received: (qmail 11014 invoked by uid 22791); 17 Oct 2008 17:00:21 -0000 X-Spam-Check-By: sourceware.org Received: from mail-gx0-f17.google.com (HELO mail-gx0-f17.google.com) (209.85.217.17) by sourceware.org (qpsmtpd/0.31) with ESMTP; Fri, 17 Oct 2008 16:59:46 +0000 Received: by gxk10 with SMTP id 10so1363813gxk.8 for ; Fri, 17 Oct 2008 09:59:43 -0700 (PDT) Received: by 10.90.89.18 with SMTP id m18mr4852554agb.73.1224262783331; Fri, 17 Oct 2008 09:59:43 -0700 (PDT) Received: by 10.90.49.12 with HTTP; Fri, 17 Oct 2008 09:59:43 -0700 (PDT) Message-ID: <4aca3dc20810170959q6f924cdene0340b1d267468ce@mail.gmail.com> Date: Fri, 17 Oct 2008 17:56:00 -0000 From: "Daniel Berlin" To: "Eric Botcazou" Subject: Re: [PATCH]: Fix missed PRE optimization discovered Cc: "Andreas Schwab" , gcc-patches@gcc.gnu.org In-Reply-To: <4aca3dc20810170814s771687dcv64f7f45ce8ad8229@mail.gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <4aca3dc20810161526r14037c7dq28f8fb2b4c7a679c@mail.gmail.com> <200810171259.02324.ebotcazou@adacore.com> <4aca3dc20810170733g6d04f968s95744ae36de2ad86@mail.gmail.com> <4aca3dc20810170814s771687dcv64f7f45ce8ad8229@mail.gmail.com> 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: 2008-10/txt/msg00757.txt.bz2 On Fri, Oct 17, 2008 at 11:14 AM, Daniel Berlin wrote: > On Fri, Oct 17, 2008 at 10:33 AM, Daniel Berlin wrote: >> On Fri, Oct 17, 2008 at 6:59 AM, Eric Botcazou wrote: >>>> This breaks Ada on ia64: >>> >>> This breaks Ada everywhere. >>> >> >> I'll take a look this morning. >> There is no good reason this should break anything. >> > > Doesn't break on x86_64-linux-gnu for me (ubuntu hardy), trying > i686-pc-linux-gnu. > Okay, i reproduced this on i686-pc-linux-gnu. It looks like we forget to add a constant to the expression set somewhere (which is a latent bug). Fix coming soon.