Merged: [cpu-profiler] use new source position information for deoptimization in cpu profiler

src/source-position.cc

 // Copyright 2016 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/source-position.h"
 #include "src/compilation-info.h"
 #include "src/objects-inl.h"
 
 namespace v8 {
 namespace internal {
 
 std::ostream& operator<<(std::ostream& out, const SourcePositionInfo& pos) {
-  Handle<SharedFunctionInfo> function;
-  if (pos.function.ToHandle(&function)) {
-    Handle<Script> script(Script::cast(function->script()));
-    out << "<";
-    if (script->name()->IsString()) {
-      out << String::cast(script->name())->ToCString(DISALLOW_NULLS).get();
-    } else {
-      out << "unknown";
-    }
-    out << ":" << pos.line + 1 << ":" << pos.column + 1 << ">";
+  Handle<SharedFunctionInfo> function(pos.function);
+  Handle<Script> script(Script::cast(function->script()));
+  out << "<";
+  if (script->name()->IsString()) {
+    out << String::cast(script->name())->ToCString(DISALLOW_NULLS).get();
   } else {
-    out << "<unknown:" << pos.position.ScriptOffset() << ">";
+    out << "unknown";
   }
+  out << ":" << pos.line + 1 << ":" << pos.column + 1 << ">";
   return out;
 }
 
 std::ostream& operator<<(std::ostream& out,
                          const std::vector<SourcePositionInfo>& stack) {
-  out << stack.back();
-  for (int i = static_cast<int>(stack.size()) - 2; i >= 0; --i) {
-    out << " inlined at ";
-    out << stack[i];
+  bool first = true;
+  for (const SourcePositionInfo& pos : stack) {
+    if (!first) out << " inlined at ";
+    out << pos;
+    first = false;
   }
   return out;
 }
 
 std::ostream& operator<<(std::ostream& out, const SourcePosition& pos) {
   if (pos.isInlined()) {
     out << "<inlined(" << pos.InliningId() << "):";
   } else {
     out << "<not inlined:";
   }
   out << pos.ScriptOffset() << ">";
   return out;
 }
 
 SourcePositionInfo SourcePosition::Info(
     Handle<SharedFunctionInfo> function) const {
+  SourcePositionInfo result(*this, function);
   Handle<Script> script(Script::cast(function->script()));
-  SourcePositionInfo result(*this);
   Script::PositionInfo pos;
   if (Script::GetPositionInfo(script, ScriptOffset(), &pos,
                               Script::WITH_OFFSET)) {
     result.line = pos.line;
     result.column = pos.column;
   }
-  result.function = function;
   return result;
 }
 
 std::vector<SourcePositionInfo> SourcePosition::InliningStack(
     CompilationInfo* cinfo) const {
-  if (!isInlined()) {
-    return std::vector<SourcePositionInfo>{Info(cinfo->shared_info())};
-  } else {
-    InliningPosition inl = cinfo->inlined_functions()[InliningId()].position;
-    std::vector<SourcePositionInfo> stack = inl.position.InliningStack(cinfo);
-    stack.push_back(Info(cinfo->inlined_functions()[InliningId()].shared_info));
-    return stack;
+  SourcePosition pos = *this;
+  std::vector<SourcePositionInfo> stack;
+  while (pos.isInlined()) {
+    const auto& inl = cinfo->inlined_functions()[pos.InliningId()];
+    stack.push_back(pos.Info(inl.shared_info));
+    pos = inl.position.position;
   }
+  stack.push_back(pos.Info(cinfo->shared_info()));
+  return stack;
 }
 
 std::vector<SourcePositionInfo> SourcePosition::InliningStack(
     Handle<Code> code) const {
   Handle<DeoptimizationInputData> deopt_data(
       DeoptimizationInputData::cast(code->deoptimization_data()));
-  if (!isInlined()) {
+  SourcePosition pos = *this;
+  std::vector<SourcePositionInfo> stack;
+  while (pos.isInlined()) {
+    InliningPosition inl =
+        deopt_data->InliningPositions()->get(pos.InliningId());
     Handle<SharedFunctionInfo> function(
-        SharedFunctionInfo::cast(deopt_data->SharedFunctionInfo()));
-
-    return std::vector<SourcePositionInfo>{Info(function)};
-  } else {
-    InliningPosition inl = deopt_data->InliningPositions()->get(InliningId());
-    std::vector<SourcePositionInfo> stack = inl.position.InliningStack(code);
-    if (inl.inlined_function_id == -1) {
-      stack.push_back(SourcePositionInfo(*this));
-    } else {
-      Handle<SharedFunctionInfo> function(SharedFunctionInfo::cast(
-          deopt_data->LiteralArray()->get(inl.inlined_function_id)));
-      stack.push_back(Info(function));
-    }
-    return stack;
+        deopt_data->GetInlinedFunction(inl.inlined_function_id));
+    stack.push_back(pos.Info(function));
+    pos = inl.position;
   }
+  Handle<SharedFunctionInfo> function(
+      SharedFunctionInfo::cast(deopt_data->SharedFunctionInfo()));
+  stack.push_back(pos.Info(function));
+  return stack;
 }
 
 void SourcePosition::Print(std::ostream& out,
                            SharedFunctionInfo* function) const {
   Script* script = Script::cast(function->script());
   Object* source_name = script->name();
   Script::PositionInfo pos;
   script->GetPositionInfo(ScriptOffset(), &pos, Script::WITH_OFFSET);
   out << "<";
   if (source_name->IsString()) {
@@ skipping 11 matching lines @@
       DeoptimizationInputData::cast(code->deoptimization_data());
   if (!isInlined()) {
     SharedFunctionInfo* function(
         SharedFunctionInfo::cast(deopt_data->SharedFunctionInfo()));
     Print(out, function);
   } else {
     InliningPosition inl = deopt_data->InliningPositions()->get(InliningId());
     if (inl.inlined_function_id == -1) {
       out << *this;
     } else {
-      SharedFunctionInfo* function = SharedFunctionInfo::cast(
-          deopt_data->LiteralArray()->get(inl.inlined_function_id));
+      SharedFunctionInfo* function =
+          deopt_data->GetInlinedFunction(inl.inlined_function_id);
       Print(out, function);
     }
     out << " inlined at ";
     inl.position.Print(out, code);
   }
 }
 
 }  // namespace internal
 }  // namespace v8