tree ee4f52990e5a333186a6fa2cbd307b5fd76a9172
parent a164829754a3cc01b2086eec5dcc9599c8e08809
author Robert Findley <rfindley@google.com> 1678918880 -0400
committer Robert Findley <rfindley@google.com> 1679071997 +0000

gopls/internal/lsp/cache: evaluate imports lazily in TypeCheck

Now that we do not need a static importMap for importing, and do not
need to eagerly parse or load export data, we can evaluate imported
packages lazily during type-checking, thereby avoiding importing
packages that will not be used.

This has a mild beneficial impact on benchmarks (because iimporting is
already cheap). The other direction -- avoiding invalidating packages
that are unaffected by changes -- should have a more significant impact.

For golang/go#57987

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