Make RegARM32 a namespace rather than a class.

Make RegARM32 a namespace rather than a class.

Cleans up code by removing unnecessary class.

BUG=None
R=stichnot@chromium.org

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

[Karl, 2016-01-14 22:34:36]

Description was changed from

==========
Make RegARM32 a namespace rather than a class.

Cleans up code by removing unnecessary class.

BUG=None
==========

to

==========
Make RegARM32 a namespace rather than a class.

Cleans up code by removing unnecessary class.

BUG=None
==========

[Karl, 2016-01-14 22:34:36]

kschimpf@google.com changed reviewers:
+ jpp@chromium.org, sehr@chromium.org, stichnot@chromium.org

[Jim Stichnoth, 2016-01-15 01:17:15]

otherwise lgtm

https://codereview.chromium.org/1581803009/diff/20001/src/IceRegistersARM32.h
File src/IceRegistersARM32.h (right):

https://codereview.chromium.org/1581803009/diff/20001/src/IceRegistersARM32.h...
src/IceRegistersARM32.h:143: static inline IceString getGPRName(SizeT RegNum) {
Remove this because it is identical to getRegName()

Also I think it's unused.

https://codereview.chromium.org/1581803009/diff/20001/src/IceRegistersARM32.h...
src/IceRegistersARM32.h:183: static inline IceString getSRegName(SizeT RegNum) {
Remove this because it is identical to getRegName()

https://codereview.chromium.org/1581803009/diff/20001/src/IceTargetLoweringAR...
File src/IceTargetLoweringARM32.cpp (right):

https://codereview.chromium.org/1581803009/diff/20001/src/IceTargetLoweringAR...
src/IceTargetLoweringARM32.cpp:99: (SizeOf alias_init).size(), alias_init       
                    \
I assume this trailing backslash alignment will get fixed by "make format"...

[Karl, 2016-01-15 16:11:02]

Description was changed from

==========
Make RegARM32 a namespace rather than a class.

Cleans up code by removing unnecessary class.

BUG=None
==========

to

==========
Make RegARM32 a namespace rather than a class.

Cleans up code by removing unnecessary class.

BUG=None
R=stichnot@chromium.org

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

[Karl, 2016-01-15 16:11:03]

Committed patchset #3 (id:40001) manually as
57ec7dfe6beadff524101efbbe05e22baa0a0706 (presubmit successful).

[Karl, 2016-01-15 16:11:20]

https://codereview.chromium.org/1581803009/diff/20001/src/IceRegistersARM32.h
File src/IceRegistersARM32.h (right):

https://codereview.chromium.org/1581803009/diff/20001/src/IceRegistersARM32.h...
src/IceRegistersARM32.h:143: static inline IceString getGPRName(SizeT RegNum) {
On 2016/01/15 01:17:15, stichnot wrote:
> Remove this because it is identical to getRegName()
> 
> Also I think it's unused.

Done.

https://codereview.chromium.org/1581803009/diff/20001/src/IceRegistersARM32.h...
src/IceRegistersARM32.h:183: static inline IceString getSRegName(SizeT RegNum) {
On 2016/01/15 01:17:15, stichnot wrote:
> Remove this because it is identical to getRegName()

Done.

https://codereview.chromium.org/1581803009/diff/20001/src/IceTargetLoweringAR...
File src/IceTargetLoweringARM32.cpp (right):

https://codereview.chromium.org/1581803009/diff/20001/src/IceTargetLoweringAR...
src/IceTargetLoweringARM32.cpp:99: (SizeOf alias_init).size(), alias_init       
                    \
On 2016/01/15 01:17:15, stichnot wrote:
> I assume this trailing backslash alignment will get fixed by "make format"...

Done.