mem-cache: Remove a std::move clang says is unnecessary.
authorGabe Black <gabeblack@google.com>
Wed, 27 Nov 2019 12:10:29 +0000 (04:10 -0800)
committerGabe Black <gabeblack@google.com>
Thu, 28 Nov 2019 07:14:36 +0000 (07:14 +0000)
It also says it prevents an optimization.

Change-Id: I9c21dc1a0c53cf70cefd1400564de07d1e845a75
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/23124
Reviewed-by: Daniel Carvalho <odanrc@yahoo.com.br>
Maintainer: Gabe Black <gabeblack@google.com>
Tested-by: kokoro <noreply+kokoro@google.com>
src/mem/cache/compressors/cpack.cc

index 5603b0bf6d45574d3d2ac8c98b8db5c6a19b53e1..5d27505c3641eb63e53a1e5ea83144e73006c977 100644 (file)
@@ -63,7 +63,7 @@ CPack::compress(const uint64_t* data, Cycles& comp_lat, Cycles& decomp_lat)
     decomp_lat = Cycles(blkSize/8);
 
     // Return compressed line
-    return std::move(comp_data);
+    return comp_data;
 }
 
 CPack*