syzaara updated this revision to Diff 77618.
syzaara added a comment.
Guard for 64-bit mode.
https://reviews.llvm.org/D26304
Files:
include/clang/Basic/BuiltinsPPC.def
lib/CodeGen/CGBuiltin.cpp
lib/Headers/altivec.h
test/CodeGen/builtins-ppc-p9vector.c
Index: test/CodeGen/builtins-ppc-
kbarton accepted this revision.
kbarton added a comment.
This revision is now accepted and ready to land.
With the suggestions above, this LGTM.
https://reviews.llvm.org/D26304
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llv
nemanjai added inline comments.
Comment at: include/clang/Basic/BuiltinsPPC.def:304
+BUILTIN(__builtin_vsx_lxvl, "V4ivC*Ui", "")
+BUILTIN(__builtin_vsx_lxvll, "V4ivC*Ui", "")
Also, in addition to guarding for 64-bit, these should be defined to take the
length
nemanjai added a comment.
Zaara, prior to committing this patch, can you write a test case that will
actually use these builtins in an executable test case and confirm the results.
We'll then run it on the simulator to ensure functional correctness.
Comment at: lib/Headers/al
syzaara created this revision.
syzaara added reviewers: nemanjai, kbarton, amehsan, sfertile, jtony, lei.
syzaara added subscribers: echristo, cfe-commits.
Add builtin functions for prototypes:
vector signed char vec_xl_len (signed char *, size_t);
vector unsigned char vec_xl_len (unsigned char *
