diff options
author | Dan Gohman <dan433584@gmail.com> | 2016-10-24 23:27:49 +0000 |
---|---|---|
committer | Dan Gohman <dan433584@gmail.com> | 2016-10-24 23:27:49 +0000 |
commit | 3acb187d95a8cbe1941459a9c6fb62cc5d44b1e5 (patch) | |
tree | 450f11c44d1568ef9e344e5163d34bc4d305250b /llvm/lib/Target/WebAssembly/InstPrinter | |
parent | 8b38ffaa986a06a6bdbd243b26eacc46ce9e5889 (diff) | |
download | bcm5719-llvm-3acb187d95a8cbe1941459a9c6fb62cc5d44b1e5.tar.gz bcm5719-llvm-3acb187d95a8cbe1941459a9c6fb62cc5d44b1e5.zip |
[WebAssembly] Implement more WebAssembly binary encoding.
This changes locals from being declared by the emitLocal hook in
WebAssemblyTargetStreamer, rather than with an instruction. After exploring
the infastructure in LLVM more, this seems to make more sense since
declaring locals doesn't use an encoded opcode.
This also adds more 0xd opcodes, type encodings, and miscellaneous
binary encoding bits.
llvm-svn: 285040
Diffstat (limited to 'llvm/lib/Target/WebAssembly/InstPrinter')
-rw-r--r-- | llvm/lib/Target/WebAssembly/InstPrinter/WebAssemblyInstPrinter.cpp | 26 |
1 files changed, 8 insertions, 18 deletions
diff --git a/llvm/lib/Target/WebAssembly/InstPrinter/WebAssemblyInstPrinter.cpp b/llvm/lib/Target/WebAssembly/InstPrinter/WebAssemblyInstPrinter.cpp index 9b59c7554bd..1b52bed0abe 100644 --- a/llvm/lib/Target/WebAssembly/InstPrinter/WebAssemblyInstPrinter.cpp +++ b/llvm/lib/Target/WebAssembly/InstPrinter/WebAssemblyInstPrinter.cpp @@ -154,22 +154,11 @@ void WebAssemblyInstPrinter::printOperand(const MCInst *MI, unsigned OpNo, (Desc.TSFlags & WebAssemblyII::VariableOpIsImmediate)) && "WebAssemblyII::VariableOpIsImmediate should be set for " "variable_ops immediate ops"); - - if (Desc.TSFlags & WebAssemblyII::VariableOpImmediateIsType) { - switch (Op.getImm()) { - case int64_t(WebAssembly::ValType::I32): O << "i32"; break; - case int64_t(WebAssembly::ValType::I64): O << "i64"; break; - case int64_t(WebAssembly::ValType::F32): O << "f32"; break; - case int64_t(WebAssembly::ValType::F64): O << "f64"; break; - default: llvm_unreachable("unknown local type"); - } - } else { - // TODO: (MII.get(MI->getOpcode()).TSFlags & - // WebAssemblyII::VariableOpImmediateIsLabel) - // can tell us whether this is an immediate referencing a label in the - // control flow stack, and it may be nice to pretty-print. - O << Op.getImm(); - } + // TODO: (MII.get(MI->getOpcode()).TSFlags & + // WebAssemblyII::VariableOpImmediateIsLabel) + // can tell us whether this is an immediate referencing a label in the + // control flow stack, and it may be nice to pretty-print. + O << Op.getImm(); } else if (Op.isFPImm()) { const MCInstrDesc &Desc = MII.get(MI->getOpcode()); assert(OpNo < Desc.getNumOperands() && @@ -220,9 +209,10 @@ WebAssemblyInstPrinter::printWebAssemblySignatureOperand(const MCInst *MI, case WebAssembly::ExprType::I8x16: O << "i8x16"; break; case WebAssembly::ExprType::I16x8: O << "i16x8"; break; case WebAssembly::ExprType::I32x4: O << "i32x4"; break; - case WebAssembly::ExprType::I64x2: O << "i32x4"; break; case WebAssembly::ExprType::F32x4: O << "f32x4"; break; - case WebAssembly::ExprType::F64x2: O << "f64x2"; break; + case WebAssembly::ExprType::B8x16: O << "b8x16"; break; + case WebAssembly::ExprType::B16x8: O << "b16x8"; break; + case WebAssembly::ExprType::B32x4: O << "b32x4"; break; } } |