[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