[cfe-commits] r162506 - in /cfe/trunk/lib: CodeGen/CGRTTI.cpp Parse/ParseDeclCXX.cpp Parse/ParseExprCXX.cpp Sema/SemaAccess.cpp Serialization/ASTWriter.cpp

[Dmitri Gribenko]

On Mon, Aug 27, 2012 at 2:25 PM, David Blaikie <dblaikie at gmail.com> wrote:
> On Thu, Aug 23, 2012 at 5:01 PM, Dmitri Gribenko <gribozavr at gmail.com> wrote:
>> Author: gribozavr
>> Date: Thu Aug 23 19:01:24 2012
>> New Revision: 162506
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=162506&view=rev
>> Log:
>> Fix a few -Wdocumentation warnings.
>
> Are Clang & LLVM generally -Wdocumentation clean?
> If you're trying to maintain that invariant, it might be best to add
> it to the LLVM Makefiles (see how -Wcovered-switch-default was
> conditionally added to the build, perhaps)

Yes, it is a goal to add -Wdocumentation to compile flags if it is
supported by the compiler.  There are just a few warnings left (IIRC,
all of them are in Clang).  Unfortunately, some are in headers which
are included all over the place, so that generates lots of noise.

Dmitri

-- 
main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if
(j){printf("%d\n",i);}}} /*Dmitri Gribenko <gribozavr at gmail.com>*/