[llvm] r181632 - PR14492: Debug Info: Support for values of non-integer non-type template parameters.

[Dmitri Gribenko]

On Sat, May 11, 2013 at 10:15 PM, David Blaikie <dblaikie at gmail.com> wrote:
> Thanks - fixed (I assume) in r181664. Should we have this warning
> on-by-default for the LLVM build (when building with a supporting
> Clang)?

Thanks.  When I started this discussion a while ago, Chandler raised a
concern about increased compile time.  No optimizations were done to
the parsing code since then (and I don't see anything obviously
implemented inefficiently).

So I have -Wdocumentation enabled on my buildbots, for example here:

http://lab.llvm.org:8011/builders/llvm-clang-lld-x86_64-debian-fast

Note that ccache ignores comments in sources (it does not consider
that a comment change is enough for doing a recompile), so warnings
still appear because ccache prints cached warnings.  I have cleared
the cache now, so warnings should disappear in the next build.

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