Skip to content

Commit ac60477

Browse files
authored
[Mips] Decode the operand of JrcRx16 (#157357)
This change will allow to remove `-ignore-non-decodable-operands` TableGen option. The J(AL)R(C) instructions still have incorrect encodings, but it seems there is only one mips16 test that only checks that `nop` can be assembled.
1 parent 96c1201 commit ac60477

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

llvm/lib/Target/Mips/Mips16InstrInfo.td

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -374,8 +374,8 @@ class FRR16_JALRC_RA_only_ins<bits<1> nd_, bits<1> l_,
374374

375375
class FRR16_JALRC_ins<bits<1> nd, bits<1> l, bits<1> ra,
376376
string asmstr, InstrItinClass itin>:
377-
FRR16_JALRC<nd, l, ra, (outs), (ins CPU16Regs:$rs),
378-
!strconcat(asmstr, "\t$rs"), [], itin> ;
377+
FRR16_JALRC<nd, l, ra, (outs), (ins CPU16Regs:$rx),
378+
!strconcat(asmstr, "\t$rx"), [], itin> ;
379379

380380
class FRR_SF16_ins
381381
<bits<5> _funct, bits<3> _subfunc,
@@ -776,7 +776,6 @@ def JrcRa16: FRR16_JALRC_RA_only_ins<1, 1, "jrc", IIM16Alu> {
776776
}
777777

778778
def JrcRx16: FRR16_JALRC_ins<1, 1, 0, "jrc", IIM16Alu> {
779-
let rx = 0b000;
780779
let isBranch = 1;
781780
let isIndirectBranch = 1;
782781
let isTerminator=1;

0 commit comments

Comments
 (0)