Revision: 16657
Author:   [email protected]
Date:     Wed Sep 11 14:36:27 2013 UTC
Log:      MIPS: remove most uses of Isolate::Current in arch specific files.

Port r16643 (b818da5)

BUG=
[email protected]

Review URL: https://codereview.chromium.org/24108002

Patch from Balazs Kilvady <[email protected]>.
http://code.google.com/p/v8/source/detail?r=16657

Modified:
 /branches/bleeding_edge/src/mips/assembler-mips-inl.h
 /branches/bleeding_edge/src/mips/assembler-mips.cc
 /branches/bleeding_edge/src/mips/code-stubs-mips.cc
 /branches/bleeding_edge/src/mips/code-stubs-mips.h
 /branches/bleeding_edge/src/mips/debug-mips.cc
 /branches/bleeding_edge/src/mips/ic-mips.cc
 /branches/bleeding_edge/src/mips/regexp-macro-assembler-mips.cc

=======================================
--- /branches/bleeding_edge/src/mips/assembler-mips-inl.h Fri Jun 21 09:24:30 2013 UTC +++ /branches/bleeding_edge/src/mips/assembler-mips-inl.h Wed Sep 11 14:36:27 2013 UTC
@@ -338,7 +338,7 @@
 }


-void RelocInfo::Visit(ObjectVisitor* visitor) {
+void RelocInfo::Visit(Isolate* isolate, ObjectVisitor* visitor) {
   RelocInfo::Mode mode = rmode();
   if (mode == RelocInfo::EMBEDDED_OBJECT) {
     visitor->VisitEmbeddedPointer(this);
@@ -351,12 +351,11 @@
   } else if (RelocInfo::IsCodeAgeSequence(mode)) {
     visitor->VisitCodeAgeSequence(this);
 #ifdef ENABLE_DEBUGGER_SUPPORT
-  // TODO(isolates): Get a cached isolate below.
   } else if (((RelocInfo::IsJSReturn(mode) &&
               IsPatchedReturnSequence()) ||
              (RelocInfo::IsDebugBreakSlot(mode) &&
              IsPatchedDebugBreakSlotSequence())) &&
-             Isolate::Current()->debug()->has_break_points()) {
+             isolate->debug()->has_break_points()) {
     visitor->VisitDebugTarget(this);
 #endif
   } else if (RelocInfo::IsRuntimeEntry(mode)) {
=======================================
--- /branches/bleeding_edge/src/mips/assembler-mips.cc Mon Aug 26 09:37:39 2013 UTC +++ /branches/bleeding_edge/src/mips/assembler-mips.cc Wed Sep 11 14:36:27 2013 UTC
@@ -238,15 +238,12 @@
 // See assembler-mips-inl.h for inlined constructors.

 Operand::Operand(Handle<Object> handle) {
-#ifdef DEBUG
-  Isolate* isolate = Isolate::Current();
-#endif
   AllowDeferredHandleDereference using_raw_address;
   rm_ = no_reg;
   // Verify all Objects referred by code are NOT in new space.
   Object* obj = *handle;
-  ASSERT(!isolate->heap()->InNewSpace(obj));
   if (obj->IsHeapObject()) {
+    ASSERT(!HeapObject::cast(obj)->GetHeap()->InNewSpace(obj));
     imm32_ = reinterpret_cast<intptr_t>(handle.location());
     rmode_ = RelocInfo::EMBEDDED_OBJECT;
   } else {
=======================================
--- /branches/bleeding_edge/src/mips/code-stubs-mips.cc Wed Sep 11 14:15:18 2013 UTC +++ /branches/bleeding_edge/src/mips/code-stubs-mips.cc Wed Sep 11 14:36:27 2013 UTC
@@ -649,7 +649,7 @@
 }


-bool WriteInt32ToHeapNumberStub::IsPregenerated() {
+bool WriteInt32ToHeapNumberStub::IsPregenerated(Isolate* isolate) {
   // These variants are compiled ahead of time.  See next method.
   if (the_int_.is(a1) &&
       the_heap_number_.is(v0) &&
@@ -2723,8 +2723,8 @@
 }


-bool CEntryStub::IsPregenerated() {
-  return (!save_doubles_ || Isolate::Current()->fp_stubs_generated()) &&
+bool CEntryStub::IsPregenerated(Isolate* isolate) {
+  return (!save_doubles_ || isolate->fp_stubs_generated()) &&
           result_size_ == 1;
 }

@@ -6585,7 +6585,7 @@
 #undef REG


-bool RecordWriteStub::IsPregenerated() {
+bool RecordWriteStub::IsPregenerated(Isolate* isolate) {
   for (const AheadOfTimeWriteBarrierStubList* entry = kAheadOfTime;
        !entry->object.is(no_reg);
        entry++) {
=======================================
--- /branches/bleeding_edge/src/mips/code-stubs-mips.h Sun Sep 1 17:22:08 2013 UTC +++ /branches/bleeding_edge/src/mips/code-stubs-mips.h Wed Sep 11 14:36:27 2013 UTC
@@ -69,7 +69,7 @@

   void Generate(MacroAssembler* masm);

-  virtual bool IsPregenerated() { return true; }
+ virtual bool IsPregenerated(Isolate* isolate) V8_OVERRIDE { return true; }
   static void GenerateFixedRegStubsAheadOfTime(Isolate* isolate);
   virtual bool SometimesSetsUpAFrame() { return false; }

@@ -240,7 +240,7 @@
     ASSERT(SignRegisterBits::is_valid(sign_.code()));
   }

-  bool IsPregenerated();
+  virtual bool IsPregenerated(Isolate* isolate) V8_OVERRIDE;
   static void GenerateFixedRegStubsAheadOfTime(Isolate* isolate);

  private:
@@ -316,7 +316,7 @@
     INCREMENTAL_COMPACTION
   };

-  virtual bool IsPregenerated();
+  virtual bool IsPregenerated(Isolate* isolate) V8_OVERRIDE;
   static void GenerateFixedRegStubsAheadOfTime(Isolate* isolate);
   virtual bool SometimesSetsUpAFrame() { return false; }

=======================================
--- /branches/bleeding_edge/src/mips/debug-mips.cc Fri Aug 2 09:53:11 2013 UTC +++ /branches/bleeding_edge/src/mips/debug-mips.cc Wed Sep 11 14:36:27 2013 UTC
@@ -60,7 +60,7 @@
   // li and Call pseudo-instructions emit two instructions each.
   patcher.masm()->li(v8::internal::t9,
       Operand(reinterpret_cast<int32_t>(
-          Isolate::Current()->debug()->debug_break_return()->entry())));
+ debug_info_->GetIsolate()->debug()->debug_break_return()->entry())));
   patcher.masm()->Call(v8::internal::t9);
   patcher.masm()->nop();
   patcher.masm()->nop();
@@ -105,7 +105,7 @@
   //   call t9          (jalr t9 / nop instruction pair)
CodePatcher patcher(rinfo()->pc(), Assembler::kDebugBreakSlotInstructions);
   patcher.masm()->li(v8::internal::t9, Operand(reinterpret_cast<int32_t>(
-      Isolate::Current()->debug()->debug_break_slot()->entry())));
+      debug_info_->GetIsolate()->debug()->debug_break_slot()->entry())));
   patcher.masm()->Call(v8::internal::t9);
 }

=======================================
--- /branches/bleeding_edge/src/mips/ic-mips.cc Thu Aug 29 03:29:24 2013 UTC
+++ /branches/bleeding_edge/src/mips/ic-mips.cc Wed Sep 11 14:36:27 2013 UTC
@@ -359,7 +359,7 @@
                                          extra_state,
                                          Code::NORMAL,
                                          argc);
-  Isolate::Current()->stub_cache()->GenerateProbe(
+  masm->isolate()->stub_cache()->GenerateProbe(
       masm, flags, a1, a2, a3, t0, t1, t2);

   // If the stub cache probing failed, the receiver might be a value.
@@ -395,7 +395,7 @@

   // Probe the stub cache for the value object.
   __ bind(&probe);
-  Isolate::Current()->stub_cache()->GenerateProbe(
+  masm->isolate()->stub_cache()->GenerateProbe(
       masm, flags, a1, a2, a3, t0, t1, t2);

   __ bind(&miss);
@@ -658,7 +658,7 @@
   Code::Flags flags = Code::ComputeFlags(
       Code::STUB, MONOMORPHIC, Code::kNoExtraICState,
       Code::NORMAL, Code::LOAD_IC);
-  Isolate::Current()->stub_cache()->GenerateProbe(
+  masm->isolate()->stub_cache()->GenerateProbe(
       masm, flags, a0, a2, a3, t0, t1, t2);

   // Cache miss: Jump to runtime.
@@ -1498,7 +1498,7 @@
   Code::Flags flags = Code::ComputeFlags(
       Code::STUB, MONOMORPHIC, strict_mode,
       Code::NORMAL, Code::STORE_IC);
-  Isolate::Current()->stub_cache()->GenerateProbe(
+  masm->isolate()->stub_cache()->GenerateProbe(
       masm, flags, a1, a2, a3, t0, t1, t2);

   // Cache miss: Jump to runtime.
=======================================
--- /branches/bleeding_edge/src/mips/regexp-macro-assembler-mips.cc Fri Jun 28 15:34:48 2013 UTC +++ /branches/bleeding_edge/src/mips/regexp-macro-assembler-mips.cc Wed Sep 11 14:36:27 2013 UTC
@@ -882,7 +882,7 @@
   masm_->GetCode(&code_desc);
   Handle<Code> code = isolate()->factory()->NewCode(
       code_desc, Code::ComputeFlags(Code::REGEXP), masm_->CodeObject());
-  LOG(Isolate::Current(), RegExpCodeCreateEvent(*code, *source));
+  LOG(masm_->isolate(), RegExpCodeCreateEvent(*code, *source));
   return Handle<HeapObject>::cast(code);
 }

@@ -1086,7 +1086,6 @@
                                                    Code* re_code,
                                                    Address re_frame) {
   Isolate* isolate = frame_entry<Isolate*>(re_frame, kIsolate);
-  ASSERT(isolate == Isolate::Current());
   if (isolate->stack_guard()->IsStackOverflow()) {
     isolate->StackOverflow();
     return EXCEPTION;

--
--
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/groups/opt_out.

Reply via email to