Added cross-build option in Makefile wrapper for gyp.

Added cross-build option in Makefile wrapper for gyp.


BUG=
TEST=

[kalmard, 2012-04-28 08:40:38]

Usage example (after setting up $CC and $CXX):
make cross target=mips BUILDTYPE=Debug 
BUILDTYPE can be omitted and defaults to Release.

[danno, 2012-04-30 07:06:35]

lgtm

[Jakob Kummerow, 2012-04-30 09:01:49]

The change itself looks OK, but I don't like the general direction this is
going. I'd prefer to have fewer targets in the Makefile, not more.

"make arm" already works for simulator builds, cross-compile builds and native
builds (on ARM hosts); what sort of binary you get just depends on the toolchain
you have set up (CXX etc. environment variables). I'd like to have the same for
MIPS, if at all possible.

I'm happy to play around with the build definitions myself if you could point me
to instructions how to set up a MIPS cross-compile toolchain.

[kalmard, 2012-04-30 10:25:24]

On 2012/04/30 09:01:49, Jakob wrote:
> "make arm" already works for simulator builds, cross-compile builds and native
> builds (on ARM hosts); what sort of binary you get just depends on the
toolchain
> you have set up (CXX etc. environment variables). I'd like to have the same
for
> MIPS, if at all possible.
> 
> I'm happy to play around with the build definitions myself if you could point
me
> to instructions how to set up a MIPS cross-compile toolchain.

Hi! Setting up for MIPS compilation should be no different from ARM. We use the
CodeSourcery toolchain:
https://sourcery.mentor.com/sgpp/lite/mips/portal/release2037
And here's a small script to set up the env variables:
#!/bin/bash
BASE=/data/bin/mips-2011.09/bin/mips-linux-gnu
export CC="$BASE"-gcc
export CXX="$BASE"-g++
export LINK="$BASE"-g++

Originally I thought that "make arm" didn't support cross-compilation, but a few
minutes ago I realized that I wasn't setting up $LINK properly. So now "make
arm" works but "make mips" doesn't - that probably means this patch really is
unnecessary and mips compilation needs to be fixed.

Thanks for the help, by the way:)

[Jakob Kummerow, 2012-04-30 15:16:15]

Have a look at https://chromiumcodereview.appspot.com/10268010. The compiler
seems happy both for cross-compilation and simulator builds; however
lithium-codegen-mips.cc currently doesn't compile at all, so I couldn't inspect
the resulting binaries.

[Paul Lind, 2012-04-30 15:47:06]

On 2012/04/30 15:16:15, Jakob wrote:
> lithium-codegen-mips.cc currently doesn't compile at all, so I couldn't
inspect
> the resulting binaries.

Hi Jakob - Thanks for helping us out with this. I've not looked at your new
patch yet, but the fix for lithium-codegen-mips.cc is in
https://chromiumcodereview.appspot.com/10233019/ which hasn't been landed yet.

[palfia, 2012-05-02 18:41:59]

On 2012/04/30 15:16:15, Jakob wrote:
> Have a look at https://chromiumcodereview.appspot.com/10268010. The compiler
> seems happy both for cross-compilation and simulator builds; however
> lithium-codegen-mips.cc currently doesn't compile at all, so I couldn't
inspect
> the resulting binaries.

Added comment at: https://chromiumcodereview.appspot.com/10268010

[Jakob Kummerow, 2012-05-03 09:20:20]

I've landed https://chromiumcodereview.appspot.com/10268010.

With the fix from https://chromiumcodereview.appspot.com/10233019/ applied in
addition, it compiles fine both for a simulator build (on Ubuntu Precise, but
Lucid should work too) and a cross-compile build (with the CodeSourcery
toolchain).

Can we close this CL?

[palfia, 2012-05-03 21:50:36]

On 2012/05/03 09:20:20, Jakob wrote:
> I've landed https://chromiumcodereview.appspot.com/10268010.
> 
> With the fix from https://chromiumcodereview.appspot.com/10233019/ applied in
> addition, it compiles fine both for a simulator build (on Ubuntu Precise, but
> Lucid should work too) and a cross-compile build (with the CodeSourcery
> toolchain).

It works for me as well on Lucid.

> Can we close this CL?

Yes, it can be closed, thank you.