[PATCH] D26479: [PowerPC] Implement remaining permute builtins in altivec.h - Clang portion

[Nemanja Ivanovic via cfe-commits]

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