[PATCH] Adding doxygen comments to the LLVM intrinsics (part 1, ammintrin.h)
Katya Romanova
Katya_Romanova at playstation.sony.com
Wed May 27 17:57:07 PDT 2015
Hi Dmitri,
One of my colleagues has a Mac, so we have looked up how the documentation for ammintrin.h is rendered the XCode today.
Good news: Regular instrinsics definitions are displayed correctly, though some small changes might be needed to make rendering of the documentation better. I don’t personally like this “code snippet” produced by this comment
/// \code
/// This intrinsic corresponds to the \c VADDPD instruction.
/// \endcode
I don't care for the gray box surrounding this text, identifying it as a code snippet. I don't find it particularly pretty, though I could live with that. Do you have better ideas?
What I definitely don't like is \c command that you asked me to add some time back. It definitely looks weird in XCode. It's interpreted literally, not like a command. I will try to attach a screenshot with generated XCode documentation for one of the intrinsics to Phabricator, so you would know what I'm talking about. If it won't work, I will send you a personal email with the screenshot. Again, thank you so much for reviewing and making helpful comments!
Bad news: Macros are not getting rendered in XCode at all. To say more, unrelated to the intrinsics headers, I wasn't even able to display documentation for the simplest macro that adds two numbers together. Do you know if we could work around it?
http://reviews.llvm.org/D8762
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the cfe-commits
mailing list