This is an archive of the discontinued LLVM Phabricator instance.

[DOXYGEN] documentation changes to emmintrin.h and tmmintrin.h
ClosedPublic

Authored by kromanova on Jan 9 2018, 3:48 PM.

Details

Summary

I comment was brought up in the patch https://reviews.llvm.org/D41507 with the request to change the notation for hex numbers in doxygen documentation from <...>h to 0x<...>.
Both of these notations were used in x86 intrinsics documentation and I promised to change hex numbers format everywhere in intrisics doxygen comments to from <...>h to 0x<...> for consistency.

Diff Detail

Repository
rL LLVM

Event Timeline

kromanova created this revision.Jan 9 2018, 3:48 PM
kromanova added a subscriber: cfe-commits.
kromanova updated this revision to Diff 134537.Feb 15 2018, 5:43 PM

Doug Yung (the reviewer for this patch) noticed that some lines exceed 80 characters limitation. I fixed that.

dyung accepted this revision.Feb 15 2018, 5:51 PM

LGTM

This revision is now accepted and ready to land.Feb 15 2018, 5:51 PM
This revision was automatically updated to reflect the committed changes.