Skip to content

[0.39.0] Backport CI fixes #4490

Merged
alexcrichton merged 3 commits into
bytecodealliance:release-0.39.0from
alexcrichton:backports-for-release-0.39.0
Jul 20, 2022
Merged

[0.39.0] Backport CI fixes #4490
alexcrichton merged 3 commits into
bytecodealliance:release-0.39.0from
alexcrichton:backports-for-release-0.39.0

Conversation

@alexcrichton
Copy link
Copy Markdown
Member

This includes an s390x-specific fix for constants but an additional all-platforms fix for the reftyped_vregs array.

This commit fixes an issue introduced as part of the fix for
GHSA-5fhj-g3p3-pq9g. The `reftyped_vregs` list given to `regalloc2` is
not allowed to have duplicates in it and while the list originally
doesn't have duplicates once aliases are applied the list may have
duplicates. The fix here is to perform another pass to remove duplicates
after the aliases have been processed.
This carries over a narrow fix from bytecodealliance#4427 to prior release branches. The
patch here was created by `@uweigand` during the investigation for bytecodealliance#4487.
@alexcrichton alexcrichton requested a review from cfallin July 20, 2022 20:34
@alexcrichton
Copy link
Copy Markdown
Member Author

@uweigand mind double-checking my backport here as well? (I can't seem to request a review from you through the GitHub UI...)

@uweigand
Copy link
Copy Markdown
Member

@uweigand mind double-checking my backport here as well?

LGTM, thanks!

(I can't seem to request a review from you through the GitHub UI...)

That's probably because I don't have any commit / approval rights on this repo?

@alexcrichton alexcrichton merged commit 0b0ee24 into bytecodealliance:release-0.39.0 Jul 20, 2022
@alexcrichton alexcrichton deleted the backports-for-release-0.39.0 branch July 20, 2022 23:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants