[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