Driver: Make FailingCommand mandatory for generateCompilationDiagnostics
authorJustin Bogner <mail@justinbogner.com>
Mon, 20 Oct 2014 21:02:05 +0000 (21:02 +0000)
committerJustin Bogner <mail@justinbogner.com>
Mon, 20 Oct 2014 21:02:05 +0000 (21:02 +0000)
commite1a33d1e4c925ae4cfe36df13475e56762780360
treeb272a85f211c8acbdf3f03ca29802420f4b9fff7
parent204607bc90b6ff4aaff96a078efa50486babb393
Driver: Make FailingCommand mandatory for generateCompilationDiagnostics

We currently use a null FailingCommand when generating crash reports
as an indication that the cause is FORCE_CLANG_DIAGNOSTICS_CRASH, the
environment variable that exists to test crash dumps. This means that
our tests don't actually cover real crashes at all, and adds a more
complicated code path that's only used in the tests.

Instead, we can have the driver synthesize that every command failed
and just call generateCompilationDiagnostics normally.

llvm-svn: 220234
clang/include/clang/Driver/Driver.h
clang/lib/Driver/Driver.cpp
clang/tools/driver/driver.cpp