-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Use discarded use site info for LookupSymbols scenario #67935
Conversation
var useSiteInfo = discardUseSiteInfo | ||
? CompoundUseSiteInfo<AssemblySymbol>.Discarded | ||
: CompoundUseSiteInfo<AssemblySymbol>.DiscardedDependencies; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This may have a behavioral change, but let's see if it will be captured by tests and check whether it's an important scenario for LookupSymbols or not.
@@ -55,7 +57,7 @@ public static MethodSymbol Create(MethodSymbol method, TypeSymbol receiverType, | |||
return null; | |||
} | |||
|
|||
if (useSiteInfo.Diagnostics != null) | |||
if (!discardUseSiteInfo && useSiteInfo.Diagnostics != null) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Before we start optimizing compiler because IDE is slow, I would like to have an understanding about how many requests IDE performs for the scenario. Compiler code might dominate the profile simply because IDE is constantly asking it to do something.
Assuming this change can be accepted, would it help with #67926 ? I'd like someone to confirm. cc @AlekseyTs @sharwell @CyrusNajmabadi