<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Jun 13, 2014 at 1:54 AM, Nikola Smiljanić <span dir="ltr"><<a href="mailto:popizdeh@gmail.com" target="_blank">popizdeh@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Turns out we didn't have correct location and range for class template friend. The same was true for case where friend is not the first token as in Richard's inline example.<br>

<br>
Manuel:<br>
Are my constructor/destructor matchers good?<br></blockquote><div><br></div><div>At a first glance they make sense, yes...</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
Richard:<br>
1. What's the TypeSourceInfo case in FriendDecl::getSourceRange?<br>
2. I couldn't find a way to hit the first call to FriendDecl::Create inside Sema::ActOnTemplatedFriendTag (some kind of explicit specialization case). I tried issuing a random diagnostic from there hoping it would break something in the test suite but it seems we don't have any coverage for this?<br>

<br>
<a href="http://reviews.llvm.org/D3906" target="_blank">http://reviews.llvm.org/D3906</a><br>
<br>
Files:<br>
  include/clang/AST/DeclFriend.h<br>
  include/clang/Sema/Sema.h<br>
  lib/Sema/SemaDecl.cpp<br>
  lib/Sema/SemaDeclCXX.cpp<br>
  lib/Sema/SemaTemplate.cpp<br>
  unittests/AST/SourceLocationTest.cpp<br>
</blockquote></div><br></div></div>