radeon/llvm: Remove AMDILLiteralManager.cpp
authorTom Stellard <thomas.stellard@amd.com>
Wed, 25 Apr 2012 01:57:14 +0000 (21:57 -0400)
committerTom Stellard <thomas.stellard@amd.com>
Wed, 25 Apr 2012 13:02:17 +0000 (09:02 -0400)
src/gallium/drivers/radeon/AMDIL.h
src/gallium/drivers/radeon/AMDILLiteralManager.cpp [deleted file]
src/gallium/drivers/radeon/AMDILTargetMachine.cpp
src/gallium/drivers/radeon/Makefile.sources

index 536fd69da312cb0c90b50f81146b8f536e797ac2..317ea124f66076c405c5354f7b04615a560fbd92 100644 (file)
@@ -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 (file)
index 52bf767..0000000
+++ /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<AMDILMachineFunctionInfo>();
-  const AMDILTargetMachine *amdtm =
-    reinterpret_cast<const AMDILTargetMachine *>(&TM);
-  mSTM = dynamic_cast<const AMDILSubtarget *>(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<int>(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";
-}
-
-
index 089a80f821b5cb5dccdc41916237ee3e81869be0..b62a8cb614ea7d8facd90974cc5df6cac70886f1 100644 (file)
@@ -184,7 +184,6 @@ bool AMDILPassConfig::addPreEmitPass()
 {
   PM.add(createAMDILCFGPreparationPass(*TM));
   PM.add(createAMDILCFGStructurizerPass(*TM));
-  PM.add(createAMDILLiteralManager(*TM));
   return true;
 }
 
index be8ec0ba8ddc445576bf93098ced3da365a3c46e..6a347945a3d59b2e4896aaf6b5102f6f36275587 100644 (file)
@@ -29,7 +29,6 @@ CPP_SOURCES := \
        AMDILIntrinsicInfo.cpp          \
        AMDILISelDAGToDAG.cpp           \
        AMDILISelLowering.cpp           \
-       AMDILLiteralManager.cpp         \
        AMDILMachineFunctionInfo.cpp    \
        AMDILMachinePeephole.cpp        \
        AMDILMCCodeEmitter.cpp          \