Clarify arm gn configuration

Clarify arm gn configuration

R=dpranke@chromium.org
TBR=machenbach@chromium.org
BUG=
LOG=n

Committed: https://crrev.com/788d9d1ba9994c1b74a1b93b4376acbbb04da617
Cr-Commit-Position: refs/heads/master@{#27209}

[jochen (gone - plz use gerrit), 2015-03-13 07:42:51]

ptal

https://codereview.chromium.org/1004113003/diff/1/BUILD.gn
File BUILD.gn (right):

https://codereview.chromium.org/1004113003/diff/1/BUILD.gn#newcode140
BUILD.gn:140: # These defines ares used for the ARM simulator.
it's not just for the snapshot toolchain - we could target_cpu == x86 and
v8_target_arch == arm

[Dirk Pranke, 2015-03-13 20:47:17]

lgtm

https://codereview.chromium.org/1004113003/diff/1/BUILD.gn
File BUILD.gn (right):

https://codereview.chromium.org/1004113003/diff/1/BUILD.gn#newcode140
BUILD.gn:140: # These defines ares used for the ARM simulator.
On 2015/03/13 07:42:51, jochen (slow) wrote:
> it's not just for the snapshot toolchain - we could target_cpu == x86 and
> v8_target_arch == arm

Um, okay ... I guess I don't understand what this is fully yet, and need to look
into where else (or how else) v8_target_arch is used.

[jochen (gone - plz use gerrit), 2015-03-16 09:23:16]

The CQ bit was checked by jochen@chromium.org

[commit-bot: I haz the power, 2015-03-16 09:23:23]

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

[commit-bot: I haz the power, 2015-03-16 09:23:23]

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

[commit-bot: I haz the power, 2015-03-16 09:23:24]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[jochen (gone - plz use gerrit), 2015-03-16 09:24:01]

jochen@chromium.org changed reviewers:
+ machenbach@chromium.org

[jochen (gone - plz use gerrit), 2015-03-16 09:24:03]

The CQ bit was checked by jochen@chromium.org

[commit-bot: I haz the power, 2015-03-16 09:24:12]

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

[commit-bot: I haz the power, 2015-03-16 09:54:50]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-03-16 09:55:06]

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