[PATCH] D45342: [DebugInfo] Examine all uses of isDebugValue() for debug instructions.

[Bjorn Pettersson via Phabricator via llvm-commits]

bjope added inline comments.


================
Comment at: llvm/trunk/lib/CodeGen/InlineSpiller.cpp:935
     // Debug values are not allowed to affect codegen.
-    if (MI->isDebugValue()) {
+    if (MI->isDebugInstr()) {
       // Modify DBG_VALUE now that the value is in a spill slot.
----------------
Isn't this is a little bit confusing?
Calling buildDbgValueForSpill is probably only correct for a DBG_VALUE. So the old check for MI->isDebugValue() is probably more correct (and more future proof).

And then perhaps there should be an assertion like this on line 943 instead:
```
assert(!MI->isDebugInstr() && "Did not expect to find a use in debug instruction that isn't a DBG_VALUE")
```



Repository:
  rL LLVM

https://reviews.llvm.org/D45342