Skip to content

[Runtime] Add the pointer location to the "failed to look up symbolic reference" error. #73138

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

Merged

Conversation

mikeash
Copy link
Contributor

@mikeash mikeash commented Apr 19, 2024

The pointer location can be computed from the symbolic reference location and offset, which we already provide, but it's not clear that you should add them together, nor is it clear why this failure would occur. Add the location of the NULL pointer itself to the error message, and also mention that it's probably caused by a missing weak symbol.

… reference" error.

The pointer location can be computed from the symbolic reference location and offset, which we already provide, but it's not clear that you should add them together, nor is it clear why this failure would occur. Add the location of the NULL pointer itself to the error message, and also mention that it's probably caused by a missing weak symbol.
@mikeash mikeash requested a review from tbkka April 19, 2024 14:49
@mikeash mikeash requested a review from al45tair as a code owner April 19, 2024 14:49
@mikeash
Copy link
Contributor Author

mikeash commented Apr 22, 2024

@swift-ci please test

@mikeash mikeash enabled auto-merge April 22, 2024 13:03
@mikeash mikeash merged commit 720ca4b into swiftlang:main Apr 22, 2024
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