hokein added inline comments.
================ Comment at: clang/lib/AST/TextNodeDumper.cpp:205 + if (T->containsErrors()) + OS << " contains-errors"; ---------------- sammccall wrote: > IIRC in the expr case we used a color, does that make sense here? yeah, in the expr we use a color, but here I don't have a strong opinion -- the other dependence bits don't use color as well, so I'd keep consistent. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D81912/new/ https://reviews.llvm.org/D81912 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
