2 Commits

Author	SHA1	Message	Date
Ben Barham	5beeca7cf1	[CursorInfo] Prefer AST based results over solver based ... Solver based results are fast within a function, where the `ASTContext` can be re-used. But it is significantly slower than the AST based results when outside of a function. Instead of using solver based as the primary results, only use them as a fallback for when AST based fails. Resolves rdar://108930110.	2023-05-04 20:53:38 -07:00
Alex Hoppen	a5f85ee10c	[SourceKit] Make sure we reuse ASTContext in function bodies for solver-based cursor info ... The main problem that prevented us from reusing the ASTContext was that we weren’t remapping the `LocToResolve` in the temporary buffer that only contains the re-parsed function back to the original buffer. Thus `NodeFinder` couldn’t find the node that we want to get cursor info for. Getting AST reuse to work for top-level items is harder because it currently heavily relies on the `HasCodeCompletion` state being set on the parser result. I’ll try that in a follow-up PR. rdar://103251263	2023-02-02 22:03:25 +01:00