[PATCH] D127756: [AMDGPU] gfx11 VINTERP intrinsics and ISel support

[Ivan Kosarev via Phabricator via llvm-commits]

kosarev added inline comments.
Herald added a subscriber: StephenFan.


================
Comment at: llvm/include/llvm/IR/IntrinsicsAMDGPU.td:1524
+  Intrinsic<[llvm_float_ty],
+            [llvm_float_ty, llvm_float_ty, llvm_float_ty, llvm_i1_ty],
+            [IntrNoMem, IntrSpeculatable, IntrWillReturn,
----------------
Wouldn't it be more natural to declare `p` and `p0` to be `llvm_v2f16_ty`?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D127756/new/

https://reviews.llvm.org/D127756