Remove substring allocation in MethodBase.AppendParameters (#21770)

Remove unnecessary propagation of stack crawl marks. (#21781)

Merge pull request #21669 from kvochko/arm_cfi_code_offsets

[Linux/ARM] Fix CFI code offsets in funclet prolog

Refactor Send to Helix step in Azure DevOps (#21700)

* Move all the test job submission logic into separate file eng/send-to-helix-step.yml.

* Pass IsExternal=true and Creator for public submissions.

* Pass IsExternal=false and HelixAccessToken for internal submissions.

* Pass the parameters to MSBuild via environment.

* Make all the properties names passed to MSBuild "from outside" starting with underscore.

* Specify HelixArchitecture and HelixConfiguration properties.

Avoid some unnecessary allocation in MethodBuilder.ToString (#21758)

This method already allocates a ton, and it's not going to be hot-path, but it's annoying me showing up in queries for string.Concat usage, so I'm fixing it :)

Replace string.Format(SR. with SR.Format(SR. (#21763)

We use SR.Format in almost all places where we format resource strings (without a format provider), but there were a few stragglers.

Change a couple string.Replace("a", "b") to string.Replace('a', 'b') (#21768)

The latter is functionally the same but less overhead.

Use Alpine container with Node.js in Azure DevOps (#21568)

* Use Alpine container with Node.js in azure-pipelines.yml
* Enable the dependsOn build_Linux_musl_x64_release in azure-pipelines.yml

Use StringBuilderCache in PathInternal.NormalizeDirectorySeparators (#21760)

When we do need to normalize, we're currently allocating the StringBuilder/char[] for the full path length.  As long as the path length is less than the max cacheable size (360), we'll now use a cached builder.

Additional ifdef changes which are ES_BUILD_STANDALONE specific for System.Diagnostics.Tracing (#21748)

Replace Array.Copy(src, dst, int) calls with Array.Copy(src, 0, dst, 0, int) (#21756)

The former calls GetLowerBound(0) on both src and dst, which is unnecessary when the arrays are T[] and thus always have a lower bound of 0.

Remove dummy globalization implementation. (dotnet/corert#6756)

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

Avoid boxing allocation and object[] allocation in BuildTraceFileName (#21752)

Temporary remove OSX.1012.Amd64.Open and OSX.1012.Amd64 Helix queues in eng/platform-matrix.yml (#21642)

Related issue: https://github.com/dotnet/core-eng/issues/4856

Delete lstrlen from Unix PAL (#21745)

strlen/wcslen works just fine.

Remove second 'license' word for readability (#21747)

The license name already has 'license' in the end, the second one is unnecessary.

Simplify and improve integer overflow checks in Interop (#21732)

- Delete unnecessary CheckStringLength calls for result of string.Length. Managed strings are guaranteed to be under 2GB bytes, so these checks were unnecessary.
- Add `checked(...)` around buffer size computations that may hit potential integer overflow. It does not look like any of these would cause a bug that would lead to buffer overrun, but it is better to catch these early.

Update CoreClr, CoreFx to preview-27302-01, preview.19051.1, respectively (#21674)

AppContext moved to shared partition - CoreRT specific changes

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

strlen to managed code and vectorize (#21729)

Cleanup ArgumentNull_Obj references in unmanaged runtime (#21738)

Tweaks permissions ifdefs in System.Diagnostics.Tracing (#21733)

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.

Fix windows build

Fix CFI code offsets in prolog on ARM

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