tlively accepted this revision. tlively added a comment. This revision is now accepted and ready to land.
LGTM with that one last comment on the test. ================ Comment at: llvm/test/CodeGen/WebAssembly/ir-locals.ll:17-20 + ; The DAG combiner infers that %reloaded is the same as %arg and + ; ultimately causes "local.get 0" to be emitted instead of + ; "local.get 1". + ; CHECK-NEXT: local.get 0 ---------------- It might be good to get the value from an external function call rather than from an argument to prevent this transformation from happening. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D101140/new/ https://reviews.llvm.org/D101140 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
