[PATCH] D53750: [AMDGPU] support image load/store a16

Nicolai Hähnle via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Oct 27 08:30:35 PDT 2018


nhaehnle accepted this revision.
nhaehnle added a comment.
This revision is now accepted and ready to land.

The function names of the a16.d16 tests should probably use vNf16 instead of vNf32. Please change that before committing, and please add a test for 2darraymsaa, to cover the v4i16 case of coordinates.

Apart from that LGTM.


Repository:
  rL LLVM

https://reviews.llvm.org/D53750





More information about the llvm-commits mailing list