[Mlir-commits] [mlir] [mlir][OpenMP] Introduce 'omp.iterators' for OpenMP iterator modifiers (PR #181322)

[llvmlistbot at llvm.org]

github-actions[bot] wrote:

<!--LLVM CODE FORMAT COMMENT: {clang-format}-->


:warning: C/C++ code formatter, clang-format found issues in your code. :warning:

<details>
<summary>
You can test this locally with the following command:
</summary>

``````````bash
git-clang-format --diff origin/main HEAD --extensions cpp -- mlir/lib/Dialect/OpenMP/IR/OpenMPDialect.cpp --diff_from_common_commit
``````````

:warning:
The reproduction instructions above might return results for more than one PR
in a stack if you are using a stacked PR workflow. You can limit the results by
changing `origin/main` to the base branch/commit you want to compare against.
:warning:

</details>

<details>
<summary>
View the diff from clang-format here.
</summary>

``````````diff
diff --git a/mlir/lib/Dialect/OpenMP/IR/OpenMPDialect.cpp b/mlir/lib/Dialect/OpenMP/IR/OpenMPDialect.cpp
index 86cb0f3f8..e7a935309 100644
--- a/mlir/lib/Dialect/OpenMP/IR/OpenMPDialect.cpp
+++ b/mlir/lib/Dialect/OpenMP/IR/OpenMPDialect.cpp
@@ -4576,13 +4576,13 @@ parseIteratorsHeader(OpAsmParser &parser, Region &region,
 
   llvm::SMLoc ivLoc = parser.getCurrentLocation();
   SmallVector<OpAsmParser::Argument> ivArgs;
-  
+
   // Parse induction variables: %i : i32, %j : i32
   if (parser.parseCommaSeparatedList([&]() -> ParseResult {
         OpAsmParser::Argument &arg = ivArgs.emplace_back();
         if (parser.parseArgument(arg))
           return failure();
-        
+
         // Optional type, default to Index if not provided
         if (succeeded(parser.parseOptionalColon())) {
           if (parser.parseType(arg.type))
@@ -4618,8 +4618,8 @@ parseIteratorsHeader(OpAsmParser &parser, Region &region,
 
   if (ivArgs.size() != lbs.size())
     return parser.emitError(ivLoc)
-           << "mismatch: " << ivArgs.size() << " variables but "
-           << lbs.size() << " ranges";
+           << "mismatch: " << ivArgs.size() << " variables but " << lbs.size()
+           << " ranges";
 
   for (auto &arg : ivArgs) {
     lbTypes.push_back(arg.type);
@@ -4662,7 +4662,7 @@ LogicalResult IteratorsOp::verify() {
 
   Block &b = getRegion().front();
   auto yield = llvm::dyn_cast<omp::YieldOp>(b.getTerminator());
-  
+
   if (!yield)
     return emitOpError() << "region must be terminated by omp.yield";
 

``````````

</details>


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