Skip to content

Conversation

@sepy97
Copy link
Contributor

@sepy97 sepy97 commented Nov 12, 2025

rdar://164596852
This flag tells the driver to include the module in the dependency scanning, but not load it. Added test for the flag.

@sepy97 sepy97 requested review from artemcm and nkcsgexi November 12, 2025 19:33
@sepy97 sepy97 force-pushed the register-module-dependency_logic branch from f12de1f to dfbfc04 Compare November 12, 2025 19:41
@sepy97 sepy97 marked this pull request as ready for review November 12, 2025 19:41
@sepy97 sepy97 force-pushed the register-module-dependency_logic branch from dfbfc04 to 7c7ec33 Compare November 12, 2025 19:44
@sepy97 sepy97 changed the title Enabled -register-module-dependency flag Added functionality to -register-module-dependency flag Nov 12, 2025
@sepy97 sepy97 force-pushed the register-module-dependency_logic branch from 7c7ec33 to d9e38b9 Compare November 12, 2025 20:06
@sepy97 sepy97 enabled auto-merge November 12, 2025 20:07
@sepy97 sepy97 disabled auto-merge November 12, 2025 22:39
@sepy97 sepy97 force-pushed the register-module-dependency_logic branch from d9e38b9 to 17f9a6b Compare November 13, 2025 17:35
@sepy97 sepy97 requested review from artemcm and nkcsgexi November 13, 2025 18:13
rdar://164596852
This flag tells the driver to include the module in the dependency scanning, but not load it.
Added test case for the flag.
@sepy97 sepy97 force-pushed the register-module-dependency_logic branch from 17f9a6b to 9c866f6 Compare November 13, 2025 18:41
@sepy97 sepy97 requested a review from artemcm November 13, 2025 18:41
Copy link
Contributor

@artemcm artemcm left a comment

Choose a reason for hiding this comment

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

Thanks!

@sepy97 sepy97 enabled auto-merge November 13, 2025 18:45
@artemcm
Copy link
Contributor

artemcm commented Nov 13, 2025

@swift-ci test

@nkcsgexi
Copy link
Contributor

@swift-ci test Windows

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