diff options
| author | Jay Phelps <hello@jayphelps.com> | 2019-08-20 15:27:58 -0500 |
|---|---|---|
| committer | Heejin Ahn <aheejin@gmail.com> | 2019-08-20 13:27:58 -0700 |
| commit | dce1fe8c16559437cae05c0dd782237474ca6082 (patch) | |
| tree | 811e57a34951ad316072d7da8838461c25fbf465 /src/passes/ConstHoisting.cpp | |
| parent | 86b8cf6c299d0189d7819cf5eabb1ea03d34ff3a (diff) | |
| download | binaryen-dce1fe8c16559437cae05c0dd782237474ca6082.tar.gz binaryen-dce1fe8c16559437cae05c0dd782237474ca6082.tar.bz2 binaryen-dce1fe8c16559437cae05c0dd782237474ca6082.zip | |
Add initial support for anyref as an opaque type (#2294)
Another round of trying to push upstream things from my fork.
This PR only adds support for anyref itself as an opaque type. It does NOT implement the full [reference types proposal](https://github.com/WebAssembly/reference-types/blob/master/proposals/reference-types/Overview.md)--so no table.get/set/grow/etc or ref.null, ref.func, etc.
Figured it was easier to review and merge as we go, especially if I did something fundamentally wrong.
***
I did put it under the `--enable-reference-types` flag as I imagine that even though this PR doesn't complete the full feature set, it probably is the right home. Lmk if not.
I'll also be adding a few github comments to places I want to point out/question.
Diffstat (limited to 'src/passes/ConstHoisting.cpp')
| -rw-r--r-- | src/passes/ConstHoisting.cpp | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/src/passes/ConstHoisting.cpp b/src/passes/ConstHoisting.cpp index 17761e765..08fede1df 100644 --- a/src/passes/ConstHoisting.cpp +++ b/src/passes/ConstHoisting.cpp @@ -91,12 +91,9 @@ private: size = getTypeSize(value.type); break; } - case v128: { - // v128 not implemented yet - return false; - } - case exnref: { - // exnref cannot have literals + case v128: // v128 not implemented yet + case anyref: // anyref cannot have literals + case exnref: { // exnref cannot have literals return false; } case none: |