nemanjai added inline comments.
================ Comment at: include/clang/Basic/BuiltinsPPC.def:385-388 +BUILTIN(__builtin_altivec_vrlwmi, "V4UiV4UiV4UiV4Ui", "") +BUILTIN(__builtin_altivec_vrldmi, "V2ULLiV2ULLiV2ULLiV2ULLi", "") +BUILTIN(__builtin_altivec_vrlwnm, "V4UiV4UiV4Ui", "") +BUILTIN(__builtin_altivec_vrldnm, "V2ULLiV2ULLiV2ULLi", "") ---------------- sfertile wrote: > A minor quibble: we have the __builtin_altivec_* functions added after the > vsx builtins, rather than with the rest of the altivec builtins. Good point. Thanks Sean. I'll move them on the commit. Repository: rL LLVM https://reviews.llvm.org/D26479 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits