[PATCH] D97054: [MachineVerifier] Confirm that both ends of a tied def/use are live together

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added a comment.

Testcase? I also think this won't handle subregister indexes correctly. How does the x86 test fail?



================
Comment at: llvm/lib/CodeGen/MachineVerifier.cpp:2724
     for (ConstMIBundleOperands MOI(*MI); MOI.isValid(); ++MOI) {
       if (!MOI->isReg() || !MOI->isDef())
         continue;
----------------
I'm surprised this isn't checking readsReg instead of isDef


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97054