MIPS64: [turbofan] Support Float64Min and Float64Max.

MIPS: [turbofan] Support Float[32|64]Min and Float[32|64]Max.

MIPS64r6 and MIPS32r6 support for min and max and Float64Max machine operators.

BUG=

Committed: https://crrev.com/099e06e2406b934453874760451a64a0db799238
Cr-Commit-Position: refs/heads/master@{#28471}

[balazs.kilvady, 2015-03-23 13:37:53]

balazs.kilvady@imgtec.com changed reviewers:
+ akos.palfi@imgtec.com, dusan.milosavljevic@rt-rk.com, gergely.kis@imgtec.com,
paul.lind@imgtec.com

[balazs.kilvady, 2015-03-23 13:37:53]

PTAL,

Is the arch variant checking is OK and should/could we add min/max to mips32r6
also (in this CL)?

[dusmil.imgtec, 2015-03-23 13:56:23]

Hold on with this, adding instructions in simulator will require simulator
refactoring.
I have patch for sim. refactoring, first CL of refactoring will contain breaking
functions to smaller ones.

[dusmil.imgtec, 2015-03-23 14:05:50]

dusan.milosavljevic@imgtec.com changed reviewers:
+ dusan.milosavljevic@imgtec.com

[balazs.kilvady, 2015-05-14 12:49:51]

balazs.kilvady@imgtec.com changed reviewers:
- dusan.milosavljevic@rt-rk.com

[balazs.kilvady, 2015-05-15 10:24:01]

PTAL

[dusmil.imgtec, 2015-05-15 10:47:11]

lgtm.

[paul.l..., 2015-05-15 19:07:48]

https://codereview.chromium.org/1027193002/diff/40001/src/compiler/mips/instr...
File src/compiler/mips/instruction-selector-mips.cc (right):

https://codereview.chromium.org/1027193002/diff/40001/src/compiler/mips/instr...
src/compiler/mips/instruction-selector-mips.cc:1068: if
((IsMipsArchVariant(kMips32r2) || IsMipsArchVariant(kMips32r6)) &&
You also need to provide implementations for the 4 visitor functions:
InstructionSelector::VisitFloat32Max() ... VisitFloat64Min(). They are currently
UNREACHABLE().

[balazs.kilvady, 2015-05-18 12:24:00]

https://codereview.chromium.org/1027193002/diff/40001/src/compiler/mips/instr...
File src/compiler/mips/instruction-selector-mips.cc (right):

https://codereview.chromium.org/1027193002/diff/40001/src/compiler/mips/instr...
src/compiler/mips/instruction-selector-mips.cc:1068: if
((IsMipsArchVariant(kMips32r2) || IsMipsArchVariant(kMips32r6)) &&
On 2015/05/15 19:07:48, paul.l... wrote:
> You also need to provide implementations for the 4 visitor functions:
> InstructionSelector::VisitFloat32Max() ... VisitFloat64Min(). They are
currently
> UNREACHABLE().

Done.

[paul.l..., 2015-05-18 16:48:36]

LGTM.

[balazs.kilvady, 2015-05-19 09:48:56]

The CQ bit was checked by balazs.kilvady@imgtec.com

[balazs.kilvady, 2015-05-19 09:48:57]

The patchset sent to the CQ was uploaded after l-g-t-m from
dusan.milosavljevic@imgtec.com
Link to the patchset: https://codereview.chromium.org/1027193002/#ps60001
(title: "Add bodies of 32 bits visitor()'s.")

[commit-bot: I haz the power, 2015-05-19 09:49:11]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1027193002/60001

[commit-bot: I haz the power, 2015-05-19 10:22:18]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-05-19 10:22:35]

Patchset 4 (id:??) landed as
https://crrev.com/099e06e2406b934453874760451a64a0db799238
Cr-Commit-Position: refs/heads/master@{#28471}