Skip to content

Skip elements in CollectAnnotatedSymbolsPsiVisitor#2863

Open
jaschdoc wants to merge 1 commit intomainfrom
psi-visitor-skips
Open

Skip elements in CollectAnnotatedSymbolsPsiVisitor#2863
jaschdoc wants to merge 1 commit intomainfrom
psi-visitor-skips

Conversation

@jaschdoc
Copy link
Copy Markdown
Collaborator

@jaschdoc jaschdoc commented Apr 9, 2026

Related to #2816

Copy link
Copy Markdown
Collaborator

@ting-yuan ting-yuan left a comment

Choose a reason for hiding this comment

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

LGTM. Do you have any experiment of how this patch affects the performance?

Copy link
Copy Markdown
Collaborator

@bcorso bcorso left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!

@jaschdoc
Copy link
Copy Markdown
Collaborator Author

LGTM. Do you have any experiment of how this patch affects the performance?

Yes, it's only a minor improvement.

@jaschdoc
Copy link
Copy Markdown
Collaborator Author

I will wait for @jsjeon's review before merging.

@jaschdoc jaschdoc requested a review from troelsbjerre April 10, 2026 10:57
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.

3 participants