Chromium Code Reviews| Index: src/arm/code-stubs-arm.cc |
| diff --git a/src/arm/code-stubs-arm.cc b/src/arm/code-stubs-arm.cc |
| index eaad9f293b330eb0936a5dceeb45ae51bb5f840b..f2aad75b4a91973a527e1b89bd2a2f435b8b224a 100644 |
| --- a/src/arm/code-stubs-arm.cc |
| +++ b/src/arm/code-stubs-arm.cc |
| @@ -4346,33 +4346,47 @@ void RegExpExecStub::Generate(MacroAssembler* masm) { |
| __ ldr(r0, FieldMemOperand(subject, HeapObject::kMapOffset)); |
| __ ldrb(r0, FieldMemOperand(r0, Map::kInstanceTypeOffset)); |
| // First check for flat string. |
| - __ tst(r0, Operand(kIsNotStringMask | kStringRepresentationMask)); |
| + __ and_(r1, r0, Operand(kIsNotStringMask | kStringRepresentationMask), SetCC); |
| STATIC_ASSERT((kStringTag | kSeqStringTag) == 0); |
| __ b(eq, &seq_string); |
| // subject: Subject string |
| // regexp_data: RegExp data (FixedArray) |
| - // Check for flat cons string. |
| + // Check for flat cons string or truncated sliced string. |
| // A flat cons string is a cons string where the second part is the empty |
| // string. In that case the subject string is just the first part of the cons |
| // string. Also in this case the first part of the cons string is known to be |
| // a sequential string or an external string. |
| - STATIC_ASSERT(kExternalStringTag !=0); |
| - STATIC_ASSERT((kConsStringTag & kExternalStringTag) == 0); |
| - __ tst(r0, Operand(kIsNotStringMask | kExternalStringTag)); |
| + // A truncated sliced string has the offset 0 and the same length as the |
| + // parent string. |
| + Label cons_string, encoding; |
| + __ cmp(r1, Operand(kConsStringTag)); |
| + __ b(eq, &cons_string); |
| + __ cmp(r1, Operand(kSlicedStringTag)); |
| + // If subject is not a sliced string, it can only be a non-string or an |
| + // external string. |
| __ b(ne, &runtime); |
| + // String is sliced, check whether it is truncated. |
| + __ ldr(r0, FieldMemOperand(subject, SlicedString::kLengthOffset)); |
| + __ ldr(subject, FieldMemOperand(subject, SlicedString::kParentOffset)); |
| + __ ldr(r1, FieldMemOperand(subject, String::kLengthOffset)); |
| + __ cmp(r0, r1); |
| + __ b(ne, &runtime); |
| + __ jmp(&encoding); |
| + // String is a cons string, check whether it is flat. |
| + __ bind(&cons_string); |
| __ ldr(r0, FieldMemOperand(subject, ConsString::kSecondOffset)); |
| __ LoadRoot(r1, Heap::kEmptyStringRootIndex); |
| __ cmp(r0, r1); |
| __ b(ne, &runtime); |
| __ ldr(subject, FieldMemOperand(subject, ConsString::kFirstOffset)); |
| + // Is first part of cons or parent of slice a flat string? |
| + __ bind(&encoding); |
| __ ldr(r0, FieldMemOperand(subject, HeapObject::kMapOffset)); |
| __ ldrb(r0, FieldMemOperand(r0, Map::kInstanceTypeOffset)); |
| - // Is first part a flat string? |
| STATIC_ASSERT(kSeqStringTag == 0); |
| __ tst(r0, Operand(kStringRepresentationMask)); |
| __ b(ne, &runtime); |
| - |
| __ bind(&seq_string); |
| // subject: Subject string |
| // regexp_data: RegExp data (FixedArray) |
| @@ -4775,6 +4789,7 @@ 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_); |
| @@ -4804,7 +4819,10 @@ void StringCharCodeAtGenerator::GenerateFast(MacroAssembler* masm) { |
| __ b(eq, &flat_string); |
| // Handle non-flat strings. |
| - __ tst(result_, Operand(kIsConsStringMask)); |
| + __ and_(result_, result_, Operand(kStringRepresentationMask)); |
| + __ cmp(result_, Operand(kSlicedStringTag)); |
| + __ b(eq, &sliced_string); |
| + __ cmp(result_, Operand(kExternalStringTag)); |
| __ b(eq, &call_runtime_); |
| // ConsString. |
| @@ -4824,6 +4842,15 @@ void StringCharCodeAtGenerator::GenerateFast(MacroAssembler* masm) { |
| STATIC_ASSERT(kSeqStringTag == 0); |
| __ tst(result_, Operand(kStringRepresentationMask)); |
| __ b(ne, &call_runtime_); |
| + __ jmp(&flat_string); |
| + |
| + // SlicedString, unpack and add offset. |
| + __ bind(&sliced_string); |
| + __ ldr(result_, FieldMemOperand(object_, SlicedString::kOffsetOffset)); |
| + __ add(scratch_, scratch_, result_); |
| + __ ldr(object_, FieldMemOperand(object_, SlicedString::kParentOffset)); |
| + __ ldr(result_, FieldMemOperand(object_, HeapObject::kMapOffset)); |
| + __ ldrb(result_, FieldMemOperand(result_, Map::kInstanceTypeOffset)); |
| // Check for 1-byte or 2-byte string. |
| __ bind(&flat_string); |
| @@ -5400,10 +5427,17 @@ void SubStringStub::Generate(MacroAssembler* masm) { |
| // Check bounds and smi-ness. |
| Register to = r6; |
| Register from = r7; |
| + |
| + __ nop(0); // Jumping as first instruction would crash the code generation. |
|
antonm
2011/08/04 12:18:48
nit: shouldn't nop go under if as well?
|
| + if (FLAG_string_slices) { |
| + __ jmp(&runtime); |
| + } |
| + |
| __ Ldrd(to, from, MemOperand(sp, kToOffset)); |
| STATIC_ASSERT(kFromOffset == kToOffset + 4); |
| STATIC_ASSERT(kSmiTag == 0); |
| STATIC_ASSERT(kSmiTagSize + kSmiShiftSize == 1); |
| + |
| // I.e., arithmetic shift right by one un-smi-tags. |
| __ mov(r2, Operand(to, ASR, 1), SetCC); |
| __ mov(r3, Operand(from, ASR, 1), SetCC, cc); |
| @@ -5412,7 +5446,6 @@ void SubStringStub::Generate(MacroAssembler* masm) { |
| __ b(mi, &runtime); // From is negative. |
| // Both to and from are smis. |
| - |
| __ sub(r2, r2, Operand(r3), SetCC); |
| __ b(mi, &runtime); // Fail if from > to. |
| // Special handling of sub-strings of length 1 and 2. One character strings |
| @@ -5918,6 +5951,8 @@ void StringAddStub::Generate(MacroAssembler* masm) { |
| __ tst(r4, Operand(kStringRepresentationMask)); |
| __ tst(r5, Operand(kStringRepresentationMask), eq); |
| __ b(ne, &string_add_runtime); |
| + // We cannot encounter sliced strings here since: |
| + STATIC_ASSERT(SlicedString::kMinLength >= String::kMinNonFlatLength); |
| // Now check if both strings have the same encoding (ASCII/Two-byte). |
| // r0: first string. |
| // r1: second string. |