Profiler: resolve top of stack address to a function

src/profile-generator.cc

 // Copyright 2012 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "src/profile-generator.h"
 
 #include "src/compiler.h"
 #include "src/debug/debug.h"
 #include "src/deoptimizer.h"
 #include "src/global-handles.h"
@@ skipping 619 matching lines @@
             pc_entry->builtin_id() == Builtins::kFunctionApply) {
           // When current function is FunctionCall or FunctionApply builtin the
           // top frame is either frame of the calling JS function or internal
           // frame. In the latter case we know the caller for sure but in the
           // former case we don't so we simply replace the frame with
           // 'unresolved' entry.
           if (sample.top_frame_type == StackFrame::JAVA_SCRIPT) {
             *entry++ = unresolved_entry_;
           }
         }
+      } else if (sample.top_frame_type == StackFrame::JAVA_SCRIPT ||
+                 sample.top_frame_type == StackFrame::OPTIMIZED) {
+        // Find out, if top of stack was pointing inside a JS function
+        // meaning that we have encountered a frameless invocation.
+        Address start;
+        *entry++ = code_map_.FindEntry(sample.tos, &start);

[alph, 2015/09/10 06:20:36]

You can omit the second argument.

[yurys, 2015/09/10 15:58:49]

Done. Also I did more clean-up in another CL:
https://codereview.chromium.org/1332683002/

       }
     }
 
     for (const Address* stack_pos = sample.stack,
            *stack_end = stack_pos + sample.frames_count;
          stack_pos != stack_end;
          ++stack_pos) {
       Address start = NULL;
       *entry = code_map_.FindEntry(*stack_pos, &start);
 
@@ skipping 44 matching lines @@
     case EXTERNAL:
       return program_entry_;
     case IDLE:
       return idle_entry_;
     default: return NULL;
   }
 }
 
 }  // namespace internal
 }  // namespace v8