Skip to content

Required changes for Golem's durable worker executor v17#2

Open
vigoo wants to merge 12 commits intowasmtime-v17.0.0-baselinefrom
golem-wasmtime-17
Open

Required changes for Golem's durable worker executor v17#2
vigoo wants to merge 12 commits intowasmtime-v17.0.0-baselinefrom
golem-wasmtime-17

Conversation

@vigoo
Copy link

@vigoo vigoo commented Feb 1, 2024

Changes applied to wasmtime v17.0.0 for the durable Golem worker executor

DO NOT MERGE

rvolosatovs and others added 2 commits February 23, 2024 13:56
…ytecodealliance#7783)

* fix: allow dynamic owned resources to be used as borrowed parameters

Signed-off-by: Roman Volosatovs <rvolosatovs@riseup.net>

* tests: add `can_use_own_for_borrow` test

Signed-off-by: Roman Volosatovs <rvolosatovs@riseup.net>

---------

Signed-off-by: Roman Volosatovs <rvolosatovs@riseup.net>
(cherry picked from commit 2c86e26)
vigoo pushed a commit that referenced this pull request Feb 27, 2026
…ealliance#12135)

* Implement simple icmp + select optimization

This optimizes the following:

a = select x, k1, k2
b = icmp eq a, k1

to

b = x

We shouldn't trigger this optimization when k1 == k2 because constant
propagation should optimize that case.

* Optimize icmp eq and ne for select-on-icmp (#2)

* Optimize icmp eq and ne for select-on-icmp

When we have a select followed by an icmp and the inner condition
to the select is also an icmp, we can remove the select + icmp and
directly use the inner condition.

We negated the values of the inner_condition via bxor x 1.

* Add trailing newline

* feat: avoid relying on icmp inner_cond

---------

Co-authored-by: school <school@schools-MacBook-Air.local>
Co-authored-by: Edward Wibowo <wibow9770@gmail.com>

* test: add extra test

---------

Co-authored-by: school <school@schools-MacBook-Air.local>
Co-authored-by: Edward Wibowo <wibow9770@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants