Skip to content

[6.0] Disable in-process target-info and supported-features queries using libSwiftScan #1745

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 1 commit into from
Dec 9, 2024

Conversation

artemcm
Copy link
Contributor

@artemcm artemcm commented Dec 3, 2024

When building the compiler toolchain, there is currently a race condition which makes it so that the 'libSwiftScan.dylib' is not ready by the time the driver is used to begin compilation of the Swift standard library. For now, disable in-process queries altogether.

On 'main', this is resolved with swiftlang/swift#77606, which seems to be a challenge to backport to 6.0.

…ibSwiftScan

When building the compiler toolchain, there is currently a race condition which makes it so that the 'libSwiftScan.dylib' is not ready by the time the driver is used to begin compilation of the Swift standard library. For now, disable in-process queries altogether.

On 'main', this is resolved with swiftlang/swift#77606, which seems to be a challenge to backport to 6.0.
@artemcm
Copy link
Contributor Author

artemcm commented Dec 3, 2024

@swift-ci test

@etcwilde
Copy link
Contributor

etcwilde commented Dec 3, 2024

@swift-ci Please Build Toolchain Ubuntu 24.04

@shahmishal
Copy link
Member

@swift-ci Please Build Toolchain Ubuntu 24.04

This trigger is only supported on swiftlang/swift repo.

@artemcm
Copy link
Contributor Author

artemcm commented Dec 9, 2024

@swift-ci test Linux platform

@artemcm artemcm marked this pull request as ready for review December 9, 2024 17:15
@artemcm artemcm merged commit 72e785e into release/6.0 Dec 9, 2024
3 checks passed
@artemcm artemcm deleted the 60_DisableInProcessTargetInfoAndFeatures branch December 9, 2024 17:15
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