[llvm] r204892 - Move MCSymbolizer's constructor into header. It's trivial - there's no need for

Lang Hames lhames at gmail.com
Wed Mar 26 19:42:53 PDT 2014


Author: lhames
Date: Wed Mar 26 21:42:52 2014
New Revision: 204892

URL: http://llvm.org/viewvc/llvm-project?rev=204892&view=rev
Log:
Move MCSymbolizer's constructor into header. It's trivial - there's no need for
it to be out-of-line.

Modified:
    llvm/trunk/include/llvm/MC/MCSymbolizer.h
    llvm/trunk/lib/MC/MCSymbolizer.cpp

Modified: llvm/trunk/include/llvm/MC/MCSymbolizer.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/MC/MCSymbolizer.h?rev=204892&r1=204891&r2=204892&view=diff
==============================================================================
--- llvm/trunk/include/llvm/MC/MCSymbolizer.h (original)
+++ llvm/trunk/include/llvm/MC/MCSymbolizer.h Wed Mar 26 21:42:52 2014
@@ -46,7 +46,9 @@ protected:
 
 public:
   /// \brief Construct an MCSymbolizer, taking ownership of \p RelInfo.
-  MCSymbolizer(MCContext &Ctx, std::unique_ptr<MCRelocationInfo> RelInfo);
+  MCSymbolizer(MCContext &Ctx, std::unique_ptr<MCRelocationInfo> RelInfo)
+    : Ctx(Ctx), RelInfo(std::move(RelInfo)) {}
+
   virtual ~MCSymbolizer();
 
   /// \brief Try to add a symbolic operand instead of \p Value to the MCInst.

Modified: llvm/trunk/lib/MC/MCSymbolizer.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/MC/MCSymbolizer.cpp?rev=204892&r1=204891&r2=204892&view=diff
==============================================================================
--- llvm/trunk/lib/MC/MCSymbolizer.cpp (original)
+++ llvm/trunk/lib/MC/MCSymbolizer.cpp Wed Mar 26 21:42:52 2014
@@ -8,13 +8,8 @@
 //===----------------------------------------------------------------------===//
 
 #include "llvm/MC/MCSymbolizer.h"
-#include "llvm/MC/MCRelocationInfo.h"
 
 using namespace llvm;
 
-MCSymbolizer::MCSymbolizer(MCContext &Ctx,
-                           std::unique_ptr<MCRelocationInfo> RelInfo)
-    : Ctx(Ctx), RelInfo(std::move(RelInfo)) {}
-
 MCSymbolizer::~MCSymbolizer() {
 }





More information about the llvm-commits mailing list