[PATCH] D89855: [PowerPC] Extend folding RLWINM + RLWINM to post-RA.

Qing Shan Zhang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 29 17:51:25 PDT 2020


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

LGTM overall with one minor comment.



================
Comment at: llvm/lib/Target/PowerPC/PPCInstrInfo.cpp:3355
+    // its flag is "killed", it's safe to delete SrcMI. Otherwise keep it.
+    if ((!MRI->isSSA() && NoUse) ||
+        (MRI->isSSA() && MRI->use_nodbg_empty(FoldingReg))) {
----------------
Can we set the NoUse for pre-ra also,so that just check the NoUse here?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89855



More information about the llvm-commits mailing list