Stream: git-wasmtime

Topic: wasmtime / PR #3708 Add syntax for cold blocks to CLIF.


view this post on Zulip Wasmtime GitHub notifications bot (Jan 20 2022 at 23:46):

cfallin opened PR #3708 from cold-blocks-clif to main:

This commit adds support for denoting cold blocks in the CLIF text
format as follows:

function %f() {
block0(...):
  ...

cold block1:
  ...

cold block2(...):
  ...

block3:
  ...

With this syntax, we are able to see the cold-block flag in CLIF, we can
write tests using it, and it is preserved when round-tripping.

Fixes #3701.

<!--

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 (Jan 21 2022 at 00:38):

cfallin updated PR #3708 from cold-blocks-clif to main.

view this post on Zulip Wasmtime GitHub notifications bot (Jan 21 2022 at 00:50):

cfallin updated PR #3708 from cold-blocks-clif to main.

view this post on Zulip Wasmtime GitHub notifications bot (Jan 21 2022 at 00:51):

cfallin edited PR #3708 from cold-blocks-clif to main:

This commit adds support for denoting cold blocks in the CLIF text
format as follows:

function %f() {
block0(...):
  ...

block1 cold:
  ...

block2(...) cold:
  ...

block3:
  ...

With this syntax, we are able to see the cold-block flag in CLIF, we can
write tests using it, and it is preserved when round-tripping.

Fixes #3701.

<!--

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 (Jan 21 2022 at 00:52):

cfallin requested fitzgen for a review on PR #3708.

view this post on Zulip Wasmtime GitHub notifications bot (Jan 21 2022 at 07:30):

bjorn3 submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Jan 21 2022 at 18:19):

fitzgen submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Jan 21 2022 at 18:19):

fitzgen merged PR #3708.


Last updated: Dec 23 2024 at 13:07 UTC