<div dir="ltr">When reverting a patch, please say in the commit message *why* you're reverting it, and also the SVN revision that you're reverting (SVN is our version control system of record, not git, so hashes from a git mirror are much less useful than a revision number).<div><br></div><div>This revert caused test failures because the cleanup change r339490 that was made to fix test failures from the original change was not also reverted. I've reverted that for you in r339635.</div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, 13 Aug 2018 at 16:12, Wouter van Oortmerssen via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: aardappel<br>
Date: Mon Aug 13 16:12:49 2018<br>
New Revision: 339630<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=339630&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=339630&view=rev</a><br>
Log:<br>
Revert "[WebAssembly] Added default stack-only instruction mode for MC."<br>
<br>
This reverts commit 917a99b71ce21c975be7bfbf66f4040f965d9f3c.<br>
<br>
Modified:<br>
    llvm/trunk/lib/Target/WebAssembly/AsmParser/WebAssemblyAsmParser.cpp<br>
    llvm/trunk/lib/Target/WebAssembly/WebAssemblyAsmPrinter.cpp<br>
    llvm/trunk/lib/Target/WebAssembly/WebAssemblyExplicitLocals.cpp<br>
    llvm/trunk/lib/Target/WebAssembly/WebAssemblyInstrControl.td<br>
    llvm/trunk/lib/Target/WebAssembly/WebAssemblyInstrMemory.td<br>
    llvm/trunk/lib/Target/WebAssembly/WebAssemblyTargetMachine.cpp<br>
    llvm/trunk/test/CodeGen/WebAssembly/address-offsets.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/atomic-mem-consistency.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/atomic-rmw.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/byval.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/call.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/cfg-stackify-eh.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/cfg-stackify.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/cfi.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/comparisons_f32.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/comparisons_f64.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/comparisons_i32.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/comparisons_i64.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/conv-trap.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/conv.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/copysign-casts.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/divrem-constant.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/exception.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/explicit-locals.mir<br>
    llvm/trunk/test/CodeGen/WebAssembly/f16.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/f32.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/f64.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/fast-isel-br-i1.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/fast-isel-i24.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/fast-isel-i256.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/fast-isel-noreg.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/fast-isel.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/frem.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/func.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/function-bitcasts-varargs.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/function-bitcasts.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/global.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/i128.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/i32-load-store-alignment.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/i32.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/i64-load-store-alignment.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/i64.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/ident.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/immediates.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/implicit-def.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/import-module.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/indirect-import.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/inline-asm-m.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/inline-asm.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/irreducible-cfg.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/legalize.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/libcalls.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/load-ext-atomic.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/load-ext.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/load-store-i1.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/load.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/lower-em-ehsjlj-options.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/lower-global-dtors.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/mem-intrinsics.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/memory-addr32.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/muloti4.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/negative-base-reg.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/offset-atomics.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/offset-fastisel.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/offset-folding.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/offset.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/phi.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/reg-stackify.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/return-int32.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/returned.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/select.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/signext-arg.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/signext-inreg.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/signext-zeroext.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/simd-arith.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/stack-alignment.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/store-trunc-atomic.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/store-trunc.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/store.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/switch.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/tls.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/umulo-i64.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/unsupported-function-bitcasts.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/unused-argument.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/userstack.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/varargs.ll<br>
    llvm/trunk/test/CodeGen/WebAssembly/vtable.ll<br>
    llvm/trunk/test/MC/Disassembler/WebAssembly/wasm.txt<br>
    llvm/trunk/test/MC/WebAssembly/basic-assembly.s<br>
    llvm/trunk/test/MC/WebAssembly/reloc-code.ll<br>
    llvm/trunk/test/MC/WebAssembly/weak-alias.ll<br>
    llvm/trunk/unittests/MC/Disassembler.cpp<br>
<br>
Modified: llvm/trunk/lib/Target/WebAssembly/AsmParser/WebAssemblyAsmParser.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/WebAssembly/AsmParser/WebAssemblyAsmParser.cpp?rev=339630&r1=339629&r2=339630&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/WebAssembly/AsmParser/WebAssemblyAsmParser.cpp?rev=339630&r1=339629&r2=339630&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/Target/WebAssembly/AsmParser/WebAssemblyAsmParser.cpp (original)<br>
+++ llvm/trunk/lib/Target/WebAssembly/AsmParser/WebAssemblyAsmParser.cpp Mon Aug 13 16:12:49 2018<br>
@@ -34,10 +34,27 @@ using namespace llvm;<br>
<br>
 namespace {<br>
<br>
+// We store register types as SimpleValueType to retain SIMD layout<br>
+// information, but must also be able to supply them as the (unnamed)<br>
+// register enum from WebAssemblyRegisterInfo.td/.inc.<br>
+static unsigned MVTToWasmReg(MVT::SimpleValueType Type) {<br>
+  switch(Type) {<br>
+    case MVT::i32: return WebAssembly::I32_0;<br>
+    case MVT::i64: return WebAssembly::I64_0;<br>
+    case MVT::f32: return WebAssembly::F32_0;<br>
+    case MVT::f64: return WebAssembly::F64_0;<br>
+    case MVT::v16i8: return WebAssembly::V128_0;<br>
+    case MVT::v8i16: return WebAssembly::V128_0;<br>
+    case MVT::v4i32: return WebAssembly::V128_0;<br>
+    case MVT::v4f32: return WebAssembly::V128_0;<br>
+    default: return MVT::INVALID_SIMPLE_VALUE_TYPE;<br>
+  }<br>
+}<br>
+<br>
 /// WebAssemblyOperand - Instances of this class represent the operands in a<br>
 /// parsed WASM machine instruction.<br>
 struct WebAssemblyOperand : public MCParsedAsmOperand {<br>
-  enum KindTy { Token, Integer, Float, Symbol } Kind;<br>
+  enum KindTy { Token, Local, Stack, Integer, Float, Symbol } Kind;<br>
<br>
   SMLoc StartLoc, EndLoc;<br>
<br>
@@ -45,6 +62,19 @@ struct WebAssemblyOperand : public MCPar<br>
     StringRef Tok;<br>
   };<br>
<br>
+  struct RegOp {<br>
+    // This is a (virtual) local or stack register represented as 0..<br>
+    unsigned RegNo;<br>
+    // In most targets, the register number also encodes the type, but for<br>
+    // wasm we have to track that seperately since we have an unbounded<br>
+    // number of registers.<br>
+    // This has the unfortunate side effect that we supply a different value<br>
+    // to the table-gen matcher at different times in the process (when it<br>
+    // calls getReg() or addRegOperands().<br>
+    // TODO: While this works, it feels brittle. and would be nice to clean up.<br>
+    MVT::SimpleValueType Type;<br>
+  };<br>
+<br>
   struct IntOp {<br>
     int64_t Val;<br>
   };<br>
@@ -59,6 +89,7 @@ struct WebAssemblyOperand : public MCPar<br>
<br>
   union {<br>
     struct TokOp Tok;<br>
+    struct RegOp Reg;<br>
     struct IntOp Int;<br>
     struct FltOp Flt;<br>
     struct SymOp Sym;<br>
@@ -66,6 +97,8 @@ struct WebAssemblyOperand : public MCPar<br>
<br>
   WebAssemblyOperand(KindTy K, SMLoc Start, SMLoc End, TokOp T)<br>
     : Kind(K), StartLoc(Start), EndLoc(End), Tok(T) {}<br>
+  WebAssemblyOperand(KindTy K, SMLoc Start, SMLoc End, RegOp R)<br>
+    : Kind(K), StartLoc(Start), EndLoc(End), Reg(R) {}<br>
   WebAssemblyOperand(KindTy K, SMLoc Start, SMLoc End, IntOp I)<br>
     : Kind(K), StartLoc(Start), EndLoc(End), Int(I) {}<br>
   WebAssemblyOperand(KindTy K, SMLoc Start, SMLoc End, FltOp F)<br>
@@ -77,12 +110,14 @@ struct WebAssemblyOperand : public MCPar<br>
   bool isImm() const override { return Kind == Integer ||<br>
                                        Kind == Float ||<br>
                                        Kind == Symbol; }<br>
+  bool isReg() const override { return Kind == Local || Kind == Stack; }<br>
   bool isMem() const override { return false; }<br>
-  bool isReg() const override { return false; }<br>
<br>
   unsigned getReg() const override {<br>
-    llvm_unreachable("Assembly inspects a register operand");<br>
-    return 0;<br>
+    assert(isReg());<br>
+    // This is called from the tablegen matcher (MatchInstructionImpl)<br>
+    // where it expects to match the type of register, see RegOp above.<br>
+    return MVTToWasmReg(Reg.Type);<br>
   }<br>
<br>
   StringRef getToken() const {<br>
@@ -93,9 +128,19 @@ struct WebAssemblyOperand : public MCPar<br>
   SMLoc getStartLoc() const override { return StartLoc; }<br>
   SMLoc getEndLoc() const override { return EndLoc; }<br>
<br>
-  void addRegOperands(MCInst &, unsigned) const {<br>
-    // Required by the assembly matcher.<br>
-    llvm_unreachable("Assembly matcher creates register operands");<br>
+  void addRegOperands(MCInst &Inst, unsigned N) const {<br>
+    assert(N == 1 && "Invalid number of operands!");<br>
+    assert(isReg() && "Not a register operand!");<br>
+    // This is called from the tablegen matcher (MatchInstructionImpl)<br>
+    // where it expects to output the actual register index, see RegOp above.<br>
+    unsigned R = Reg.RegNo;<br>
+    if (Kind == Stack) {<br>
+      // A stack register is represented as a large negative number.<br>
+      // See WebAssemblyRegNumbering::runOnMachineFunction and<br>
+      // getWARegStackId for why this | is needed.<br>
+      R |= INT32_MIN;<br>
+    }<br>
+    Inst.addOperand(MCOperand::createReg(R));<br>
   }<br>
<br>
   void addImmOperands(MCInst &Inst, unsigned N) const {<br>
@@ -115,6 +160,12 @@ struct WebAssemblyOperand : public MCPar<br>
     case Token:<br>
       OS << "Tok:" << Tok.Tok;<br>
       break;<br>
+    case Local:<br>
+      OS << "Loc:" << Reg.RegNo << ":" << static_cast<int>(Reg.Type);<br>
+      break;<br>
+    case Stack:<br>
+      OS << "Stk:" << Reg.RegNo << ":" << static_cast<int>(Reg.Type);<br>
+      break;<br>
     case Integer:<br>
       OS << "Int:" << Int.Val;<br>
       break;<br>
@@ -131,6 +182,11 @@ struct WebAssemblyOperand : public MCPar<br>
 class WebAssemblyAsmParser final : public MCTargetAsmParser {<br>
   MCAsmParser &Parser;<br>
   MCAsmLexer &Lexer;<br>
+  // These are for the current function being parsed:<br>
+  // These are vectors since register assignments are so far non-sparse.<br>
+  // Replace by map if necessary.<br>
+  std::vector<MVT::SimpleValueType> LocalTypes;<br>
+  std::vector<MVT::SimpleValueType> StackTypes;<br>
   MCSymbol *LastLabel;<br>
<br>
 public:<br>
@@ -180,6 +236,68 @@ public:<br>
         .Default(MVT::INVALID_SIMPLE_VALUE_TYPE);<br>
   }<br>
<br>
+  MVT::SimpleValueType &GetType(<br>
+      std::vector<MVT::SimpleValueType> &Types, size_t i) {<br>
+    Types.resize(std::max(i + 1, Types.size()), MVT::INVALID_SIMPLE_VALUE_TYPE);<br>
+    return Types[i];<br>
+  }<br>
+<br>
+  bool ParseReg(OperandVector &Operands, StringRef TypePrefix) {<br>
+    if (Lexer.is(AsmToken::Integer)) {<br>
+      auto &Local = Lexer.getTok();<br>
+      // This is a reference to a local, turn it into a virtual register.<br>
+      auto LocalNo = static_cast<unsigned>(Local.getIntVal());<br>
+      Operands.push_back(make_unique<WebAssemblyOperand>(<br>
+                           WebAssemblyOperand::Local, Local.getLoc(),<br>
+                           Local.getEndLoc(),<br>
+                           WebAssemblyOperand::RegOp{LocalNo,<br>
+                               GetType(LocalTypes, LocalNo)}));<br>
+      Parser.Lex();<br>
+    } else if (Lexer.is(AsmToken::Identifier)) {<br>
+      auto &StackRegTok = Lexer.getTok();<br>
+      // These are push/pop/drop pseudo stack registers, which we turn<br>
+      // into virtual registers also. The stackify pass will later turn them<br>
+      // back into implicit stack references if possible.<br>
+      auto StackReg = StackRegTok.getString();<br>
+      auto StackOp = StackReg.take_while([](char c) { return isalpha(c); });<br>
+      auto Reg = StackReg.drop_front(StackOp.size());<br>
+      unsigned long long ParsedRegNo = 0;<br>
+      if (!Reg.empty() && getAsUnsignedInteger(Reg, 10, ParsedRegNo))<br>
+        return Error("Cannot parse stack register index: ", StackRegTok);<br>
+      unsigned RegNo = static_cast<unsigned>(ParsedRegNo);<br>
+      if (StackOp == "push") {<br>
+        // This defines a result, record register type.<br>
+        auto RegType = ParseRegType(TypePrefix);<br>
+        GetType(StackTypes, RegNo) = RegType;<br>
+        Operands.push_back(make_unique<WebAssemblyOperand>(<br>
+                             WebAssemblyOperand::Stack,<br>
+                             StackRegTok.getLoc(),<br>
+                             StackRegTok.getEndLoc(),<br>
+                             WebAssemblyOperand::RegOp{RegNo, RegType}));<br>
+      } else if (StackOp == "pop") {<br>
+        // This uses a previously defined stack value.<br>
+        auto RegType = GetType(StackTypes, RegNo);<br>
+        Operands.push_back(make_unique<WebAssemblyOperand>(<br>
+                             WebAssemblyOperand::Stack,<br>
+                             StackRegTok.getLoc(),<br>
+                             StackRegTok.getEndLoc(),<br>
+                             WebAssemblyOperand::RegOp{RegNo, RegType}));<br>
+      } else if (StackOp == "drop") {<br>
+        // This operand will be dropped, since it is part of an instruction<br>
+        // whose result is void.<br>
+      } else {<br>
+        return Error("Unknown stack register prefix: ", StackRegTok);<br>
+      }<br>
+      Parser.Lex();<br>
+    } else {<br>
+      return Error(<br>
+            "Expected identifier/integer following $, instead got: ",<br>
+            Lexer.getTok());<br>
+    }<br>
+    IsNext(AsmToken::Equal);<br>
+    return false;<br>
+  }<br>
+<br>
   void ParseSingleInteger(bool IsNegative, OperandVector &Operands) {<br>
     auto &Int = Lexer.getTok();<br>
     int64_t Val = Int.getIntVal();<br>
@@ -192,26 +310,36 @@ public:<br>
<br>
   bool ParseOperandStartingWithInteger(bool IsNegative,<br>
                                        OperandVector &Operands,<br>
-                                       StringRef InstName) {<br>
+                                       StringRef InstType) {<br>
     ParseSingleInteger(IsNegative, Operands);<br>
-    // FIXME: there is probably a cleaner way to do this.<br>
-    auto IsLoadStore = InstName.startswith("load") ||<br>
-                       InstName.startswith("store") ||<br>
-                       InstName.startswith("atomic_load") ||<br>
-                       InstName.startswith("atomic_store");<br>
-    if (IsLoadStore) {<br>
-      // Parse load/store operands of the form: offset align<br>
-      auto &Offset = Lexer.getTok();<br>
-      if (Offset.is(AsmToken::Integer)) {<br>
+    if (Lexer.is(AsmToken::LParen)) {<br>
+      // Parse load/store operands of the form: offset($reg)align<br>
+      auto &LParen = Lexer.getTok();<br>
+      Operands.push_back(<br>
+            make_unique<WebAssemblyOperand>(WebAssemblyOperand::Token,<br>
+                                            LParen.getLoc(),<br>
+                                            LParen.getEndLoc(),<br>
+                                            WebAssemblyOperand::TokOp{<br>
+                                              LParen.getString()}));<br>
+      Parser.Lex();<br>
+      if (Expect(AsmToken::Dollar, "register")) return true;<br>
+      if (ParseReg(Operands, InstType)) return true;<br>
+      auto &RParen = Lexer.getTok();<br>
+      Operands.push_back(<br>
+            make_unique<WebAssemblyOperand>(WebAssemblyOperand::Token,<br>
+                                            RParen.getLoc(),<br>
+                                            RParen.getEndLoc(),<br>
+                                            WebAssemblyOperand::TokOp{<br>
+                                              RParen.getString()}));<br>
+      if (Expect(AsmToken::RParen, ")")) return true;<br>
+      if (Lexer.is(AsmToken::Integer)) {<br>
         ParseSingleInteger(false, Operands);<br>
       } else {<br>
         // Alignment not specified.<br>
         // FIXME: correctly derive a default from the instruction.<br>
-        // We can't just call WebAssembly::GetDefaultP2Align since we don't have<br>
-        // an opcode until after the assembly matcher.<br>
         Operands.push_back(make_unique<WebAssemblyOperand>(<br>
-                             WebAssemblyOperand::Integer, Offset.getLoc(),<br>
-                             Offset.getEndLoc(), WebAssemblyOperand::IntOp{0}));<br>
+                             WebAssemblyOperand::Integer, RParen.getLoc(),<br>
+                             RParen.getEndLoc(), WebAssemblyOperand::IntOp{0}));<br>
       }<br>
     }<br>
     return false;<br>
@@ -232,6 +360,11 @@ public:<br>
     while (Lexer.isNot(AsmToken::EndOfStatement)) {<br>
       auto &Tok = Lexer.getTok();<br>
       switch (Tok.getKind()) {<br>
+      case AsmToken::Dollar: {<br>
+        Parser.Lex();<br>
+        if (ParseReg(Operands, NamePair.first)) return true;<br>
+        break;<br>
+      }<br>
       case AsmToken::Identifier: {<br>
         auto &Id = Lexer.getTok();<br>
         const MCExpr *Val;<br>
@@ -247,11 +380,11 @@ public:<br>
         Parser.Lex();<br>
         if (Lexer.isNot(AsmToken::Integer))<br>
           return Error("Expected integer instead got: ", Lexer.getTok());<br>
-        if (ParseOperandStartingWithInteger(true, Operands, NamePair.second))<br>
+        if (ParseOperandStartingWithInteger(true, Operands, NamePair.first))<br>
           return true;<br>
         break;<br>
       case AsmToken::Integer:<br>
-        if (ParseOperandStartingWithInteger(false, Operands, NamePair.second))<br>
+        if (ParseOperandStartingWithInteger(false, Operands, NamePair.first))<br>
           return true;<br>
         break;<br>
       case AsmToken::Real: {<br>
@@ -272,6 +405,35 @@ public:<br>
       }<br>
     }<br>
     Parser.Lex();<br>
+    // Call instructions are vararg, but the tablegen matcher doesn't seem to<br>
+    // support that, so for now we strip these extra operands.<br>
+    // This is problematic if these arguments are not simple $pop stack<br>
+    // registers, since e.g. a local register would get lost, so we check for<br>
+    // this. This can be the case when using -disable-wasm-explicit-locals<br>
+    // which currently s2wasm requires.<br>
+    // TODO: Instead, we can move this code to MatchAndEmitInstruction below and<br>
+    // actually generate get_local instructions on the fly.<br>
+    // Or even better, improve the matcher to support vararg?<br>
+    auto IsIndirect = NamePair.second == "call_indirect";<br>
+    if (IsIndirect || NamePair.second == "call") {<br>
+      // Figure out number of fixed operands from the instruction.<br>
+      size_t CallOperands = 1;  // The name token.<br>
+      if (!IsIndirect) CallOperands++;  // The function index.<br>
+      if (!NamePair.first.empty()) CallOperands++;  // The result register.<br>
+      if (Operands.size() > CallOperands) {<br>
+        // Ensure operands we drop are all $pop.<br>
+        for (size_t I = CallOperands; I < Operands.size(); I++) {<br>
+          auto Operand =<br>
+              reinterpret_cast<WebAssemblyOperand *>(Operands[I].get());<br>
+          if (Operand->Kind != WebAssemblyOperand::Stack)<br>
+            Parser.Error(NameLoc,<br>
+              "Call instruction has non-stack arguments, if this code was "<br>
+              "generated with -disable-wasm-explicit-locals please remove it");<br>
+        }<br>
+        // Drop unneeded operands.<br>
+        Operands.resize(CallOperands);<br>
+      }<br>
+    }<br>
     // Block instructions require a signature index, but these are missing in<br>
     // assembly, so we add a dummy one explicitly (since we have no control<br>
     // over signature tables here, we assume these will be regenerated when<br>
@@ -281,6 +443,17 @@ public:<br>
                            WebAssemblyOperand::Integer, NameLoc,<br>
                            NameLoc, WebAssemblyOperand::IntOp{-1}));<br>
     }<br>
+    // These don't specify the type, which has to derived from the local index.<br>
+    if (NamePair.second == "get_local" || NamePair.second == "tee_local") {<br>
+      if (Operands.size() >= 3 && Operands[1]->isReg() &&<br>
+          Operands[2]->isImm()) {<br>
+        auto Op1 = reinterpret_cast<WebAssemblyOperand *>(Operands[1].get());<br>
+        auto Op2 = reinterpret_cast<WebAssemblyOperand *>(Operands[2].get());<br>
+        auto Type = GetType(LocalTypes, static_cast<size_t>(Op2->Int.Val));<br>
+        Op1->Reg.Type = Type;<br>
+        GetType(StackTypes, Op1->Reg.RegNo) = Type;<br>
+      }<br>
+    }<br>
     return false;<br>
   }<br>
<br>
@@ -304,6 +477,11 @@ public:<br>
             IsNext(AsmToken::At) &&<br>
             Lexer.is(AsmToken::Identifier)))<br>
         return Error("Expected label,@type declaration, got: ", Lexer.getTok());<br>
+      if (Lexer.getTok().getString() == "function") {<br>
+        // Track locals from start of function.<br>
+        LocalTypes.clear();<br>
+        StackTypes.clear();<br>
+      }<br>
       Parser.Lex();<br>
       //Out.EmitSymbolAttribute(??, MCSA_ELF_TypeFunction);<br>
     } else if (DirectiveID.getString() == ".param" ||<br>
@@ -316,6 +494,7 @@ public:<br>
       while (Lexer.is(AsmToken::Identifier)) {<br>
         auto RegType = ParseRegType(Lexer.getTok().getString());<br>
         if (RegType == MVT::INVALID_SIMPLE_VALUE_TYPE) return true;<br>
+        LocalTypes.push_back(RegType);<br>
         if (DirectiveID.getString() == ".param") {<br>
           Params.push_back(RegType);<br>
         } else {<br>
<br>
Modified: llvm/trunk/lib/Target/WebAssembly/WebAssemblyAsmPrinter.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/WebAssembly/WebAssemblyAsmPrinter.cpp?rev=339630&r1=339629&r2=339630&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/WebAssembly/WebAssemblyAsmPrinter.cpp?rev=339630&r1=339629&r2=339630&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/Target/WebAssembly/WebAssemblyAsmPrinter.cpp (original)<br>
+++ llvm/trunk/lib/Target/WebAssembly/WebAssemblyAsmPrinter.cpp Mon Aug 13 16:12:49 2018<br>
@@ -169,62 +169,45 @@ void WebAssemblyAsmPrinter::EmitInstruct<br>
<br>
   switch (MI->getOpcode()) {<br>
   case WebAssembly::ARGUMENT_I32:<br>
-  case WebAssembly::ARGUMENT_I32_S:<br>
   case WebAssembly::ARGUMENT_I64:<br>
-  case WebAssembly::ARGUMENT_I64_S:<br>
   case WebAssembly::ARGUMENT_F32:<br>
-  case WebAssembly::ARGUMENT_F32_S:<br>
   case WebAssembly::ARGUMENT_F64:<br>
-  case WebAssembly::ARGUMENT_F64_S:<br>
   case WebAssembly::ARGUMENT_v16i8:<br>
-  case WebAssembly::ARGUMENT_v16i8_S:<br>
   case WebAssembly::ARGUMENT_v8i16:<br>
-  case WebAssembly::ARGUMENT_v8i16_S:<br>
   case WebAssembly::ARGUMENT_v4i32:<br>
-  case WebAssembly::ARGUMENT_v4i32_S:<br>
   case WebAssembly::ARGUMENT_v2i64:<br>
-  case WebAssembly::ARGUMENT_v2i64_S:<br>
   case WebAssembly::ARGUMENT_v4f32:<br>
-  case WebAssembly::ARGUMENT_v4f32_S:<br>
   case WebAssembly::ARGUMENT_v2f64:<br>
-  case WebAssembly::ARGUMENT_v2f64_S:<br>
     // These represent values which are live into the function entry, so there's<br>
     // no instruction to emit.<br>
     break;<br>
   case WebAssembly::FALLTHROUGH_RETURN_I32:<br>
-  case WebAssembly::FALLTHROUGH_RETURN_I32_S:<br>
   case WebAssembly::FALLTHROUGH_RETURN_I64:<br>
-  case WebAssembly::FALLTHROUGH_RETURN_I64_S:<br>
   case WebAssembly::FALLTHROUGH_RETURN_F32:<br>
-  case WebAssembly::FALLTHROUGH_RETURN_F32_S:<br>
   case WebAssembly::FALLTHROUGH_RETURN_F64:<br>
-  case WebAssembly::FALLTHROUGH_RETURN_F64_S:<br>
   case WebAssembly::FALLTHROUGH_RETURN_v16i8:<br>
-  case WebAssembly::FALLTHROUGH_RETURN_v16i8_S:<br>
   case WebAssembly::FALLTHROUGH_RETURN_v8i16:<br>
-  case WebAssembly::FALLTHROUGH_RETURN_v8i16_S:<br>
   case WebAssembly::FALLTHROUGH_RETURN_v4i32:<br>
-  case WebAssembly::FALLTHROUGH_RETURN_v4i32_S:<br>
   case WebAssembly::FALLTHROUGH_RETURN_v2i64:<br>
-  case WebAssembly::FALLTHROUGH_RETURN_v2i64_S:<br>
   case WebAssembly::FALLTHROUGH_RETURN_v4f32:<br>
-  case WebAssembly::FALLTHROUGH_RETURN_v4f32_S:<br>
-  case WebAssembly::FALLTHROUGH_RETURN_v2f64:<br>
-  case WebAssembly::FALLTHROUGH_RETURN_v2f64_S: {<br>
+  case WebAssembly::FALLTHROUGH_RETURN_v2f64: {<br>
     // These instructions represent the implicit return at the end of a<br>
-    // function body. Always pops one value off the stack.<br>
+    // function body. The operand is always a pop.<br>
+    assert(MFI->isVRegStackified(MI->getOperand(0).getReg()));<br>
+<br>
     if (isVerbose()) {<br>
-      OutStreamer->AddComment("fallthrough-return-value");<br>
+      OutStreamer->AddComment("fallthrough-return: $pop" +<br>
+                              Twine(MFI->getWARegStackId(<br>
+                                  MFI->getWAReg(MI->getOperand(0).getReg()))));<br>
       OutStreamer->AddBlankLine();<br>
     }<br>
     break;<br>
   }<br>
   case WebAssembly::FALLTHROUGH_RETURN_VOID:<br>
-  case WebAssembly::FALLTHROUGH_RETURN_VOID_S:<br>
     // This instruction represents the implicit return at the end of a<br>
     // function body with no return value.<br>
     if (isVerbose()) {<br>
-      OutStreamer->AddComment("fallthrough-return-void");<br>
+      OutStreamer->AddComment("fallthrough-return");<br>
       OutStreamer->AddBlankLine();<br>
     }<br>
     break;<br>
@@ -265,9 +248,6 @@ bool WebAssemblyAsmPrinter::PrintAsmOper<br>
       OS << MO.getImm();<br>
       return false;<br>
     case MachineOperand::MO_Register:<br>
-      // FIXME: only opcode that still contains registers, as required by<br>
-      // MachineInstr::getDebugVariable().<br>
-      assert(MI->getOpcode() == WebAssembly::INLINEASM);<br>
       OS << regToString(MO);<br>
       return false;<br>
     case MachineOperand::MO_GlobalAddress:<br>
<br>
Modified: llvm/trunk/lib/Target/WebAssembly/WebAssemblyExplicitLocals.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/WebAssembly/WebAssemblyExplicitLocals.cpp?rev=339630&r1=339629&r2=339630&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/WebAssembly/WebAssemblyExplicitLocals.cpp?rev=339630&r1=339629&r2=339630&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/Target/WebAssembly/WebAssemblyExplicitLocals.cpp (original)<br>
+++ llvm/trunk/lib/Target/WebAssembly/WebAssemblyExplicitLocals.cpp Mon Aug 13 16:12:49 2018<br>
@@ -31,21 +31,12 @@ using namespace llvm;<br>
<br>
 #define DEBUG_TYPE "wasm-explicit-locals"<br>
<br>
-// A command-line option to disable this pass, and keep implicit locals and<br>
-// stackified registers for the purpose of testing with lit/llc ONLY.<br>
-// This produces output which is not valid WebAssembly, and is not supported<br>
-// by assemblers/disassemblers and other MC based tools.<br>
-static cl::opt<bool> RegisterCodeGenTestMode(<br>
-    "wasm-register-codegen-test-mode", cl::Hidden,<br>
-    cl::desc("WebAssembly: output stack registers and implicit locals in"<br>
-             " instruction output for test purposes only."),<br>
-    cl::init(false));<br>
-// This one does explicit locals but keeps stackified registers, as required<br>
-// by some current tests.<br>
-static cl::opt<bool> ExplicitLocalsCodeGenTestMode(<br>
-    "wasm-explicit-locals-codegen-test-mode", cl::Hidden,<br>
-    cl::desc("WebAssembly: output stack registers and explicit locals in"<br>
-             " instruction output for test purposes only."),<br>
+// A command-line option to disable this pass. Note that this produces output<br>
+// which is not valid WebAssembly, though it may be more convenient for writing<br>
+// LLVM unit tests with.<br>
+static cl::opt<bool> DisableWebAssemblyExplicitLocals(<br>
+    "disable-wasm-explicit-locals", cl::ReallyHidden,<br>
+    cl::desc("WebAssembly: Disable emission of get_local/set_local."),<br>
     cl::init(false));<br>
<br>
 namespace {<br>
@@ -68,8 +59,6 @@ public:<br>
 };<br>
 } // end anonymous namespace<br>
<br>
-unsigned regInstructionToStackInstruction(unsigned OpCode);<br>
-<br>
 char WebAssemblyExplicitLocals::ID = 0;<br>
 INITIALIZE_PASS(WebAssemblyExplicitLocals, DEBUG_TYPE,<br>
                 "Convert registers to WebAssembly locals", false, false)<br>
@@ -173,7 +162,7 @@ static MVT typeForRegClass(const TargetR<br>
<br>
 /// Given a MachineOperand of a stackified vreg, return the instruction at the<br>
 /// start of the expression tree.<br>
-static MachineInstr *findStartOfTree(MachineOperand &MO,<br>
+static MachineInstr *FindStartOfTree(MachineOperand &MO,<br>
                                      MachineRegisterInfo &MRI,<br>
                                      WebAssemblyFunctionInfo &MFI) {<br>
   unsigned Reg = MO.getReg();<br>
@@ -184,7 +173,7 @@ static MachineInstr *findStartOfTree(Mac<br>
   for (MachineOperand &DefMO : Def->explicit_uses()) {<br>
     if (!DefMO.isReg())<br>
       continue;<br>
-    return findStartOfTree(DefMO, MRI, MFI);<br>
+    return FindStartOfTree(DefMO, MRI, MFI);<br>
   }<br>
<br>
   // If there were no stackified uses, we've reached the start.<br>
@@ -197,7 +186,7 @@ bool WebAssemblyExplicitLocals::runOnMac<br>
                     << MF.getName() << '\n');<br>
<br>
   // Disable this pass if directed to do so.<br>
-  if (RegisterCodeGenTestMode)<br>
+  if (DisableWebAssemblyExplicitLocals)<br>
     return false;<br>
<br>
   bool Changed = false;<br>
@@ -217,19 +206,19 @@ bool WebAssemblyExplicitLocals::runOnMac<br>
       break;<br>
     unsigned Reg = MI.getOperand(0).getReg();<br>
     assert(!MFI.isVRegStackified(Reg));<br>
-    Reg2Local[Reg] = static_cast<unsigned>(MI.getOperand(1).getImm());<br>
+    Reg2Local[Reg] = MI.getOperand(1).getImm();<br>
     MI.eraseFromParent();<br>
     Changed = true;<br>
   }<br>
<br>
   // Start assigning local numbers after the last parameter.<br>
-  unsigned CurLocal = static_cast<unsigned>(MFI.getParams().size());<br>
+  unsigned CurLocal = MFI.getParams().size();<br>
<br>
   // Precompute the set of registers that are unused, so that we can insert<br>
   // drops to their defs.<br>
   BitVector UseEmpty(MRI.getNumVirtRegs());<br>
-  for (unsigned I = 0, E = MRI.getNumVirtRegs(); I < E; ++I)<br>
-    UseEmpty[I] = MRI.use_empty(TargetRegisterInfo::index2VirtReg(I));<br>
+  for (unsigned i = 0, e = MRI.getNumVirtRegs(); i < e; ++i)<br>
+    UseEmpty[i] = MRI.use_empty(TargetRegisterInfo::index2VirtReg(i));<br>
<br>
   // Visit each instruction in the function.<br>
   for (MachineBasicBlock &MBB : MF) {<br>
@@ -333,7 +322,7 @@ bool WebAssemblyExplicitLocals::runOnMac<br>
         // If we see a stackified register, prepare to insert subsequent<br>
         // get_locals before the start of its tree.<br>
         if (MFI.isVRegStackified(OldReg)) {<br>
-          InsertPt = findStartOfTree(MO, MRI, MFI);<br>
+          InsertPt = FindStartOfTree(MO, MRI, MFI);<br>
           continue;<br>
         }<br>
<br>
@@ -367,414 +356,37 @@ bool WebAssemblyExplicitLocals::runOnMac<br>
         Changed = true;<br>
       }<br>
     }<br>
-<br>
-    if (!ExplicitLocalsCodeGenTestMode) {<br>
-      // Remove all uses of stackified registers to bring the instruction format<br>
-      // into its final stack form, and transition opcodes to their _S variant.<br>
-      // We do this in a seperate loop, since the previous loop adds/removes<br>
-      // instructions.<br>
-      // See comments in lib/Target/WebAssembly/WebAssemblyInstrFormats.td for<br>
-      // details.<br>
-      // TODO: the code above creates new registers which are then removed here.<br>
-      // That code could be slightly simplified by not doing that, though maybe<br>
-      // it is simpler conceptually to keep the code above in "register mode"<br>
-      // until this transition point.<br>
-      for (MachineBasicBlock::iterator I = MBB.begin(), E = MBB.end();<br>
-           I != E;) {<br>
-        MachineInstr &MI = *I++;<br>
-        // FIXME: we are not processing inline assembly, which contains register<br>
-        // operands, because it is used by later target generic code.<br>
-        if (MI.isDebugInstr() || MI.isLabel() || MI.isInlineAsm())<br>
-          continue;<br>
-        auto RegOpcode = MI.getOpcode();<br>
-        auto StackOpcode = regInstructionToStackInstruction(RegOpcode);<br>
-        MI.setDesc(TII->get(StackOpcode));<br>
-        // Now remove all register operands.<br>
-        for (auto I = MI.getNumOperands(); I; --I) {<br>
-          auto &MO = MI.getOperand(I - 1);<br>
-          if (MO.isReg()) {<br>
-            MI.RemoveOperand(I - 1);<br>
-            // TODO: we should also update the MFI here or below to reflect the<br>
-            // removed registers? The MFI is about to be deleted anyway, so<br>
-            // maybe that is not worth it?<br>
-          }<br>
-        }<br>
-      }<br>
-    }<br>
   }<br>
<br>
   // Define the locals.<br>
   // TODO: Sort the locals for better compression.<br>
   MFI.setNumLocals(CurLocal - MFI.getParams().size());<br>
-  for (unsigned I = 0, E = MRI.getNumVirtRegs(); I < E; ++I) {<br>
-    unsigned Reg = TargetRegisterInfo::index2VirtReg(I);<br>
-    auto RL = Reg2Local.find(Reg);<br>
-    if (RL == Reg2Local.end() || RL->second < MFI.getParams().size())<br>
+  for (size_t i = 0, e = MRI.getNumVirtRegs(); i < e; ++i) {<br>
+    unsigned Reg = TargetRegisterInfo::index2VirtReg(i);<br>
+    auto I = Reg2Local.find(Reg);<br>
+    if (I == Reg2Local.end() || I->second < MFI.getParams().size())<br>
       continue;<br>
<br>
-    MFI.setLocal(RL->second - MFI.getParams().size(),<br>
+    MFI.setLocal(I->second - MFI.getParams().size(),<br>
                  typeForRegClass(MRI.getRegClass(Reg)));<br>
     Changed = true;<br>
   }<br>
<br>
-  return Changed;<br>
-}<br>
-<br>
-unsigned regInstructionToStackInstruction(unsigned OpCode) {<br>
-  switch (OpCode) {<br>
-  default:<br>
-    // You may hit this if you add new instructions, please add them below.<br>
-    // For most of these opcodes, this function could have been implemented<br>
-    // as "return OpCode + 1", but since table-gen alphabetically sorts them,<br>
-    // this cannot be guaranteed (see e.g. BR and BR_IF).<br>
-    // The approach below is the same as what the x87 backend does.<br>
-    // TODO(wvo): to make this code cleaner, create a custom tablegen<br>
-    // code generator that emits the table below automatically.<br>
-    llvm_unreachable(<br>
-          "unknown WebAssembly instruction in Explicit Locals pass");<br>
-  case WebAssembly::ABS_F32: return WebAssembly::ABS_F32_S;<br>
-  case WebAssembly::ABS_F64: return WebAssembly::ABS_F64_S;<br>
-  case WebAssembly::ADD_F32: return WebAssembly::ADD_F32_S;<br>
-  case WebAssembly::ADD_F32x4: return WebAssembly::ADD_F32x4_S;<br>
-  case WebAssembly::ADD_F64: return WebAssembly::ADD_F64_S;<br>
-  case WebAssembly::ADD_I16x8: return WebAssembly::ADD_I16x8_S;<br>
-  case WebAssembly::ADD_I32: return WebAssembly::ADD_I32_S;<br>
-  case WebAssembly::ADD_I32x4: return WebAssembly::ADD_I32x4_S;<br>
-  case WebAssembly::ADD_I64: return WebAssembly::ADD_I64_S;<br>
-  case WebAssembly::ADD_I8x16: return WebAssembly::ADD_I8x16_S;<br>
-  case WebAssembly::ADJCALLSTACKDOWN: return WebAssembly::ADJCALLSTACKDOWN_S;<br>
-  case WebAssembly::ADJCALLSTACKUP: return WebAssembly::ADJCALLSTACKUP_S;<br>
-  case WebAssembly::AND_I32: return WebAssembly::AND_I32_S;<br>
-  case WebAssembly::AND_I64: return WebAssembly::AND_I64_S;<br>
-  case WebAssembly::ARGUMENT_EXCEPT_REF: return WebAssembly::ARGUMENT_EXCEPT_REF_S;<br>
-  case WebAssembly::ARGUMENT_F32: return WebAssembly::ARGUMENT_F32_S;<br>
-  case WebAssembly::ARGUMENT_F64: return WebAssembly::ARGUMENT_F64_S;<br>
-  case WebAssembly::ARGUMENT_I32: return WebAssembly::ARGUMENT_I32_S;<br>
-  case WebAssembly::ARGUMENT_I64: return WebAssembly::ARGUMENT_I64_S;<br>
-  case WebAssembly::ARGUMENT_v16i8: return WebAssembly::ARGUMENT_v16i8_S;<br>
-  case WebAssembly::ARGUMENT_v4f32: return WebAssembly::ARGUMENT_v4f32_S;<br>
-  case WebAssembly::ARGUMENT_v4i32: return WebAssembly::ARGUMENT_v4i32_S;<br>
-  case WebAssembly::ARGUMENT_v8i16: return WebAssembly::ARGUMENT_v8i16_S;<br>
-  case WebAssembly::ARGUMENT_v2f64: return WebAssembly::ARGUMENT_v2f64_S;<br>
-  case WebAssembly::ARGUMENT_v2i64: return WebAssembly::ARGUMENT_v2i64_S;<br>
-  case WebAssembly::ATOMIC_LOAD16_U_I32: return WebAssembly::ATOMIC_LOAD16_U_I32_S;<br>
-  case WebAssembly::ATOMIC_LOAD16_U_I64: return WebAssembly::ATOMIC_LOAD16_U_I64_S;<br>
-  case WebAssembly::ATOMIC_LOAD32_U_I64: return WebAssembly::ATOMIC_LOAD32_U_I64_S;<br>
-  case WebAssembly::ATOMIC_LOAD8_U_I32: return WebAssembly::ATOMIC_LOAD8_U_I32_S;<br>
-  case WebAssembly::ATOMIC_LOAD8_U_I64: return WebAssembly::ATOMIC_LOAD8_U_I64_S;<br>
-  case WebAssembly::ATOMIC_LOAD_I32: return WebAssembly::ATOMIC_LOAD_I32_S;<br>
-  case WebAssembly::ATOMIC_LOAD_I64: return WebAssembly::ATOMIC_LOAD_I64_S;<br>
-  case WebAssembly::ATOMIC_STORE16_I32: return WebAssembly::ATOMIC_STORE16_I32_S;<br>
-  case WebAssembly::ATOMIC_STORE16_I64: return WebAssembly::ATOMIC_STORE16_I64_S;<br>
-  case WebAssembly::ATOMIC_STORE32_I64: return WebAssembly::ATOMIC_STORE32_I64_S;<br>
-  case WebAssembly::ATOMIC_STORE8_I32: return WebAssembly::ATOMIC_STORE8_I32_S;<br>
-  case WebAssembly::ATOMIC_STORE8_I64: return WebAssembly::ATOMIC_STORE8_I64_S;<br>
-  case WebAssembly::ATOMIC_STORE_I32: return WebAssembly::ATOMIC_STORE_I32_S;<br>
-  case WebAssembly::ATOMIC_STORE_I64: return WebAssembly::ATOMIC_STORE_I64_S;<br>
-  case WebAssembly::BLOCK: return WebAssembly::BLOCK_S;<br>
-  case WebAssembly::BR: return WebAssembly::BR_S;<br>
-  case WebAssembly::BR_IF: return WebAssembly::BR_IF_S;<br>
-  case WebAssembly::BR_TABLE_I32: return WebAssembly::BR_TABLE_I32_S;<br>
-  case WebAssembly::BR_TABLE_I64: return WebAssembly::BR_TABLE_I64_S;<br>
-  case WebAssembly::BR_UNLESS: return WebAssembly::BR_UNLESS_S;<br>
-  case WebAssembly::CALL_EXCEPT_REF: return WebAssembly::CALL_EXCEPT_REF_S;<br>
-  case WebAssembly::CALL_F32: return WebAssembly::CALL_F32_S;<br>
-  case WebAssembly::CALL_F64: return WebAssembly::CALL_F64_S;<br>
-  case WebAssembly::CALL_I32: return WebAssembly::CALL_I32_S;<br>
-  case WebAssembly::CALL_I64: return WebAssembly::CALL_I64_S;<br>
-  case WebAssembly::CALL_INDIRECT_EXCEPT_REF: return WebAssembly::CALL_INDIRECT_EXCEPT_REF_S;<br>
-  case WebAssembly::CALL_INDIRECT_F32: return WebAssembly::CALL_INDIRECT_F32_S;<br>
-  case WebAssembly::CALL_INDIRECT_F64: return WebAssembly::CALL_INDIRECT_F64_S;<br>
-  case WebAssembly::CALL_INDIRECT_I32: return WebAssembly::CALL_INDIRECT_I32_S;<br>
-  case WebAssembly::CALL_INDIRECT_I64: return WebAssembly::CALL_INDIRECT_I64_S;<br>
-  case WebAssembly::CALL_INDIRECT_VOID: return WebAssembly::CALL_INDIRECT_VOID_S;<br>
-  case WebAssembly::CALL_INDIRECT_v16i8: return WebAssembly::CALL_INDIRECT_v16i8_S;<br>
-  case WebAssembly::CALL_INDIRECT_v4f32: return WebAssembly::CALL_INDIRECT_v4f32_S;<br>
-  case WebAssembly::CALL_INDIRECT_v4i32: return WebAssembly::CALL_INDIRECT_v4i32_S;<br>
-  case WebAssembly::CALL_INDIRECT_v8i16: return WebAssembly::CALL_INDIRECT_v8i16_S;<br>
-  case WebAssembly::CALL_VOID: return WebAssembly::CALL_VOID_S;<br>
-  case WebAssembly::CALL_v16i8: return WebAssembly::CALL_v16i8_S;<br>
-  case WebAssembly::CALL_v4f32: return WebAssembly::CALL_v4f32_S;<br>
-  case WebAssembly::CALL_v4i32: return WebAssembly::CALL_v4i32_S;<br>
-  case WebAssembly::CALL_v8i16: return WebAssembly::CALL_v8i16_S;<br>
-  case WebAssembly::CATCHRET: return WebAssembly::CATCHRET_S;<br>
-  case WebAssembly::CATCH_ALL: return WebAssembly::CATCH_ALL_S;<br>
-  case WebAssembly::CATCH_I32: return WebAssembly::CATCH_I32_S;<br>
-  case WebAssembly::CATCH_I64: return WebAssembly::CATCH_I64_S;<br>
-  case WebAssembly::CEIL_F32: return WebAssembly::CEIL_F32_S;<br>
-  case WebAssembly::CEIL_F64: return WebAssembly::CEIL_F64_S;<br>
-  case WebAssembly::CLEANUPRET: return WebAssembly::CLEANUPRET_S;<br>
-  case WebAssembly::CLZ_I32: return WebAssembly::CLZ_I32_S;<br>
-  case WebAssembly::CLZ_I64: return WebAssembly::CLZ_I64_S;<br>
-  case WebAssembly::CONST_F32: return WebAssembly::CONST_F32_S;<br>
-  case WebAssembly::CONST_F64: return WebAssembly::CONST_F64_S;<br>
-  case WebAssembly::CONST_I32: return WebAssembly::CONST_I32_S;<br>
-  case WebAssembly::CONST_I64: return WebAssembly::CONST_I64_S;<br>
-  case WebAssembly::COPYSIGN_F32: return WebAssembly::COPYSIGN_F32_S;<br>
-  case WebAssembly::COPYSIGN_F64: return WebAssembly::COPYSIGN_F64_S;<br>
-  case WebAssembly::COPY_EXCEPT_REF: return WebAssembly::COPY_EXCEPT_REF_S;<br>
-  case WebAssembly::COPY_F32: return WebAssembly::COPY_F32_S;<br>
-  case WebAssembly::COPY_F64: return WebAssembly::COPY_F64_S;<br>
-  case WebAssembly::COPY_I32: return WebAssembly::COPY_I32_S;<br>
-  case WebAssembly::COPY_I64: return WebAssembly::COPY_I64_S;<br>
-  case WebAssembly::COPY_V128: return WebAssembly::COPY_V128_S;<br>
-  case WebAssembly::CTZ_I32: return WebAssembly::CTZ_I32_S;<br>
-  case WebAssembly::CTZ_I64: return WebAssembly::CTZ_I64_S;<br>
-  case WebAssembly::CURRENT_MEMORY_I32: return WebAssembly::CURRENT_MEMORY_I32_S;<br>
-  case WebAssembly::DIV_F32: return WebAssembly::DIV_F32_S;<br>
-  case WebAssembly::DIV_F64: return WebAssembly::DIV_F64_S;<br>
-  case WebAssembly::DIV_S_I32: return WebAssembly::DIV_S_I32_S;<br>
-  case WebAssembly::DIV_S_I64: return WebAssembly::DIV_S_I64_S;<br>
-  case WebAssembly::DIV_U_I32: return WebAssembly::DIV_U_I32_S;<br>
-  case WebAssembly::DIV_U_I64: return WebAssembly::DIV_U_I64_S;<br>
-  case WebAssembly::DROP_EXCEPT_REF: return WebAssembly::DROP_EXCEPT_REF_S;<br>
-  case WebAssembly::DROP_F32: return WebAssembly::DROP_F32_S;<br>
-  case WebAssembly::DROP_F64: return WebAssembly::DROP_F64_S;<br>
-  case WebAssembly::DROP_I32: return WebAssembly::DROP_I32_S;<br>
-  case WebAssembly::DROP_I64: return WebAssembly::DROP_I64_S;<br>
-  case WebAssembly::DROP_V128: return WebAssembly::DROP_V128_S;<br>
-  case WebAssembly::END_BLOCK: return WebAssembly::END_BLOCK_S;<br>
-  case WebAssembly::END_FUNCTION: return WebAssembly::END_FUNCTION_S;<br>
-  case WebAssembly::END_LOOP: return WebAssembly::END_LOOP_S;<br>
-  case WebAssembly::END_TRY: return WebAssembly::END_TRY_S;<br>
-  case WebAssembly::EQZ_I32: return WebAssembly::EQZ_I32_S;<br>
-  case WebAssembly::EQZ_I64: return WebAssembly::EQZ_I64_S;<br>
-  case WebAssembly::EQ_F32: return WebAssembly::EQ_F32_S;<br>
-  case WebAssembly::EQ_F64: return WebAssembly::EQ_F64_S;<br>
-  case WebAssembly::EQ_I32: return WebAssembly::EQ_I32_S;<br>
-  case WebAssembly::EQ_I64: return WebAssembly::EQ_I64_S;<br>
-  case WebAssembly::F32_CONVERT_S_I32: return WebAssembly::F32_CONVERT_S_I32_S;<br>
-  case WebAssembly::F32_CONVERT_S_I64: return WebAssembly::F32_CONVERT_S_I64_S;<br>
-  case WebAssembly::F32_CONVERT_U_I32: return WebAssembly::F32_CONVERT_U_I32_S;<br>
-  case WebAssembly::F32_CONVERT_U_I64: return WebAssembly::F32_CONVERT_U_I64_S;<br>
-  case WebAssembly::F32_DEMOTE_F64: return WebAssembly::F32_DEMOTE_F64_S;<br>
-  case WebAssembly::F32_REINTERPRET_I32: return WebAssembly::F32_REINTERPRET_I32_S;<br>
-  case WebAssembly::F64_CONVERT_S_I32: return WebAssembly::F64_CONVERT_S_I32_S;<br>
-  case WebAssembly::F64_CONVERT_S_I64: return WebAssembly::F64_CONVERT_S_I64_S;<br>
-  case WebAssembly::F64_CONVERT_U_I32: return WebAssembly::F64_CONVERT_U_I32_S;<br>
-  case WebAssembly::F64_CONVERT_U_I64: return WebAssembly::F64_CONVERT_U_I64_S;<br>
-  case WebAssembly::F64_PROMOTE_F32: return WebAssembly::F64_PROMOTE_F32_S;<br>
-  case WebAssembly::F64_REINTERPRET_I64: return WebAssembly::F64_REINTERPRET_I64_S;<br>
-  case WebAssembly::FALLTHROUGH_RETURN_EXCEPT_REF: return WebAssembly::FALLTHROUGH_RETURN_EXCEPT_REF_S;<br>
-  case WebAssembly::FALLTHROUGH_RETURN_F32: return WebAssembly::FALLTHROUGH_RETURN_F32_S;<br>
-  case WebAssembly::FALLTHROUGH_RETURN_F64: return WebAssembly::FALLTHROUGH_RETURN_F64_S;<br>
-  case WebAssembly::FALLTHROUGH_RETURN_I32: return WebAssembly::FALLTHROUGH_RETURN_I32_S;<br>
-  case WebAssembly::FALLTHROUGH_RETURN_I64: return WebAssembly::FALLTHROUGH_RETURN_I64_S;<br>
-  case WebAssembly::FALLTHROUGH_RETURN_VOID: return WebAssembly::FALLTHROUGH_RETURN_VOID_S;<br>
-  case WebAssembly::FALLTHROUGH_RETURN_v16i8: return WebAssembly::FALLTHROUGH_RETURN_v16i8_S;<br>
-  case WebAssembly::FALLTHROUGH_RETURN_v4f32: return WebAssembly::FALLTHROUGH_RETURN_v4f32_S;<br>
-  case WebAssembly::FALLTHROUGH_RETURN_v4i32: return WebAssembly::FALLTHROUGH_RETURN_v4i32_S;<br>
-  case WebAssembly::FALLTHROUGH_RETURN_v8i16: return WebAssembly::FALLTHROUGH_RETURN_v8i16_S;<br>
-  case WebAssembly::FALLTHROUGH_RETURN_v2f64: return WebAssembly::FALLTHROUGH_RETURN_v2f64_S;<br>
-  case WebAssembly::FALLTHROUGH_RETURN_v2i64: return WebAssembly::FALLTHROUGH_RETURN_v2i64_S;<br>
-  case WebAssembly::FLOOR_F32: return WebAssembly::FLOOR_F32_S;<br>
-  case WebAssembly::FLOOR_F64: return WebAssembly::FLOOR_F64_S;<br>
-  case WebAssembly::FP_TO_SINT_I32_F32: return WebAssembly::FP_TO_SINT_I32_F32_S;<br>
-  case WebAssembly::FP_TO_SINT_I32_F64: return WebAssembly::FP_TO_SINT_I32_F64_S;<br>
-  case WebAssembly::FP_TO_SINT_I64_F32: return WebAssembly::FP_TO_SINT_I64_F32_S;<br>
-  case WebAssembly::FP_TO_SINT_I64_F64: return WebAssembly::FP_TO_SINT_I64_F64_S;<br>
-  case WebAssembly::FP_TO_UINT_I32_F32: return WebAssembly::FP_TO_UINT_I32_F32_S;<br>
-  case WebAssembly::FP_TO_UINT_I32_F64: return WebAssembly::FP_TO_UINT_I32_F64_S;<br>
-  case WebAssembly::FP_TO_UINT_I64_F32: return WebAssembly::FP_TO_UINT_I64_F32_S;<br>
-  case WebAssembly::FP_TO_UINT_I64_F64: return WebAssembly::FP_TO_UINT_I64_F64_S;<br>
-  case WebAssembly::GET_GLOBAL_EXCEPT_REF: return WebAssembly::GET_GLOBAL_EXCEPT_REF_S;<br>
-  case WebAssembly::GET_GLOBAL_F32: return WebAssembly::GET_GLOBAL_F32_S;<br>
-  case WebAssembly::GET_GLOBAL_F64: return WebAssembly::GET_GLOBAL_F64_S;<br>
-  case WebAssembly::GET_GLOBAL_I32: return WebAssembly::GET_GLOBAL_I32_S;<br>
-  case WebAssembly::GET_GLOBAL_I64: return WebAssembly::GET_GLOBAL_I64_S;<br>
-  case WebAssembly::GET_GLOBAL_V128: return WebAssembly::GET_GLOBAL_V128_S;<br>
-  case WebAssembly::GET_LOCAL_EXCEPT_REF: return WebAssembly::GET_LOCAL_EXCEPT_REF_S;<br>
-  case WebAssembly::GET_LOCAL_F32: return WebAssembly::GET_LOCAL_F32_S;<br>
-  case WebAssembly::GET_LOCAL_F64: return WebAssembly::GET_LOCAL_F64_S;<br>
-  case WebAssembly::GET_LOCAL_I32: return WebAssembly::GET_LOCAL_I32_S;<br>
-  case WebAssembly::GET_LOCAL_I64: return WebAssembly::GET_LOCAL_I64_S;<br>
-  case WebAssembly::GET_LOCAL_V128: return WebAssembly::GET_LOCAL_V128_S;<br>
-  case WebAssembly::GE_F32: return WebAssembly::GE_F32_S;<br>
-  case WebAssembly::GE_F64: return WebAssembly::GE_F64_S;<br>
-  case WebAssembly::GE_S_I32: return WebAssembly::GE_S_I32_S;<br>
-  case WebAssembly::GE_S_I64: return WebAssembly::GE_S_I64_S;<br>
-  case WebAssembly::GE_U_I32: return WebAssembly::GE_U_I32_S;<br>
-  case WebAssembly::GE_U_I64: return WebAssembly::GE_U_I64_S;<br>
-  case WebAssembly::GROW_MEMORY_I32: return WebAssembly::GROW_MEMORY_I32_S;<br>
-  case WebAssembly::GT_F32: return WebAssembly::GT_F32_S;<br>
-  case WebAssembly::GT_F64: return WebAssembly::GT_F64_S;<br>
-  case WebAssembly::GT_S_I32: return WebAssembly::GT_S_I32_S;<br>
-  case WebAssembly::GT_S_I64: return WebAssembly::GT_S_I64_S;<br>
-  case WebAssembly::GT_U_I32: return WebAssembly::GT_U_I32_S;<br>
-  case WebAssembly::GT_U_I64: return WebAssembly::GT_U_I64_S;<br>
-  case WebAssembly::I32_EXTEND16_S_I32: return WebAssembly::I32_EXTEND16_S_I32_S;<br>
-  case WebAssembly::I32_EXTEND8_S_I32: return WebAssembly::I32_EXTEND8_S_I32_S;<br>
-  case WebAssembly::I32_REINTERPRET_F32: return WebAssembly::I32_REINTERPRET_F32_S;<br>
-  case WebAssembly::I32_TRUNC_S_F32: return WebAssembly::I32_TRUNC_S_F32_S;<br>
-  case WebAssembly::I32_TRUNC_S_F64: return WebAssembly::I32_TRUNC_S_F64_S;<br>
-  case WebAssembly::I32_TRUNC_S_SAT_F32: return WebAssembly::I32_TRUNC_S_SAT_F32_S;<br>
-  case WebAssembly::I32_TRUNC_S_SAT_F64: return WebAssembly::I32_TRUNC_S_SAT_F64_S;<br>
-  case WebAssembly::I32_TRUNC_U_F32: return WebAssembly::I32_TRUNC_U_F32_S;<br>
-  case WebAssembly::I32_TRUNC_U_F64: return WebAssembly::I32_TRUNC_U_F64_S;<br>
-  case WebAssembly::I32_TRUNC_U_SAT_F32: return WebAssembly::I32_TRUNC_U_SAT_F32_S;<br>
-  case WebAssembly::I32_TRUNC_U_SAT_F64: return WebAssembly::I32_TRUNC_U_SAT_F64_S;<br>
-  case WebAssembly::I32_WRAP_I64: return WebAssembly::I32_WRAP_I64_S;<br>
-  case WebAssembly::I64_EXTEND16_S_I64: return WebAssembly::I64_EXTEND16_S_I64_S;<br>
-  case WebAssembly::I64_EXTEND32_S_I64: return WebAssembly::I64_EXTEND32_S_I64_S;<br>
-  case WebAssembly::I64_EXTEND8_S_I64: return WebAssembly::I64_EXTEND8_S_I64_S;<br>
-  case WebAssembly::I64_EXTEND_S_I32: return WebAssembly::I64_EXTEND_S_I32_S;<br>
-  case WebAssembly::I64_EXTEND_U_I32: return WebAssembly::I64_EXTEND_U_I32_S;<br>
-  case WebAssembly::I64_REINTERPRET_F64: return WebAssembly::I64_REINTERPRET_F64_S;<br>
-  case WebAssembly::I64_TRUNC_S_F32: return WebAssembly::I64_TRUNC_S_F32_S;<br>
-  case WebAssembly::I64_TRUNC_S_F64: return WebAssembly::I64_TRUNC_S_F64_S;<br>
-  case WebAssembly::I64_TRUNC_S_SAT_F32: return WebAssembly::I64_TRUNC_S_SAT_F32_S;<br>
-  case WebAssembly::I64_TRUNC_S_SAT_F64: return WebAssembly::I64_TRUNC_S_SAT_F64_S;<br>
-  case WebAssembly::I64_TRUNC_U_F32: return WebAssembly::I64_TRUNC_U_F32_S;<br>
-  case WebAssembly::I64_TRUNC_U_F64: return WebAssembly::I64_TRUNC_U_F64_S;<br>
-  case WebAssembly::I64_TRUNC_U_SAT_F32: return WebAssembly::I64_TRUNC_U_SAT_F32_S;<br>
-  case WebAssembly::I64_TRUNC_U_SAT_F64: return WebAssembly::I64_TRUNC_U_SAT_F64_S;<br>
-  case WebAssembly::LE_F32: return WebAssembly::LE_F32_S;<br>
-  case WebAssembly::LE_F64: return WebAssembly::LE_F64_S;<br>
-  case WebAssembly::LE_S_I32: return WebAssembly::LE_S_I32_S;<br>
-  case WebAssembly::LE_S_I64: return WebAssembly::LE_S_I64_S;<br>
-  case WebAssembly::LE_U_I32: return WebAssembly::LE_U_I32_S;<br>
-  case WebAssembly::LE_U_I64: return WebAssembly::LE_U_I64_S;<br>
-  case WebAssembly::LOAD16_S_I32: return WebAssembly::LOAD16_S_I32_S;<br>
-  case WebAssembly::LOAD16_S_I64: return WebAssembly::LOAD16_S_I64_S;<br>
-  case WebAssembly::LOAD16_U_I32: return WebAssembly::LOAD16_U_I32_S;<br>
-  case WebAssembly::LOAD16_U_I64: return WebAssembly::LOAD16_U_I64_S;<br>
-  case WebAssembly::LOAD32_S_I64: return WebAssembly::LOAD32_S_I64_S;<br>
-  case WebAssembly::LOAD32_U_I64: return WebAssembly::LOAD32_U_I64_S;<br>
-  case WebAssembly::LOAD8_S_I32: return WebAssembly::LOAD8_S_I32_S;<br>
-  case WebAssembly::LOAD8_S_I64: return WebAssembly::LOAD8_S_I64_S;<br>
-  case WebAssembly::LOAD8_U_I32: return WebAssembly::LOAD8_U_I32_S;<br>
-  case WebAssembly::LOAD8_U_I64: return WebAssembly::LOAD8_U_I64_S;<br>
-  case WebAssembly::LOAD_F32: return WebAssembly::LOAD_F32_S;<br>
-  case WebAssembly::LOAD_F64: return WebAssembly::LOAD_F64_S;<br>
-  case WebAssembly::LOAD_I32: return WebAssembly::LOAD_I32_S;<br>
-  case WebAssembly::LOAD_I64: return WebAssembly::LOAD_I64_S;<br>
-  case WebAssembly::LOOP: return WebAssembly::LOOP_S;<br>
-  case WebAssembly::LT_F32: return WebAssembly::LT_F32_S;<br>
-  case WebAssembly::LT_F64: return WebAssembly::LT_F64_S;<br>
-  case WebAssembly::LT_S_I32: return WebAssembly::LT_S_I32_S;<br>
-  case WebAssembly::LT_S_I64: return WebAssembly::LT_S_I64_S;<br>
-  case WebAssembly::LT_U_I32: return WebAssembly::LT_U_I32_S;<br>
-  case WebAssembly::LT_U_I64: return WebAssembly::LT_U_I64_S;<br>
-  case WebAssembly::MAX_F32: return WebAssembly::MAX_F32_S;<br>
-  case WebAssembly::MAX_F64: return WebAssembly::MAX_F64_S;<br>
-  case WebAssembly::MEMORY_GROW_I32: return WebAssembly::MEMORY_GROW_I32_S;<br>
-  case WebAssembly::MEMORY_SIZE_I32: return WebAssembly::MEMORY_SIZE_I32_S;<br>
-  case WebAssembly::MEM_GROW_I32: return WebAssembly::MEM_GROW_I32_S;<br>
-  case WebAssembly::MEM_SIZE_I32: return WebAssembly::MEM_SIZE_I32_S;<br>
-  case WebAssembly::MIN_F32: return WebAssembly::MIN_F32_S;<br>
-  case WebAssembly::MIN_F64: return WebAssembly::MIN_F64_S;<br>
-  case WebAssembly::MUL_F32: return WebAssembly::MUL_F32_S;<br>
-  case WebAssembly::MUL_F32x4: return WebAssembly::MUL_F32x4_S;<br>
-  case WebAssembly::MUL_F64: return WebAssembly::MUL_F64_S;<br>
-  case WebAssembly::MUL_I16x8: return WebAssembly::MUL_I16x8_S;<br>
-  case WebAssembly::MUL_I32: return WebAssembly::MUL_I32_S;<br>
-  case WebAssembly::MUL_I32x4: return WebAssembly::MUL_I32x4_S;<br>
-  case WebAssembly::MUL_I64: return WebAssembly::MUL_I64_S;<br>
-  case WebAssembly::MUL_I8x16: return WebAssembly::MUL_I8x16_S;<br>
-  case WebAssembly::NEAREST_F32: return WebAssembly::NEAREST_F32_S;<br>
-  case WebAssembly::NEAREST_F64: return WebAssembly::NEAREST_F64_S;<br>
-  case WebAssembly::NEG_F32: return WebAssembly::NEG_F32_S;<br>
-  case WebAssembly::NEG_F64: return WebAssembly::NEG_F64_S;<br>
-  case WebAssembly::NE_F32: return WebAssembly::NE_F32_S;<br>
-  case WebAssembly::NE_F64: return WebAssembly::NE_F64_S;<br>
-  case WebAssembly::NE_I32: return WebAssembly::NE_I32_S;<br>
-  case WebAssembly::NE_I64: return WebAssembly::NE_I64_S;<br>
-  case WebAssembly::NOP: return WebAssembly::NOP_S;<br>
-  case WebAssembly::OR_I32: return WebAssembly::OR_I32_S;<br>
-  case WebAssembly::OR_I64: return WebAssembly::OR_I64_S;<br>
-  case WebAssembly::PCALL_INDIRECT_EXCEPT_REF: return WebAssembly::PCALL_INDIRECT_EXCEPT_REF_S;<br>
-  case WebAssembly::PCALL_INDIRECT_F32: return WebAssembly::PCALL_INDIRECT_F32_S;<br>
-  case WebAssembly::PCALL_INDIRECT_F64: return WebAssembly::PCALL_INDIRECT_F64_S;<br>
-  case WebAssembly::PCALL_INDIRECT_I32: return WebAssembly::PCALL_INDIRECT_I32_S;<br>
-  case WebAssembly::PCALL_INDIRECT_I64: return WebAssembly::PCALL_INDIRECT_I64_S;<br>
-  case WebAssembly::PCALL_INDIRECT_VOID: return WebAssembly::PCALL_INDIRECT_VOID_S;<br>
-  case WebAssembly::PCALL_INDIRECT_v16i8: return WebAssembly::PCALL_INDIRECT_v16i8_S;<br>
-  case WebAssembly::PCALL_INDIRECT_v4f32: return WebAssembly::PCALL_INDIRECT_v4f32_S;<br>
-  case WebAssembly::PCALL_INDIRECT_v4i32: return WebAssembly::PCALL_INDIRECT_v4i32_S;<br>
-  case WebAssembly::PCALL_INDIRECT_v8i16: return WebAssembly::PCALL_INDIRECT_v8i16_S;<br>
-  case WebAssembly::POPCNT_I32: return WebAssembly::POPCNT_I32_S;<br>
-  case WebAssembly::POPCNT_I64: return WebAssembly::POPCNT_I64_S;<br>
-  case WebAssembly::REM_S_I32: return WebAssembly::REM_S_I32_S;<br>
-  case WebAssembly::REM_S_I64: return WebAssembly::REM_S_I64_S;<br>
-  case WebAssembly::REM_U_I32: return WebAssembly::REM_U_I32_S;<br>
-  case WebAssembly::REM_U_I64: return WebAssembly::REM_U_I64_S;<br>
-  case WebAssembly::RETHROW: return WebAssembly::RETHROW_S;<br>
-  case WebAssembly::RETHROW_TO_CALLER: return WebAssembly::RETHROW_TO_CALLER_S;<br>
-  case WebAssembly::RETURN_EXCEPT_REF: return WebAssembly::RETURN_EXCEPT_REF_S;<br>
-  case WebAssembly::RETURN_F32: return WebAssembly::RETURN_F32_S;<br>
-  case WebAssembly::RETURN_F64: return WebAssembly::RETURN_F64_S;<br>
-  case WebAssembly::RETURN_I32: return WebAssembly::RETURN_I32_S;<br>
-  case WebAssembly::RETURN_I64: return WebAssembly::RETURN_I64_S;<br>
-  case WebAssembly::RETURN_VOID: return WebAssembly::RETURN_VOID_S;<br>
-  case WebAssembly::RETURN_v16i8: return WebAssembly::RETURN_v16i8_S;<br>
-  case WebAssembly::RETURN_v4f32: return WebAssembly::RETURN_v4f32_S;<br>
-  case WebAssembly::RETURN_v4i32: return WebAssembly::RETURN_v4i32_S;<br>
-  case WebAssembly::RETURN_v8i16: return WebAssembly::RETURN_v8i16_S;<br>
-  case WebAssembly::ROTL_I32: return WebAssembly::ROTL_I32_S;<br>
-  case WebAssembly::ROTL_I64: return WebAssembly::ROTL_I64_S;<br>
-  case WebAssembly::ROTR_I32: return WebAssembly::ROTR_I32_S;<br>
-  case WebAssembly::ROTR_I64: return WebAssembly::ROTR_I64_S;<br>
-  case WebAssembly::SELECT_EXCEPT_REF: return WebAssembly::SELECT_EXCEPT_REF_S;<br>
-  case WebAssembly::SELECT_F32: return WebAssembly::SELECT_F32_S;<br>
-  case WebAssembly::SELECT_F64: return WebAssembly::SELECT_F64_S;<br>
-  case WebAssembly::SELECT_I32: return WebAssembly::SELECT_I32_S;<br>
-  case WebAssembly::SELECT_I64: return WebAssembly::SELECT_I64_S;<br>
-  case WebAssembly::SET_GLOBAL_EXCEPT_REF: return WebAssembly::SET_GLOBAL_EXCEPT_REF_S;<br>
-  case WebAssembly::SET_GLOBAL_F32: return WebAssembly::SET_GLOBAL_F32_S;<br>
-  case WebAssembly::SET_GLOBAL_F64: return WebAssembly::SET_GLOBAL_F64_S;<br>
-  case WebAssembly::SET_GLOBAL_I32: return WebAssembly::SET_GLOBAL_I32_S;<br>
-  case WebAssembly::SET_GLOBAL_I64: return WebAssembly::SET_GLOBAL_I64_S;<br>
-  case WebAssembly::SET_GLOBAL_V128: return WebAssembly::SET_GLOBAL_V128_S;<br>
-  case WebAssembly::SET_LOCAL_EXCEPT_REF: return WebAssembly::SET_LOCAL_EXCEPT_REF_S;<br>
-  case WebAssembly::SET_LOCAL_F32: return WebAssembly::SET_LOCAL_F32_S;<br>
-  case WebAssembly::SET_LOCAL_F64: return WebAssembly::SET_LOCAL_F64_S;<br>
-  case WebAssembly::SET_LOCAL_I32: return WebAssembly::SET_LOCAL_I32_S;<br>
-  case WebAssembly::SET_LOCAL_I64: return WebAssembly::SET_LOCAL_I64_S;<br>
-  case WebAssembly::SET_LOCAL_V128: return WebAssembly::SET_LOCAL_V128_S;<br>
-  case WebAssembly::SHL_I32: return WebAssembly::SHL_I32_S;<br>
-  case WebAssembly::SHL_I64: return WebAssembly::SHL_I64_S;<br>
-  case WebAssembly::SHR_S_I32: return WebAssembly::SHR_S_I32_S;<br>
-  case WebAssembly::SHR_S_I64: return WebAssembly::SHR_S_I64_S;<br>
-  case WebAssembly::SHR_U_I32: return WebAssembly::SHR_U_I32_S;<br>
-  case WebAssembly::SHR_U_I64: return WebAssembly::SHR_U_I64_S;<br>
-  case WebAssembly::SQRT_F32: return WebAssembly::SQRT_F32_S;<br>
-  case WebAssembly::SQRT_F64: return WebAssembly::SQRT_F64_S;<br>
-  case WebAssembly::STORE16_I32: return WebAssembly::STORE16_I32_S;<br>
-  case WebAssembly::STORE16_I64: return WebAssembly::STORE16_I64_S;<br>
-  case WebAssembly::STORE32_I64: return WebAssembly::STORE32_I64_S;<br>
-  case WebAssembly::STORE8_I32: return WebAssembly::STORE8_I32_S;<br>
-  case WebAssembly::STORE8_I64: return WebAssembly::STORE8_I64_S;<br>
-  case WebAssembly::STORE_F32: return WebAssembly::STORE_F32_S;<br>
-  case WebAssembly::STORE_F64: return WebAssembly::STORE_F64_S;<br>
-  case WebAssembly::STORE_I32: return WebAssembly::STORE_I32_S;<br>
-  case WebAssembly::STORE_I64: return WebAssembly::STORE_I64_S;<br>
-  case WebAssembly::SUB_F32: return WebAssembly::SUB_F32_S;<br>
-  case WebAssembly::SUB_F32x4: return WebAssembly::SUB_F32x4_S;<br>
-  case WebAssembly::SUB_F64: return WebAssembly::SUB_F64_S;<br>
-  case WebAssembly::SUB_I16x8: return WebAssembly::SUB_I16x8_S;<br>
-  case WebAssembly::SUB_I32: return WebAssembly::SUB_I32_S;<br>
-  case WebAssembly::SUB_I32x4: return WebAssembly::SUB_I32x4_S;<br>
-  case WebAssembly::SUB_I64: return WebAssembly::SUB_I64_S;<br>
-  case WebAssembly::SUB_I8x16: return WebAssembly::SUB_I8x16_S;<br>
-  case WebAssembly::TEE_EXCEPT_REF: return WebAssembly::TEE_EXCEPT_REF_S;<br>
-  case WebAssembly::TEE_F32: return WebAssembly::TEE_F32_S;<br>
-  case WebAssembly::TEE_F64: return WebAssembly::TEE_F64_S;<br>
-  case WebAssembly::TEE_I32: return WebAssembly::TEE_I32_S;<br>
-  case WebAssembly::TEE_I64: return WebAssembly::TEE_I64_S;<br>
-  case WebAssembly::TEE_LOCAL_EXCEPT_REF: return WebAssembly::TEE_LOCAL_EXCEPT_REF_S;<br>
-  case WebAssembly::TEE_LOCAL_F32: return WebAssembly::TEE_LOCAL_F32_S;<br>
-  case WebAssembly::TEE_LOCAL_F64: return WebAssembly::TEE_LOCAL_F64_S;<br>
-  case WebAssembly::TEE_LOCAL_I32: return WebAssembly::TEE_LOCAL_I32_S;<br>
-  case WebAssembly::TEE_LOCAL_I64: return WebAssembly::TEE_LOCAL_I64_S;<br>
-  case WebAssembly::TEE_LOCAL_V128: return WebAssembly::TEE_LOCAL_V128_S;<br>
-  case WebAssembly::TEE_V128: return WebAssembly::TEE_V128_S;<br>
-  case WebAssembly::THROW_I32: return WebAssembly::THROW_I32_S;<br>
-  case WebAssembly::THROW_I64: return WebAssembly::THROW_I64_S;<br>
-  case WebAssembly::TRUNC_F32: return WebAssembly::TRUNC_F32_S;<br>
-  case WebAssembly::TRUNC_F64: return WebAssembly::TRUNC_F64_S;<br>
-  case WebAssembly::TRY: return WebAssembly::TRY_S;<br>
-  case WebAssembly::UNREACHABLE: return WebAssembly::UNREACHABLE_S;<br>
-  case WebAssembly::XOR_I32: return WebAssembly::XOR_I32_S;<br>
-  case WebAssembly::XOR_I64: return WebAssembly::XOR_I64_S;<br>
+#ifndef NDEBUG<br>
+  // Assert that all registers have been stackified at this point.<br>
+  for (const MachineBasicBlock &MBB : MF) {<br>
+    for (const MachineInstr &MI : MBB) {<br>
+      if (MI.isDebugInstr() || MI.isLabel())<br>
+        continue;<br>
+      for (const MachineOperand &MO : MI.explicit_operands()) {<br>
+        assert(<br>
+            (!MO.isReg() || MRI.use_empty(MO.getReg()) ||<br>
+             MFI.isVRegStackified(MO.getReg())) &&<br>
+            "WebAssemblyExplicitLocals failed to stackify a register operand");<br>
+      }<br>
+    }<br>
   }<br>
+#endif<br>
+<br>
+  return Changed;<br>
 }<br>
<br>
Modified: llvm/trunk/lib/Target/WebAssembly/WebAssemblyInstrControl.td<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/WebAssembly/WebAssemblyInstrControl.td?rev=339630&r1=339629&r2=339630&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/WebAssembly/WebAssemblyInstrControl.td?rev=339630&r1=339629&r2=339630&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/Target/WebAssembly/WebAssemblyInstrControl.td (original)<br>
+++ llvm/trunk/lib/Target/WebAssembly/WebAssemblyInstrControl.td Mon Aug 13 16:12:49 2018<br>
@@ -53,9 +53,9 @@ def BR_TABLE_I32 : NI<(outs), (ins I32:$<br>
   let TSFlags{0} = 1;<br>
   let TSFlags{1} = 1;<br>
 }<br>
-def BR_TABLE_I32_S : NI<(outs), (ins variable_ops),<br>
+def BR_TABLE_I32_S : NI<(outs), (ins I32:$index),<br>
                         [], 1,<br>
-                        "br_table", 0x0e> {<br>
+                        "br_table \t$index", 0x0e> {<br>
   let TSFlags{0} = 1;<br>
   let TSFlags{1} = 1;<br>
 }<br>
@@ -65,9 +65,9 @@ def BR_TABLE_I64 : NI<(outs), (ins I64:$<br>
   let TSFlags{0} = 1;<br>
   let TSFlags{1} = 1;<br>
 }<br>
-def BR_TABLE_I64_S : NI<(outs), (ins variable_ops),<br>
+def BR_TABLE_I64_S : NI<(outs), (ins I64:$index),<br>
                         [], 1,<br>
-                        "br_table"> {<br>
+                        "br_table \t$index"> {<br>
   let TSFlags{0} = 1;<br>
   let TSFlags{1} = 1;<br>
 }<br>
<br>
Modified: llvm/trunk/lib/Target/WebAssembly/WebAssemblyInstrMemory.td<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/WebAssembly/WebAssemblyInstrMemory.td?rev=339630&r1=339629&r2=339630&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/WebAssembly/WebAssemblyInstrMemory.td?rev=339630&r1=339629&r2=339630&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/Target/WebAssembly/WebAssemblyInstrMemory.td (original)<br>
+++ llvm/trunk/lib/Target/WebAssembly/WebAssemblyInstrMemory.td Mon Aug 13 16:12:49 2018<br>
@@ -57,12 +57,11 @@ let Defs = [ARGUMENTS] in {<br>
<br>
 // Defines atomic and non-atomic loads, regular and extending.<br>
 multiclass WebAssemblyLoad<WebAssemblyRegClass rc, string Name, int Opcode> {<br>
-  let mayLoad = 1 in<br>
   defm "": I<(outs rc:$dst),<br>
              (ins P2Align:$p2align, offset32_op:$off, I32:$addr),<br>
              (outs), (ins P2Align:$p2align, offset32_op:$off),<br>
              [], !strconcat(Name, "\t$dst, ${off}(${addr})${p2align}"),<br>
-             !strconcat(Name, "\t${off}${p2align}"), Opcode>;<br>
+             !strconcat(Name, "\t${off}, ${p2align}"), Opcode>;<br>
 }<br>
<br>
 // Basic load.<br>
@@ -308,13 +307,12 @@ let Defs = [ARGUMENTS] in {<br>
<br>
 // Defines atomic and non-atomic stores, regular and truncating<br>
 multiclass WebAssemblyStore<WebAssemblyRegClass rc, string Name, int Opcode> {<br>
-  let mayStore = 1 in<br>
   defm "" : I<(outs),<br>
               (ins P2Align:$p2align, offset32_op:$off, I32:$addr, rc:$val),<br>
               (outs),<br>
               (ins P2Align:$p2align, offset32_op:$off), [],<br>
               !strconcat(Name, "\t${off}(${addr})${p2align}, $val"),<br>
-              !strconcat(Name, "\t${off}${p2align}"), Opcode>;<br>
+              !strconcat(Name, "\t${off}, ${p2align}"), Opcode>;<br>
 }<br>
 // Basic store.<br>
 // Note: WebAssembly inverts SelectionDAG's usual operand order.<br>
@@ -472,12 +470,12 @@ defm CURRENT_MEMORY_I32 : I<(outs I32:$d<br>
<br>
 // Grow memory.<br>
 defm MEMORY_GROW_I32 : I<(outs I32:$dst), (ins i32imm:$flags, I32:$delta),<br>
-                         (outs), (ins i32imm:$flags),<br>
+                         (outs), (ins i32imm:$flags, I32:$delta),<br>
                          [(set I32:$dst,<br>
                            (int_wasm_memory_grow (i32 imm:$flags),<br>
                              I32:$delta))],<br>
                          "memory.grow\t$dst, $flags, $delta",<br>
-                         "memory.grow\t$flags", 0x3f>,<br>
+                         "memory.grow\t$flags, $delta", 0x3f>,<br>
                        Requires<[HasAddr32]>;<br>
 defm MEM_GROW_I32 : I<(outs I32:$dst), (ins i32imm:$flags, I32:$delta),<br>
                       (outs), (ins i32imm:$flags),<br>
<br>
Modified: llvm/trunk/lib/Target/WebAssembly/WebAssemblyTargetMachine.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/WebAssembly/WebAssemblyTargetMachine.cpp?rev=339630&r1=339629&r2=339630&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/WebAssembly/WebAssemblyTargetMachine.cpp?rev=339630&r1=339629&r2=339630&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/Target/WebAssembly/WebAssemblyTargetMachine.cpp (original)<br>
+++ llvm/trunk/lib/Target/WebAssembly/WebAssemblyTargetMachine.cpp Mon Aug 13 16:12:49 2018<br>
@@ -317,6 +317,9 @@ void WebAssemblyPassConfig::addPreEmitPa<br>
   // converted into a local.<br>
   addPass(createWebAssemblyFixIrreducibleControlFlow());<br>
<br>
+  // Insert explicit get_local and set_local operators.<br>
+  addPass(createWebAssemblyExplicitLocals());<br>
+<br>
   // Do various transformations for exception handling<br>
   addPass(createWebAssemblyLateEHPrepare());<br>
<br>
@@ -334,9 +337,6 @@ void WebAssemblyPassConfig::addPreEmitPa<br>
   if (getOptLevel() != CodeGenOpt::None)<br>
     addPass(createWebAssemblyPeephole());<br>
<br>
-  // Insert explicit get_local and set_local operators.<br>
-  addPass(createWebAssemblyExplicitLocals());<br>
-<br>
   // Create a mapping from LLVM CodeGen virtual registers to wasm registers.<br>
   addPass(createWebAssemblyRegNumbering());<br>
 }<br>
<br>
Modified: llvm/trunk/test/CodeGen/WebAssembly/address-offsets.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/WebAssembly/address-offsets.ll?rev=339630&r1=339629&r2=339630&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/WebAssembly/address-offsets.ll?rev=339630&r1=339629&r2=339630&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/CodeGen/WebAssembly/address-offsets.ll (original)<br>
+++ llvm/trunk/test/CodeGen/WebAssembly/address-offsets.ll Mon Aug 13 16:12:49 2018<br>
@@ -1,4 +1,4 @@<br>
-; RUN: llc < %s -asm-verbose=false -disable-wasm-fallthrough-return-opt -wasm-register-codegen-test-mode | FileCheck %s<br>
+; RUN: llc < %s -asm-verbose=false -disable-wasm-fallthrough-return-opt -disable-wasm-explicit-locals | FileCheck %s<br>
<br>
 ; Test folding constant offsets and symbols into load and store addresses under<br>
 ; a variety of circumstances.<br>
<br>
Modified: llvm/trunk/test/CodeGen/WebAssembly/atomic-mem-consistency.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/WebAssembly/atomic-mem-consistency.ll?rev=339630&r1=339629&r2=339630&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/WebAssembly/atomic-mem-consistency.ll?rev=339630&r1=339629&r2=339630&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/CodeGen/WebAssembly/atomic-mem-consistency.ll (original)<br>
+++ llvm/trunk/test/CodeGen/WebAssembly/atomic-mem-consistency.ll Mon Aug 13 16:12:49 2018<br>
@@ -1,5 +1,5 @@<br>
 ; RUN: not llc < %s -asm-verbose=false -disable-wasm-fallthrough-return-opt<br>
-; RUN: llc < %s -asm-verbose=false -disable-wasm-fallthrough-return-opt -wasm-register-codegen-test-mode -mattr=+atomics,+sign-ext | FileCheck %s<br>
+; RUN: llc < %s -asm-verbose=false -disable-wasm-fallthrough-return-opt -disable-wasm-explicit-locals -mattr=+atomics,+sign-ext | FileCheck %s<br>
<br>
 ; Currently all wasm atomic memory access instructions are sequentially<br>
 ; consistent, so even if LLVM IR specifies weaker orderings than that, we<br>
<br>
Modified: llvm/trunk/test/CodeGen/WebAssembly/atomic-rmw.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/WebAssembly/atomic-rmw.ll?rev=339630&r1=339629&r2=339630&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/WebAssembly/atomic-rmw.ll?rev=339630&r1=339629&r2=339630&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/CodeGen/WebAssembly/atomic-rmw.ll (original)<br>
+++ llvm/trunk/test/CodeGen/WebAssembly/atomic-rmw.ll Mon Aug 13 16:12:49 2018<br>
@@ -1,5 +1,5 @@<br>
-; RUN: not llc < %s -asm-verbose=false -disable-wasm-fallthrough-return-opt -wasm-explicit-locals-codegen-test-mode<br>
-; RUN: llc < %s -asm-verbose=false -disable-wasm-fallthrough-return-opt -wasm-register-codegen-test-mode -mattr=+atomics,+sign-ext | FileCheck %s<br>
+; RUN: not llc < %s -asm-verbose=false -disable-wasm-fallthrough-return-opt<br>
+; RUN: llc < %s -asm-verbose=false -disable-wasm-fallthrough-return-opt -disable-wasm-explicit-locals -mattr=+atomics,+sign-ext | FileCheck %s<br>
<br>
 ; Test atomic RMW (read-modify-write) instructions are assembled properly.<br>
<br>
<br>
Modified: llvm/trunk/test/CodeGen/WebAssembly/byval.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/WebAssembly/byval.ll?rev=339630&r1=339629&r2=339630&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/WebAssembly/byval.ll?rev=339630&r1=339629&r2=339630&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/CodeGen/WebAssembly/byval.ll (original)<br>
+++ llvm/trunk/test/CodeGen/WebAssembly/byval.ll Mon Aug 13 16:12:49 2018<br>
@@ -1,5 +1,5 @@<br>
-; RUN: llc < %s -asm-verbose=false -disable-wasm-fallthrough-return-opt -wasm-register-codegen-test-mode -verify-machineinstrs | FileCheck %s<br>
-; RUN: llc < %s -asm-verbose=false -disable-wasm-fallthrough-return-opt -wasm-register-codegen-test-mode -verify-machineinstrs -fast-isel | FileCheck %s<br>
+; RUN: llc < %s -asm-verbose=false -disable-wasm-fallthrough-return-opt -disable-wasm-explicit-locals -verify-machineinstrs | FileCheck %s<br>
+; RUN: llc < %s -asm-verbose=false -disable-wasm-fallthrough-return-opt -disable-wasm-explicit-locals -verify-machineinstrs -fast-isel | FileCheck %s<br>
<br>
 target datalayout = "e-m:e-p:32:32-i64:64-n32:64-S128"<br>
 target triple = "wasm32-unknown-unknown"<br>
<br>
Modified: llvm/trunk/test/CodeGen/WebAssembly/call.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/WebAssembly/call.ll?rev=339630&r1=339629&r2=339630&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/WebAssembly/call.ll?rev=339630&r1=339629&r2=339630&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/CodeGen/WebAssembly/call.ll (original)<br>
+++ llvm/trunk/test/CodeGen/WebAssembly/call.ll Mon Aug 13 16:12:49 2018<br>
@@ -1,5 +1,5 @@<br>
-; RUN: llc < %s -asm-verbose=false -disable-wasm-fallthrough-return-opt -wasm-explicit-locals-codegen-test-mode -wasm-temporary-workarounds=false | FileCheck %s<br>
-; RUN: llc < %s -asm-verbose=false -disable-wasm-fallthrough-return-opt -wasm-explicit-locals-codegen-test-mode -fast-isel -fast-isel-abort=1 -wasm-temporary-workarounds=false | FileCheck %s<br>
+; RUN: llc < %s -asm-verbose=false -disable-wasm-fallthrough-return-opt -wasm-temporary-workarounds=false | FileCheck %s<br>
+; RUN: llc < %s -asm-verbose=false -disable-wasm-fallthrough-return-opt -fast-isel -fast-isel-abort=1 -wasm-temporary-workarounds=false | FileCheck %s<br>
<br>
 ; Test that basic call operations assemble as expected.<br>
<br>
<br>
Modified: llvm/trunk/test/CodeGen/WebAssembly/cfg-stackify-eh.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/WebAssembly/cfg-stackify-eh.ll?rev=339630&r1=339629&r2=339630&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/WebAssembly/cfg-stackify-eh.ll?rev=339630&r1=339629&r2=339630&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/CodeGen/WebAssembly/cfg-stackify-eh.ll (original)<br>
+++ llvm/trunk/test/CodeGen/WebAssembly/cfg-stackify-eh.ll Mon Aug 13 16:12:49 2018<br>
@@ -1,4 +1,4 @@<br>
-; RUN: llc < %s -asm-verbose=false -disable-wasm-fallthrough-return-opt -wasm-register-codegen-test-mode -disable-block-placement -verify-machineinstrs -fast-isel=false -machine-sink-split-probability-threshold=0 -cgp-freq-ratio-to-skip-merge=1000 -exception-model=wasm -mattr=+exception-handling | FileCheck %s<br>
+; RUN: llc < %s -asm-verbose=false -disable-wasm-fallthrough-return-opt -disable-wasm-explicit-locals -disable-block-placement -verify-machineinstrs -fast-isel=false -machine-sink-split-probability-threshold=0 -cgp-freq-ratio-to-skip-merge=1000 -exception-model=wasm -mattr=+exception-handling | FileCheck %s<br>
<br>
 target datalayout = "e-m:e-p:32:32-i64:64-n32:64-S128"<br>
 target triple = "wasm32-unknown-unknown"<br>
<br>
Modified: llvm/trunk/test/CodeGen/WebAssembly/cfg-stackify.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/WebAssembly/cfg-stackify.ll?rev=339630&r1=339629&r2=339630&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/WebAssembly/cfg-stackify.ll?rev=339630&r1=339629&r2=339630&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/test/CodeGen/WebAssembly/cfg-stackify.ll (original)<br>
+++ llvm/trunk/test/CodeGen/WebAssembly/cfg-stackify.ll Mon Aug 13 16:12:49 2018<br>
@@ -1,5 +1,5 @@<br>
-; RUN: llc < %s -asm-verbose=false -disable-wasm-fallthrough-return-opt -wasm-register-codegen-test-mode -disable-block-placement -verify-machineinstrs -fast-isel=false -machine-sink-split-probability-threshold=0 -cgp-freq-ratio-to-skip-merge=1000 | FileCheck %s<br>
-; RUN: llc < %s -asm-verbose=false -disable-wasm-fallthrough-return-opt -wasm-register-codegen-test-mode -tail-dup-placement=0 -verify-machineinstrs -fast-isel=false -machine-sink-split-probability-threshold=0 -cgp-freq-ratio-to-skip-merge=1000 | FileCheck -check-prefix=OPT %s<br>
+; RUN: llc < %s -asm-verbose=false -disable-wasm-fallthrough-return-opt -disable-wasm-explicit-locals -disable-block-placement -verify-machineinstrs -fast-isel=false -machine-sink-split-probability-threshold=0 -cgp-freq-ratio-to-skip-merge=1000 | FileCheck %s<br>
+; RUN: llc < %s -asm-verbose=false -disable-wasm-fallthrough-return-opt -disable-wasm-explicit-locals -tail-dup-placement=0 -verify-machineinstrs -fast-isel=false -machine-sink-split-probability-threshold=0 -cgp-freq-ratio-to-skip-merge=1000 | FileCheck -check-prefix=OPT %s<br>
<br>
 ; Test the CFG stackifier pass.<br>
<br>
<br>
Modified: llvm/trunk/test/CodeGen/WebAssembly/cfi.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/WebAssembly/cfi.ll?rev=339630&r1=339629&r2=339630&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/WebAssembly/cfi.ll?rev=339630&r1=339629&r2=339630&view=diff</a><br>
============</blockquote></div>