[Mlir-commits] [mlir] [MLIR][DLTI] Fix access to Data Layout info in -to-llvm's type converter (PR #127239)

llvmlistbot at llvm.org llvmlistbot at llvm.org
Fri Feb 14 10:40:52 PST 2025


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 ec17161c6e87bb061daee4603ed21ab988f9d03d 2791e8e07da15290072f5191429aeec0fb538633 --extensions cpp -- mlir/lib/Conversion/ConvertToLLVM/ConvertToLLVMPass.cpp
``````````

</details>

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

``````````diff
diff --git a/mlir/lib/Conversion/ConvertToLLVM/ConvertToLLVMPass.cpp b/mlir/lib/Conversion/ConvertToLLVM/ConvertToLLVMPass.cpp
index 093ac02bb0..9448db3e9d 100644
--- a/mlir/lib/Conversion/ConvertToLLVM/ConvertToLLVMPass.cpp
+++ b/mlir/lib/Conversion/ConvertToLLVM/ConvertToLLVMPass.cpp
@@ -169,8 +169,7 @@ struct DynamicConvertToLLVM : public ConvertToLLVMPassInterface {
     target.addLegalDialect<LLVM::LLVMDialect>();
     // Get the data layout analysis.
     const auto &dlAnalysis = manager.getAnalysis<DataLayoutAnalysis>();
-    LowerToLLVMOptions options(op->getContext(),
-                               dlAnalysis.getAtOrAbove(op));
+    LowerToLLVMOptions options(op->getContext(), dlAnalysis.getAtOrAbove(op));
     LLVMTypeConverter typeConverter(context, options, &dlAnalysis);
 
     // Configure the conversion with dialect level interfaces.

``````````

</details>


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


More information about the Mlir-commits mailing list