gAlfonso-bit added a comment. In D107775#2967134 <https://reviews.llvm.org/D107775#2967134>, @dgoldman wrote:
>> I put a case for it in the declprinter so it is handled appropriately so it >> would get the pointee type, whereas before this was neglected. >> >> Other than that, there should be no difference. > > It looks like DeclPrinter only uses that method here > <https://github.com/llvm/llvm-project/blob/ef227b32b63c53ca81ebd410c7fbd5af8fc22ec5/clang/lib/AST/DeclPrinter.cpp#L415> > - so it shouldn't make a difference. > > but TypePrinter does call `isSpecifierType()` so this changes that behavior, > right? Can you add a test to TypePrinterTest for this change? Like for the function itself or an integration test? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107775/new/ https://reviews.llvm.org/D107775 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
