[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