[PATCH] D45463: [AST] Print correct tag decl for tag specifier
Joel E. Denny via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri May 11 16:06:55 PDT 2018
jdenny added a comment.
In https://reviews.llvm.org/D45463#1096629, @rsmith wrote:
> In https://reviews.llvm.org/D45463#1096499, @jdenny wrote:
>
> > A few comments ago <https://reviews.llvm.org/D45463#1094614>, I mentioned that IncludeTagDefinition's documentation and name is drifting farther from its functionality. Should we do something about that?
>
>
> I think we should deprecate that flag, and change it to have no effect. It is an implementation detail of the AST printer and should never have been exposed by libclang in the first place.
Thanks. I'll address that in a separate patch then.
https://reviews.llvm.org/D45463
More information about the cfe-commits
mailing list