Revision: 4119 Author: [email protected] Date: Fri Mar 12 06:06:04 2010 Log: Simple bugfix in reaching definitions.
We only track definitions of stack-allocated variables. Trying to look up other variables in the environment is a bug. Review URL: http://codereview.chromium.org/843006 http://code.google.com/p/v8/source/detail?r=4119 Modified: /branches/bleeding_edge/src/data-flow.cc ======================================= --- /branches/bleeding_edge/src/data-flow.cc Fri Mar 12 05:12:08 2010 +++ /branches/bleeding_edge/src/data-flow.cc Fri Mar 12 06:06:04 2010 @@ -1931,7 +1931,7 @@ Expression* expr = instructions_[i]->AsExpression(); if (expr == NULL) continue; Variable* var = expr->AssignedVar(); - if (var == NULL) continue; + if (var == NULL || !var->IsStackAllocated()) continue; // All definitions of this variable are killed. BitVector* def_set = -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
