<div dir="ltr">On Tue, Sep 10, 2013 at 5:28 AM, Alexey Samsonov <span dir="ltr"><<a href="mailto:samsonov@google.com" target="_blank">samsonov@google.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr">Hi Eli,<div><br></div><div>Recently sanitizer-x86_64-linux buildbot which bootstraps Clang started failing with -Wunused warnings which looks like false positives to me:</div>
<div><a href="http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux/builds/848/steps/build%2064-bit%20llvm%20using%20clang/logs/stdio" target="_blank">http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux/builds/848/steps/build%2064-bit%20llvm%20using%20clang/logs/stdio</a></div>

<div><div><br></div><div><div>/build/llvm/lib/Transforms/Instrumentation/ThreadSanitizer.cpp:108:23: error: variable 'kNumberOfAccessSizes' is not needed and will not be emitted [-Werror,-Wunneeded-internal-declaration]</div>

<div>  static const size_t kNumberOfAccessSizes = 5;</div></div></div><div><br></div><div>This static member is actually used to define array sizes, and in the function code as well.</div><div><br></div><div>Can this be related to your change?</div>

</div><div class="gmail_extra"><div><div class="h5"><br></div></div><div class="h5"><br></div></div></blockquote><div><span style="font-family:arial,sans-serif;font-size:13px">r190382.  Thanks for the report.</span></div>
<div><br></div><div>-Eli </div></div><br></div></div>