Fixed bug in while-loops that caused an assertion to fail. ...

Fixed bug in while-loops that caused an assertion to fail. 

We forgot resetting true-/false-label to NULL after evaluating the 
condition expression in dowhile- and while-loops. 
This change fixes this.

This causes an assertion to fail in VisitIfStatement whenever there is an 
if-statement after a while-loop before. e.g. like in:

  var i=0, j=0;
  while(j<5) { j++; }
  if (i ==0 ) { j++; }

[fschneider, 2009-11-09 16:10:35]

Small fix that re-activates while-loops and dowhile-loops and fixes the debug
assertion failure in test/mjsunit/array-concat.js

[Lasse Reichstein, 2009-11-10 09:35:34]

LGTM

http://codereview.chromium.org/371070/diff/1/3
File src/fast-codegen.cc (right):

http://codereview.chromium.org/371070/diff/1/3#newcode319
Line 319: true_label_ = &body;
We should probably have some scoped object that set and unset/reset both labels.
This seems like unnecessary book-keeping.

[fschneider, 2009-11-10 09:57:18]

http://codereview.chromium.org/371070/diff/1/3
File src/fast-codegen.cc (right):

http://codereview.chromium.org/371070/diff/1/3#newcode319
Line 319: true_label_ = &body;
On 2009/11/10 09:35:34, Lasse Reichstein wrote:
> We should probably have some scoped object that set and unset/reset both
labels.
> This seems like unnecessary book-keeping.

Good point.