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

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)

[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.

Merge pull request #11035 from BruceForstall/NumSuccImprovement

Hoist calls to NumSucc() out of loops

[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.

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

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).

Merge pull request #10955 from adityamandaleeka/makeHandleStoreClass

Make a GCHandleStore class and interface for use by the VM

Merge pull request #10956 from AndyAyersMS/FixMorphUnnecessaryNullCheck

Jit: fix unnecessary null checks in some field accesses

Make HandleStores contain their underlying bucket directly.

[Local GC] Move operations on CLREventStatic to the EE interface (#10813)

* [Local GC] Move operations on CLREventStatic to the EE and add their functionality to the interface

* Fix a missed case

* Split GetWaitForGCEvent into two smaller interface methods to avoid exposing the event itself on the interface

* Initial implementation for Unix

* Complete unix implementation

* Make it work on Windows

* Remove redudant methods from GCToEEInterface

* Fix the Linux build

* First part of code review feedback: make GCEvent dispatch statically (Windows)

* Second part of code review feedback: make GCEvent dispatch statically (Unix)

* Standardize implementation across Windows/Unix (apparently MSVC is more lenient about constructor names than clang)

* Address code review feedback: Add Create*Event methods back onto GCEvent and remove them from GCToOSInterface

* Address code review feedback: remove a dead define

* Remove a bad comment, remove an unnecessary friend class, fix some formatting issues

* Fix an issue when initializing a GCEvent on Linux (should not be allocating an Impl in the constructor)

* Fix the same issue on Windows (less bad, just leaks memory instead of asserting)

[x86/Linux] Use appropriate FCALL macro for DoToDecimal (#10963)

Implement MEM_RESET handling in VirtualAlloc on Unix (#10939)

Rename IGCHandleTable to IGCHandleManager.

Merge pull request #10945 from OmarTawfik/update-runtimefeature-docs

Update comment to reflect how RuntimeFeature members should be defined

Reenable incremental linking (#10943)

Preserve VNs in fgMorphIntoHelperCall.

This method is used to transform a node into a semantically-equivalent
helper call, and should preserve any value numbers that are present on
the the node.

[Local GC] Handle table low-hanging interface violations (#10929)

Jit: fix unnecessary null checks in some field accesses

Morph was sometimes passing the existing MorphAddressContext down
to fgMorphField even when the field access was for a field value.
If that context contained indefinite offsets, morph would then
insert an explicit null check on the object pointer for the field
access.

Typically the field offset is small enough that this explicit check
is not needed. The implicit check done when fetching the field's value
is sufficient protection.

The fix is to have `fgMorphSmpOp` clear out the context for child
`GT_FIELD` nodes, unless the field parent is a `GT_ADDR`. Note
if there is an `op2` node the parent cannot be `GT_ADDR` so these
field children always get an empty context.

No tests added since this kicks in reasonably frequently in corelib and
elsewhere in frameworks.

Closes #10942.

Make a GCHandleStore class and interface for use by the VM.

Merge pull request #10953 from wtgodbe/crossImage

Use updated Docker images for Crossbuild

Use updated Docker images for Crossbuild

Merge pull request #10927 from wtgodbe/PortableOSXWin

Enable building Core_Root for any RID against packages

Merge pull request #10949 from wtgodbe/ImageVersion

Update Ubuntu14.04 Cross Docker image

Update Ubuntu14.04 Cross Docker image

Fix perf of DateTime.Now on Unix

Profiling DateTime.Now on Unix shows that a lot of time is spent in GetPreviousAdjustmentRule and AdjustmentRule::Equals. Instead, it is safe to use ReferenceEquals, which increases the performance of DateTime.Now and other time zone operations on Unix.

Update comment to reflect how RuntimeFeatures should be defined

Merge pull request #10922 from adiaaida/fixXUnitPerf

Fix run-xunit-perf.cmd for local use

[x86/Linux] Use appropriate FCALL macro for SafeBuffer methods (#10935)

Remove distro specific RIDs from SOS.NETCore's project.json. (#10925)

Ensure Environment.ExitCode works correctly after app domain unloaded. (#10842)

* Ensure Environment.ExitCode works correctly after app domain unloaded.

This PR addresses 2 problems of Environment.ExitCode:

1. Can't get correct exit code of main function.
2. Can't set %errorlevel%.

Details can be found on #6206

Fix #6206

Update CoreClr to preview1-25213-02 (#10937)

Merge pull request #10875 from rahku/fix_x86_gcstress

modify lastAvAddress when adjusting context for Stub

[ARM32/Linux] Build libcoreclrtraceptprovider.so (#10406)

* Build libcoreclrtraceptprovider.so for ARM32 and modify cmake

- Build libcoreclrtraceptprovider.so for ARM32
- Include libcoreclrtraceptprovider.so in nuget package
- Don't build libcoreclrtraceptprovider.so for
Tizen: lttng is not supported in Tizen
- Exclude packaging libcoreclrtraceptprovider.so based on os-name
excluded for tizen

* Remove libcoreclrtraceptprovider.so before test for ARM32/Linux

Remove libcoreclrtraceptprovider.so before test for ARM32/Linux in CI
CI test environment using docker with qemu has some problem to use lttng library

Create ELF core dump on coreclr exceptions and signals. (#10902)

* Create ELF core dump on coreclr exceptions and signals.

When coreclr aborts because of a unhandled managed exception or hardware signal, the
"createdump" utility is launched if the "COMPlus_DbgEnableMiniDump" environment
variable is set to "1".

This utility uses the DAC enumerate memory interface to determine what memory regions
should be written for the managed state plus all the normal Linux memory regions for
the native state like threads, stacks, DSO, etc.

There still needs some to configure some of the "policies" like the name/location of
the core dump and the type of minidump (MiniDumpNormal, MiniDumpWithFullAuxiliaryState,
MiniDumpWithPrivateReadWriteMemory, etc.).

Currently only supported on Ubuntu Linux.

Make createdump smaller by using PAL in DAC module.

Add platform (x64 & arm64) specific optimized jit helpers for statics access (#10912)

Merge pull request #10930 from rahku/marshalfix

HelperMethodFrame missing at certain places in method MarshalNative::PtrToStructureHelper

HelperMethodFrame missing at certain places in method MarshalNative::PtrToStructureHelper

Merge pull request #10914 from hqueue/jit/remove_unused_fuction

Remove unused function in BasicBlock

Merge changes from TFS (#10926)

* Fix desktop build break

[tfs-changeset: 1654125]

* JIT formatting

Merge pull request #10917 from stephentoub/fix_badges

Fix build badges in OfficialAndDailyBuilds.md

[x86/Linux] Fix ResumeEsp before resume (#10749)

* [x86/Linux] Correctly unwind FCALL frames

* Fix Up ResumeEsp inside ProcessCLRException

* Fix Context Before PC Update

[x86/Linux] 16-byte aligned BackPatchWorkerAsmStub (#10759)

* [x86/Linux] 16-byte aligned BackPatchWorkerAsmStub

* Use fixed alignment padding

* Do NOT push ebx (no longer required)

Merge pull request #10905 from pgavlin/VSO406158

Fix a number of reordering issues in shift decomposition.

[x86/Linux] 16-byte aligned DelayLoad_MethodCall (#10760)

Enable building Core_Root for any RID against packages

Merge pull request #10706 from sdmaclea/PR-ARM64-GC-EmitInsLdStOp

[Arm64] Fix GC Holes in emitInsLoadStoreOp