* Simplify codegen for struct declaration patterns
* Move box to perform the conversion only when the test succeeds

* Reject ref as an invalid modifier for accessors

* Updated test plan

* PR Comments

* Accept ref in parsing, and error out in binding

* Update test plan

* Fix build break

MemberSemanticModel.GetBoundLambdaOrQuery should use GetAdjustedNodePosition helper to ensure proper handling of zero-length tokens at the beginning of a syntax node. (#24049)

Fixes #23883.

Merging on behalf of @alrz. Thanks!

There were two separate bugs here:
    1) Fixing up the MVID of the binary was being done *after* signing,
    when it should be done before.

    2) System.Reflection.Metadata has a bug where calculating the
    checksum occurs before signing, when it should happen after.

Both of these isues have been fixed and the helper utility that checks
to see whether a binary was signed has been changed to verify that the
signature and checksums of the binaries are valid, not just that the
signed bit is set on the assembly.

Fixes #23521

Merging on behalf of @alrz. Thanks!

This change removes the following APIs:
- TypeSymbol.BaseType
- TypeSymbol.Interfaces
- TypeSymbol.AllInterfaces
- NamedTypeSymbol.TypeArguments
- TypeParameterSymbol.ConstraintTypes

These APIs simply delegate the work to APIs with "NoUseSiteDiagnostics" suffix and are not supposed to be used inside compilers. They were left in the code-base because they used to be part of the public surface and the names were nicer, but they are no longer part of the public surface.

* Pass in optional arguments by ref

* IOperation tree tests

* PR Comments

Merged it into the non-desktop specific project. Will make porting our
test projects to netcoreapp easier.

NormalizeWhitespace: no space between keyword and open paren, add newline before constraints (#23686)

* Fixes #11497 - Error on empty key path for signing

* Address PR Feedback

* Forbid 'default' as a case constant or a pattern.

Fixes #23499

* Accept PE Operator Symbols having in parameters

* Address PR Comments

* Prefer by-val methods over in methdos in overload resolution

* More tests

* Moved check to the end of OR chain

* Address PR comments

* Operators overloading

* Address PR Comments

* Clean up

Reduce the number of attributes that should be bound in order to determine if there are NoPia local types explicitly defined in code. (#23505)

Fixes #23179.

* 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

Fixes #23378.
Also, preserve and adjust arguments passed to 'new T'.

Fixes #23464