nridge added a comment.

In D104619#2831966 <https://reviews.llvm.org/D104619#2831966>, @dblaikie wrote:

> So likely what you'd need would be a unit test of some kind (check around in 
> clang/unittests to see if there are other AST pretty printer tests, for 
> instance).

Thanks for the suggestion! I did find existing pretty printer tests and was 
able to write a test for this fix by doing something similar.

I added you as reviewer for the patch, hope that's ok.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D104619/new/

https://reviews.llvm.org/D104619

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to