Remove the DisasmEnabled AsmPrinter variable and just look it
authorEric Christopher <echristo@gmail.com>
Thu, 19 Feb 2015 01:10:49 +0000 (01:10 +0000)
committerEric Christopher <echristo@gmail.com>
Thu, 19 Feb 2015 01:10:49 +0000 (01:10 +0000)
up on the subtarget where it's set anyhow than looking it up
2-3 times in the same place.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@229802 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Target/R600/AMDGPUAsmPrinter.cpp
lib/Target/R600/AMDGPUAsmPrinter.h
lib/Target/R600/AMDGPUMCInstLower.cpp

index 4721a44e2778df7feb06ae34b427ceec3f376a91..1c65e7e3566296f043812e4fa3c26a6ddf929e2f 100644 (file)
@@ -86,9 +86,7 @@ extern "C" void LLVMInitializeR600AsmPrinter() {
 
 AMDGPUAsmPrinter::AMDGPUAsmPrinter(TargetMachine &TM,
                                    std::unique_ptr<MCStreamer> Streamer)
-    : AsmPrinter(TM, std::move(Streamer)) {
-  DisasmEnabled = TM.getSubtarget<AMDGPUSubtarget>().dumpCode();
-}
+    : AsmPrinter(TM, std::move(Streamer)) {}
 
 void AMDGPUAsmPrinter::EmitEndOfAsmFile(Module &M) {
 
@@ -160,7 +158,7 @@ bool AMDGPUAsmPrinter::runOnMachineFunction(MachineFunction &MF) {
     }
   }
 
-  if (STM.dumpCode() && DisasmEnabled) {
+  if (STM.dumpCode()) {
 
     OutStreamer.SwitchSection(
         Context.getELFSection(".AMDGPU.disasm", ELF::SHT_NOTE, 0));
index 375b3e1b4b6fbb85f08a537435013bad521e5426..58ffb1ed4edfb7a75dd2614fe60fcba973a5fc35 100644 (file)
@@ -100,7 +100,6 @@ public:
   void EmitEndOfAsmFile(Module &M) override;
 
 protected:
-  bool DisasmEnabled;
   std::vector<std::string> DisasmLines, HexLines;
   size_t DisasmLineMaxLen;
 };
index bfb36e80916b01ae8294bb403e19809bf30ca278..19fe2b372e5cf06042935a3ab9a603d47da10686 100644 (file)
@@ -116,7 +116,7 @@ void AMDGPUAsmPrinter::EmitInstruction(const MachineInstr *MI) {
     MCInstLowering.lower(MI, TmpInst);
     EmitToStreamer(OutStreamer, TmpInst);
 
-    if (DisasmEnabled) {
+    if (MF->getSubtarget<AMDGPUSubtarget>().dumpCode()) {
       // Disassemble instruction/operands to text.
       DisasmLines.resize(DisasmLines.size() + 1);
       std::string &DisasmLine = DisasmLines.back();