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

Unified Diff: src/full-codegen/ppc/full-codegen-ppc.cc

Issue 1708523002: [fullcodegen] Remove the hacky %_FastOneByteArrayJoin intrinsic. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 4 years, 10 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/full-codegen/mips64/full-codegen-mips64.cc ('k') | src/full-codegen/x64/full-codegen-x64.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/full-codegen/ppc/full-codegen-ppc.cc
diff --git a/src/full-codegen/ppc/full-codegen-ppc.cc b/src/full-codegen/ppc/full-codegen-ppc.cc
index 78c91096ff60b7004f97bccc9998e98b29d596ef..84095e9548e9f798730ed8d3011289c123675ddc 100644
--- a/src/full-codegen/ppc/full-codegen-ppc.cc
+++ b/src/full-codegen/ppc/full-codegen-ppc.cc
@@ -3439,261 +3439,6 @@ void FullCodeGenerator::EmitGetSuperConstructor(CallRuntime* expr) {
}
-void FullCodeGenerator::EmitFastOneByteArrayJoin(CallRuntime* expr) {
- Label bailout, done, one_char_separator, long_separator, non_trivial_array,
- not_size_one_array, loop, empty_separator_loop, one_char_separator_loop,
- one_char_separator_loop_entry, long_separator_loop;
- ZoneList<Expression*>* args = expr->arguments();
- DCHECK(args->length() == 2);
- VisitForStackValue(args->at(1));
- VisitForAccumulatorValue(args->at(0));
-
- // All aliases of the same register have disjoint lifetimes.
- Register array = r3;
- Register elements = no_reg; // Will be r3.
- Register result = no_reg; // Will be r3.
- Register separator = r4;
- Register array_length = r5;
- Register result_pos = no_reg; // Will be r5
- Register string_length = r6;
- Register string = r7;
- Register element = r8;
- Register elements_end = r9;
- Register scratch1 = r10;
- Register scratch2 = r11;
-
- // Separator operand is on the stack.
- __ pop(separator);
-
- // Check that the array is a JSArray.
- __ JumpIfSmi(array, &bailout);
- __ CompareObjectType(array, scratch1, scratch2, JS_ARRAY_TYPE);
- __ bne(&bailout);
-
- // Check that the array has fast elements.
- __ CheckFastElements(scratch1, scratch2, &bailout);
-
- // If the array has length zero, return the empty string.
- __ LoadP(array_length, FieldMemOperand(array, JSArray::kLengthOffset));
- __ SmiUntag(array_length);
- __ cmpi(array_length, Operand::Zero());
- __ bne(&non_trivial_array);
- __ LoadRoot(r3, Heap::kempty_stringRootIndex);
- __ b(&done);
-
- __ bind(&non_trivial_array);
-
- // Get the FixedArray containing array's elements.
- elements = array;
- __ LoadP(elements, FieldMemOperand(array, JSArray::kElementsOffset));
- array = no_reg; // End of array's live range.
-
- // Check that all array elements are sequential one-byte strings, and
- // accumulate the sum of their lengths, as a smi-encoded value.
- __ li(string_length, Operand::Zero());
- __ addi(element, elements, Operand(FixedArray::kHeaderSize - kHeapObjectTag));
- __ ShiftLeftImm(elements_end, array_length, Operand(kPointerSizeLog2));
- __ add(elements_end, element, elements_end);
- // Loop condition: while (element < elements_end).
- // Live values in registers:
- // elements: Fixed array of strings.
- // array_length: Length of the fixed array of strings (not smi)
- // separator: Separator string
- // string_length: Accumulated sum of string lengths (smi).
- // element: Current array element.
- // elements_end: Array end.
- if (generate_debug_code_) {
- __ cmpi(array_length, Operand::Zero());
- __ Assert(gt, kNoEmptyArraysHereInEmitFastOneByteArrayJoin);
- }
- __ bind(&loop);
- __ LoadP(string, MemOperand(element));
- __ addi(element, element, Operand(kPointerSize));
- __ JumpIfSmi(string, &bailout);
- __ LoadP(scratch1, FieldMemOperand(string, HeapObject::kMapOffset));
- __ lbz(scratch1, FieldMemOperand(scratch1, Map::kInstanceTypeOffset));
- __ JumpIfInstanceTypeIsNotSequentialOneByte(scratch1, scratch2, &bailout);
- __ LoadP(scratch1, FieldMemOperand(string, SeqOneByteString::kLengthOffset));
-
- __ AddAndCheckForOverflow(string_length, string_length, scratch1, scratch2,
- r0);
- __ BranchOnOverflow(&bailout);
-
- __ cmp(element, elements_end);
- __ blt(&loop);
-
- // If array_length is 1, return elements[0], a string.
- __ cmpi(array_length, Operand(1));
- __ bne(&not_size_one_array);
- __ LoadP(r3, FieldMemOperand(elements, FixedArray::kHeaderSize));
- __ b(&done);
-
- __ bind(&not_size_one_array);
-
- // Live values in registers:
- // separator: Separator string
- // array_length: Length of the array.
- // string_length: Sum of string lengths (smi).
- // elements: FixedArray of strings.
-
- // Check that the separator is a flat one-byte string.
- __ JumpIfSmi(separator, &bailout);
- __ LoadP(scratch1, FieldMemOperand(separator, HeapObject::kMapOffset));
- __ lbz(scratch1, FieldMemOperand(scratch1, Map::kInstanceTypeOffset));
- __ JumpIfInstanceTypeIsNotSequentialOneByte(scratch1, scratch2, &bailout);
-
- // Add (separator length times array_length) - separator length to the
- // string_length to get the length of the result string.
- __ LoadP(scratch1,
- FieldMemOperand(separator, SeqOneByteString::kLengthOffset));
- __ sub(string_length, string_length, scratch1);
-#if V8_TARGET_ARCH_PPC64
- __ SmiUntag(scratch1, scratch1);
- __ Mul(scratch2, array_length, scratch1);
- // Check for smi overflow. No overflow if higher 33 bits of 64-bit result are
- // zero.
- __ ShiftRightImm(ip, scratch2, Operand(31), SetRC);
- __ bne(&bailout, cr0);
- __ SmiTag(scratch2, scratch2);
-#else
- // array_length is not smi but the other values are, so the result is a smi
- __ mullw(scratch2, array_length, scratch1);
- __ mulhw(ip, array_length, scratch1);
- // Check for smi overflow. No overflow if higher 33 bits of 64-bit result are
- // zero.
- __ cmpi(ip, Operand::Zero());
- __ bne(&bailout);
- __ cmpwi(scratch2, Operand::Zero());
- __ blt(&bailout);
-#endif
-
- __ AddAndCheckForOverflow(string_length, string_length, scratch2, scratch1,
- r0);
- __ BranchOnOverflow(&bailout);
- __ SmiUntag(string_length);
-
- // Bailout for large object allocations.
- __ Cmpi(string_length, Operand(Page::kMaxRegularHeapObjectSize), r0);
- __ bgt(&bailout);
-
- // Get first element in the array to free up the elements register to be used
- // for the result.
- __ addi(element, elements, Operand(FixedArray::kHeaderSize - kHeapObjectTag));
- result = elements; // End of live range for elements.
- elements = no_reg;
- // Live values in registers:
- // element: First array element
- // separator: Separator string
- // string_length: Length of result string (not smi)
- // array_length: Length of the array.
- __ AllocateOneByteString(result, string_length, scratch1, scratch2,
- elements_end, &bailout);
- // Prepare for looping. Set up elements_end to end of the array. Set
- // result_pos to the position of the result where to write the first
- // character.
- __ ShiftLeftImm(elements_end, array_length, Operand(kPointerSizeLog2));
- __ add(elements_end, element, elements_end);
- result_pos = array_length; // End of live range for array_length.
- array_length = no_reg;
- __ addi(result_pos, result,
- Operand(SeqOneByteString::kHeaderSize - kHeapObjectTag));
-
- // Check the length of the separator.
- __ LoadP(scratch1,
- FieldMemOperand(separator, SeqOneByteString::kLengthOffset));
- __ CmpSmiLiteral(scratch1, Smi::FromInt(1), r0);
- __ beq(&one_char_separator);
- __ bgt(&long_separator);
-
- // Empty separator case
- __ bind(&empty_separator_loop);
- // Live values in registers:
- // result_pos: the position to which we are currently copying characters.
- // element: Current array element.
- // elements_end: Array end.
-
- // Copy next array element to the result.
- __ LoadP(string, MemOperand(element));
- __ addi(element, element, Operand(kPointerSize));
- __ LoadP(string_length, FieldMemOperand(string, String::kLengthOffset));
- __ SmiUntag(string_length);
- __ addi(string, string,
- Operand(SeqOneByteString::kHeaderSize - kHeapObjectTag));
- __ CopyBytes(string, result_pos, string_length, scratch1);
- __ cmp(element, elements_end);
- __ blt(&empty_separator_loop); // End while (element < elements_end).
- DCHECK(result.is(r3));
- __ b(&done);
-
- // One-character separator case
- __ bind(&one_char_separator);
- // Replace separator with its one-byte character value.
- __ lbz(separator, FieldMemOperand(separator, SeqOneByteString::kHeaderSize));
- // Jump into the loop after the code that copies the separator, so the first
- // element is not preceded by a separator
- __ b(&one_char_separator_loop_entry);
-
- __ bind(&one_char_separator_loop);
- // Live values in registers:
- // result_pos: the position to which we are currently copying characters.
- // element: Current array element.
- // elements_end: Array end.
- // separator: Single separator one-byte char (in lower byte).
-
- // Copy the separator character to the result.
- __ stb(separator, MemOperand(result_pos));
- __ addi(result_pos, result_pos, Operand(1));
-
- // Copy next array element to the result.
- __ bind(&one_char_separator_loop_entry);
- __ LoadP(string, MemOperand(element));
- __ addi(element, element, Operand(kPointerSize));
- __ LoadP(string_length, FieldMemOperand(string, String::kLengthOffset));
- __ SmiUntag(string_length);
- __ addi(string, string,
- Operand(SeqOneByteString::kHeaderSize - kHeapObjectTag));
- __ CopyBytes(string, result_pos, string_length, scratch1);
- __ cmpl(element, elements_end);
- __ blt(&one_char_separator_loop); // End while (element < elements_end).
- DCHECK(result.is(r3));
- __ b(&done);
-
- // Long separator case (separator is more than one character). Entry is at the
- // label long_separator below.
- __ bind(&long_separator_loop);
- // Live values in registers:
- // result_pos: the position to which we are currently copying characters.
- // element: Current array element.
- // elements_end: Array end.
- // separator: Separator string.
-
- // Copy the separator to the result.
- __ LoadP(string_length, FieldMemOperand(separator, String::kLengthOffset));
- __ SmiUntag(string_length);
- __ addi(string, separator,
- Operand(SeqOneByteString::kHeaderSize - kHeapObjectTag));
- __ CopyBytes(string, result_pos, string_length, scratch1);
-
- __ bind(&long_separator);
- __ LoadP(string, MemOperand(element));
- __ addi(element, element, Operand(kPointerSize));
- __ LoadP(string_length, FieldMemOperand(string, String::kLengthOffset));
- __ SmiUntag(string_length);
- __ addi(string, string,
- Operand(SeqOneByteString::kHeaderSize - kHeapObjectTag));
- __ CopyBytes(string, result_pos, string_length, scratch1);
- __ cmpl(element, elements_end);
- __ blt(&long_separator_loop); // End while (element < elements_end).
- DCHECK(result.is(r3));
- __ b(&done);
-
- __ bind(&bailout);
- __ LoadRoot(r3, Heap::kUndefinedValueRootIndex);
- __ bind(&done);
- context()->Plug(r3);
-}
-
-
void FullCodeGenerator::EmitDebugIsActive(CallRuntime* expr) {
DCHECK(expr->arguments()->length() == 0);
ExternalReference debug_is_active =
« no previous file with comments | « src/full-codegen/mips64/full-codegen-mips64.cc ('k') | src/full-codegen/x64/full-codegen-x64.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698