[Mesa-dev] [PATCH 5/6] R600: Remove LowerConstCopyPass and lower CONST_COPY right after ISel.
Tom Stellard
tom at stellard.net
Tue Feb 19 07:29:19 PST 2013
On Mon, Feb 18, 2013 at 05:27:29PM +0100, Vincent Lejeune wrote:
> Maintaining CONST_COPY Instructions until Pre Emit may prevent some ifcvt case
> and taking them in account for scheduling is difficult for no real benefit.
> ---
> lib/Target/R600/AMDGPU.h | 1 -
> lib/Target/R600/AMDGPUTargetMachine.cpp | 1 -
> lib/Target/R600/R600ISelLowering.cpp | 8 +-
> lib/Target/R600/R600Instructions.td | 7 +-
> lib/Target/R600/R600LowerConstCopy.cpp | 222 --------------------------------
Don't forget to remove this file from CMakeLists.txt
> 5 files changed, 11 insertions(+), 228 deletions(-)
> delete mode 100644 lib/Target/R600/R600LowerConstCopy.cpp
>
> diff --git a/lib/Target/R600/AMDGPU.h b/lib/Target/R600/AMDGPU.h
> index ba87918..67073ab 100644
> --- a/lib/Target/R600/AMDGPU.h
> +++ b/lib/Target/R600/AMDGPU.h
> @@ -23,7 +23,6 @@ class AMDGPUTargetMachine;
> // R600 Passes
> FunctionPass* createR600KernelParametersPass(const DataLayout *TD);
> FunctionPass *createR600ExpandSpecialInstrsPass(TargetMachine &tm);
> -FunctionPass *createR600LowerConstCopy(TargetMachine &tm);
>
> // SI Passes
> FunctionPass *createSIAnnotateControlFlowPass();
> diff --git a/lib/Target/R600/AMDGPUTargetMachine.cpp b/lib/Target/R600/AMDGPUTargetMachine.cpp
> index e2f00be..70b34b0 100644
> --- a/lib/Target/R600/AMDGPUTargetMachine.cpp
> +++ b/lib/Target/R600/AMDGPUTargetMachine.cpp
> @@ -143,7 +143,6 @@ bool AMDGPUPassConfig::addPreEmitPass() {
> addPass(createAMDGPUCFGStructurizerPass(*TM));
> addPass(createR600ExpandSpecialInstrsPass(*TM));
> addPass(&FinalizeMachineBundlesID);
> - addPass(createR600LowerConstCopy(*TM));
> } else {
> addPass(createSILowerControlFlowPass(*TM));
> }
> diff --git a/lib/Target/R600/R600ISelLowering.cpp b/lib/Target/R600/R600ISelLowering.cpp
> index ece0b9a..f25ced1 100644
> --- a/lib/Target/R600/R600ISelLowering.cpp
> +++ b/lib/Target/R600/R600ISelLowering.cpp
> @@ -150,7 +150,13 @@ MachineBasicBlock * R600TargetLowering::EmitInstrWithCustomInserter(
> TII->buildMovImm(*BB, I, MI->getOperand(0).getReg(),
> MI->getOperand(1).getImm());
> break;
> -
> + case AMDGPU::CONST_COPY: {
> + MachineInstr *NewMI = TII->buildDefaultInstruction(*BB, MI, AMDGPU::MOV,
> + MI->getOperand(0).getReg(), AMDGPU::ALU_CONST);
> + TII->setImmOperand(NewMI, R600Operands::SRC0_SEL,
> + MI->getOperand(1).getImm());
> + break;
> + }
>
> case AMDGPU::RAT_WRITE_CACHELESS_32_eg:
> case AMDGPU::RAT_WRITE_CACHELESS_128_eg: {
> diff --git a/lib/Target/R600/R600Instructions.td b/lib/Target/R600/R600Instructions.td
> index 74106c9..10bcdcf 100644
> --- a/lib/Target/R600/R600Instructions.td
> +++ b/lib/Target/R600/R600Instructions.td
> @@ -1650,17 +1650,18 @@ let isTerminator = 1, isReturn = 1, isBarrier = 1, hasCtrlDep = 1,
> // Constant Buffer Addressing Support
> //===----------------------------------------------------------------------===//
>
> -let isCodeGenOnly = 1, isPseudo = 1, Namespace = "AMDGPU" in {
> +let usesCustomInserter = 1, isCodeGenOnly = 1, isPseudo = 1, Namespace = "AMDGPU" in {
> def CONST_COPY : Instruction {
> let OutOperandList = (outs R600_Reg32:$dst);
> let InOperandList = (ins i32imm:$src);
> - let Pattern = [(set R600_Reg32:$dst, (CONST_ADDRESS ADDRGA_CONST_OFFSET:$src))];
> + let Pattern =
> + [(set R600_Reg32:$dst, (CONST_ADDRESS ADDRGA_CONST_OFFSET:$src))];
> let AsmString = "CONST_COPY";
> let neverHasSideEffects = 1;
> let isAsCheapAsAMove = 1;
> let Itinerary = NullALU;
> }
> -} // end isCodeGenOnly = 1, isPseudo = 1, Namespace = "AMDGPU"
> +} // end usesCustomInserter = 1, isCodeGenOnly = 1, isPseudo = 1, Namespace = "AMDGPU"
>
> def TEX_VTX_CONSTBUF :
> InstR600ISA <(outs R600_Reg128:$dst), (ins MEMxi:$ptr, i32imm:$BUFFER_ID), "VTX_READ_eg $dst, $ptr",
> diff --git a/lib/Target/R600/R600LowerConstCopy.cpp b/lib/Target/R600/R600LowerConstCopy.cpp
> deleted file mode 100644
> index 3ebe653..0000000
> --- a/lib/Target/R600/R600LowerConstCopy.cpp
> +++ /dev/null
> @@ -1,222 +0,0 @@
> -//===-- R600LowerConstCopy.cpp - Propagate ConstCopy / lower them to MOV---===//
> -//
> -// The LLVM Compiler Infrastructure
> -//
> -// This file is distributed under the University of Illinois Open Source
> -// License. See LICENSE.TXT for details.
> -//
> -//===----------------------------------------------------------------------===//
> -//
> -/// \file
> -/// This pass is intended to handle remaining ConstCopy pseudo MachineInstr.
> -/// ISel will fold each Const Buffer read inside scalar ALU. However it cannot
> -/// fold them inside vector instruction, like DOT4 or Cube ; ISel emits
> -/// ConstCopy instead. This pass (executed after ExpandingSpecialInstr) will try
> -/// to fold them if possible or replace them by MOV otherwise.
> -//
> -//===----------------------------------------------------------------------===//
> -
> -#include "AMDGPU.h"
> -#include "R600InstrInfo.h"
> -#include "llvm/CodeGen/MachineFunction.h"
> -#include "llvm/CodeGen/MachineFunctionPass.h"
> -#include "llvm/CodeGen/MachineInstrBuilder.h"
> -#include "llvm/IR/GlobalValue.h"
> -
> -namespace llvm {
> -
> -class R600LowerConstCopy : public MachineFunctionPass {
> -private:
> - static char ID;
> - const R600InstrInfo *TII;
> -
> - struct ConstPairs {
> - unsigned XYPair;
> - unsigned ZWPair;
> - };
> -
> - bool canFoldInBundle(ConstPairs &UsedConst, unsigned ReadConst) const;
> -public:
> - R600LowerConstCopy(TargetMachine &tm);
> - virtual bool runOnMachineFunction(MachineFunction &MF);
> -
> - const char *getPassName() const { return "R600 Eliminate Symbolic Operand"; }
> -};
> -
> -char R600LowerConstCopy::ID = 0;
> -
> -R600LowerConstCopy::R600LowerConstCopy(TargetMachine &tm) :
> - MachineFunctionPass(ID),
> - TII (static_cast<const R600InstrInfo *>(tm.getInstrInfo()))
> -{
> -}
> -
> -bool R600LowerConstCopy::canFoldInBundle(ConstPairs &UsedConst,
> - unsigned ReadConst) const {
> - unsigned ReadConstChan = ReadConst & 3;
> - unsigned ReadConstIndex = ReadConst & (~3);
> - if (ReadConstChan < 2) {
> - if (!UsedConst.XYPair) {
> - UsedConst.XYPair = ReadConstIndex;
> - }
> - return UsedConst.XYPair == ReadConstIndex;
> - } else {
> - if (!UsedConst.ZWPair) {
> - UsedConst.ZWPair = ReadConstIndex;
> - }
> - return UsedConst.ZWPair == ReadConstIndex;
> - }
> -}
> -
> -static bool isControlFlow(const MachineInstr &MI) {
> - return (MI.getOpcode() == AMDGPU::IF_PREDICATE_SET) ||
> - (MI.getOpcode() == AMDGPU::ENDIF) ||
> - (MI.getOpcode() == AMDGPU::ELSE) ||
> - (MI.getOpcode() == AMDGPU::WHILELOOP) ||
> - (MI.getOpcode() == AMDGPU::BREAK);
> -}
> -
> -bool R600LowerConstCopy::runOnMachineFunction(MachineFunction &MF) {
> -
> - for (MachineFunction::iterator BB = MF.begin(), BB_E = MF.end();
> - BB != BB_E; ++BB) {
> - MachineBasicBlock &MBB = *BB;
> - DenseMap<unsigned, MachineInstr *> RegToConstIndex;
> - for (MachineBasicBlock::instr_iterator I = MBB.instr_begin(),
> - E = MBB.instr_end(); I != E;) {
> -
> - if (I->getOpcode() == AMDGPU::CONST_COPY) {
> - MachineInstr &MI = *I;
> - I = llvm::next(I);
> - unsigned DstReg = MI.getOperand(0).getReg();
> - DenseMap<unsigned, MachineInstr *>::iterator SrcMI =
> - RegToConstIndex.find(DstReg);
> - if (SrcMI != RegToConstIndex.end()) {
> - SrcMI->second->eraseFromParent();
> - RegToConstIndex.erase(SrcMI);
> - }
> - MachineInstr *NewMI =
> - TII->buildDefaultInstruction(MBB, &MI, AMDGPU::MOV,
> - MI.getOperand(0).getReg(), AMDGPU::ALU_CONST);
> - TII->setImmOperand(NewMI, R600Operands::SRC0_SEL,
> - MI.getOperand(1).getImm());
> - RegToConstIndex[DstReg] = NewMI;
> - MI.eraseFromParent();
> - continue;
> - }
> -
> - std::vector<unsigned> Defs;
> - // We consider all Instructions as bundled because algorithm that handle
> - // const read port limitations inside an IG is still valid with single
> - // instructions.
> - std::vector<MachineInstr *> Bundle;
> -
> - if (I->isBundle()) {
> - unsigned BundleSize = I->getBundleSize();
> - for (unsigned i = 0; i < BundleSize; i++) {
> - I = llvm::next(I);
> - Bundle.push_back(I);
> - }
> - } else if (TII->isALUInstr(I->getOpcode())){
> - Bundle.push_back(I);
> - } else if (isControlFlow(*I)) {
> - RegToConstIndex.clear();
> - I = llvm::next(I);
> - continue;
> - } else {
> - MachineInstr &MI = *I;
> - for (MachineInstr::mop_iterator MOp = MI.operands_begin(),
> - MOpE = MI.operands_end(); MOp != MOpE; ++MOp) {
> - MachineOperand &MO = *MOp;
> - if (!MO.isReg())
> - continue;
> - if (MO.isDef()) {
> - Defs.push_back(MO.getReg());
> - } else {
> - // Either a TEX or an Export inst, prevent from erasing def of used
> - // operand
> - RegToConstIndex.erase(MO.getReg());
> - for (MCSubRegIterator SR(MO.getReg(), &TII->getRegisterInfo());
> - SR.isValid(); ++SR) {
> - RegToConstIndex.erase(*SR);
> - }
> - }
> - }
> - }
> -
> -
> - R600Operands::Ops OpTable[3][2] = {
> - {R600Operands::SRC0, R600Operands::SRC0_SEL},
> - {R600Operands::SRC1, R600Operands::SRC1_SEL},
> - {R600Operands::SRC2, R600Operands::SRC2_SEL},
> - };
> -
> - for(std::vector<MachineInstr *>::iterator It = Bundle.begin(),
> - ItE = Bundle.end(); It != ItE; ++It) {
> - MachineInstr *MI = *It;
> - if (TII->isPredicated(MI)) {
> - // We don't want to erase previous assignment
> - RegToConstIndex.erase(MI->getOperand(0).getReg());
> - } else {
> - int WriteIDX = TII->getOperandIdx(MI->getOpcode(), R600Operands::WRITE);
> - if (WriteIDX < 0 || MI->getOperand(WriteIDX).getImm())
> - Defs.push_back(MI->getOperand(0).getReg());
> - }
> - }
> -
> - ConstPairs CP = {0,0};
> - for (unsigned SrcOp = 0; SrcOp < 3; SrcOp++) {
> - for(std::vector<MachineInstr *>::iterator It = Bundle.begin(),
> - ItE = Bundle.end(); It != ItE; ++It) {
> - MachineInstr *MI = *It;
> - int SrcIdx = TII->getOperandIdx(MI->getOpcode(), OpTable[SrcOp][0]);
> - if (SrcIdx < 0)
> - continue;
> - MachineOperand &MO = MI->getOperand(SrcIdx);
> - DenseMap<unsigned, MachineInstr *>::iterator SrcMI =
> - RegToConstIndex.find(MO.getReg());
> - if (SrcMI != RegToConstIndex.end()) {
> - MachineInstr *CstMov = SrcMI->second;
> - int ConstMovSel =
> - TII->getOperandIdx(CstMov->getOpcode(), R600Operands::SRC0_SEL);
> - unsigned ConstIndex = CstMov->getOperand(ConstMovSel).getImm();
> - if (MI->isInsideBundle() && canFoldInBundle(CP, ConstIndex)) {
> - TII->setImmOperand(MI, OpTable[SrcOp][1], ConstIndex);
> - MI->getOperand(SrcIdx).setReg(AMDGPU::ALU_CONST);
> - } else {
> - RegToConstIndex.erase(SrcMI);
> - }
> - }
> - }
> - }
> -
> - for (std::vector<unsigned>::iterator It = Defs.begin(), ItE = Defs.end();
> - It != ItE; ++It) {
> - DenseMap<unsigned, MachineInstr *>::iterator SrcMI =
> - RegToConstIndex.find(*It);
> - if (SrcMI != RegToConstIndex.end()) {
> - SrcMI->second->eraseFromParent();
> - RegToConstIndex.erase(SrcMI);
> - }
> - }
> - I = llvm::next(I);
> - }
> -
> - if (MBB.succ_empty()) {
> - for (DenseMap<unsigned, MachineInstr *>::iterator
> - DI = RegToConstIndex.begin(), DE = RegToConstIndex.end();
> - DI != DE; ++DI) {
> - DI->second->eraseFromParent();
> - }
> - }
> - }
> - return false;
> -}
> -
> -FunctionPass *createR600LowerConstCopy(TargetMachine &tm) {
> - return new R600LowerConstCopy(tm);
> -}
> -
> -}
> -
> -
> --
> 1.8.1.2
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
More information about the mesa-dev
mailing list