[PATCH] D122215: [WebAssembly] Initial support for reference type externref in clang
Paulo Matos via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Dec 14 07:17:23 PST 2022
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
More information about the llvm-commits
mailing list