Merge pull request #11172 from mskvortsov/ArithOverflowChecks

[RyuJIT/ARM32] Enable arithmetic overflow checks

Support other toolsets in Win/ARM64 build

Fix cmake warning about mismatched if/endif comment (#11176)

Remove PAL FileAlignment restriction (#10959)

* Remove PAL FileAlignment restriction

* Address PR #10959 feedback

* Fix amd64 crossgen error

* Respond to review feedback

* Fix arm32 regression

* Prepare to remove VIRTUAL_PAGE_* from map.cpp

Also simplify previous section code

* Rename function to GetVirtualPageSize()

[Local GC] Ensure that handle creation returns null on failure instead of throwing (#11092)

* [Local GC] Ensure that handle creation returns null on failure instead of throwing

* Fix some clang pedantry about jumping past variable initialization

* Throw OOM if initialization of handle store fails

* Perform the handle OOM check in each handle helper

do not use = {nullptr} initialization. (#11153)

It worked incorrectly on desktop.

Merge pull request #11169 from hseok-oh/ryujit/fix_11128

[ARM32/RyuJIT] Fix failure related with struct argument passing

Merge pull request #11091 from BruceForstall/BlockRefactor

Refactoring: move some BasicBlock functions to block.cpp

RyuJIT/ARM32: Enable arithmetic overflow checks.

Support FEATURE_DBGIPC override (#11164)

[x86/Linux] Remove unnecessary ResumeEsp field (#11167)

[ARM32/RyuJIT] Fix failure related with struct argument passing

Block using NYI_ARM when struct value split between registers and stack
Fix type assign error for struct slot

[x86/Linux] Fix unmatched NESTED_END in CallEHFilterFunclet (#11163)

11106: Use RuntimeHelpers.IsReferenceOrContainsReferences in Dictionary.Remove methods (#11158)

Mark GetGenerationWR2 as GCStressIncompatible across all platforms (#11151)

Remove destructor from GCEvent and instead rely on the OS to clean up (#11132)

* Remove destructor from GCEvent and instead rely on the OS to clean up

* Add a comment justifying the lack of destructor

* wording: many -> all

Two improvments for LCG jump stubs (dynamic methods): (#9160)

When requesting a jump stub block for LCG, ask for a block of 4 jump stubs instead of 32
    Enable caching and cache lookup for LCG jump stubs

Move xplat minidump doc to botr. (#11149)

Change default minidump type to MiniDumpWithPrivateReadWriteMemory.

Remove language about uploading triage dumps.

Turn off diagnostic output.

Move CoreCLR to Preview2 (#11146)

Merge pull request #11140 from AndyAyersMS/FixRunBenchmarks

Update jit RunBenchmarks test to no longer use XPath

Merge pull request #11148 from wtgodbe/DisableCoreMangLib

Disable DynMethodJumpStubTests test

Disable DynMethodJumpStubTests test

Merge pull request #11117 from briansull/zapsig-fix

ReadyToRun/Crossgen: Disable cross module generic instantiations from profile data

Merge pull request #11147 from dotnet/ReEnableHashCodeTest

Re-enable Hashcode Test

Re-enable Hashcode Test

Merge pull request #11144 from wtgodbe/DisableSomeTests

Temporarily disable failing tests in Helix

Temporarily disable failing tests in Helix

Update jit RunBenchmarks test to no longer use XPath

Rewrite to use linq-to-xml instead. Verified at least that this
can still parse the adjoining xml file. Also now able to crossgen
this test case.

Closes #11082.

Merge pull request #11138 from wtgodbe/WrongCrossgen

Copy Runtime-specific Crossgen to Core_Root

Copy Runtime-specific Crossgen to Core_Root

Merge pull request #11055 from jorive/dev/xUnit-Performance-API

Improving robustness of the run-xunit-perf.cmd script.

[x86/Linux] Correctly initialize nonVolRegPtrs in LazyMachState::unwindLazyState (#10888)

Mark CharEnumerator as [Serializable] (#11124)

Fixes https://github.com/dotnet/corefx/issues/18704

[x86/Linux] Fix pow of tryrun (#11099)

x86/Linux has compatible math pow()
This will fix CoreFX System.Tests.MathFTests.Pow() failures

Add Marshal.GetExceptionPointers (#11125)

* Add Marshal.GetExceptionPointers

* Remove SecurityCritical

[ARM32/RyuJIT] Argument passing for multireg struct (#10846)

* [Linux/ARM32] Enable fgMorphMultiregStructArgs

Enable fgMorphMultiregStructArgs: morph struct argument to GT_FILED_LIST when size <= 4
Cannot handle split struct that using register and stack

* Fix formatting

Fix formatting error

* Fix typo and commit with not added file

- Fix typo
- Commit with not added file: src/jit/compiler.h

* Fix formatting and build error

* Fix comment and block using NYI macro

- Fix comment
- Block using NYI when struct passed by value can be split between registers and stack

[x86/Linux] Relax inst_JMP assert condition (#11062)

* [x86/Linux] Do NOT use exception-throwing block on ESP framed method

* Fix format error

* Relax assert condition

* Fix format error

* Revert changes in codegencommon.cpp

* Add parentheses

Merge pull request #11024 from dotnet-bot/master-UpdateDependencies

Update CoreClr, CoreFx to preview1-25221-01, preview1-25221-01, respectively (master)

Update CoreClr, CoreFx to preview1-25221-01, preview1-25221-01, respectively

Update StringBuilder.AppendJoin to approved API  (#11059)

* Updated StringBuilder.AppendJoin to approved API

* Organized StringBuilder.AppendJoin overloads

* Added ThrowHelper, removed Contract.Ensures

* Inline array accesses

Enable corerun to select Globalization Invariant mode (#11097)

* Enable corerun to select Globalization Invariant mode

* Support 'true' also

Fix R2R workaround for Diasymreader (#11116)

Inline List.Clear (#11104)

When we have profile data we can have cross module references in our ZapSig's when building ReadyToRun images.
We should return a null TypeHandle to indicate failure so that we don't attempt to add generic types/methods
that have cross module references.  (i.e. MyList<System.Guid> )

Fixing Delegate's hash code's distribution (#11019)

Delegate's GetHashCode just returns the hash code of the delegate type. For a scenario where delegates are used as keys in a dictionary, this leads to obvious performance problems. We should look at coming up with a better GetHashCode that properly factors in the target object and method, in all of the various forms a delegate can take.

Merge pull request #11115 from wtgodbe/readytoruntests

Move readytorun tests into directory one level deeper

Merge pull request #11033 from adityamandaleeka/handle_table_local_gc_store

Move handle object manipulation functions to interface

Move readytorun tests into directory one level deeper

Merge pull request #11020 from adiaaida/hardenPerf

Strip quotes out of BenchviewCommitName

Rename CompareAndSwap to InterlockedCompareExchange.

Merge pull request #11086 from wtgodbe/SecurityPermissions

Add dependency on System.Security.Permissions

Add dependency on System.Security.Permissions

Add the Linux minidump support documentation (#11087)

Add the Linux minidump support documentation

One minor fix to createdump.

[x86/Linux] Use CDECL as STDAPICALLTYPE (#10969)

* [x86/Linux] Use CDECL as STDAPICALLTYPE

* Use system default calling convention for internal functions

Strip quotes out of BenchviewCommitName

When we pass the name of the pr to submission-metadata.py, we need the
name of the pr to not include quotes, or submission-metadata.py will fail
because it has the "wrong" parameters (because parts of the name are no
longer quoted). This change strips quotation marks out of the benchview
commit name to harden against this kind of failure.

[x86/Linux] fix a problem that stack was broken by changing stdcall to cdecl in case of readytorun. (#11063)

* [x86/Linux] fix a problem that stack was broken by changing stdcall to cdecl in case of readytorun.

add DynamicHelperArgsStub and change jmp to call.

* [x86/Linux] set cfi_def_cfa_offset of DynamicHelperArgsStub to 16.

Merge pull request #11083 from AndyAyersMS/FixNonDevUseOfCheckProjects

Update CheckProjects to not fail for non-dev CORE_ROOT

Refactoring: move some BasicBlock functions to block.cpp

(where they belong)

Delete unused containsStatement().

Merge pull request #11081 from mikedn/static-noway

Convert some noway_assert to static_assert

Allow to run all benchmarks and even upload, but fail the job.

Add LinkBench to CoreCLR BenchView

This change adds Linker tests as scenario tests tracked in
BenchView for CoreCLR benchmarks.

The Benchmarks added are:
HelloWorld, WebAPI, CoreFx, MusicStore (MSIL and ReadyToRun), Roslyn.

The metrics are:
MSIL size before and after ILLINK
Total publish-dir size before and after ILLINK
% Reduction in MSIL and Total dir-size after linking.

Move ResetOBJECTHANDLE and the associated holder to the VM side.

Merge pull request #11077 from pgavlin/gh10940

Spill side-effects in impAssignMultiRegTypeToVar.

Add NUMA and thread affinity support for Unix (#10938)

* Add NUMA and thread affinity support for Unix

This change adds new PAL functions for NUMA and thread affinity support
for Unix and also enables related code in GC and VM for FEATURE_PAL.

It doesn't reflect the limits imposed by CGROUPS on systems with
CGROUPS enables yet.

Merge pull request #10257 from jashook/reflection_invoke_arm64

[Arm64] For reflection invoke, use ArgLocDesc

Merge pull request #11085 from wtgodbe/SkipTests

Skip failing tests in Helix

Skip failing tests in Helix

Update CheckProjects to not fail for non-dev CORE_ROOT

CheckProjects expects to be able to find the jit test project files
relative to the value of the `CORE_ROOT` environment variable. This
fails when we have test setups like cross-tests or helix where the
test binary tree has been copied away from the repo after building.

Tolerate these cases by returning a success code if the `CORE_ROOT`
path is not pointing at the expected place.

Fixes #11079.
Fixes #10635.

Convert some noway_assert to static_assert

[x86/Linux] Corretly pop args after unmanaged function call (#11007)

* [x86/Linux] Corretly pop args after unmanaged function call

* Fix x86/Windows build error

Add a regression test.

Merge pull request #11049 from briansull/fixMaxOffset

Fix incorrect test of structSize during struct promotion

Add test coverage for Rank 1 multidimensional arrays (#11034)

These have some weird behaviors worth covering.

IBC: Disable `-partialNGEN -minify` for SPC.dll (#10950)

Add some fixes around the use of IBCMerge in CoreCLR, along with a BuildTools patch (which the fixes are dependent upon):
* don't restore the default optdata package through BuildTools
* don't use partial ngen by default in CoreCLR for SPC.dll

Spill side-effects in impAssignMultiRegTypeToVar.

This importer function converts IR from the form

    tree

to
            /- tree
    STMT - =
            \- lclVar

in order to conform to the JIT's representation of multi-register values.
Today, it does not check whether or not the tree being spilled has any
side-effects that may interfere with the side-effects of trees that are
already on the evaluation stack, which can lead to bad codegen. This
change fixes this function to spill any side-effects on the stack
before generating the temp assign.

Fixes #10940.

Merge pull request #11044 from alpencolt/ryu-arm-localloc-subs-fix

[RyuJIT/ARM32] Use 'subs' instead of 'sub' on localloc

Merge pull request #11070 from stephentoub/add_stat_fields

Add Dev/Ino fields to Interop.Sys.FileStatus

For Reflection invoke, use ArgLocDesc

For arm64 if the struct needs to be passed in registers, then pass
that information along when we do the reflection invoke.

Note this adds more test cases for passing different struct types via reflection.
I am guessing similar work would need to be done for Arm32.

Change ICLRRuntimeHost2 to ICLRRuntimeHost4 for Windows corerun. (#10967)

* Change ICLRRuntimeHost2 to ICLRRuntimeHost4 for Windows corerun.

Since we've introduced ICLRRuntimeHost4 in PR #10842 for Unix, we apply it for Windows corerun as well.

Fix #10960

Stop running all tests on first failure, if the run is private or rolling.

Stop treating test files as executable and fix encoding (#11065)

* change encoding to UTF-8

* change line ending to UNIX

* remove executable attribute

Add Dev/Ino fields to Interop.Sys.FileStatus

Mac OS X is now macOS (README.md) (#11068)

[x86/Linux] Do NOT run GC on Transition (#11056)

Revert "Add SecurityZone stub" (#11053)

Updating the measurement.py call to be setup like the submission.py call

Set default BenchView run type to local
- This is the case when running on the dev boxes (outside automation)

Merge pull request #10933 from hqueue/ryujit/enable_nodebash_stats

Fix clang build error when enabling NODEBASH_STATS

Merge pull request #11021 from mskvortsov/array-offset-mla

[RyuJIT/ARM32] Use mla for array offset computation

Some fixes for the GC Reliability Framework (#11029)

* Quality-of-life fixes for the GC reliability framework to make it run
a little better on CoreCLR

* Remove some p/invokes to kernel32 when building on Core

* Minor fixes to run on non-Windows platforms

* Re-enable ReliabilityFramework sanity run

Improving robustness of the run-xunit-perf.cmd script.
- Added error checking to the script and make it fail fast
- Updated the xUnit Performance API version (build0002 -> build0003)

Implement the createdump policy options and env variables (#11032)

Added these command line options:

-f, --name - dump path and file name. The pid can be placed in the name with %d. The default is "/tmp/coredump.%d"
-n, --normal - create minidump (default).
-h, --withheap - create minidump with heap.
-m, --micro - create triage minidump.
-d, --diag - enable diagnostic messages.

Added these environment variables:

COMPlus_DbgMiniDumpType - if set to "1" generate MiniDumpNormal, "2" MiniDumpWithPrivateReadWriteMemory, "3" MiniDumpFilterTriage. Default is MiniDumpNormal.
COMPlus_DbgMiniDumpName - if set, use as the template to create the dump path and file name. The pid can be placed in the name with %d. The default is "/tmp/coredump.%d".

Merge pull request #11054 from wtgodbe/spacing

Fix spacing in build-test.cmd in Pipeline

Fix spacing in build-test.cmd in Pipeline

Merge pull request #11050 from ellismg/remove-duplicate-resources

Remove some duplicate resource strings

Merge pull request #11052 from wtgodbe/emptyArgs

Create default variables for Crossgen, TargetsNonWindows in pipebuild

Create default variables for Crossgen, TargetsNonWindows in pipebuild

Convert a few test files from UTF16 to ASCII and add license (#11051)

Merge pull request #10979 from wtgodbe/R2RWindows

Enable running Crossgen tests on Windows in Helix

Remove /RELEASE option required for device drivers (#11023)