[PATCH] D75789: [AMDGPU] Allow struct.buffer.*.format intrinsics to accept i32
Carl Ritson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Mar 6 21:59:45 PST 2020
critson added a comment.
In D75789#1910835 <https://reviews.llvm.org/D75789#1910835>, @arsenm wrote:
> Needs d16 tests, and needs GlobalISel tests. I believe there was a reason format buffer intrinsics needed to be assumed to be FP, but maybe it doesn't matter
I have added d16 tests. This already includes GlobalISel tests, unless I missed something?
I am hoping Tim Renouf (@tpr) can comment why the original intrinsics assumed FP.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D75789/new/
https://reviews.llvm.org/D75789
More information about the llvm-commits
mailing list