[Mlir-commits] [flang] [mlir] [MLIR][LLVMIR][DLTI] Add #llvm.target, #llvm.data_layout and TargetAttrInterface (PR #145899)

[Fabian Mora]

================
@@ -14,12 +14,15 @@
 #ifndef MLIR_DIALECT_LLVMIR_LLVMATTRS_H_
 #define MLIR_DIALECT_LLVMIR_LLVMATTRS_H_
 
-#include "mlir/Dialect/LLVMIR/LLVMTypes.h"
 #include "mlir/IR/OpImplementation.h"
+#include "mlir/Interfaces/DataLayoutInterfaces.h"
 #include <optional>
 
 #include "mlir/Dialect/LLVMIR/LLVMOpsEnums.h.inc"
 
+#include "llvm/IR/DataLayout.h"
+#include "llvm/Target/TargetMachine.h"
----------------
fabianmcg wrote:

Actually those dependencies there are spurious and legacy, see https://github.com/llvm/llvm-project/pull/150692 for removal

Ideally the LLVM dialect shouldn't depend on `llvm/IR`.

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