<div dir="ltr">On 6 February 2013 16:17, Richard Smith <span dir="ltr"><<a href="mailto:richard@metafoo.co.uk" target="_blank">richard@metafoo.co.uk</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">We seem to have the same issue in CreateOverloadedUnaryOp, CreateOverloadedBinOp, CreateOverloadedArraySubscriptExpr, and BuildOverloadedArrowExpr.</blockquote>

<div><br></div><div style>Right you are.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>How about moving the call to MarkFunctionReferenced (and DiagnoseUseOfDecl) into CreateFunctionRefExpr, and changing it to call MarkDeclRefReferenced?<br>

</div></blockquote><div><br></div><div style>That works out perfectly. Updated patch attached!</div><div style><br></div><div style>Nick</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div><div class="gmail_quote"><div class="im">On Tue, Feb 5, 2013 at 10:48 PM, Nick Lewycky <span dir="ltr"><<a href="mailto:nlewycky@google.com" target="_blank">nlewycky@google.com</a>></span> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im">
<div dir="ltr">This patch changes Sema::BuildCallToObjectOfClassType to not mark a virtual operator() as odr used when called via functor syntax "(*obj)();". There's no way this syntax can be used to call the body of a pure virtual function. Please review!<span><font color="#888888"><div>




<br></div><div>Nick</div><div><br></div></font></span></div>
<br></div>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div>
</blockquote></div><br></div></div>