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.

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

refactor dlsym: dispatch symbols via the normal shim mechanism #3108

Merged
merged 3 commits into from Oct 6, 2023

Conversation

RalfJung
Copy link
Member

@RalfJung RalfJung commented Oct 6, 2023

This avoids having to adjust Miri when switching between invoking the function via a linked symbol vs via dlsym.

@RalfJung
Copy link
Member Author

RalfJung commented Oct 6, 2023

@bors r+

@bors
Copy link
Collaborator

bors commented Oct 6, 2023

📌 Commit 1fe89cc has been approved by RalfJung

It is now in the queue for this repository.

@bors
Copy link
Collaborator

bors commented Oct 6, 2023

⌛ Testing commit 1fe89cc with merge c98a177...

@bors
Copy link
Collaborator

bors commented Oct 6, 2023

☀️ Test successful - checks-actions
Approved by: RalfJung
Pushing c98a177 to master...

1 similar comment
@bors
Copy link
Collaborator

bors commented Oct 6, 2023

☀️ Test successful - checks-actions
Approved by: RalfJung
Pushing c98a177 to master...

@bors bors merged commit c98a177 into rust-lang:master Oct 6, 2023
5 checks passed
@RalfJung RalfJung deleted the dlsym branch October 6, 2023 09:04
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.

None yet

2 participants