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

[Fabian Mora via flang-commits]

fabianmcg wrote:

>  The cost model will be making a lot of enquires about the target over the life of the compiler passes, so this cache is necessary to avoid constructing it for every invocation.

Do you already have the design of how that looks? I'm not saying it's not necessary, I'm arguing that until such use is ready to land we shouldn't put it in, as we would be adding unused features, and it's easy to add later when the use is ready.

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