r181081 - [doc parsing]: Make warning about unknown command

Dmitri Gribenko gribozavr at gmail.com
Wed May 8 16:00:24 PDT 2013


On Thu, May 9, 2013 at 1:54 AM, jahanian <fjahanian at apple.com> wrote:
>
> On May 8, 2013, at 3:52 PM, Dmitri Gribenko <gribozavr at gmail.com> wrote:
>
> On Sat, May 4, 2013 at 3:47 AM, Fariborz Jahanian <fjahanian at apple.com>
> wrote:
>
> Author: fjahanian
> Date: Fri May  3 19:47:28 2013
> New Revision: 181081
>
> URL: http://llvm.org/viewvc/llvm-project?rev=181081&view=rev
> Log:
> [doc parsing]: Make warning about unknown command
> tags off by default for now. Move diagnostic code
> to DiagnosticCommentKinds.td. // rdar://12381408
>
>
> I think this should be moved a separate group (for example, a separate
> flag under DocumentationPedantic group).  It causes way too much noise
> even for LLVM and Clang.
>
> http://lab.llvm.org:8011/builders/llvm-clang-lld-x86_64-debian-fast/builds/2344/steps/build/logs/stdio
>
>
> I agree. I was just looking at the log.

What do you think about -Wdocumentation-unknown-command flag name?  I
can not think of anything shorter...

> (There's also a crash somewhere in diagnostic printing, BTW…)
>
>
> I just fixed the crash

r181479?  Erm, sort of.  The root cause of the crash is still there.

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




More information about the cfe-commits mailing list