search

ykjit / yk

yk packages
https://ykjit.github.io/yk/
Other
29 stars 7 forks source link

Largely complete dead code elimination #1277

Closed ltratt closed 3 months ago

ltratt commented 3 months ago

This PR largely completes dead code elimination in the final commit (https://github.com/ykjit/yk/commit/49374b42bef49df52786d4f2c4303baaf5bc256a). On the way to doing that, I realised that I had to more carefully think about how we handle Proxy* instructions (https://github.com/ykjit/yk/commit/b47ddc07e4c2211076d3cd09e5efdde3672b4f0c). With this, I can successfully run yklua and eliminate dead code in its traces.

vext01 commented 3 months ago

Superficially looks OK to me.

ptersilie commented 3 months ago

Just some typos and a question. Otherwise looks good.

ltratt commented 3 months ago

OK to squash?

ptersilie commented 3 months ago

Please squash.

ltratt commented 3 months ago

Squashed.

ltratt commented 3 months ago

Force pushed a rebase.