Revision: 23807
Author:   [email protected]
Date:     Tue Sep  9 14:11:18 2014 UTC
Log:      MIPS: Do not use CodeStub::FindInCache if not necessary.

Port r23795 (b41f4a7)

Original commit message:
GetCode does the FindInCache check as well.

BUG=
[email protected], [email protected]

Review URL: https://codereview.chromium.org/549403004
https://code.google.com/p/v8/source/detail?r=23807

Modified:
 /branches/bleeding_edge/src/mips/code-stubs-mips.cc
 /branches/bleeding_edge/src/mips64/code-stubs-mips64.cc

=======================================
--- /branches/bleeding_edge/src/mips/code-stubs-mips.cc Mon Sep 8 19:09:21 2014 UTC +++ /branches/bleeding_edge/src/mips/code-stubs-mips.cc Tue Sep 9 14:11:18 2014 UTC
@@ -1081,20 +1081,10 @@


 void CodeStub::GenerateFPStubs(Isolate* isolate) {
+  // Generate if not already in cache.
   SaveFPRegsMode mode = kSaveFPRegs;
-  CEntryStub save_doubles(isolate, 1, mode);
-  StoreBufferOverflowStub stub(isolate, mode);
-  // These stubs might already be in the snapshot, detect that and don't
- // regenerate, which would lead to code stub initialization state being messed
-  // up.
-  Code* save_doubles_code;
-  if (!save_doubles.FindCodeInCache(&save_doubles_code)) {
-    save_doubles_code = *save_doubles.GetCode();
-  }
-  Code* store_buffer_overflow_code;
-  if (!stub.FindCodeInCache(&store_buffer_overflow_code)) {
-      store_buffer_overflow_code = *stub.GetCode();
-  }
+  CEntryStub(isolate, 1, mode).GetCode();
+  StoreBufferOverflowStub(isolate, mode).GetCode();
   isolate->set_fp_stubs_generated(true);
 }

=======================================
--- /branches/bleeding_edge/src/mips64/code-stubs-mips64.cc Mon Sep 8 19:09:21 2014 UTC +++ /branches/bleeding_edge/src/mips64/code-stubs-mips64.cc Tue Sep 9 14:11:18 2014 UTC
@@ -1076,20 +1076,10 @@


 void CodeStub::GenerateFPStubs(Isolate* isolate) {
+  // Generate if not already in cache.
   SaveFPRegsMode mode = kSaveFPRegs;
-  CEntryStub save_doubles(isolate, 1, mode);
-  StoreBufferOverflowStub stub(isolate, mode);
-  // These stubs might already be in the snapshot, detect that and don't
- // regenerate, which would lead to code stub initialization state being messed
-  // up.
-  Code* save_doubles_code;
-  if (!save_doubles.FindCodeInCache(&save_doubles_code)) {
-    save_doubles_code = *save_doubles.GetCode();
-  }
-  Code* store_buffer_overflow_code;
-  if (!stub.FindCodeInCache(&store_buffer_overflow_code)) {
-      store_buffer_overflow_code = *stub.GetCode();
-  }
+  CEntryStub(isolate, 1, mode).GetCode();
+  StoreBufferOverflowStub(isolate, mode).GetCode();
   isolate->set_fp_stubs_generated(true);
 }

--
--
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.

Reply via email to