Introduce code serializer/deserializer.

src/parser.cc

Index: src/parser.cc
diff --git a/src/parser.cc b/src/parser.cc
index 261de4eed78e2021a3c8ff4dfa21dd7513bcb977..4ab9659dd2ff662f3fc4d3e09b1eb38df2cf0aee 100644
--- a/src/parser.cc
+++ b/src/parser.cc
@@ -182,7 +182,7 @@ void RegExpBuilder::AddQuantifierToAtom(
 }
 
 
-ScriptData* ScriptData::New(const char* data, int length) {
+ScriptData* ScriptData::New(const char* data, int length, bool owns_store) {
   // The length is obviously invalid.
   if (length % sizeof(unsigned) != 0) {
     return NULL;
@@ -190,7 +190,8 @@ ScriptData* ScriptData::New(const char* data, int length) {
 
   int deserialized_data_length = length / sizeof(unsigned);
   unsigned* deserialized_data;
-  bool owns_store = reinterpret_cast<intptr_t>(data) % sizeof(unsigned) != 0;
+  owns_store =
+      owns_store || reinterpret_cast<intptr_t>(data) % sizeof(unsigned) != 0;

[vogelheim, 2014/07/07 15:20:00]

The naming is weird: I had read the 'owns_store' parameter as a hint that
ScriptData::New should take posession (own) the data that's passed in. But the
implementation is the opposite: If owns_store is set, ScriptData will always
make a copy and hence not 'own' the passed in data, but will instead own its own
copy.

Maybe a better name would be 'always_copy'?

[Yang, 2014/07/08 09:01:05]

I'll address this in a follow-up CL.

   if (owns_store) {
     // Copy the data to align it.
     deserialized_data = i::NewArray<unsigned>(deserialized_data_length);