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

Djordje Todorovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 28 07:08:23 PDT 2021


djtodoro accepted this revision.
djtodoro added inline comments.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/SelectionDAGISel.cpp:1663
     // sequence, so we return true in that case.
-    return MI.isDebugValue();
+    return MI.isDebugInstr();
 
----------------
Orlando wrote:
> 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?
All good. I agree with Jeremy that other meta instructions shouldn't affect this piece of code.


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

https://reviews.llvm.org/D106660



More information about the llvm-commits mailing list