modocache added a comment. Thank you, @bruno! Good idea, I'll add a `-cc1` invocation test.
================ Comment at: lib/Driver/Driver.cpp:191 if (A->getOption().hasFlag(options::Unsupported)) { - Diag(diag::err_drv_unsupported_opt) << A->getAsString(Args); - ContainsError |= Diags.getDiagnosticLevel(diag::err_drv_unsupported_opt, - SourceLocation()) > + unsigned DiagID; + auto ArgString = A->getAsString(Args); ---------------- jroelofs wrote: > No need for this variable. There's a call below to `Diags.getDiagnosticsLevel`, which takes this ID as an argument. Are you suggesting I leave that line alone, and have it use the diagnostic level of `err_drv_unsupported_opt`, even if `err_drv_unsupported_opt_with_suggestion` is actually emitted here? That seems reasonable to me, but just want to make sure that's what you had in mind. Repository: rC Clang https://reviews.llvm.org/D41733 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits