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/class/test.cpp | |
parent | 7de9969bb0853f3331a29825ae5a959515d59d87 (diff) | |
download | llvm-ee6d3fa07eed0d3187ded0cbc63fdbbdc4a8480c.zip llvm-ee6d3fa07eed0d3187ded0cbc63fdbbdc4a8480c.tar.gz llvm-ee6d3fa07eed0d3187ded0cbc63fdbbdc4a8480c.tar.bz2 |
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/class/test.cpp')
-rw-r--r-- | clang/test/ASTMerge/class/test.cpp | 14 |
1 files changed, 14 insertions, 0 deletions
diff --git a/clang/test/ASTMerge/class/test.cpp b/clang/test/ASTMerge/class/test.cpp new file mode 100644 index 0000000..a68a2d1 --- /dev/null +++ b/clang/test/ASTMerge/class/test.cpp @@ -0,0 +1,14 @@ +// RUN: %clang_cc1 -emit-pch -o %t.1.ast %S/Inputs/class1.cpp +// RUN: %clang_cc1 -emit-pch -o %t.2.ast %S/Inputs/class2.cpp +// RUN: %clang_cc1 -ast-merge %t.1.ast -ast-merge %t.2.ast -fsyntax-only %s 2>&1 | FileCheck %s +// RUN: %clang_cc1 -ast-merge %t.1.ast -ast-merge %t.2.ast -fsyntax-only %s 2>&1 -Wno-odr -Werror + +// CHECK: class1.cpp:6:8: warning: type 'B' has incompatible definitions in different translation units +// CHECK: class1.cpp:7:9: note: field 'y' has type 'float' here +// CHECK: class2.cpp:7:7: note: field 'y' has type 'int' here + +// FIXME: we should also complain about mismatched types on the method + +// CHECK: class1.cpp:18:6: warning: type 'E' has incompatible definitions in different translation units +// CHECK: class1.cpp:19:3: note: enumerator 'b' with value 1 here +// CHECK: class2.cpp:12:3: note: enumerator 'a' with value 0 here |