r190382 - Make -Wunused warning rules more consistent.

Eli Friedman eli.friedman at gmail.com
Tue Sep 10 14:22:14 PDT 2013


On Tue, Sep 10, 2013 at 5:28 AM, Alexey Samsonov <samsonov at google.com>wrote:

> Hi Eli,
>
> Recently sanitizer-x86_64-linux buildbot which bootstraps Clang started
> failing with -Wunused warnings which looks like false positives to me:
>
> http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux/builds/848/steps/build%2064-bit%20llvm%20using%20clang/logs/stdio
>
> /build/llvm/lib/Transforms/Instrumentation/ThreadSanitizer.cpp:108:23:
> error: variable 'kNumberOfAccessSizes' is not needed and will not be
> emitted [-Werror,-Wunneeded-internal-declaration]
>   static const size_t kNumberOfAccessSizes = 5;
>
> This static member is actually used to define array sizes, and in the
> function code as well.
>
> Can this be related to your change?
>
>
> r190382.  Thanks for the report.

-Eli
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130910/c6efd6ea/attachment.html>


More information about the cfe-commits mailing list