[Mlir-commits] [mlir] [mlir][openacc] Update acc.loop to be a proper loop like operation (PR #67355)

[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 740ee00a4c1d841204f57c88c04fa62e38bd1322 a2f4fa15f68c473c2ee162a7358f824b199ee2d0 -- mlir/include/mlir/Dialect/OpenACC/OpenACC.h mlir/lib/Dialect/OpenACC/IR/OpenACC.cpp
``````````

</details>

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

``````````diff
diff --git a/mlir/lib/Dialect/OpenACC/IR/OpenACC.cpp b/mlir/lib/Dialect/OpenACC/IR/OpenACC.cpp
index 91e83e1ffa28..06d0d29eb446 100644
--- a/mlir/lib/Dialect/OpenACC/IR/OpenACC.cpp
+++ b/mlir/lib/Dialect/OpenACC/IR/OpenACC.cpp
@@ -858,7 +858,7 @@ void printVectorClause(OpAsmPrinter &p, Operation *op, Value vectorLength,
 }
 
 LogicalResult acc::LoopOp::verify() {
-  if (!getUpperbound().empty() && 
+  if (!getUpperbound().empty() &&
       (getUpperbound().size() != getInclusiveUpperbound()->size()))
     return emitError() << "inclusiveUpperbound size is expected to be the same"
                        << " as upperbound size";
@@ -911,7 +911,7 @@ parseLoopControl(OpAsmParser &parser, Region &region,
   SmallVector<OpAsmParser::Argument> inductionVars;
   if (succeeded(parser.parseOptionalLParen())) {
     if (parser.parseArgumentList(inductionVars, OpAsmParser::Delimiter::None,
-                                /*allowType=*/true) ||
+                                 /*allowType=*/true) ||
         parser.parseRParen() || parser.parseEqual() || parser.parseLParen() ||
         parser.parseOperandList(lowerbound, inductionVars.size(),
                                 OpAsmParser::Delimiter::None) ||

``````````

</details>


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