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

Reply via email to