Eugene.Zelenko closed this revision.
Eugene.Zelenko added a comment.
Committed in https://reviews.llvm.org/rL286455. See
http://llvm.org/docs/Phabricator.html#committing-a-change for review automatic
closing.
https://reviews.llvm.org/D26282
___
cf
kbarton accepted this revision.
kbarton added a comment.
This revision is now accepted and ready to land.
LGTM
https://reviews.llvm.org/D26282
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe
jtony updated this revision to Diff 77190.
jtony added a comment.
Add -U99 option to git diff to show context for this patch.
https://reviews.llvm.org/D26282
Files:
lib/Headers/altivec.h
test/CodeGen/builtins-ppc-altivec.c
test/CodeGen/builtins-ppc-quadword.c
test/CodeGen/builtins-p
jtony updated this revision to Diff 77037.
jtony added a comment.
I also migrated this commit from the old anonymous repository to the new jtony
repository to prepare for committing upstream (use git svn dcommit). Should not
have any difference with the previous patch, update it just in case.
jtony updated this revision to Diff 76945.
jtony added a comment.
Reorder the overloads according to their size.
https://reviews.llvm.org/D26282
Files:
lib/Headers/altivec.h
test/CodeGen/builtins-ppc-altivec.c
test/CodeGen/builtins-ppc-quadword.c
test/CodeGen/builtins-ppc-vsx.c
Index:
nemanjai added inline comments.
Comment at: lib/Headers/altivec.h:15618
+
+#if defined(__POWER8_VECTOR__) && defined(__powerpc64__)
+static inline __ATTRS_o_ai vector signed __int128
Please move the __int128 overloads below the 64-bit ones to keep the overloads
jtony created this revision.
jtony added reviewers: nemanjai, amehsan, kbarton, sfertile, syzaara, lei.
jtony added subscribers: llvm-commits, cfe-commits, echristo.
Implement all the different 24 overloads for vec_xl and vec_xst.
Signatures:
vector signed char vec_xl (signed long long, signed ch