This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: ifcvt/crossjump patch: Fix PR 42496, 21803
- From: Bernd Schmidt <bernds at codesourcery dot com>
- To: Jeff Law <law at redhat dot com>
- Cc: Eric Botcazou <ebotcazou at adacore dot com>, gcc-patches at gcc dot gnu dot org, Steven Bosscher <stevenb dot gcc at gmail dot com>, Jim Wilson <wilson at codesourcery dot com>
- Date: Thu, 23 Sep 2010 12:07:51 +0200
- Subject: Re: ifcvt/crossjump patch: Fix PR 42496, 21803
- References: <4BB3CCCA.7000600@codesourcery.com> <201004101235.54302.ebotcazou@adacore.com> <4BC62EB2.40609@codesourcery.com> <201004200005.53901.ebotcazou@adacore.com> <4C460A1F.4000509@codesourcery.com> <4C56EABB.4030401@redhat.com> <4C56EB55.7010408@codesourcery.com> <4C56ECB1.9060302@redhat.com> <4C56EF21.1060801@codesourcery.com> <4C58232D.8090805@codesourcery.com> <4C58329F.6010902@redhat.com> <4C583639.9070704@codesourcery.com> <4C584E0E.2040408@redhat.com> <4C596CA5.2060207@codesourcery.com> <4C7BCCFA.90409@codesourcery.com> <4C97316B.6090902@codesourcery.com> <4C97843D.7070104@redhat.com>
On 09/20/2010 05:56 PM, Jeff Law wrote:
> On 09/20/10 04:03, Bernd Schmidt wrote:
>> On 08/30/2010 05:23 PM, Bernd Schmidt wrote:
>>> On 08/04/2010 03:35 PM, Bernd Schmidt wrote:
>>>> On 08/03/2010 07:12 PM, Jeff Law wrote:
>>>>> OK. WRT the comment, we might want to just say that BB_MODIFIED is
>>>>> set
>>>>> at the same time as a block is marked dirty, but is not cleared
>>>>> during a
>>>>> df_analyze allowing a pass to update the DF information and still know
>>>>> what blocks were modified.
>>>>
>>>> New patch below.
>>
>> Ping^2.
>> http://gcc.gnu.org/ml/gcc-patches/2010-08/msg00280.html
> Sorry, I thought this had been approved long ago. Approved :-)
Thanks. Committed with a small fix found while testing on x86_64: In
try_head_merge_bb, in the case where we try to move across multiple
blocks to optimize for a switch statement, the final destination block
must be the only predecessor of the block we're looking at.
Bernd