Skip to content

Conversation

@cachemeifyoucan
Copy link

@cachemeifyoucan cachemeifyoucan commented Dec 23, 2025

Fix a typo in the default module search path which are added to help implicit import when using explicit module build. Also added a default path that usually can find Darwin swift shim modules.

Note when turning on implicit clang module in swift requires disable relocatable PCM checks since the search path for implicit modules can pick up an incompatible module map that defines a module with the same name that was built explicitly. Make sure we don't apply base directory relocation for explicitly loaded modules.

@cachemeifyoucan cachemeifyoucan requested a review from a team as a code owner December 23, 2025 01:05
@cachemeifyoucan
Copy link
Author

@swift-ci please test

@cachemeifyoucan cachemeifyoucan force-pushed the eng/PR-swift-ebm-default-search branch from 4705a1a to dce6935 Compare January 2, 2026 23:59
@cachemeifyoucan
Copy link
Author

@swift-ci please test

Fix a typo in the default module search path which are added to help
implicit import when using explicit module build. Also added a default
path that usually can find Darwin swift shim modules.

Note when turning on implicit clang module in swift requires disable
relocatable PCM checks since the search path for implicit modules can
pick up an incompatible module map that defines a module with the same
name that was built explicitly. Make sure we don't apply base directory
relocation for explicitly loaded modules.
@cachemeifyoucan cachemeifyoucan force-pushed the eng/PR-swift-ebm-default-search branch from dce6935 to 6f83316 Compare January 3, 2026 00:01
@cachemeifyoucan
Copy link
Author

@swift-ci please test

@cachemeifyoucan
Copy link
Author

@swift-ci please test windows platform

@adrian-prantl adrian-prantl merged commit fc2924c into swiftlang:swift/release/6.3 Jan 5, 2026
3 checks passed
@adrian-prantl
Copy link

@cachemeifyoucan Please also cherry-pick this to next.

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