Reviewers: Michael Starzinger,
Message:
PTAL.
Description:
[test] Remove FLAG_always_opt special case in NotifyDeoptimized
Always unlink optimized code on deopt, even when FLAG_always_opt is present,
because assumptions that the code made could have become invalid.
BUG=v8:4375
LOG=n
[email protected]
Please review this at https://codereview.chromium.org/1284103006/
Base URL: https://chromium.googlesource.com/v8/v8.git@master
Affected files (+1, -3 lines):
M src/runtime/runtime-compiler.cc
Index: src/runtime/runtime-compiler.cc
diff --git a/src/runtime/runtime-compiler.cc
b/src/runtime/runtime-compiler.cc
index
a60252717573be68b9cc9b8d9d535863da0aeea4..cf76fa0b5d541c3c4f0053e1f1b0b3435c39b9f6
100644
--- a/src/runtime/runtime-compiler.cc
+++ b/src/runtime/runtime-compiler.cc
@@ -132,9 +132,7 @@ RUNTIME_FUNCTION(Runtime_NotifyDeoptimized) {
RUNTIME_ASSERT(frame->function()->IsJSFunction());
DCHECK(frame->function() == *function);
- // Avoid doing too much work when running with --always-opt and keep
- // the optimized code around.
- if (FLAG_always_opt || type == Deoptimizer::LAZY) {
+ if (type == Deoptimizer::LAZY) {
return isolate->heap()->undefined_value();
}
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
