[PATCH] D41516: emmintrin.h documentation fixes and updates

2018-01-09 Thread Eli Friedman via Phabricator via cfe-commits
efriedma added inline comments. Comment at: cfe/trunk/lib/Headers/emmintrin.h:4683 /// -/// This intrinsic has no corresponding instruction. +/// This intrinsic corresponds to the MOVDQ2Q instruction. /// kromanova wrote: > efriedma wrote: > > kromanova

[PATCH] D41516: emmintrin.h documentation fixes and updates

2018-01-08 Thread Katya Romanova via Phabricator via cfe-commits
kromanova added inline comments. Comment at: cfe/trunk/lib/Headers/emmintrin.h:4683 /// -/// This intrinsic has no corresponding instruction. +/// This intrinsic corresponds to the MOVDQ2Q instruction. /// efriedma wrote: > kromanova wrote: > > kromanova

[PATCH] D41516: emmintrin.h documentation fixes and updates

2018-01-08 Thread Eli Friedman via Phabricator via cfe-commits
efriedma added inline comments. Comment at: cfe/trunk/lib/Headers/emmintrin.h:4683 /// -/// This intrinsic has no corresponding instruction. +/// This intrinsic corresponds to the MOVDQ2Q instruction. /// kromanova wrote: > kromanova wrote: > > I'm not sure

[PATCH] D41516: emmintrin.h documentation fixes and updates

2018-01-08 Thread Katya Romanova via Phabricator via cfe-commits
kromanova added inline comments. Herald added a subscriber: llvm-commits. Comment at: cfe/trunk/lib/Headers/emmintrin.h:3865 /// -/// This intrinsic corresponds to the VPUNPCKLQDQ / PUNPCKLQDQ -/// instruction. +/// This intrinsic does not correspond to a specific

[PATCH] D41516: emmintrin.h documentation fixes and updates

2018-01-04 Thread Katya Romanova via Phabricator via cfe-commits
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. ///

[PATCH] D41516: emmintrin.h documentation fixes and updates

2018-01-02 Thread Douglas Yung via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL321669: [DOXYGEN] Fix doxygen and content issues in emmintrin.h (authored by dyung, committed by ). Changed prior to commit: https://reviews.llvm.org/D41516?vs=127939=128447#toc Repository: rL LLVM

[PATCH] D41516: emmintrin.h documentation fixes and updates

2017-12-22 Thread Craig Topper via Phabricator via cfe-commits
craig.topper accepted this revision. craig.topper added a comment. This revision is now accepted and ready to land. LGTM https://reviews.llvm.org/D41516 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D41516: emmintrin.h documentation fixes and updates

2017-12-21 Thread Douglas Yung via Phabricator via cfe-commits
dyung created this revision. This is the result of several patches we made internally to update the documentation that we would like to have reviewed for possible submission. The changes include: 1. Fixed innaccurate instruction mappings for various intrinsics. 2. Fixed description of NaN