From ffcce6679dc401d1da9427d4076c68b824cf30e9 Mon Sep 17 00:00:00 2001
From: Nishant Patel <nishant.b.patel@intel.com>
Date: Mon, 13 Nov 2023 17:07:37 -0800
Subject: [PATCH] Fix coverity low impact issue

---
 lib/Conversion/XeTileToXeGPU/SCFOpConversion.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/lib/Conversion/XeTileToXeGPU/SCFOpConversion.cpp b/lib/Conversion/XeTileToXeGPU/SCFOpConversion.cpp
index a89595b2b..8b83ca066 100644
--- a/lib/Conversion/XeTileToXeGPU/SCFOpConversion.cpp
+++ b/lib/Conversion/XeTileToXeGPU/SCFOpConversion.cpp
@@ -31,7 +31,7 @@ struct SgSCFForOpBlockPattern
     llvm::SmallVector<mlir::Value> convertedArgs;
     // OpAdaptor is defined with ValueRange, so it contains results after
     // One-to-N mapping
-    for (auto values : adaptor.getInitArgs())
+    for (auto &values : adaptor.getInitArgs())
       convertedArgs.append(values.begin(), values.end());
 
     auto argumentTys = op.getRegion().getArgumentTypes();
@@ -74,7 +74,7 @@ struct SgSCFYieldOpPattern
   matchAndRewrite(mlir::scf::YieldOp op, OpAdaptor adaptor,
                   imex::XeGPUOneToNPatterRewriter &rewriter) const override {
     llvm::SmallVector<mlir::Value> convertedResults;
-    for (auto values : adaptor.getResults())
+    for (auto &values : adaptor.getResults())
       convertedResults.append(values.begin(), values.end());
 
     auto newOp =