check if we can split (linear scan)

check if we can split (linear scan)

Revert "[wasm] Quickfix for register allocation problem on ia32."

This reverts commit 2ca31b636bf03b67867ff94cc82fca84f48eb1d1.

BUG=

Committed: https://crrev.com/5ec339c4f04f76c2bf90e2deefc3250ba7e77682
Cr-Commit-Position: refs/heads/master@{#35559}

[titzer, 2016-03-29 08:12:06]

titzer@chromium.org changed reviewers:
+ titzer@chromium.org

[titzer, 2016-03-29 08:12:06]

lgtm

https://codereview.chromium.org/1824303002/diff/60001/src/compiler/register-a...
File src/compiler/register-allocator.h (right):

https://codereview.chromium.org/1824303002/diff/60001/src/compiler/register-a...
src/compiler/register-allocator.h:53: LifetimePosition next_gap =
Clearer with an explicit if?

if(next.IsGapPosition()) return next < pos2;
return next.FullStart() < pos2

[Mircea Trofin, 2016-04-15 19:47:13]

Patchset #5 (id:80001) has been deleted

[Mircea Trofin, 2016-04-18 05:17:43]

The CQ bit was checked by mtrofin@chromium.org

[Mircea Trofin, 2016-04-18 05:17:43]

The patchset sent to the CQ was uploaded after l-g-t-m from titzer@chromium.org
Link to the patchset: https://codereview.chromium.org/1824303002/#ps100001
(title: " ")

[commit-bot: I haz the power, 2016-04-18 05:17:53]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1824303002/100001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1824303002/100001

[Mircea Trofin, 2016-04-18 05:18:37]

https://codereview.chromium.org/1824303002/diff/60001/src/compiler/register-a...
File src/compiler/register-allocator.h (right):

https://codereview.chromium.org/1824303002/diff/60001/src/compiler/register-a...
src/compiler/register-allocator.h:53: LifetimePosition next_gap =
On 2016/03/29 08:12:06, titzer wrote:
> Clearer with an explicit if?
> 
> if(next.IsGapPosition()) return next < pos2;
> return next.FullStart() < pos2
> 

Done.

[commit-bot: I haz the power, 2016-04-18 05:41:12]

Committed patchset #5 (id:100001)

[commit-bot: I haz the power, 2016-04-18 05:42:45]

Description was changed from

==========
check if we can split (linear scan)


Revert "[wasm] Quickfix for register allocation problem on ia32."

This reverts commit 2ca31b636bf03b67867ff94cc82fca84f48eb1d1.

BUG=
==========

to

==========
check if we can split (linear scan)

Revert "[wasm] Quickfix for register allocation problem on ia32."

This reverts commit 2ca31b636bf03b67867ff94cc82fca84f48eb1d1.

BUG=

Committed: https://crrev.com/5ec339c4f04f76c2bf90e2deefc3250ba7e77682
Cr-Commit-Position: refs/heads/master@{#35559}
==========

[commit-bot: I haz the power, 2016-04-18 05:42:46]

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