[Mlir-commits] [mlir] [MLIR][OpenMP] Add canonical loop LLVM-IR lowering (PR #147069)

[Michael Kruse]

Meinersbur wrote:

> @Meinersbur is it expected that _mlir/include/mlir/Target/LLVMIR/ModuleTranslation.h_ now includes OpenMP specific code, given that the remainder of the header seems to be mostly on LLVM IR? If so, could there be a missing dependency in some CMakeLists file?

`ModuleTranslation.h` always included OpenMP-specific code in form of `getOpenMPBuilder()`. This is because instead of translating the OpenMP Dialect to the LLVM Dialect, the LLVM and OpenMP Dialect are translated together to LLVM-IR. This was a comprimise reached in the past in order to reuse the OpenMPIRBuilder. The header is affected now because the `loopMapping` field is needed to remember the mapping between OpenMP Dialect operations and OpenMPIRBuilder objects.

I will add the additional dependency.

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