diff options
author | Stephen Hines <srhines@google.com> | 2011-09-23 15:53:16 -0700 |
---|---|---|
committer | Stephen Hines <srhines@google.com> | 2011-09-23 15:53:16 -0700 |
commit | c632be206ac4fe49a5db05cfa54942d774329dbe (patch) | |
tree | bad18d44a17a1a18c82f80953a36fe12dac4a9d5 /llvm-rs-cc.cpp | |
parent | 3f175af8a0644fb5fc53aa90e01c24f75855c5f7 (diff) | |
download | frameworks_compile_slang-c632be206ac4fe49a5db05cfa54942d774329dbe.tar.gz frameworks_compile_slang-c632be206ac4fe49a5db05cfa54942d774329dbe.tar.bz2 frameworks_compile_slang-c632be206ac4fe49a5db05cfa54942d774329dbe.zip |
Generate warnings properly for multi-file compiles.
BUG=5367019
Slang (llvm-rs-cc) was clobbering warnings during multi-file compilation
because they are not part of the error count. When the diagnostics get reset,
we should be dumping whichever non-error (i.e. warning) diagnostics we have
accumulated. During investigation, I also found then that our dependency
generation path can cause duplicate warnings to be added to the diagnostic
client. Thus, to fix this bug, the change need dumps relevant diagnostics, but
then suppresses warnings if we are compiling the same file multiple times
(once for code generation and once for dependencies).
Change-Id: I4b94e962f93b96f31ff7e7d6cce45418fe39e1d1
Diffstat (limited to 'llvm-rs-cc.cpp')
-rw-r--r-- | llvm-rs-cc.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm-rs-cc.cpp b/llvm-rs-cc.cpp index ae5618c..2a29a3b 100644 --- a/llvm-rs-cc.cpp +++ b/llvm-rs-cc.cpp @@ -444,7 +444,7 @@ int main(int argc, const char **argv) { Opts.mTargetAPI, Opts.mJavaReflectionPathBase, Opts.mJavaReflectionPackageName); - llvm::errs() << Compiler->getErrorMessage(); + Compiler->reset(); return CompileFailed; } |