Support for arm64 build.

Support for arm64 build.

Needed for fnl on n6p.

Change-Id: Ief00f8190087437944a937e010f2a97d668fb610

BUG=
R=qsr@chromium.org

Committed: https://chromium.googlesource.com/external/mojo/+/0d68ff269ecf68310681f4af67a64c60a68bae2a

[cdotstout, 2015-12-21 23:04:33]

cstout@chromium.org changed reviewers:
+ jamesr@chromium.org, qsr@chromium.org, viettrungluu@chromium.org

[cdotstout, 2015-12-21 23:04:34]

For fnl on n6p.

[qsr, 2015-12-22 08:51:29]

https://codereview.chromium.org/1540293003/diff/1/build/config/arm.gni
File build/config/arm.gni (right):

https://codereview.chromium.org/1540293003/diff/1/build/config/arm.gni#newcode10
build/config/arm.gni:10: arm_version = 8
So, this is a gn arg. You can change it in out/.../args.gn

What do you think about having changes only in mojob and setting this, instead
of adding a new cpu?

[cdotstout, 2015-12-22 18:11:55]

On 2015/12/22 08:51:29, qsr wrote:
> https://codereview.chromium.org/1540293003/diff/1/build/config/arm.gni
> File build/config/arm.gni (right):
> 
>
https://codereview.chromium.org/1540293003/diff/1/build/config/arm.gni#newcode10
> build/config/arm.gni:10: arm_version = 8
> So, this is a gn arg. You can change it in out/.../args.gn
> 
> What do you think about having changes only in mojob and setting this, instead
> of adding a new cpu?

arm64 seems already used in several places, mostly for android, so I thought
it's best to just add it here.  Let me know if you disagree.

[qsr, 2015-12-23 10:35:09]

On 2015/12/22 18:11:55, cdotstout wrote:
> On 2015/12/22 08:51:29, qsr wrote:
> > https://codereview.chromium.org/1540293003/diff/1/build/config/arm.gni
> > File build/config/arm.gni (right):
> > 
> >
>
https://codereview.chromium.org/1540293003/diff/1/build/config/arm.gni#newcode10
> > build/config/arm.gni:10: arm_version = 8
> > So, this is a gn arg. You can change it in out/.../args.gn
> > 
> > What do you think about having changes only in mojob and setting this,
instead
> > of adding a new cpu?
> 
> arm64 seems already used in several places, mostly for android, so I thought
> it's best to just add it here.  Let me know if you disagree.

You are right, sorry.

LGTM

[viettrungluu, 2015-12-27 18:02:50]

https://codereview.chromium.org/1540293003/diff/1/mojo/tools/mopy/config.py
File mojo/tools/mopy/config.py (right):

https://codereview.chromium.org/1540293003/diff/1/mojo/tools/mopy/config.py#n...
mojo/tools/mopy/config.py:30: ARCH_ARM64 = "arm64"
Please add arm64 to mojo/tools/mopy/gn_unittest.py.

[cdotstout, 2016-01-04 17:37:35]

Description was changed from

==========
Support for arm64 build.

Needed for fnl on n6p.

Change-Id: Ief00f8190087437944a937e010f2a97d668fb610

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

to

==========
Support for arm64 build.

Needed for fnl on n6p.

Change-Id: Ief00f8190087437944a937e010f2a97d668fb610

BUG=
R=qsr@chromium.org

Committed:
https://chromium.googlesource.com/external/mojo/+/0d68ff269ecf68310681f4af67a...
==========

[cdotstout, 2016-01-04 17:37:40]

Committed patchset #3 (id:40001) manually as
0d68ff269ecf68310681f4af67a64c60a68bae2a (presubmit successful).