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

Side by Side Diff: src/ia32/codegen-ia32.cc

Issue 4469002: Improve positions recording for calls. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Created 10 years, 1 month 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 2010 the V8 project authors. All rights reserved. 1 // Copyright 2010 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 3716 matching lines...) Expand 10 before | Expand all | Expand 10 after
3727 } 3727 }
3728 3728
3729 3729
3730 void CodeGenerator::VisitReturnStatement(ReturnStatement* node) { 3730 void CodeGenerator::VisitReturnStatement(ReturnStatement* node) {
3731 ASSERT(!in_spilled_code()); 3731 ASSERT(!in_spilled_code());
3732 Comment cmnt(masm_, "[ ReturnStatement"); 3732 Comment cmnt(masm_, "[ ReturnStatement");
3733 3733
3734 CodeForStatementPosition(node); 3734 CodeForStatementPosition(node);
3735 Load(node->expression()); 3735 Load(node->expression());
3736 Result return_value = frame_->Pop(); 3736 Result return_value = frame_->Pop();
3737 masm()->WriteRecordedPositions(); 3737 masm()->positions_recorder()->WriteRecordedPositions();
3738 if (function_return_is_shadowed_) { 3738 if (function_return_is_shadowed_) {
3739 function_return_.Jump(&return_value); 3739 function_return_.Jump(&return_value);
3740 } else { 3740 } else {
3741 frame_->PrepareForReturn(); 3741 frame_->PrepareForReturn();
3742 if (function_return_.is_bound()) { 3742 if (function_return_.is_bound()) {
3743 // If the function return label is already bound we reuse the 3743 // If the function return label is already bound we reuse the
3744 // code by jumping to the return site. 3744 // code by jumping to the return site.
3745 function_return_.Jump(&return_value); 3745 function_return_.Jump(&return_value);
3746 } else { 3746 } else {
3747 function_return_.Bind(&return_value); 3747 function_return_.Bind(&return_value);
(...skipping 6321 matching lines...) Expand 10 before | Expand all | Expand 10 after
10069 masm.GetCode(&desc); 10069 masm.GetCode(&desc);
10070 // Call the function from C++. 10070 // Call the function from C++.
10071 return FUNCTION_CAST<MemCopyFunction>(buffer); 10071 return FUNCTION_CAST<MemCopyFunction>(buffer);
10072 } 10072 }
10073 10073
10074 #undef __ 10074 #undef __
10075 10075
10076 } } // namespace v8::internal 10076 } } // namespace v8::internal
10077 10077
10078 #endif // V8_TARGET_ARCH_IA32 10078 #endif // V8_TARGET_ARCH_IA32
OLDNEW
« src/assembler.h ('K') | « src/ia32/assembler-ia32.cc ('k') | src/ia32/full-codegen-ia32.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698