[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