[PATCH] D44401: [AMDGPU] Always use IDX for load/store format intrinsics.
Marek Olšák via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Mar 16 13:31:14 PDT 2018
mareko added a comment.
In https://reviews.llvm.org/D44401#1040175, @arsenm wrote:
> I think the unit change is a hint we should use different intrinsics for this
The current intrinsic is unusable on GFX9 though. Why should we keep it as-is if it's unusable? This patch is reasonable because it picks the most expected behavior for GFX9 while it disallows the least expected and problematic behavior.
Repository:
rL LLVM
https://reviews.llvm.org/D44401
More information about the llvm-commits
mailing list