[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.


More information about the cfe-commits mailing list