Account for both parts of partial methods while computing dependent s…

UseLocalFunction should not report diagnostics on invalid decleration code

* Display compiler generated members in raw view.

* Fix dumping of PDBs in DumpAssemblyData

* wip

* FIx for 18409

* Additional fix

* remove commented out code
Co-Authored-By: Nchborl <chborl@users.noreply.github.com>

* Fix spacing issue
Co-Authored-By: Nchborl <chborl@users.noreply.github.com>

* Use default instead of 0
Co-Authored-By: Nchborl <chborl@users.noreply.github.com>

* Respond to feedback

* Add test

Move toolset to the dev16.0p2 compiler

Allow locals, parameters, range variables inside a lambda to shadow names outside the lambda (#32754)

Remove workaround for persistent errors

Allow installing on Dev16.
Allow to specify rootSuffix.
Bump min required VS version to 15.8.
Misc cleanup.

The serialization bug has popped up again. Disabling to get CI green
while Mono takes a look.

https://github.com/mono/mono/issues/10945

This version of the build task can run correctly on .NET Core 3.0. Our
current one is suffering from a private reflection bug that causes the
build tasks to crash on startup. This is the PR which previously fixed
that bug.

https://github.com/dotnet/roslyn/pull/31763

Merge pull request #32649 from jasonmalinowski/fix-metadata-references-not-always-having-file-watchers

Fix metadata references not always having file watchers

Gracefully handle documented exceptions from AssemblyMetadata.GetModules

Remove additive classifications when generating ClassifiedText from ClassifiedSpans

 Add structure provider for interpolated strings

Tweak bail out approach from unused member analyzer for struct layout types

Issue 27925 - Squiggle first line / entire parenthetical expression in remove unnecessary parentheses diagnostic (#32500)

* Update unnecessary parentheses analyzer to squiggle the entire
expression / first line.
Resolves #27925 

Fix cast exception in AbstractProject.RemoveDocument

Many of these can be called fairly often, so this avoids allocations in
the common case we don't refresh anything.

this is so that we don't get this kind of issue (https://github.com/dotnet/roslyn/issues/31928) by throwing argument exception invalid argument is given

Fix preview changes application for linked files

We have done so in past for other callsites to this API, such as #17687. This change does the same for the remaining two callsites in the IDE layer.

Fixes VSO Watson [234447](https://devdiv.visualstudio.com/DevDiv/_queries/edit/234447): [Watson] clr20r3: CLR_EXCEPTION_System.BadImageFormatException_8007000b_Microsoft.CodeAnalysis.dll!Microsoft.CodeAnalysis.PEModule.get_ReferencedAssemblies

Fixes #32216

* Remove obsolete comment.
Fixes #32315

* Add additional verifications in tests for recursive patterns.
Related to an issue in https://github.com/dotnet/roslyn/issues/32012

We invoke symbol end actions for named type symbols after symbol callbacks have been made for all its member symbols. However, we did not account for both parts of partial methods, which means we would end up with a symbol end callback for a named type prior to the partial implementation part being analyzed. This caused the remove unused member analyzer in IDE to incorrectly flag private members referenced in partial methods.

Fixes #32702

Dispose pattern based enumerators in foreach

AbstractProject.RemoveDocument was trying to support removing
ContainedDocuments, but had a misplaced cast which negated the whole
attempt.

Fixes https://devdiv.visualstudio.com/DevDiv/_workitems/edit/776782

Enable Mono testing

2 new options. NullableContextOptions and MetadataImportOptions

if compiler team do this (https://github.com/dotnet/roslyn/issues/12795), IDE side can simply use compiler's serialization rather than doing it in IDE side

Current implementation bailed out in symbol start action, which caused both analysis and reporting to be skipped for unused members in a type with StructLayout attribute. New implementation bails out in symbol end action, so we only skip reporting not analysis.

Fixes #32727

Update the Spanish_Unit_Tests test results path

https://github.com/mono/mono/issues/12603

These issues should be fixed now. Enabling the assemblise again.