Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2133)

Unified Diff: test/cctest/test-assembler-arm.cc

Issue 12716010: Added a version of the v8::HandleScope constructor with an Isolate and use that consistently. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Fixed rest Created 7 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: test/cctest/test-assembler-arm.cc
diff --git a/test/cctest/test-assembler-arm.cc b/test/cctest/test-assembler-arm.cc
index cfbdb124e3dfac3a2f8c7bc31abd97d1e9afefff..5cb4ab3238e6049b644e8521189487cadf975d60 100644
--- a/test/cctest/test-assembler-arm.cc
+++ b/test/cctest/test-assembler-arm.cc
@@ -57,9 +57,9 @@ static void InitializeVM() {
TEST(0) {
InitializeVM();
- v8::HandleScope scope;
-
Isolate* isolate = Isolate::Current();
+ HandleScope scope(isolate);
+
Assembler assm(isolate, NULL, 0);
__ add(r0, r0, Operand(r1));
@@ -84,9 +84,9 @@ TEST(0) {
TEST(1) {
InitializeVM();
- v8::HandleScope scope;
-
Isolate* isolate = Isolate::Current();
+ HandleScope scope(isolate);
+
Assembler assm(isolate, NULL, 0);
Label L, C;
@@ -122,9 +122,9 @@ TEST(1) {
TEST(2) {
InitializeVM();
- v8::HandleScope scope;
-
Isolate* isolate = Isolate::Current();
+ HandleScope scope(isolate);
+
Assembler assm(isolate, NULL, 0);
Label L, C;
@@ -169,7 +169,8 @@ TEST(2) {
TEST(3) {
InitializeVM();
- v8::HandleScope scope;
+ Isolate* isolate = Isolate::Current();
+ HandleScope scope(isolate);
typedef struct {
int i;
@@ -178,7 +179,6 @@ TEST(3) {
} T;
T t;
- Isolate* isolate = Isolate::Current();
Assembler assm(isolate, NULL, 0);
Label L, C;
@@ -225,7 +225,8 @@ TEST(3) {
TEST(4) {
// Test the VFP floating point instructions.
InitializeVM();
- v8::HandleScope scope;
+ Isolate* isolate = Isolate::Current();
+ HandleScope scope(isolate);
typedef struct {
double a;
@@ -246,7 +247,6 @@ TEST(4) {
// Create a function that accepts &t, and loads, manipulates, and stores
// the doubles and floats.
- Isolate* isolate = Isolate::Current();
Assembler assm(isolate, NULL, 0);
Label L, C;
@@ -364,9 +364,9 @@ TEST(4) {
TEST(5) {
// Test the ARMv7 bitfield instructions.
InitializeVM();
- v8::HandleScope scope;
-
Isolate* isolate = Isolate::Current();
+ HandleScope scope(isolate);
+
Assembler assm(isolate, NULL, 0);
if (CpuFeatures::IsSupported(ARMv7)) {
@@ -401,9 +401,9 @@ TEST(5) {
TEST(6) {
// Test saturating instructions.
InitializeVM();
- v8::HandleScope scope;
-
Isolate* isolate = Isolate::Current();
+ HandleScope scope(isolate);
+
Assembler assm(isolate, NULL, 0);
if (CpuFeatures::IsSupported(ARMv7)) {
@@ -445,9 +445,9 @@ static void TestRoundingMode(VCVTTypes types,
int expected,
bool expected_exception = false) {
InitializeVM();
- v8::HandleScope scope;
-
Isolate* isolate = Isolate::Current();
+ HandleScope scope(isolate);
+
Assembler assm(isolate, NULL, 0);
if (CpuFeatures::IsSupported(VFP3)) {
@@ -623,7 +623,8 @@ TEST(7) {
TEST(8) {
// Test VFP multi load/store with ia_w.
InitializeVM();
- v8::HandleScope scope;
+ Isolate* isolate = Isolate::Current();
+ HandleScope scope(isolate);
typedef struct {
double a;
@@ -651,7 +652,6 @@ TEST(8) {
// Create a function that uses vldm/vstm to move some double and
// single precision values around in memory.
- Isolate* isolate = Isolate::Current();
Assembler assm(isolate, NULL, 0);
if (CpuFeatures::IsSupported(VFP2)) {
@@ -735,7 +735,8 @@ TEST(8) {
TEST(9) {
// Test VFP multi load/store with ia.
InitializeVM();
- v8::HandleScope scope;
+ Isolate* isolate = Isolate::Current();
+ HandleScope scope(isolate);
typedef struct {
double a;
@@ -763,7 +764,6 @@ TEST(9) {
// Create a function that uses vldm/vstm to move some double and
// single precision values around in memory.
- Isolate* isolate = Isolate::Current();
Assembler assm(isolate, NULL, 0);
if (CpuFeatures::IsSupported(VFP2)) {
@@ -851,7 +851,8 @@ TEST(9) {
TEST(10) {
// Test VFP multi load/store with db_w.
InitializeVM();
- v8::HandleScope scope;
+ Isolate* isolate = Isolate::Current();
+ HandleScope scope(isolate);
typedef struct {
double a;
@@ -879,7 +880,6 @@ TEST(10) {
// Create a function that uses vldm/vstm to move some double and
// single precision values around in memory.
- Isolate* isolate = Isolate::Current();
Assembler assm(isolate, NULL, 0);
if (CpuFeatures::IsSupported(VFP2)) {
@@ -963,7 +963,8 @@ TEST(10) {
TEST(11) {
// Test instructions using the carry flag.
InitializeVM();
- v8::HandleScope scope;
+ Isolate* isolate = Isolate::Current();
+ HandleScope scope(isolate);
typedef struct {
int32_t a;
@@ -976,7 +977,6 @@ TEST(11) {
i.a = 0xabcd0001;
i.b = 0xabcd0000;
- Isolate* isolate = Isolate::Current();
Assembler assm(isolate, NULL, 0);
// Test HeapObject untagging.
@@ -1029,9 +1029,10 @@ TEST(11) {
TEST(12) {
// Test chaining of label usages within instructions (issue 1644).
InitializeVM();
- v8::HandleScope scope;
- Assembler assm(Isolate::Current(), NULL, 0);
+ Isolate* isolate = Isolate::Current();
+ HandleScope scope(isolate);
+ Assembler assm(isolate, NULL, 0);
Label target;
__ b(eq, &target);
__ b(ne, &target);
@@ -1043,7 +1044,8 @@ TEST(12) {
TEST(13) {
// Test VFP instructions using registers d16-d31.
InitializeVM();
- v8::HandleScope scope;
+ Isolate* isolate = Isolate::Current();
+ HandleScope scope(isolate);
if (!CpuFeatures::IsSupported(VFP32DREGS)) {
return;
@@ -1064,7 +1066,6 @@ TEST(13) {
// Create a function that accepts &t, and loads, manipulates, and stores
// the doubles and floats.
- Isolate* isolate = Isolate::Current();
Assembler assm(isolate, NULL, 0);
Label L, C;

Powered by Google App Engine
This is Rietveld 408576698