[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