[Mlir-commits] [mlir] [MLIR][OpenMP] Simplify OpenMP to LLVM dialect conversion (PR #132009)

Sergio Afonso llvmlistbot at llvm.org
Wed Mar 19 07:35:57 PDT 2025


skatrak wrote:

> Do I understand correctly that this is not NFC because in some situations attributes were not forwarded previously but now they will be?

Thank you for the quick review! In general this will look at operands, results, attributes and entry block arguments of all operations, so it's very likely that it plugs some holes on different places. I also completed the list of operations passed to `configureOpenMPToLLVMConversionLegality` and `populateOpenMPToLLVMConversionPatterns`, since some were missing previously. So that'd be a functional change too.

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


More information about the Mlir-commits mailing list