Delete unused fFullReset argument (#21814)

Avoid passing stack crawl mark unnecessarily deep in the call stack. (#21783)

* Avoid passing stack crawl mark unnecessarily deep in the call stack.

* Move stack crawl from TypeName::GetTypeManaged to RuntimeTypeHandle::GetTypeByName.

If the debugger is stopped on a garbage collection event, resuming the threads during detach should simply be setting the event (#21764)

Merge pull request #21806 from BruceForstall/CheckTestEnvCall

Check result of call to TestEnv script

Support faster null checks (#21765)

JIT: don't optimize struct copies for call args in debug or minopts (#21792)

The jit will opportunistically optimize away copies for some struct call
arguments, if that argument is a "last use" for the struct (and some other
conditions apply).

In cases like #21544 this leads to confusing debug experiences as inputs
to a call appear to be modified by the call.

Also we really should not be optimizing the code this way in debug or in
minopts codegen modes. So, block this optimization for debug and minopts.

Closes #21544.

Fix Windows arm32/arm64 testing (#21809)

* Fix Windows arm32/arm64 testing

After the conversion to runtest.py, we were not getting any
JIT or GC stress mode testing:
1. The generated test env batch file didn't have a ".bat" extension,
so Windows wouldn't execute it.
2. The Python generated temp file was locked, so couldn't be read.

Added more logging.

Specify value of __RuntimeId for non-portable Linux test builds in build-test.sh (#21777)

Update dependencies from https://github.com/dotnet/arcade build 1060 (#21726)

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

Add StringExtensions.Concat to fix Microsoft.IO.Redist assembly

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

EventPipePayloadDecoder clean-up (#21716)

* Remove ReadUnalignedGuid

* Make EventPipePayloadDecoder unconditionally little-endian

Removes AppDomain dependency from TPLETWProvider (#21709)

which always returns 1, to make code sharing easier

Skip additional cast from common EC.Run calls (#21789)

* Skip additional cast from common EC.Run calls

* Unsafe.As

Reflection performance tweaks & cleanup (#21737)

- Delete internal "Unsafe" field getters/setters. They performance was very close (within 10%) of the regular getters/setters.
- Made a few performance tweaks in reflection to make reflection faster overall. The regular field getters/setters are faster than what the unsafe ones used to be with this change.

Use ValueStringBuilder instead of StringBuilderCache in PathInternal.Windows.cs

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

Add test for StringBuilder null terminator implementation detail (#21800)

* Remove some commented out code.

* Add test verifying that we put a null terminator 2-past the end of the native buffer allocated for a StringBuilder.

Merge pull request #21803 from BruceForstall/FixTypo

Fix batch script errors running JitOptimizationSensitive tests

Check result of call to TestEnv script

If you give an illegal, not executable TestEnv script, then this
will not fail silently.

Fix batch script errors running JitOptimizationSensitive tests

Simple typo: REN instead of REM, leading to:

"The syntax of the command is incorrect."

Use Kernel32 instead of api-ms-win-core-* in DllImports in SafeFileHandle.cs

Eliminate some bounds checks in Path (#21721)

Adds portable version of EncodingTable (#21735)

* Adds portable version of EncodingTable

Most of the implementation is extracted from CoreRT

* Use string comparer directly

* Remove no longer used COMNlsInfo

* Adds localization support

* Removes FeatureCoreFxGlobalization configuration

* Remove redudant encodings look up from GetEncoding

* Keep Hashtable for nameToCodePage as it does not lock on read

* Replace locked dictionary lookup with short switch

* Include comment with msbuild task link used to generate the data file

Merge pull request #21619 from CarolEidt/FixDesktopSIMD

Fix desktop SIMD failures

Merge pull request #21776 from fiigii/fixAs

Fix Vector128/256 As<U> intrinsic recognition and exception

Improve performance of Path.ChangeExtension (#21766)

In the common case where it need to replace a non-empty extension with a non-empty extension, it currently incurs a substring without the original extension prior to then concatenating with the new extension.  This PR avoids that.

(As the Path implementation is in corelib, this uses string.FastAllocateString and then formats into it with a span; if we wanted to avoid that, string.Create could also be used.)

* Add internal String.Concat overloads for spans

* Use span-based string.Concat overloads in several places

Merge pull request #21774 from hoyosjs/juhoyosa/event_size

Add payload size case for GC events

Fix Vector128/256 As<U> intrinsic recognition and exception

Fix GC for collectible array of pure value types (#21782)

For collectible arrays, we enter the mark_object_simple1 method even if
the array contains no pointers so that the reference to LoaderAllocator
can be walked. But we were missing skipping the call to go_through_object
which is both not necessary and it also doesn't work correctly for
objects without pointers (the loop over the fields reports the first 8
bytes of each array element as if it was a pointer).
The fix is to skip the call to go_through_object for objects without
pointers in the mark_object_simple1 and background_mark_simple1.

Reduce allocation in Index/Range.ToString (#21755)

* Reduce allocation in Index/Range.ToString

* Address PR feedback

Remove vc-runtime package dependency (#21643)

Bring Debug CRT DLLs when build IJW tests (#21699)

Removes the code moved to shared partition

Moves Attribute instance bits to shared location (dotnet/corert#6747)

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

Disable the [Intrinsic] check for desktop SIMD

Fix desktop SIMD failures

Fix ISA checking to work with desktop.
Also fix Value Numbering jitdump to not assert for struct types without a handle.

Cleanup current culture handling in the unmanaged runtime (#21706)

Large portion of the current culture handling in the unmanaged runtime inherited from desktop has been no-op. The nativeInitCultureAccessors QCall that it used to depend on desktop got (almost) never called in CoreCLR.

- Delete resetting of current culture on threadpool threads. It was needed in desktop because of a very tricky flow of current culture between appdomains. It is superseded by the flowing the current culture via AsyncLocal in CoreCLR.
- Comment out fetch of managed current culture for unmanaged resource lookup. It has number of problems that are not easy to fix. We are not localizing the unmanaged runtime currently anyway, so it is ok to just comment it out.
- Fix the rest to call CultureInfo directly without going through Thread.CurrentThread

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)