================
Comment at: lib/Driver/Driver.cpp:444
@@ -443,3 +443,3 @@
if (FailingCommand)
- C.PrintDiagnosticJob(OS, *FailingCommand);
+ FailingCommand->Print(OS, "\n", /*Quote*/ false, /*CrashReportFlagsOnly*/
true);
else
----------------
Reid Kleckner wrote:
> This is 84 cols? Also, you named the bool "CrashReport", not
> "CrashReportFlagsOnly".
Fixed.
================
Comment at: lib/Driver/Job.cpp:31
@@ +30,3 @@
+ // arguments. Therefore, we need to skip the flag and the next argument.
+ bool Res = llvm::StringSwitch<bool>(Flag)
+ .Cases("-I", "-MF", "-MT", "-MQ", true)
----------------
Reid Kleckner wrote:
> Ouch. Can we get this from Options.td, perhaps in a future change?
Yeah, this is not pretty. Using Options.td sounds like a good idea.
http://llvm-reviews.chandlerc.com/D1653
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
