Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Better LICM #1038

Open
sunfishcode opened this issue Jul 7, 2018 · 2 comments
Open

Better LICM #1038

sunfishcode opened this issue Jul 7, 2018 · 2 comments
Labels
cranelift:E-compiler Compiler issues. cranelift:goal:optimize-speed Focus area: the speed of the code produced by Cranelift. cranelift Issues related to the Cranelift code generator

Comments

@sunfishcode
Copy link
Member

bytecodealliance/cranelift#133 and bytecodealliance/cranelift#138 started some work toward improving LICM, in particular to better handle loops that end in the middles of ebbs; see the actual PRs for more discussion.

I've now merged those patches into a branch here:

https://github.com/CraneStation/cranelift/tree/better_licm
@sunfishcode
Copy link
Member Author

I want to rethink the design of spitting blocks that contain more than two loops. It would be nice for LICM to only change the IR for preheader formation and actual hoisting, even if we need somewhat more per-loop bookkeeping.

ms2300 referenced this issue in ms2300/cranelift Aug 20, 2018
@ms2300
Fixing a couple clippy warnings : #393
cee835d
ms2300 referenced this issue in ms2300/cranelift Aug 20, 2018
@ms2300
Fixing a couple clippy warnings : #393
8a31d67
@alexcrichton alexcrichton transferred this issue from bytecodealliance/cranelift Feb 28, 2020
@alexcrichton alexcrichton added cranelift:E-compiler Compiler issues. cranelift:goal:optimize-speed Focus area: the speed of the code produced by Cranelift. cranelift Issues related to the Cranelift code generator labels Feb 28, 2020
arkpar pushed a commit to paritytech/wasmtime that referenced this issue Mar 4, 2020
@jyn514 @abrown
Add more documentation for entities, fixes bytecodealliance#1038 (byt…
59139c6 
…ecodealliance#1041)

Also fixes broken links to point to docs.rs; see
bytecodealliance/cranelift#1041 (comment),
there doesn't currently seem to be a way to link to a type in a
dependent crate.
@bjorn3
Copy link
Contributor

bjorn3 commented Feb 3, 2021

With the move from extended basic blocks to basic blocks has this become obsolete?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
cranelift:E-compiler Compiler issues. cranelift:goal:optimize-speed Focus area: the speed of the code produced by Cranelift. cranelift Issues related to the Cranelift code generator
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@alexcrichton @sunfishcode @bjorn3