dyung added inline comments.
================ Comment at: lib/Headers/xmmintrin.h:1927 /// -/// This intrinsic corresponds to the <c> VPEXTRQ / MOVQ </c> instruction. +/// This intrinsic corresponds to the <c> VPEXTRQ / PEXTRQ </c> instruction. /// ---------------- RKSimon wrote: > Not necessarily, it could be (v)movhps/(v)movhpd - but those are tricky to > generate in current isel. Would it be preferred then for this to be the following: ``` /// This intrinsic corresponds to the <c> VPEXTRQ / PEXTRQ / VMOVHPS / VMOVHPD / MOVHPS / MOVHPD </c> instruction. ``` https://reviews.llvm.org/D41523 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
