[PATCH] D88716: [TwoAddressInstruction][PowerPC] Call `regOverlapsSet` to find out real clobbers and uses

Nemanja Ivanovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 7 05:24:30 PDT 2020


nemanjai accepted this revision.
nemanjai added a comment.
This revision is now accepted and ready to land.

LGTM. Checking for overlap rather than simply the same register should be the right thing to do. However, I wonder if we might have other places where a similar assumption is made that could become a problem since we presumably only mark the subregister as an implicit def.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88716



More information about the llvm-commits mailing list