diff options
| author | Clement Courbet <courbet@google.com> | 2018-11-07 16:52:50 +0000 |
|---|---|---|
| committer | Clement Courbet <courbet@google.com> | 2018-11-07 16:52:50 +0000 |
| commit | 5b0d783078965bc22201259bd61f86301c97376f (patch) | |
| tree | 23f6c06ada5a50bc2f49b391a17f0c95db01c35f /llvm/tools/llvm-exegesis/lib/X86/Target.cpp | |
| parent | a4d9e2293a8491b23410044d5309ad066126a0ce (diff) | |
| download | bcm5719-llvm-5b0d783078965bc22201259bd61f86301c97376f.tar.gz bcm5719-llvm-5b0d783078965bc22201259bd61f86301c97376f.zip | |
[llvm-exegesis] Remove superfluous move.
/Users/buildslave/as-bldslv9_new/lld-x86_64-darwin13/llvm.src/tools/llvm-exegesis/lib/X86/Target.cpp:155:12: error: moving a local object in a return statement prevents copy elision [-Werror,-Wpessimizing-move]
return std::move(Error);
^
/Users/buildslave/as-bldslv9_new/lld-x86_64-darwin13/llvm.src/tools/llvm-exegesis/lib/X86/Target.cpp:155:12: note: remove std::move call here
return std::move(Error);
^~~~~~~~~~ ~
llvm-svn: 346333
Diffstat (limited to 'llvm/tools/llvm-exegesis/lib/X86/Target.cpp')
| -rw-r--r-- | llvm/tools/llvm-exegesis/lib/X86/Target.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/tools/llvm-exegesis/lib/X86/Target.cpp b/llvm/tools/llvm-exegesis/lib/X86/Target.cpp index 7f26adbe237..618e4d77db4 100644 --- a/llvm/tools/llvm-exegesis/lib/X86/Target.cpp +++ b/llvm/tools/llvm-exegesis/lib/X86/Target.cpp @@ -152,7 +152,7 @@ static llvm::Error IsInvalidOpcode(const Instruction &Instr) { return llvm::make_error<BenchmarkFailure>( "unsupported opcode: Push/Pop/AdjCallStack"); if (llvm::Error Error = isInvalidMemoryInstr(Instr)) - return std::move(Error); + return Error; // We do not handle instructions with OPERAND_PCREL. for (const Operand &Op : Instr.Operands) if (Op.isExplicit() && |
