Revision: 8760
Author: [email protected]
Date: Thu Jul 28 07:56:08 2011
Log: Optimization: Do not assign an environment to LBranch when are
sure that we will never deopt later.
Review URL: http://codereview.chromium.org/7524025
http://code.google.com/p/v8/source/detail?r=8760
Modified:
/branches/bleeding_edge/src/ia32/lithium-ia32.cc
=======================================
--- /branches/bleeding_edge/src/ia32/lithium-ia32.cc Thu Jul 28 06:33:51
2011
+++ /branches/bleeding_edge/src/ia32/lithium-ia32.cc Thu Jul 28 07:56:08
2011
@@ -1042,9 +1042,15 @@
return new LGoto(successor->block_id());
}
ToBooleanStub::Types expected = instr->expected_input_types();
+ // We need a temporary register when we have to access the map *or* we
have
+ // no type info yet, in which case we handle all cases (including the
ones
+ // involving maps).
bool needs_temp = expected.NeedsMap() || expected.IsEmpty();
LOperand* temp = needs_temp ? TempRegister() : NULL;
- return AssignEnvironment(new LBranch(UseRegister(v), temp));
+ LInstruction* branch = new LBranch(UseRegister(v), temp);
+ // When we handle all cases, we never deopt, so we don't need to assign
the
+ // environment then.
+ return expected.IsAll() ? branch : AssignEnvironment(branch);
}
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
