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