[PATCH] D44312: Retain both sets of debug intrinsics in HoistThenElseCodeToIf (fixes PR 36410)

Adrian Prantl via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 12 09:12:03 PDT 2018


aprantl added inline comments.


================
Comment at: lib/Transforms/Utils/SimplifyCFG.cpp:1295
+      assert (isa<DbgInfoIntrinsic>(I1) && isa<DbgInfoIntrinsic>(I2));
+      // For a debug info intrinsic, merging the debug locations can result
+      // in invalid IR (since the location must match the location of the
----------------
Maybe formulate this even stronger:
"The debug location is an integral part of a debug info intrinsic and can't be separated from it or replaced."


https://reviews.llvm.org/D44312





More information about the llvm-commits mailing list