[PATCH] D70676: [DebugInfo] Don't repeatedly created undef DBG_VALUEs during machine-sinking

[Mikael Holmén via Phabricator via llvm-commits]

uabelho added a comment.

Hi,

I've started seeing a verifier complaint with this patch.
I don't have a full testcase to share since I get this on our out-of-tree
target but maybe you can make somethng out of this anyway.

In the input to MachineSink we have this in one MBB

  %5:an32pairs = mv [...]
  %6:an32_rn_pairs = COPY %5.hiAcc:an32pairs
  %8:an32_0_7 = COPY %6:an32_rn_pairs
  DBG_VALUE %8:an32_0_7 [...]

and then in the output we get this, sinked into another MBB

  %6:an32_rn_pairs = COPY %5.hiAcc:an32pairs
  %8:an32_0_7 = COPY %6:an32_rn_pairs
  DBG_VALUE %8.hiAcc:an32_0_7 [...]

Then the verifier complains that there is no sub register hiAcc in the an32_0_7
register class, i.e. the "hiAcc" in the

  DBG_VALUE %8.hiAcc:an32_0_7

shouldn't be there.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70676