[PATCH] D109302: [PowerPC] Check subregister defines before removing XXMFACC/XXMTACC

Thu Sep 29 20:00:36 PDT 2022

shchenz added a comment.

Do we plan to move forward @qiucf

Deleting the instruction seems can not work for all cases, for example, `%r = and 0, undef`, this is well defined.

IMO, can we just change the input case? For example, don't store `undef`(element 2) to memory, we can extract and store element 3, maybe? Storing undef should not be intended by the user and the machineverify failure can exactly tell the user there is something wrong in the source or in compiler optimizations?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109302