Skip to content

Commit

Permalink
LibWasm: Ignore tests that check that we don't support 2+ memories
Browse files Browse the repository at this point in the history
Since we support the multi-memory proposal, we should skip tests that
validate that we have only one memory. Once multi-memory gets included
in the main WebAssembly specification (and the testsuite is updated), we
can revert this commit.
  • Loading branch information
dzfrias authored and alimpfard committed Jul 14, 2024
1 parent 9da3e29 commit 3a0f80b
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions Meta/generate-libwasm-spec-test.py
Original file line number Diff line number Diff line change
Expand Up @@ -314,6 +314,11 @@ def gen_module_command(command: ModuleCommand, ctx: Context):


def gen_invalid(invalid: AssertInvalid, ctx: Context):
# TODO: Remove this once the multiple memories proposal is standardized.
# We support the multiple memories proposal, so spec-tests that check that
# we don't do not make any sense to include right now.
if invalid.message == "multiple memories":
return
if ctx.has_unclosed:
print("});")
ctx.has_unclosed = False
Expand Down

0 comments on commit 3a0f80b

Please sign in to comment.