[Mlir-commits] [mlir] [mlir][EmitC]Add a Reflection Map to a Class (PR #150572)

[Gil Rapaport]

aniragil wrote:

[Apologies if I missed some earlier community discussion on this]
This patch seems to be one in a series aimed at supporting specific MLGO features. Would be good if we could separate generic contributions that benefit most/all EmitC users (e.g. adding an `emit.class` op) from downstream-specific ones. For instance, the pass added here seems to perform a rather specific transformation and rely on existing dialect components. Could you elaborate on why it belongs upstream in MLIR core?
If you believe these patterns (reflection map, func-to-class for AoT) to be beneficial for many EmitC users, would be great if you could post an RFC on the MLIR Discourse to facilitate a wider discussion in the community.

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