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

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added a comment.

Couldn't we also optimize the loads at least based on used bits like a normal load?

In https://reviews.llvm.org/D42885#997648, @timcorringham wrote:

> Matt, we do actually need these intrinsics as we have an urgent requirement for them Open Vulkan (which is of course my motivation for implementing them).
>
> As Tim commented, the load ubyte and load short instructions extend to 32 bits. While float is a little odd, it does match the behavior of the other buffer_load instructions. Also I think changing it would require a disproportionate amount of effort.


Yes, they do but the intrinsic doesn't need to care.


Repository:
  rL LLVM

https://reviews.llvm.org/D42885