This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH] Remove bogus assert from CCP's insert_clobbers_for_var

From: Martin Jambor <mjambor at suse dot cz>
To: Eric Botcazou <ebotcazou at adacore dot com>
Cc: gcc-patches at gcc dot gnu dot org
Date: Sat, 31 Mar 2012 09:17:17 +0200
Subject: Re: [PATCH] Remove bogus assert from CCP's insert_clobbers_for_var
References: <20120328183936.GA2817@virgil.arch.suse.de> <201203301323.01290.ebotcazou@adacore.com>

Hi,

On Fri, Mar 30, 2012 at 01:23:00PM +0200, Eric Botcazou wrote:
> > On IRC I've been told that is OK and the that CCP cannot make such
> > assumtions.  Since it is only a missed-optimization if the call to the
> > builtin is not found and processed (basically PR 51491 again but only
> > in cases like these), I thought it best to just remove the assert by
> > the following simple patch, bootstrapped and tested on x86_64-linux.
> 
> Please do that on the 4.7 branch as well if the assertion is incorrect.

Done.

Thanks,

Martin

> 
> -- 
> Eric Botcazou

Follow-Ups:
- Re: [PATCH] Remove bogus assert from CCP's insert_clobbers_for_var
  - From: Eric Botcazou

References:
- [PATCH] Remove bogus assert from CCP's insert_clobbers_for_var
  - From: Martin Jambor
- Re: [PATCH] Remove bogus assert from CCP's insert_clobbers_for_var
  - From: Eric Botcazou

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]