[Mlir-commits] [mlir] [mlir][spirv] Support function argument decorations for ptr in the PhysicalStorageBuffer (PR #76353)

llvmlistbot at llvm.org llvmlistbot at llvm.org
Mon Dec 25 17:29:55 PST 2023


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 9971b9ab195dd629fb2625c5c5c674d355760231 4ee3313d9f4d9301bbf5fb0e329486281d69beb3 -- mlir/lib/Dialect/SPIRV/IR/SPIRVDialect.cpp mlir/lib/Dialect/SPIRV/IR/SPIRVOps.cpp
``````````

</details>

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

``````````diff
diff --git a/mlir/lib/Dialect/SPIRV/IR/SPIRVOps.cpp b/mlir/lib/Dialect/SPIRV/IR/SPIRVOps.cpp
index 497d85f042..800f4a9003 100644
--- a/mlir/lib/Dialect/SPIRV/IR/SPIRVOps.cpp
+++ b/mlir/lib/Dialect/SPIRV/IR/SPIRVOps.cpp
@@ -978,10 +978,10 @@ LogicalResult spirv::FuncOp::verifyType() {
 
   auto hasDecorationAttr = [op = getOperation()](spirv::Decoration decoration,
                                                  unsigned i) {
-    if(auto funcOp = dyn_cast<FunctionOpInterface>(op)) {
+    if (auto funcOp = dyn_cast<FunctionOpInterface>(op)) {
       for (auto argAttr : funcOp.getArgAttrs(i))
-          if (auto decAttr = dyn_cast<spirv::DecorationAttr>(argAttr.getValue()))
-            return decAttr.getValue() == decoration;
+        if (auto decAttr = dyn_cast<spirv::DecorationAttr>(argAttr.getValue()))
+          return decAttr.getValue() == decoration;
     }
     return false;
   };

``````````

</details>


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


More information about the Mlir-commits mailing list