[PATCH] D20404: [Driver] Fix driver support for color diagnostics
Duncan P. N. Exon Smith via cfe-commits
cfe-commits at lists.llvm.org
Wed May 18 19:01:10 PDT 2016
dexonsmith added a comment.
I don't think we even need a helper.
Clang::ConstructJob takes a `Compilation`, which has a `Driver`, which has a `DiagnosticsEngine`, which has a `DiagnosticsOptions`.
In other words, I think you can delete the code in Clang::ConstructJob (change it to a lookup) now that you've changed clang::ParseDiagnosticArgs to update `DiagnosticsOptions`.
http://reviews.llvm.org/D20404
More information about the cfe-commits
mailing list