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

Side by Side Diff: src/factory.cc

Issue 178583003: Remove bogus assertion regarding code marked for deopt. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2013 the V8 project authors. All rights reserved. 1 // Copyright 2013 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 950 matching lines...) Expand 10 before | Expand all | Expand 10 after
961 } 961 }
962 result->set_literals(*literals); 962 result->set_literals(*literals);
963 } 963 }
964 964
965 if (index > 0) { 965 if (index > 0) {
966 // Caching of optimized code enabled and optimized code found. 966 // Caching of optimized code enabled and optimized code found.
967 FixedArray* literals = 967 FixedArray* literals =
968 function_info->GetLiteralsFromOptimizedCodeMap(index); 968 function_info->GetLiteralsFromOptimizedCodeMap(index);
969 if (literals != NULL) result->set_literals(literals); 969 if (literals != NULL) result->set_literals(literals);
970 Code* code = function_info->GetCodeFromOptimizedCodeMap(index); 970 Code* code = function_info->GetCodeFromOptimizedCodeMap(index);
971 ASSERT(!code->marked_for_deoptimization());
972 result->ReplaceCode(code); 971 result->ReplaceCode(code);
973 return result; 972 return result;
974 } 973 }
975 974
976 if (isolate()->use_crankshaft() && 975 if (isolate()->use_crankshaft() &&
977 FLAG_always_opt && 976 FLAG_always_opt &&
978 result->is_compiled() && 977 result->is_compiled() &&
979 !function_info->is_toplevel() && 978 !function_info->is_toplevel() &&
980 function_info->allows_lazy_compilation() && 979 function_info->allows_lazy_compilation() &&
981 !function_info->optimization_disabled() && 980 !function_info->optimization_disabled() &&
(...skipping 1008 matching lines...) Expand 10 before | Expand all | Expand 10 after
1990 return Handle<Object>::null(); 1989 return Handle<Object>::null();
1991 } 1990 }
1992 1991
1993 1992
1994 Handle<Object> Factory::ToBoolean(bool value) { 1993 Handle<Object> Factory::ToBoolean(bool value) {
1995 return value ? true_value() : false_value(); 1994 return value ? true_value() : false_value();
1996 } 1995 }
1997 1996
1998 1997
1999 } } // namespace v8::internal 1998 } } // namespace v8::internal
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698