[PATCH] D31993: [AMDGPU] Combine DS operations with offsets bigger than byte

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 13 10:42:28 PDT 2017


rampitec marked an inline comment as done.
rampitec added inline comments.


================
Comment at: lib/Target/AMDGPU/SILoadStoreOptimizer.cpp:78
+    unsigned Offset1;
+    unsigned BaseOff;
+    bool UseST64;
----------------
vpykhtin wrote:
> Why 3 offsets?
Two offsets as it will be encoded into an LDS instruction, and then base offset which needs to be added with v_add_i32 to the pointer if non zero.


Repository:
  rL LLVM

https://reviews.llvm.org/D31993





More information about the llvm-commits mailing list