Fix a failure to correctly set the static type on a frame element at a...

Fix a failure to correctly set the static type on a frame element at a
backward jump.  The frame entering the backward block is not used, so
the this is mostly just a bookkeeping change.

Committed: http://code.google.com/p/v8/source/detail?r=1989

[William Hesse, 2009-05-18 12:50:46]

http://codereview.chromium.org/115464/diff/1/4
File src/frame-element.h (right):

http://codereview.chromium.org/115464/diff/1/4#newcode160
Line 160: !(static_type() == other.static_type())) {
Why doesn't this use != as the previous two lines do?  The form should be
consistent, right?

http://codereview.chromium.org/115464/diff/1/3
File src/ia32/virtual-frame-ia32.cc (right):

http://codereview.chromium.org/115464/diff/1/3#newcode409
Line 409: // Nothing except the register backing use_index has changed.
Sync bit may be changed without actually syncing.  Fixed in next revision of
this patch already.

[William Hesse, 2009-05-18 12:56:06]

LGTM.

[Kevin Millikin (Chromium), 2009-05-18 12:59:13]

http://codereview.chromium.org/115464/diff/1/3
File src/ia32/virtual-frame-ia32.cc (right):

http://codereview.chromium.org/115464/diff/1/3#newcode409
Line 409: // Nothing except the register backing use_index has changed.
On 2009/05/18 12:50:46, William Hesse wrote:
> Sync bit may be changed without actually syncing.  Fixed in next revision of
> this patch already.

Good catch.  New change uploaded, check it out.