<br><br><div class="gmail_quote">On Fri, Oct 26, 2012 at 8:51 AM, Richard Smith <span dir="ltr"><<a href="mailto:richard@metafoo.co.uk" target="_blank">richard@metafoo.co.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Thu, Oct 25, 2012 at 10:48 PM, Craig Topper <<a href="mailto:craig.topper@gmail.com">craig.topper@gmail.com</a>> wrote:<br>
> I think this change broke bootstrap builds with C++11 and -Werror enabled.<br>
><br>
> lib/VMCore/Verifier.cpp:116:14: error: variable 'PreVerifyID' is not needed<br>
> and will not be emitted [-Werror,-Wunneeded-internal-declaration]<br>
> static char &PreVerifyID = PreVerifier::ID;<br>
<br>
</div>Yes, I think this change probably caused that. I'm unclear on what the<br>
purpose of that warning is: it appears to be warning on variables<br>
which are referenced but not odr-used, which seems like a pretty<br>
questionable thing to warn on. It'd be easy enough to fix this by<br>
teaching Sema::ShouldWarnIfUnusedFileScopedDecl to ignore references<br>
(along with its existing check for const variables), but I'm not sure<br>
that's the right fix, since I'm not really sure what the intent is<br>
here.<br></blockquote></div><br><br>Is not PreVerifyID just an unused variable ? (not the difference with PreVerifier::ID)<br><br>-- Matthieu<br>