[Mlir-commits] [mlir] [mlir][LLVM] Preserve !associated and !absolute_symbol on llvm.mlir.global (PR #218839)

[Tobias Gysi]

gysit wrote:

> @gysit, do you think this import path should be addressed in this patch?

That is a good point! 

We should not silently drop the attribute. However, failing the import is also problematic if that happens on regular inputs since it would be quite a regression for the users of the import.

One option could be to fail the import but make the import of `associated` metadata optional / controllable by a flag. Alternatively, we could also consider how to introduce some special handling for global_ctors and global_dtors to ensure the import of the associated metadata succeeds in all cases.


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