[turbofan] OSR loops assign all local variables.

[turbofan] OSR loops assign all local variables.

R=mstarzinger@chromium.org
BUG=

Committed: https://crrev.com/d9892bbd5a78b5a1e389cbf4854ec7fcbd645567
Cr-Commit-Position: refs/heads/master@{#26493}

[Michael Starzinger, 2015-02-06 14:59:32]

LGTM. Sigh ... :/

https://codereview.chromium.org/886733005/diff/1/src/bit-vector.h
File src/bit-vector.h (right):

https://codereview.chromium.org/886733005/diff/1/src/bit-vector.h#newcode157
src/bit-vector.h:157: for (int i = 0; i < data_length_; i++) {
Just out of curiosity, is there a reason Clear doesn't use memset here?

https://codereview.chromium.org/886733005/diff/1/src/compiler/ast-loop-assign...
File src/compiler/ast-loop-assignment-analyzer.cc (right):

https://codereview.chromium.org/886733005/diff/1/src/compiler/ast-loop-assign...
src/compiler/ast-loop-assignment-analyzer.cc:33: if (info()->is_osr() &&
info()->osr_ast_id() == loop->OsrEntryId())
Sigh ... :/

[titzer, 2015-02-06 15:01:54]

https://codereview.chromium.org/886733005/diff/1/src/bit-vector.h
File src/bit-vector.h (right):

https://codereview.chromium.org/886733005/diff/1/src/bit-vector.h#newcode157
src/bit-vector.h:157: for (int i = 0; i < data_length_; i++) {
On 2015/02/06 14:59:32, Michael Starzinger wrote:
> Just out of curiosity, is there a reason Clear doesn't use memset here?

No idea. I can fix it, if you like. I did the memset above because I wasn't sure
that static_cast<uintptr_t>(-1) was a good idea, so I used the big hammer.

[titzer, 2015-02-06 15:03:24]

The CQ bit was checked by titzer@chromium.org

[commit-bot: I haz the power, 2015-02-06 15:04:05]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/886733005/1

[commit-bot: I haz the power, 2015-02-06 15:57:04]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-02-06 15:57:23]

Patchset 1 (id:??) landed as
https://crrev.com/d9892bbd5a78b5a1e389cbf4854ec7fcbd645567
Cr-Commit-Position: refs/heads/master@{#26493}