<div dir="ltr">Vedant,<div><br></div><div>I have tried the example in your test case and could not get a spurious warning from it.  Further, having two different functions map to the same canonical function decl would be problematic for Clang.  Can you give any more information about where you saw this spurious warning?</div><div><br></div><div>Richard</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jul 29, 2015 at 4:41 PM, Vedant Kumar <span dir="ltr"><<a href="mailto:vsk@apple.com" target="_blank">vsk@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Sema::checkRecursiveFunction() currently cannot tell overloaded functions apart.<br>
<br>
This results in spurious warnings -- clang claims that a function is infinitely recursive when it in fact just calls a different function. The attached patch fixes the problem by keeping the original FunctionDecl around, instead of switching to the 'canonical' decl.<br>
<br>
I don't have commit rights, so I'd appreciate someone taking a look at the patch and guiding it into trunk :).<br>
<span class="HOEnZb"><font color="#888888"><br>
vedant<br>
<br>
</font></span><br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div>