Skip to content

[6.0][Macros] Remove unused dependency to SwiftOperators #74503

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
Jun 18, 2024

Conversation

rintaro
Copy link
Member

@rintaro rintaro commented Jun 17, 2024

Cherry-pick #74502 into release/6.0

  • Explanation: ObservationMacros and SwiftMacros don't use anything in SwiftOperators. Just remove the dependencies.
  • Scope: Macro plugins
  • Risk: Low. This makes those dylibs not directly link with libSwiftOperators.dylib, but any symbols in it weren't used.
  • Testing: Passes current test suite
  • Issues: N/A
  • Reviewers: Ben Barham (@bnbarham)

ObservationMacros and SwiftMacros don't use anything in SwiftOperators

(cherry picked from commit 1fb7839)
@rintaro rintaro requested a review from a team as a code owner June 17, 2024 23:13
@rintaro
Copy link
Member Author

rintaro commented Jun 17, 2024

@swift-ci Please test

@rintaro rintaro enabled auto-merge June 18, 2024 00:36
@rintaro rintaro merged commit 3637533 into swiftlang:release/6.0 Jun 18, 2024
5 checks passed
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