Skip to content

[5.9][CodeComplete] Offer code completion for attached macro attributes #65639

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

Conversation

ahoppen
Copy link
Member

@ahoppen ahoppen commented May 4, 2023

  • Explanation: To complete inside attached macro attributes, we need to teach code completion how to invoke the type checker for attached macro attributes. After that, everything started working.
  • Scope: Type checking of macros
  • Risk: The biggest risk is changing the parameter of ResolveMacroRequest from DeclContext * to Decl *. Doug thought that he had made the change to avoid a cycle in the request evaluator but no test are failing and he can’t remember why it’s needed. So we should be fine
  • Testing: Added test case
  • Issue: rdar://105232015
  • Reviewer: @bnbarham on [CodeComplete] Offer code completion for attached macro attributes #65572

We need to teach code completion how to invoke the type checker for attached macro attributes. After that, everything started working.

rdar://105232015
@ahoppen ahoppen added 🍒 release cherry pick Flag: Release branch cherry picks swift 5.9 labels May 4, 2023
@ahoppen ahoppen requested a review from a team as a code owner May 4, 2023 02:10
@ahoppen
Copy link
Member Author

ahoppen commented May 4, 2023

@swift-ci Please test

@ahoppen ahoppen merged commit 86fdadd into swiftlang:release/5.9 May 4, 2023
@ahoppen ahoppen deleted the ahoppen/5.9/macro-arg-completion branch May 4, 2023 23:37
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 5.9
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants