jameysharp opened PR #8240 from jameysharp:resolve-egraph-aliases
to bytecodealliance:main
:
This way we can use the linear-time alias rewriting pass, and then avoid having to think about value aliases ever again.
jameysharp requested abrown for a review on PR #8240.
jameysharp requested wasmtime-compiler-reviewers for a review on PR #8240.
cfallin submitted PR review:
Nice!
jameysharp updated PR #8240.
jameysharp commented on PR #8240:
@cfallin, could you have a look at my additional changes in 70b082781c93a3a3359597e36a9f5ef5f8feeca7 and make sure they make sense?
CI informed me that values deleted due to this alias-rewrite pass were still referenced. While auditing everything in the
DataFlowGraph
for references toValue
s, I realized I think it makes sense to merge PCC facts onto the targets of value aliases. So that may affect your work on PCC, hopefully in good ways.
cfallin submitted PR review:
New changes look good!
jameysharp merged PR #8240.
Last updated: Dec 23 2024 at 13:07 UTC