r276361 - Reverting r275115 which caused PR28634.

Pieb, Wolfgang via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 25 10:02:35 PDT 2016


Yes, it should. I'm just now realizing that the original change got into 3.9.

Thanks,
Wolfgang

> -----Original Message-----
> From: hwennborg at google.com [mailto:hwennborg at google.com] On Behalf Of
> Hans Wennborg
> Sent: Monday, July 25, 2016 9:56 AM
> To: Pieb, Wolfgang
> Cc: cfe-commits
> Subject: Re: r276361 - Reverting r275115 which caused PR28634.
> 
> Should this revert be merged to 3.9?
> 
> Thanks,
> Hans
> 
> On Thu, Jul 21, 2016 at 4:28 PM, Wolfgang Pieb via cfe-commits <cfe-
> commits at lists.llvm.org> wrote:
> > Author: wolfgangp
> > Date: Thu Jul 21 18:28:18 2016
> > New Revision: 276361
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=276361&view=rev
> > Log:
> > Reverting r275115 which caused PR28634.
> > When empty (forwarding) basic blocks that are referenced by user
> > labels are removed, incorrect code may be generated.
> >
> >
> > Removed:
> >     cfe/trunk/test/CodeGen/forwarding-blocks-if.c
> > Modified:
> >     cfe/trunk/lib/CodeGen/CGStmt.cpp


More information about the cfe-commits mailing list