[PATCH] D41523: xmmintrin.h documentation fixes and updates

Simon Pilgrim via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 5 06:46:58 PST 2018


RKSimon added a comment.

Sort of related - should we enable -Wdocumentation (it's currently -Wall and -Weverything might be too much....) on the respective clang builtin tests? Doesn't have to be part of this patch.



================
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.
 ///
----------------
Not necessarily, it could be (v)movhps/(v)movhpd - but those are tricky to generate in current isel.


https://reviews.llvm.org/D41523





More information about the cfe-commits mailing list