[llvm-branch-commits] [llvm-branch] r232162 - Try to make AsmPrinter vtable compatible with previous shared library.

Daniel Sanders daniel.sanders at imgtec.com
Fri Mar 13 04:54:32 PDT 2015


Author: dsanders
Date: Fri Mar 13 06:54:32 2015
New Revision: 232162

URL: http://llvm.org/viewvc/llvm-project?rev=232162&view=rev
Log:
Try to make AsmPrinter vtable compatible with previous shared library.

r232083 added AsmPrinter::emitInlineAsmStart() before
AsmPrinter::emitInlineAsmEnd(), changing the vtable in an incompatible way.
This patch swaps the declaration order so that the new vtable entry is at the
end of the vtable.


Modified:
    llvm/branches/release_35/include/llvm/CodeGen/AsmPrinter.h
    llvm/branches/release_35/lib/Target/Mips/MipsAsmPrinter.h

Modified: llvm/branches/release_35/include/llvm/CodeGen/AsmPrinter.h
URL: http://llvm.org/viewvc/llvm-project/llvm/branches/release_35/include/llvm/CodeGen/AsmPrinter.h?rev=232162&r1=232161&r2=232162&view=diff
==============================================================================
--- llvm/branches/release_35/include/llvm/CodeGen/AsmPrinter.h (original)
+++ llvm/branches/release_35/include/llvm/CodeGen/AsmPrinter.h Fri Mar 13 06:54:32 2015
@@ -461,10 +461,6 @@ public:
                                      unsigned AsmVariant, const char *ExtraCode,
                                      raw_ostream &OS);
 
-  /// Let the target do anything it needs to do before emitting inlineasm.
-  /// \p StartInfo - the subtarget info before parsing inline asm
-  virtual void emitInlineAsmStart(const MCSubtargetInfo &StartInfo) const;
-
   /// Let the target do anything it needs to do after emitting inlineasm.
   /// This callback can be used restore the original mode in case the
   /// inlineasm contains directives to switch modes.
@@ -474,6 +470,10 @@ public:
   virtual void emitInlineAsmEnd(const MCSubtargetInfo &StartInfo,
                                 const MCSubtargetInfo *EndInfo) const;
 
+  /// Let the target do anything it needs to do before emitting inlineasm.
+  /// \p StartInfo - the subtarget info before parsing inline asm
+  virtual void emitInlineAsmStart(const MCSubtargetInfo &StartInfo) const;
+
 private:
   /// Private state for PrintSpecial()
   // Assign a unique ID to this machine instruction.

Modified: llvm/branches/release_35/lib/Target/Mips/MipsAsmPrinter.h
URL: http://llvm.org/viewvc/llvm-project/llvm/branches/release_35/lib/Target/Mips/MipsAsmPrinter.h?rev=232162&r1=232161&r2=232162&view=diff
==============================================================================
--- llvm/branches/release_35/lib/Target/Mips/MipsAsmPrinter.h (original)
+++ llvm/branches/release_35/lib/Target/Mips/MipsAsmPrinter.h Fri Mar 13 06:54:32 2015
@@ -60,11 +60,11 @@ private:
   std::map<const char *, const llvm::Mips16HardFloatInfo::FuncSignature *>
   StubsNeeded;
 
-  void emitInlineAsmStart(const MCSubtargetInfo &StartInfo) const override;
-
   void emitInlineAsmEnd(const MCSubtargetInfo &StartInfo,
                         const MCSubtargetInfo *EndInfo) const override;
 
+  void emitInlineAsmStart(const MCSubtargetInfo &StartInfo) const override;
+
   void EmitJal(MCSymbol *Symbol);
 
   void EmitInstrReg(unsigned Opcode, unsigned Reg);





More information about the llvm-branch-commits mailing list