Moves AppContext to shared location (#21734)

Delete useless ArgumentNullException message (#21728)

"Object cannot be null" does not add anything over the default "Value cannot be null"

Remove EventPipeController static constructor (#21718)

* Remove EventPipeController static constructor

* Don't cache DefaultProviderConfiguration

Moves ExceptionDispatchInfo to shared location (#21725)

* Moves ExceptionDispatchInfo to shared location

* Use struct for Exception.DispatchState, avoid TLAs in names, misc cleanup

Improve MdUtf8String (#21720)

* Move MdUtf8String::EqualsCaseSensitive to managed code

* Move MdUtf8String.ToString to safe code

* Use Encoding.UTF8.GetString

Add license, keep more of the CriticalHandle logic in the shared file

Moves CriticalHandle to shared location

Fix CoreRT build breaks

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

Readability improvement in MemoryStream (#21710)

Don't Sleep(1) in some spin-wait loops (#21722)

- In spin-wait loops that are not expected to last too long, Sleep(1) significantly delays threads from completing the operation
- From eliminating Sleep(1) in such spin-wait loops, there can be a tradeoff, better fairness vs worse throughput, because Sleep(1) removes threads from contention and in some cases fewer threads can make faster progress at the cost of delaying the Sleep(1) threads relatively significantly. Eliminating the Sleep(1) in such spin-wait loops seems to be a good tradeoff.

Remove parsing from GUID constants (#21717)

Remove Environment static constructor (#21715)

Fix CoreCLR build breaks

Delete file moved to shared partition

Moves ThrowHelper to shared location (dotnet/corert#6741)

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

Shorter ExecutionContext.Capture (#21707)

Use EventSource guid ctor for RuntimeEventSource (#21714)

Update dependency files (#21703)

* Update dependencies from https://github.com/dotnet/arcade build 1026

This change updates the following dependencies
- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.18627.1
- Microsoft.DotNet.Helix.Sdk - 1.0.0-beta.18627.1

* Update dependencies from https://github.com/dotnet/arcade build 1039

This change updates the following dependencies
- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.18629.1
- Microsoft.DotNet.Helix.Sdk - 1.0.0-beta.18629.1

Moves generic comparers to shared CoreLib (#21649)

* Moves generic comparer to shared CoreLib

* Keep Index/LastIndexOf optimizations in CoreCLR only

* Moved runtime-specific enum helper to runtime-specific partial type

Separate ThreadPool...ThreadLocals get from initalization (#21704)

* Seperate ThreadPool...ThreadLocals get from initalization

* Assert null

Move CultureInfo to shared CoreLib (#21672)

* Move thread CurrentCulture and CurrentUICulture variables to CultureInfo.

* Move CultureInfo to shared CoreLib

* Code style fixes for consistency.

* Fold down APPX code into CultureInfo.CurrentCulture/CurrentUICulture.

* Remove unnecessary parentheses around return statements.

* Remove GetCurrentUICultureNoAppX.

Fix warning causing error in warning as error (Windows) (#21697)

Remove usages of GetCurrentUICultureNoAppX from ResourceManager (#21691)

* Remove usages of GetCurrentUICultureNoAppX from ResourceManager to make GetString and GetObject internally consistent. This matches the current behaviour of CoreRT.

Replace Interop.ReadDir.cs with Interop.ReadDir.cs from System.IO.FileSystem (#21622)

* Use Interop.ReadDir.cs from FileSystem (corefx)

* Use ArrayPool

* Remove ReadBufferSize (and static cctor)

* use NameBufferSize for 256 const

* Delete ReadDir and SafeDirectoryHandle.Unix.cs

* Introduce GetDirectoryEntryFullPath, remove uint casts

* Refactor EnumerateFilesRecursively

Remove dead Guid parsing code (#21123)

* Remove dead checks in Guid parsing

* Minor cleanups in Guid

* Delete unused failureMessageFormatArgument

Update dependency files (#21686)

* Update dependencies from https://github.com/dotnet/arcade build 970

This change updates the following dependencies
- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.18621.6
- Microsoft.DotNet.Helix.Sdk - 1.0.0-beta.18621.6

* Update dependencies from https://github.com/dotnet/arcade build 970

This change updates the following dependencies
- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.18621.6
- Microsoft.DotNet.Helix.Sdk - 1.0.0-beta.18621.6

Use common copy of System.Numerics.Hashing.HashHelpers

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

Delete file moved to shared partition

Move Contracts.cs (CoreCLR version) to shared CoreLib and update code to compile. (#6734)

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

Delete file moved to shared CoreLib.

Move System/Numerics/Hashing to shared CoreLib. (dotnet/corert#6732)

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

desktop port (#21523)

+alloc lock split into SOH and LOH
+provisional mode to fix too many gen2 GCs triggered in low mem situation when the heap has heavy pinning fragmentation
+better free list usage
+premature OOM fixes
+3 new configs: GCHeapAffinitizeMask, GCHighMemPercent, GCLOHThreshold (will be documented)
YieldProcessor scaling factor is different on core due to the different implementation on core.

Update CoreClr, CoreFx to preview-27225-01, preview.18624.2, respectively (#21663)

Makes CultureInfo.get_Parent thread safe. (#21671)

Move DebugProvider.Windows to shared CoreLib. (#21657)

Fix warning causing error in warning as error (Windows) (#21656)

Use TZCNT and LZCNT for Locate{First|Last}Found{Byte|Char} (#21073)

Add Module Override Recognition to R2RDump (#21617)

This was added to help debug work with large version bubbles. The changes should allow R2RDump to have some rudimentary understanding of ENCODE_METHOD_OVERRIDE.

The big TODO here is allow loading of external dependency assemblies to be able to resolve the tokens coming after the override.

Fix ProjectN regression in `ByReference<T>` (dotnet/corert#6713)

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

Update CoreClr, CoreFx to preview-27223-02, preview.18623.1, respectively (#21658)

Add optimization in GetHashCode for Int16 and SByte (#21652)

Delete CoreLib.FixupCoreLibName (#21655)

This method doing blind find&replace of "mscorlib" with "System.Private.CoreLib". It is both incorrect (e.g. if the typename happened to contain "mscorlib" string) and inefficient (unnecessary allocations and use of reflection).

Fix ArrayStack's call to default constructor for <T>. (#21624)

* Add ArrayStack::Empty

* Add ArrayStack::BottomRef

* Delete unused `ReverseTop`.

* do not use default constructor.

Update CoreClr, CoreFx to preview-27223-01, preview.18622.2, respectively (master) (#21620)

* Update CoreClr, CoreFx to preview-27223-01, preview.18622.2, respectively

* Update issues json

Extracts public IThreadPoolWorkItem interface to shared CoreLib (#21653)

Moves CoreLib to shared location (#21654)

SpanHelpers deal with nullable reference types (#21232)

Move some Task related files to shared CoreLib partition (#21650)

Synchronize Task, Awaiter and ThreadPool implementation with CoreCLR (#6706)

Synchronized the code as much as possible with CoreCLR. The one major piece that I have excluded is tracing. I will look into doing that separately.

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

Update dependency files (#21634)

* Update dependencies from https://github.com/dotnet/arcade build 943

This change updates the following dependencies
- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.18620.6
- Microsoft.DotNet.Helix.Sdk - 1.0.0-beta.18620.6

* Update dependencies from https://github.com/dotnet/arcade build 946

This change updates the following dependencies
- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.18620.7
- Microsoft.DotNet.Helix.Sdk - 1.0.0-beta.18620.7

CoreRT-specific Object.cs changes - Fix build breaks - Apply same cleanup to all related files

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

Improve removal of dead calls to allocator helpers.

This change improves detection of allocators with side effects.
Allocators can cause side effects if the allocated object may have a finalizer.
This change adds a pHasSideEffects parameter to getNewHelper JitEE interface
method. It's used by the jit to check for allocator side effects instead of
guessing from helper ids.

Fixes #21530.

Remove dead ICLRPrivBinder methods (#21637)

* Remove dead FindAssemblyBySpec

All paths returned E_FAIL

* Remove dead BinderFlags

* Remove dead VerifyBind

Correctly disable tests (#21645)

Cleanup to match conventions

Moves common part of Object to CoreLib shared

Don't mark calls to allocation helpers as CSE candidates.

Marking them as CSE candidates usually blocks CSEs rather than enables them.
A typical case is:
 [1] GT_IND(x) = GT_CALL ALLOC_HELPER
...
[2] y = GT_IND(x)
...
[3] z = GT_IND(x)
If we mark CALL ALLOC_HELPER as a CSE candidate, we later discover
that it can't be a CSE def because GT_INDs in [2] and [3] can cause
more exceptions (NullRef) so we abandon this CSE.
If we don't mark CALL ALLOC_HELPER as a CSE candidate, we are able
to use GT_IND(x) in [2] as a CSE def and do a CSE on [2] and [3].

Remove _syncRoot field from collections (#21628)

* SyncRoot

* Temp disable tests

* Disable outdated CoreFX test

* Tidier

Disable outdated CoreFX test (#21636)

Ensure that the S.N.Vector methods are marked as [Intrinsic] and that the attribute is respected by the runtime (#21601)

* Updating the SIMD intrinsics to check for the `[Intrinsic]` attribute

* Ensure that the existing Vector/Vector<T> intrinsic methods are marked as such.

* Fixing Vector<T>.DotProduct and Vector.AndNot to be treated as intrinsic

* Fixing `SIMDIntrinsicBitwiseAndNot` to swap the operands for XARCH

Fix arm64 prolog generation for register masks with holes. (#21395)

* Check that `genSaveCalleeSavedRegistersHelp` doesn't accept `REG_LR`.

There are two callers and both save `LR` with `FP` before calling this helper.
In case if somebody calls this function wth`REG_LR` it won't work because unwinding info doesn't expect holes in the reg pairs.

* Extract `genPushOrPopCalleeSavedRegisters`.

It will be used for both float and int types.

* Extend `genSaveCaleeSavedRegisterGroup` to support float.

The previous version was a copy-paste with an additional case for `REG_LR` that was unreachable. Fix that.

* Use `genSaveCaleeSavedRegisterGroup` for floats.

* Extract `genRestoreCaleeSavedRegisterGroup`.

It will be used for both int and float groups.

* Prepare `genPopCalleeSavedRegisters` to work with float.

* Use `genRestoreCaleeSavedRegisterGroup` for float.

* Check that `genRestoreCalleeSavedRegistersHelp` doesn't restore `REG_LR`.

Both callers do it later.

* Extract `CheckSPOffset`.

and move it out of loops because it works only before the first save intruction.

* Format `genRestoreCaleeSavedRegisterGroup` as `genSaveCalleeSavedRegistersHelp`.

* Extract `buildRegPairsStack` from `genSaveCaleeSavedRegisterGroup` and `genRestoreCaleeSavedRegisterGroup`.

Build a stack of registers that we want to save/restore and then iterate over it doing the actual saving/restoring.

* Extract `GetSlotSizeForRegsInMask`.

* Tolerate holes in arm64 prolog/epilog register masks.

Fixes #21363

* Reenable the test.

* Add new methods headers.

* Fix typos.

* Do not use non-const references.

* Describe `buildRegPairsStack` better.

* Change signature of `buildRegPairsStack` to avoid copyings of big structs.

* Return the logic for `RBM_LR`.

For future use.

* Clean-up some unused variables.

* Fix comments.

* Get rid of file-level functions.

* Make new methods static.

Update tiering guide to reference the new TieredCompilation label (#21618)

Moves ByReference to shared CoreLib (#21633)

Streamline default EqualityComparer and Comparer for Enums (#21604)

This borrows the implementation strategy for these from CoreRT. It makes it both simpler (fewer types and lines of code) and faster in some cases since we always use the exact right underlying type.

E.g. The following micro-benchmark is 25% faster with this change:

```
enum MyEnum : byte { x, y };

var comparer = Comparer<MyEnum>.Default;

for (int i = 0; i < 100000000; i++)
{
    comparer.Compare(MyEnum.x, MyEnum.y);
    comparer.Compare(MyEnum.y, MyEnum.x);
}
```

Updating gtGetSIMDZero to only return the NI_Base_VectorXXX_Zero node if they are supported by the compiler (#21605)

 Updating `FusedMultiplyAdd` for `System.Math` and `System.MathF` to use intrinsics when available. (#21602)

* Removing the explicit value assignment to NamedIntrinsic enum values

* Renaming NI_Math_Round/NI_MathF_Round to NI_System_Math_Round/NI_System_MathF_Round

* Updating `FusedMultiplyAdd` for `System.Math` and `System.MathF` to use intrinsics when available

Implement the remaining BMI1/2 intrinsic (#21480)

* Add tests for BMI1/2 intrinsic

* Implement the remaining BMI1/2 intrinsic

* Fix emitDispIns for BMI instruction

Add AggressiveInlining for Vector<T>.op_Multiply (#21587)

Fix "harmless" bug in ZapImport - use AppendByte to emit fixup type (#21621)

During my work on the CPAOT compiler I found out that in the past
I had believed we should use the compressed uint signature encoding
for emitting fixup types. This place may have contributed to my
mistake. I have audited all places in CoreCLR where signatures are
decoded and in all cases the fixup type is assumed to be a byte,
not a signature-compressed uint.

As I said, the bug is harmless (after all, Crossgen works) because
the enum code of READYTORUN_FIXUP_Helper is less than 128 and for
values 0-127 the compressed uint encoding is identical with the
byte value; I however believe it's useful to make this change
nonetheless for the sake of code clarity and to help avoid future
confusions similar to mine.

Fix jit warning in crossgen desktop build.

Don't morph volatile IND(ADDR(LCL_VAR)) (#20843)

Besides the fact that volatile indirections aren't supposed to be removed doing so in this case results in incorrect code being generated.

The GT_IND node has GTF_DONT_CSE set and this gets copied to the GT_LCL_VAR node. Later this prevents the insertion of a normalization cast because GTF_DONT_CSE on a GT_LCL_VAR node is assumed to mean that the variable address is being taken.

Merge pull request #21589 from CarolEidt/Fix21546

Transform SIMD8 to FIELD_LIST if promoted

Update CoreClr, CoreFx to preview-27220-03, preview.18619.1, respectively (#21592)

Freebsd symbols3 (#21607)

* enable symbols on freebsd once again

Replace more of JitHelpers with S.R.CS.Unsafe (#21598)

- Replace JitHelpers.UnsafeCastToStackPtr with Unsafe.AsPointer
- Delete PinningHelper that was duplicate of RawData helper class

Update dependency files (#21580)

* Update dependencies from https://github.com/dotnet/arcade build 844

This change updates the following dependencies
- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.18617.7
- Microsoft.DotNet.Helix.Sdk - 1.0.0-beta.18617.7

* Update dependencies from https://github.com/dotnet/arcade build 885

This change updates the following dependencies
- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.18618.7
- Microsoft.DotNet.Helix.Sdk - 1.0.0-beta.18618.7

* Update dependencies from https://github.com/dotnet/arcade build 908

This change updates the following dependencies
- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.18619.4
- Microsoft.DotNet.Helix.Sdk - 1.0.0-beta.18619.4

Fix jit warning. (#21600)

Add build timeout (#21591)

This should fix official build failures that are timing out because
the jobs are waiting to obtain an exclusive lock on the blob
feed.

Update CoreClr, CoreSetup to preview-27218-02, preview-27218-01, respectively (#21558)

Transform SIMD8 to FIELD_LIST if promoted

Fix #21546

Update named mutex PAL tests to include case of file locks being inherited by child process (#21496)

Included case fixed by https://github.com/dotnet/coreclr/pull/21458. Fixes https://github.com/dotnet/coreclr/issues/21494.

Add support for parsing field signatures (#21573)

This change adds basic field signature parsing support to R2RDump
and it improves parsing of two fixup types (FIELD_ADDRESS and
CCTOR_TRIGGER).

Thanks

Tomas

Merge pull request #21567 from CarolEidt/Issue20657

Test for #20657 SIMD byref bug

Merge pull request #21571 from BruceForstall/DisableArm32Test

Disable GitHub_19361 for Linux arm32

Fix stack walking and reporting of default interface methods (#21525)

Default interface methods in their unresolved state don't have a generic context. The generic context is only added once the method is resolved to its implementation.

Add .NET Core version to EventLog (#21566)

Generate ARM CFIs in prolog (#21505)

Fixed suppressed XML documentation errors (dotnet/corefx#34020)

* Fixed XML documentation tags for CoreFx.Private.TestUtilities

* Fixed XML documentation tags for System.Diagnostics.Process

* Fixed XML documentation tags for System.ComponentModel.Composition

* Fixed XML documentation tags for System.ComponentModel.Primitives

* Fixed XML documentation tags for System.ComponentModel.TypeConverter

* Fixed XML documentation tags for System.Data.Common

* Fixed XML documentation tags for System.Data.SqlClient

* Fixed XML documentation tags for System.Diagnostics.PerformanceCounter

* Fixed XML documentation tags for System.Diagnostics.StackTrace

* Fixed XML documentation for System.Diagnostics.TraceSource

* Fixed XML documentation for System.IO.Compression.ZipFile

* Fixed XML documentation for System.IO.Compression

* Fixed XMl documentation reference according to review notes

* Fixed XML documentation tags for System.IO.MemoryMappedFiles

* Fixed XML documentation tags for System.IO.Pipes

* Fixed XML documentation tag for OSX interop in System.Diagnostics.Process

* Fixed XML documentation tags for CoreFx.Private.TestUtilities for Unix

* Fixed XML documentation tags for System.Linq.Expressions

* Fixed XML documentation for System.Linq.Parallel

* Fixed XML documentationt ags for System.Linq

* Fixed XML documentation tags for System.Management

* Fixed XMl documentation tags for System.Net.Http (left CS1573)

* Fixed XMl documentation tags for System.Net.Mail

* Fixed XML documentation tags for System.Net.NameResolution

* Fixed XML documentation tags for System.Net.Primitives - added T prefix for Socket as Primitives do not reference System.Net.Sockets

* Fixed XML documentation tags for System.Net.Sockets

* Fixed XML documentation tags for Unix interop files

* Fixed XML documentation tags for System.Net.Http (Unix)

* Fixed XML documentation tags for PlatformDetection.Unix.cs according to review notes

* Fixed XML documentation tags for OSX interop

* Ignore CS1574 for System.Linq as there is no ConcatIterator{TSource}.ToList for uap.

* Fixed XML documentation tags for FreeBSD interop

* Fixed XML documentation tags for System.Net.WebSockets

* Fixed XMLm documentation tags for System.Transactions.Local

* Fixed XML documentation for System.Security.Principal.Windows

* Fixed XML documentation for System.Drawing.Common

* Fixed XML documentation for System.Net.NetworkInformation

* Fixed XML documentation tags for System.Private.XML (left CS1589)

* Fixed XML documentation tags for System.Security.Cryptography

* Fixed XML documentation for System.Runtime.WindowsRuntime

* Fixed XML documentation tags for System.Security.Cryptography.Xml

* Fixed XML documentation tags for System.Security.Cryptography.Primitives
CS3016 was added by mistake as part of #32934 so I removed it.

* Fixed XML documentation tags for System.Linq: added M to ToList method as it is not accessible by uap builds.

* Fixed XML documentation tags for System.IO.FileSystem

* Fixed XML documentation tags for System.Security.Cryptography.*

* Fixed XML documentation for System.IO.Ports

* Fixed XML documentation tags for System.Private.Reflection.Metadata.Ecma335

* Fixed XML documentation tags for System.Windows.Extensions

* Fixed XMl documentation tags for System.Console

* Fixed XML documentation for System.IO.FileSystem.DriveInfo

* Fixed XML documentation tags for System.Runtime.Extensions

* Fixed XML documentation tags for System.Drawing.Primitives.
Left CS1589 as there is no 'doc\KnownColor.uex' file.

* Fixed XMl documentation for System.Runtime.WindowsRuntime.
Had to add M prefix to 'Windows.Foundation.IAsyncAction' as Windows.Foundation is not accessible at compilation time.

* Fixed XML documentation tags for System.Private.Xml

* Fixed XML documentation tags for System.Private.Reflection.Metadata.Ecma335
Had to add a prefixes to: 'T:System.Collections.Immutable' and 'M:Enumerable.ToArray{TSource}' as they are not available for uap at compilation time.

* Fixed XML documentation tags for System.Net.WebSockets.WebSocketProtocol

* Fixed XML documentation tags for System.Net.WebSockets.WebSocketProtocol

* XML documentation tags fixes: changes after review.

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

Do not use CorHost2::m_fStarted to gate execution (#20953)

* ExecuteInDefaultAppDomain w/o Start

Do not require Start() call before
calling ExecuteInDefaultAppDomain()

* Remove unused variable
* Move flags as needed to CorHost2

Merge pull request #21574 from BruceForstall/AddDotnetDiagnosticsInfo

Add dotnet info to PMI asm diffs output

Merge pull request #21572 from BruceForstall/DisableArm64Test

Disable System.Runtime.Tests for Linux arm64 corefx tests

Merge pull request #21532 from BruceForstall/FixPython3Issue

Fix Python 3 issues

Add tests for marshalling a System.Drawing.Color. (#21569)

Add dotnet info to PMI asm diffs output

This might help diagnose issues.

Disable System.Runtime.Tests for Linux arm64 corefx tests

Failure with System.Tests.ExceptionTests.ThrowStatementDoesNotResetExceptionStackLineOtherMethod
in JitStress=2.

Tracking: #21223

Disable GitHub_19361 for Linux arm32

It currently times out and fails on Debug.

Tracking: #20232

Switch to using resources in BenchmarksGames tests (#21554)

Embed the following test files as the resources in their corresponding test assemblies:

* regexdna-input25.txt regexdna-input25000.txt in BenchmarksGame/regex-redux

* knucleotide-input.txt knucleotide-input-big.txt in BenchmarksGame/k-nucleotide

* revcomp-input25.txt revcomp-input25000.txt in BenchmarksGame/reverse-complement

Test for #20657 SIMD byref bug

Delete dead code (#21562)

- Delete clix support from PAL. "clix" was launcher in the Rotor days - like "dotnet" today.
- Delete URL parsing from PALRT

Update dependency files (#21560)

* Update dependencies from https://github.com/dotnet/arcade build 802

This change updates the following dependencies
- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.18615.1
- Microsoft.DotNet.Helix.Sdk - 1.0.0-beta.18615.1

* Update dependencies from https://github.com/dotnet/arcade build 817

This change updates the following dependencies
- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.18616.5
- Microsoft.DotNet.Helix.Sdk - 1.0.0-beta.18616.5