-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Cranelift: eliminate dead stores #4167
Copy link
Copy link
Open
Labels
craneliftIssues related to the Cranelift code generatorIssues related to the Cranelift code generatorcranelift:goal:optimize-speedFocus area: the speed of the code produced by Cranelift.Focus area: the speed of the code produced by Cranelift.enhancement
Metadata
Metadata
Assignees
Labels
craneliftIssues related to the Cranelift code generatorIssues related to the Cranelift code generatorcranelift:goal:optimize-speedFocus area: the speed of the code produced by Cranelift.Focus area: the speed of the code produced by Cranelift.enhancement
Type
Fields
Give feedbackNo fields configured for issues without a type.
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: