diff options
author | Daniel Dunbar <daniel@zuster.org> | 2010-05-26 17:50:16 +0000 |
---|---|---|
committer | Daniel Dunbar <daniel@zuster.org> | 2010-05-26 17:50:16 +0000 |
commit | dae5b9f804019a127e941f8f889297d5d62b81fb (patch) | |
tree | 387cc088ffd5a46aa1e84474776420d089308671 /lib/MC/MCMachOStreamer.cpp | |
parent | 84882528551bd816464a0657ad581c1fed0ac865 (diff) |
MC: Eliminate an unnecessary copy.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@104709 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/MC/MCMachOStreamer.cpp')
-rw-r--r-- | lib/MC/MCMachOStreamer.cpp | 13 |
1 files changed, 4 insertions, 9 deletions
diff --git a/lib/MC/MCMachOStreamer.cpp b/lib/MC/MCMachOStreamer.cpp index 50348e4ea3..6c738aa4a8 100644 --- a/lib/MC/MCMachOStreamer.cpp +++ b/lib/MC/MCMachOStreamer.cpp @@ -434,11 +434,6 @@ void MCMachOStreamer::EmitInstruction(const MCInst &Inst) { Assembler.getEmitter().EncodeInstruction(Inst, VecOS, Fixups); VecOS.flush(); - // FIXME: Eliminate this copy. - SmallVector<MCFixup, 4> AsmFixups; - for (unsigned i = 0, e = Fixups.size(); i != e; ++i) - AsmFixups.push_back(Fixups[i]); - // See if we might need to relax this instruction, if so it needs its own // fragment. // @@ -461,16 +456,16 @@ void MCMachOStreamer::EmitInstruction(const MCInst &Inst) { // FIXME: Revisit this design decision when relaxation is done, we may be // able to get away with not storing any extra data in the MCInst. IF->getCode() = Code; - IF->getFixups() = AsmFixups; + IF->getFixups() = Fixups; return; } // Add the fixups and data. MCDataFragment *DF = getOrCreateDataFragment(); - for (unsigned i = 0, e = AsmFixups.size(); i != e; ++i) { - AsmFixups[i].setOffset(AsmFixups[i].getOffset() + DF->getContents().size()); - DF->addFixup(AsmFixups[i]); + for (unsigned i = 0, e = Fixups.size(); i != e; ++i) { + Fixups[i].setOffset(Fixups[i].getOffset() + DF->getContents().size()); + DF->addFixup(Fixups[i]); } DF->getContents().append(Code.begin(), Code.end()); } |