Skip to content

[swift-api-extract] BumpPtrAllocate API structs #36727

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
merged 1 commit into from
Apr 2, 2021

Conversation

cachemeifyoucan
Copy link
Contributor

Fix a bug that pointers to ObjCInterfaces can be invalid when std vector
expended. Now allocate all the interfaces in BumpPtrAllocator so the
pointers to APIs are always valid as long as API class is alive.

rdar://76155450

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

rdar://76155450

Fix a bug that pointers to ObjCInterfaces can be invalid when std vector
expended. Now allocate all the interfaces in BumpPtrAllocator so the
pointers to APIs are always valid as long as API class is alive.

rdar://76155450
@cachemeifyoucan
Copy link
Contributor Author

@swift-ci Please smoke test

Copy link
Contributor

@cyndyishida cyndyishida left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@cachemeifyoucan
Copy link
Contributor Author

windows failure is not related.

@cachemeifyoucan cachemeifyoucan merged commit 717a132 into swiftlang:main Apr 2, 2021
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