[PATCH] D41516: emmintrin.h documentation fixes and updates
Katya Romanova via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Jan 4 20:12:53 PST 2018
kromanova added inline comments.
================
Comment at: cfe/trunk/lib/Headers/emmintrin.h:1143
///
-/// If either of the two lower double-precision values is NaN, 1 is returned.
+/// If either of the two lower double-precision values is NaN, 0 is returned.
///
----------------
Formatting is inconsistent with the rest of the changes above or below. One sentence here separated by the empty lines, where everywhere else it's 2 sentences.
================
Comment at: cfe/trunk/lib/Headers/emmintrin.h:4683
///
-/// This intrinsic has no corresponding instruction.
+/// This intrinsic corresponds to the <c> MOVDQ2Q </c> instruction.
///
----------------
I'm not sure about this change.
Intel documentation says they generate MOVDQ2Q (don't have icc handy to try).
However, I've tried on Linux/X86_64 with clang and gcc, - and we just return.
Repository:
rL LLVM
https://reviews.llvm.org/D41516
More information about the cfe-commits
mailing list