[PATCH] D99352: [AMDGPU] ds_read_*/ds_write_* operations require strict alignment.

[Jay Foad via Phabricator via llvm-commits]

foad added a comment.

In D99352#2653253 <https://reviews.llvm.org/D99352#2653253>, @rampitec wrote:

> @foad can we run it trough gfx perf suite? With exception of one ds_write_b64 store it seems reasonable to me. Of course these splits to b8 would be unfortunate but I assume it shall not happen in real life too often and the data is usually better aligned.

Vulkan graphics rarely uses DS instructions. Vulkan compute uses them, but we don't have any standard set of benchmarks, and in any case the workloads would be very similar to OpenCL compute. So I don't think we need to do any Vulkan-specific benchmarking.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D99352/new/

https://reviews.llvm.org/D99352