Revert r354244 "[DAGCombiner] Eliminate dead stores to stack."
[llvm-complete.git] / lib / Target / X86 / X86FixupSetCC.cpp
blob5bfad71ec05bdf5efa95312296dda9afb13952b6
1 //===---- X86FixupSetCC.cpp - optimize usage of LEA instructions ----------===//
2 //
3 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
4 // See https://llvm.org/LICENSE.txt for license information.
5 // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
6 //
7 //===----------------------------------------------------------------------===//
8 //
9 // This file defines a pass that fixes zero-extension of setcc patterns.
10 // X86 setcc instructions are modeled to have no input arguments, and a single
11 // GR8 output argument. This is consistent with other similar instructions
12 // (e.g. movb), but means it is impossible to directly generate a setcc into
13 // the lower GR8 of a specified GR32.
14 // This means that ISel must select (zext (setcc)) into something like
15 // seta %al; movzbl %al, %eax.
16 // Unfortunately, this can cause a stall due to the partial register write
17 // performed by the setcc. Instead, we can use:
18 // xor %eax, %eax; seta %al
19 // This both avoids the stall, and encodes shorter.
20 //===----------------------------------------------------------------------===//
22 #include "X86.h"
23 #include "X86InstrInfo.h"
24 #include "X86Subtarget.h"
25 #include "llvm/ADT/Statistic.h"
26 #include "llvm/CodeGen/MachineFunctionPass.h"
27 #include "llvm/CodeGen/MachineInstrBuilder.h"
28 #include "llvm/CodeGen/MachineRegisterInfo.h"
30 using namespace llvm;
32 #define DEBUG_TYPE "x86-fixup-setcc"
34 STATISTIC(NumSubstZexts, "Number of setcc + zext pairs substituted");
36 namespace {
37 class X86FixupSetCCPass : public MachineFunctionPass {
38 public:
39 X86FixupSetCCPass() : MachineFunctionPass(ID) {}
41 StringRef getPassName() const override { return "X86 Fixup SetCC"; }
43 bool runOnMachineFunction(MachineFunction &MF) override;
45 private:
46 // Find the preceding instruction that imp-defs eflags.
47 MachineInstr *findFlagsImpDef(MachineBasicBlock *MBB,
48 MachineBasicBlock::reverse_iterator MI);
50 // Return true if MI imp-uses eflags.
51 bool impUsesFlags(MachineInstr *MI);
53 // Return true if this is the opcode of a SetCC instruction with a register
54 // output.
55 bool isSetCCr(unsigned Opode);
57 MachineRegisterInfo *MRI;
58 const X86InstrInfo *TII;
60 enum { SearchBound = 16 };
62 static char ID;
65 char X86FixupSetCCPass::ID = 0;
68 FunctionPass *llvm::createX86FixupSetCC() { return new X86FixupSetCCPass(); }
70 bool X86FixupSetCCPass::isSetCCr(unsigned Opcode) {
71 switch (Opcode) {
72 default:
73 return false;
74 case X86::SETOr:
75 case X86::SETNOr:
76 case X86::SETBr:
77 case X86::SETAEr:
78 case X86::SETEr:
79 case X86::SETNEr:
80 case X86::SETBEr:
81 case X86::SETAr:
82 case X86::SETSr:
83 case X86::SETNSr:
84 case X86::SETPr:
85 case X86::SETNPr:
86 case X86::SETLr:
87 case X86::SETGEr:
88 case X86::SETLEr:
89 case X86::SETGr:
90 return true;
94 // We expect the instruction *immediately* before the setcc to imp-def
95 // EFLAGS (because of scheduling glue). To make this less brittle w.r.t
96 // scheduling, look backwards until we hit the beginning of the
97 // basic-block, or a small bound (to avoid quadratic behavior).
98 MachineInstr *
99 X86FixupSetCCPass::findFlagsImpDef(MachineBasicBlock *MBB,
100 MachineBasicBlock::reverse_iterator MI) {
101 // FIXME: Should this be instr_rend(), and MI be reverse_instr_iterator?
102 auto MBBStart = MBB->rend();
103 for (int i = 0; (i < SearchBound) && (MI != MBBStart); ++i, ++MI)
104 for (auto &Op : MI->implicit_operands())
105 if (Op.isReg() && (Op.getReg() == X86::EFLAGS) && Op.isDef())
106 return &*MI;
108 return nullptr;
111 bool X86FixupSetCCPass::impUsesFlags(MachineInstr *MI) {
112 for (auto &Op : MI->implicit_operands())
113 if (Op.isReg() && (Op.getReg() == X86::EFLAGS) && Op.isUse())
114 return true;
116 return false;
119 bool X86FixupSetCCPass::runOnMachineFunction(MachineFunction &MF) {
120 bool Changed = false;
121 MRI = &MF.getRegInfo();
122 TII = MF.getSubtarget<X86Subtarget>().getInstrInfo();
124 SmallVector<MachineInstr*, 4> ToErase;
126 for (auto &MBB : MF) {
127 for (auto &MI : MBB) {
128 // Find a setcc that is used by a zext.
129 // This doesn't have to be the only use, the transformation is safe
130 // regardless.
131 if (!isSetCCr(MI.getOpcode()))
132 continue;
134 MachineInstr *ZExt = nullptr;
135 for (auto &Use : MRI->use_instructions(MI.getOperand(0).getReg()))
136 if (Use.getOpcode() == X86::MOVZX32rr8)
137 ZExt = &Use;
139 if (!ZExt)
140 continue;
142 // Find the preceding instruction that imp-defs eflags.
143 MachineInstr *FlagsDefMI = findFlagsImpDef(
144 MI.getParent(), MachineBasicBlock::reverse_iterator(&MI));
145 if (!FlagsDefMI)
146 continue;
148 // We'd like to put something that clobbers eflags directly before
149 // FlagsDefMI. This can't hurt anything after FlagsDefMI, because
150 // it, itself, by definition, clobbers eflags. But it may happen that
151 // FlagsDefMI also *uses* eflags, in which case the transformation is
152 // invalid.
153 if (impUsesFlags(FlagsDefMI))
154 continue;
156 ++NumSubstZexts;
157 Changed = true;
159 // On 32-bit, we need to be careful to force an ABCD register.
160 const TargetRegisterClass *RC = MF.getSubtarget<X86Subtarget>().is64Bit()
161 ? &X86::GR32RegClass
162 : &X86::GR32_ABCDRegClass;
163 unsigned ZeroReg = MRI->createVirtualRegister(RC);
164 unsigned InsertReg = MRI->createVirtualRegister(RC);
166 // Initialize a register with 0. This must go before the eflags def
167 BuildMI(MBB, FlagsDefMI, MI.getDebugLoc(), TII->get(X86::MOV32r0),
168 ZeroReg);
170 // X86 setcc only takes an output GR8, so fake a GR32 input by inserting
171 // the setcc result into the low byte of the zeroed register.
172 BuildMI(*ZExt->getParent(), ZExt, ZExt->getDebugLoc(),
173 TII->get(X86::INSERT_SUBREG), InsertReg)
174 .addReg(ZeroReg)
175 .addReg(MI.getOperand(0).getReg())
176 .addImm(X86::sub_8bit);
177 MRI->replaceRegWith(ZExt->getOperand(0).getReg(), InsertReg);
178 ToErase.push_back(ZExt);
182 for (auto &I : ToErase)
183 I->eraseFromParent();
185 return Changed;