[PATCH] D42885: [AMDGPU] intrintrics for byte/short load/store

[Matt Arsenault via Phabricator via llvm-commits]

arsenm requested changes to this revision.
arsenm added a comment.
This revision now requires changes to proceed.

I don't think we need intrinsics for these. At most we should add a mangled type to the existing buffer intrinsics.



================
Comment at: include/llvm/IR/IntrinsicsAMDGPU.td:494
+def int_amdgcn_buffer_load_ubyte : Intrinsic <
+  [llvm_float_ty],
+  [llvm_v4i32_ty,     // rsrc(SGPR)
----------------
float return type doesn't make sense


Repository:
  rL LLVM

https://reviews.llvm.org/D42885