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

Side by Side Diff: src/arm/builtins-arm.cc

Issue 14738009: Ensure that soft-deopts don't count against opt_count (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Implement x64 and ARM, review feedback Created 7 years, 7 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
« no previous file with comments | « no previous file | src/arm/deoptimizer-arm.cc » ('j') | src/x64/builtins-x64.cc » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 1322 matching lines...) Expand 10 before | Expand all | Expand 10 after
1333 __ bind(&unknown_state); 1333 __ bind(&unknown_state);
1334 __ stop("no cases left"); 1334 __ stop("no cases left");
1335 } 1335 }
1336 1336
1337 1337
1338 void Builtins::Generate_NotifyDeoptimized(MacroAssembler* masm) { 1338 void Builtins::Generate_NotifyDeoptimized(MacroAssembler* masm) {
1339 Generate_NotifyDeoptimizedHelper(masm, Deoptimizer::EAGER); 1339 Generate_NotifyDeoptimizedHelper(masm, Deoptimizer::EAGER);
1340 } 1340 }
1341 1341
1342 1342
1343 void Builtins::Generate_NotifySoftDeoptimized(MacroAssembler* masm) {
1344 Generate_NotifyDeoptimizedHelper(masm, Deoptimizer::LAZY);
Jakob Kummerow 2013/05/14 11:39:05 same here
1345 }
1346
1347
1343 void Builtins::Generate_NotifyLazyDeoptimized(MacroAssembler* masm) { 1348 void Builtins::Generate_NotifyLazyDeoptimized(MacroAssembler* masm) {
1344 Generate_NotifyDeoptimizedHelper(masm, Deoptimizer::LAZY); 1349 Generate_NotifyDeoptimizedHelper(masm, Deoptimizer::LAZY);
1345 } 1350 }
1346 1351
1347 1352
1348 void Builtins::Generate_NotifyOSR(MacroAssembler* masm) { 1353 void Builtins::Generate_NotifyOSR(MacroAssembler* masm) {
1349 // For now, we are relying on the fact that Runtime::NotifyOSR 1354 // For now, we are relying on the fact that Runtime::NotifyOSR
1350 // doesn't do any garbage collection which allows us to save/restore 1355 // doesn't do any garbage collection which allows us to save/restore
1351 // the registers without worrying about which of them contain 1356 // the registers without worrying about which of them contain
1352 // pointers. This seems a bit fragile. 1357 // pointers. This seems a bit fragile.
(...skipping 517 matching lines...) Expand 10 before | Expand all | Expand 10 after
1870 __ bind(&dont_adapt_arguments); 1875 __ bind(&dont_adapt_arguments);
1871 __ Jump(r3); 1876 __ Jump(r3);
1872 } 1877 }
1873 1878
1874 1879
1875 #undef __ 1880 #undef __
1876 1881
1877 } } // namespace v8::internal 1882 } } // namespace v8::internal
1878 1883
1879 #endif // V8_TARGET_ARCH_ARM 1884 #endif // V8_TARGET_ARCH_ARM
OLDNEW
« no previous file with comments | « no previous file | src/arm/deoptimizer-arm.cc » ('j') | src/x64/builtins-x64.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698