This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rG5d1fe3f903b9: [PowerPC] Implemented Vector Multiply Builtins
(authored by Conanap).
Changed prior to commit:
amyk accepted this revision as: amyk.
amyk added a comment.
I think this LGTM now. The file is already upstream, so your tests will need to
be added to that file.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D83955/new/
Conanap updated this revision to Diff 279637.
Conanap marked 2 inline comments as done.
Conanap added a comment.
Changed if def to use pwer10_vector
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D83955/new/
https://reviews.llvm.org/D83955
Files:
amyk requested changes to this revision.
amyk added inline comments.
This revision now requires changes to proceed.
Comment at: clang/lib/Headers/altivec.h:5472
+
+#ifdef _ARCH_PWR10
+static __inline__ vector unsigned __int128 __ATTRS_o_ai
Please use
Conanap created this revision.
Conanap added reviewers: PowerPC, saghir, nemanjai, hfinkel.
Conanap added projects: LLVM, clang, PowerPC.
Herald added subscribers: steven.zhang, kbarton.
This patch implements 128-bit Binary Vector Multiply builtins for PowerPC10.
Repository:
rG LLVM Github