Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(271)

Unified Diff: src/objects.cc

Issue 1169103004: [deoptimizer] Basic support inlining based on SharedFunctionInfo. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Fix Jaros comment. Created 5 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « src/mips64/lithium-codegen-mips64.cc ('k') | src/runtime/runtime-function.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/objects.cc
diff --git a/src/objects.cc b/src/objects.cc
index 249597bc1ccf0fc30556043634c6ce48927de3c4..fa98881bb5e9eba0cd11a3f4a23dff9079a2fa71 100644
--- a/src/objects.cc
+++ b/src/objects.cc
@@ -11378,16 +11378,17 @@ void DeoptimizationInputData::DeoptimizationInputDataPrint(
case Translation::JS_FRAME: {
int ast_id = iterator.Next();
- int function_id = iterator.Next();
+ int shared_info_id = iterator.Next();
unsigned height = iterator.Next();
- os << "{ast_id=" << ast_id << ", function=";
- if (function_id != Translation::kSelfLiteralId) {
- Object* function = LiteralArray()->get(function_id);
- os << Brief(JSFunction::cast(function)->shared()->DebugName());
- } else {
- os << "<self>";
- }
- os << ", height=" << height << "}";
+ Object* shared_info = LiteralArray()->get(shared_info_id);
+ os << "{ast_id=" << ast_id << ", function="
+ << Brief(SharedFunctionInfo::cast(shared_info)->DebugName())
+ << ", height=" << height << "}";
+ break;
+ }
+
+ case Translation::JS_FRAME_FUNCTION: {
+ os << "{function}";
break;
}
@@ -11399,21 +11400,21 @@ void DeoptimizationInputData::DeoptimizationInputDataPrint(
case Translation::ARGUMENTS_ADAPTOR_FRAME:
case Translation::CONSTRUCT_STUB_FRAME: {
- int function_id = iterator.Next();
- JSFunction* function =
- JSFunction::cast(LiteralArray()->get(function_id));
+ int shared_info_id = iterator.Next();
+ Object* shared_info = LiteralArray()->get(shared_info_id);
unsigned height = iterator.Next();
- os << "{function=" << Brief(function->shared()->DebugName())
+ os << "{function="
+ << Brief(SharedFunctionInfo::cast(shared_info)->DebugName())
<< ", height=" << height << "}";
break;
}
case Translation::GETTER_STUB_FRAME:
case Translation::SETTER_STUB_FRAME: {
- int function_id = iterator.Next();
- JSFunction* function =
- JSFunction::cast(LiteralArray()->get(function_id));
- os << "{function=" << Brief(function->shared()->DebugName()) << "}";
+ int shared_info_id = iterator.Next();
+ Object* shared_info = LiteralArray()->get(shared_info_id);
+ os << "{function=" << Brief(SharedFunctionInfo::cast(shared_info)
+ ->DebugName()) << "}";
break;
}
« no previous file with comments | « src/mips64/lithium-codegen-mips64.cc ('k') | src/runtime/runtime-function.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698