Yes, it should. I'm just now realizing that the original change got into 3.9.
Thanks, Wolfgang > -----Original Message----- > From: hwennb...@google.com [mailto:hwennb...@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- > comm...@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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits