Introduce short external strings without pointer cache.

src/ia32/codegen-ia32.cc

 // Copyright 2011 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 514 matching lines @@
 }
 
 
 void StringCharLoadGenerator::Generate(MacroAssembler* masm,
                                        Factory* factory,
                                        Register string,
                                        Register index,
                                        Register result,
                                        Label* call_runtime) {
   // Fetch the instance type of the receiver into result register.
   __ mov(result, FieldOperand(string, HeapObject::kMapOffset));

[Lasse Reichstein, 2011/11/23 08:43:20]

We're absolutely sure that only strings reach here, right?

[Yang, 2011/11/23 09:58:48]

Yes.

   __ movzx_b(result, FieldOperand(result, Map::kInstanceTypeOffset));
 
   // We need special handling for indirect strings.
   Label check_sequential;
   __ test(result, Immediate(kIsIndirectStringMask));
   __ j(zero, &check_sequential);
 
   // Dispatch on the indirect string shape: slice or cons.
   Label cons_string;
   __ test(result, Immediate(kSlicedNotConsMask));
@@ skipping 10 matching lines @@
   // Handle external strings.
   Label external_string, ascii_external, done;
   __ bind(&external_string);
   if (FLAG_debug_code) {
     // Assert that we do not have a cons or slice (indirect strings) here.
     // Sequential strings have already been ruled out.
     __ test(result, Immediate(kIsIndirectStringMask));
     __ Assert(zero, "external string expected, but not found");
   }
   __ mov(result, FieldOperand(string, ExternalString::kResourceDataOffset));
-  // Assert that the external string has not been finalized yet.
-  __ test(result, result);
-  __ j(zero, call_runtime);
   Register scratch = string;
   __ mov(scratch, FieldOperand(string, HeapObject::kMapOffset));
-  __ cmp(scratch, Immediate(factory->external_ascii_string_map()));
-  __ j(equal, &ascii_external, Label::kNear);
-  __ cmp(scratch, Immediate(factory->external_ascii_symbol_map()));
-  __ j(equal, &ascii_external, Label::kNear);
+  __ movzx_b(scratch, FieldOperand(scratch, Map::kInstanceTypeOffset));
+  // Rule out short external strings.
+  STATIC_CHECK(kShortExternalStringTag != 0);
+  __ test_b(scratch, kShortExternalStringTag);

[Lasse Reichstein, 2011/11/23 08:43:20]

Test with kShortExternalStringMask instead of ...Tag.

[Yang, 2011/11/23 09:58:48]

Done.

+  __ j(not_zero, call_runtime);
+  // Check encoding.
+  STATIC_ASSERT(kTwoByteStringTag == 0);
+  __ test_b(scratch, kStringEncodingMask);
+  __ j(not_equal, &ascii_external, Label::kNear);
   // Two-byte string.
   __ movzx_w(result, Operand(result, index, times_2, 0));
   __ jmp(&done);
   __ bind(&ascii_external);
   // Ascii string.
   __ movzx_b(result, Operand(result, index, times_1, 0));
   __ jmp(&done);
 
   // Handle conses.
   // Check whether the right hand side is the empty string (i.e. if
@@ skipping 44 matching lines @@
                                   times_1,
                                   SeqAsciiString::kHeaderSize));
   __ bind(&done);
 }
 
 #undef __
 
 } }  // namespace v8::internal
 
 #endif  // V8_TARGET_ARCH_IA32