diff options
author | Benjamin Kramer <benny.kra@googlemail.com> | 2016-05-18 09:28:45 +0000 |
---|---|---|
committer | Benjamin Kramer <benny.kra@googlemail.com> | 2016-05-18 09:28:45 +0000 |
commit | 2ecd0902c723f7b56fd145a60c2e50610ab73dc6 (patch) | |
tree | 18b93a816940184fca90b09aa040822857cf32b0 | |
parent | 1549a2f46a8da1dac4a305df3b8c7dbbf9ba403c (diff) | |
download | bcm5719-llvm-2ecd0902c723f7b56fd145a60c2e50610ab73dc6.tar.gz bcm5719-llvm-2ecd0902c723f7b56fd145a60c2e50610ab73dc6.zip |
[include-fixer] Run tests with -fno-ms-compatibility.
Something behind that flag makes us get fewer typo correction callbacks,
unbreak the tests on windows for now.
llvm-svn: 269894
-rw-r--r-- | clang-tools-extra/unittests/include-fixer/IncludeFixerTest.cpp | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/clang-tools-extra/unittests/include-fixer/IncludeFixerTest.cpp b/clang-tools-extra/unittests/include-fixer/IncludeFixerTest.cpp index 39af81b7c1c..dacd10eb640 100644 --- a/clang-tools-extra/unittests/include-fixer/IncludeFixerTest.cpp +++ b/clang-tools-extra/unittests/include-fixer/IncludeFixerTest.cpp @@ -27,7 +27,9 @@ static bool runOnCode(tooling::ToolAction *ToolAction, StringRef Code, new vfs::InMemoryFileSystem); llvm::IntrusiveRefCntPtr<FileManager> Files( new FileManager(FileSystemOptions(), InMemoryFileSystem)); - std::vector<std::string> Args = {"include_fixer", "-fsyntax-only", FileName}; + // FIXME: Investigate why -fms-compatibility breaks tests. + std::vector<std::string> Args = {"include_fixer", "-fsyntax-only", + "-fno-ms-compatibility", FileName}; Args.insert(Args.end(), ExtraArgs.begin(), ExtraArgs.end()); tooling::ToolInvocation Invocation( Args, ToolAction, Files.get(), @@ -130,8 +132,6 @@ TEST(IncludeFixer, MinimizeInclude) { runIncludeFixer("a::b::foo bar;\n", IncludePath)); } -#ifndef _WIN32 -// It doesn't pass for targeting win32. Investigating. TEST(IncludeFixer, NestedName) { // Some tests don't pass for target *-win32. std::vector<std::string> args = {"-target", "x86_64-unknown-unknown"}; @@ -149,7 +149,6 @@ TEST(IncludeFixer, NestedName) { "namespace a {}\nint a = a::b::foo(0);\n", runIncludeFixer("namespace a {}\nint a = a::b::foo(0);\n", args)); } -#endif TEST(IncludeFixer, MultipleMissingSymbols) { EXPECT_EQ("#include <string>\nstd::string bar;\nstd::sting foo;\n", |