[Mlir-commits] [flang] [llvm] [mlir] [flang][OpenMP] Enable tiling (PR #143715)

[Sergio Afonso]

================
@@ -522,6 +535,19 @@ bool ClauseProcessor::processProcBind(
   return false;
 }
 
+bool ClauseProcessor::processTileSizes(
+    lower::pft::Evaluation &eval, mlir::omp::LoopNestOperands &result) const {
+  bool found = false;
+  llvm::SmallVector<int64_t> sizeValues;
+  auto *ompCons{eval.getIf<parser::OpenMPConstruct>()};
+  collectTileSizesFromOpenMPConstruct(ompCons, sizeValues, semaCtx);
+  if (sizeValues.size() > 0) {
+    found = true;
+    result.tileSizes = sizeValues;
+  }
+  return found;
----------------
skatrak wrote:

Nit: I think this can be simplified a bit. It is already assumed that result structures are passed empty to all other `ClauseProcessor` methods as well.
```suggestion
  auto *ompCons{eval.getIf<parser::OpenMPConstruct>()};
  collectTileSizesFromOpenMPConstruct(ompCons, result.tileSizes, semaCtx);
  return !result.tileSizes.empty();
```

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