[PATCH] D140537: SIInsertWait: Skip tied source of d16 buffer instruction

[Ruiling, Song via Phabricator via llvm-commits]

ruiling added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/SIInsertWaitcnts.cpp:1181
+
+        // D16 buffer instructions does not actually read the tied source
+        // operand, so we can skip the source operand.
----------------
foad wrote:
> Should we do the same thing for _d16 DS, FLAT, SCRATCH and GLOBAL instructions?
For _d16 DS/FLAT, skipping the tied source does not change the code generation. As we always need a s_waitcnt for two successive _d16 ds/flat load because they may return out of order. So, I would rather not handle them here. I am actually setting the D16Buf bit for global/scratch load in parent change.  Should I change `D16Buf` to something else? I will update the comment.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140537