Skip to content

[IRGen] Use ConditionalDominanceScope in AbstractMetadataAccessor::emit #64585

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 2 commits into from
Mar 24, 2023

Conversation

drexin
Copy link
Contributor

@drexin drexin commented Mar 23, 2023

rdar://103179745

Under certain conditions the missing dominance scope caused a cached witness table ref to be used in a block that it was not available in.

rdar://103179745

Under certain conditions the missing dominance scope caused a cached witness table ref to be used in a block that it was not available in.
@drexin drexin requested a review from aschwaighofer March 23, 2023 20:48
@drexin
Copy link
Contributor Author

drexin commented Mar 23, 2023

@swift-ci smoke test

Copy link
Contributor

@aschwaighofer aschwaighofer left a comment

Choose a reason for hiding this comment

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

lgtm

@drexin
Copy link
Contributor Author

drexin commented Mar 23, 2023

@swift-ci smoke test

@drexin drexin merged commit bdbab35 into swiftlang:main Mar 24, 2023
@drexin drexin deleted the wip-103179745 branch March 24, 2023 01:36
@drexin drexin restored the wip-103179745 branch March 24, 2023 01:36
@drexin drexin deleted the wip-103179745 branch March 24, 2023 01:36
@drexin drexin restored the wip-103179745 branch March 24, 2023 01:36
drexin added a commit to drexin/swift that referenced this pull request Apr 13, 2023
…it (swiftlang#64585)

* [IRGen] Use ConditionalDominanceScope in AbstractMetadataAccessor::emit

rdar://103179745

Under certain conditions the missing dominance scope caused a cached witness table ref to be used in a block that it was not available in.

* Fix target in test
etcwilde pushed a commit to etcwilde/swift that referenced this pull request Apr 19, 2023
…it (swiftlang#64585)

* [IRGen] Use ConditionalDominanceScope in AbstractMetadataAccessor::emit

rdar://103179745

Under certain conditions the missing dominance scope caused a cached witness table ref to be used in a block that it was not available in.

* Fix target in test
drexin added a commit that referenced this pull request Apr 21, 2023
…it (#64585) (#65163)

* [IRGen] Use ConditionalDominanceScope in AbstractMetadataAccessor::emit

rdar://103179745

Under certain conditions the missing dominance scope caused a cached witness table ref to be used in a block that it was not available in.

* Fix target in test
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.

2 participants