Request execution queue

Merge master to features/lspClassifications

Skip test CustomizableTagsForUnnecessaryCode

Merge release/dev16.8 to master

Terminate the integration test process if cleanup fails

Avoid heavy contention in OOP reading from temporary storage

* Better handling of OOM and exceptions in EnC analyzer

* Do not trigger fail fast on OOM exception unless the process actually runs out of memory

Check if $insertionDir exists before getting children.

Merge release/dev16.8 to master

Analyzer driver performance improvements

Move DateAndTime embedded language to the features layer

AreStringValuesEqual for real

Add a cache for AnalyzerConfigSets

Ensure the Compilation's options are changed on .editorconfig changes

Implement ability to extract code into a new base class if there is no other base class already. 

* Uses shared code with similar things like Extract Interface and Pull Members Up
* Shares UI with those two features as well 

Quick info for inline parameter name hints

Diagnostics for CommitOnSave failures

There's a combination of flags to build.ps1 that doesn't do the VSSetup related stuff, so the insertion dir doesn't exist. This is used in CI that indexes Roslyn source code. Otherwise it fails the build.

Use C# 'not' pattern when available.

We were correctly producing a new CompilationOptions but failed to
update an existing Compilation if one had been computed first.

Fixes https://github.com/dotnet/roslyn/issues/46784

The change to move .editorconfig off of syntax trees meant that we were
recomputing a bunch of information again and again, since previously
the cached syntax trees implicitly acted as a cache of this data.
This adds a quick cache back to avoid some extra overhead.

This cache is a bit more expensive than it probably needs to be, but
it's not too bad as the underlying AnalyerConfigSet has some logic
to avoid recreating duplicate arrays/dictionaries of the underlying
data, so much of the data is ultimately shared in the end.