[PATCH] D115360: [MachineVerifier] Undef subreg operands do not require subranges

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 8 14:58:16 PST 2021


arsenm added inline comments.


================
Comment at: llvm/lib/CodeGen/MachineVerifier.cpp:2232
       LI = &LiveInts->getInterval(Reg);
-      if (SubRegIdx != 0 && !LI->empty() && !LI->hasSubRanges() &&
-          MRI->shouldTrackSubRegLiveness(Reg))
+      if (SubRegIdx != 0 && !MO->isUndef() && !LI->empty() &&
+          !LI->hasSubRanges() && MRI->shouldTrackSubRegLiveness(Reg))
----------------
undef subreg defs do read the register (i.e. see the below readsReg check)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115360



More information about the llvm-commits mailing list