Skip to content

NCGenerics: add test for feature flag & force-enable all #73522

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 5 commits into from
May 13, 2024

Conversation

kavon
Copy link
Member

@kavon kavon commented May 8, 2024

Ensure that when using noncopyable generics when building a module, the compiler can re-ingest it and will pick the part guarded by $NoncopyableGenerics.

verifies the concern in rdar://127701059

@kavon
Copy link
Member Author

kavon commented May 8, 2024

@swift-ci smoke test

@kavon kavon enabled auto-merge May 8, 2024 21:06
@kavon
Copy link
Member Author

kavon commented May 12, 2024

@swift-ci smoke test Linux

@kavon
Copy link
Member Author

kavon commented May 13, 2024

@swift-ci smoke test windows

@kavon
Copy link
Member Author

kavon commented May 13, 2024

@swift-ci smoke test macOS

@kavon kavon force-pushed the ncgeneric-coverage-rdar127701059 branch from 0003267 to 7799c4a Compare May 13, 2024 16:09
@kavon kavon changed the title NCGenerics: add test for feature flag NCGenerics: add test for feature flag & force-enable all May 13, 2024
@kavon
Copy link
Member Author

kavon commented May 13, 2024

@swift-ci smoke test

kavon added 3 commits May 13, 2024 10:43
Regardless of whether someone made it non-experimental, or deleted all
the checks in the compiler, really, truly, make sure the bit is set to
indicate we have the darn feature.
Ensure that when using noncopyable generics when building a module,
the compiler can re-ingest it and will pick the part guarded by
$NoncopyableGenerics.

verifies the concern in rdar://127701059
@kavon kavon force-pushed the ncgeneric-coverage-rdar127701059 branch from 7799c4a to be46fe4 Compare May 13, 2024 18:07
@kavon kavon requested review from zoecarver, hyp and egorzhdan as code owners May 13, 2024 18:07
@kavon
Copy link
Member Author

kavon commented May 13, 2024

@swift-ci smoke test

kavon added 2 commits May 13, 2024 13:08
We haven't yet solved the underlying issue in rdar://128013193 and the
workaround to make it conditionally use the better importing strategy of
Unsafe{Mutable}Pointer no longer will apply, since NoncopyableGenerics
is here.
@kavon kavon force-pushed the ncgeneric-coverage-rdar127701059 branch from be46fe4 to 89ff3fe Compare May 13, 2024 20:08
@kavon
Copy link
Member Author

kavon commented May 13, 2024

@swift-ci smoke test

@kavon kavon merged commit 3dffcc6 into swiftlang:main May 13, 2024
@kavon kavon deleted the ncgeneric-coverage-rdar127701059 branch May 13, 2024 23:31
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.

1 participant