This patch implements the function prototypes vec_mulh and vec_dive in order to utilize
the vector multiply high (vmulh[s|u][w|d]) and vector divide extended (vdive[s|u][w|d])
instructions introduced in Power10.
Depends on D82576.
Paths
| Differential D82609
[PowerPC] Implement Vector Multiply High/Divide Extended Builtins in LLVM/Clang ClosedPublic Authored by amyk on Jun 25 2020, 4:12 PM.
Details
Summary This patch implements the function prototypes vec_mulh and vec_dive in order to utilize Depends on D82576.
Diff Detail
Event Timelineanil9 added inline comments.
amyk added inline comments. amyk added a parent revision: D82576: [PowerPC][Power10] Implement low-order Vector Modulus Builtins, and add Vector Multiply/Divide/Modulus Builtins Tests. Comment ActionsRebase patch, remove MC tests from this patch. amyk added inline comments. amyk retitled this revision from [PowerPC][Power10] Implement Vector Multiply High/Divide Extended Builtins in LLVM/Clang to [PowerPC] Implement Vector Multiply High/Divide Extended Builtins in LLVM/Clang.Jul 27 2020, 3:43 PM Comment Actions Addressed the following comments:
This revision is now accepted and ready to land.Aug 26 2020, 1:32 PM This revision was landed with ongoing or failed builds.Aug 26 2020, 9:14 PM Closed by commit rG76b0f99ea854: [PowerPC] Implement Vector Multiply High/Divide Extended Builtins in LLVM/Clang (authored by amyk). · Explain Why This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 288177 clang/include/clang/Basic/BuiltinsPPC.def
clang/lib/Headers/altivec.h
clang/test/CodeGen/builtins-ppc-p10vector.c
llvm/include/llvm/IR/IntrinsicsPowerPC.td
llvm/lib/Target/PowerPC/PPCInstrPrefix.td
llvm/test/CodeGen/PowerPC/p10-vector-divide.ll
llvm/test/CodeGen/PowerPC/p10-vector-multiply.ll
|
why does the ck stops matching at the first param? Shouldn't we check the remaining param type and number of param are correct as well?