-
Notifications
You must be signed in to change notification settings - Fork 10.7k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[clang-tidy] Move incomplete type test into separate test file
Summary: Move in complete type test which does not compile into its own test file. Reviewers: alexfh, sbenza, aaron.ballman Subscribers: cfe-commits Differential Revision: https://reviews.llvm.org/D26369 llvm-svn: 286155
- Loading branch information
Felix Berger
committed
Nov 7, 2016
1 parent
9d8ab72
commit 4a3f94c
Showing
2 changed files
with
10 additions
and
8 deletions.
There are no files selected for viewing
9 changes: 9 additions & 0 deletions
9
clang-tools-extra/test/clang-tidy/performance-unnecessary-value-param-incomplete-type.cpp
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,9 @@ | ||
// RUN: %check_clang_tidy %s performance-unnecessary-value-param %t -- -fix-errors -- --std=c++11 | ||
|
||
// Ensure that incomplete types result in an error from the frontend and not a | ||
// clang-tidy diagnostic about IncompleteType being expensive to copy. | ||
struct IncompleteType; | ||
void NegativeForIncompleteType(IncompleteType I) { | ||
// CHECK-MESSAGES: [[@LINE-1]]:47: error: variable has incomplete type 'IncompleteType' [clang-diagnostic-error] | ||
} | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters