Skip to content

Conversation

@artemcm
Copy link
Contributor

@artemcm artemcm commented May 30, 2025

Cherry-pick of #81850

  • Explanation: The Swift dependency scanner collects diagnostics as it performs a scan, including those emitted by the integrated Clang dependency scanner. Such diagnostics are then presented to the client, most-likely the swift-driver or another build system. This change adds such diagnostic handling to those emitted by the clang dependency scanner when scanning the bridging header file.

  • Scope: Specifically dependency scanning actions (as part of a build with explicitly-built modules) which involve scanning a bridging header file and emitting diagnostics during said header scan.

  • Risk: Low, the code-path this changes affects only how Clang scanner-emitted diagnostics are handled by collecting them into the scanner's diagnostic handler, like any other diagnostics, instead of just dumping them to stderr.

  • Problem: rdar://151993075

  • Reviewed By: @cachemeifyoucan

  • Original PR: [Dependency Scanning] Bridge diagnostics emitted during bridging header scanning #81850

…er scanning

Instead of simply dumping them to stderr.

Resolves rdar://151993075
@artemcm artemcm requested a review from nkcsgexi May 30, 2025 21:00
@artemcm artemcm requested a review from a team as a code owner May 30, 2025 21:00
@artemcm artemcm added 🍒 release cherry pick Flag: Release branch cherry picks swift 6.2 labels May 30, 2025
@nkcsgexi
Copy link
Contributor

This is to simply adding more diagnostic messages to an already failing build.

@artemcm
Copy link
Contributor Author

artemcm commented Jun 2, 2025

@swift-ci test

@artemcm artemcm enabled auto-merge June 2, 2025 16:01
@artemcm artemcm merged commit cf36c44 into swiftlang:release/6.2 Jun 2, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🍒 release cherry pick Flag: Release branch cherry picks swift 6.2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants