Skip to content

Fix CI for main#4486

Merged
alexcrichton merged 2 commits into
bytecodealliance:mainfrom
alexcrichton:try-fix-main-ci
Jul 20, 2022
Merged

Fix CI for main#4486
alexcrichton merged 2 commits into
bytecodealliance:mainfrom
alexcrichton:try-fix-main-ci

Conversation

@alexcrichton
Copy link
Copy Markdown
Member

@alexcrichton alexcrichton commented Jul 20, 2022

Applies fixes found in #4487 to main

@alexcrichton alexcrichton mentioned this pull request Jul 20, 2022
3 tasks
@github-actions github-actions Bot added the fuzzing Issues related to our fuzzing infrastructure label Jul 20, 2022
@github-actions
Copy link
Copy Markdown

Subscribe to Label Action

cc @fitzgen

Details This issue or pull request has been labeled: "fuzzing"

Thus the following users have been cc'd because of the following labels:

  • fitzgen: fuzzing

To subscribe or unsubscribe from this label, edit the .github/subscribe-to-label.json configuration file.

Learn more.

@alexcrichton
Copy link
Copy Markdown
Member Author

Depending on how CI goes I will update #4487. This will not pass s390x CI but I'd like to see if it passes aarch64 CI.

When emulating we already have to disable most pooling-allocator related
tests so this commit carries over that logic to the new fuzz test which
may run some configurations with the pooling allocator depending on the
random input.
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.
@alexcrichton alexcrichton changed the title Skip new table_ops test under emulation Fix CI for main Jul 20, 2022
@alexcrichton alexcrichton requested a review from cfallin July 20, 2022 20:24
@alexcrichton alexcrichton enabled auto-merge (squash) July 20, 2022 20:35
@alexcrichton alexcrichton merged commit 2127c3a into bytecodealliance:main Jul 20, 2022
@alexcrichton alexcrichton deleted the try-fix-main-ci branch July 20, 2022 21:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fuzzing Issues related to our fuzzing infrastructure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants