[PATCH] Refactor Compilation::PrintJob and PrintDiagnosticJob into Job::Print

[Hans Wennborg]

================
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