[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