tree d795f1b6fca6131f3636eb1b471aed548ab274a4
parent bc35a697886ef7c3e69cfd72d1be87a2f1d9c331
author Robert Findley <rfindley@google.com> 1680048374 -0400
committer Gopher Robot <gobot@golang.org> 1680125109 +0000

gopls/internal/lsp/cache: parse the builtin file through the cache

Likely because I assumed it was pinned to the snapshot, the builtin file
was not being parsed through the parse cache.

Completion builds a lot of builtin signatures, snapshot.BuiltinFile was
showing up as a hot spot in benchmarks and test profiles.

For golang/go#59184

Change-Id: Ic919562d2194f8f42736a4d6ea2d32e76f06e3ef
Reviewed-on: https://go-review.googlesource.com/c/tools/+/480075
Reviewed-by: Alan Donovan <adonovan@google.com>
gopls-CI: kokoro <noreply+kokoro@google.com>
Auto-Submit: Robert Findley <rfindley@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Run-TryBot: Robert Findley <rfindley@google.com>