ARM: make sure that the expected code size is computed in predictable scope.

ARM: make sure that the expected code size is computed in predictable scope.

BUG=v8:4277
LOG=NO

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

[ulan, 2015-07-21 09:57:35]

ulan@chromium.org changed reviewers:
+ jacob.bramley@arm.com

[ulan, 2015-07-21 09:57:36]

Hi Jacob, would you mind taking a look?

[jbramley, 2015-07-21 10:04:35]

What's the purpose of it? Will there be future cases where we don't know the
expected size until after the start of the scope?

[ulan, 2015-07-21 10:06:54]

On 2015/07/21 10:04:35, jbramley wrote:
> What's the purpose of it? Will there be future cases where we don't know the
> expected size until after the start of the scope?

The background is in https://code.google.com/p/v8/issues/detail?id=4277

CallCodeSize() depends on the predictable_code_size_ flag, so we need to call it
in the scope, not before the scope.

[jbramley, 2015-07-21 10:11:08]

It might be more robust to be able to pass the flag to CallCodeSize(), and make
it stateless, but that might be a complicated change.

LGTM

[ulan, 2015-07-21 10:49:00]

The CQ bit was checked by ulan@chromium.org

[commit-bot: I haz the power, 2015-07-21 10:49:07]

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

[commit-bot: I haz the power, 2015-07-21 11:18:28]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-07-21 11:18:45]

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