[llvm] [WebAssembly] Fix lowering of (extending) loads from addrspace(1) globals (PR #155937)

[Demetrius Kanios via llvm-commits]

QuantumSegfault wrote:

I would say that, yes, this is certainly a correctness bug.

So correctness in an unusual use-case wins over optimizations in a more common one... got it.

In that case, I should rebase, make sure it still works, and request a final review?

Last we left off, you were experimenting with #157627

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