aaron.ballman added a comment. In https://reviews.llvm.org/D39722#934783, @aaron.ballman wrote:
> In https://reviews.llvm.org/D39722#934781, @tk1012 wrote: > > > Hello Aaron, > > > > I remove the semicolon. > > > > > Is this type actually correct for C++? > > > > Yes, it is. > > clang generates the AST for `declToImport` struct like this. > > > > |-CXXRecordDecl 0x8b19fe0 <col:22, col:29> col:29 implicit struct > > declToImport > > `-CXXMethodDecl 0x8b1a0d0 <line:2:3, col:27> col:8 m 'void (void)' > > `-CompoundStmt 0x8b1a1e8 <col:12, col:27> > > `-TypeTraitExpr 0x8b1a1c8 <col:14, col:24> '_Bool' > > > > > Hmm, that looks like a bug to me; but not one that should impact this review. Yeah, it seems as though the ASTDumper does not receive a printing policy, and the default behavior of `QualType::getAsString()` is to gin up a very dumb printing policy from default language options, hence the poor quality of the name here. https://reviews.llvm.org/D39722 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
