<br><br><div class="gmail_quote">On Fri, Dec 9, 2011 at 1:56 PM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@google.com">chandlerc@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; padding-left: 1ex; border-left-color: rgb(204, 204, 204); border-left-width: 1px; border-left-style: solid;">
<div class="gmail_quote">On Fri, Dec 9, 2011 at 1:42 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; padding-left: 1ex; border-left-color: rgb(204, 204, 204); border-left-width: 1px; border-left-style: solid;">

<div>+          S.isExprCallable(*E, ReturnType, NonTemplateOverloads);<br>
+          if (!ReturnType.isNull()</div></blockquote></div><br><div>We should make Sema::isExprCallable return ReturnType.isNull() to simplify this pattern...</div><div><br></div><div>I mention this as you were looking at fixing other parts isExprCallable...</div>

</blockquote></div><div> </div><div>Yeah, that bit mystified me too - the Boolean return from isExprCallable isn't entirely clear to me - I suspect it returns true if it's overloaded too, perhaps. (ie: if either of th eout parameters can be used - then you're meant to test them to disambiguate)<br>
</div>