[llvm-commits] [llvm] r162404 - /llvm/trunk/include/llvm/MC/MCAssembler.h

[Jim Grosbach]

Author: grosbach
Date: Wed Aug 22 17:19:40 2012
New Revision: 162404

URL: http://llvm.org/viewvc/llvm-project?rev=162404&view=rev
Log:
MCInstFragment constructor should take by-reference MCInst.

The MCInst is immediately passed to the copy-constructor for local
storage, so there's no need for the parameter itself to be by-value.

Modified:
    llvm/trunk/include/llvm/MC/MCAssembler.h

Modified: llvm/trunk/include/llvm/MC/MCAssembler.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/MC/MCAssembler.h?rev=162404&r1=162403&r2=162404&view=diff
==============================================================================
--- llvm/trunk/include/llvm/MC/MCAssembler.h (original)
+++ llvm/trunk/include/llvm/MC/MCAssembler.h Wed Aug 22 17:19:40 2012
@@ -176,7 +176,7 @@
   typedef SmallVectorImpl<MCFixup>::iterator fixup_iterator;
 
 public:
-  MCInstFragment(MCInst _Inst, MCSectionData *SD = 0)
+  MCInstFragment(const MCInst &_Inst, MCSectionData *SD = 0)
     : MCFragment(FT_Inst, SD), Inst(_Inst) {
   }