Skip to content

Minor update to borrow and mutate accessors #84924

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
Merged
merged 6 commits into from
Oct 20, 2025
Merged

Minor update to borrow and mutate accessors #84924

merged 6 commits into from
Oct 20, 2025

Conversation

meg-gupta
Copy link
Contributor

This PR does the following:

  • Add @inout result convention for mutate accessors
  • Update mangling for borrow and mutate accessors
  • Support opaque values mode

@meg-gupta meg-gupta force-pushed the borrowandmutateupdate branch from 5a4a030 to fda2145 Compare October 15, 2025 19:04
@meg-gupta
Copy link
Contributor Author

@swift-ci test

@meg-gupta meg-gupta force-pushed the borrowandmutateupdate branch from fda2145 to 2709f9d Compare October 15, 2025 23:00
@meg-gupta meg-gupta marked this pull request as ready for review October 15, 2025 23:00
@meg-gupta
Copy link
Contributor Author

@swift-ci test

1 similar comment
@meg-gupta
Copy link
Contributor Author

@swift-ci test

@meg-gupta meg-gupta force-pushed the borrowandmutateupdate branch from 2709f9d to 2c61dbc Compare October 15, 2025 23:11
@meg-gupta
Copy link
Contributor Author

@swift-ci test

@meg-gupta meg-gupta requested a review from atrick October 16, 2025 19:16
@meg-gupta meg-gupta force-pushed the borrowandmutateupdate branch from 478965e to 7e65180 Compare October 16, 2025 19:34
@meg-gupta
Copy link
Contributor Author

@swift-ci smoke test

@meg-gupta meg-gupta enabled auto-merge October 16, 2025 19:35
@meg-gupta
Copy link
Contributor Author

@swift-ci smoke test macOS platform

@meg-gupta meg-gupta force-pushed the borrowandmutateupdate branch from 7e65180 to 35e89fb Compare October 17, 2025 17:14
@meg-gupta
Copy link
Contributor Author

@swift-ci smoke test

@meg-gupta meg-gupta force-pushed the borrowandmutateupdate branch from 35e89fb to 6c4dd2b Compare October 20, 2025 16:06
@meg-gupta
Copy link
Contributor Author

@swift-ci smoke test

@meg-gupta meg-gupta merged commit 00e272d into swiftlang:main Oct 20, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@xymus xymus Awaiting requested review from xymus xymus is a code owner

@jckarter jckarter Awaiting requested review from jckarter jckarter is a code owner

@AnthonyLatsis AnthonyLatsis Awaiting requested review from AnthonyLatsis AnthonyLatsis is a code owner

@hborla hborla Awaiting requested review from hborla hborla is a code owner

@slavapestov slavapestov Awaiting requested review from slavapestov slavapestov is a code owner

@xedin xedin Awaiting requested review from xedin xedin is a code owner

@kavon kavon Awaiting requested review from kavon kavon is a code owner

@eeckstein eeckstein Awaiting requested review from eeckstein eeckstein is a code owner

@rjmccall rjmccall Awaiting requested review from rjmccall rjmccall is a code owner

@tshortli tshortli Awaiting requested review from tshortli tshortli is a code owner

@atrick atrick Awaiting requested review from atrick

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

@meg-gupta