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

Side by Side Diff: src/debug.cc

Issue 172523002: Create a function call IC (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Addressed comments. Created 6 years, 9 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright 2012 the V8 project authors. All rights reserved. 1 // Copyright 2012 the V8 project authors. All rights reserved.
2 // Redistribution and use in source and binary forms, with or without 2 // Redistribution and use in source and binary forms, with or without
3 // modification, are permitted provided that the following conditions are 3 // modification, are permitted provided that the following conditions are
4 // met: 4 // met:
5 // 5 //
6 // * Redistributions of source code must retain the above copyright 6 // * Redistributions of source code must retain the above copyright
7 // notice, this list of conditions and the following disclaimer. 7 // notice, this list of conditions and the following disclaimer.
8 // * Redistributions in binary form must reproduce the above 8 // * Redistributions in binary form must reproduce the above
9 // copyright notice, this list of conditions and the following 9 // copyright notice, this list of conditions and the following
10 // disclaimer in the documentation and/or other materials provided 10 // disclaimer in the documentation and/or other materials provided
(...skipping 388 matching lines...) Expand 10 before | Expand all | Expand 10 after
399 bool BreakLocationIterator::IsStepInLocation(Isolate* isolate) { 399 bool BreakLocationIterator::IsStepInLocation(Isolate* isolate) {
400 if (RelocInfo::IsConstructCall(original_rmode())) { 400 if (RelocInfo::IsConstructCall(original_rmode())) {
401 return true; 401 return true;
402 } else if (RelocInfo::IsCodeTarget(rmode())) { 402 } else if (RelocInfo::IsCodeTarget(rmode())) {
403 HandleScope scope(debug_info_->GetIsolate()); 403 HandleScope scope(debug_info_->GetIsolate());
404 Address target = original_rinfo()->target_address(); 404 Address target = original_rinfo()->target_address();
405 Handle<Code> target_code(Code::GetCodeFromTargetAddress(target)); 405 Handle<Code> target_code(Code::GetCodeFromTargetAddress(target));
406 if (target_code->kind() == Code::STUB) { 406 if (target_code->kind() == Code::STUB) {
407 return target_code->major_key() == CodeStub::CallFunction; 407 return target_code->major_key() == CodeStub::CallFunction;
408 } 408 }
409 return target_code->is_call_stub();
409 } 410 }
410 return false; 411 return false;
411 } 412 }
412 413
413 414
414 void BreakLocationIterator::PrepareStepIn(Isolate* isolate) { 415 void BreakLocationIterator::PrepareStepIn(Isolate* isolate) {
415 #ifdef DEBUG 416 #ifdef DEBUG
416 HandleScope scope(isolate); 417 HandleScope scope(isolate);
417 // Step in can only be prepared if currently positioned on an IC call, 418 // Step in can only be prepared if currently positioned on an IC call,
418 // construct call or CallFunction stub call. 419 // construct call or CallFunction stub call.
(...skipping 998 matching lines...) Expand 10 before | Expand all | Expand 10 after
1417 bool is_load_or_store = false; 1418 bool is_load_or_store = false;
1418 bool is_inline_cache_stub = false; 1419 bool is_inline_cache_stub = false;
1419 bool is_at_restarted_function = false; 1420 bool is_at_restarted_function = false;
1420 Handle<Code> call_function_stub; 1421 Handle<Code> call_function_stub;
1421 1422
1422 if (thread_local_.restarter_frame_function_pointer_ == NULL) { 1423 if (thread_local_.restarter_frame_function_pointer_ == NULL) {
1423 if (RelocInfo::IsCodeTarget(it.rinfo()->rmode())) { 1424 if (RelocInfo::IsCodeTarget(it.rinfo()->rmode())) {
1424 bool is_call_target = false; 1425 bool is_call_target = false;
1425 Address target = it.rinfo()->target_address(); 1426 Address target = it.rinfo()->target_address();
1426 Code* code = Code::GetCodeFromTargetAddress(target); 1427 Code* code = Code::GetCodeFromTargetAddress(target);
1428 if (code->is_call_stub()) {
1429 is_call_target = true;
1430 }
1427 if (code->is_inline_cache_stub()) { 1431 if (code->is_inline_cache_stub()) {
1428 is_inline_cache_stub = true; 1432 is_inline_cache_stub = true;
1429 is_load_or_store = !is_call_target; 1433 is_load_or_store = !is_call_target;
1430 } 1434 }
1431 1435
1432 // Check if target code is CallFunction stub. 1436 // Check if target code is CallFunction stub.
1433 Code* maybe_call_function_stub = code; 1437 Code* maybe_call_function_stub = code;
1434 // If there is a breakpoint at this line look at the original code to 1438 // If there is a breakpoint at this line look at the original code to
1435 // check if it is a CallFunction stub. 1439 // check if it is a CallFunction stub.
1436 if (it.IsDebugBreak()) { 1440 if (it.IsDebugBreak()) {
1437 Address original_target = it.original_rinfo()->target_address(); 1441 Address original_target = it.original_rinfo()->target_address();
1438 maybe_call_function_stub = 1442 maybe_call_function_stub =
1439 Code::GetCodeFromTargetAddress(original_target); 1443 Code::GetCodeFromTargetAddress(original_target);
1440 } 1444 }
1441 if (maybe_call_function_stub->kind() == Code::STUB && 1445 if ((maybe_call_function_stub->kind() == Code::STUB &&
1442 maybe_call_function_stub->major_key() == CodeStub::CallFunction) { 1446 maybe_call_function_stub->major_key() == CodeStub::CallFunction) ||
1447 maybe_call_function_stub->kind() == Code::CALL_IC) {
1443 // Save reference to the code as we may need it to find out arguments 1448 // Save reference to the code as we may need it to find out arguments
1444 // count for 'step in' later. 1449 // count for 'step in' later.
1445 call_function_stub = Handle<Code>(maybe_call_function_stub); 1450 call_function_stub = Handle<Code>(maybe_call_function_stub);
1446 } 1451 }
1447 } 1452 }
1448 } else { 1453 } else {
1449 is_at_restarted_function = true; 1454 is_at_restarted_function = true;
1450 } 1455 }
1451 1456
1452 // If this is the last break code target step out is the only possibility. 1457 // If this is the last break code target step out is the only possibility.
(...skipping 35 matching lines...) Expand 10 before | Expand all | Expand 10 after
1488 } else { 1493 } else {
1489 // If there's restarter frame on top of the stack, just get the pointer 1494 // If there's restarter frame on top of the stack, just get the pointer
1490 // to function which is going to be restarted. 1495 // to function which is going to be restarted.
1491 if (is_at_restarted_function) { 1496 if (is_at_restarted_function) {
1492 Handle<JSFunction> restarted_function( 1497 Handle<JSFunction> restarted_function(
1493 JSFunction::cast(*thread_local_.restarter_frame_function_pointer_)); 1498 JSFunction::cast(*thread_local_.restarter_frame_function_pointer_));
1494 FloodWithOneShot(restarted_function); 1499 FloodWithOneShot(restarted_function);
1495 } else if (!call_function_stub.is_null()) { 1500 } else if (!call_function_stub.is_null()) {
1496 // If it's CallFunction stub ensure target function is compiled and flood 1501 // If it's CallFunction stub ensure target function is compiled and flood
1497 // it with one shot breakpoints. 1502 // it with one shot breakpoints.
1503 bool is_call_ic = call_function_stub->kind() == Code::CALL_IC;
1498 1504
1499 // Find out number of arguments from the stub minor key. 1505 // Find out number of arguments from the stub minor key.
1500 // Reverse lookup required as the minor key cannot be retrieved 1506 // Reverse lookup required as the minor key cannot be retrieved
1501 // from the code object. 1507 // from the code object.
1502 Handle<Object> obj( 1508 Handle<Object> obj(
1503 isolate_->heap()->code_stubs()->SlowReverseLookup( 1509 isolate_->heap()->code_stubs()->SlowReverseLookup(
1504 *call_function_stub), 1510 *call_function_stub),
1505 isolate_); 1511 isolate_);
1506 ASSERT(!obj.is_null()); 1512 ASSERT(!obj.is_null());
1507 ASSERT(!(*obj)->IsUndefined()); 1513 ASSERT(!(*obj)->IsUndefined());
1508 ASSERT(obj->IsSmi()); 1514 ASSERT(obj->IsSmi());
1509 // Get the STUB key and extract major and minor key. 1515 // Get the STUB key and extract major and minor key.
1510 uint32_t key = Smi::cast(*obj)->value(); 1516 uint32_t key = Smi::cast(*obj)->value();
1511 // Argc in the stub is the number of arguments passed - not the 1517 // Argc in the stub is the number of arguments passed - not the
1512 // expected arguments of the called function. 1518 // expected arguments of the called function.
1513 int call_function_arg_count = 1519 int call_function_arg_count = is_call_ic
1514 CallFunctionStub::ExtractArgcFromMinorKey( 1520 ? CallICStub::ExtractArgcFromMinorKey(CodeStub::MinorKeyFromKey(key))
1521 : CallFunctionStub::ExtractArgcFromMinorKey(
1515 CodeStub::MinorKeyFromKey(key)); 1522 CodeStub::MinorKeyFromKey(key));
1516 ASSERT(call_function_stub->major_key() == 1523
1517 CodeStub::MajorKeyFromKey(key)); 1524 ASSERT(is_call_ic ||
1525 call_function_stub->major_key() == CodeStub::MajorKeyFromKey(key));
1518 1526
1519 // Find target function on the expression stack. 1527 // Find target function on the expression stack.
1520 // Expression stack looks like this (top to bottom): 1528 // Expression stack looks like this (top to bottom):
1521 // argN 1529 // argN
1522 // ... 1530 // ...
1523 // arg0 1531 // arg0
1524 // Receiver 1532 // Receiver
1525 // Function to call 1533 // Function to call
1526 int expressions_count = frame->ComputeExpressionsCount(); 1534 int expressions_count = frame->ComputeExpressionsCount();
1527 ASSERT(expressions_count - 2 - call_function_arg_count >= 0); 1535 ASSERT(expressions_count - 2 - call_function_arg_count >= 0);
(...skipping 107 matching lines...) Expand 10 before | Expand all | Expand 10 after
1635 1643
1636 1644
1637 // Find the builtin to use for invoking the debug break 1645 // Find the builtin to use for invoking the debug break
1638 Handle<Code> Debug::FindDebugBreak(Handle<Code> code, RelocInfo::Mode mode) { 1646 Handle<Code> Debug::FindDebugBreak(Handle<Code> code, RelocInfo::Mode mode) {
1639 Isolate* isolate = code->GetIsolate(); 1647 Isolate* isolate = code->GetIsolate();
1640 1648
1641 // Find the builtin debug break function matching the calling convention 1649 // Find the builtin debug break function matching the calling convention
1642 // used by the call site. 1650 // used by the call site.
1643 if (code->is_inline_cache_stub()) { 1651 if (code->is_inline_cache_stub()) {
1644 switch (code->kind()) { 1652 switch (code->kind()) {
1653 case Code::CALL_IC:
1654 return isolate->builtins()->CallICStub_DebugBreak();
1655
1645 case Code::LOAD_IC: 1656 case Code::LOAD_IC:
1646 return isolate->builtins()->LoadIC_DebugBreak(); 1657 return isolate->builtins()->LoadIC_DebugBreak();
1647 1658
1648 case Code::STORE_IC: 1659 case Code::STORE_IC:
1649 return isolate->builtins()->StoreIC_DebugBreak(); 1660 return isolate->builtins()->StoreIC_DebugBreak();
1650 1661
1651 case Code::KEYED_LOAD_IC: 1662 case Code::KEYED_LOAD_IC:
1652 return isolate->builtins()->KeyedLoadIC_DebugBreak(); 1663 return isolate->builtins()->KeyedLoadIC_DebugBreak();
1653 1664
1654 case Code::KEYED_STORE_IC: 1665 case Code::KEYED_STORE_IC:
1655 return isolate->builtins()->KeyedStoreIC_DebugBreak(); 1666 return isolate->builtins()->KeyedStoreIC_DebugBreak();
1656 1667
1657 case Code::COMPARE_NIL_IC: 1668 case Code::COMPARE_NIL_IC:
1658 return isolate->builtins()->CompareNilIC_DebugBreak(); 1669 return isolate->builtins()->CompareNilIC_DebugBreak();
1659 1670
1660 default: 1671 default:
1661 UNREACHABLE(); 1672 UNREACHABLE();
1662 } 1673 }
1663 } 1674 }
1664 if (RelocInfo::IsConstructCall(mode)) { 1675 if (RelocInfo::IsConstructCall(mode)) {
1665 if (code->has_function_cache()) { 1676 if (code->has_function_cache()) {
1666 return isolate->builtins()->CallConstructStub_Recording_DebugBreak(); 1677 return isolate->builtins()->CallConstructStub_Recording_DebugBreak();
1667 } else { 1678 } else {
1668 return isolate->builtins()->CallConstructStub_DebugBreak(); 1679 return isolate->builtins()->CallConstructStub_DebugBreak();
1669 } 1680 }
1670 } 1681 }
1671 if (code->kind() == Code::STUB) { 1682 if (code->kind() == Code::STUB) {
1672 ASSERT(code->major_key() == CodeStub::CallFunction); 1683 ASSERT(code->major_key() == CodeStub::CallFunction);
1673 if (code->has_function_cache()) { 1684 return isolate->builtins()->CallFunctionStub_DebugBreak();
1674 return isolate->builtins()->CallFunctionStub_Recording_DebugBreak();
1675 } else {
1676 return isolate->builtins()->CallFunctionStub_DebugBreak();
1677 }
1678 } 1685 }
1679 1686
1680 UNREACHABLE(); 1687 UNREACHABLE();
1681 return Handle<Code>::null(); 1688 return Handle<Code>::null();
1682 } 1689 }
1683 1690
1684 1691
1685 // Simple function for returning the source positions for active break points. 1692 // Simple function for returning the source positions for active break points.
1686 Handle<Object> Debug::GetSourceBreakLocations( 1693 Handle<Object> Debug::GetSourceBreakLocations(
1687 Handle<SharedFunctionInfo> shared, 1694 Handle<SharedFunctionInfo> shared,
(...skipping 2121 matching lines...) Expand 10 before | Expand all | Expand 10 after
3809 { 3816 {
3810 Locker locker(reinterpret_cast<v8::Isolate*>(isolate_)); 3817 Locker locker(reinterpret_cast<v8::Isolate*>(isolate_));
3811 isolate_->debugger()->CallMessageDispatchHandler(); 3818 isolate_->debugger()->CallMessageDispatchHandler();
3812 } 3819 }
3813 } 3820 }
3814 } 3821 }
3815 3822
3816 #endif // ENABLE_DEBUGGER_SUPPORT 3823 #endif // ENABLE_DEBUGGER_SUPPORT
3817 3824
3818 } } // namespace v8::internal 3825 } } // namespace v8::internal
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698