r260333 - This patch adds doxygen comments for all the intrinsincs in the header file f16cintrin.h. The doxygen comments are automatically generated based on Sony's intrinsics document.

Joerg Sonnenberger via cfe-commits cfe-commits at lists.llvm.org
Wed Feb 10 17:07:35 PST 2016


On Thu, Feb 11, 2016 at 12:18:49AM +0000, Eric Christopher via cfe-commits wrote:
> Mostly that you didn't mention it in your commit message.
> 
> I'd rather prefer the rest of the file be changed to match the __ rather
> than the other way around :)

Depends. If it is a macro, not using __ is fine. If it is an inline
function, it *must* use the protected namespace, since an application can
#define a fancy junk here.

Joerg


More information about the cfe-commits mailing list