From d4bcbb70895c851fe44d3be0c48d8ff8754e5ef2 Mon Sep 17 00:00:00 2001 From: "plind44@gmail.com" Date: Tue, 18 Feb 2014 19:29:56 +0000 Subject: [PATCH] MIPS: Second attempt at introducing a premonomorphic state in the call target caches. Port: r19457 (9d8d5f3) Original commit message: This time we don't go through the premonomorphic state for the Array call target caches to avoid losing information from allocation sites that aren't only used once, but where the resulting array is used heavily. Patch from Kasper Lund . BUG= R=plind44@gmail.com Review URL: https://codereview.chromium.org/170903002 Patch from Balazs Kilvady . git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@19463 ce2b1a6d-e550-0410-aec6-3dcde31c8c00 --- src/mips/code-stubs-mips.cc | 80 ++++++++++++++++++++++++++------------------- 1 file changed, 46 insertions(+), 34 deletions(-) diff --git a/src/mips/code-stubs-mips.cc b/src/mips/code-stubs-mips.cc index 4b3107f..e83447a 100644 --- a/src/mips/code-stubs-mips.cc +++ b/src/mips/code-stubs-mips.cc @@ -3159,12 +3159,17 @@ static void GenerateRecordCallTarget(MacroAssembler* masm) { // a1 : the function to call // a2 : Feedback vector // a3 : slot in feedback vector (Smi) - Label initialize, done, miss, megamorphic, not_array_function; + Label check_array, initialize_array, initialize_non_array, megamorphic, done; ASSERT_EQ(*TypeFeedbackInfo::MegamorphicSentinel(masm->isolate()), masm->isolate()->heap()->undefined_value()); + Heap::RootListIndex kMegamorphicRootIndex = Heap::kUndefinedValueRootIndex; ASSERT_EQ(*TypeFeedbackInfo::UninitializedSentinel(masm->isolate()), masm->isolate()->heap()->the_hole_value()); + Heap::RootListIndex kUninitializedRootIndex = Heap::kTheHoleValueRootIndex; + ASSERT_EQ(*TypeFeedbackInfo::PremonomorphicSentinel(masm->isolate()), + masm->isolate()->heap()->null_value()); + Heap::RootListIndex kPremonomorphicRootIndex = Heap::kNullValueRootIndex; // Load the cache state into t0. __ sll(t0, a3, kPointerSizeLog2 - kSmiTagSize); @@ -3174,44 +3179,51 @@ static void GenerateRecordCallTarget(MacroAssembler* masm) { // A monomorphic cache hit or an already megamorphic state: invoke the // function without changing the state. __ Branch(&done, eq, t0, Operand(a1)); - - // If we came here, we need to see if we are the array function. - // If we didn't have a matching function, and we didn't find the megamorph - // sentinel, then we have in the slot either some other function or an - // AllocationSite. Do a map check on the object in a3. + __ LoadRoot(at, kMegamorphicRootIndex); + __ Branch(&done, eq, t0, Operand(at)); + + // Check if we're dealing with the Array function or not. + __ LoadArrayFunction(t1); + __ Branch(&check_array, eq, a1, Operand(t1)); + + // Non-array cache: Check the cache state. + __ LoadRoot(at, kPremonomorphicRootIndex); + __ Branch(&initialize_non_array, eq, t0, Operand(at)); + __ LoadRoot(at, kUninitializedRootIndex); + __ Branch(&megamorphic, ne, t0, Operand(at)); + + // Non-array cache: Uninitialized -> premonomorphic. The sentinel is an + // immortal immovable object (null) so no write-barrier is needed. + __ sll(at, a3, kPointerSizeLog2 - kSmiTagSize); + __ Addu(t0, a2, at); + __ LoadRoot(at, kPremonomorphicRootIndex); + __ Branch(USE_DELAY_SLOT, &done); + __ sw(at, FieldMemOperand(t0, FixedArray::kHeaderSize)); // In delay slot. + + // Array cache: Check the cache state to see if we're in a monomorphic + // state where the state object is an AllocationSite object. + __ bind(&check_array); __ lw(t1, FieldMemOperand(t0, 0)); __ LoadRoot(at, Heap::kAllocationSiteMapRootIndex); - __ Branch(&miss, ne, t1, Operand(at)); - - // Make sure the function is the Array() function - __ LoadArrayFunction(t0); - __ Branch(&megamorphic, ne, a1, Operand(t0)); - __ jmp(&done); + __ Branch(&done, eq, t1, Operand(at)); - __ bind(&miss); + // Array cache: Uninitialized or premonomorphic -> monomorphic. + __ LoadRoot(at, kUninitializedRootIndex); + __ Branch(&initialize_array, eq, t0, Operand(at)); + __ LoadRoot(at, kPremonomorphicRootIndex); + __ Branch(&initialize_array, eq, t0, Operand(at)); - // A monomorphic miss (i.e, here the cache is not uninitialized) goes - // megamorphic. - __ LoadRoot(at, Heap::kTheHoleValueRootIndex); - __ Branch(&initialize, eq, t0, Operand(at)); - // MegamorphicSentinel is an immortal immovable object (undefined) so no - // write-barrier is needed. + // Both caches: Monomorphic -> megamorphic. The sentinel is an + // immortal immovable object (undefined) so no write-barrier is needed. __ bind(&megamorphic); __ sll(t0, a3, kPointerSizeLog2 - kSmiTagSize); __ Addu(t0, a2, Operand(t0)); - __ LoadRoot(at, Heap::kUndefinedValueRootIndex); - __ sw(at, FieldMemOperand(t0, FixedArray::kHeaderSize)); - __ jmp(&done); - - // An uninitialized cache is patched with the function or sentinel to - // indicate the ElementsKind if function is the Array constructor. - __ bind(&initialize); - // Make sure the function is the Array() function - __ LoadArrayFunction(t0); - __ Branch(¬_array_function, ne, a1, Operand(t0)); - - // The target function is the Array constructor. - // Create an AllocationSite if we don't already have it, store it in the slot. + __ LoadRoot(at, kMegamorphicRootIndex); + __ Branch(USE_DELAY_SLOT, &done); + __ sw(at, FieldMemOperand(t0, FixedArray::kHeaderSize)); // In delay slot. + + // Array cache: Uninitialized or premonomorphic -> monomorphic. + __ bind(&initialize_array); { FrameScope scope(masm, StackFrame::INTERNAL); const RegList kSavedRegs = @@ -3232,8 +3244,8 @@ static void GenerateRecordCallTarget(MacroAssembler* masm) { } __ Branch(&done); - __ bind(¬_array_function); - + // Non-array cache: Premonomorphic -> monomorphic. + __ bind(&initialize_non_array); __ sll(t0, a3, kPointerSizeLog2 - kSmiTagSize); __ Addu(t0, a2, Operand(t0)); __ Addu(t0, t0, Operand(FixedArray::kHeaderSize - kHeapObjectTag)); -- 2.7.4