Fix coverity issues (#2515)

anmyachev · web-flow · commit 8d72744aabb5 · 2024-10-21T15:15:47.000+02:00
Signed-off-by: Anatoly Myachev &lt;anatoly.myachev@intel.com&gt;
diff --git a/third_party/intel/include/Analysis/AxisInfo.h b/third_party/intel/include/Analysis/AxisInfo.h
@@ -27,11 +27,12 @@ class AxisInfo {
 public:
   AxisInfo() : AxisInfo({}, {}, {}) {}
 
-  AxisInfo(DimVectorT contiguity, DimVectorT divisibility, DimVectorT constancy)
+  AxisInfo(const DimVectorT &contiguity, const DimVectorT &divisibility,
+           const DimVectorT &constancy)
       : AxisInfo(contiguity, divisibility, constancy, std::nullopt) {}
 
-  AxisInfo(DimVectorT contiguity, DimVectorT divisibility, DimVectorT constancy,
-           std::optional<int64_t> constantValue)
+  AxisInfo(const DimVectorT &contiguity, const DimVectorT &divisibility,
+           const DimVectorT &constancy, std::optional<int64_t> constantValue)
       : contiguity(contiguity), divisibility(divisibility),
         constancy(constancy), constantValue(constantValue) {
     assert(divisibility.size() == contiguity.size());
diff --git a/third_party/intel/lib/Analysis/AxisInfo.cpp b/third_party/intel/lib/Analysis/AxisInfo.cpp
@@ -1344,7 +1344,7 @@ void ModuleAxisInfoAnalysis::initialize(FunctionOpInterface funcOp) {
     } else {
       curAxisInfo = axisInfo;
     }
-    (*axisInfoMap)[value] = curAxisInfo;
+    (*axisInfoMap)[value] = std::move(curAxisInfo);
   };
   funcOp.walk([&](Operation *op) {
     for (auto value : op->getResults()) {

Original file line number	Diff line number	Diff line change
`@@ -1344,7 +1344,7 @@ void ModuleAxisInfoAnalysis::initialize(FunctionOpInterface funcOp) {`
`1344`	`1344`	`} else {`
`1345`	`1345`	`curAxisInfo = axisInfo;`
`1346`	`1346`	`}`
`1347`		`- (*axisInfoMap)[value] = curAxisInfo;`
	`1347`	`+ (*axisInfoMap)[value] = std::move(curAxisInfo);`
`1348`	`1348`	`};`
`1349`	`1349`	`funcOp.walk([&](Operation *op) {`
`1350`	`1350`	`for (auto value : op->getResults()) {`