[PATCH] D13858: Add an MCTargetMachine and have it construct MC classes.
Daniel Sanders via llvm-commits
llvm-commits at lists.llvm.org
Mon Oct 19 03:51:56 PDT 2015
dsanders added inline comments.
================
Comment at: lib/Target/X86/Disassembler/X86Disassembler.cpp:993
@@ -992,4 +992,3 @@
MCContext &Ctx) {
- std::unique_ptr<const MCInstrInfo> MII(T.createMCInstrInfo());
return new X86Disassembler::X86GenericDisassembler(STI, Ctx, std::move(MII));
}
----------------
I just noticed this redundant std::move(). I'll remove it since it's a leftover from when createX86Disassembler() created it's own redundant copy of MCInstrInfo.
http://reviews.llvm.org/D13858
More information about the llvm-commits
mailing list