Great catch! Is there a test case we can add for this? Anders
On Dec 2, 2009, at 3:28 PM, Mike Stump wrote: > Author: mrs > Date: Wed Dec 2 17:28:08 2009 > New Revision: 90356 > > URL: http://llvm.org/viewvc/llvm-project?rev=90356&view=rev > Log: > Add missing branch to exit. Seemingly obvious when I look at the > code, but to track this down was laborious. > > Modified: > cfe/trunk/lib/CodeGen/CGDecl.cpp > > Modified: cfe/trunk/lib/CodeGen/CGDecl.cpp > URL: > http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/CGDecl.cpp?rev=90356&r1=90355&r2=90356&view=diff > > ============================================================================== > --- cfe/trunk/lib/CodeGen/CGDecl.cpp (original) > +++ cfe/trunk/lib/CodeGen/CGDecl.cpp Wed Dec 2 17:28:08 2009 > @@ -534,6 +534,9 @@ > } else { > DelayedCleanupBlock Scope(*this); > EmitCXXDestructorCall(D, Dtor_Complete, DeclPtr); > + > + // Make sure to jump to the exit block. > + EmitBranch(Scope.getCleanupExitBlock()); > } > } > } > > > _______________________________________________ > cfe-commits mailing list > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
