[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