[turbofan] Regalloc validator: support same block pending assessment

[turbofan] Regalloc validator: support same block pending assessment

Previous fuzzer fix broke the case when the pending assessment came from the same
block. In that case, the assessments table does not have an entry yet for the block,
because we register only when we're done processing a block.

BUG=667745
Committed: https://crrev.com/7a1ad0c581e1ac5536723fcddbf8e623bd59b1c7
Cr-Commit-Position: refs/heads/master@{#41193}

[Mircea Trofin, 2016-11-22 17:02:53]

The CQ bit was checked by mtrofin@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-22 17:02:57]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Mircea Trofin, 2016-11-22 17:05:31]

The CQ bit was checked by mtrofin@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-22 17:05:37]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Mircea Trofin, 2016-11-22 17:07:50]

Description was changed from

==========
[turbofan] Regalloc validator: support same block pending assessment

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

to

==========
[turbofan] Regalloc validator: support same block pending assessment

Previous fuzzer fix broke the case when the pending assessment came from the
same
block. In that case, the assessments table does not have an entry yet for the
block,
because we register only when we're done processing a block.

BUG=667745
==========

[Mircea Trofin, 2016-11-22 17:07:51]

mtrofin@chromium.org changed reviewers:
+ bradnelson@chromium.org

[Mircea Trofin, 2016-11-22 17:08:15]

Patch#1 introduces the regression test, Patch#2 fixes the issue.

[bradnelson, 2016-11-22 17:18:25]

lgtm

[bradnelson, 2016-11-22 17:18:29]

The CQ bit was unchecked by bradnelson@chromium.org

[bradnelson, 2016-11-22 17:18:29]

The CQ bit was checked by bradnelson@chromium.org

[commit-bot: I haz the power, 2016-11-22 17:18:38]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-22 17:30:53]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1479835109512290,
"parent_rev": "b062593721a4fad85929894309b74a5ff1b010fb", "commit_rev":
"1e666f8819c0932582767ea0ed2c3ec5363cf891"}

[commit-bot: I haz the power, 2016-11-22 17:30:58]

Description was changed from

==========
[turbofan] Regalloc validator: support same block pending assessment

Previous fuzzer fix broke the case when the pending assessment came from the
same
block. In that case, the assessments table does not have an entry yet for the
block,
because we register only when we're done processing a block.

BUG=667745
==========

to

==========
[turbofan] Regalloc validator: support same block pending assessment

Previous fuzzer fix broke the case when the pending assessment came from the
same
block. In that case, the assessments table does not have an entry yet for the
block,
because we register only when we're done processing a block.

BUG=667745
==========

[commit-bot: I haz the power, 2016-11-22 17:30:59]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-22 17:31:12]

Description was changed from

==========
[turbofan] Regalloc validator: support same block pending assessment

Previous fuzzer fix broke the case when the pending assessment came from the
same
block. In that case, the assessments table does not have an entry yet for the
block,
because we register only when we're done processing a block.

BUG=667745
==========

to

==========
[turbofan] Regalloc validator: support same block pending assessment

Previous fuzzer fix broke the case when the pending assessment came from the
same
block. In that case, the assessments table does not have an entry yet for the
block,
because we register only when we're done processing a block.

BUG=667745

Committed: https://crrev.com/7a1ad0c581e1ac5536723fcddbf8e623bd59b1c7
Cr-Commit-Position: refs/heads/master@{#41193}
==========

[commit-bot: I haz the power, 2016-11-22 17:31:12]

Patchset 2 (id:??) landed as
https://crrev.com/7a1ad0c581e1ac5536723fcddbf8e623bd59b1c7
Cr-Commit-Position: refs/heads/master@{#41193}