dgoldman added a comment.

> 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?


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

Reply via email to