diff options
author | Sean Callanan <scallanan@apple.com> | 2016-11-16 18:21:00 +0000 |
---|---|---|
committer | Sean Callanan <scallanan@apple.com> | 2016-11-16 18:21:00 +0000 |
commit | ee6d3fa07eed0d3187ded0cbc63fdbbdc4a8480c (patch) | |
tree | 6578eb05e2f5031c7d30586ea47f019c2483942d /clang/test/ASTMerge/namespace/test.cpp | |
parent | 7de9969bb0853f3331a29825ae5a959515d59d87 (diff) | |
download | bcm5719-llvm-ee6d3fa07eed0d3187ded0cbc63fdbbdc4a8480c.tar.gz bcm5719-llvm-ee6d3fa07eed0d3187ded0cbc63fdbbdc4a8480c.zip |
Fixed layout of test/ASTMerge.
As outlined in a previous RFC, the test/ASTMerge/Inputs folder is getting full and the tests are starting to become interdependent. This is undesirable because
- it makes it harder to write new tests
- it makes it harder to figure out at a glance what old tests are doing, and
- it adds the risk of breaking one test while changing a different one, because of the interdependencies.
To fix this, according to the conversation in the RFC, I have changed the layout from
a.c
Inputs/a1.c
Inputs/a2.c
to
a/test.c
a/Inputs/a1.c
a/Inputs/a2.c
for all existing tests. I have also eliminated interdependencies by replicating the input files for each test that uses them.
https://reviews.llvm.org/D26571
llvm-svn: 287129
Diffstat (limited to 'clang/test/ASTMerge/namespace/test.cpp')
-rw-r--r-- | clang/test/ASTMerge/namespace/test.cpp | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/clang/test/ASTMerge/namespace/test.cpp b/clang/test/ASTMerge/namespace/test.cpp new file mode 100644 index 00000000000..8cc0fa2a6d9 --- /dev/null +++ b/clang/test/ASTMerge/namespace/test.cpp @@ -0,0 +1,6 @@ +// RUN: %clang_cc1 -emit-pch -o %t.1.ast %S/Inputs/namespace1.cpp +// RUN: %clang_cc1 -emit-pch -o %t.2.ast %S/Inputs/namespace2.cpp +// RUN: not %clang_cc1 -ast-merge %t.1.ast -ast-merge %t.2.ast -fsyntax-only %s 2>&1 | FileCheck %s + +// CHECK: namespace2.cpp:16:17: error: external variable 'z' declared with incompatible types in different translation units ('double' vs. 'float') +// CHECK: namespace1.cpp:16:16: note: declared here with type 'float' |