[Mlir-commits] [mlir] [mlir][acc] Fix build error for tiling API return value (PR #171546)

[llvmlistbot at llvm.org]

llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-openacc

Author: Razvan Lupusoru (razvanlupusoru)

<details>
<summary>Changes</summary>

The build error looks like:
error: could not convert 'newLoops' from 'SmallVector<[...],3>' to 'SmallVector<[...],6>'
  310 |   return newLoops;

The fix is to remove the explicit size in the local declaration for the SmallVector being returned.

---
Full diff: https://github.com/llvm/llvm-project/pull/171546.diff


1 Files Affected:

- (modified) mlir/lib/Dialect/OpenACC/Utils/OpenACCUtilsTiling.cpp (+1-1) 


``````````diff
diff --git a/mlir/lib/Dialect/OpenACC/Utils/OpenACCUtilsTiling.cpp b/mlir/lib/Dialect/OpenACC/Utils/OpenACCUtilsTiling.cpp
index bf82d247028b9..0b344ba2f8316 100644
--- a/mlir/lib/Dialect/OpenACC/Utils/OpenACCUtilsTiling.cpp
+++ b/mlir/lib/Dialect/OpenACC/Utils/OpenACCUtilsTiling.cpp
@@ -253,7 +253,7 @@ llvm::SmallVector<mlir::acc::LoopOp>
 mlir::acc::uncollapseLoops(mlir::acc::LoopOp origLoop, unsigned tileCount,
                            unsigned collapseCount,
                            mlir::RewriterBase &rewriter) {
-  llvm::SmallVector<mlir::acc::LoopOp, 3> newLoops;
+  llvm::SmallVector<mlir::acc::LoopOp> newLoops;
   llvm::SmallVector<mlir::Value, 3> newIVs;
   mlir::Location loc = origLoop.getLoc();
   llvm::SmallVector<bool> newInclusiveUBs;

``````````

</details>


https://github.com/llvm/llvm-project/pull/171546