From 5c0e64fcd6d93ed1033165ed74b524831fff285e Mon Sep 17 00:00:00 2001 From: Benjamin Kramer Date: Sat, 21 Feb 2015 16:22:48 +0000 Subject: [PATCH] Calling memmove on a MachineOperand is totally safe. While it's not POD due to the user-defined constructor, it's still a trivially copyable type. No functional change. llvm-svn: 230141 --- llvm/lib/CodeGen/MachineInstr.cpp | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/llvm/lib/CodeGen/MachineInstr.cpp b/llvm/lib/CodeGen/MachineInstr.cpp index 0514eb6..50c4e81 100644 --- a/llvm/lib/CodeGen/MachineInstr.cpp +++ b/llvm/lib/CodeGen/MachineInstr.cpp @@ -674,14 +674,8 @@ static void moveOperands(MachineOperand *Dst, MachineOperand *Src, if (MRI) return MRI->moveOperands(Dst, Src, NumOps); - // Here it would be convenient to call memmove, so that isn't allowed because - // MachineOperand has a constructor and so isn't a POD type. - if (Dst < Src) - for (unsigned i = 0; i != NumOps; ++i) - new (Dst + i) MachineOperand(Src[i]); - else - for (unsigned i = NumOps; i ; --i) - new (Dst + i - 1) MachineOperand(Src[i - 1]); + // MachineOperand is a trivially copyable type so we can just use memmove. + std::memmove(Dst, Src, NumOps * sizeof(MachineOperand)); } /// addOperand - Add the specified operand to the instruction. If it is an -- 2.7.4