MIPS: Rename "mips" arch to "mipsel" in the GYP build.

MIPS: Rename "mips" arch to "mipsel" in the GYP build.

In the traditional MIPS naming scheme, "mips" is used for
big-endian mips and "mipsel" is used for little-endian mips.
In V8 the "mips" build is little-endian, so the "mips" target is
renamed to "mipsel" to be compliant with the traditional MIPS
naming scheme.

This change is also required for supporting the Chromium project on MIPS.

BUG=
TEST=

Committed: https://code.google.com/p/v8/source/detail?r=12047

[Jakob Kummerow, 2012-07-06 16:52:24]

LGTM.

I'll land this on Monday.

[Jakob Kummerow, 2012-07-07 12:34:37]

Come to think of it, these changes aren't quite enough; in particular, they will
make it impossible to run the test suite.
tools/disasm.py, tools/ll_prof.py, tools/test.py,
tools/test-wrapper-gypbuild.py, test/{mjsunit,mozilla,test262}/*.status need
updating too.

Please make sure "make mipsel.release.check" works as expected.

http://chromiumcodereview.appspot.com/10695114/diff/1/build/common.gypi
File build/common.gypi (right):

http://chromiumcodereview.appspot.com/10695114/diff/1/build/common.gypi#newco...
build/common.gypi:167: 'mipscompiler': '<!($(echo ${CXX:-$(which g++)}) -v 2>&1
| grep -q "^Target: mips-" && echo "yes" || echo "no")',
Is "^Target: mips-" the right pattern to match against, or does your
cross-compiler identify itself differently? "Target: mipsel-" maybe?

http://chromiumcodereview.appspot.com/10695114/diff/1/build/standalone.gypi
File build/standalone.gypi (right):

http://chromiumcodereview.appspot.com/10695114/diff/1/build/standalone.gypi#n...
build/standalone.gypi:49:
s/x86_64/x64/;s/amd64/x64/;s/arm.*/arm/;s/mips.*/mips/")',
missing update, the last replacement must be:
s/mips.*/mipsel/

[palfia, 2012-07-09 17:07:21]

On 2012/07/07 12:34:37, Jakob wrote:
> Come to think of it, these changes aren't quite enough; in particular, they
will
> make it impossible to run the test suite.
> tools/disasm.py, tools/ll_prof.py, tools/test.py,
> tools/test-wrapper-gypbuild.py, test/{mjsunit,mozilla,test262}/*.status need
> updating too.
> 
> Please make sure "make mipsel.release.check" works as expected.
> 
> http://chromiumcodereview.appspot.com/10695114/diff/1/build/common.gypi
> File build/common.gypi (right):
> 
>
http://chromiumcodereview.appspot.com/10695114/diff/1/build/common.gypi#newco...
> build/common.gypi:167: 'mipscompiler': '<!($(echo ${CXX:-$(which g++)}) -v
2>&1
> | grep -q "^Target: mips-" && echo "yes" || echo "no")',
> Is "^Target: mips-" the right pattern to match against, or does your
> cross-compiler identify itself differently? "Target: mipsel-" maybe?
> 
> http://chromiumcodereview.appspot.com/10695114/diff/1/build/standalone.gypi
> File build/standalone.gypi (right):
> 
>
http://chromiumcodereview.appspot.com/10695114/diff/1/build/standalone.gypi#n...
> build/standalone.gypi:49:
> s/x86_64/x64/;s/amd64/x64/;s/arm.*/arm/;s/mips.*/mips/")',
> missing update, the last replacement must be:
> s/mips.*/mipsel/

Thanks for the review!

I've uploaded a new patchset, which includes the required updates, also we have
tested "make mipsel.release.check" and it works as expected now.

[palfia, 2012-07-09 17:07:41]

http://chromiumcodereview.appspot.com/10695114/diff/1/build/common.gypi
File build/common.gypi (right):

http://chromiumcodereview.appspot.com/10695114/diff/1/build/common.gypi#newco...
build/common.gypi:167: 'mipscompiler': '<!($(echo ${CXX:-$(which g++)}) -v 2>&1
| grep -q "^Target: mips-" && echo "yes" || echo "no")',
On 2012/07/07 12:34:37, Jakob wrote:
> Is "^Target: mips-" the right pattern to match against, or does your
> cross-compiler identify itself differently? "Target: mipsel-" maybe?

The CodeSourcery toolchain returns "Target: mips-linux-gnu", but with the -EL
flag, it builds little-endian code, so I think, that the pattern is correct.

http://chromiumcodereview.appspot.com/10695114/diff/1/build/standalone.gypi
File build/standalone.gypi (right):

http://chromiumcodereview.appspot.com/10695114/diff/1/build/standalone.gypi#n...
build/standalone.gypi:49:
s/x86_64/x64/;s/amd64/x64/;s/arm.*/arm/;s/mips.*/mips/")',
On 2012/07/07 12:34:37, Jakob wrote:
> missing update, the last replacement must be:
> s/mips.*/mipsel/

Thanks, done.

[Jakob Kummerow, 2012-07-11 15:12:43]

LGTM, landing.

Found another missing update: test/cctest/cctest.gyp. Fixed that for you.

Note that our MIPS buildbot will be red now until we update it too.

[Jakob Kummerow, 2012-07-11 15:17:23]

Oops, forgot to specify the change's author when committing. Sorry.

[Paul Lind, 2012-07-11 15:20:15]

On 2012/07/11 15:12:43, Jakob wrote:
> Note that our MIPS buildbot will be red now until we update it too.

Hi Jakob - Just noticed now that you had switched the buildbot over to GYP. Is
fixing that something that we should submit a patch or issue for, or will you be
able to take care of it?

[Jakob Kummerow, 2012-07-11 15:23:36]

On 2012/07/11 15:20:15, Paul Lind wrote:
> Hi Jakob - Just noticed now that you had switched the buildbot over to GYP. Is
> fixing that something that we should submit a patch or issue for, or will you
be
> able to take care of it?

I'm taking care of it. It'll require a restart of the buildbot master, though,
and we won't do that immediately.

[palfia, 2012-07-11 15:27:44]

On 2012/07/11 15:17:23, Jakob wrote:
> Oops, forgot to specify the change's author when committing. Sorry.

No problem, thanks for landing!