[PATCH] D32241: Don't process debug intrinsics in InstCombine

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 20 23:11:27 PDT 2017


craig.topper added inline comments.


================
Comment at: lib/Transforms/InstCombine/InstructionCombining.cpp:3074-3075
 
-      InstrsForInstCombineWorklist.push_back(Inst);
+      // Skip processing intrinsics in InstCombine
+      if (!isa<DbgInfoIntrinsic>(Inst))
+        InstrsForInstCombineWorklist.push_back(Inst);
----------------
davide wrote:
> Please expand this comment to explain why we want to skip them.
I'm really sorry to keep asking this to be moved, but is there any reason to run any of the code above here for debug intrinsics?


https://reviews.llvm.org/D32241





More information about the llvm-commits mailing list