[Mlir-commits] [mlir] [mlir][func] Fix multiple bugs in `DuplicateFunctionElimination` (PR #109571)

llvmlistbot at llvm.org llvmlistbot at llvm.org
Sun Sep 22 01:52:23 PDT 2024


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 766ec7d50f207b7e17f23b16bdf4fca1cdc8dd23 780b3d5e7288d5990d82d897aba4332e5d1bb3ca --extensions cpp -- mlir/lib/Dialect/Func/Transforms/DuplicateFunctionElimination.cpp
``````````

</details>

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

``````````diff
diff --git a/mlir/lib/Dialect/Func/Transforms/DuplicateFunctionElimination.cpp b/mlir/lib/Dialect/Func/Transforms/DuplicateFunctionElimination.cpp
index 8dc4cdd5a5..ba754a0394 100644
--- a/mlir/lib/Dialect/Func/Transforms/DuplicateFunctionElimination.cpp
+++ b/mlir/lib/Dialect/Func/Transforms/DuplicateFunctionElimination.cpp
@@ -106,7 +106,8 @@ struct DuplicateFunctionEliminationPass
     for (auto it : toBeErased) {
       auto oldSymbol = it.getSymNameAttr();
       auto newSymbol = getRepresentant[oldSymbol].getSymNameAttr();
-      if (failed(SymbolTable::replaceAllSymbolUses(oldSymbol, newSymbol, module)))
+      if (failed(
+              SymbolTable::replaceAllSymbolUses(oldSymbol, newSymbol, module)))
         return failure();
       it.erase();
     }

``````````

</details>


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


More information about the Mlir-commits mailing list