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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
