[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