[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