[Mlir-commits] [mlir] [MLIR][WASM] Control flow, conversion and comparison in Wasm importer (PR #154674)

[Luc Forget]

================

----------------
lforg37 wrote:

I think it is because we rely on the fact that the visibility attribute stream should be present somewhere else, and `setSymbolVisibility` remove the attribute for public symbols.

I'll investigate to see where we have this assumption to see if we can instead have follow the usual assumption that no attribute means public visibility.

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