[PATCH] D42551: [Debug] Add dbg.value intrinsics for PHIs created during LCSSA.

[Vedant Kumar via Phabricator via llvm-commits]

vsk added inline comments.


================
Comment at: lib/Transforms/Utils/Local.cpp:1371
+    if (isa<CatchSwitchInst>(*Parent->getFirstNonPHI()))
+      continue;
+    auto PhiMAV = MetadataAsValue::get(C, ValueAsMetadata::get(PHI));
----------------
Would isEHPad work better here (http://llvm.org/doxygen/classllvm_1_1Instruction.html#a5d3d315f678bc76e43b27d18e5d72829)?


================
Comment at: test/Transforms/LCSSA/avoid-intrinsics-in-catchswitch.ll:36
+  %0 = bitcast %struct.e* %n to i8*
+  call void @llvm.lifetime.start.p0i8(i64 4, i8* nonnull %0)
+  %1 = bitcast i32* %db to i8*
----------------
Does the bug reproduce with the lifetime intrinsics stripped out? That might be a way to reduce the regression test a bit.


https://reviews.llvm.org/D42551