This revision was automatically updated to reflect the committed changes.
Closed by commit rL300068: [x86] fix AVX FP cmp intrinsic documentation
(PR28110) (authored by spatel).
Changed prior to commit:
https://reviews.llvm.org/D31428?vs=93366=94976#toc
Repository:
rL LLVM
spatel added a comment.
Ping.
https://reviews.llvm.org/D31428
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
spatel updated this revision to Diff 93366.
spatel added a comment.
Patch updated:
Standardize the spelling of "non-signaling" and don't abbreviate "unordered".
https://reviews.llvm.org/D31428
Files:
lib/Headers/avxintrin.h
Index: lib/Headers/avxintrin.h
RKSimon added inline comments.
Comment at: lib/Headers/avxintrin.h:1752
+///0x1e : Greater-than (ordered, non-signaling)
+///0x1f : True (unordered, signaling)
/// \returns A 128-bit vector of [4 x float] containing the comparison results.
Minor: clean
spatel created this revision.
Herald added a subscriber: mcrosier.
This copies the text used in the #define statements to the code comments. The
conflicting text comes from AMD manuals, but those are wrong. Sadly, this part
has not been updated even after some docs were updated for Zen: