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

Unified Diff: src/builtins/mips64/builtins-mips64.cc

Issue 2263533002: [builtins] Migrate DatePrototype_GetField to TurboFan builtin. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Fix C call return type Created 4 years, 3 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
« no previous file with comments | « src/builtins/mips/builtins-mips.cc ('k') | src/builtins/ppc/builtins-ppc.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/builtins/mips64/builtins-mips64.cc
diff --git a/src/builtins/mips64/builtins-mips64.cc b/src/builtins/mips64/builtins-mips64.cc
index 3e70c006f20f44ea0ed3f0e374c8b302053c0706..284a1783744964fddcd1777ca21ea4a3412ea074 100644
--- a/src/builtins/mips64/builtins-mips64.cc
+++ b/src/builtins/mips64/builtins-mips64.cc
@@ -1847,71 +1847,6 @@ void Builtins::Generate_InterpreterOnStackReplacement(MacroAssembler* masm) {
}
// static
-void Builtins::Generate_DatePrototype_GetField(MacroAssembler* masm,
- int field_index) {
- // ----------- S t a t e -------------
- // -- a0 : number of arguments
- // -- a1 : function
- // -- cp : context
- // -- sp[0] : receiver
- // -----------------------------------
-
- // 1. Pop receiver into a0 and check that it's actually a JSDate object.
- Label receiver_not_date;
- {
- __ Pop(a0);
- __ JumpIfSmi(a0, &receiver_not_date);
- __ GetObjectType(a0, t0, t0);
- __ Branch(&receiver_not_date, ne, t0, Operand(JS_DATE_TYPE));
- }
-
- // 2. Load the specified date field, falling back to the runtime as necessary.
- if (field_index == JSDate::kDateValue) {
- __ Ret(USE_DELAY_SLOT);
- __ ld(v0, FieldMemOperand(a0, JSDate::kValueOffset)); // In delay slot.
- } else {
- if (field_index < JSDate::kFirstUncachedField) {
- Label stamp_mismatch;
- __ li(a1, Operand(ExternalReference::date_cache_stamp(masm->isolate())));
- __ ld(a1, MemOperand(a1));
- __ ld(t0, FieldMemOperand(a0, JSDate::kCacheStampOffset));
- __ Branch(&stamp_mismatch, ne, t0, Operand(a1));
- __ Ret(USE_DELAY_SLOT);
- __ ld(v0, FieldMemOperand(
- a0, JSDate::kValueOffset +
- field_index * kPointerSize)); // In delay slot.
- __ bind(&stamp_mismatch);
- }
- FrameScope scope(masm, StackFrame::INTERNAL);
- __ PrepareCallCFunction(2, t0);
- __ li(a1, Operand(Smi::FromInt(field_index)));
- __ CallCFunction(
- ExternalReference::get_date_field_function(masm->isolate()), 2);
- }
- __ Ret();
-
- // 3. Raise a TypeError if the receiver is not a date.
- __ bind(&receiver_not_date);
- {
- FrameScope scope(masm, StackFrame::MANUAL);
- __ Push(a0);
- __ Move(a0, Smi::FromInt(0));
- __ EnterBuiltinFrame(cp, a1, a0);
- __ CallRuntime(Runtime::kThrowNotDateError);
-
- // It's far from obvious, but this final trailing instruction after the call
- // is required for StackFrame::LookupCode to work correctly. To illustrate
- // why: if call were the final instruction in the code object, then the pc
- // (== return address) would point beyond the code object when the stack is
- // traversed. When we then try to look up the code object through
- // StackFrame::LookupCode, we actually return the next code object that
- // happens to be on the same page in memory.
- // TODO(jgruber): A proper fix for this would be nice.
- __ nop();
- }
-}
-
-// static
void Builtins::Generate_FunctionPrototypeApply(MacroAssembler* masm) {
// ----------- S t a t e -------------
// -- a0 : argc
« no previous file with comments | « src/builtins/mips/builtins-mips.cc ('k') | src/builtins/ppc/builtins-ppc.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698