<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Sep 18, 2014 at 9:15 AM, Aaron Ballman <span dir="ltr"><<a href="mailto:aaron@aaronballman.com" target="_blank">aaron@aaronballman.com</a>></span> wrote:<br><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 class=""><div class="h5">On Thu, Sep 18, 2014 at 12:12 PM, David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br>
><br>
><br>
> On Thu, Sep 18, 2014 at 9:04 AM, Aaron Ballman <<a href="mailto:aaron@aaronballman.com">aaron@aaronballman.com</a>><br>
> wrote:<br>
>><br>
>> On Thu, Sep 18, 2014 at 11:55 AM, David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>><br>
>> wrote:<br>
>> ><br>
>> ><br>
>> > On Thu, Sep 18, 2014 at 6:27 AM, Aaron Ballman <<a href="mailto:aaron@aaronballman.com">aaron@aaronballman.com</a>><br>
>> > wrote:<br>
>> >><br>
>> >> Author: aaronballman<br>
>> >> Date: Thu Sep 18 08:27:14 2014<br>
>> >> New Revision: 218050<br>
>> >><br>
>> >> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=218050&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=218050&view=rev</a><br>
>> >> Log:<br>
>> >> Fixing a bunch of -Woverloaded-virtual warnings due to hiding<br>
>> >> getSubtargetImpl from the base class. NFC.<br>
>> ><br>
>> ><br>
>> > Which compiler is producing that warning here?<br>
>><br>
>> I believe this is GCC warning on it. It's our attribute documentation<br>
>> building bot that is kicking off these warnings, so I don't have a<br>
>> link to a bot to show you. If you would like, I can forward the bot<br>
>> report email.<br>
><br>
><br>
> That's OK - it looks most likely like the GCC flavor of this warning.<br>
><br>
>><br>
>><br>
>> > I /think/ this is one of those cases where we improved the Clang warning<br>
>> > and<br>
>> > should just disable the GCC one.<br>
>> ><br>
>> > (there's some debate about what the /point/ of this warning is, which is<br>
>> > fair - but I tuned Clang's to detect the cases where someone tried to<br>
>> > override and accidentally overloaded - which isn't the case here. The<br>
>> > base<br>
>> > class has a couple of virtual methods, the derived class correctly<br>
>> > overrode<br>
>> > one of them. I don't think this is a major source of bugs/worth fixing,<br>
>> > possibly)<br>
>> ><br>
>> > (the gap of course is that you can still end up with this problem:<br>
>> > struct base { virtual void func(int); virtual void func(bool); };<br>
>> > struct derived : base { void func(bool) override; void stuff() {<br>
>> > func(42);<br>
>> > /* oops, this calls func(bool) because shadowing */ } };<br>
>> > hence the debate about what the purpose of the warning is)<br>
>><br>
>> I don't have any strong opinions on whether the warning provides value<br>
>> or not, but I do have strong opinions on ensuring a warning-free build<br>
>> for this bot, which is the only reason I cleaned these up.<br>
><br>
><br>
> *nod* if it's not too much hassle - could we clean it up by disabling the<br>
> warning instead? I assume we already have a bunch of GCC warnings we disable<br>
> & it's just a matter of adding this one to the list...<br>
<br>
</div></div>I don't know how much hassle is involved, I've not disabled any GCC<br>
warnings (only MSVC ones). If someone who knows more about where that<br>
magic lives in the build system would like to tackle it, I am okay<br>
with the idea.<br></blockquote><div><br></div><div>Little confusion when I started to look. Addressed in <span style="color:rgb(0,0,0)">r218059.<br><br>(if you like, you can revert this commit now that it won't flag on the bots - I don't really mind all that much either way)<br><br>Thanks!</span></div><div> </div><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">
<span class=""><font color="#888888"><br>
~Aaron<br>
</font></span></blockquote></div><br></div></div>