[stubs] Fixing issues found by machine graph verifier in code stubs.

[stubs] Fixing issues found by machine graph verifier in code stubs.

BUG=

Committed: https://crrev.com/9978f90381a41f9abe5d4d2f95ae6848ed732098
Cr-Commit-Position: refs/heads/master@{#41651}

[Igor Sheludko, 2016-12-11 16:30:11]

Patchset #1 (id:1) has been deleted

[Igor Sheludko, 2016-12-11 18:36:05]

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

[commit-bot: I haz the power, 2016-12-11 18:36:16]

Dry run: 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-12-11 19:05:00]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-11 19:05:01]

Dry run: This issue passed the CQ dry run.

[Igor Sheludko, 2016-12-12 14:29:20]

Patchset #1 (id:20001) has been deleted

[Igor Sheludko, 2016-12-12 14:29:52]

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

[commit-bot: I haz the power, 2016-12-12 14:29:57]

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

[Igor Sheludko, 2016-12-12 14:30:04]

ishell@chromium.org changed reviewers:
+ cbruni@chromium.org

[Igor Sheludko, 2016-12-12 14:30:05]

PTAL

[commit-bot: I haz the power, 2016-12-12 14:55:42]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-12 14:55:43]

Dry run: This issue passed the CQ dry run.

[Camillo Bruni, 2016-12-12 15:17:37]

LGTM with nits

https://codereview.chromium.org/2568713002/diff/40001/src/builtins/builtins.cc
File src/builtins/builtins.cc (right):

https://codereview.chromium.org/2568713002/diff/40001/src/builtins/builtins.c...
src/builtins/builtins.cc:114: bool sav_csa_verify = FLAG_csa_verify;
nit: saved_csa_verify ?

https://codereview.chromium.org/2568713002/diff/40001/src/code-stubs.cc
File src/code-stubs.cc (right):

https://codereview.chromium.org/2568713002/diff/40001/src/code-stubs.cc#newco...
src/code-stubs.cc:443: bool sav_csa_verify = FLAG_csa_verify;
nit: saved_csa_verify

[Igor Sheludko, 2016-12-12 15:24:51]

Patchset #2 (id:60001) has been deleted

[Igor Sheludko, 2016-12-12 15:25:12]

https://codereview.chromium.org/2568713002/diff/40001/src/builtins/builtins.cc
File src/builtins/builtins.cc (right):

https://codereview.chromium.org/2568713002/diff/40001/src/builtins/builtins.c...
src/builtins/builtins.cc:114: bool sav_csa_verify = FLAG_csa_verify;
On 2016/12/12 15:17:37, Camillo Bruni wrote:
> nit: saved_csa_verify ?

Done.

https://codereview.chromium.org/2568713002/diff/40001/src/code-stubs.cc
File src/code-stubs.cc (right):

https://codereview.chromium.org/2568713002/diff/40001/src/code-stubs.cc#newco...
src/code-stubs.cc:443: bool sav_csa_verify = FLAG_csa_verify;
On 2016/12/12 15:17:37, Camillo Bruni wrote:
> nit: saved_csa_verify

Done.

[Igor Sheludko, 2016-12-12 15:25:17]

The CQ bit was checked by ishell@chromium.org

[Igor Sheludko, 2016-12-12 15:25:17]

The patchset sent to the CQ was uploaded after l-g-t-m from cbruni@chromium.org
Link to the patchset: https://codereview.chromium.org/2568713002/#ps80001
(title: "Addressing nits")

[commit-bot: I haz the power, 2016-12-12 15:25:21]

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-12-12 15:53:22]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1481556317049720,
"parent_rev": "2b13cf57eefa56ac57b1413513ebf4ca6acd19fa", "commit_rev":
"3baabedb638ac9a9b2553ea201f1f58dffdfab7f"}

[commit-bot: I haz the power, 2016-12-12 15:53:25]

Description was changed from

==========
[stubs] Fixing issues found by machine graph verifier in code stubs.

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

to

==========
[stubs] Fixing issues found by machine graph verifier in code stubs.

BUG=

Review-Url: https://codereview.chromium.org/2568713002
==========

[commit-bot: I haz the power, 2016-12-12 15:53:26]

Committed patchset #2 (id:80001)

[commit-bot: I haz the power, 2016-12-12 15:54:06]

Description was changed from

==========
[stubs] Fixing issues found by machine graph verifier in code stubs.

BUG=

Review-Url: https://codereview.chromium.org/2568713002
==========

to

==========
[stubs] Fixing issues found by machine graph verifier in code stubs.

BUG=

Committed: https://crrev.com/9978f90381a41f9abe5d4d2f95ae6848ed732098
Cr-Commit-Position: refs/heads/master@{#41651}
==========

[commit-bot: I haz the power, 2016-12-12 15:54:07]

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