[PATCH] D72737: [AMDGPU] Bundle loads before post-RA scheduler

Valery Pykhtin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 21 09:13:48 PST 2020


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

LGTM



================
Comment at: llvm/lib/Target/AMDGPU/SIMemoryLegalizer.cpp:1300
+            if (MO.isReg())
+              MO.setIsInternalRead(false);
+        }
----------------
just out of curiosity: why is this needed?


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

https://reviews.llvm.org/D72737





More information about the llvm-commits mailing list