Introduce accessors on builtins instance

src/builtins.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 1558 matching lines @@
     functions->s_name = #aname;                                        \
     functions->name = c_##aname;                                       \
     functions->flags = Code::ComputeFlags(Code::BUILTIN);              \
     functions->extra_args = aextra_args;                               \
     ++functions;
 
 #define DEF_FUNCTION_PTR_A(aname, kind, state, extra)                       \
     functions->generator = FUNCTION_ADDR(Generate_##aname);                 \
     functions->c_code = NULL;                                               \
     functions->s_name = #aname;                                             \
-    functions->name = aname;                                                \
+    functions->name = k##aname;                                             \
     functions->flags = Code::ComputeFlags(Code::kind,                       \
                                           NOT_IN_LOOP,                      \
                                           state,                            \
                                           extra);                           \
     functions->extra_args = NO_EXTRA_ARGUMENTS;                             \
     ++functions;
 
   BUILTIN_LIST_C(DEF_FUNCTION_PTR_C)
   BUILTIN_LIST_A(DEF_FUNCTION_PTR_A)
   BUILTIN_LIST_DEBUG_A(DEF_FUNCTION_PTR_A)
@@ skipping 88 matching lines @@
       Code* entry = Code::cast(builtins_[i]);
       if (entry->contains(pc)) {
         return names_[i];
       }
     }
   }
   return NULL;
 }
 
 
+#define DEFINE_BUILTIN_ACCESSOR_C(name, ignore)                 \
+  Handle<Code> Builtins::name() {                               \

[Vitaly Repeshko, 2011/03/23 12:13:41]

This shouldn't be indented, I think.

[Mads Ager (chromium), 2011/03/23 12:28:06]

Done.

+    Code** code_address =                                       \
+        reinterpret_cast<Code**>(builtin_address(k##name));     \
+    return Handle<Code>(code_address);                          \
+  }
+#define DEFINE_BUILTIN_ACCESSOR_A(name, kind, state, extra)  \
+  Handle<Code> Builtins::name() {                            \

[Vitaly Repeshko, 2011/03/23 12:13:41]

Ditto.

[Mads Ager (chromium), 2011/03/23 12:28:06]

Done.

+    Code** code_address =                                    \
+        reinterpret_cast<Code**>(builtin_address(k##name));  \
+    return Handle<Code>(code_address);                       \
+  }
+  BUILTIN_LIST_C(DEFINE_BUILTIN_ACCESSOR_C)
+  BUILTIN_LIST_A(DEFINE_BUILTIN_ACCESSOR_A)
+  BUILTIN_LIST_DEBUG_A(DEFINE_BUILTIN_ACCESSOR_A)
+#undef DEFINE_BUILTIN_ACCESSOR_C
+#undef DEFINE_BUILTIN_ACCESSOR_A
+
+
 } }  // namespace v8::internal