First patch for Mips subzero compiler

First patch for Mips subzero compiler

BUG= https://code.google.com/p/nativeclient/issues/detail?id=4167

Move issue https://codereview.chromium.org/1159823004/ here so that
it's under the proper email.

Committed: https://gerrit.chromium.org/gerrit/gitweb?p=native_client/pnacl-subzero.git;a=commit;h=6da4cef77cdfbd8d42ee08171d550cc6414d06ab

[reed.kotler, 2015-06-06 22:08:44]

reed.kotler@imgtec.com changed reviewers:
+ jvoung@chromium.org, stichnot@chromium.org

[reed.kotler, 2015-06-06 22:09:26]

Move the issue here so that it's under the proper email.

[reed.kotler, 2015-06-09 02:20:41]

On 2015/06/06 22:09:26, reed.kotler wrote:
> Move the issue here so that it's under the proper email.

This seems to build okay in the current native_client tree.

Can someone push it upstream if it still gets a LGTM?

Tia.

Reed

[reed.kotler, 2015-06-09 16:24:46]

On 2015/06/09 02:20:41, reed.kotler wrote:
> On 2015/06/06 22:09:26, reed.kotler wrote:
> > Move the issue here so that it's under the proper email.
> 
> This seems to build okay in the current native_client tree.
> 
> Can someone push it upstream if it still gets a LGTM?
> 
> Tia.
> 
> Reed

Something interesting... if you apply the patch from this tool using linux
patch, it messes up.
The patch tool for rietvel is clever and noticed that I made IceRegisterMIPS32.h
from the ARM one and so it made the patch by first copying IceRegegisterARM32.h

...
Index: src/IceRegistersMIPS32.h
diff --git a/src/IceRegistersARM32.h b/src/IceRegistersMIPS32.h
similarity index 78%
copy from src/IceRegistersARM32.h
copy to src/IceRegistersMIPS32.h



When linux patch applies this patch, it ends up patching ...IceRegistersARM32.h

I ended up using this other mechanism for patching from a codereview patch.

https://sites.google.com/site/skiadocs/developer-documentation/contributing-c...

[Jim Stichnoth, 2015-06-11 20:26:37]

Committed patchset #1 (id:1) manually as
6da4cef77cdfbd8d42ee08171d550cc6414d06ab (presubmit successful).

[Jim Stichnoth, 2015-06-11 20:30:38]

I landed this patch.  Unfortunately, since I waited too long, there had been
some refactoring in Subzero that required some manual changes to the patch.

It would be good to rename assembler_mips32.h ==> IceAssemblerMIPS32.h in the
near future.