123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377 |
- //===-------------- RISCVSExtWRemoval.cpp - MI sext.w Removal -------------===//
- //
- // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
- // See https://llvm.org/LICENSE.txt for license information.
- // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
- //
- //===---------------------------------------------------------------------===//
- //
- // This pass removes unneeded sext.w instructions at the MI level. Either
- // because the sign extended bits aren't consumed or because the input was
- // already sign extended by an earlier instruction.
- //
- //===---------------------------------------------------------------------===//
- #include "RISCV.h"
- #include "RISCVMachineFunctionInfo.h"
- #include "RISCVSubtarget.h"
- #include "llvm/ADT/Statistic.h"
- #include "llvm/CodeGen/MachineFunctionPass.h"
- #include "llvm/CodeGen/TargetInstrInfo.h"
- using namespace llvm;
- #define DEBUG_TYPE "riscv-sextw-removal"
- STATISTIC(NumRemovedSExtW, "Number of removed sign-extensions");
- STATISTIC(NumTransformedToWInstrs,
- "Number of instructions transformed to W-ops");
- static cl::opt<bool> DisableSExtWRemoval("riscv-disable-sextw-removal",
- cl::desc("Disable removal of sext.w"),
- cl::init(false), cl::Hidden);
- namespace {
- class RISCVSExtWRemoval : public MachineFunctionPass {
- public:
- static char ID;
- RISCVSExtWRemoval() : MachineFunctionPass(ID) {
- initializeRISCVSExtWRemovalPass(*PassRegistry::getPassRegistry());
- }
- bool runOnMachineFunction(MachineFunction &MF) override;
- void getAnalysisUsage(AnalysisUsage &AU) const override {
- AU.setPreservesCFG();
- MachineFunctionPass::getAnalysisUsage(AU);
- }
- StringRef getPassName() const override { return "RISCV sext.w Removal"; }
- };
- } // end anonymous namespace
- char RISCVSExtWRemoval::ID = 0;
- INITIALIZE_PASS(RISCVSExtWRemoval, DEBUG_TYPE, "RISCV sext.w Removal", false,
- false)
- FunctionPass *llvm::createRISCVSExtWRemovalPass() {
- return new RISCVSExtWRemoval();
- }
- // This function returns true if the machine instruction always outputs a value
- // where bits 63:32 match bit 31.
- static bool isSignExtendingOpW(const MachineInstr &MI,
- const MachineRegisterInfo &MRI) {
- uint64_t TSFlags = MI.getDesc().TSFlags;
- // Instructions that can be determined from opcode are marked in tablegen.
- if (TSFlags & RISCVII::IsSignExtendingOpWMask)
- return true;
- // Special cases that require checking operands.
- switch (MI.getOpcode()) {
- // shifting right sufficiently makes the value 32-bit sign-extended
- case RISCV::SRAI:
- return MI.getOperand(2).getImm() >= 32;
- case RISCV::SRLI:
- return MI.getOperand(2).getImm() > 32;
- // The LI pattern ADDI rd, X0, imm is sign extended.
- case RISCV::ADDI:
- return MI.getOperand(1).isReg() && MI.getOperand(1).getReg() == RISCV::X0;
- // An ANDI with an 11 bit immediate will zero bits 63:11.
- case RISCV::ANDI:
- return isUInt<11>(MI.getOperand(2).getImm());
- // An ORI with an >11 bit immediate (negative 12-bit) will set bits 63:11.
- case RISCV::ORI:
- return !isUInt<11>(MI.getOperand(2).getImm());
- // Copying from X0 produces zero.
- case RISCV::COPY:
- return MI.getOperand(1).getReg() == RISCV::X0;
- }
- return false;
- }
- static bool isSignExtendedW(Register SrcReg, const MachineRegisterInfo &MRI,
- const RISCVInstrInfo &TII,
- SmallPtrSetImpl<MachineInstr *> &FixableDef) {
- SmallPtrSet<const MachineInstr *, 4> Visited;
- SmallVector<MachineInstr *, 4> Worklist;
- auto AddRegDefToWorkList = [&](Register SrcReg) {
- if (!SrcReg.isVirtual())
- return false;
- MachineInstr *SrcMI = MRI.getVRegDef(SrcReg);
- if (!SrcMI)
- return false;
- // Add SrcMI to the worklist.
- Worklist.push_back(SrcMI);
- return true;
- };
- if (!AddRegDefToWorkList(SrcReg))
- return false;
- while (!Worklist.empty()) {
- MachineInstr *MI = Worklist.pop_back_val();
- // If we already visited this instruction, we don't need to check it again.
- if (!Visited.insert(MI).second)
- continue;
- // If this is a sign extending operation we don't need to look any further.
- if (isSignExtendingOpW(*MI, MRI))
- continue;
- // Is this an instruction that propagates sign extend?
- switch (MI->getOpcode()) {
- default:
- // Unknown opcode, give up.
- return false;
- case RISCV::COPY: {
- const MachineFunction *MF = MI->getMF();
- const RISCVMachineFunctionInfo *RVFI =
- MF->getInfo<RISCVMachineFunctionInfo>();
- // If this is the entry block and the register is livein, see if we know
- // it is sign extended.
- if (MI->getParent() == &MF->front()) {
- Register VReg = MI->getOperand(0).getReg();
- if (MF->getRegInfo().isLiveIn(VReg) && RVFI->isSExt32Register(VReg))
- continue;
- }
- Register CopySrcReg = MI->getOperand(1).getReg();
- if (CopySrcReg == RISCV::X10) {
- // For a method return value, we check the ZExt/SExt flags in attribute.
- // We assume the following code sequence for method call.
- // PseudoCALL @bar, ...
- // ADJCALLSTACKUP 0, 0, implicit-def dead $x2, implicit $x2
- // %0:gpr = COPY $x10
- //
- // We use the PseudoCall to look up the IR function being called to find
- // its return attributes.
- const MachineBasicBlock *MBB = MI->getParent();
- auto II = MI->getIterator();
- if (II == MBB->instr_begin() ||
- (--II)->getOpcode() != RISCV::ADJCALLSTACKUP)
- return false;
- const MachineInstr &CallMI = *(--II);
- if (!CallMI.isCall() || !CallMI.getOperand(0).isGlobal())
- return false;
- auto *CalleeFn =
- dyn_cast_if_present<Function>(CallMI.getOperand(0).getGlobal());
- if (!CalleeFn)
- return false;
- auto *IntTy = dyn_cast<IntegerType>(CalleeFn->getReturnType());
- if (!IntTy)
- return false;
- const AttributeSet &Attrs = CalleeFn->getAttributes().getRetAttrs();
- unsigned BitWidth = IntTy->getBitWidth();
- if ((BitWidth <= 32 && Attrs.hasAttribute(Attribute::SExt)) ||
- (BitWidth < 32 && Attrs.hasAttribute(Attribute::ZExt)))
- continue;
- }
- if (!AddRegDefToWorkList(CopySrcReg))
- return false;
- break;
- }
- // For these, we just need to check if the 1st operand is sign extended.
- case RISCV::BCLRI:
- case RISCV::BINVI:
- case RISCV::BSETI:
- if (MI->getOperand(2).getImm() >= 31)
- return false;
- [[fallthrough]];
- case RISCV::REM:
- case RISCV::ANDI:
- case RISCV::ORI:
- case RISCV::XORI:
- // |Remainder| is always <= |Dividend|. If D is 32-bit, then so is R.
- // DIV doesn't work because of the edge case 0xf..f 8000 0000 / (long)-1
- // Logical operations use a sign extended 12-bit immediate.
- if (!AddRegDefToWorkList(MI->getOperand(1).getReg()))
- return false;
- break;
- case RISCV::PseudoCCADDW:
- case RISCV::PseudoCCSUBW:
- // Returns operand 4 or an ADDW/SUBW of operands 5 and 6. We only need to
- // check if operand 4 is sign extended.
- if (!AddRegDefToWorkList(MI->getOperand(4).getReg()))
- return false;
- break;
- case RISCV::REMU:
- case RISCV::AND:
- case RISCV::OR:
- case RISCV::XOR:
- case RISCV::ANDN:
- case RISCV::ORN:
- case RISCV::XNOR:
- case RISCV::MAX:
- case RISCV::MAXU:
- case RISCV::MIN:
- case RISCV::MINU:
- case RISCV::PseudoCCMOVGPR:
- case RISCV::PseudoCCAND:
- case RISCV::PseudoCCOR:
- case RISCV::PseudoCCXOR:
- case RISCV::PHI: {
- // If all incoming values are sign-extended, the output of AND, OR, XOR,
- // MIN, MAX, or PHI is also sign-extended.
- // The input registers for PHI are operand 1, 3, ...
- // The input registers for PseudoCCMOVGPR are 4 and 5.
- // The input registers for PseudoCCAND/OR/XOR are 4, 5, and 6.
- // The input registers for others are operand 1 and 2.
- unsigned B = 1, E = 3, D = 1;
- switch (MI->getOpcode()) {
- case RISCV::PHI:
- E = MI->getNumOperands();
- D = 2;
- break;
- case RISCV::PseudoCCMOVGPR:
- B = 4;
- E = 6;
- break;
- case RISCV::PseudoCCAND:
- case RISCV::PseudoCCOR:
- case RISCV::PseudoCCXOR:
- B = 4;
- E = 7;
- break;
- }
- for (unsigned I = B; I != E; I += D) {
- if (!MI->getOperand(I).isReg())
- return false;
- if (!AddRegDefToWorkList(MI->getOperand(I).getReg()))
- return false;
- }
- break;
- }
- case RISCV::VT_MASKC:
- case RISCV::VT_MASKCN:
- // Instructions return zero or operand 1. Result is sign extended if
- // operand 1 is sign extended.
- if (!AddRegDefToWorkList(MI->getOperand(1).getReg()))
- return false;
- break;
- // With these opcode, we can "fix" them with the W-version
- // if we know all users of the result only rely on bits 31:0
- case RISCV::SLLI:
- // SLLIW reads the lowest 5 bits, while SLLI reads lowest 6 bits
- if (MI->getOperand(2).getImm() >= 32)
- return false;
- [[fallthrough]];
- case RISCV::ADDI:
- case RISCV::ADD:
- case RISCV::LD:
- case RISCV::LWU:
- case RISCV::MUL:
- case RISCV::SUB:
- if (TII.hasAllWUsers(*MI, MRI)) {
- FixableDef.insert(MI);
- break;
- }
- return false;
- }
- }
- // If we get here, then every node we visited produces a sign extended value
- // or propagated sign extended values. So the result must be sign extended.
- return true;
- }
- static unsigned getWOp(unsigned Opcode) {
- switch (Opcode) {
- case RISCV::ADDI:
- return RISCV::ADDIW;
- case RISCV::ADD:
- return RISCV::ADDW;
- case RISCV::LD:
- case RISCV::LWU:
- return RISCV::LW;
- case RISCV::MUL:
- return RISCV::MULW;
- case RISCV::SLLI:
- return RISCV::SLLIW;
- case RISCV::SUB:
- return RISCV::SUBW;
- default:
- llvm_unreachable("Unexpected opcode for replacement with W variant");
- }
- }
- bool RISCVSExtWRemoval::runOnMachineFunction(MachineFunction &MF) {
- if (skipFunction(MF.getFunction()) || DisableSExtWRemoval)
- return false;
- MachineRegisterInfo &MRI = MF.getRegInfo();
- const RISCVSubtarget &ST = MF.getSubtarget<RISCVSubtarget>();
- const RISCVInstrInfo &TII = *ST.getInstrInfo();
- if (!ST.is64Bit())
- return false;
- bool MadeChange = false;
- for (MachineBasicBlock &MBB : MF) {
- for (auto I = MBB.begin(), IE = MBB.end(); I != IE;) {
- MachineInstr *MI = &*I++;
- // We're looking for the sext.w pattern ADDIW rd, rs1, 0.
- if (!RISCV::isSEXT_W(*MI))
- continue;
- Register SrcReg = MI->getOperand(1).getReg();
- SmallPtrSet<MachineInstr *, 4> FixableDefs;
- // If all users only use the lower bits, this sext.w is redundant.
- // Or if all definitions reaching MI sign-extend their output,
- // then sext.w is redundant.
- if (!TII.hasAllWUsers(*MI, MRI) &&
- !isSignExtendedW(SrcReg, MRI, TII, FixableDefs))
- continue;
- Register DstReg = MI->getOperand(0).getReg();
- if (!MRI.constrainRegClass(SrcReg, MRI.getRegClass(DstReg)))
- continue;
- // Convert Fixable instructions to their W versions.
- for (MachineInstr *Fixable : FixableDefs) {
- LLVM_DEBUG(dbgs() << "Replacing " << *Fixable);
- Fixable->setDesc(TII.get(getWOp(Fixable->getOpcode())));
- Fixable->clearFlag(MachineInstr::MIFlag::NoSWrap);
- Fixable->clearFlag(MachineInstr::MIFlag::NoUWrap);
- Fixable->clearFlag(MachineInstr::MIFlag::IsExact);
- LLVM_DEBUG(dbgs() << " with " << *Fixable);
- ++NumTransformedToWInstrs;
- }
- LLVM_DEBUG(dbgs() << "Removing redundant sign-extension\n");
- MRI.replaceRegWith(DstReg, SrcReg);
- MRI.clearKillFlags(SrcReg);
- MI->eraseFromParent();
- ++NumRemovedSExtW;
- MadeChange = true;
- }
- }
- return MadeChange;
- }
|