From d96682169eee693000846cddab6928e141ac274e Mon Sep 17 00:00:00 2001 From: Tom Stellard Date: Tue, 24 Apr 2012 21:57:14 -0400 Subject: [PATCH] radeon/llvm: Remove AMDILLiteralManager.cpp --- src/gallium/drivers/radeon/AMDIL.h | 2 - .../drivers/radeon/AMDILLiteralManager.cpp | 125 ------------------ .../drivers/radeon/AMDILTargetMachine.cpp | 1 - src/gallium/drivers/radeon/Makefile.sources | 1 - 4 files changed, 129 deletions(-) delete mode 100644 src/gallium/drivers/radeon/AMDILLiteralManager.cpp diff --git a/src/gallium/drivers/radeon/AMDIL.h b/src/gallium/drivers/radeon/AMDIL.h index 536fd69da31..317ea124f66 100644 --- a/src/gallium/drivers/radeon/AMDIL.h +++ b/src/gallium/drivers/radeon/AMDIL.h @@ -104,8 +104,6 @@ FunctionPass* createAMDILCFGPreparationPass(TargetMachine &TM AMDIL_OPT_LEVEL_DECL); FunctionPass* createAMDILCFGStructurizerPass(TargetMachine &TM AMDIL_OPT_LEVEL_DECL); -FunctionPass* - createAMDILLiteralManager(TargetMachine &TM AMDIL_OPT_LEVEL_DECL); extern Target TheAMDILTarget; extern Target TheAMDGPUTarget; diff --git a/src/gallium/drivers/radeon/AMDILLiteralManager.cpp b/src/gallium/drivers/radeon/AMDILLiteralManager.cpp deleted file mode 100644 index 52bf7674e7d..00000000000 --- a/src/gallium/drivers/radeon/AMDILLiteralManager.cpp +++ /dev/null @@ -1,125 +0,0 @@ -//===--- AMDILLiteralManager.cpp - AMDIL Literal Manager Pass --*- C++ -*--===// -// -// The LLVM Compiler Infrastructure -// -// This file is distributed under the University of Illinois Open Source -// License. See LICENSE.TXT for details. -// -//==-----------------------------------------------------------------------===// - -#define DEBUG_TYPE "literal_manager" - -#include "AMDIL.h" - -#include "AMDILAlgorithms.tpp" -#include "AMDILMachineFunctionInfo.h" -#include "AMDILSubtarget.h" -#include "AMDILTargetMachine.h" -#include "llvm/CodeGen/MachineFunctionPass.h" -#include "llvm/CodeGen/Passes.h" -#include "llvm/Support/Debug.h" -#include "llvm/Target/TargetMachine.h" - -using namespace llvm; - - -// AMDIL Literal Manager traverses through all of the LOADCONST instructions and -// converts them from an immediate value to the literal index. The literal index -// is valid IL, but the immediate values are not. The Immediate values must be -// aggregated and declared for clarity and to reduce the number of literals that -// are used. It is also illegal to declare the same literal twice, so this keeps -// that from occuring. - -namespace { - class AMDILLiteralManager : public MachineFunctionPass { - public: - static char ID; - AMDILLiteralManager(TargetMachine &tm AMDIL_OPT_LEVEL_DECL); - virtual const char *getPassName() const; - - bool runOnMachineFunction(MachineFunction &MF); - private: - bool trackLiterals(MachineBasicBlock::iterator *bbb); - TargetMachine &TM; - const AMDILSubtarget *mSTM; - AMDILMachineFunctionInfo *mMFI; - int32_t mLitIdx; - bool mChanged; - }; - char AMDILLiteralManager::ID = 0; -} - -namespace llvm { - FunctionPass * - createAMDILLiteralManager(TargetMachine &tm AMDIL_OPT_LEVEL_DECL) { - return new AMDILLiteralManager(tm AMDIL_OPT_LEVEL_VAR); - } - -} - -AMDILLiteralManager::AMDILLiteralManager(TargetMachine &tm - AMDIL_OPT_LEVEL_DECL) - : MachineFunctionPass(ID), - TM(tm) { -} - -bool AMDILLiteralManager::runOnMachineFunction(MachineFunction &MF) { - mChanged = false; - mMFI = MF.getInfo(); - const AMDILTargetMachine *amdtm = - reinterpret_cast(&TM); - mSTM = dynamic_cast(amdtm->getSubtargetImpl()); - safeNestedForEach(MF.begin(), MF.end(), MF.begin()->begin(), - std::bind1st(std::mem_fun(&AMDILLiteralManager::trackLiterals), this)); - return mChanged; -} - -bool AMDILLiteralManager::trackLiterals(MachineBasicBlock::iterator *bbb) { - MachineInstr *MI = *bbb; - uint32_t Opcode = MI->getOpcode(); - switch(Opcode) { - default: - return false; - case AMDIL::LOADCONST_i8: - case AMDIL::LOADCONST_i16: - case AMDIL::LOADCONST_i32: - case AMDIL::LOADCONST_i64: - case AMDIL::LOADCONST_f32: - case AMDIL::LOADCONST_f64: - break; - }; - MachineOperand &dstOp = MI->getOperand(0); - MachineOperand &litOp = MI->getOperand(1); - if (!litOp.isImm() && !litOp.isFPImm()) { - return false; - } - if (!dstOp.isReg()) { - return false; - } - // Change the literal to the correct index for each literal that is found. - if (litOp.isImm()) { - int64_t immVal = litOp.getImm(); - uint32_t idx = MI->getOpcode() == AMDIL::LOADCONST_i64 - ? mMFI->addi64Literal(immVal) - : mMFI->addi32Literal(static_cast(immVal), Opcode); - litOp.ChangeToImmediate(idx); - return false; - } - - if (litOp.isFPImm()) { - const ConstantFP *fpVal = litOp.getFPImm(); - uint32_t idx = MI->getOpcode() == AMDIL::LOADCONST_f64 - ? mMFI->addf64Literal(fpVal) - : mMFI->addf32Literal(fpVal); - litOp.ChangeToImmediate(idx); - return false; - } - - return false; -} - -const char* AMDILLiteralManager::getPassName() const { - return "AMDIL Constant Propagation"; -} - - diff --git a/src/gallium/drivers/radeon/AMDILTargetMachine.cpp b/src/gallium/drivers/radeon/AMDILTargetMachine.cpp index 089a80f821b..b62a8cb614e 100644 --- a/src/gallium/drivers/radeon/AMDILTargetMachine.cpp +++ b/src/gallium/drivers/radeon/AMDILTargetMachine.cpp @@ -184,7 +184,6 @@ bool AMDILPassConfig::addPreEmitPass() { PM.add(createAMDILCFGPreparationPass(*TM)); PM.add(createAMDILCFGStructurizerPass(*TM)); - PM.add(createAMDILLiteralManager(*TM)); return true; } diff --git a/src/gallium/drivers/radeon/Makefile.sources b/src/gallium/drivers/radeon/Makefile.sources index be8ec0ba8dd..6a347945a3d 100644 --- a/src/gallium/drivers/radeon/Makefile.sources +++ b/src/gallium/drivers/radeon/Makefile.sources @@ -29,7 +29,6 @@ CPP_SOURCES := \ AMDILIntrinsicInfo.cpp \ AMDILISelDAGToDAG.cpp \ AMDILISelLowering.cpp \ - AMDILLiteralManager.cpp \ AMDILMachineFunctionInfo.cpp \ AMDILMachinePeephole.cpp \ AMDILMCCodeEmitter.cpp \ -- 2.30.2