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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to