<div dir="auto">Ok, sorry for getting the wrong end of the stick re the history.<div dir="auto"><br></div><div dir="auto">Arguments vs parameters - are we sure this is a distinction worth surfacing to users/uis? I guess modeling both using the same struct is awkward.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Dec 18, 2019, 10:57 AM Kadir Cetinkaya via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">kadircet marked an inline comment as done.<br>
kadircet added inline comments.<br>
<br>
<br>
================<br>
Comment at: clang-tools-extra/clangd/unittests/HoverTests.cpp:1209<br>
             // FIXME: Print template instantiation parameters.<br>
-            HI.Name = "initializer_list";<br>
+            HI.Name = "initializer_list<int>";<br>
             HI.Kind = index::SymbolKind::Class;<br>
----------------<br>
sammccall wrote:<br>
> Hang on, I think we're going round in circles with this design.<br>
> <br>
> IIRC the idea was that `Name` doesn't include template parameters, signature, etc, so clients control rendering.<br>
> <br>
> Isn't it easy to reconstitute this from the template argument list in the hover info?<br>
we were only storing template parameters, not arguments. they've always been the part of the name, they had disappeared after a previous patch, as an intermediate state and this was introducing them back.<br>
<br>
i am planning to move template arguments into a different field though, as template parameters, which should also help with dropping default arguments.<br>
<br>
<br>
Repository:<br>
  rG LLVM Github Monorepo<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D71545/new/" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D71545/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D71545" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D71545</a><br>
<br>
<br>
<br>
</blockquote></div>