[llvm-commits] [llvm] r166014 - in /llvm/trunk/lib/Target/X86: X86AsmPrinter.cpp X86MCInstLower.cpp X86MCInstLower.h

[Jim Grosbach]

Nice!

-j
On Oct 15, 2012, at 11:01 PM, Craig Topper <craig.topper at gmail.com> wrote:

> Author: ctopper
> Date: Tue Oct 16 01:01:50 2012
> New Revision: 166014
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=166014&view=rev
> Log:
> Move X86MCInstLower class definition into implementation file. It's not needed outside.
> 
> Removed:
>    llvm/trunk/lib/Target/X86/X86MCInstLower.h
> Modified:
>    llvm/trunk/lib/Target/X86/X86AsmPrinter.cpp
>    llvm/trunk/lib/Target/X86/X86MCInstLower.cpp
> 
> Modified: llvm/trunk/lib/Target/X86/X86AsmPrinter.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86AsmPrinter.cpp?rev=166014&r1=166013&r2=166014&view=diff
> ==============================================================================
> --- llvm/trunk/lib/Target/X86/X86AsmPrinter.cpp (original)
> +++ llvm/trunk/lib/Target/X86/X86AsmPrinter.cpp Tue Oct 16 01:01:50 2012
> @@ -13,7 +13,6 @@
> //===----------------------------------------------------------------------===//
> 
> #include "X86AsmPrinter.h"
> -#include "X86MCInstLower.h"
> #include "X86.h"
> #include "X86COFFMachineModuleInfo.h"
> #include "X86MachineFunctionInfo.h"
> 
> Modified: llvm/trunk/lib/Target/X86/X86MCInstLower.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86MCInstLower.cpp?rev=166014&r1=166013&r2=166014&view=diff
> ==============================================================================
> --- llvm/trunk/lib/Target/X86/X86MCInstLower.cpp (original)
> +++ llvm/trunk/lib/Target/X86/X86MCInstLower.cpp Tue Oct 16 01:01:50 2012
> @@ -12,7 +12,6 @@
> //
> //===----------------------------------------------------------------------===//
> 
> -#include "X86MCInstLower.h"
> #include "X86AsmPrinter.h"
> #include "X86COFFMachineModuleInfo.h"
> #include "InstPrinter/X86ATTInstPrinter.h"
> @@ -29,6 +28,31 @@
> #include "llvm/ADT/SmallString.h"
> using namespace llvm;
> 
> +namespace {
> +
> +/// X86MCInstLower - This class is used to lower an MachineInstr into an MCInst.
> +class X86MCInstLower {
> +  MCContext &Ctx;
> +  Mangler *Mang;
> +  const MachineFunction &MF;
> +  const TargetMachine &TM;
> +  const MCAsmInfo &MAI;
> +  X86AsmPrinter &AsmPrinter;
> +public:
> +  X86MCInstLower(Mangler *mang, const MachineFunction &MF,
> +                 X86AsmPrinter &asmprinter);
> +
> +  void Lower(const MachineInstr *MI, MCInst &OutMI) const;
> +
> +  MCSymbol *GetSymbolFromOperand(const MachineOperand &MO) const;
> +  MCOperand LowerSymbolOperand(const MachineOperand &MO, MCSymbol *Sym) const;
> +
> +private:
> +  MachineModuleInfoMachO &getMachOMMI() const;
> +};
> +
> +} // end anonymous namespace
> +
> X86MCInstLower::X86MCInstLower(Mangler *mang, const MachineFunction &mf,
>                                X86AsmPrinter &asmprinter)
> : Ctx(mf.getContext()), Mang(mang), MF(mf), TM(mf.getTarget()),
> 
> Removed: llvm/trunk/lib/Target/X86/X86MCInstLower.h
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86MCInstLower.h?rev=166013&view=auto
> ==============================================================================
> --- llvm/trunk/lib/Target/X86/X86MCInstLower.h (original)
> +++ llvm/trunk/lib/Target/X86/X86MCInstLower.h (removed)
> @@ -1,52 +0,0 @@
> -//===-- X86MCInstLower.h - Lower MachineInstr to MCInst ---------*- C++ -*-===//
> -//
> -//                     The LLVM Compiler Infrastructure
> -//
> -// This file is distributed under the University of Illinois Open Source
> -// License. See LICENSE.TXT for details.
> -//
> -//===----------------------------------------------------------------------===//
> -
> -#ifndef X86_MCINSTLOWER_H
> -#define X86_MCINSTLOWER_H
> -
> -#include "llvm/Support/Compiler.h"
> -
> -namespace llvm {
> -  class MCAsmInfo;
> -  class MCContext;
> -  class MCInst;
> -  class MCOperand;
> -  class MCSymbol;
> -  class MachineInstr;
> -  class MachineFunction;
> -  class MachineModuleInfoMachO;
> -  class MachineOperand;
> -  class Mangler;
> -  class TargetMachine;
> -  class X86AsmPrinter;
> -
> -/// X86MCInstLower - This class is used to lower an MachineInstr into an MCInst.
> -class LLVM_LIBRARY_VISIBILITY X86MCInstLower {
> -  MCContext &Ctx;
> -  Mangler *Mang;
> -  const MachineFunction &MF;
> -  const TargetMachine &TM;
> -  const MCAsmInfo &MAI;
> -  X86AsmPrinter &AsmPrinter;
> -public:
> -  X86MCInstLower(Mangler *mang, const MachineFunction &MF,
> -                 X86AsmPrinter &asmprinter);
> -
> -  void Lower(const MachineInstr *MI, MCInst &OutMI) const;
> -
> -  MCSymbol *GetSymbolFromOperand(const MachineOperand &MO) const;
> -  MCOperand LowerSymbolOperand(const MachineOperand &MO, MCSymbol *Sym) const;
> -
> -private:
> -  MachineModuleInfoMachO &getMachOMMI() const;
> -};
> -
> -}
> -
> -#endif
> 
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits