[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