Skip to content

Update to regalloc.rs 0.0.32.#3466

Merged
cfallin merged 1 commit into
bytecodealliance:mainfrom
cfallin:regalloc-0.0.32
Oct 20, 2021
Merged

Update to regalloc.rs 0.0.32.#3466
cfallin merged 1 commit into
bytecodealliance:mainfrom
cfallin:regalloc-0.0.32

Conversation

@cfallin
Copy link
Copy Markdown
Member

@cfallin cfallin commented Oct 20, 2021

This pulls in a change from regalloc.rs to address some quadratic behavior
(thanks @adamrk!).

It appears that some allocation heuristics have changed slightly since
0.0.31, so some of the golden-output filetests are updated as well.
Ideally we would rely more on runtests rather than golden-compilation
tests; but for now this is sufficient. (I'm not sure exactly what in
regalloc.rs changed to alter these heuristics; it's actually been almost
a year since the 0.0.31 release with several refactorings and tweaks
merged since then.)

Fixes #3441.

It appears that some allocation heuristics have changed slightly since
0.0.31, so some of the golden-output filetests are updated as well.
Ideally we would rely more on runtests rather than golden-compilation
tests; but for now this is sufficient. (I'm not sure exactly what in
regalloc.rs changed to alter these heuristics; it's actually been almost
a year since the 0.0.31 release with several refactorings and tweaks
merged since then.)

Fixes bytecodealliance#3441.
@github-actions github-actions Bot added the cranelift Issues related to the Cranelift code generator label Oct 20, 2021
@cfallin cfallin merged commit e043575 into bytecodealliance:main Oct 20, 2021
@cfallin cfallin deleted the regalloc-0.0.32 branch October 20, 2021 23:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cranelift Issues related to the Cranelift code generator

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Seemingly-quadratic behavior compiling function with many empty loops

2 participants