[PATCH] Merge empty landing pads in SimplifyCFG

Andy Kaylor andrew.kaylor at intel.com
Thu Mar 12 12:09:44 PDT 2015


LGTM (with a minor question about debug info intrinsics)


================
Comment at: lib/Transforms/Utils/SimplifyCFG.cpp:4462
@@ +4461,3 @@
+  if (LandingPadInst *LPad = dyn_cast<LandingPadInst>(I)) {
+    for (++I; isa<DbgInfoIntrinsic>(I); ++I) {}
+    if (I->isTerminator() &&
----------------
If I'm reading this right these debug intrinsics just get thrown away if a merge is performed.  Do they not contain anything that might be different from the corresponding debug intrinsics in the potential merge block?

http://reviews.llvm.org/D8297

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list