[PATCH] D129208: [MachineVerifier] Try harder to verify LiveIntervals

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 24 02:47:28 PDT 2023


foad added a subscriber: kparzysz.
foad added a comment.

Alternatively I could just remove this case from `test/CodeGen/AMDGPU/subreg-intervals.mir`:

  ---
  name: test0
  registers:
    - { id: 0, class: sreg_64 }
  body: |
    bb.0:
      S_NOP 0, implicit-def %0
      S_NOP 0, implicit %0
  
      S_NOP 0, implicit-def undef %0.sub0
      S_NOP 0, implicit %0
  ...

I have gone back to the original commit of D21189 <https://reviews.llvm.org/D21189> and confirmed that this test case passes both before and after that commit, so it wasn't testing any new functionality. @kparzysz would you be happy with that?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129208



More information about the llvm-commits mailing list