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)

Merge pull request #11047 from stephentoub/task_iscompletedsuccessfully

Expose Task.IsCompletedSuccessfully

Remove some duplicate resource strings

Enable running Crossgen tests on Windows in Helix

Merge pull request #10948 from eerhardt/GetPreviousAdjRulePerf

Fix perf of DateTime.Now on Unix

Merge pull request #11045 from JosephTremoulet/TraceEventVersion

Run-xunit-perf: update TraceEvents version number

fail fast on exceptions that indicate corrupted process state (#10957)

Fix incorrect test of structSize in struct promoted

Fixes the desktop test Crossgen NI and Ngen NI validation

[x86/Linux] EH Support for Per-Frame P/Invoke Init (#10966)

Expose Task.IsCompletedSuccessfully

small clean-up and bug fix in genIntToIntCast (#10989)

genintToIntCast was cleaned-up, the assert that was wrong for x86 was deleted. The repro was added.

Run-xunit-perf: update TraceEvents version number

The perf harness needs 1.0.3-alpha-experimental now.

Assert GetAdjustmentRuleForTime sets ruleIndex correctly.

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

Previous implementation used sub instruction for counter
conditional execution for iterating over loops which cause incorrect
behavior.

Merge pull request #11035 from BruceForstall/NumSuccImprovement

Hoist calls to NumSucc() out of loops

RyuJIT/ARM32: use mla for array offset computation

[x86/Linux] changed to the AT&T syntax to solve a problem related broken stack. (#10932)

This is a problem similar to #10383.

Signed-off-by: ragmani <ragmani0216@gmail.com>

* fixed class size calculation (#10916)

Fix String.Replace implementation (#11039)

* Fix String.Replace implementation

String.Replace was callin ga low level CompareInfo method IndexOfCore which cannot be called in some conditions like when using ordinal ignore case option or when running in invariant mode.
The fix is to call a higher level method in CompareInfo which can handle such cases becfore calling the lower level method

* fix typo

Merge pull request #10995 from svick/patch-1

Update Viewing JIT dumps to 2.0 and csproj

Fix error caused by passing -TargetsWindows at end of Command Line (#11031)

Do not throw an assertion on no mscorlib

It's okay to use debuggable attributes from other BaseAsmRef's

Fixes #10608

Merge pull request #11036 from brianrob/remove_eventcounter

Remove EventCounter from System.Private.CoreLib

Fix build break

Rename attribute to IsReadOnlyAttribute (#11026)

Use isInternalRegDelayFree to avoid temp reg overallocation (#11025)

Use isInternalRegDelayFree to avoid temp reg overallocation

Remove EventCounter from System.Private.CoreLib.

Hoist calls to NumSucc() out of loops

Should be a minor throughput improvement.

Remove handle assignment validation from GC side.

Add handle assignment validation to VM side.

Improve performance of Object.GetType for arrays (#10992)

* Improve performance of Object.GetType for arrays

Cache managed Type object on MethodTable for arrays. Makes Object.GetType for arrays about 5x faster.

Merge pull request #11028 from danmosemsft/api.zone

Add SecurityZone stub

Move handle manipulation functions to interface.

Format spmi sources with clang-format (#11027)

Apply jit style format on spmi sources.

Jit intrinsics for Span<T>.get_Item and ReadOnlySpan<T>.get_Item. (#10910)

Implement these two methods as optional-expand jit intrinsics.
Uses `GT_ARR_BOUNDS_CHECK` for the bounds check so in some cases
downstream code is able to eliminate redundant checks. Fully general
support (on par with arrays in most cases) is still work in progress.

Update one bit of code in the optimizer that assumed it knew the
tree types that appeared in a `GT_ARR_BOUNDS_CHECK`.

Add benchmark tests for Span and ReadOnlySpan indexers.

Tests ability of jit to reason about indexer properties with respect
to loop bounds and related indexer uses. Some cases inspired by span
indexer usage in Kestrel.

Closes #10785.

Also addresses lack of indexer inlining noted in #10031. Span indexers
should now always be inlined, even when invoked from shared methods.

Add scenario-based tests

Modified Xunit runner to run Scenario Benchmarks
Updating xunit-performance packages

Add SecurityZone stub

In function Assembly.Load ignore AssemblyName.CodeBase even if it is set (#11010)

Remove Alpine for pipe builds (#11022)

Support FEATURE_EVENT_TRACE override (#10961)

Allow build.sh to have multiple cmakeargs options (#10962)

Update CoreClr, CoreFx to preview1-25215-03, preview1-25214-03, respectively (#10952)

[RyuJit/ARM32] Fix assertion of failure on "Instruction cannot be encoded" (#10789)

Fix assertion of failure on "Instruction cannot be encoded"

I figured out the bit set mask for '0xffffff00' which is the imm is not supportted on ARM.
So at first, It would be checked the validation of using instruction with immediate constants.
If not, it would be use the instruction with registers.

Merge pull request #11013 from mskvortsov/nullcheck

[RyuJIT/ARM32] Update nullcheck codegen

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

[RyuJIT/ARM32] Fix amount of allocated registers for Localloc.

Fix ARM32/Linux release CI test failure (#11008)

* Fix ARM32/Linux release CI test failure

Fix ARM32/Linux release CI test failure by removing libcoreclrptprovider.so from CoreFX build results.

* Fix typo

Fix typo in netci.groovy

Add Span and ReadOnlySpan from CoreCLR

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

[RyuJIT/ARM32] Fix amount of allocated registers for Localloc.

Now for small allocation (up to 4 store instructions) we always have
1 temporary register which is different from register where result
of stackalloc is returned. Also one additional register will be allocated
for PSP if needed.

RyuJIT/ARM32: update nullcheck codegen

Fix comment about pinning bit in object.h (#11004)

Fix warning in LPTSTRTestNative (#10973)

* Fix warning in LPTSTRTestNative and add support for wcsncmp in xplatform.h

Fix OOM exception thrown in case insensitive replace for an empty string (#11001)

Fixed CI link (#11003)

Updated link referenced in CI builds; old link was broken

Move Span/ReadOnlySpan to shared CoreLib partition (#10988)

Fix a few method names to better names used in CoreRT

Contributes to https://github.com/dotnet/corert/issues/2966

Make some Dictionary code smaller (#10993)

Store a reference to the relevant entry at the start of the loop to avoid indexing `entries` multiple times.
This avoids some redundant range checks in `Remove` and allows the elimination of a duplicate `index++` in `Enumerator.MoveNext`.

Saves ~7KB of code.

Merge pull request #10983 from danmosemsft/revert.jeremy

Revert "Strip out unused reg code. (#10741)"

Update Viewing JIT dumps to 2.0 and csproj

Make createdump build and work on Centos (#10985)

Remember AdjustementRule index

Profiling TimeZoneInfo scenarios on Unix shows most of the time being spent in GetPreviousAdjustmentRule. This is because we need to scan all of the AdjustmentRules, looking for the specified rule, and then getting the one before it.

This change remembers the AdjustmentRule index when passing the AdjustmentRule into GetPreviousAdjustmentRule. This allows us to not scan all the rules, and increases the performance of DateTime.Now significantly in time zones that have daylight savings time.

Fix https://github.com/dotnet/corefx/issues/3571

[Arm64] Fix assembly errors (#10977)

* [Arm64] Fix assembly errors

* [Arm64] Static use ldrb w2, [x2, w1, UXTW]

* Revert changes to src/vm/arm64/asmhelpers.asm

Merge pull request #10958 from pgavlin/VSO397793

Preserve VNs in fgMorphIntoHelperCall.

Revert "Strip out unused reg code. (#10741)"

This reverts commit ed4f594abf41a71b126152bb8755051d0831e12d.

Merge pull request #10936 from BruceForstall/TempRegApi

Introduce API for codegen getting temp registers from gtRsvdRegs

Add a brief comment to the new test.

Small clean up of stackEntry (#10820)

Make all stackDepth equal unsigned int.
Delete possible wrong function (the comment said : "// used in the inliner, where we can assume typesafe code. please don't use in the importer!!", but it was used).
Delete methods, that used arguments as output buffers.

* delete code without effects

Introduce API for codegen getting temp registers from gtRsvdRegs

LSRA puts a set of temporary registers needed by a node in the
gtRsvdRegs register mask. Currently, evey codegen function that
needs to use a temporary register manually manipulates this mask.

Introduce a few functions to make this simpler and more regular:

1. GetSingleTempReg() // Gets a temp; asserts there is exactly one temp reg.
2. ExtractTempReg() // Gets the lowest temp, removes it from gtRsvdRegs so subsequent calls don't see it.
3. AvailableTempRegCount() // Returns a count of available temp registers.

All take an optional register mask, so you can extract a register from
just the set specified by the mask (typically RBM_ALLINT or RBM_ALLFLOAT).