[PATCH] D25708: [WebAssembly] Fix for 0xc call_indirect changes

Dan Gohman via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 18 13:22:55 PDT 2016


sunfish added a comment.

Hrm, I see. My preference would be for the approach in this patch, rather than the other. It has the advantage that we handle it in one place, and don't require as much special-casing elsewhere.

As one minor nit, the CallIndirectFixup pass should be registered outside the getOptLevel() != CodeGenOpt::None test, since it's needed for correctness.


https://reviews.llvm.org/D25708





More information about the llvm-commits mailing list