<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Mar 11, 2014 at 1:10 AM, Stephan Bergmann <span dir="ltr"><<a href="mailto:sbergman@redhat.com" target="_blank">sbergman@redhat.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The code<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  bool ShouldCheckUse = true;<br>
  if (CXXMethodDecl *MD = dyn_cast<CXXMethodDecl>(<u></u>MemberDecl)) {<br>
    // Don't diagnose the use of a virtual member function unless it's<br>
    // explicitly qualified.<br>
    if (MD->isVirtual() && !SS.isSet())<br>
      ShouldCheckUse = false;<br>
  }<br>
</blockquote>
<br>
in lib/Sema/SemaExprMember.cpp, originally introduced with r81460 to fix <<a href="http://llvm.org/bugs/show_bug.cgi?id=4878" target="_blank">http://llvm.org/bugs/show_<u></u>bug.cgi?id=4878</a>> "ability to call deprecated virtual methods in the same class without warning" prevents warnings on all virtual calls to such deprecated functions.  Not just when the call is from the same class, but also when a client of that class calls the function.<br>

<br>
Is that intentional?  (I just had a case where it would have been useful to get the warning for the client-call case at least, and was surprised to not get a warning, esp. since GCC does give one.)</blockquote><div><br></div>
<div>This looks like a bug to me, please file a PR! </div></div></div></div>