[Mlir-commits] [mlir] [mlir][acc] Refactor and expand OpenACC utils (PR #115119)

[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 cbc7812565b0b0d60c0dadbd3743650f863237d4 1d816056c03b3d0d533a6c7e7c4bac75b80b4c4c --extensions cpp,h -- mlir/include/mlir/Dialect/OpenACC/Utils/OpenACCUtils.h mlir/lib/Dialect/OpenACC/Utils/OpenACCUtils.cpp mlir/unittests/Dialect/OpenACC/OpenACCUtilsTest.cpp mlir/include/mlir/Dialect/OpenACC/OpenACC.h mlir/lib/Dialect/OpenACC/IR/OpenACC.cpp mlir/lib/Dialect/OpenACC/Transforms/LegalizeDataValues.cpp
``````````

</details>

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

``````````diff
diff --git a/mlir/include/mlir/Dialect/OpenACC/Utils/OpenACCUtils.h b/mlir/include/mlir/Dialect/OpenACC/Utils/OpenACCUtils.h
index e1d01aa251..4a93d3ebe6 100644
--- a/mlir/include/mlir/Dialect/OpenACC/Utils/OpenACCUtils.h
+++ b/mlir/include/mlir/Dialect/OpenACC/Utils/OpenACCUtils.h
@@ -54,8 +54,7 @@ mlir::SmallVector<mlir::Value> getBounds(mlir::Operation *accDataClauseOp);
 /// an acc dat aclause operation).
 bool setBounds(mlir::Operation *accDataClauseOp,
                mlir::SmallVector<mlir::Value> &bounds);
-bool setBounds(mlir::Operation *accDataClauseOp,
-               mlir::Value bound);
+bool setBounds(mlir::Operation *accDataClauseOp, mlir::Value bound);
 
 /// Used to obtain the `dataClause` from a data clause operation.
 /// Returns empty optional if not a data operation.
diff --git a/mlir/lib/Dialect/OpenACC/Transforms/LegalizeDataValues.cpp b/mlir/lib/Dialect/OpenACC/Transforms/LegalizeDataValues.cpp
index 028f53d05b..897c2dabd8 100644
--- a/mlir/lib/Dialect/OpenACC/Transforms/LegalizeDataValues.cpp
+++ b/mlir/lib/Dialect/OpenACC/Transforms/LegalizeDataValues.cpp
@@ -8,8 +8,8 @@
 
 #include "mlir/Dialect/Func/IR/FuncOps.h"
 #include "mlir/Dialect/OpenACC/OpenACC.h"
-#include "mlir/Dialect/OpenACC/Utils/OpenACCUtils.h"
 #include "mlir/Dialect/OpenACC/Transforms/Passes.h"
+#include "mlir/Dialect/OpenACC/Utils/OpenACCUtils.h"
 #include "mlir/Pass/Pass.h"
 #include "llvm/Support/ErrorHandling.h"
 

``````````

</details>


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