Fix nullable annotation on IEnumerator.Current (#37969)

For the non-generic IEnumerator, we don't have any idea whether the type contains nulls or not.  If we're forced to annotate it, `?` is correct, because it says "it's possible this is null" and we can't prove it's not.  But that ends up leading to lots of spurious warnings, in particular when enumerating over supplied collections that won't ever contain null, e.g. enumerating a CookieCollection.  Since the non-generic IEnumerator is effectively legacy, our best path forward while maintaining correctness and avoiding causing unnecessary warnings is to simply not annotate `IEnumerator.Current` at all, leaving it "oblivious"; in that way, we don't make any claims about its state, and leave it up to tooling how to best convey that.

Annotate System.Threading.AccessControl for nullable reference types (#37797)

* Annotate System.Threading.AccessControl for nullable reference types

* Address PR feedback

Add int AsnWriter.Encode(Span)

Various createdump changes (#37998)

Only run interpreter jobs on internal queues (#38039)

The interpreter runs were too much for Helix to run per pr. This
moves their execution to only on the internal queues.

Fix XML .tt/.ttinclude files to match sources (#37771)

* Fix XML .tt/.ttinclude files to match sources

[Arm64] ASIMD By Element Intrinsics (#36916)

* FusedMultiplyAddByScalar

* FusedMultiplyAddBySelectedScalar

* FusedMultiplyAddScalarBySelectedScalar

* FusedMultiplySubtractByScalar

* FusedMultiplySubtractBySelectedScalar

* FusedMultiplySubtractScalarBySelectedScalar

* InsertSelectedScalar

* LoadAndInsertScalar

* LoadAndReplicateToVector128

* LoadAndReplicateToVector64

* MultiplyAddByScalar

* MultiplyAddBySelectedScalar

* MultiplyByScalar

* MultiplyBySelectedScalar

* MultiplyBySelectedScalarWideningLower

* MultiplyBySelectedScalarWideningLowerAndAdd

* MultiplyBySelectedScalarWideningLowerAndSubtract

* MultiplyBySelectedScalarWideningUpper

* MultiplyBySelectedScalarWideningUpperAndAdd

* MultiplyBySelectedScalarWideningUpperAndSubtract

* MultiplyExtendedByScalar

* MultiplyExtendedBySelectedScalar

* MultiplyExtendedScalarBySelectedScalar

* MultiplyScalarBySelectedScalar

* MultiplySubtractByScalar

* MultiplySubtractBySelectedScalar

* StoreSelectedScalar

Fix MemoryCache Trim test (#38007)

Nullable: System.Xml, part 4 (schema) (#37724)

* Nullable: System.Xml, part 4 (schema)

Improve Crossgen2 compiler perf measurement (#37839)

Capture the full execution of Crossgen2 with ETW events by measuring command line parsing and moving the point we begin `Loading` measurement to the start of `Program.Run`. This accounts for about 300ms of time we were not measuring previously.

* Add ETW start/stop events around the command line parsing so we can time it and have a number to improve
* Fix up r2rtest `--measure-perf` switch which I broke when I renamed the graph provider and also add the new event.
* Move easily factorable pieces of `Run` into subroutines.

On small compile inputs such as a "Hello World" console app, a surprising amount of time is spent parsing the input arguments in the `System.CommandLine` library. Example perf of helloworld CG2 compilation:
```
Command line processing time: 192.46 ms
Total average compilation time: 594.46 ms
Phase breakdown (average):
    Loading time: 350.26 ms
    Graph processing time: 136.67 ms
        Added 108 nodes to mark stack
        Dependency analysis time: 57.38 ms
        Wall clock JIT time: 71.09 ms
        Total JIT time: 136.93 ms (sum of all threads)
        2 methods JITed
    Emitting time: 103.80 ms
```

[interp] Switch locals and execution stack order (#37927)

The order before was execution stack, vtstack, locals. The order now is locals, vtstack, execution stack. The reason for this is that in the future we would like to access arguments as locals, so the args pushed by the caller will end up being locals in the new frame. Additionally, this enables us to further compact the stack use in the future since a new frame will be allocated immediately following the current stack top. This also can also make calls up to 10% faster.

Co-authored-by: BrzVlad <BrzVlad@users.noreply.github.com>

[libraries] Add ConditionalFact to wasm failing tests (#37822)

* [libraries] Add IsThreadingSupported

* [libraries] Add IsThreadingSupported attribute to failing wasm tests

* [libraries] Add IsObdcNotAvailableAndThreadingSupported property to DependencyCheckTest

* [libraries] Include system namespace in RedactedLogValueIntegrationTest and fix DependencyCheckTest ConditionalFact

* [libraries] Revert System.Console IsThreadingSupported changes

* [libraries] Remove IsThreadingSupported attribute from EncodingCodePages

Co-authored-by: Mitchell Hwang <mitchell.hwang@microsoft.com>

Add proposal for breaking changes (#37779)

* Add proposal for breaking changes

Co-authored-by: Stephen Toub <stoub@microsoft.com>
Co-authored-by: Jan Kotas <jkotas@microsoft.com>
Co-authored-by: Dan Moseley <danmose@microsoft.com>

Merge pull request #38012 from maryamariyan/bugfix

Makes DI InternalsVisibleTo MicroBenchmarks in perf repo

Add 2 more areas to area-owners (#37922)

* 2 more areas

* typo

* merge

Disable mono perf jobs. (#38030)

Drains the pool: https://github.com/dotnet/core-eng/issues/10070

Address edge scenarios with JsonSerializer's property visibility (#37720)

* Ignore properties that were hidden in a more derived type

* Update EnumConverterTests comment

* Add more tests and align with Newtonsoft.Json behavior

* Clean up using statement

Merge pull request #37968 from akoeplinger/sqlclient-packagereference

Essentially reverts #31850 and fixes the underlying issue, which was that alpine-arm64 was missing from the RID graph.

This allows us to switch to PackageReference for SqlClient as the old approach doesn't work for mobile targets where we do self-contained publishing since we don't have the SqlClient dll in the in-tree runtime pack (and we don't want to put it there).
While working on this I also removed some unnecessary duplication between Mono and CoreCLR in runtime.depproj.

Also bumps Microsoft.DotNet.ProjectModel version in Microsoft.Extensions.DependencyModel.Tests.csproj

The older version references a non-existing assembly "System.Runtime.InteropServices.Pinvoke".

Add build instructions for pkg-config (#38017)

Update steps to rebuild System.Runtime ref source (#38020)

Fixes #37966.

Mark token based resolution APIs trimming unfriendly (#37965)

Annotate delegate APIs for trimming friendliness (#37964)

Annotate GetUninitializedObject (#37741)

We don't have a good annotation for this so going with a superset. We could choose to intrinsify it in the linker to restrict this if necessary.

delete low value tests (#38002)

Disable two more tests that use RemoteExecutor on platforms where it's not supported (#37992)

Those sneaked in with https://github.com/dotnet/runtime/pull/37528 which was merged at the same time as my PR https://github.com/dotnet/runtime/pull/37479.

removed unused local (#38000)

Remove some stale nullable suppressions / TODOs (#37978)

[browser][wasm] Default Debug Write to StdErr (#37886)

* [browser][wasm] Default Debug Write to StdErr

- COMPlus_DebugWriteToStdErr environment variable when set to `1` will write to `stderr` so it show up in the browser console log.

* Address review comment about adding a comment

* Address review comments

Add conditional compile for DEBUG build to only set debug environment variables when /p:Configuration=Debug is specified

- Make IVT MicroBenchmarks on DI to enable remaining DI tests in performance repo

Needed to enable DI tests that require internal ServiceProviderMode

[browser][wasm] Fix already disposed error (#37887)

* [browser][wasm] Fix already disposed error

* Remove CancellationTokenSource redundant disposal

* Address redundant initialization

* Address review comments bailing first think if already disposed

* Address review comment

Fix build break in System.Net.Sockets (#38006)

Use Latest Linker for our Trimming tests and reenable test (#37999)

* Use Latest Linker for our Trimming tests and reenable test

* Address feedback

* Add environment variables to the Publish command so that it won't try to use a globally installed sdk

ComWrappers isolation (#37861)

- Assign/track ID of ComWrappers used to create object/wrapper.
- Only reuse object/wrapper created by same instance for GetOrCreate*
- Rehydrate RCW on WeakReference for global ComWrapper instances only

Unix: add mode to inline Socket continuations (#37974)

* Unix: add mode to inline Socket continuations

On Unix, socket continuations are dispatched to the ThreadPool
from an event thread. This avoids continuations blocking the
event handling.

This adds an option to disable that dispatch. Continuations for
socket operations will be executed on the event thread directly.
This removes the overhead of context switching to the ThreadPool.

Currently this is implemented as an application level switch for
benchmarking, and experimentation. It may be made controllable
at the Socket level.

To avoid the event threads being a bottleneck, ProcessorCount
event threads are created.

* Add assert to verify Error is filtered out

* Add internal Socket.PreferInlineCompletions

* Add envvar to control PreferInlineCompletions default

* PR feedback

* Squash DOTNET_SYSTEM_NET_SOCKETS_INLINE_COMPLETIONS envvars

* Don't use Unix specific SocketAsyncEngine in common SafeHandle code

* PR feedback

Use sync IO when the physical path is available (#37846)

* Use sync IO when the physical path is available
- This should remove the overhead of doing async io for the synchronous configuration system

Disable GenericArraySortHelperTest to fix CI (#37997)

reduce use of TRACE_VERBOSE in SslStream (#37917)

* reduce use of TRACE_VERBOSE

* add NetEventSource.Verbose

Co-authored-by: Tomas Weinfurt <furt@Shining.local>

Adding linker correctness tests to test library annotations for the linker (#37618)

Exclude the AdvSimd_Part* tests for mono (#37979)

Enable startup events over EventPipe (#36720)

* Add PauseOnStart inside ceemain
* Add DOTNET_ env vars for opting out of pause on start
* Add Resume command to Diagnostics IPC

Bump Microsoft.DotNet.ProjectModel version in Microsoft.Extensions.DependencyModel.Tests.csproj

The older version references a non-existing assembly "System.Runtime.InteropServices.Pinvoke".

Use PackageReference for System.Data.SqlClient again

The old approach doesn't work for mobile targets where we do self-contained publishing since we don't have the SqlClient dll in the in-tree runtime pack (and we don't want to put it there).
While working on this I also removed some unnecessary duplication between Mono and CoreCLR in runtime.depproj.

Add alpine-arm64 to the RID graph

It was missing and caused a wrong RID graph to be generated.

Use DynamicDependencyAttribute (#37780)

* Use DynamicDependencyAttribute

* PR feedback

- Use typeof in more places
- Add comments about DynamicallyAccessedMembers
- Add comments about cases where we work around linker limitations
- Also move AssemblyBuilder`s DynamicDependency closer to where it
is actually needed.

* Add issue links

* Include internal attribute for OOB packages

On down-level platforms where DynamicDependencyAttribute doesn't exist
in corelib, this includes it in the OOB assemblies.

Help illinker remove all ICU dependencies in Invariant mode (#37832)

* Help illinker remove all ICU dependencies in Invariant mode

Data for browser-wasm hello world
|Mode  | SPC Size | Delta
|-|-|-|
|Default| 1449 KB | |
|Invariant| 1345 KB| -50 KB |

* Review feedback

[eng] Build Mono Arm64 Release on runtime test builds (#37916)

This change will build Mono on Arm64 if the runtime tests are changed. This is required so that the runtime tests will run on arm64 if the tests are changed.

Fix a bug in 64-bit signed integer division. (#37570)

Calculation of `absNc` involves adding the sign bit of the denominator. It incorrectly used bit 31 of the denominator instead of bit 63.

Disable test on mono interpreter (#37957)

Which relies on a particular locals init behavior. Fixing this naively right now could have a noticeable perf regression, so we should postpone fixing this, until we have a better optimization pass design.

Move the Buffer.Memmove<T> implementation into the runtime specific files since it requires a runtime specific BulkMoveWithWriteBarrier icall. (#37920)

The mono version has an extra type argument since it uses copy functions which are type specific.
Also add an elementCount > 0 check to avoid icall overhead if possible.

Allow X509Certificates to be trimmed in HttpClientHandler. (#37909)

* Allow X509Certificates to be trimmed in HttpClientHandler.

When DangerousAcceptAnyServerCertificateValidator is never used, the backing static field won't be trimmed by the ILLinker. This will keep empty X509Certificate2 and X509Chain types around, which is the only thing in the X509Certificates assembly.

See https://github.com/mono/linker/issues/1270

[crossgen2][arm32]Fix nonGcOffset (#37980)

Fix DomainLocalModuleNormalDynamicEntryOffsetOfDataBlobArm offset for public
variables in class.

Fixes dotnet#37161

Signed-off-by: Timur <t.mustafin@partner.samsung.com>

Strip the ILLinkTrim.xml file from the System.Private.Xml.Linq assembly (#37935)

Use Microsoft prefix for merge conflict SDK entry (#37948)

Use the Microsoft prefix which is reserved on NuGet for Microsoft employees.

2 small CQ optimizations. (#37967)

* `REF(0)==INT(0)` for reg reuse.

* Avoid field by field init when not profitable.

Fix generic delegate and MD Address method handling (#37937)

- Fix delegates to generic methods defined in the same compilation unit
- Fix usage of taking the address of a multidimensional array
- Pull the generics test from CoreRT into the ready to run generics test

Fixes issue #31654

Reserve return reg for cross-reg-file copy (#37863)

Fix #37829

Add JIT stress mode testing using libraries tests (#36486)

* Add JIT stress mode testing using libraries tests

There are three separate pipelines, with different sets of JIT
stress modes enabled for each:

- libraries-jitstress.yml
- libraries-jitstress2-jitstressregs.yml
- libraries-jitstressregs.yml

The live built Release libraries are used with a Checked CoreCLR.

The non-stress configuration is tested during normal PR testing.

The libraries `sendtohelix.proj` file is changed to take an optional
list of CoreCLR stress scenarios to run. If given, a set of environment
setting scripts is created and added to the Helix correlation payload.
Then, an auxiliary `sendtohelixhelp.proj` file is invoked for each
scenario (or just the default, empty scenario) to create and run the
Helix tasks.

* Remove remnants of old corefx-on-coreclr test run infrastructure

* Rename corefx-*.yml to libraries-*.yml

* Disable tests

```
src/libraries/System.Numerics.Vectors/tests/Matrix3x2Tests.cs
https://github.com/dotnet/runtime/issues/36587
Matrix3x2CreateRotationCenterTest()

https://github.com/dotnet/runtime/issues/36587
Matrix3x2CreateScaleCenterTest1()

https://github.com/dotnet/runtime/issues/36587
Matrix3x2CreateScaleCenterTest3()

src/libraries/System.Numerics.Vectors/tests/Matrix4x4Tests.cs
https://github.com/dotnet/runtime/issues/36586
Matrix4x4CreateFromAxisAngleTest()
```

* Fix run-test-job.yml

* Fix BuildAllConfigurations case

Annotate Microsoft.Extensions.DependencyInjection.Abstractions for nu… (#37488)

* Annotate Microsoft.Extensions.DependencyInjection.Abstractions for nullable reference types
Contributes to https://github.com/dotnet/runtime/issues/2339

Optimize SpanHelpers.IndexOfAny() for byte (#37934)

Implement hdt_get_function_pointer (#37696)

[wasm] Make some System.Console test pass with accordance to WASM PNSE behavior (#37645)

* Add a separate wasm-specific test

* Address feedback

Replace Create(ulong) with CreateScalar(uint) in PopCount (#37912)

* Replace Create(ulong) with CreateScalar(uint)
* Add CreateScalar() in shim

Fix JavaScript core object creation and interop binding. (#37885)

-  Missing parameter for ownsHandle.

improve correlation for EventSource events in SslStream (#36633)

* improve corelation for EventSource events in SslStream

* fix UnitTests build

* feedback from review

* add line

* add missing reference

Co-authored-by: Tomas Weinfurt <furt@Shining.local>

Use jeffhandley (GitHub handle) for Lead column (#37931)

Implement timers for wasm. (#37811)

Fix detection of static readonly field re-initialization via reflection (#37849)

Move the check for static readonly field re-initialization after the static constructor is triggered.

Fixes #37796

Bundler: Fix a path computation.

The bundler recognizes json config files based on the input host-name as follows:

baseName = Path.GetFileNameWithoutExtension(hostName).
depsJsonName = baseName + "deps.json"
On Linux, since executables don't have an .exe extension, this logic is incorrect for apps whose name contains a . in their name.
For example, if hostName is Some.App, depsJson is incorrectly computed as Some.deps.json instead of Some.App.deps.json.

This change fixes the problem.

Improve performance of Enum.CompareTo (#37845)

Enum.CompareTo was rewritten in C# recently. This set of fixes gets the performance on par with the previous C++ implementation

- Add optimization for obj1.GetType() == obj2.GetType() pattern the JIT
- Optimize FCall used by Enum.CompareTo

Fixes https://github.com/DrewScoggins/performance-2/issues/525

SingleFileBundler: Fix a path computation.

The bundler recognizes json config files based on the input host-name as follows:

```C#
baseName = Path.GetFileNameWithoutExtension(hostName).
depsJsonName = baseName + "deps.json"
```

On Linux, since executables don't have an `.exe` extension, this logic is incorrect for apps whose name contains a `.` in their name.
For example, if `hostName` is `Some.App`, `depsJson` is incorrectly computed as `Some.deps.json` instead of `Some.App.deps.json`.

This change fixes the problem.

update area-SDK entries (#37908)

Updating the ARM HWIntrinsic test generation script to automatically split the tests (#37859)

* Updating the ARM HWIntrinsic test generation script to automatically split the tests

* Removing the combined Program.AdvSimd.cs and Program.AdvSimd.Arm64.cs files

* Fixing the csproj to use the file group name

* Regenerate the ARM HWIntrinsic test projects

Revert "Fix tizen rootfs build (#37891)" (#37902)

This reverts commit 279940471dacb387d78e00fd2c2dff724eafa251.

Fix CultureInfo creation with special Windows LCID values (#37834)

* Fix CultureInfo creation with special Windows LCID values

* Restrict the test to Windows 10.

Use RELOAD tree when unspilling multi-reg node (#37773)

* Use RELOAD tree when unspilling multi-reg node

Delete dead code (#37847)

.editorconfig: replace invalid `refactoring` severity (#37367)

Valid severity levels are: `error`, `warning`, `suggestion`, `silent`, `none`

https://docs.microsoft.com/en-us/visualstudio/ide/editorconfig-language-conventions?view=vs-2019#severity-levels

[browser][wasm] Fix key already exists error (#37889)

* [browser][wasm] Fix key already exists error

- During the last cleanup the key that was added for bridged object was messed up.

* Address review comments

Fix tizen rootfs build (#37891)

Update covariant returns - add support for pointers (#37454)

* Update covariant returns - add support for pointers

This change adds support for managed, unmanaged and function pointers as
return values. It also adds related tests.

[wasm] Disable aot support in release builds. (#37765)

[wasm] Update library tests for System.Runtime.InteropServices.JavaScript namespace (#37740)

* Update library tests for System.Runtime.InteropServices.JavaScript namespace

* Add using

* Make the tests wasm-specific

* Make the tests wasm-specific

* Remove commented out tests

Fix propagation of basic block flags for inlinee return expressions. (#37840)

block flags from inlinee return expressions. The assumption there
was that `fgUpdateInlineReturnExpressionPlaceHolder` is the only
place where `GT_RET_EXPR`'s are replaced with the actual nodes
from inlinees. That assumption was incorrect and this change fixes
the remaining places.

Fixes #37574.

Include libnethost.lib in apphost package (#37878)

disable crossgen2 composite scenario (#37842)

Prevent use of LINQ in managed type system (#37854)

//
    // The type system needs to be low level enough to be usable as
    // an actual runtime type system.
    //
    // LINQ is not low level enough to be allowable in the type system.
    //
    // It also has performance characteristics that make it a poor choice
    // in high performance components such as the type system.
    //

Sync shared compiler files (#37850)

Add StringSplitOptions.TrimEntries (#35740)

Also changes some existing string.Split call sites to use new API

Refactor ComponentActivator (#37784)

* Refactor ComponentActivator.

* Add ComponentActivation test for initialization from self-contained app.

* Rename tests around load_assembly_and_get_function_pointer.

* Update app ComponentActivation tests to initialize from an app.
Address PR feedback.

* Rely on PtrToStringAuto to call Uni or UTF8

Fix Vector2/4 Lerp methods (#37764)

* Fix LERP and add appropriate tests

* Fix syntax error

All tests passed :white_check_mark:

Update area owners to include lead owners column (#37760)

* prettify

* add leads

* single owner for system.runtime

* jared

* note

* Update area-owners.md

* Sort and add some more leads

* Update area-owners.md

* Add more leads

* s.r.m and s.ds

* couple more

* Fix lee/jeff

* S.DS

* Add David-Engel

* tweaks

Co-authored-by: Marek Safar <marek.safar@gmail.com>

Balance out arm jobs due to number of machines we have available (#37835)

Fixes and improvements for removal of redundant zero inits (#37786)

1. Bug fix: when processing a zero assignment to a field of a promoted struct,
check the reference count of the parent struct.

2. Bug fix: when processing a zero assignment to a promoted struct, check the
reference counts of all fields.

3. Bug fix and improvement: a dependently promoted field is initialized in the prolog
iff its parent struct is initialized in the prolog so we can remove a field zero initialization
if the parent struct is already zero initialized.

4. Improvement: keep track of explicitly zero-initialized locals so that duplicate
explicit zero initializations can be eliminated.

Fixes #37666.

Plumb interpreter support into perf runs (#37514)

* Plumb interpreter support into perf runs

* Add interpreter variable to Linux

* Fix space issue

* Pass codeGenType to final yaml

* Fix whitespace issue

* Fix auth

* Fix log name

* Fix log name

* Update HelixPreCommand steps

Android: align xharness output folder name to iOS/WASM (#37799)

Use `xharness-output` instead of `TestResults`.

[interp] Refactor ldfld from vt (#37788)

Add separate faster opcode and remove use of MINT_VTRESULT

Co-authored-by: BrzVlad <BrzVlad@users.noreply.github.com>

Fix HFA/HVA classification (#37499)

* Fix HFA/HVA classification

Fix #35144

Add System.Runtime.CompilerServices.IsExternalInit (#37763)

Huge page support for composite images loaded on Linux (#37673)

Add support for loading composite R2R images utilizing huge pages on Linux

Support is broken into 3 major portions

- Changes to the compiler to add a switch which can compile the composite image with higher than normal alignment
- Changes to the runtime to make some slight tweaks to PE file loading on Linux to support these images correctly
- Documentation on how to tie these various features together to achieve large page loading on Linux

Consistently use inline stackallocs with ValueStringBuilder (#37795)

Primarily for style consistency.