[PATCH] D38906: AMDGPU/SI: Implement d16 support for buffer intrinsics

[Nicolai Hähnle via Phabricator via llvm-commits]

nhaehnle added a comment.

In https://reviews.llvm.org/D38906#922226, @b-sumner wrote:

> Pardon my ignorance, but why isn't include/llvm/IR/IntrinsicsAMDGPU.td being updated?


The intrinsics use overloading based on the type, so no new intrinsics need to be defined.



================
Comment at: lib/Target/AMDGPU/BUFInstructions.td:1464-1475
+let SubtargetPredicate = HasUnpackedD16VMem in {
+defm : MTBUF_LoadIntrinsicPat<SItbuffer_load, f16,   "TBUFFER_LOAD_FORMAT_D16_X_gfx80">;
+defm : MTBUF_LoadIntrinsicPat<SItbuffer_load_d16, v2i32, "TBUFFER_LOAD_FORMAT_D16_XY_gfx80">;
+defm : MTBUF_LoadIntrinsicPat<SItbuffer_load_d16, v4i32, "TBUFFER_LOAD_FORMAT_D16_XYZW_gfx80">;
+} // End HasUnpackedD16VMem.
+
+let SubtargetPredicate = HasPackedD16VMem in {
----------------
Are the pattens with SItbuffer_load and 16-bit types ever used? I believe the f16 maybe, but isn't the v2f16 getting replaced by ReplaceNodeResults?



https://reviews.llvm.org/D38906