[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