| Index: src/arm/code-stubs-arm.cc
|
| diff --git a/src/arm/code-stubs-arm.cc b/src/arm/code-stubs-arm.cc
|
| index f1e8c6d53b5d3ffd07b37e20dbef4ced4b8d54b9..3391686e9f1e01ac523599daac0ed53b200ebb12 100644
|
| --- a/src/arm/code-stubs-arm.cc
|
| +++ b/src/arm/code-stubs-arm.cc
|
| @@ -688,19 +688,19 @@ void CompareICStub::GenerateGeneric(MacroAssembler* masm) {
|
| masm, lhs, rhs, &flat_string_check, &slow);
|
| }
|
|
|
| - // Check for both being sequential ASCII strings, and inline if that is the
|
| - // case.
|
| + // Check for both being sequential one-byte strings,
|
| + // and inline if that is the case.
|
| __ bind(&flat_string_check);
|
|
|
| - __ JumpIfNonSmisNotBothSequentialAsciiStrings(lhs, rhs, r2, r3, &slow);
|
| + __ JumpIfNonSmisNotBothSequentialOneByteStrings(lhs, rhs, r2, r3, &slow);
|
|
|
| __ IncrementCounter(isolate()->counters()->string_compare_native(), 1, r2,
|
| r3);
|
| if (cc == eq) {
|
| - StringHelper::GenerateFlatAsciiStringEquals(masm, lhs, rhs, r2, r3, r4);
|
| + StringHelper::GenerateFlatOneByteStringEquals(masm, lhs, rhs, r2, r3, r4);
|
| } else {
|
| - StringHelper::GenerateCompareFlatAsciiStrings(masm, lhs, rhs, r2, r3, r4,
|
| - r5);
|
| + StringHelper::GenerateCompareFlatOneByteStrings(masm, lhs, rhs, r2, r3, r4,
|
| + r5);
|
| }
|
| // Never falls through to here.
|
|
|
| @@ -2037,7 +2037,8 @@ void RegExpExecStub::Generate(MacroAssembler* masm) {
|
| STATIC_ASSERT(kTwoByteStringTag == 0);
|
| __ and_(r0, r0, Operand(kStringEncodingMask));
|
| __ mov(r3, Operand(r0, ASR, 2), SetCC);
|
| - __ ldr(r6, FieldMemOperand(regexp_data, JSRegExp::kDataAsciiCodeOffset), ne);
|
| + __ ldr(r6, FieldMemOperand(regexp_data, JSRegExp::kDataOneByteCodeOffset),
|
| + ne);
|
| __ ldr(r6, FieldMemOperand(regexp_data, JSRegExp::kDataUC16CodeOffset), eq);
|
|
|
| // (E) Carry on. String handling is done.
|
| @@ -2048,7 +2049,7 @@ void RegExpExecStub::Generate(MacroAssembler* masm) {
|
| __ JumpIfSmi(r6, &runtime);
|
|
|
| // r1: previous index
|
| - // r3: encoding of subject string (1 if ASCII, 0 if two_byte);
|
| + // r3: encoding of subject string (1 if one_byte, 0 if two_byte);
|
| // r6: code
|
| // subject: Subject string
|
| // regexp_data: RegExp data (FixedArray)
|
| @@ -2091,7 +2092,7 @@ void RegExpExecStub::Generate(MacroAssembler* masm) {
|
| __ str(r0, MemOperand(sp, 1 * kPointerSize));
|
|
|
| // For arguments 4 and 3 get string length, calculate start of string data and
|
| - // calculate the shift of the index (0 for ASCII and 1 for two byte).
|
| + // calculate the shift of the index (0 for one-byte and 1 for two-byte).
|
| __ add(r7, subject, Operand(SeqString::kHeaderSize - kHeapObjectTag));
|
| __ eor(r3, r3, Operand(1));
|
| // Load the length from the original subject string from the previous stack
|
| @@ -2724,11 +2725,6 @@ void CallICStub::GenerateMiss(MacroAssembler* masm, IC::UtilityId id) {
|
|
|
| // StringCharCodeAtGenerator
|
| void StringCharCodeAtGenerator::GenerateFast(MacroAssembler* masm) {
|
| - Label flat_string;
|
| - Label ascii_string;
|
| - Label got_char_code;
|
| - Label sliced_string;
|
| -
|
| // If the receiver is a smi trigger the non-string case.
|
| __ JumpIfSmi(object_, receiver_not_string_);
|
|
|
| @@ -2827,7 +2823,7 @@ void StringCharFromCodeGenerator::GenerateFast(MacroAssembler* masm) {
|
| __ b(ne, &slow_case_);
|
|
|
| __ LoadRoot(result_, Heap::kSingleCharacterStringCacheRootIndex);
|
| - // At this point code register contains smi tagged ASCII char code.
|
| + // At this point code register contains smi tagged one-byte char code.
|
| __ add(result_, result_, Operand::PointerOffsetFromSmiKey(code_));
|
| __ ldr(result_, FieldMemOperand(result_, FixedArray::kHeaderSize));
|
| __ CompareRoot(result_, Heap::kUndefinedValueRootIndex);
|
| @@ -2853,10 +2849,7 @@ void StringCharFromCodeGenerator::GenerateSlow(
|
| }
|
|
|
|
|
| -enum CopyCharactersFlags {
|
| - COPY_ASCII = 1,
|
| - DEST_ALWAYS_ALIGNED = 2
|
| -};
|
| +enum CopyCharactersFlags { COPY_ONE_BYTE = 1, DEST_ALWAYS_ALIGNED = 2 };
|
|
|
|
|
| void StringHelper::GenerateCopyCharacters(MacroAssembler* masm,
|
| @@ -3017,7 +3010,7 @@ void SubStringStub::Generate(MacroAssembler* masm) {
|
| STATIC_ASSERT((kStringEncodingMask & kTwoByteStringTag) == 0);
|
| __ tst(r1, Operand(kStringEncodingMask));
|
| __ b(eq, &two_byte_slice);
|
| - __ AllocateAsciiSlicedString(r0, r2, r6, r4, &runtime);
|
| + __ AllocateOneByteSlicedString(r0, r2, r6, r4, &runtime);
|
| __ jmp(&set_slice_header);
|
| __ bind(&two_byte_slice);
|
| __ AllocateTwoByteSlicedString(r0, r2, r6, r4, &runtime);
|
| @@ -3060,8 +3053,8 @@ void SubStringStub::Generate(MacroAssembler* masm) {
|
| __ tst(r1, Operand(kStringEncodingMask));
|
| __ b(eq, &two_byte_sequential);
|
|
|
| - // Allocate and copy the resulting ASCII string.
|
| - __ AllocateAsciiString(r0, r2, r4, r6, r1, &runtime);
|
| + // Allocate and copy the resulting one-byte string.
|
| + __ AllocateOneByteString(r0, r2, r4, r6, r1, &runtime);
|
|
|
| // Locate first character of substring to copy.
|
| __ add(r5, r5, r3);
|
| @@ -3120,11 +3113,9 @@ void SubStringStub::Generate(MacroAssembler* masm) {
|
| }
|
|
|
|
|
| -void StringHelper::GenerateFlatAsciiStringEquals(MacroAssembler* masm,
|
| - Register left, Register right,
|
| - Register scratch1,
|
| - Register scratch2,
|
| - Register scratch3) {
|
| +void StringHelper::GenerateFlatOneByteStringEquals(
|
| + MacroAssembler* masm, Register left, Register right, Register scratch1,
|
| + Register scratch2, Register scratch3) {
|
| Register length = scratch1;
|
|
|
| // Compare lengths.
|
| @@ -3148,9 +3139,8 @@ void StringHelper::GenerateFlatAsciiStringEquals(MacroAssembler* masm,
|
|
|
| // Compare characters.
|
| __ bind(&compare_chars);
|
| - GenerateAsciiCharsCompareLoop(masm,
|
| - left, right, length, scratch2, scratch3,
|
| - &strings_not_equal);
|
| + GenerateOneByteCharsCompareLoop(masm, left, right, length, scratch2, scratch3,
|
| + &strings_not_equal);
|
|
|
| // Characters are equal.
|
| __ mov(r0, Operand(Smi::FromInt(EQUAL)));
|
| @@ -3158,7 +3148,7 @@ void StringHelper::GenerateFlatAsciiStringEquals(MacroAssembler* masm,
|
| }
|
|
|
|
|
| -void StringHelper::GenerateCompareFlatAsciiStrings(
|
| +void StringHelper::GenerateCompareFlatOneByteStrings(
|
| MacroAssembler* masm, Register left, Register right, Register scratch1,
|
| Register scratch2, Register scratch3, Register scratch4) {
|
| Label result_not_equal, compare_lengths;
|
| @@ -3174,9 +3164,8 @@ void StringHelper::GenerateCompareFlatAsciiStrings(
|
| __ b(eq, &compare_lengths);
|
|
|
| // Compare loop.
|
| - GenerateAsciiCharsCompareLoop(masm,
|
| - left, right, min_length, scratch2, scratch4,
|
| - &result_not_equal);
|
| + GenerateOneByteCharsCompareLoop(masm, left, right, min_length, scratch2,
|
| + scratch4, &result_not_equal);
|
|
|
| // Compare lengths - strings up to min-length are equal.
|
| __ bind(&compare_lengths);
|
| @@ -3192,7 +3181,7 @@ void StringHelper::GenerateCompareFlatAsciiStrings(
|
| }
|
|
|
|
|
| -void StringHelper::GenerateAsciiCharsCompareLoop(
|
| +void StringHelper::GenerateOneByteCharsCompareLoop(
|
| MacroAssembler* masm, Register left, Register right, Register length,
|
| Register scratch1, Register scratch2, Label* chars_not_equal) {
|
| // Change index to run from -length to -1 by adding length to string
|
| @@ -3240,13 +3229,13 @@ void StringCompareStub::Generate(MacroAssembler* masm) {
|
|
|
| __ bind(¬_same);
|
|
|
| - // Check that both objects are sequential ASCII strings.
|
| - __ JumpIfNotBothSequentialAsciiStrings(r1, r0, r2, r3, &runtime);
|
| + // Check that both objects are sequential one-byte strings.
|
| + __ JumpIfNotBothSequentialOneByteStrings(r1, r0, r2, r3, &runtime);
|
|
|
| - // Compare flat ASCII strings natively. Remove arguments from stack first.
|
| + // Compare flat one-byte strings natively. Remove arguments from stack first.
|
| __ IncrementCounter(counters->string_compare_native(), 1, r2, r3);
|
| __ add(sp, sp, Operand(2 * kPointerSize));
|
| - StringHelper::GenerateCompareFlatAsciiStrings(masm, r1, r0, r2, r3, r4, r5);
|
| + StringHelper::GenerateCompareFlatOneByteStrings(masm, r1, r0, r2, r3, r4, r5);
|
|
|
| // Call the runtime; it returns -1 (less), 0 (equal), or 1 (greater)
|
| // tagged as a small integer.
|
| @@ -3512,18 +3501,18 @@ void CompareICStub::GenerateStrings(MacroAssembler* masm) {
|
| __ Ret(eq);
|
| }
|
|
|
| - // Check that both strings are sequential ASCII.
|
| + // Check that both strings are sequential one-byte.
|
| Label runtime;
|
| - __ JumpIfBothInstanceTypesAreNotSequentialAscii(
|
| - tmp1, tmp2, tmp3, tmp4, &runtime);
|
| + __ JumpIfBothInstanceTypesAreNotSequentialOneByte(tmp1, tmp2, tmp3, tmp4,
|
| + &runtime);
|
|
|
| - // Compare flat ASCII strings. Returns when done.
|
| + // Compare flat one-byte strings. Returns when done.
|
| if (equality) {
|
| - StringHelper::GenerateFlatAsciiStringEquals(masm, left, right, tmp1, tmp2,
|
| - tmp3);
|
| + StringHelper::GenerateFlatOneByteStringEquals(masm, left, right, tmp1, tmp2,
|
| + tmp3);
|
| } else {
|
| - StringHelper::GenerateCompareFlatAsciiStrings(masm, left, right, tmp1, tmp2,
|
| - tmp3, tmp4);
|
| + StringHelper::GenerateCompareFlatOneByteStrings(masm, left, right, tmp1,
|
| + tmp2, tmp3, tmp4);
|
| }
|
|
|
| // Handle more complex cases in runtime.
|
|
|