r285825 - Simplify the test case from r285289.
Alex Lorenz via cfe-commits
cfe-commits at lists.llvm.org
Wed Nov 2 09:11:30 PDT 2016
Author: arphaman
Date: Wed Nov 2 11:11:30 2016
New Revision: 285825
URL: http://llvm.org/viewvc/llvm-project?rev=285825&view=rev
Log:
Simplify the test case from r285289.
This commit simplifies and clarifies the test code
that was added in r285289.
Suggested by David Blaikie.
Modified:
cfe/trunk/test/SemaCXX/warn-unused-variables.cpp
Modified: cfe/trunk/test/SemaCXX/warn-unused-variables.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/SemaCXX/warn-unused-variables.cpp?rev=285825&r1=285824&r2=285825&view=diff
==============================================================================
--- cfe/trunk/test/SemaCXX/warn-unused-variables.cpp (original)
+++ cfe/trunk/test/SemaCXX/warn-unused-variables.cpp Wed Nov 2 11:11:30 2016
@@ -153,36 +153,29 @@ namespace ctor_with_cleanups {
namespace arrayRecords {
-int total = 0;
-
-class Adder {
+class NonTriviallyDestructible {
public:
- Adder(int x); // out of line below
- ~Adder() {}
+ ~NonTriviallyDestructible() {}
};
-Adder::Adder(int x) {
- total += x;
-}
-
struct Foo {
int x;
Foo(int x) : x(x) {}
};
-struct S1 {
- S1();
+struct Elidable {
+ Elidable();
};
void foo(int size) {
- S1 y; // no warning
- S1 yarray[2]; // no warning
- S1 dynArray[size]; // no warning
- S1 nestedArray[1][2][3]; // no warning
-
- Adder scalerInFuncScope = 134; // no warning
- Adder arrayInFuncScope[] = { 135, 136 }; // no warning
- Adder nestedArrayInFuncScope[2][2] = { {1,2}, {3,4} }; // no warning
+ Elidable elidable; // no warning
+ Elidable elidableArray[2]; // no warning
+ Elidable elidableDynArray[size]; // no warning
+ Elidable elidableNestedArray[1][2][3]; // no warning
+
+ NonTriviallyDestructible scalar; // no warning
+ NonTriviallyDestructible array[2]; // no warning
+ NonTriviallyDestructible nestedArray[2][2]; // no warning
Foo fooScalar = 1; // expected-warning {{unused variable 'fooScalar'}}
Foo fooArray[] = {1,2}; // expected-warning {{unused variable 'fooArray'}}
@@ -191,8 +184,8 @@ void foo(int size) {
template<int N>
void bar() {
- Adder scaler = 123; // no warning
- Adder array[N] = {1,2}; // no warning
+ NonTriviallyDestructible scaler; // no warning
+ NonTriviallyDestructible array[N]; // no warning
}
void test() {
More information about the cfe-commits
mailing list