[PATCH] D150381: [AMDGPU] Add optional tied-op for wwm-register's epilog spill restore

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 15 11:49:58 PDT 2023


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

LGTM with nit



================
Comment at: llvm/lib/Target/AMDGPU/SIRegisterInfo.cpp:1669
+    // so that it marks a usage for the preceding COPY.
+    if (!IsStore && MBB.isReturnBlock() && MI->isReturn() &&
+        MI->readsRegister(SubReg, this)) {
----------------
MBB.isReturnBlock is redundant with MI->isReturn. isReturnBlock just checks if the last instruction is a return 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150381



More information about the llvm-commits mailing list