Create an abstraction for the string type flags so that they can be cached....

src/jsregexp.cc

 // Copyright 2006-2008 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 111 matching lines @@
   Handle<String> two_byte_string = StringToTwoByte(subject);
   last_ascii_string_ = *subject;
   two_byte_cached_string_ = *two_byte_string;
   return two_byte_string;
 }
 
 
 // Converts a source string to a 16 bit flat string or a SlicedString containing
 // a 16 bit flat string).
 Handle<String> RegExpImpl::StringToTwoByte(Handle<String> pattern) {
-  if (!pattern->IsFlat()) {
+  StringShape shape(*pattern);
+  if (!pattern->IsFlat(shape)) {
     FlattenString(pattern);
   }
-  Handle<String> flat_string(pattern->IsConsString() ?
+  Handle<String> flat_string(shape.IsCons() ?
     String::cast(ConsString::cast(*pattern)->first()) :
     *pattern);
-  ASSERT(!flat_string->IsConsString());
-  ASSERT(flat_string->IsSeqString() || flat_string->IsSlicedString() ||
-         flat_string->IsExternalString());
-  if (!flat_string->IsAsciiRepresentation()) {
+  ASSERT(flat_string->IsString());
+  StringShape flat_shape(*flat_string);
+  ASSERT(!flat_shape.IsCons());
+  ASSERT(flat_shape.IsSequential() ||
+         flat_shape.IsSliced() ||
+         flat_shape.IsExternal());
+  if (!flat_shape.IsAsciiRepresentation()) {
     return flat_string;
   }
 
+  int len = flat_string->length(flat_shape);
   Handle<String> two_byte_string =
-    Factory::NewRawTwoByteString(flat_string->length(), TENURED);
-  static StringInputBuffer convert_to_two_byte_buffer;
-  convert_to_two_byte_buffer.Reset(*flat_string);
-  for (int i = 0; convert_to_two_byte_buffer.has_more(); i++) {
-    two_byte_string->Set(i, convert_to_two_byte_buffer.GetNext());
-  }
+    Factory::NewRawTwoByteString(len, TENURED);
+  uc16* dest = SeqTwoByteString::cast(*two_byte_string)->GetChars();
+  String::WriteToFlat(*flat_string, flat_shape, dest, 0, len);
   return two_byte_string;
 }
 
 
 static JSRegExp::Flags RegExpFlagsFromString(Handle<String> str) {
   int flags = JSRegExp::NONE;
-  for (int i = 0; i < str->length(); i++) {
-    switch (str->Get(i)) {
+  StringShape shape(*str);
+  for (int i = 0; i < str->length(shape); i++) {
+    switch (str->Get(shape, i)) {
       case 'i':
         flags |= JSRegExp::IGNORE_CASE;
         break;
       case 'g':
         flags |= JSRegExp::GLOBAL;
         break;
       case 'm':
         flags |= JSRegExp::MULTILINE;
         break;
     }
   }
   return JSRegExp::Flags(flags);
 }
 
 
 unibrow::Predicate<unibrow::RegExpSpecialChar, 128> is_reg_exp_special_char;
 
 
 Handle<Object> RegExpImpl::Compile(Handle<JSRegExp> re,
                                    Handle<String> pattern,
                                    Handle<String> flag_str) {
   JSRegExp::Flags flags = RegExpFlagsFromString(flag_str);
   Handle<FixedArray> cached = CompilationCache::LookupRegExp(pattern, flags);
   bool in_cache = !cached.is_null();
   Handle<Object> result;
+  StringShape shape(*pattern);
   if (in_cache) {
     re->set_data(*cached);
     result = re;
   } else {
     bool is_atom = !flags.is_ignore_case();
-    for (int i = 0; is_atom && i < pattern->length(); i++) {
-      if (is_reg_exp_special_char.get(pattern->Get(i)))
+    for (int i = 0; is_atom && i < pattern->length(shape); i++) {
+      if (is_reg_exp_special_char.get(pattern->Get(shape, i)))
         is_atom = false;
     }
     if (is_atom) {
       result = AtomCompile(re, pattern, flags);
     } else {
       result = JsreCompile(re, pattern, flags);
     }
     Object* data = re->data();
     if (data->IsFixedArray()) {
       // If compilation succeeded then the data is set on the regexp
@@ skipping 359 matching lines @@
   return Smi::cast(value->get(CAPTURE_INDEX))->value();
 }
 
 
 ByteArray* RegExpImpl::JsreInternal(Handle<JSRegExp> re) {
   FixedArray* value = FixedArray::cast(re->DataAt(JSRegExp::kJscreDataIndex));
   return ByteArray::cast(value->get(INTERNAL_INDEX));
 }
 
 }}  // namespace v8::internal