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

[build] Fix AOT no-write-symbols test #6900

Closed
wants to merge 1 commit into from
Closed

[build] Fix AOT no-write-symbols test #6900

wants to merge 1 commit into from

Conversation

pjcollins
Copy link
Member

No description provided.

@pjcollins
Copy link
Member Author

This change doesn't seem to be needed when using our own ld when LLVM is enabled (see #6901). If that approach is viable than this PR may not be needed.

@pjcollins pjcollins closed this Sep 12, 2022
@pjcollins pjcollins deleted the fix-nowrite branch September 12, 2022 14:19
@github-actions github-actions bot locked and limited conversation to collaborators Jan 24, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@dellis1972 dellis1972 Awaiting requested review from dellis1972 dellis1972 will be requested when the pull request is marked ready for review dellis1972 is a code owner

@jonathanpeppers jonathanpeppers Awaiting requested review from jonathanpeppers jonathanpeppers will be requested when the pull request is marked ready for review jonathanpeppers is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@pjcollins