echristo added a comment. One inline comment for the clang part and then let's split the llvm part out into a separate patch please.
Thanks! -eric ================ Comment at: clang/lib/Headers/altivec.h:8045 /* vec_sl */ static __inline__ vector unsigned char __ATTRS_o_ai ---------------- Can you add a note here that these functions are documented to return a shift with the shift operand modulo the size of the type? https://reviews.llvm.org/D28037 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
