Correctly report callstack when current function is FunctionCall builtin

src/profile-generator.h

Index: src/profile-generator.h
diff --git a/src/profile-generator.h b/src/profile-generator.h
index 38451d11b7579d09866960888bd7191d38d03fb0..7af4df9bc0e6712f181b53ff415aada9292c0987 100644
--- a/src/profile-generator.h
+++ b/src/profile-generator.h
@@ -120,6 +120,9 @@ class CodeEntry {
     no_frame_ranges_ = ranges;
   }
 
+  void SetBuiltinId(Builtins::Name id);
+  Builtins::Name builtin_id() const { return builtin_id_; }
+
   void CopyData(const CodeEntry& source);
   uint32_t GetCallUid() const;
   bool IsSameAs(CodeEntry* entry) const;
@@ -128,7 +131,8 @@ class CodeEntry {
   static const char* const kEmptyResourceName;
 
  private:
-  Logger::LogEventsAndTags tag_;
+  Logger::LogEventsAndTags tag_ : 8;
+  Builtins::Name builtin_id_ : 8;

[Jakob Kummerow, 2013/07/01 12:16:06]

Can you add a STATIC_ASSERT(Builtins::builtin_count < 256) or something like
that?

[yurys, 2013/07/01 12:50:16]

Compilation will fail with a error like the following

../src/profile-generator.h:135:32: error: 'v8::internal::CodeEntry::builtin_id_'
is too small to hold all values of 'enum v8::internal::Builtins::Name' [-Werror]

if 8 bits is not enough to fit all enum values so there is no need in extra
check.

   const char* name_prefix_;
   const char* name_;
   const char* resource_name_;