MIPS64: Eliminate ABI-dependent definitions.

MIPS64: Eliminate ABI-dependent definitions.

Compilation dependencies for O32 ABI are removed from the code and now
compilation will be done according n64 ABI only.

TEST=
BUG=

Committed: https://crrev.com/dd64a6d8a1ead06a67490c869ab4fdd13ac80b00
Cr-Commit-Position: refs/heads/master@{#33589}

[Ilija.Pavlovic1, 2016-01-28 12:20:04]

ilija.pavlovic@imgtec.com changed reviewers:
+ akos.palfi@imgtec.com, balazs.kilvady@imgtec.com, gergely.kis@imgtec.com,
ivica.bogosavljevic@imgtec.com, marija.antic@imgtec.com

[Ilija.Pavlovic1, 2016-01-28 12:20:04]

PTAL

[balazs.kilvady, 2016-01-28 12:48:23]

LGTM with a small nit.

https://codereview.chromium.org/1638303005/diff/1/src/regexp/mips64/regexp-ma...
File src/regexp/mips64/regexp-macro-assembler-mips64.cc (right):

https://codereview.chromium.org/1638303005/diff/1/src/regexp/mips64/regexp-ma...
src/regexp/mips64/regexp-macro-assembler-mips64.cc:674: argument_registers |=
a4.bit() | a5.bit() | a6.bit() | a7.bit();
I think a4-a7 bits could be 'or'-ed to the above list, separated expression is
not needed.

[Ilija.Pavlovic1, 2016-01-28 15:07:30]

https://codereview.chromium.org/1638303005/diff/1/src/regexp/mips64/regexp-ma...
File src/regexp/mips64/regexp-macro-assembler-mips64.cc (right):

https://codereview.chromium.org/1638303005/diff/1/src/regexp/mips64/regexp-ma...
src/regexp/mips64/regexp-macro-assembler-mips64.cc:674: argument_registers |=
a4.bit() | a5.bit() | a6.bit() | a7.bit();
On 2016/01/28 12:48:22, balazs.kilvady wrote:
> I think a4-a7 bits could be 'or'-ed to the above list, separated expression is
> not needed.
Yes, you are right. I didn't noted that definition is right above this line. I
decided to leave as it is written because this current notation is also readable
and understandable.

Acknowledged.

[Ilija.Pavlovic1, 2016-01-28 15:11:26]

The CQ bit was checked by ilija.pavlovic@imgtec.com

[commit-bot: I haz the power, 2016-01-28 15:11:37]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1638303005/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1638303005/1

[commit-bot: I haz the power, 2016-01-28 16:03:46]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-28 16:04:26]

Description was changed from

==========
MIPS64: Eliminate ABI-dependent definitions.

Compilation dependencies for O32 ABI are removed from the code and now
compilation will be done according n64 ABI only.

TEST=
BUG=
==========

to

==========
MIPS64: Eliminate ABI-dependent definitions.

Compilation dependencies for O32 ABI are removed from the code and now
compilation will be done according n64 ABI only.

TEST=
BUG=

Committed: https://crrev.com/dd64a6d8a1ead06a67490c869ab4fdd13ac80b00
Cr-Commit-Position: refs/heads/master@{#33589}
==========

[commit-bot: I haz the power, 2016-01-28 16:04:27]

Patchset 1 (id:??) landed as
https://crrev.com/dd64a6d8a1ead06a67490c869ab4fdd13ac80b00
Cr-Commit-Position: refs/heads/master@{#33589}