[PATCH] D41523: xmmintrin.h documentation fixes and updates
Douglas Yung via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Jan 8 16:27:22 PST 2018
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
More information about the cfe-commits
mailing list