[MIPS] Fix missing definitions for MIPS arch in build and driver scripts

[MIPS] Fix missing definitions for MIPS arch in build and driver scripts

pnacl/build.sh and pnacl/driver/* scripts missed MIPS arch in a couple
of places. The toolchain can be built for MIPS now.

BUG= https://code.google.com/p/nativeclient/issues/detail?id=2275
TEST= pnacl/build.sh all

Committed: http://src.chromium.org/viewvc/native_client?view=rev&revision=10814

[Mark Seaborn, 2013-02-13 19:17:52]

LGTM, but please ensure toolchain trybots pass before committing.

https://codereview.chromium.org/12256013/diff/1/pnacl/driver/pnacl-driver.py
File pnacl/driver/pnacl-driver.py (right):

https://codereview.chromium.org/12256013/diff/1/pnacl/driver/pnacl-driver.py#...
pnacl/driver/pnacl-driver.py:59: 'BIAS_MIPS32' : '-D__MIPS__ -D__mips__
-D__MIPSEL__ -U__MIPSEB__',
Setting both __MIPSEL__ (little endian) and __MIPSEB__ (big endian) doesn't seem
to make sense.  Remove __MIPSEB__, since PNaCl assumes little endian?

[jvoung - send to chromium..., 2013-02-13 19:30:43]

https://codereview.chromium.org/12256013/diff/1/pnacl/driver/driver_env.py
File pnacl/driver/driver_env.py (right):

https://codereview.chromium.org/12256013/diff/1/pnacl/driver/driver_env.py#ne...
pnacl/driver/driver_env.py:67: 'BIAS'        : 'NONE', # This can be 'NONE',
'ARM', 'X8632', or 'X8664'.
Can also add MIPS32 to this list.

[petarj, 2013-02-13 20:45:01]

https://codereview.chromium.org/12256013/diff/1/pnacl/driver/pnacl-driver.py
File pnacl/driver/pnacl-driver.py (right):

https://codereview.chromium.org/12256013/diff/1/pnacl/driver/pnacl-driver.py#...
pnacl/driver/pnacl-driver.py:59: 'BIAS_MIPS32' : '-D__MIPS__ -D__mips__
-D__MIPSEL__ -U__MIPSEB__',
On 2013/02/13 19:17:52, Mark Seaborn wrote:

It's "-U__MIPSEB__", not "-D__MIPSEB__".

[Mark Seaborn, 2013-02-13 20:53:17]

https://codereview.chromium.org/12256013/diff/1/pnacl/driver/pnacl-driver.py
File pnacl/driver/pnacl-driver.py (right):

https://codereview.chromium.org/12256013/diff/1/pnacl/driver/pnacl-driver.py#...
pnacl/driver/pnacl-driver.py:59: 'BIAS_MIPS32' : '-D__MIPS__ -D__mips__
-D__MIPSEL__ -U__MIPSEB__',
On 2013/02/13 20:45:02, petarj wrote:
> It's "-U__MIPSEB__", not "-D__MIPSEB__".

Oh, I didn't notice that.  But still, we shouldn't need to undefine it, surely,
any more than we need -U__i386__?  It shouldn't be defined to begin with, should
it?

[petarj, 2013-02-14 01:06:52]

Second patch set uploaded.

https://codereview.chromium.org/12256013/diff/1/pnacl/driver/driver_env.py
File pnacl/driver/driver_env.py (right):

https://codereview.chromium.org/12256013/diff/1/pnacl/driver/driver_env.py#ne...
pnacl/driver/driver_env.py:67: 'BIAS'        : 'NONE', # This can be 'NONE',
'ARM', 'X8632', or 'X8664'.
On 2013/02/13 19:30:43, jvoung wrote:
> Can also add MIPS32 to this list.

Done.

https://codereview.chromium.org/12256013/diff/1/pnacl/driver/pnacl-driver.py
File pnacl/driver/pnacl-driver.py (right):

https://codereview.chromium.org/12256013/diff/1/pnacl/driver/pnacl-driver.py#...
pnacl/driver/pnacl-driver.py:59: 'BIAS_MIPS32' : '-D__MIPS__ -D__mips__
-D__MIPSEL__ -U__MIPSEB__',
On 2013/02/13 20:53:17, Mark Seaborn wrote:
> On 2013/02/13 20:45:02, petarj wrote:
> > It's "-U__MIPSEB__", not "-D__MIPSEB__".
> 
> Oh, I didn't notice that.  But still, we shouldn't need to undefine it,
surely,
> any more than we need -U__i386__?  It shouldn't be defined to begin with,
should
> it?
I have removed it now as I can not recall why we put it originally
there, and from the smoke testing I have just done, it seems that it
is not needed.

[jvoung - send to chromium..., 2013-02-14 01:37:08]

LGTM

[commit-bot: I haz the power, 2013-02-14 17:30:35]

CQ is trying da patch. Follow status at
https://nativeclient-status.appspot.com/cq/petarj@mips.com/12256013/8001

[commit-bot: I haz the power, 2013-02-14 17:47:58]

CQ is trying da patch. Follow status at
https://nativeclient-status.appspot.com/cq/petarj@mips.com/12256013/8001

[commit-bot: I haz the power, 2013-02-14 17:58:09]

CQ is trying da patch. Follow status at
https://nativeclient-status.appspot.com/cq/petarj@mips.com/12256013/8001

[commit-bot: I haz the power, 2013-02-14 18:08:33]

CQ is trying da patch. Follow status at
https://nativeclient-status.appspot.com/cq/petarj@mips.com/12256013/8001

[commit-bot: I haz the power, 2013-02-14 18:10:44]

CQ is trying da patch. Follow status at
https://nativeclient-status.appspot.com/cq/petarj@mips.com/12256013/8001

[commit-bot: I haz the power, 2013-02-14 20:32:36]

Change committed as 10814