pmatos added a comment. In D122215#3991648 <https://reviews.llvm.org/D122215#3991648>, @MoritzS wrote:
> Thanks for the patch! I just tried it out and I think this enables many > interesting use cases for WebAssembly when using C/C++. > > Currently the lowering to wasm does not work when using externrefs when > compiling without optimizations. Is that intended behavior? No - thanks for trying this out. This is definitely not intended behaviour. Somehow I probably didn't test enough without optimizations. I was expecting that the pass mem2reg would get rid of the allocas, but maybe it's not running at those optimization levels. I will have to take a closer look. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D122215/new/ https://reviews.llvm.org/D122215 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
