2017-08-02 07:14:32 +08:00
|
|
|
//===-- SIOptimizeExecMaskingPreRA.cpp ------------------------------------===//
|
|
|
|
//
|
2019-01-19 16:50:56 +08:00
|
|
|
// 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
|
2017-08-02 07:14:32 +08:00
|
|
|
//
|
|
|
|
//===----------------------------------------------------------------------===//
|
|
|
|
//
|
|
|
|
/// \file
|
2020-03-13 02:09:45 +08:00
|
|
|
/// This pass performs exec mask handling peephole optimizations which needs
|
|
|
|
/// to be done before register allocation to reduce register pressure.
|
2017-08-02 07:14:32 +08:00
|
|
|
///
|
|
|
|
//===----------------------------------------------------------------------===//
|
|
|
|
|
|
|
|
#include "AMDGPU.h"
|
|
|
|
#include "AMDGPUSubtarget.h"
|
AMDGPU: Remove #include "MCTargetDesc/AMDGPUMCTargetDesc.h" from common headers
Summary:
MCTargetDesc/AMDGPUMCTargetDesc.h contains enums for all the instuction
and register defintions, which are huge so we only want to include
them where needed.
This will also make it easier if we want to split the R600 and GCN
definitions into separate tablegenerated files.
I was unable to remove AMDGPUMCTargetDesc.h from SIMachineFunctionInfo.h
because it uses some enums from the header to initialize default values
for the SIMachineFunction class, so I ended up having to remove includes of
SIMachineFunctionInfo.h from headers too.
Reviewers: arsenm, nhaehnle
Reviewed By: nhaehnle
Subscribers: MatzeB, kzhuravl, wdng, yaxunl, dstuttard, tpr, t-tye, javed.absar, llvm-commits
Differential Revision: https://reviews.llvm.org/D46272
llvm-svn: 332930
2018-05-22 10:03:23 +08:00
|
|
|
#include "MCTargetDesc/AMDGPUMCTargetDesc.h"
|
Sink all InitializePasses.h includes
This file lists every pass in LLVM, and is included by Pass.h, which is
very popular. Every time we add, remove, or rename a pass in LLVM, it
caused lots of recompilation.
I found this fact by looking at this table, which is sorted by the
number of times a file was changed over the last 100,000 git commits
multiplied by the number of object files that depend on it in the
current checkout:
recompiles touches affected_files header
342380 95 3604 llvm/include/llvm/ADT/STLExtras.h
314730 234 1345 llvm/include/llvm/InitializePasses.h
307036 118 2602 llvm/include/llvm/ADT/APInt.h
213049 59 3611 llvm/include/llvm/Support/MathExtras.h
170422 47 3626 llvm/include/llvm/Support/Compiler.h
162225 45 3605 llvm/include/llvm/ADT/Optional.h
158319 63 2513 llvm/include/llvm/ADT/Triple.h
140322 39 3598 llvm/include/llvm/ADT/StringRef.h
137647 59 2333 llvm/include/llvm/Support/Error.h
131619 73 1803 llvm/include/llvm/Support/FileSystem.h
Before this change, touching InitializePasses.h would cause 1345 files
to recompile. After this change, touching it only causes 550 compiles in
an incremental rebuild.
Reviewers: bkramer, asbirlea, bollu, jdoerfert
Differential Revision: https://reviews.llvm.org/D70211
2019-11-14 05:15:01 +08:00
|
|
|
#include "SIInstrInfo.h"
|
2017-12-13 10:51:04 +08:00
|
|
|
#include "llvm/CodeGen/LiveIntervals.h"
|
2017-08-02 07:14:32 +08:00
|
|
|
#include "llvm/CodeGen/MachineFunctionPass.h"
|
Sink all InitializePasses.h includes
This file lists every pass in LLVM, and is included by Pass.h, which is
very popular. Every time we add, remove, or rename a pass in LLVM, it
caused lots of recompilation.
I found this fact by looking at this table, which is sorted by the
number of times a file was changed over the last 100,000 git commits
multiplied by the number of object files that depend on it in the
current checkout:
recompiles touches affected_files header
342380 95 3604 llvm/include/llvm/ADT/STLExtras.h
314730 234 1345 llvm/include/llvm/InitializePasses.h
307036 118 2602 llvm/include/llvm/ADT/APInt.h
213049 59 3611 llvm/include/llvm/Support/MathExtras.h
170422 47 3626 llvm/include/llvm/Support/Compiler.h
162225 45 3605 llvm/include/llvm/ADT/Optional.h
158319 63 2513 llvm/include/llvm/ADT/Triple.h
140322 39 3598 llvm/include/llvm/ADT/StringRef.h
137647 59 2333 llvm/include/llvm/Support/Error.h
131619 73 1803 llvm/include/llvm/Support/FileSystem.h
Before this change, touching InitializePasses.h would cause 1345 files
to recompile. After this change, touching it only causes 550 compiles in
an incremental rebuild.
Reviewers: bkramer, asbirlea, bollu, jdoerfert
Differential Revision: https://reviews.llvm.org/D70211
2019-11-14 05:15:01 +08:00
|
|
|
#include "llvm/InitializePasses.h"
|
2017-08-02 07:14:32 +08:00
|
|
|
|
|
|
|
using namespace llvm;
|
|
|
|
|
|
|
|
#define DEBUG_TYPE "si-optimize-exec-masking-pre-ra"
|
|
|
|
|
|
|
|
namespace {
|
|
|
|
|
|
|
|
class SIOptimizeExecMaskingPreRA : public MachineFunctionPass {
|
2019-03-28 22:01:39 +08:00
|
|
|
private:
|
|
|
|
const SIRegisterInfo *TRI;
|
|
|
|
const SIInstrInfo *TII;
|
|
|
|
MachineRegisterInfo *MRI;
|
|
|
|
|
2017-08-02 07:14:32 +08:00
|
|
|
public:
|
2019-03-28 22:01:39 +08:00
|
|
|
static char ID;
|
|
|
|
|
2017-08-02 07:14:32 +08:00
|
|
|
SIOptimizeExecMaskingPreRA() : MachineFunctionPass(ID) {
|
|
|
|
initializeSIOptimizeExecMaskingPreRAPass(*PassRegistry::getPassRegistry());
|
|
|
|
}
|
|
|
|
|
|
|
|
bool runOnMachineFunction(MachineFunction &MF) override;
|
|
|
|
|
|
|
|
StringRef getPassName() const override {
|
|
|
|
return "SI optimize exec mask operations pre-RA";
|
|
|
|
}
|
|
|
|
|
|
|
|
void getAnalysisUsage(AnalysisUsage &AU) const override {
|
|
|
|
AU.addRequired<LiveIntervals>();
|
|
|
|
AU.setPreservesAll();
|
|
|
|
MachineFunctionPass::getAnalysisUsage(AU);
|
|
|
|
}
|
|
|
|
};
|
|
|
|
|
|
|
|
} // End anonymous namespace.
|
|
|
|
|
|
|
|
INITIALIZE_PASS_BEGIN(SIOptimizeExecMaskingPreRA, DEBUG_TYPE,
|
|
|
|
"SI optimize exec mask operations pre-RA", false, false)
|
|
|
|
INITIALIZE_PASS_DEPENDENCY(LiveIntervals)
|
|
|
|
INITIALIZE_PASS_END(SIOptimizeExecMaskingPreRA, DEBUG_TYPE,
|
|
|
|
"SI optimize exec mask operations pre-RA", false, false)
|
|
|
|
|
|
|
|
char SIOptimizeExecMaskingPreRA::ID = 0;
|
|
|
|
|
|
|
|
char &llvm::SIOptimizeExecMaskingPreRAID = SIOptimizeExecMaskingPreRA::ID;
|
|
|
|
|
|
|
|
FunctionPass *llvm::createSIOptimizeExecMaskingPreRAPass() {
|
|
|
|
return new SIOptimizeExecMaskingPreRA();
|
|
|
|
}
|
|
|
|
|
2019-06-17 01:13:09 +08:00
|
|
|
static bool isFullExecCopy(const MachineInstr& MI, const GCNSubtarget& ST) {
|
|
|
|
unsigned Exec = ST.isWave32() ? AMDGPU::EXEC_LO : AMDGPU::EXEC;
|
|
|
|
|
2020-04-13 09:44:45 +08:00
|
|
|
if (MI.isFullCopy() && MI.getOperand(1).getReg() == Exec)
|
2019-04-22 22:54:39 +08:00
|
|
|
return true;
|
|
|
|
|
|
|
|
return false;
|
2017-08-02 07:14:32 +08:00
|
|
|
}
|
|
|
|
|
2018-12-13 11:17:40 +08:00
|
|
|
// Optimize sequence
|
|
|
|
// %sel = V_CNDMASK_B32_e64 0, 1, %cc
|
|
|
|
// %cmp = V_CMP_NE_U32 1, %1
|
|
|
|
// $vcc = S_AND_B64 $exec, %cmp
|
|
|
|
// S_CBRANCH_VCC[N]Z
|
|
|
|
// =>
|
|
|
|
// $vcc = S_ANDN2_B64 $exec, %cc
|
|
|
|
// S_CBRANCH_VCC[N]Z
|
|
|
|
//
|
|
|
|
// It is the negation pattern inserted by DAGCombiner::visitBRCOND() in the
|
|
|
|
// rebuildSetCC(). We start with S_CBRANCH to avoid exhaustive search, but
|
|
|
|
// only 3 first instructions are really needed. S_AND_B64 with exec is a
|
|
|
|
// required part of the pattern since V_CNDMASK_B32 writes zeroes for inactive
|
|
|
|
// lanes.
|
|
|
|
//
|
|
|
|
// Returns %cc register on success.
|
|
|
|
static unsigned optimizeVcndVcmpPair(MachineBasicBlock &MBB,
|
|
|
|
const GCNSubtarget &ST,
|
|
|
|
MachineRegisterInfo &MRI,
|
|
|
|
LiveIntervals *LIS) {
|
|
|
|
const SIRegisterInfo *TRI = ST.getRegisterInfo();
|
|
|
|
const SIInstrInfo *TII = ST.getInstrInfo();
|
2019-06-17 01:13:09 +08:00
|
|
|
bool Wave32 = ST.isWave32();
|
|
|
|
const unsigned AndOpc = Wave32 ? AMDGPU::S_AND_B32 : AMDGPU::S_AND_B64;
|
|
|
|
const unsigned Andn2Opc = Wave32 ? AMDGPU::S_ANDN2_B32 : AMDGPU::S_ANDN2_B64;
|
|
|
|
const unsigned CondReg = Wave32 ? AMDGPU::VCC_LO : AMDGPU::VCC;
|
|
|
|
const unsigned ExecReg = Wave32 ? AMDGPU::EXEC_LO : AMDGPU::EXEC;
|
2018-12-13 11:17:40 +08:00
|
|
|
|
|
|
|
auto I = llvm::find_if(MBB.terminators(), [](const MachineInstr &MI) {
|
|
|
|
unsigned Opc = MI.getOpcode();
|
|
|
|
return Opc == AMDGPU::S_CBRANCH_VCCZ ||
|
|
|
|
Opc == AMDGPU::S_CBRANCH_VCCNZ; });
|
|
|
|
if (I == MBB.terminators().end())
|
|
|
|
return AMDGPU::NoRegister;
|
|
|
|
|
|
|
|
auto *And = TRI->findReachingDef(CondReg, AMDGPU::NoSubRegister,
|
|
|
|
*I, MRI, LIS);
|
|
|
|
if (!And || And->getOpcode() != AndOpc ||
|
|
|
|
!And->getOperand(1).isReg() || !And->getOperand(2).isReg())
|
|
|
|
return AMDGPU::NoRegister;
|
|
|
|
|
|
|
|
MachineOperand *AndCC = &And->getOperand(1);
|
Apply llvm-prefer-register-over-unsigned from clang-tidy to LLVM
Summary:
This clang-tidy check is looking for unsigned integer variables whose initializer
starts with an implicit cast from llvm::Register and changes the type of the
variable to llvm::Register (dropping the llvm:: where possible).
Partial reverts in:
X86FrameLowering.cpp - Some functions return unsigned and arguably should be MCRegister
X86FixupLEAs.cpp - Some functions return unsigned and arguably should be MCRegister
X86FrameLowering.cpp - Some functions return unsigned and arguably should be MCRegister
HexagonBitSimplify.cpp - Function takes BitTracker::RegisterRef which appears to be unsigned&
MachineVerifier.cpp - Ambiguous operator==() given MCRegister and const Register
PPCFastISel.cpp - No Register::operator-=()
PeepholeOptimizer.cpp - TargetInstrInfo::optimizeLoadInstr() takes an unsigned&
MachineTraceMetrics.cpp - MachineTraceMetrics lacks a suitable constructor
Manual fixups in:
ARMFastISel.cpp - ARMEmitLoad() now takes a Register& instead of unsigned&
HexagonSplitDouble.cpp - Ternary operator was ambiguous between unsigned/Register
HexagonConstExtenders.cpp - Has a local class named Register, used llvm::Register instead of Register.
PPCFastISel.cpp - PPCEmitLoad() now takes a Register& instead of unsigned&
Depends on D65919
Reviewers: arsenm, bogner, craig.topper, RKSimon
Reviewed By: arsenm
Subscribers: RKSimon, craig.topper, lenary, aemerson, wuzish, jholewinski, MatzeB, qcolombet, dschuff, jyknight, dylanmckay, sdardis, nemanjai, jvesely, wdng, nhaehnle, sbc100, jgravelle-google, kristof.beyls, hiraditya, aheejin, kbarton, fedor.sergeev, javed.absar, asb, rbar, johnrusso, simoncook, apazos, sabuasal, niosHD, jrtc27, MaskRay, zzheng, edward-jones, atanasyan, rogfer01, MartinMosbeck, brucehoult, the_o, tpr, PkmX, jocewei, jsji, Petar.Avramovic, asbirlea, Jim, s.egerton, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D65962
llvm-svn: 369041
2019-08-16 03:22:08 +08:00
|
|
|
Register CmpReg = AndCC->getReg();
|
2018-12-13 11:17:40 +08:00
|
|
|
unsigned CmpSubReg = AndCC->getSubReg();
|
|
|
|
if (CmpReg == ExecReg) {
|
|
|
|
AndCC = &And->getOperand(2);
|
|
|
|
CmpReg = AndCC->getReg();
|
|
|
|
CmpSubReg = AndCC->getSubReg();
|
|
|
|
} else if (And->getOperand(2).getReg() != ExecReg) {
|
|
|
|
return AMDGPU::NoRegister;
|
|
|
|
}
|
|
|
|
|
|
|
|
auto *Cmp = TRI->findReachingDef(CmpReg, CmpSubReg, *And, MRI, LIS);
|
|
|
|
if (!Cmp || !(Cmp->getOpcode() == AMDGPU::V_CMP_NE_U32_e32 ||
|
|
|
|
Cmp->getOpcode() == AMDGPU::V_CMP_NE_U32_e64) ||
|
|
|
|
Cmp->getParent() != And->getParent())
|
|
|
|
return AMDGPU::NoRegister;
|
|
|
|
|
|
|
|
MachineOperand *Op1 = TII->getNamedOperand(*Cmp, AMDGPU::OpName::src0);
|
|
|
|
MachineOperand *Op2 = TII->getNamedOperand(*Cmp, AMDGPU::OpName::src1);
|
|
|
|
if (Op1->isImm() && Op2->isReg())
|
|
|
|
std::swap(Op1, Op2);
|
|
|
|
if (!Op1->isReg() || !Op2->isImm() || Op2->getImm() != 1)
|
|
|
|
return AMDGPU::NoRegister;
|
|
|
|
|
Apply llvm-prefer-register-over-unsigned from clang-tidy to LLVM
Summary:
This clang-tidy check is looking for unsigned integer variables whose initializer
starts with an implicit cast from llvm::Register and changes the type of the
variable to llvm::Register (dropping the llvm:: where possible).
Partial reverts in:
X86FrameLowering.cpp - Some functions return unsigned and arguably should be MCRegister
X86FixupLEAs.cpp - Some functions return unsigned and arguably should be MCRegister
X86FrameLowering.cpp - Some functions return unsigned and arguably should be MCRegister
HexagonBitSimplify.cpp - Function takes BitTracker::RegisterRef which appears to be unsigned&
MachineVerifier.cpp - Ambiguous operator==() given MCRegister and const Register
PPCFastISel.cpp - No Register::operator-=()
PeepholeOptimizer.cpp - TargetInstrInfo::optimizeLoadInstr() takes an unsigned&
MachineTraceMetrics.cpp - MachineTraceMetrics lacks a suitable constructor
Manual fixups in:
ARMFastISel.cpp - ARMEmitLoad() now takes a Register& instead of unsigned&
HexagonSplitDouble.cpp - Ternary operator was ambiguous between unsigned/Register
HexagonConstExtenders.cpp - Has a local class named Register, used llvm::Register instead of Register.
PPCFastISel.cpp - PPCEmitLoad() now takes a Register& instead of unsigned&
Depends on D65919
Reviewers: arsenm, bogner, craig.topper, RKSimon
Reviewed By: arsenm
Subscribers: RKSimon, craig.topper, lenary, aemerson, wuzish, jholewinski, MatzeB, qcolombet, dschuff, jyknight, dylanmckay, sdardis, nemanjai, jvesely, wdng, nhaehnle, sbc100, jgravelle-google, kristof.beyls, hiraditya, aheejin, kbarton, fedor.sergeev, javed.absar, asb, rbar, johnrusso, simoncook, apazos, sabuasal, niosHD, jrtc27, MaskRay, zzheng, edward-jones, atanasyan, rogfer01, MartinMosbeck, brucehoult, the_o, tpr, PkmX, jocewei, jsji, Petar.Avramovic, asbirlea, Jim, s.egerton, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D65962
llvm-svn: 369041
2019-08-16 03:22:08 +08:00
|
|
|
Register SelReg = Op1->getReg();
|
2018-12-13 11:17:40 +08:00
|
|
|
auto *Sel = TRI->findReachingDef(SelReg, Op1->getSubReg(), *Cmp, MRI, LIS);
|
|
|
|
if (!Sel || Sel->getOpcode() != AMDGPU::V_CNDMASK_B32_e64)
|
|
|
|
return AMDGPU::NoRegister;
|
|
|
|
|
2019-03-19 03:25:39 +08:00
|
|
|
if (TII->hasModifiersSet(*Sel, AMDGPU::OpName::src0_modifiers) ||
|
2019-04-15 18:36:24 +08:00
|
|
|
TII->hasModifiersSet(*Sel, AMDGPU::OpName::src1_modifiers))
|
2019-03-19 03:25:39 +08:00
|
|
|
return AMDGPU::NoRegister;
|
|
|
|
|
2018-12-13 11:17:40 +08:00
|
|
|
Op1 = TII->getNamedOperand(*Sel, AMDGPU::OpName::src0);
|
|
|
|
Op2 = TII->getNamedOperand(*Sel, AMDGPU::OpName::src1);
|
|
|
|
MachineOperand *CC = TII->getNamedOperand(*Sel, AMDGPU::OpName::src2);
|
|
|
|
if (!Op1->isImm() || !Op2->isImm() || !CC->isReg() ||
|
|
|
|
Op1->getImm() != 0 || Op2->getImm() != 1)
|
|
|
|
return AMDGPU::NoRegister;
|
|
|
|
|
2019-10-08 20:46:32 +08:00
|
|
|
LLVM_DEBUG(dbgs() << "Folding sequence:\n\t" << *Sel << '\t' << *Cmp << '\t'
|
|
|
|
<< *And);
|
2018-12-13 11:17:40 +08:00
|
|
|
|
Apply llvm-prefer-register-over-unsigned from clang-tidy to LLVM
Summary:
This clang-tidy check is looking for unsigned integer variables whose initializer
starts with an implicit cast from llvm::Register and changes the type of the
variable to llvm::Register (dropping the llvm:: where possible).
Partial reverts in:
X86FrameLowering.cpp - Some functions return unsigned and arguably should be MCRegister
X86FixupLEAs.cpp - Some functions return unsigned and arguably should be MCRegister
X86FrameLowering.cpp - Some functions return unsigned and arguably should be MCRegister
HexagonBitSimplify.cpp - Function takes BitTracker::RegisterRef which appears to be unsigned&
MachineVerifier.cpp - Ambiguous operator==() given MCRegister and const Register
PPCFastISel.cpp - No Register::operator-=()
PeepholeOptimizer.cpp - TargetInstrInfo::optimizeLoadInstr() takes an unsigned&
MachineTraceMetrics.cpp - MachineTraceMetrics lacks a suitable constructor
Manual fixups in:
ARMFastISel.cpp - ARMEmitLoad() now takes a Register& instead of unsigned&
HexagonSplitDouble.cpp - Ternary operator was ambiguous between unsigned/Register
HexagonConstExtenders.cpp - Has a local class named Register, used llvm::Register instead of Register.
PPCFastISel.cpp - PPCEmitLoad() now takes a Register& instead of unsigned&
Depends on D65919
Reviewers: arsenm, bogner, craig.topper, RKSimon
Reviewed By: arsenm
Subscribers: RKSimon, craig.topper, lenary, aemerson, wuzish, jholewinski, MatzeB, qcolombet, dschuff, jyknight, dylanmckay, sdardis, nemanjai, jvesely, wdng, nhaehnle, sbc100, jgravelle-google, kristof.beyls, hiraditya, aheejin, kbarton, fedor.sergeev, javed.absar, asb, rbar, johnrusso, simoncook, apazos, sabuasal, niosHD, jrtc27, MaskRay, zzheng, edward-jones, atanasyan, rogfer01, MartinMosbeck, brucehoult, the_o, tpr, PkmX, jocewei, jsji, Petar.Avramovic, asbirlea, Jim, s.egerton, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D65962
llvm-svn: 369041
2019-08-16 03:22:08 +08:00
|
|
|
Register CCReg = CC->getReg();
|
2018-12-13 11:17:40 +08:00
|
|
|
LIS->RemoveMachineInstrFromMaps(*And);
|
2019-10-08 20:46:32 +08:00
|
|
|
MachineInstr *Andn2 =
|
|
|
|
BuildMI(MBB, *And, And->getDebugLoc(), TII->get(Andn2Opc),
|
|
|
|
And->getOperand(0).getReg())
|
|
|
|
.addReg(ExecReg)
|
|
|
|
.addReg(CCReg, getUndefRegState(CC->isUndef()), CC->getSubReg());
|
2018-12-13 11:17:40 +08:00
|
|
|
And->eraseFromParent();
|
|
|
|
LIS->InsertMachineInstrInMaps(*Andn2);
|
|
|
|
|
|
|
|
LLVM_DEBUG(dbgs() << "=>\n\t" << *Andn2 << '\n');
|
|
|
|
|
|
|
|
// Try to remove compare. Cmp value should not used in between of cmp
|
|
|
|
// and s_and_b64 if VCC or just unused if any other register.
|
2019-08-02 07:27:28 +08:00
|
|
|
if ((Register::isVirtualRegister(CmpReg) && MRI.use_nodbg_empty(CmpReg)) ||
|
2018-12-13 11:17:40 +08:00
|
|
|
(CmpReg == CondReg &&
|
|
|
|
std::none_of(std::next(Cmp->getIterator()), Andn2->getIterator(),
|
2018-12-13 13:52:11 +08:00
|
|
|
[&](const MachineInstr &MI) {
|
2019-08-02 07:27:28 +08:00
|
|
|
return MI.readsRegister(CondReg, TRI);
|
|
|
|
}))) {
|
2018-12-13 11:17:40 +08:00
|
|
|
LLVM_DEBUG(dbgs() << "Erasing: " << *Cmp << '\n');
|
|
|
|
|
|
|
|
LIS->RemoveMachineInstrFromMaps(*Cmp);
|
|
|
|
Cmp->eraseFromParent();
|
|
|
|
|
|
|
|
// Try to remove v_cndmask_b32.
|
2019-08-02 07:27:28 +08:00
|
|
|
if (Register::isVirtualRegister(SelReg) && MRI.use_nodbg_empty(SelReg)) {
|
2018-12-13 11:17:40 +08:00
|
|
|
LLVM_DEBUG(dbgs() << "Erasing: " << *Sel << '\n');
|
|
|
|
|
|
|
|
LIS->RemoveMachineInstrFromMaps(*Sel);
|
|
|
|
Sel->eraseFromParent();
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
return CCReg;
|
|
|
|
}
|
|
|
|
|
2017-08-02 07:14:32 +08:00
|
|
|
bool SIOptimizeExecMaskingPreRA::runOnMachineFunction(MachineFunction &MF) {
|
2017-12-16 06:22:58 +08:00
|
|
|
if (skipFunction(MF.getFunction()))
|
2017-08-02 07:14:32 +08:00
|
|
|
return false;
|
|
|
|
|
2018-07-12 04:59:01 +08:00
|
|
|
const GCNSubtarget &ST = MF.getSubtarget<GCNSubtarget>();
|
2019-03-28 22:01:39 +08:00
|
|
|
TRI = ST.getRegisterInfo();
|
|
|
|
TII = ST.getInstrInfo();
|
|
|
|
MRI = &MF.getRegInfo();
|
|
|
|
|
2017-08-02 07:14:32 +08:00
|
|
|
MachineRegisterInfo &MRI = MF.getRegInfo();
|
|
|
|
LiveIntervals *LIS = &getAnalysis<LiveIntervals>();
|
2017-08-16 12:43:49 +08:00
|
|
|
DenseSet<unsigned> RecalcRegs({AMDGPU::EXEC_LO, AMDGPU::EXEC_HI});
|
2019-06-17 01:13:09 +08:00
|
|
|
unsigned Exec = ST.isWave32() ? AMDGPU::EXEC_LO : AMDGPU::EXEC;
|
2017-08-02 07:14:32 +08:00
|
|
|
bool Changed = false;
|
|
|
|
|
|
|
|
for (MachineBasicBlock &MBB : MF) {
|
2017-08-16 12:43:49 +08:00
|
|
|
|
2018-12-13 11:17:40 +08:00
|
|
|
if (unsigned Reg = optimizeVcndVcmpPair(MBB, ST, MRI, LIS)) {
|
|
|
|
RecalcRegs.insert(Reg);
|
|
|
|
RecalcRegs.insert(AMDGPU::VCC_LO);
|
|
|
|
RecalcRegs.insert(AMDGPU::VCC_HI);
|
|
|
|
RecalcRegs.insert(AMDGPU::SCC);
|
|
|
|
Changed = true;
|
|
|
|
}
|
|
|
|
|
2017-08-16 12:43:49 +08:00
|
|
|
// Try to remove unneeded instructions before s_endpgm.
|
|
|
|
if (MBB.succ_empty()) {
|
2018-08-29 02:55:55 +08:00
|
|
|
if (MBB.empty())
|
|
|
|
continue;
|
|
|
|
|
|
|
|
// Skip this if the endpgm has any implicit uses, otherwise we would need
|
|
|
|
// to be careful to update / remove them.
|
[AMDGPU] Add support for immediate operand for S_ENDPGM
Summary:
Add support for immediate operand in S_ENDPGM
Change-Id: I0c56a076a10980f719fb2a8f16407e9c301013f6
Reviewers: alexshap
Subscribers: qcolombet, arsenm, kzhuravl, jvesely, wdng, nhaehnle, yaxunl, tpr, t-tye, eraman, arphaman, Petar.Avramovic, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D59213
llvm-svn: 355902
2019-03-12 17:52:58 +08:00
|
|
|
// S_ENDPGM always has a single imm operand that is not used other than to
|
|
|
|
// end up in the encoding
|
2018-08-29 02:55:55 +08:00
|
|
|
MachineInstr &Term = MBB.back();
|
[AMDGPU] Add support for immediate operand for S_ENDPGM
Summary:
Add support for immediate operand in S_ENDPGM
Change-Id: I0c56a076a10980f719fb2a8f16407e9c301013f6
Reviewers: alexshap
Subscribers: qcolombet, arsenm, kzhuravl, jvesely, wdng, nhaehnle, yaxunl, tpr, t-tye, eraman, arphaman, Petar.Avramovic, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D59213
llvm-svn: 355902
2019-03-12 17:52:58 +08:00
|
|
|
if (Term.getOpcode() != AMDGPU::S_ENDPGM || Term.getNumOperands() != 1)
|
2017-08-16 12:43:49 +08:00
|
|
|
continue;
|
|
|
|
|
|
|
|
SmallVector<MachineBasicBlock*, 4> Blocks({&MBB});
|
|
|
|
|
|
|
|
while (!Blocks.empty()) {
|
|
|
|
auto CurBB = Blocks.pop_back_val();
|
|
|
|
auto I = CurBB->rbegin(), E = CurBB->rend();
|
|
|
|
if (I != E) {
|
|
|
|
if (I->isUnconditionalBranch() || I->getOpcode() == AMDGPU::S_ENDPGM)
|
|
|
|
++I;
|
|
|
|
else if (I->isBranch())
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
|
|
|
while (I != E) {
|
2018-05-09 10:42:00 +08:00
|
|
|
if (I->isDebugInstr()) {
|
2017-12-06 02:23:17 +08:00
|
|
|
I = std::next(I);
|
2017-08-16 12:43:49 +08:00
|
|
|
continue;
|
2017-12-06 02:23:17 +08:00
|
|
|
}
|
|
|
|
|
2017-08-16 12:43:49 +08:00
|
|
|
if (I->mayStore() || I->isBarrier() || I->isCall() ||
|
|
|
|
I->hasUnmodeledSideEffects() || I->hasOrderedMemoryRef())
|
|
|
|
break;
|
|
|
|
|
2018-05-14 20:53:11 +08:00
|
|
|
LLVM_DEBUG(dbgs()
|
|
|
|
<< "Removing no effect instruction: " << *I << '\n');
|
2017-08-16 12:43:49 +08:00
|
|
|
|
2017-09-09 02:51:26 +08:00
|
|
|
for (auto &Op : I->operands()) {
|
2017-08-16 12:43:49 +08:00
|
|
|
if (Op.isReg())
|
|
|
|
RecalcRegs.insert(Op.getReg());
|
2017-09-09 02:51:26 +08:00
|
|
|
}
|
2017-08-16 12:43:49 +08:00
|
|
|
|
|
|
|
auto Next = std::next(I);
|
|
|
|
LIS->RemoveMachineInstrFromMaps(*I);
|
|
|
|
I->eraseFromParent();
|
|
|
|
I = Next;
|
|
|
|
|
|
|
|
Changed = true;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (I != E)
|
|
|
|
continue;
|
|
|
|
|
|
|
|
// Try to ascend predecessors.
|
|
|
|
for (auto *Pred : CurBB->predecessors()) {
|
|
|
|
if (Pred->succ_size() == 1)
|
|
|
|
Blocks.push_back(Pred);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
2020-03-13 02:09:45 +08:00
|
|
|
// If the only user of a logical operation is move to exec, fold it now
|
|
|
|
// to prevent forming of saveexec. I.e:
|
|
|
|
//
|
|
|
|
// %0:sreg_64 = COPY $exec
|
|
|
|
// %1:sreg_64 = S_AND_B64 %0:sreg_64, %2:sreg_64
|
|
|
|
// =>
|
|
|
|
// %1 = S_AND_B64 $exec, %2:sreg_64
|
|
|
|
unsigned ScanThreshold = 10;
|
|
|
|
for (auto I = MBB.rbegin(), E = MBB.rend(); I != E
|
|
|
|
&& ScanThreshold--; ++I) {
|
|
|
|
if (!isFullExecCopy(*I, ST))
|
|
|
|
continue;
|
2017-08-02 07:44:35 +08:00
|
|
|
|
2020-03-13 02:09:45 +08:00
|
|
|
Register SavedExec = I->getOperand(0).getReg();
|
|
|
|
if (SavedExec.isVirtual() && MRI.hasOneNonDBGUse(SavedExec) &&
|
|
|
|
MRI.use_instr_nodbg_begin(SavedExec)->getParent() == I->getParent()) {
|
|
|
|
LLVM_DEBUG(dbgs() << "Redundant EXEC COPY: " << *I << '\n');
|
|
|
|
LIS->RemoveMachineInstrFromMaps(*I);
|
|
|
|
I->eraseFromParent();
|
|
|
|
MRI.replaceRegWith(SavedExec, Exec);
|
|
|
|
LIS->removeInterval(SavedExec);
|
|
|
|
Changed = true;
|
2017-08-02 07:44:35 +08:00
|
|
|
}
|
2020-03-13 02:09:45 +08:00
|
|
|
break;
|
2017-08-02 07:44:35 +08:00
|
|
|
}
|
2017-08-02 07:14:32 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
if (Changed) {
|
2017-08-16 12:43:49 +08:00
|
|
|
for (auto Reg : RecalcRegs) {
|
2019-08-02 07:27:28 +08:00
|
|
|
if (Register::isVirtualRegister(Reg)) {
|
2017-08-16 12:43:49 +08:00
|
|
|
LIS->removeInterval(Reg);
|
|
|
|
if (!MRI.reg_empty(Reg))
|
|
|
|
LIS->createAndComputeVirtRegInterval(Reg);
|
|
|
|
} else {
|
2019-02-23 03:03:36 +08:00
|
|
|
LIS->removeAllRegUnitsForPhysReg(Reg);
|
2017-08-16 12:43:49 +08:00
|
|
|
}
|
|
|
|
}
|
2017-08-02 07:14:32 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
return Changed;
|
|
|
|
}
|