Comparison in effect context lazy deopt fix.

Comparison in effect context lazy deopt fix.

R=jkummerow@chromium.org
BUG=

Committed: https://code.google.com/p/v8/source/detail?r=19396

[Jakob Kummerow, 2014-02-13 15:55:32]

lgtm

https://codereview.chromium.org/163623002/diff/1/src/hydrogen.cc
File src/hydrogen.cc (right):

https://codereview.chromium.org/163623002/diff/1/src/hydrogen.cc#newcode9616
src/hydrogen.cc:9616: if (push_sim_result == NO_PUSH_BEFORE_SIMULATE) {
nit: I'd find it slightly more readable to swap the condition; then you can also
drop the ASSERT (it's clear enough what's going on):

if (push_sim_result == PUSH_BEFORE_SIMULATE) {
  Push(result);
  AddSimulate(bailout_id, REMOVABLE_SIMULATE);
  Drop(1);
} else {
  AddSimulate(bailout_id, REMOVABLE_SIMULATE);
}

[Jarin, 2014-02-16 05:51:17]

Committed patchset #2 manually as r19396 (presubmit successful).