[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