[PATCH] D99352: [AMDGPU] ds_read_*/ds_write_* operations require strict alignment.
Mahesha S via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Mar 25 22:28:46 PDT 2021
hsmhsm added inline comments.
================
Comment at: llvm/lib/Target/AMDGPU/SIISelLowering.cpp:1435
+ // ds_read/write_b128 require 16-byte alignment on gfx8 and older.
+ bool Aligned = Alignment >= Align(16);
if (IsFast)
----------------
arsenm wrote:
> I don't like changing this function to lie and say it doesn't work when it does
My understanding is that the function allowsMisalignedMemoryAccessesImpl() should check for the strict alignment requirment for ds_read/ds_write operations. I am now further updated this function accordingly.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D99352/new/
https://reviews.llvm.org/D99352
More information about the llvm-commits
mailing list