rnk added inline comments.
================
Comment at: lib/CodeGen/CGCleanup.cpp:458
InsertBefore = Invoke->getNormalDest()->getFirstInsertionPt();
+ else if (isa<llvm::AllocaInst>(Inst))
+ InsertBefore = std::next(AllocaInsertPt->getIterator());
----------------
This doesn't seem right, `Inst` could be a dynamic alloca. If it's static, we
definitely don't need to store and reload it. All static allocas better be in
the entry block... You might want to use `isStaticAlloca`, but that still feels
like we're hacking around some deeper problem.
https://reviews.llvm.org/D33663
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
