[llvm] [mlir] [mlir][openmp] - Fix crash in OpenMPIRBuilder when converting to LLVMIR (PR #84611)

Sergio Afonso via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 12 04:50:09 PDT 2024


skatrak wrote:

My preference would be to still merge it, since it does fix some problems. We just need to be aware of its limitations and start thinking of a better way to deal with the root cause.

I think @agozillon and @jsjodin are more familiar with the reasons for the early outlining to be removed, but I seem to remember that it may have been done too early, resulting in complications when dealing with map clauses. I might be misremembering, though, so they might be able to correct me on that.

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


More information about the llvm-commits mailing list