[PATCH] D106660: [DebugInfo][InstrRef] Don't break up return-sequences on debug-info instructions

[Orlando Cazalet-Hyams via Phabricator via llvm-commits]

Orlando accepted this revision.
Orlando added a comment.
This revision is now accepted and ready to land.

LGTM with the new test which appears to be the only previously outstanding request.



================
Comment at: llvm/lib/CodeGen/SelectionDAG/SelectionDAGISel.cpp:1663
     // sequence, so we return true in that case.
-    return MI.isDebugValue();
+    return MI.isDebugInstr();
 
----------------
jmorse wrote:
> djtodoro wrote:
> > Should it be `isMetaInstruction()` ?
> The implications of this are unclear to me -- the "KILL" instruction for example manually edits register liveness information, and I'm not sure where in a termination sequence it would belong. IMO, best to keep the change so as small an amount of behaviours as possible, hence picking isDebugInstr.
SGTM. I just want to check that @djtodoro is happy with this too?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D106660/new/

https://reviews.llvm.org/D106660