Skip to content

[IDE] Signature Help PoC #1

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

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from
Draft

Conversation

a7medev
Copy link
Owner

@a7medev a7medev commented Mar 23, 2025

This is a work-in-progress, proof-of-concept implementation of signature help in SourceKitD.

a7medev added 5 commits March 23, 2025 23:02
This is to be used in signature help for the documentation of each signature as the parameter documentation will be separate.
This doesn't currently work due to the constraint system ignoring some score increases (e.g. implicit conversion) when doing code completion and only retrieves items with the best score.
@a7medev a7medev self-assigned this Mar 23, 2025
@a7medev a7medev force-pushed the feature/signature-help-poc branch from 8e7c4c0 to 882d3b3 Compare March 25, 2025 00:38
a7medev pushed a commit that referenced this pull request Jul 8, 2025
…g#81643)

which generates IR without a llvm.trap function

All the normal CI generated this:
```
ret i32 %1
}

; Function Attrs: cold noreturn nounwind memory(inaccessiblemem: write)
declare void @llvm.trap() #1

attributes #0 = { "frame-pointer"=
```
But the test-simulator CI doesn't for some reason, so just check for the
closing brace instead.
a7medev pushed a commit that referenced this pull request Jul 8, 2025
…g#81643)

which generates IR without a llvm.trap function

All the normal CI generated this:
```
ret i32 %1
}

; Function Attrs: cold noreturn nounwind memory(inaccessiblemem: write)
declare void @llvm.trap() #1

attributes #0 = { "frame-pointer"=
```
But the test-simulator CI doesn't for some reason, so just check for the
closing brace instead.
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.

1 participant