Skip to content

Backport: "Cranelift: avoid quadratic behavior in label-fixup processing" to 12.0.1#6897

Merged
cfallin merged 2 commits intobytecodealliance:release-12.0.0from
TimonPost:timon/12.0.1
Aug 24, 2023
Merged

Backport: "Cranelift: avoid quadratic behavior in label-fixup processing" to 12.0.1#6897
cfallin merged 2 commits intobytecodealliance:release-12.0.0from
TimonPost:timon/12.0.1

Conversation

@TimonPost
Copy link
Contributor

This PR backports #6804 to 12.0.1.

Fixes: #6889

I am unfamiliar with backporting and followed those docs let me know if there are additional steps to follow.

cfallin and others added 2 commits August 24, 2023 10:41
…dealliance#6804)

* WIP: two-tier processing for MachBuffer veneer processing in island emission

* Unconditionally emit veneers for label forward-refs that cross islands.

* Clean up and fix tests

* Review feedback

* Add some more detailed comments.
@TimonPost TimonPost marked this pull request as ready for review August 24, 2023 09:05
@TimonPost TimonPost requested review from a team as code owners August 24, 2023 09:05
@TimonPost TimonPost requested review from cfallin and removed request for a team August 24, 2023 09:05
@github-actions github-actions bot added cranelift Issues related to the Cranelift code generator cranelift:area:machinst Issues related to instruction selection and the new MachInst backend. cranelift:area:aarch64 Issues related to AArch64 backend. wasmtime:docs Issues related to Wasmtime's documentation labels Aug 24, 2023
@alexcrichton
Copy link
Member

To confirm, @cfallin do you have any reservations about the backport here and a point release? If not I can do the motions after merging this.

Copy link
Member

@cfallin cfallin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks! @alexcrichton indeed feel free to do the release; thank you for offering!

@cfallin cfallin merged commit ee7cc00 into bytecodealliance:release-12.0.0 Aug 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cranelift:area:aarch64 Issues related to AArch64 backend. cranelift:area:machinst Issues related to instruction selection and the new MachInst backend. cranelift Issues related to the Cranelift code generator wasmtime:docs Issues related to Wasmtime's documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants