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. |