MIPS: Add float instructions and test coverage, part two

MIPS: Add float instructions and test coverage, part two

Implement assembler, disassembler tests for all instructions for mips32 and mips64. Additionally, add missing single precision float instructions for r2 and r6 architecture variants in assembler, simulator and disassembler with corresponding tests.

Committed: https://crrev.com/874c54e05e92e04590b1026b495108abfd6063b7
Cr-Commit-Position: refs/heads/master@{#28595}

[Djordje.Pesic, 2015-05-20 15:12:25]

djordje.pesic@imgtec.com changed reviewers:
+ akos.palfi@imgtec.com, balazs.kilvady@imgtec.com,
dusan.milosavljevic@imgtec.com, paul.lind@imgtec.com

[dusmil.imgtec, 2015-05-22 11:03:37]

https://codereview.chromium.org/1145223002/diff/1/test/cctest/test-disasm-mip...
File test/cctest/test-disasm-mips.cc (right):

https://codereview.chromium.org/1145223002/diff/1/test/cctest/test-disasm-mip...
test/cctest/test-disasm-mips.cc:635: if (kArchVariant == kMips32r6) {
It should be IsMipsArchVariant() as above.

https://codereview.chromium.org/1145223002/diff/1/test/cctest/test-disasm-mip...
test/cctest/test-disasm-mips.cc:680: if (kArchVariant == kMips32r6) {
ditto.

https://codereview.chromium.org/1145223002/diff/1/test/cctest/test-disasm-mip...
test/cctest/test-disasm-mips.cc:725: if (kArchVariant == kMips32r6 ||
kArchVariant == kMips32r2) {
ditto.

[Djordje.Pesic, 2015-05-22 11:46:35]

https://codereview.chromium.org/1145223002/diff/1/test/cctest/test-disasm-mip...
File test/cctest/test-disasm-mips.cc (right):

https://codereview.chromium.org/1145223002/diff/1/test/cctest/test-disasm-mip...
test/cctest/test-disasm-mips.cc:635: if (kArchVariant == kMips32r6) {
On 2015/05/22 11:03:37, dusmil.imgtec wrote:
> It should be IsMipsArchVariant() as above.

Done.

https://codereview.chromium.org/1145223002/diff/1/test/cctest/test-disasm-mip...
test/cctest/test-disasm-mips.cc:680: if (kArchVariant == kMips32r6) {
On 2015/05/22 11:03:37, dusmil.imgtec wrote:
> ditto.

Done.

https://codereview.chromium.org/1145223002/diff/1/test/cctest/test-disasm-mip...
test/cctest/test-disasm-mips.cc:725: if (kArchVariant == kMips32r6 ||
kArchVariant == kMips32r2) {
On 2015/05/22 11:03:37, dusmil.imgtec wrote:
> ditto.

Done.

[dusmil.imgtec, 2015-05-22 12:50:04]

LGTM.

[Djordje.Pesic, 2015-05-22 13:08:16]

The CQ bit was checked by djordje.pesic@imgtec.com

[commit-bot: I haz the power, 2015-05-22 13:08:30]

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

[commit-bot: I haz the power, 2015-05-22 13:49:27]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-05-22 13:49:39]

Patchset 2 (id:??) landed as
https://crrev.com/874c54e05e92e04590b1026b495108abfd6063b7
Cr-Commit-Position: refs/heads/master@{#28595}

[Paul.Lind_imgtec.com, 2015-05-22 13:49:50]

I am out of the office from Friday May 22 through Monday June 1, 2015.

I will have occasional email access during this time. There may be a multiple
day delay in my response.

Thanks,

Paul