[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