[llvm] r302256 - [Float2Int] Replace a ConstantRange copy with a move. Remove an extra call to MapVector::find.

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Fri May 5 10:09:29 PDT 2017


Author: ctopper
Date: Fri May  5 12:09:29 2017
New Revision: 302256

URL: http://llvm.org/viewvc/llvm-project?rev=302256&view=rev
Log:
[Float2Int] Replace a ConstantRange copy with a move. Remove an extra call to MapVector::find.

Modified:
    llvm/trunk/lib/Transforms/Scalar/Float2Int.cpp

Modified: llvm/trunk/lib/Transforms/Scalar/Float2Int.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Scalar/Float2Int.cpp?rev=302256&r1=302255&r2=302256&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/Scalar/Float2Int.cpp (original)
+++ llvm/trunk/lib/Transforms/Scalar/Float2Int.cpp Fri May  5 12:09:29 2017
@@ -139,10 +139,11 @@ void Float2IntPass::findRoots(Function &
 // Helper - mark I as having been traversed, having range R.
 void Float2IntPass::seen(Instruction *I, ConstantRange R) {
   DEBUG(dbgs() << "F2I: " << *I << ":" << R << "\n");
-  if (SeenInsts.find(I) != SeenInsts.end())
-    SeenInsts.find(I)->second = R;
+  auto IT = SeenInsts.find(I);
+  if (IT != SeenInsts.end())
+    IT->second = std::move(R);
   else
-    SeenInsts.insert(std::make_pair(I, R));
+    SeenInsts.insert(std::make_pair(I, std::move(R)));
 }
 
 // Helper - get a range representing a poison value.




More information about the llvm-commits mailing list