Stream: git-wasmtime

Topic: wasmtime / PR #4559 ISLE: support more flexible integer c...


view this post on Zulip Wasmtime GitHub notifications bot (Jul 29 2022 at 19:11):

cfallin requested fitzgen for a review on PR #4559.

view this post on Zulip Wasmtime GitHub notifications bot (Jul 29 2022 at 19:11):

cfallin opened PR #4559 from isle-int-constants to main:

The ISLE language's lexer previously used a very primitive
i64::from_str_radix call to parse integer constants, allowing values
in the range -2^63..2^63 only. Also, underscores to separate digits (as
is allwoed in Rust) were not supported. Finally, 128-bit constants were
not supported at all.

This PR addresses all issues above:

This PR also adds a new kind of ISLE test that actually runs a driver
linked to compiled ISLE code; we previously didn't have any such tests,
but it is now quite useful to assert correct interpretation of constant
values.

<!--

Please ensure that the following steps are all taken care of before submitting
the PR.

Please ensure all communication adheres to the code of conduct.
-->

view this post on Zulip Wasmtime GitHub notifications bot (Jul 29 2022 at 19:11):

cfallin requested akirilov-arm for a review on PR #4559.

view this post on Zulip Wasmtime GitHub notifications bot (Jul 29 2022 at 20:01):

alexcrichton submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Jul 29 2022 at 21:15):

cfallin updated PR #4559 from isle-int-constants to main.

view this post on Zulip Wasmtime GitHub notifications bot (Jul 29 2022 at 21:17):

cfallin has enabled auto merge for PR #4559.

view this post on Zulip Wasmtime GitHub notifications bot (Jul 29 2022 at 21:52):

cfallin merged PR #4559.


Last updated: Dec 23 2024 at 12:05 UTC