[PATCH] D15139: [IR] Reformulate LLVM's EH funclet IR
Reid Kleckner via llvm-commits
llvm-commits at lists.llvm.org
Fri Dec 11 10:10:58 PST 2015
rnk added inline comments.
================
Comment at: lib/Transforms/Utils/SimplifyCFG.cpp:3284
@@ +3283,3 @@
+ // DestParentFunclet is an ancestor of CPInst.
+ if (CPInst->getOuterScope() != DestParentFunclet)
+ return false;
----------------
I made this change and I think I completely misunderstood what the code was trying to do. I thought it was trying to merge consecutive cleanups together, not just remove empty cleanups. We can probably revert this.
http://reviews.llvm.org/D15139
More information about the llvm-commits
mailing list