Reduce the number of intermediary IEnumerables we create while buildind indices

* ignore fixed-field check if inside nameof binder

* redo is nameof-binder strategy

* pick a good name for the test

* use existing method for determining if we are inside of a nameof node

* fix nameof(outer field) crash

* minor style tweaks

* allow nulls in VisitLValue

* harden vb VisitLValue/VisitRValue

* Revert "harden vb VisitLValue/VisitRValue"

This reverts commit edeaddd00ce310b0408b5fc6609b15f17b674654.

Just return the array from the `.Split` function as array implements `IList`

MoveFile moves existing file to root of project when renaming them.

Don't fetch IVsSolutionWorkspaceService until we actually need it

Don't create definition-entries for items that have no references and have specified DisplayIfNoReferences=false

Don't show duplicate FAR results on an object creation expression.

This PR both adds the missing branches (yield/await) for async and iterator
local functions and also incorporates the implicit branch before the first
statement of an iterator local function.

Fixes #13762

temporary turn off code lens check for this RI.

Properly calculate OverridenProperty for synthesized WithEvents overrides.

Fixes #7659.