xref: /freebsd/contrib/llvm-project/llvm/lib/Target/X86/X86FixupSetCC.cpp (revision e8d8bef961a50d4dc22501cde4fb9fb0be1b2532)
10b57cec5SDimitry Andric //===---- X86FixupSetCC.cpp - optimize usage of LEA instructions ----------===//
20b57cec5SDimitry Andric //
30b57cec5SDimitry Andric // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
40b57cec5SDimitry Andric // See https://llvm.org/LICENSE.txt for license information.
50b57cec5SDimitry Andric // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
60b57cec5SDimitry Andric //
70b57cec5SDimitry Andric //===----------------------------------------------------------------------===//
80b57cec5SDimitry Andric //
90b57cec5SDimitry Andric // This file defines a pass that fixes zero-extension of setcc patterns.
100b57cec5SDimitry Andric // X86 setcc instructions are modeled to have no input arguments, and a single
110b57cec5SDimitry Andric // GR8 output argument. This is consistent with other similar instructions
120b57cec5SDimitry Andric // (e.g. movb), but means it is impossible to directly generate a setcc into
130b57cec5SDimitry Andric // the lower GR8 of a specified GR32.
140b57cec5SDimitry Andric // This means that ISel must select (zext (setcc)) into something like
150b57cec5SDimitry Andric // seta %al; movzbl %al, %eax.
160b57cec5SDimitry Andric // Unfortunately, this can cause a stall due to the partial register write
170b57cec5SDimitry Andric // performed by the setcc. Instead, we can use:
180b57cec5SDimitry Andric // xor %eax, %eax; seta %al
190b57cec5SDimitry Andric // This both avoids the stall, and encodes shorter.
200b57cec5SDimitry Andric //===----------------------------------------------------------------------===//
210b57cec5SDimitry Andric 
220b57cec5SDimitry Andric #include "X86.h"
230b57cec5SDimitry Andric #include "X86InstrInfo.h"
240b57cec5SDimitry Andric #include "X86Subtarget.h"
250b57cec5SDimitry Andric #include "llvm/ADT/Statistic.h"
260b57cec5SDimitry Andric #include "llvm/CodeGen/MachineFunctionPass.h"
270b57cec5SDimitry Andric #include "llvm/CodeGen/MachineInstrBuilder.h"
280b57cec5SDimitry Andric #include "llvm/CodeGen/MachineRegisterInfo.h"
290b57cec5SDimitry Andric 
300b57cec5SDimitry Andric using namespace llvm;
310b57cec5SDimitry Andric 
320b57cec5SDimitry Andric #define DEBUG_TYPE "x86-fixup-setcc"
330b57cec5SDimitry Andric 
340b57cec5SDimitry Andric STATISTIC(NumSubstZexts, "Number of setcc + zext pairs substituted");
350b57cec5SDimitry Andric 
360b57cec5SDimitry Andric namespace {
370b57cec5SDimitry Andric class X86FixupSetCCPass : public MachineFunctionPass {
380b57cec5SDimitry Andric public:
395ffd83dbSDimitry Andric   static char ID;
405ffd83dbSDimitry Andric 
410b57cec5SDimitry Andric   X86FixupSetCCPass() : MachineFunctionPass(ID) {}
420b57cec5SDimitry Andric 
430b57cec5SDimitry Andric   StringRef getPassName() const override { return "X86 Fixup SetCC"; }
440b57cec5SDimitry Andric 
450b57cec5SDimitry Andric   bool runOnMachineFunction(MachineFunction &MF) override;
460b57cec5SDimitry Andric 
470b57cec5SDimitry Andric private:
48480093f4SDimitry Andric   MachineRegisterInfo *MRI = nullptr;
49480093f4SDimitry Andric   const X86InstrInfo *TII = nullptr;
500b57cec5SDimitry Andric 
510b57cec5SDimitry Andric   enum { SearchBound = 16 };
520b57cec5SDimitry Andric };
535ffd83dbSDimitry Andric } // end anonymous namespace
540b57cec5SDimitry Andric 
550b57cec5SDimitry Andric char X86FixupSetCCPass::ID = 0;
565ffd83dbSDimitry Andric 
575ffd83dbSDimitry Andric INITIALIZE_PASS(X86FixupSetCCPass, DEBUG_TYPE, DEBUG_TYPE, false, false)
580b57cec5SDimitry Andric 
590b57cec5SDimitry Andric FunctionPass *llvm::createX86FixupSetCC() { return new X86FixupSetCCPass(); }
600b57cec5SDimitry Andric 
610b57cec5SDimitry Andric bool X86FixupSetCCPass::runOnMachineFunction(MachineFunction &MF) {
620b57cec5SDimitry Andric   bool Changed = false;
630b57cec5SDimitry Andric   MRI = &MF.getRegInfo();
640b57cec5SDimitry Andric   TII = MF.getSubtarget<X86Subtarget>().getInstrInfo();
650b57cec5SDimitry Andric 
660b57cec5SDimitry Andric   SmallVector<MachineInstr*, 4> ToErase;
670b57cec5SDimitry Andric 
680b57cec5SDimitry Andric   for (auto &MBB : MF) {
69480093f4SDimitry Andric     MachineInstr *FlagsDefMI = nullptr;
700b57cec5SDimitry Andric     for (auto &MI : MBB) {
71480093f4SDimitry Andric       // Remember the most recent preceding eflags defining instruction.
72480093f4SDimitry Andric       if (MI.definesRegister(X86::EFLAGS))
73480093f4SDimitry Andric         FlagsDefMI = &MI;
74480093f4SDimitry Andric 
750b57cec5SDimitry Andric       // Find a setcc that is used by a zext.
760b57cec5SDimitry Andric       // This doesn't have to be the only use, the transformation is safe
770b57cec5SDimitry Andric       // regardless.
780b57cec5SDimitry Andric       if (MI.getOpcode() != X86::SETCCr)
790b57cec5SDimitry Andric         continue;
800b57cec5SDimitry Andric 
810b57cec5SDimitry Andric       MachineInstr *ZExt = nullptr;
820b57cec5SDimitry Andric       for (auto &Use : MRI->use_instructions(MI.getOperand(0).getReg()))
830b57cec5SDimitry Andric         if (Use.getOpcode() == X86::MOVZX32rr8)
840b57cec5SDimitry Andric           ZExt = &Use;
850b57cec5SDimitry Andric 
860b57cec5SDimitry Andric       if (!ZExt)
870b57cec5SDimitry Andric         continue;
880b57cec5SDimitry Andric 
890b57cec5SDimitry Andric       if (!FlagsDefMI)
900b57cec5SDimitry Andric         continue;
910b57cec5SDimitry Andric 
920b57cec5SDimitry Andric       // We'd like to put something that clobbers eflags directly before
930b57cec5SDimitry Andric       // FlagsDefMI. This can't hurt anything after FlagsDefMI, because
940b57cec5SDimitry Andric       // it, itself, by definition, clobbers eflags. But it may happen that
950b57cec5SDimitry Andric       // FlagsDefMI also *uses* eflags, in which case the transformation is
960b57cec5SDimitry Andric       // invalid.
97480093f4SDimitry Andric       if (FlagsDefMI->readsRegister(X86::EFLAGS))
980b57cec5SDimitry Andric         continue;
990b57cec5SDimitry Andric 
1000b57cec5SDimitry Andric       // On 32-bit, we need to be careful to force an ABCD register.
1010b57cec5SDimitry Andric       const TargetRegisterClass *RC = MF.getSubtarget<X86Subtarget>().is64Bit()
1020b57cec5SDimitry Andric                                           ? &X86::GR32RegClass
1030b57cec5SDimitry Andric                                           : &X86::GR32_ABCDRegClass;
104*e8d8bef9SDimitry Andric       if (!MRI->constrainRegClass(ZExt->getOperand(0).getReg(), RC)) {
105*e8d8bef9SDimitry Andric         // If we cannot constrain the register, we would need an additional copy
106*e8d8bef9SDimitry Andric         // and are better off keeping the MOVZX32rr8 we have now.
107*e8d8bef9SDimitry Andric         continue;
108*e8d8bef9SDimitry Andric       }
109*e8d8bef9SDimitry Andric 
110*e8d8bef9SDimitry Andric       ++NumSubstZexts;
111*e8d8bef9SDimitry Andric       Changed = true;
1120b57cec5SDimitry Andric 
1130b57cec5SDimitry Andric       // Initialize a register with 0. This must go before the eflags def
114*e8d8bef9SDimitry Andric       Register ZeroReg = MRI->createVirtualRegister(RC);
1150b57cec5SDimitry Andric       BuildMI(MBB, FlagsDefMI, MI.getDebugLoc(), TII->get(X86::MOV32r0),
1160b57cec5SDimitry Andric               ZeroReg);
1170b57cec5SDimitry Andric 
1180b57cec5SDimitry Andric       // X86 setcc only takes an output GR8, so fake a GR32 input by inserting
1190b57cec5SDimitry Andric       // the setcc result into the low byte of the zeroed register.
1200b57cec5SDimitry Andric       BuildMI(*ZExt->getParent(), ZExt, ZExt->getDebugLoc(),
121*e8d8bef9SDimitry Andric               TII->get(X86::INSERT_SUBREG), ZExt->getOperand(0).getReg())
1220b57cec5SDimitry Andric           .addReg(ZeroReg)
1230b57cec5SDimitry Andric           .addReg(MI.getOperand(0).getReg())
1240b57cec5SDimitry Andric           .addImm(X86::sub_8bit);
1250b57cec5SDimitry Andric       ToErase.push_back(ZExt);
1260b57cec5SDimitry Andric     }
1270b57cec5SDimitry Andric   }
1280b57cec5SDimitry Andric 
1290b57cec5SDimitry Andric   for (auto &I : ToErase)
1300b57cec5SDimitry Andric     I->eraseFromParent();
1310b57cec5SDimitry Andric 
1320b57cec5SDimitry Andric   return Changed;
1330b57cec5SDimitry Andric }
134