[interpreter][stubs] Don't modify FLAG_csa_verify flag to make TSAN happy.

[interpreter][stubs] Don't modify FLAG_csa_verify flag to make TSAN happy.

BUG=

Committed: https://crrev.com/769442a1b8257211a913d7a5cfb85362120b6f18
Cr-Commit-Position: refs/heads/master@{#41653}

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

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

[commit-bot: I haz the power, 2016-12-12 16:30:34]

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 16:48:09]

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

[Igor Sheludko, 2016-12-12 16:48:09]

PTAL

[Igor Sheludko, 2016-12-12 16:48:39]

Description was changed from

==========
[stubs] Don't modify FLAG_csa_verify flag to avoid TSAN issues.

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

to

==========
[interpreter][stubs] Don't modify FLAG_csa_verify flag to avoid TSAN issues.

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

[rmcilroy, 2016-12-12 16:50:14]

LGTM. 

Any plan for how we will re-enable the verifier for the interpreter, or are we
just waiting until we can enable it universally?

[Igor Sheludko, 2016-12-12 16:52:06]

On 2016/12/12 16:50:14, rmcilroy wrote:
> LGTM. 
> 
> Any plan for how we will re-enable the verifier for the interpreter, or are we
> just waiting until we can enable it universally?

I'll just enable it universally, once I fix issues in bulitins.

[Igor Sheludko, 2016-12-12 16:52:46]

Description was changed from

==========
[interpreter][stubs] Don't modify FLAG_csa_verify flag to avoid TSAN issues.

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

to

==========
[interpreter][stubs] Don't modify FLAG_csa_verify flag to make TSAN happy.

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

[Igor Sheludko, 2016-12-12 16:52:52]

The CQ bit was unchecked by ishell@chromium.org

[Igor Sheludko, 2016-12-12 16:52:55]

The CQ bit was checked by ishell@chromium.org

[commit-bot: I haz the power, 2016-12-12 16:53:04]

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 17:02:53]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481561575810870, "parent_rev":
"f607f548ff21d33c5caca628e66dd19bca773763", "commit_rev":
"4a563fb99d06531ba75dbbbcaf2adc1fef0affe6"}

[commit-bot: I haz the power, 2016-12-12 17:03:01]

Description was changed from

==========
[interpreter][stubs] Don't modify FLAG_csa_verify flag to make TSAN happy.

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

to

==========
[interpreter][stubs] Don't modify FLAG_csa_verify flag to make TSAN happy.

BUG=

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

[commit-bot: I haz the power, 2016-12-12 17:03:02]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-12 17:21:14]

Description was changed from

==========
[interpreter][stubs] Don't modify FLAG_csa_verify flag to make TSAN happy.

BUG=

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

to

==========
[interpreter][stubs] Don't modify FLAG_csa_verify flag to make TSAN happy.

BUG=

Committed: https://crrev.com/769442a1b8257211a913d7a5cfb85362120b6f18
Cr-Commit-Position: refs/heads/master@{#41653}
==========

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

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