<div dir="ltr">I've finally had time to circle back to this, and just sent a version of this through <a href="http://reviews.llvm.org">reviews.llvm.org</a>.<div><br></div><div>It is highly edited, and we may as well start from scratch. In particular, my usecase has no need for all the desugaring, so I have removed it all. So no std::vector special casing, and no desugaring code even exists to simplify.</div><div><br></div><div>Take a look if you get a chance.</div><div><br></div><div><a href="http://reviews.llvm.org/D15861">http://reviews.llvm.org/D15861</a><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Oct 26, 2015 at 6:03 AM, Benjamin Kramer <span dir="ltr"><<a href="mailto:benny.kra@gmail.com" target="_blank">benny.kra@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Some high level style comments:<br>
<br>
1. Please convert the file to LLVM style for the things that<br>
clang-format doesn't change. In particular PascalCase for all<br>
variables and cameCase for all function names.<br>
2. We don't do author attribution in file comments, sorry.<br>
3. Try to avoid commented out/#if 0'd code.<br>
4. I'd really prefer to review this on <a href="http://reviews.llvm.org" rel="noreferrer" target="_blank">reviews.llvm.org</a>, makes things easier.<br>
<br>
On the functional side:<br>
1. Why is std::vector and friends handled in a special case, don't we<br>
want this for all default arguments?<br>
2. Can the desugaring code be simplified somehow? it feels extremely<br>
repetitive and the corresponding code in Type.cpp looks much simpler?<br>
Where is the additional complexity coming from?<br>
<br>
- Ben<br>
<div><div class="h5"><br>
<br>
On Sat, Oct 24, 2015 at 12:29 AM, Sterling Augustine via cfe-commits<br>
<<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a>> wrote:<br>
> As requested on cfe-dev, enclosed is a patch (that needs some<br>
> style-related work) to calculate the complete fully qualified name of<br>
> arbitrary Qualtypes, including recursively expanding template<br>
> parameters and subtypes.<br>
><br>
> Seems to me like it follows most of the conventions described here:<br>
><br>
> <a href="http://llvm.org/docs/CodingStandards.html" rel="noreferrer" target="_blank">http://llvm.org/docs/CodingStandards.html</a><br>
><br>
> but I'm told it needs more work. Happy to oblige.<br>
><br>
</div></div>> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a><br>
> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
><br>
</blockquote></div><br></div>