diff options
| author | Alon Zakai <azakai@google.com> | 2021-01-11 21:11:03 +0000 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-01-11 13:11:03 -0800 |
| commit | eadb53c442209962f98dde22f3b769c691398cad (patch) | |
| tree | bb7362dc6195a2bc058709e1a2b302cd49f6a9db /src/literal.h | |
| parent | fa3a74351be3c9903a95cff45dc8f903e1017461 (diff) | |
| download | binaryen-eadb53c442209962f98dde22f3b769c691398cad.tar.gz binaryen-eadb53c442209962f98dde22f3b769c691398cad.tar.bz2 binaryen-eadb53c442209962f98dde22f3b769c691398cad.zip | |
[GC] More HeapType instead of Type (#3475)
To handle both nullable and non-nullable i31s and other heap types, we cannot
just look at the isBasic case (which is just one of the two).
This may fix this issue on the release builder:
https://github.com/WebAssembly/binaryen/runs/1669944081?check_suite_focus=true
but the issue does not reproduce locally, so I worry it is something else...
Diffstat (limited to 'src/literal.h')
| -rw-r--r-- | src/literal.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/literal.h b/src/literal.h index 5b590e07f..fb5396147 100644 --- a/src/literal.h +++ b/src/literal.h @@ -279,7 +279,7 @@ public: return i32; } int32_t geti31(bool signed_ = true) const { - assert(type == Type::i31ref); + assert(type.getHeapType() == HeapType::i31); return signed_ ? (i32 << 1) >> 1 : i32; } int64_t geti64() const { |