This is an archive of the discontinued LLVM Phabricator instance.

Set of all x86 intrinsic header files with generated doxygen comments (as per Sean Silva request).
AbandonedPublic

Authored by kromanova on Jan 21 2016, 8:05 PM.

Details

Reviewers
None
Summary

Please note that no code review is actually needed.

In review D15999 (http://reviews.llvm.org/D15999) Sean Silva asked me to post a patch with "all" the headers with doxygen comments, so that others can test and verify.
Here is the patch automatically generated by DCG tool (doxygen comment generator) that I wrote.

There are a couple of things that the people should be aware:
(a) *only* intrinsics that are supported on PS4 will have doxygen comments. PS4 documentation is parsed to generate the comments
and it doesn't document the intrinsics that are *not* supported on PS4.
(b) the doxygen comments for all the headers files (except ammintrin.h that was already accepted upstream and _wmmintrin_pclmul.h that is currently in the review)
might have some known problems (typos, incorrect parameter types, bugs in the description, etc). DCG might have some glitches too (e.g. currently it doesn't parse HTML tables in the documentation).
Most of these things will get fixed before I submit the Phabricator code review for a specific file. So, please don't assume that this is the final version of the headers with doxygen comments. It's a rough dratf. Use with caution! :)
However, it should be perfectly acceptable for measuring build time performance impact.

Diff Detail

Repository
rL LLVM

Event Timeline

kromanova updated this revision to Diff 45643.Jan 21 2016, 8:05 PM
kromanova retitled this revision from to Set of all x86 intrinsic header files with generated doxygen comments (as per Sean Silva request)..
kromanova updated this object.
kromanova set the repository for this revision to rL LLVM.
kromanova added a subscriber: silvas.
kromanova abandoned this revision.Jul 24 2017, 4:12 PM

This was for information purpose only (by Sean Silva's request) and no review was needed. All doxygen comments for x86 intrinsic had already landed, so I am abandoning this review.

lib/Headers/bmiintrin.h