diff options
| author | Hans Wennborg <hans@hanshq.net> | 2013-09-18 22:26:39 +0000 |
|---|---|---|
| committer | Hans Wennborg <hans@hanshq.net> | 2013-09-18 22:26:39 +0000 |
| commit | 9cb7d9ba94769d77b077b813536d35f27a9918f1 (patch) | |
| tree | 5926745e39981c2985515bf9361b43798a7c2b3a /clang/test | |
| parent | 0db88a7d7e96cc75c53ea04878b003453cfc9023 (diff) | |
| download | bcm5719-llvm-9cb7d9ba94769d77b077b813536d35f27a9918f1.tar.gz bcm5719-llvm-9cb7d9ba94769d77b077b813536d35f27a9918f1.zip | |
clang-cl: Don't warn about overriding /MD with /MT, /Fo with another /Fo, etc.
I put in the warnings because MSVC has them, but I don't think they're very
useful.
Clang does not warn about overriding flags in general, e.g. it's perfectly
fine to have -fomit-frame-pointer followed by -fno-omit-frame-pointer.
We should focus on warning where things get confusing, such as with the
/TP and /TC options. In "clang-cl /TC a.c /TP b.cc", the user might not
realize that the /TP flag will apply to both files, and we warn about that.
Differential Revision: http://llvm-reviews.chandlerc.com/D1718
llvm-svn: 190964
Diffstat (limited to 'clang/test')
| -rw-r--r-- | clang/test/Driver/cl-outputs.c | 2 | ||||
| -rw-r--r-- | clang/test/Driver/cl-runtime-flags.c | 1 |
2 files changed, 0 insertions, 3 deletions
diff --git a/clang/test/Driver/cl-outputs.c b/clang/test/Driver/cl-outputs.c index 636ef96f60d..82134b05626 100644 --- a/clang/test/Driver/cl-outputs.c +++ b/clang/test/Driver/cl-outputs.c @@ -11,7 +11,6 @@ // FoNAME: "-o" "a.obj" // RUN: %clang_cl /Foa.ext /Fob.ext -### -- %s 2>&1 | FileCheck -check-prefix=FoNAMEEXT %s -// FoNAMEEXT: warning: overriding '/Foa.ext' option with '/Fob.ext' // FoNAMEEXT: "-o" "b.ext" // RUN: %clang_cl /Fofoo.dir/ -### -- %s 2>&1 | FileCheck -check-prefix=FoDIR %s @@ -88,5 +87,4 @@ // FeMISSINGARG: error: argument to '/Fe' is missing (expected 1 value) // RUN: %clang_cl /Fefoo /Febar -### -- %s 2>&1 | FileCheck -check-prefix=FeOVERRIDE %s -// FeOVERRIDE: warning: overriding '/Fefoo' option with '/Febar' // FeOVERRIDE: "-out:bar.exe" diff --git a/clang/test/Driver/cl-runtime-flags.c b/clang/test/Driver/cl-runtime-flags.c index 11a5e948767..8367531f9cc 100644 --- a/clang/test/Driver/cl-runtime-flags.c +++ b/clang/test/Driver/cl-runtime-flags.c @@ -86,5 +86,4 @@ // CHECK-LDMDd: "--dependent-lib=msvcrtd" // RUN: %clang_cl /MD /MT -### -- %s 2>&1 | FileCheck -check-prefix=MTOVERRIDE %s -// MTOVERRIDE: warning: overriding '/MD' option with '/MT' // MTOVERRIDE: "--dependent-lib=libcmt" |
