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] Missing barrier after removing empty basic block

From: Andreas Krebbel <krebbel at linux dot vnet dot ibm dot com>
To: Richard Henderson <rth at redhat dot com>
Cc: gcc-patches at gcc dot gnu dot org
Date: Thu, 17 Sep 2009 11:11:21 +0200
Subject: Re: [PATCH] Missing barrier after removing empty basic block
References: <20090916102915.GA8346@bart> <4AB10AF1.5060608@redhat.com>

Richard Henderson wrote:

On 09/16/2009 05:29 AM, Andreas Krebbel wrote:
    * cfglayout.c (fixup_reorder_chain): Accept conditional jumps
    without a fallthrough edge.
Ok.

Thanks! I've committed the patch without the unlink_insn_chain invokation since the header/footer insns at that point are already linked back into the insn stream. It happens just a few lines above the lines I've changed.

+ bb->il.rtl->footer = unlink_insn_chain (barrier, barrier);

Bye,

-Andreas-

References:
- [PATCH] Missing barrier after removing empty basic block
  - From: Andreas Krebbel
- Re: [PATCH] Missing barrier after removing empty basic block
  - From: Richard Henderson

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