diff options
| author | Shengchen Kan <shengchen.kan@intel.com> | 2020-06-08 21:43:24 +0800 |
|---|---|---|
| committer | Shengchen Kan <shengchen.kan@intel.com> | 2020-06-08 23:14:16 +0800 |
| commit | 2c63ea6eded3eed4018306363c28b7f63d7b5898 (patch) | |
| tree | dff808b8eb52f055dfecee8fd17a5e6fa5f2ad2e /clang/unittests/Tooling/Syntax/TreeTest.cpp | |
| parent | f70912f885f991d5af11d8ecb10b703f3cbed982 (diff) | |
| download | llvm-2c63ea6eded3eed4018306363c28b7f63d7b5898.zip llvm-2c63ea6eded3eed4018306363c28b7f63d7b5898.tar.gz llvm-2c63ea6eded3eed4018306363c28b7f63d7b5898.tar.bz2 | |
[TEST] TreeTest.cpp - Add a comma to avoid build error with -werror
Summary:
The macro `INSTANTIATE_TEST_CASE_P` is defined as
```
\# define INSTANTIATE_TEST_CASE_P(prefix, test_case_name, generator, ...) \
...
```
If we build the test case with -werror, we will get an error like
```
error: ISO C++11 requires at least one argument for the "..." in a
variadic macro
testing::ValuesIn(TestClangConfig::allConfigs()));
^
```
This patch fixes that.
Reviewers: gribozavr, hlopko, eduucaldas, gribozavr2
Reviewed By: gribozavr2
Subscribers: cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D81388
Diffstat (limited to 'clang/unittests/Tooling/Syntax/TreeTest.cpp')
| -rw-r--r-- | clang/unittests/Tooling/Syntax/TreeTest.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/clang/unittests/Tooling/Syntax/TreeTest.cpp b/clang/unittests/Tooling/Syntax/TreeTest.cpp index 49fdcb7..64acfe3 100644 --- a/clang/unittests/Tooling/Syntax/TreeTest.cpp +++ b/clang/unittests/Tooling/Syntax/TreeTest.cpp @@ -2995,6 +2995,6 @@ TEST_P(SyntaxTreeTest, SynthesizedNodes) { } INSTANTIATE_TEST_CASE_P(SyntaxTreeTests, SyntaxTreeTest, - testing::ValuesIn(TestClangConfig::allConfigs())); + testing::ValuesIn(TestClangConfig::allConfigs()), ); } // namespace |