<div dir="ltr">Anyone able to review this? It's minor since it doesn't seem to affect valid code, but still would like to get this fixed if my patch is OK....<div><br></div><div>Also, I've attached patch updated against the latest revision<br>
<div><br></div><div style>Thanks,</div><div style>Stephen</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Mar 12, 2013 at 5:30 PM, Stephen Lin <span dir="ltr"><<a href="mailto:swlin@post.harvard.edu" target="_blank">swlin@post.harvard.edu</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi,<div><br></div><div>This is a bug fix for <span style="font-family:arial,sans-serif;font-size:13px">LLVM Bugzilla bug #15485 (</span><a href="http://llvm.org/bugs/show_bug.cgi?id=15485" target="_blank">http://llvm.org/bugs/show_bug.cgi?id=15485</a>)</div>

<div><br></div><div>Basically, llvm_unreachable is currently hit because CheckOverload assumes that a new function declaration in record scope should trigger member function hiding rules, but this is not the case for "friend" declarations.</div>

<div><br></div><div><div>I believe the code causing the issue is illegal (the friend declaration should not friend the function brought into scope by the using) and should result in an error; this is what happens after the patch without any extra effort (although the error could possibly be improved?).</div>

<div><br></div><div>Please let me know if you have any comments.<br></div></div><div><br></div><div>Best,</div><div>Stephen</div><div><br></div><div>P.S. The patch is in git diff format; hope that's ok</div>
</div>
</blockquote></div><br></div></div>