[PATCH] D67684: MachineScheduler: Fix missing dependency with multiple subreg defs

Quentin Colombet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 19 16:22:10 PDT 2019


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

LGTM



================
Comment at: lib/CodeGen/ScheduleDAGInstrs.cpp:406
+      // register in later operands. The lanes of other defs will now be live
+      // after this instruction, so these should not be treated as killd by the
+      // instruction even though they appear to be killed in this one operand.
----------------
typo: killd => killed


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

https://reviews.llvm.org/D67684





More information about the llvm-commits mailing list