[PATCH] D39903: [libclang] Allow pretty printing declarations
Nikolai Kosjar via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Dec 8 05:47:56 PST 2017
nik added a comment.
In https://reviews.llvm.org/D39903#944182, @cameron314 wrote:
> Locally we've done something similar (adding a `clang_getCursorPrettyPrintedDeclaration` function, though without exposing the `PrintingPolicy`) and overhauled `DeclPrinter` to produce proper pretty names. This is a hack that works well for us, but can never be upstreamed since it breaks too much existing code (and some of the printing decisions are subjective). Your way is better.
You might consider to enhance PrintingPolicy for your use cases?
> I can point out differences in our implementations if you like. The diff is rather long, though.
That would be interesting, yes, but rather later.
First I would like to get a review for this one...
Ping.
Repository:
rC Clang
https://reviews.llvm.org/D39903
More information about the cfe-commits
mailing list