<div dir="ltr">So clang folks, where should this new code to calculate the complete, fully qualified name of arbitrary qualtypes go? <div><br></div><div>My best guess is as a new file in clang/lib/AST, but it's not a great fit.</div><div><br></div><div>Does the license as listed in the file (reproduced below) create any issues?</div><div><div><br></div><div>// This file is dual-licensed: you can choose to license it under the University</div><div>// of Illinois Open Source License or the GNU Lesser General Public License. See</div><div>// LICENSE.TXT for details.</div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Oct 15, 2015 at 8:47 AM, Philippe Canal <span dir="ltr"><<a href="mailto:pcanal@fnal.gov" target="_blank">pcanal@fnal.gov</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div bgcolor="#FFFFFF" text="#000000">
Hi Sterling,<span class=""><br>
<br>
> Would you all be amenable to contributing
cling/lib/Utils/AST.cpp to clang?<br>
<br></span>
Of course :). I agree that this feature belongs in clang proper.<br>
<br>
AST.cpp is pretty much standalone (depends only on clang and llvm)<br>
<br>
Thanks,<br>
Philippe.<br>
<br>
PS. A part that has not yet been integrated in AST.cpp is the
adding of default<br>
template parameters (for user classes where that parameter might
matter for I/O)<br>
and can be found in:<br>
<a href="https://root.cern.ch/gitweb?p=root.git;a=blob;f=core/metautils/src/TMetaUtils.cxx" target="_blank">https://root.cern.ch/gitweb?p=root.git;a=blob;f=core/metautils/src/TMetaUtils.cxx</a><br>
in the routines<br>
ROOT::TMetaUtils::GetNormalizedType<br>
ROOT::TMetaUtils::AddDefaultParameters<br>
ROOT::TMetaUtils::KeepNParams<div><div class="h5"><br>
<br>
<div>On 10/14/15 11:53 AM, Sterling
Augustine wrote:<br>
</div>
<blockquote type="cite">
<div dir="ltr">
<div class="gmail_extra">
<div class="gmail_quote">On Tue, Oct 13, 2015 at 2:48 PM,
Philippe Canal <span dir="ltr"><<a href="mailto:pcanal@fnal.gov" target="_blank"></a><a href="mailto:pcanal@fnal.gov" target="_blank">pcanal@fnal.gov</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div bgcolor="#FFFFFF" text="#000000"> Hi,<br>
<br>
We also discovered that we could not find in clang a
routine that reliably generate the name (or the
equivalent<br>
decl/type) corresponding to a type as it would need to
be typed from the global namespace (i.e. a unique
identifier<br>
of the class); </div>
</blockquote>
<div><br>
</div>
<div>Philippe,</div>
<div><br>
</div>
<div>Would you all be amenable to contributing
cling/lib/Utils/AST.cpp to clang?</div>
<div><br>
</div>
<div>I would be willing to write-up the patch if that would
be helpful. I'm not sure how deep the dependencies go.</div>
</div>
</div>
</div>
</blockquote>
<br>
</div></div></div>
</blockquote></div><br></div>