Charusso accepted this revision. Charusso added a comment. This revision is now accepted and ready to land.
I like the idea to put connecting stuff together in the same file to create more understandable code. LGTM. ================ Comment at: clang/lib/StaticAnalyzer/Checkers/Taint.h:8 +// +//===----------------------------------------------------------------------===// +// ---------------- Outdated header-blurb. ================ Comment at: clang/lib/StaticAnalyzer/Checkers/Taint.h:81 + +void dumpTaint(ProgramStateRef State); + ---------------- We left the `ProgramState::dump()' context so what about just `dump()`? Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59861/new/ https://reviews.llvm.org/D59861 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
