Skip to content

Conversation

@tshortli
Copy link
Contributor

@tshortli tshortli commented Jun 5, 2025

  • Explanation: To allow tooling to analyze use of language mode and experimental/upcoming features, add that information to the .SWIFT_TRACE file. Also, includes some NFC fixes for warnings encountered when building the compiler.
  • Scope: Affects any compilation jobs that emit a trace file.
  • Issue/Radar: rdar://152673681
  • Original PR: Frontend: Add language mode and enabled features to .SWIFT_TRACE #82032
  • Risk: Low. The change only adds new keys to the .SWIFT_TRACE json format and existing tools will ignore them. The code for generating the newly emitted information is simple and tested.
  • Testing: New tests in the compiler test suite.
  • Reviewer: @artemcm @xymus @nkcsgexi

tshortli and others added 4 commits June 5, 2025 15:09
To allow tooling to analyze use of language mode and experimental/upcoming
features, add that information to the `.SWIFT_TRACE` file.

Resolves rdar://152673681.
Move a static inline function into the cpp file.

NFC.
@tshortli tshortli requested a review from a team as a code owner June 5, 2025 22:49
@tshortli tshortli added 🍒 release cherry pick Flag: Release branch cherry picks swift 6.2 labels Jun 5, 2025
@tshortli
Copy link
Contributor Author

tshortli commented Jun 5, 2025

@swift-ci please test

@tshortli tshortli enabled auto-merge June 5, 2025 22:50
Copy link
Contributor

@nkcsgexi nkcsgexi left a comment

Choose a reason for hiding this comment

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

This should have minimum impacts on most compiler users.

@tshortli tshortli merged commit c9d5e4b into swiftlang:release/6.2 Jun 6, 2025
5 checks passed
@tshortli tshortli deleted the language-modes-and-features-in-module-trace-6.2 branch June 6, 2025 15:24
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.

3 participants