[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