diff options
author | 2017-03-31 15:26:54 -0700 | |
---|---|---|
committer | 2017-04-10 10:19:30 -0700 | |
commit | cd0295d81b6d53bbade117a0531b2453e8cb7c7f (patch) | |
tree | eceada4e7329ce8fc2e993f414f515c186b81f10 /compiler/utils/mips/assembler_mips.cc | |
parent | ef6787bd892b55588ebb2835cc3a3bc4e9e08d04 (diff) |
MIPS: Use Lsa/Dlsa when possible.
For MIPS32R6 replace instances of "sll/addu" to calculate the
address of an item in an array with "lsa". For other versions of
MIPS32 use the "sll/addu" sequence. Encapsulate this logic in an
assembler method to eliminate having a lot of statements like
"if (IsR6()) { ... } else { ... }" scattered throughout the code.
MIPS64 always supports R6. This means that all instances of
"dsll/daddu" used to calculate the address of an item in an array
can be replaced by "dlsa" so there is no need to encapsulate
conditional logic in a special method. The code can just emit
"dlsa" directly.
Test: mma -j2 ART_TEST_OPTIMIZING=true test-art-target-run-test
Tested on MIPS32, and MIPS64 QEMU.
Test: "make test-art-target-gtest32" on CI20 board.
Test: "cd art; test/testrunner/testrunner.py --target --optimizing --32"
on CI20 board.
Change-Id: Ibe5facc1bc2a6a7a6584e23d3a48e163ae38077d
Diffstat (limited to 'compiler/utils/mips/assembler_mips.cc')
-rw-r--r-- | compiler/utils/mips/assembler_mips.cc | 19 |
1 files changed, 19 insertions, 0 deletions
diff --git a/compiler/utils/mips/assembler_mips.cc b/compiler/utils/mips/assembler_mips.cc index 2e2231b07d..a99d02d4d0 100644 --- a/compiler/utils/mips/assembler_mips.cc +++ b/compiler/utils/mips/assembler_mips.cc @@ -635,6 +635,7 @@ void MipsAssembler::Ins(Register rd, Register rt, int pos, int size) { DsFsmInstrRrr(EmitR(0x1f, rt, rd, static_cast<Register>(pos + size - 1), pos, 0x04), rd, rd, rt); } +// TODO: This instruction is available in both R6 and MSA and it should be used when available. void MipsAssembler::Lsa(Register rd, Register rs, Register rt, int saPlusOne) { CHECK(IsR6()); CHECK(1 <= saPlusOne && saPlusOne <= 4) << saPlusOne; @@ -642,6 +643,24 @@ void MipsAssembler::Lsa(Register rd, Register rs, Register rt, int saPlusOne) { DsFsmInstrRrr(EmitR(0x0, rs, rt, rd, sa, 0x05), rd, rs, rt); } +void MipsAssembler::ShiftAndAdd(Register dst, + Register src_idx, + Register src_base, + int shamt, + Register tmp) { + CHECK(0 <= shamt && shamt <= 4) << shamt; + CHECK_NE(src_base, tmp); + if (shamt == TIMES_1) { + // Catch the special case where the shift amount is zero (0). + Addu(dst, src_base, src_idx); + } else if (IsR6()) { + Lsa(dst, src_idx, src_base, shamt); + } else { + Sll(tmp, src_idx, shamt); + Addu(dst, src_base, tmp); + } +} + void MipsAssembler::Lb(Register rt, Register rs, uint16_t imm16) { DsFsmInstrRrr(EmitI(0x20, rs, rt, imm16), rt, rs, rs); } |