This is an archive of the discontinued LLVM Phabricator instance.

[WebAssembly] Run ExplicitLocals pass after CFGStackify
ClosedPublic

Authored by aheejin on Mar 21 2019, 9:59 AM.

Details

Summary

While this does not change any final output, this will greatly simplify
fixing unwind destination mismatches in CFGStackify (D48345), because we
have to create some new registers there.

Diff Detail

Repository
rL LLVM

Event Timeline

aheejin created this revision.Mar 21 2019, 9:59 AM
aheejin marked an inline comment as done.Mar 21 2019, 10:00 AM
aheejin added inline comments.
test/CodeGen/WebAssembly/cfg-stackify-dbg-skip.ll
10 ↗(On Diff #191730)

This does not mean this patch changes the final output. The reason for the change is this test stops after CFGStackify and now we run ExplicitLocals after CFGStackify.

aheejin edited the summary of this revision. (Show Details)Mar 21 2019, 10:10 AM
dschuff accepted this revision.Mar 25 2019, 3:37 PM
This revision is now accepted and ready to land.Mar 25 2019, 3:37 PM
This revision was automatically updated to reflect the committed changes.