diff options
author | Heejin Ahn <aheejin@gmail.com> | 2019-11-26 02:36:38 -0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-11-26 02:36:38 -0800 |
commit | 6e23f8e5f4d28eb2056d0b3636b8317b9f299bfc (patch) | |
tree | 86072ef1f11098c4c721602c833f4fa4ebcacede /src/wasm/literal.cpp | |
parent | 98ab07f9e471e713e7b00732a84e260aaaecc25f (diff) | |
download | binaryen-6e23f8e5f4d28eb2056d0b3636b8317b9f299bfc.tar.gz binaryen-6e23f8e5f4d28eb2056d0b3636b8317b9f299bfc.tar.bz2 binaryen-6e23f8e5f4d28eb2056d0b3636b8317b9f299bfc.zip |
Print only literal values when printing literals (#2469)
Current `<<` operator on `Literal` prints `[type].const` with it. But
`[type].const` is rather an instruction than a literal itself, and
printing it with the literals makes less sense when we later have
literals whose type don't have `const` instructions (such as reference
types).
This patch
- Makes `<<` operator on `Literal` print only its value
- Makes wasm-shell's shell interface comply with the spec interpreter's
printing format (`value : type`).
- Prints wasm-shell's `[trap]` message to stderr
These make all `fix_` routines for spec tests in check.py unnecessary.
Diffstat (limited to 'src/wasm/literal.cpp')
-rw-r--r-- | src/wasm/literal.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/wasm/literal.cpp b/src/wasm/literal.cpp index 41d8d2924..3b2930c67 100644 --- a/src/wasm/literal.cpp +++ b/src/wasm/literal.cpp @@ -252,7 +252,7 @@ void Literal::printVec128(std::ostream& o, const std::array<uint8_t, 16>& v) { } std::ostream& operator<<(std::ostream& o, Literal literal) { - prepareMinorColor(o) << literal.type << ".const "; + prepareMinorColor(o); switch (literal.type) { case Type::none: o << "?"; |