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



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

Reply via email to