summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorBenjamin Kramer <benny.kra@googlemail.com>2016-05-18 09:28:45 +0000
committerBenjamin Kramer <benny.kra@googlemail.com>2016-05-18 09:28:45 +0000
commit2ecd0902c723f7b56fd145a60c2e50610ab73dc6 (patch)
tree18b93a816940184fca90b09aa040822857cf32b0
parent1549a2f46a8da1dac4a305df3b8c7dbbf9ba403c (diff)
downloadbcm5719-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.cpp7
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",
OpenPOWER on IntegriCloud