On Mon, Feb 21, 2011 at 6:02 PM, <[email protected]> wrote: > http://codereview.chromium.org/6526047/diff/1/src/lithium.h#newcode540 > src/lithium.h:540: return op == NULL || op->IsConstantOperand() || > op->IsArgument(); > Just wondering: Was this already a problem before to have LArgument > operands in the enumeration of environment uses not skipped? > > There was an assert that would have triggered if we had seen one. We didn't get them because PushArgument can't deoptimize and doesn't have side effects, so even if the call itself deoptimized we would target an environment without the arguments pushed.
-- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
