[llvm-commits] [llvm] r166024 - /llvm/trunk/include/llvm/CodeGen/LiveIntervalAnalysis.h

[Dmitri Gribenko]

On Tue, Oct 16, 2012 at 6:53 PM, David Blaikie <dblaikie at gmail.com> wrote:
> On Tue, Oct 16, 2012 at 8:37 AM, Dmitri Gribenko <gribozavr at gmail.com> wrote:
>> Author: gribozavr
>> Date: Tue Oct 16 10:37:50 2012
>> New Revision: 166024
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=166024&view=rev
>> Log:
>> Fix function parameter spelling in comments.  Caught by -Wdocumentation.
>
> Are Clang & LLVM -Wdocumentation clean now? Could we add some build
> magic (to both CMake and configure+make) to enable this flag when it's
> available (we already do something similar for
> -Wcovered-switch-default)

They are 99% clean.  There are just a few warnings left that refer to
parameters that are no longer there and SVN history was not helpful in
understanding how that comment and declaration evolved.

If you are interested, I build LLVM and clang with
--with-extra-options=-Wdocumentation\
-Wno-documentation-deprecated-sync

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>*/