various minor cleanups, no functionality change.
authorChris Lattner <sabre@nondot.org>
Thu, 9 Jul 2009 05:42:07 +0000 (05:42 +0000)
committerChris Lattner <sabre@nondot.org>
Thu, 9 Jul 2009 05:42:07 +0000 (05:42 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@75108 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Target/X86/AsmPrinter/X86ATTAsmPrinter.cpp

index f6f486fa9aa16b807c5755575ed990533f6f24e4..ede0fa98e8f58d971e8f0ba6aabc878cfa4802aa 100644 (file)
@@ -428,6 +428,10 @@ void X86ATTAsmPrinter::printOperand(const MachineInstr *MI, unsigned OpNo,
       needCloseParen = true;
     }
 
+    // Handle dllimport linkage.
+    if (MO.getTargetFlags() == X86II::MO_DLLIMPORT)
+      O << "__imp_";
+    
     if (Subtarget->isPICStyleStub()) {
       // DARWIN/X86-32 in != static mode.
 
@@ -457,24 +461,20 @@ void X86ATTAsmPrinter::printOperand(const MachineInstr *MI, unsigned OpNo,
         PrintPICBaseSymbol();
       }        
     } else {
-      // Handle dllimport linkage.
-      if (MO.getTargetFlags() == X86II::MO_DLLIMPORT)
-        O << "__imp_";
       O << Name;
     }
 
-    printOffset(MO.getOffset());
-
     if (needCloseParen)
       O << ')';
     
+    printOffset(MO.getOffset());
     break;
   }
   case MachineOperand::MO_ExternalSymbol:
     /// NOTE: MO_ExternalSymbol in a non-pcrel_imm context is *only* generated
     /// by _GLOBAL_OFFSET_TABLE_ on X86-32.  All others are call operands, which
     /// are pcrel_imm's.
-    assert(!Subtarget->is64Bit() && !Subtarget->isPICStyleRIPRel());
+    assert(!Subtarget->is64Bit());
     // These are never used as memory operands.
     assert(Modifier == 0 || strcmp(Modifier, "mem"));
     O << '$';