Skip AttributeTests_WellKnownAttributes.ObsoleteOnVirtual_OnBase on Spanish while investigating failure.

* Produce errors on emitting attributes using default constructor of structs

* PR Comments

- Fixed build break in the DevDivInsertionFiles
- Fixed a couple of test failures

Key problem with the tests is they were being run in parallel. This is
due to using CoreClr which doesn't respect our .config based options.

Also fixed bad uses of Thread.Abort in the try finally tests which were
causing resource cleanup issues

147 failures remaining to handle

This unifies our API naming convention. When standard is in the name
then we are using the netstandard2.0 API set and when it is not then it
is the simple desktop mscorlib46.

Both variants are necessary for CompileAndVerify style functions because
many of our tests that use it can only run on desktop or simply need to
have some of the netstandard2.0 API set missing.

* Refactored synthesized attributes tests

* Remove rest of calls to CompilationUtils.GetSynthesizedAttributes()

* Fix EE tests

* Fix [Dynamic] tests

* Clean up

* Remove EmitMetadataTestBase.ValidateDeclSecurity(Compilation)

* Remove C# GetDefaultPEBuilder

* Fix Assembly Attributes tests

* Clean up

* Address PR Comments

* Clean up

* In parameters cannot have Out attributes

* PR feedback

* Follow up

* Rename Foo to Goo

* Rename ifoo to igoo

* Add back BOMs

LocalFunctionTypeParameterSymbol isn't necessary as long as the functionality around type constraints can be moved into a shared base class between ordinary method symbols and local function symbols.
In fact, it would be very useful to have a shared base symbol aside from MethodSymbol between local functions, lambdas, and other source method symbols. To this end, I've renamed SourceMemberMethodSymbol to SourceOrdinaryMethodSymbol (since it is just used for ordinary member-level methods), SourceMethodSymbol to SourceMemberMethodSymbol (since it is only used as a base class for member-level method-like functions like ordinary methods, constructors, property accessors, et al.), and added a new abstract class SourceMethodSymbol that is the root of both LocalFunctionSymbol and SourceMemberMethodSymbol. This lets us unify all source method-like symbols under a single base class, SourceMethodSymbol.
The first three commits perform the refactoring, while the last commit actually removes LocalFunctionTypeParameterSymbol and moves the necessary functionality into SourceMethodSymbol. I expect more functionality to be moved over in subsequent PRs.

Fixes #17244

Fixes #12182.

* Bug 217740: Repro and test harness for C#

* Testing and fixing loading scenario in C# and VB

* Verifying VB case using attribute on field

* Adding more tests

* Adding VB tests

* Adding tests

* More tests on fields with double-attributes

* New test constraint

* Fixing VB side to new test constraints

* Fixing C# side to new test constraints

* Fixing remaining tests

* Adding comments

* Fixing VB field scenario too

So far compiler knows about the following signatures of DeprecatedAttribute constructor.
   DeprecatedAttribute(String, DeprecationType, UInt32)
   DeprecatedAttribute(String, DeprecationType, UInt32, Platform)
   DeprecatedAttribute(String, DeprecationType, UInt32, Type)

As of TH2 and onwards  Windows SDK  uses the following signature for the DeprecatedAttribute:

     DeprecatedAttribute(String, DeprecationType, UInt32, String)

Notice the "String" at the end of the signature.
This change adds the TH2 attribute signature to the known set.

Fixes: #10630

Propagate Debugger… attributes from container of async/iterator method to the corresponding state machine type.

Fixes #4521.