[v8-dev] [v8] r22745 committed - Fixed a few space leaks....

codesite-noreply via v8-dev Thu, 31 Jul 2014 03:18:13 -0700

Revision: 22745
Author:   [email protected]
Date:     Thu Jul 31 10:17:25 2014 UTC
Log:      Fixed a few space leaks.


[email protected]

Review URL: https://codereview.chromium.org/430133002
http://code.google.com/p/v8/source/detail?r=22745

Modified:
 /branches/bleeding_edge/src/compiler/instruction.cc
 /branches/bleeding_edge/test/cctest/compiler/test-scheduler.cc

=======================================

--- /branches/bleeding_edge/src/compiler/instruction.cc Wed Jul 30 16:21:362014 UTC+++ /branches/bleeding_edge/src/compiler/instruction.cc Thu Jul 31 10:17:252014 UTC

@@ -450,7 +450,7 @@
       os << "\n";
     }

-    Vector<char> buf = Vector<char>::New(32);
+    ScopedVector<char> buf(32);
     for (int j = block->first_instruction_index();
          j <= block->last_instruction_index(); j++) {
       // TODO(svenpanne) Add some basic formatting to our streams.
=======================================

--- /branches/bleeding_edge/test/cctest/compiler/test-scheduler.cc Wed Jul30 16:21:36 2014 UTC+++ /branches/bleeding_edge/test/cctest/compiler/test-scheduler.cc Thu Jul31 10:17:25 2014 UTC

@@ -146,7 +146,7 @@
   HandleAndZoneScope scope;
   Schedule schedule(scope.main_zone());
   Scheduler scheduler(scope.main_zone(), NULL, &schedule);
-  TestLoop* loop1 = CreateLoop(&schedule, 2);
+  SmartPointer<TestLoop> loop1(CreateLoop(&schedule, 2));
   schedule.AddSuccessor(schedule.entry(), loop1->header());
   BasicBlockVector* order = scheduler.ComputeSpecialRPO();
   CheckRPONumbers(order, 3, true);
@@ -158,7 +158,7 @@
   HandleAndZoneScope scope;
   Schedule schedule(scope.main_zone());
   Scheduler scheduler(scope.main_zone(), NULL, &schedule);
-  TestLoop* loop1 = CreateLoop(&schedule, 2);
+  SmartPointer<TestLoop> loop1(CreateLoop(&schedule, 2));
   schedule.AddSuccessor(schedule.entry(), loop1->header());
   schedule.AddSuccessor(loop1->last(), schedule.entry());
   BasicBlockVector* order = scheduler.ComputeSpecialRPO();
@@ -354,8 +354,8 @@
   Schedule schedule(scope.main_zone());
   Scheduler scheduler(scope.main_zone(), NULL, &schedule);

-  TestLoop* loop1 = CreateLoop(&schedule, 1);
-  TestLoop* loop2 = CreateLoop(&schedule, 1);
+  SmartPointer<TestLoop> loop1(CreateLoop(&schedule, 1));
+  SmartPointer<TestLoop> loop2(CreateLoop(&schedule, 1));

   BasicBlock* A = schedule.entry();
   BasicBlock* E = schedule.exit();
@@ -371,8 +371,6 @@
   CHECK_EQ(schedule.BasicBlockCount(), static_cast<int>(order->size()));
   CheckLoopContains(loop1->nodes, loop1->count);
   CheckLoopContains(loop2->nodes, loop2->count);
-  delete loop1;
-  delete loop2;
 }


@@ -381,8 +379,8 @@
   Schedule schedule(scope.main_zone());
   Scheduler scheduler(scope.main_zone(), NULL, &schedule);

-  TestLoop* loop1 = CreateLoop(&schedule, 1);
-  TestLoop* loop2 = CreateLoop(&schedule, 1);
+  SmartPointer<TestLoop> loop1(CreateLoop(&schedule, 1));
+  SmartPointer<TestLoop> loop2(CreateLoop(&schedule, 1));

   BasicBlock* A = schedule.entry();
   BasicBlock* S = schedule.NewBasicBlock();
@@ -400,8 +398,6 @@
   CHECK_EQ(schedule.BasicBlockCount(), static_cast<int>(order->size()));
   CheckLoopContains(loop1->nodes, loop1->count);
   CheckLoopContains(loop2->nodes, loop2->count);
-  delete loop1;
-  delete loop2;
 }


@@ -412,8 +408,8 @@
     for (int exit = 0; exit < size; exit++) {
       Schedule schedule(scope.main_zone());
       Scheduler scheduler(scope.main_zone(), NULL, &schedule);
-      TestLoop* loop1 = CreateLoop(&schedule, size);
-      TestLoop* loop2 = CreateLoop(&schedule, size);
+      SmartPointer<TestLoop> loop1(CreateLoop(&schedule, size));
+      SmartPointer<TestLoop> loop2(CreateLoop(&schedule, size));
       BasicBlock* A = schedule.entry();
       BasicBlock* E = schedule.exit();

@@ -426,8 +422,6 @@

CHECK_EQ(schedule.BasicBlockCount(),static_cast<int>(order->size()));

       CheckLoopContains(loop1->nodes, loop1->count);
       CheckLoopContains(loop2->nodes, loop2->count);
-      delete loop1;
-      delete loop2;
     }
   }
 }
@@ -438,8 +432,8 @@
   Schedule schedule(scope.main_zone());
   Scheduler scheduler(scope.main_zone(), NULL, &schedule);

-  TestLoop* loop1 = CreateLoop(&schedule, 1);
-  TestLoop* loop2 = CreateLoop(&schedule, 1);
+  SmartPointer<TestLoop> loop1(CreateLoop(&schedule, 1));
+  SmartPointer<TestLoop> loop2(CreateLoop(&schedule, 1));

   BasicBlock* A = schedule.entry();
   BasicBlock* B = schedule.NewBasicBlock();
@@ -463,8 +457,6 @@

   BasicBlock* loop3[] = {B, loop1->nodes[0], loop2->nodes[0], C};
   CheckLoopContains(loop3, 4);
-  delete loop1;
-  delete loop2;
 }


@@ -479,7 +471,7 @@
       BasicBlock* A = schedule.entry();
       BasicBlock* E = schedule.exit();

-      TestLoop* loop1 = CreateLoop(&schedule, size);
+      SmartPointer<TestLoop> loop1(CreateLoop(&schedule, size));
       schedule.AddSuccessor(A, loop1->header());
       schedule.AddSuccessor(loop1->last(), E);

@@ -489,7 +481,6 @@
       BasicBlockVector* order = scheduler.ComputeSpecialRPO();
       CheckRPONumbers(order, schedule.BasicBlockCount(), true);
       CheckLoopContains(loop1->nodes, loop1->count);
-      delete loop1;
     }
   }
 }
@@ -507,7 +498,7 @@
       BasicBlock* D = schedule.NewBasicBlock();
       BasicBlock* E = schedule.exit();

-      TestLoop* loop1 = CreateLoop(&schedule, size);
+      SmartPointer<TestLoop> loop1(CreateLoop(&schedule, size));
       schedule.AddSuccessor(A, loop1->header());
       schedule.AddSuccessor(loop1->last(), E);

@@ -518,7 +509,6 @@
       BasicBlockVector* order = scheduler.ComputeSpecialRPO();
       CheckRPONumbers(order, schedule.BasicBlockCount(), true);
       CheckLoopContains(loop1->nodes, loop1->count);
-      delete loop1;
     }
   }
 }
@@ -534,7 +524,7 @@
     BasicBlock* A = schedule.entry();
     BasicBlock* E = schedule.exit();

-    TestLoop* loop1 = CreateLoop(&schedule, size);
+    SmartPointer<TestLoop> loop1(CreateLoop(&schedule, size));
     schedule.AddSuccessor(A, loop1->header());
     schedule.AddSuccessor(loop1->last(), E);

@@ -547,7 +537,6 @@
     BasicBlockVector* order = scheduler.ComputeSpecialRPO();
     CheckRPONumbers(order, schedule.BasicBlockCount(), true);
     CheckLoopContains(loop1->nodes, loop1->count);
-    delete loop1;
   }
 }

@@ -561,7 +550,7 @@
     Scheduler scheduler(scope.main_zone(), NULL, &schedule);
     BasicBlock* A = schedule.entry();
     BasicBlock* E = schedule.exit();
-    TestLoop* loop1 = CreateLoop(&schedule, size);
+    SmartPointer<TestLoop> loop1(CreateLoop(&schedule, size));
     schedule.AddSuccessor(A, loop1->header());
     schedule.AddSuccessor(loop1->last(), E);

@@ -581,7 +570,6 @@
       delete loopN[j];
     }
     delete[] loopN;
-    delete loop1;
   }
 }

--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

---You received this message because you are subscribed to the Google Groups "v8-dev" group.

To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

[v8-dev] [v8] r22745 committed - Fixed a few space leaks....

Reply via email to