MIPS: Correct handling of Nan values on MIPS R6

MIPS: Correct handling of Nan values on MIPS R6

MIPS R6 introduced new behavior for handling of NaN values
for TRUNC, FLOOR, CEIL and CVT instructions. Adding support for
the new behavior in MIPS and MIPS64 simulators. Fixing tests
for MIPS and MIPS64 to align them with the new behavior.

BUG=

Committed: https://crrev.com/60d77c8a4378f8af68c384989e4fcbdb4eb062df
Cr-Commit-Position: refs/heads/master@{#32499}

[ivica.bogosavljevic, 2015-12-01 11:01:38]

Description was changed from

==========
MIPS: Correct handling of Nan values on MIPS R6

MIPS R6 introduced new behavior for handling of NaN values
for TRUNC, FLOOR, CEIL and CVT instructions. Adding support for
the new behavior in MIPS and MIPS64 simulators. Fixing tests
for MIPS and MIPS64 to align them with the new behavior.

BUG=
==========

to

==========
MIPS: Correct handling of Nan values on MIPS R6

MIPS R6 introduced new behavior for handling of NaN values
for TRUNC, FLOOR, CEIL and CVT instructions. Adding support for
the new behavior in MIPS and MIPS64 simulators. Fixing tests
for MIPS and MIPS64 to align them with the new behavior.

BUG=
==========

[ivica.bogosavljevic, 2015-12-01 11:01:38]

ivica.bogosavljevic@imgtec.com changed reviewers:
+ akos.palfi@imgtec.com, balazs.kilvady@imgtec.com, djordje.pesic@imgtec.com,
ilija.pavlovic@imgtec.com, paul.lind@imgtec.com

[paul.l..., 2015-12-02 02:49:52]

LGTM, after fixing the nits.

https://codereview.chromium.org/1488613007/diff/1/src/mips/simulator-mips.cc
File src/mips/simulator-mips.cc (right):

https://codereview.chromium.org/1488613007/diff/1/src/mips/simulator-mips.cc#...
src/mips/simulator-mips.cc:1580: printf("Rounded int = %d\n", rounded_int);
Leftover debugging prints? Please remove, here and just above.

[Ilija.Pavlovic1, 2015-12-02 09:12:16]

ilija.pavlovic@imgtec.com changed reviewers:
+ Ilija.Pavlovic@imgtec.com

[Ilija.Pavlovic1, 2015-12-02 09:12:17]

lgtm

https://codereview.chromium.org/1488613007/diff/1/test/cctest/test-assembler-...
File test/cctest/test-assembler-mips.cc (right):

https://codereview.chromium.org/1488613007/diff/1/test/cctest/test-assembler-...
test/cctest/test-assembler-mips.cc:2299: printf("Type = %d, i = %d\n", j, i);
Should be deleted/removed?

[ivica.bogosavljevic, 2015-12-02 10:00:34]

The CQ bit was checked by ivica.bogosavljevic@imgtec.com to run a CQ dry run

[commit-bot: I haz the power, 2015-12-02 10:00:41]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1488613007/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1488613007/20001

[ivica.bogosavljevic, 2015-12-02 10:31:47]

The CQ bit was checked by ivica.bogosavljevic@imgtec.com to run a CQ dry run

[commit-bot: I haz the power, 2015-12-02 10:31:55]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1488613007/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1488613007/40001

[commit-bot: I haz the power, 2015-12-02 10:51:06]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-12-02 10:51:07]

Dry run: This issue passed the CQ dry run.

[ivica.bogosavljevic, 2015-12-02 10:53:10]

The CQ bit was checked by ivica.bogosavljevic@imgtec.com

[ivica.bogosavljevic, 2015-12-02 10:53:10]

The patchset sent to the CQ was uploaded after l-g-t-m from
Ilija.Pavlovic@imgtec.com, paul.lind@imgtec.com
Link to the patchset: https://codereview.chromium.org/1488613007/#ps40001
(title: "Removing leftover printfs")

[commit-bot: I haz the power, 2015-12-02 10:53:14]

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

[commit-bot: I haz the power, 2015-12-02 10:54:41]

Description was changed from

==========
MIPS: Correct handling of Nan values on MIPS R6

MIPS R6 introduced new behavior for handling of NaN values
for TRUNC, FLOOR, CEIL and CVT instructions. Adding support for
the new behavior in MIPS and MIPS64 simulators. Fixing tests
for MIPS and MIPS64 to align them with the new behavior.

BUG=
==========

to

==========
MIPS: Correct handling of Nan values on MIPS R6

MIPS R6 introduced new behavior for handling of NaN values
for TRUNC, FLOOR, CEIL and CVT instructions. Adding support for
the new behavior in MIPS and MIPS64 simulators. Fixing tests
for MIPS and MIPS64 to align them with the new behavior.

BUG=
==========

[commit-bot: I haz the power, 2015-12-02 10:54:42]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-12-02 10:55:32]

Description was changed from

==========
MIPS: Correct handling of Nan values on MIPS R6

MIPS R6 introduced new behavior for handling of NaN values
for TRUNC, FLOOR, CEIL and CVT instructions. Adding support for
the new behavior in MIPS and MIPS64 simulators. Fixing tests
for MIPS and MIPS64 to align them with the new behavior.

BUG=
==========

to

==========
MIPS: Correct handling of Nan values on MIPS R6

MIPS R6 introduced new behavior for handling of NaN values
for TRUNC, FLOOR, CEIL and CVT instructions. Adding support for
the new behavior in MIPS and MIPS64 simulators. Fixing tests
for MIPS and MIPS64 to align them with the new behavior.

BUG=

Committed: https://crrev.com/60d77c8a4378f8af68c384989e4fcbdb4eb062df
Cr-Commit-Position: refs/heads/master@{#32499}
==========

[commit-bot: I haz the power, 2015-12-02 10:55:32]

Patchset 3 (id:??) landed as
https://crrev.com/60d77c8a4378f8af68c384989e4fcbdb4eb062df
Cr-Commit-Position: refs/heads/master@{#32499}