[PATCH] D101656: [WebAssembly] Fixup order of ins variables for table instructions

Thomas Lively via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 3 10:20:04 PDT 2021


tlively added a comment.

Oh right, this only changes the register-based printing, which is used exclusively in llc tests. The stacky assembly that would be used in production is unaffected, so it's not surprising that none of the existing assembly tests are affected.

I'm fine merging this as-is. Presumably once we have a way to generate these instructions from LLVM IR, we will also have tests for them reflecting this change.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D101656/new/

https://reviews.llvm.org/D101656



More information about the llvm-commits mailing list