[llvm] [X86][AVX-VNNI] Fix VNNI intrinsics argument types (PR #122649)

via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 13 17:46:31 PST 2025


================
@@ -1867,29 +1867,29 @@ let TargetPrefix = "x86" in {  // All intrinsics start with "llvm.x86.".
 let TargetPrefix = "x86" in {  // All intrinsics start with "llvm.x86.".
   def int_x86_avx512_vpdpbusd_128 :
       ClangBuiltin<"__builtin_ia32_vpdpbusd128">,
-      DefaultAttrsIntrinsic<[llvm_v4i32_ty], [llvm_v4i32_ty, llvm_v4i32_ty,
-                             llvm_v4i32_ty], [IntrNoMem]>;
+      DefaultAttrsIntrinsic<[llvm_v4i32_ty], [llvm_v4i32_ty, llvm_v16i8_ty,
+                             llvm_v16i8_ty], [IntrNoMem]>;
----------------
BaiXilin wrote:

Hi Phoebe, thank you for the feedbacks! By front end tests, do you mean `check-clang`? I updated the header files, but don't see failing tests in `check-clang`.

In addition, although the correct assembly is currently generated, is there a need to update `X86InstrSSE.td` given `vpdpb*` intrinsic argument types have been updated?

https://github.com/llvm/llvm-project/pull/122649


More information about the llvm-commits mailing list