Commit e5fc7fe5 authored by mbrandy's avatar mbrandy Committed by Commit bot

PPC: VectorICs: Cleanup, remove unnecessary arguments from HandleArrayCases()

Port c29a4061

R=mvstanton@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com, michael_dawson@ca.ibm.com, dstence@us.ibm.com
BUG=

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

Cr-Commit-Position: refs/heads/master@{#30545}
parent d779181b
...@@ -4569,11 +4569,10 @@ void LoadICStub::GenerateForTrampoline(MacroAssembler* masm) { ...@@ -4569,11 +4569,10 @@ void LoadICStub::GenerateForTrampoline(MacroAssembler* masm) {
} }
static void HandleArrayCases(MacroAssembler* masm, Register receiver, static void HandleArrayCases(MacroAssembler* masm, Register feedback,
Register key, Register vector, Register slot, Register receiver_map, Register scratch1,
Register feedback, Register receiver_map, Register scratch2, bool is_polymorphic,
Register scratch1, Register scratch2, Label* miss) {
bool is_polymorphic, Label* miss) {
// feedback initially contains the feedback array // feedback initially contains the feedback array
Label next_loop, prepare_next; Label next_loop, prepare_next;
Label start_polymorphic; Label start_polymorphic;
...@@ -4690,8 +4689,7 @@ void LoadICStub::GenerateImpl(MacroAssembler* masm, bool in_frame) { ...@@ -4690,8 +4689,7 @@ void LoadICStub::GenerateImpl(MacroAssembler* masm, bool in_frame) {
__ LoadP(scratch1, FieldMemOperand(feedback, HeapObject::kMapOffset)); __ LoadP(scratch1, FieldMemOperand(feedback, HeapObject::kMapOffset));
__ CompareRoot(scratch1, Heap::kFixedArrayMapRootIndex); __ CompareRoot(scratch1, Heap::kFixedArrayMapRootIndex);
__ bne(&not_array); __ bne(&not_array);
HandleArrayCases(masm, receiver, name, vector, slot, feedback, receiver_map, HandleArrayCases(masm, feedback, receiver_map, scratch1, r10, true, &miss);
scratch1, r10, true, &miss);
__ bind(&not_array); __ bind(&not_array);
__ CompareRoot(feedback, Heap::kmegamorphic_symbolRootIndex); __ CompareRoot(feedback, Heap::kmegamorphic_symbolRootIndex);
...@@ -4751,8 +4749,7 @@ void KeyedLoadICStub::GenerateImpl(MacroAssembler* masm, bool in_frame) { ...@@ -4751,8 +4749,7 @@ void KeyedLoadICStub::GenerateImpl(MacroAssembler* masm, bool in_frame) {
// We have a polymorphic element handler. // We have a polymorphic element handler.
Label polymorphic, try_poly_name; Label polymorphic, try_poly_name;
__ bind(&polymorphic); __ bind(&polymorphic);
HandleArrayCases(masm, receiver, key, vector, slot, feedback, receiver_map, HandleArrayCases(masm, feedback, receiver_map, scratch1, r10, true, &miss);
scratch1, r10, true, &miss);
__ bind(&not_array); __ bind(&not_array);
// Is it generic? // Is it generic?
...@@ -4772,8 +4769,7 @@ void KeyedLoadICStub::GenerateImpl(MacroAssembler* masm, bool in_frame) { ...@@ -4772,8 +4769,7 @@ void KeyedLoadICStub::GenerateImpl(MacroAssembler* masm, bool in_frame) {
__ add(feedback, vector, r0); __ add(feedback, vector, r0);
__ LoadP(feedback, __ LoadP(feedback,
FieldMemOperand(feedback, FixedArray::kHeaderSize + kPointerSize)); FieldMemOperand(feedback, FixedArray::kHeaderSize + kPointerSize));
HandleArrayCases(masm, receiver, key, vector, slot, feedback, receiver_map, HandleArrayCases(masm, feedback, receiver_map, scratch1, r10, false, &miss);
scratch1, r10, false, &miss);
__ bind(&miss); __ bind(&miss);
KeyedLoadIC::GenerateMiss(masm); KeyedLoadIC::GenerateMiss(masm);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment