[llvm] r360025 - [Analysis] Remove duplicated std::move from LocRange constructor
Simon Pilgrim via llvm-commits
llvm-commits at lists.llvm.org
Mon May 6 03:04:25 PDT 2019
Author: rksimon
Date: Mon May 6 03:04:23 2019
New Revision: 360025
URL: http://llvm.org/viewvc/llvm-project?rev=360025&view=rev
Log:
[Analysis] Remove duplicated std::move from LocRange constructor
scan-build was reporting that we were referencing a moved variable - in fact we were moving it twice.....
Modified:
llvm/trunk/include/llvm/Analysis/LoopInfo.h
Modified: llvm/trunk/include/llvm/Analysis/LoopInfo.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Analysis/LoopInfo.h?rev=360025&r1=360024&r2=360025&view=diff
==============================================================================
--- llvm/trunk/include/llvm/Analysis/LoopInfo.h (original)
+++ llvm/trunk/include/llvm/Analysis/LoopInfo.h Mon May 6 03:04:23 2019
@@ -470,7 +470,7 @@ public:
public:
LocRange() {}
- LocRange(DebugLoc Start) : Start(std::move(Start)), End(std::move(Start)) {}
+ LocRange(DebugLoc Start) : Start(Start), End(Start) {}
LocRange(DebugLoc Start, DebugLoc End)
: Start(std::move(Start)), End(std::move(End)) {}
More information about the llvm-commits
mailing list