Skip to content

Commit 2bf7018

Browse files
authored
[RISCV] Move let statement for hasSideEffects, mayLoad, mayStore into BranchCC_rri. NFC (#130721)
This is consistent with the isBranch and isTerminator flags already in the class. Addresses feedback given on #130714 where I copied the inconsistent split into RISCVInstrInfoXCV.td.
1 parent 90c11ad commit 2bf7018

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

llvm/lib/Target/RISCV/RISCVInstrInfo.td

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -523,14 +523,16 @@ include "RISCVInstrFormats.td"
523523
// Instruction Class Templates
524524
//===----------------------------------------------------------------------===//
525525

526-
let hasSideEffects = 0, mayLoad = 0, mayStore = 0 in
527526
class BranchCC_rri<bits<3> funct3, string opcodestr>
528527
: RVInstB<funct3, OPC_BRANCH, (outs),
529528
(ins GPR:$rs1, GPR:$rs2, simm13_lsb0:$imm12),
530529
opcodestr, "$rs1, $rs2, $imm12">,
531530
Sched<[WriteJmp, ReadJmp, ReadJmp]> {
532531
let isBranch = 1;
533532
let isTerminator = 1;
533+
let hasSideEffects = 0;
534+
let mayLoad = 0;
535+
let mayStore = 0;
534536
}
535537

536538
let hasSideEffects = 0, mayLoad = 1, mayStore = 0 in {

0 commit comments

Comments
 (0)