Stream: git-wasmtime

Topic: wasmtime / issue #4167 Cranelift: eliminate dead stores


view this post on Zulip Wasmtime GitHub notifications bot (May 19 2022 at 23:22):

cfallin opened issue #4167:

In #4163, we have an alias analysis that right now we will use to eliminate redundant loads.

However, in principle we could also eliminate dead stores -- stores that are not observed and that are known to be overwritten by some other store, hence completely invisible.

This is quite a bit more complicated in a world where traps exist and precise state must be observed at the trap point (and where the store itself can trap); so we should carefully evaluate whether this is actually worth it, and whether much opportunity exists under these constraints. But it may still be worthwhile.

We will at least need to ensure that:

view this post on Zulip Wasmtime GitHub notifications bot (May 19 2022 at 23:22):

cfallin labeled issue #4167:

In #4163, we have an alias analysis that right now we will use to eliminate redundant loads.

However, in principle we could also eliminate dead stores -- stores that are not observed and that are known to be overwritten by some other store, hence completely invisible.

This is quite a bit more complicated in a world where traps exist and precise state must be observed at the trap point (and where the store itself can trap); so we should carefully evaluate whether this is actually worth it, and whether much opportunity exists under these constraints. But it may still be worthwhile.

We will at least need to ensure that:

view this post on Zulip Wasmtime GitHub notifications bot (May 19 2022 at 23:22):

cfallin labeled issue #4167:

In #4163, we have an alias analysis that right now we will use to eliminate redundant loads.

However, in principle we could also eliminate dead stores -- stores that are not observed and that are known to be overwritten by some other store, hence completely invisible.

This is quite a bit more complicated in a world where traps exist and precise state must be observed at the trap point (and where the store itself can trap); so we should carefully evaluate whether this is actually worth it, and whether much opportunity exists under these constraints. But it may still be worthwhile.

We will at least need to ensure that:

view this post on Zulip Wasmtime GitHub notifications bot (May 19 2022 at 23:22):

cfallin labeled issue #4167:

In #4163, we have an alias analysis that right now we will use to eliminate redundant loads.

However, in principle we could also eliminate dead stores -- stores that are not observed and that are known to be overwritten by some other store, hence completely invisible.

This is quite a bit more complicated in a world where traps exist and precise state must be observed at the trap point (and where the store itself can trap); so we should carefully evaluate whether this is actually worth it, and whether much opportunity exists under these constraints. But it may still be worthwhile.

We will at least need to ensure that:


Last updated: Nov 22 2024 at 17:03 UTC