Split OrdinalIgnoreCaseComparer from OrdinalComparer

They're both used very frequently, and we can eliminate the branches on checking _ignoreCase from every operation.

Reject byref-like object in FormatterServices.GetUninitializedObject (#9754)

Fixes #9739

[x86/Linux] Enable FEATURE_ARRAYSTUB_AS_IL (#9752)

* [x86/Linux] Enable FEATURE_ARRAYSTUB_AS_IL
* Move FeatureStubsAsIL into Windows-specific block

Merge pull request #9681 from mskvortsov/ryujit-arm32-reload

[RyuJIT/ARM32] Fix helper kill mask and call target consuming

Use GetSystemTimePreciseAsFileTime if available in DateTime.UtcNow (#9736)

Merge pull request #9717 from tannergooding/math-intrinsics2

Updating codegenxarch to support single-precision CORINFO_INTRINSIC_Sqrt

Merge pull request #9742 from sivarv/singleField

Enable promotion of structs containing fields of structs with a single pointer-sized scalar type field.

Merge pull request #9748 from AndyAyersMS/FixJitTestDir

Move jit tests to directory with proper casing

Merge pull request #9726 from alexperovich/codeFormat

Run dotnet/codeformatter on S.P.CoreLib sources.

Add jit test project checker as jit test (#9721)

Add the tool used to fix/repair projects as a test case. It will
fail with an error code if any project file in the jit test subtree
needs fixing.

This test requires CORE_ROOT to point at the test overlay and
that directory be rooted under the repo (other test cases already
have similar requirements).

CodeGenBringUp tests are noted as needing fixes, but are accounted
for under the category of "deferred fixes".

JIT: defer setting outgoing args size until after optimization (#9683)

For fixed out args ABIs, the jit currently computes the size of the
out arg area during morph, as it encounters calls that must pass arguments
via the stack.

Subsequently, the optimizer may delete calls, either because they are
pure and the results are unused, or because they are conditional and the
guarding predicates are resolved by the optimizer in such a way that
the call is no longer reachable.

In particular if all the calls seen by morph are subsequently removed by
the optimizer, the jit may generate a stack frame that it never uses and
doesn't need. If only some calls are removed then the stack frame may end
up larger than necessary.

One motivating example is the inlining of a shared generic method that
ignores its generic context parameter. The caller typically must invoke a pure
helper to determine the proper argument to pass. Post inline the call's result
is unused, and the helper call is deleted by the optimizer.

This change defers the outgoing arg size computation until fgSimpleLowering,
which runs after optimization. The code in morph now simply records the
needed size for each call in the associated fgArgInfo.

As before, the outgoing arg size computation ignores fast tail calls, since
they can use the caller-supplied scratch area for memory arguments.

The jit may introduce helper calls after optimization which could seemingly
invalidate the out args area size. The presumption here is that these calls
are carefully designed not to use the caller-supplied scratch area. The
current code makes the same assumption.

This change introduces some unanticipated diffs. Optcse estimates the
frame size to decide how aggressive it should be about cses of local
variable references. This size estimate included the out args area, which
appears to be something of an oversight; there are no cse opportunities
in this area and the offsets of the other local variables are not impacted
by the size of this area. With this change the out args area size is seen
as zero during optimization and so the cse strategy for a method may be
different.

Merge pull request #9582 from pgavlin/vzeroupper_x86_epilog

Do not report FP restores in x86 epilogs.

Merge pull request #9454 from mikedn/cgtun-to-cne

Morph (x GT_GT.unsigned 0) into (x GT_NE 0)

Move jit tests to directory with proper casing

Closes #9747.

Run codeformatter on S.P.CoreLib sources

Manual change to make codeformatter work

Enable promotion of structs containing fields of structs with a single pointer-sized scalar type field.

Merge pull request #9743 from weshaggard/UpdateUsingYourBuildDocs

Update UsingYourBuildDocs to use .NET Core 2.0 SDK

Update UsingYourBuild docs to use .NET Core 2.0 SDK

Merge pull request #9741 from brianrob/issue-9733

Disable Test Broken By CoreFX PR # 15767

Merge pull request #9655 from sdmaclea/PR-ARM64-JITGC-WIP3

[Arm64] Correct CORINFO_HELP_ASSIGN_BYREF killed mask

Format code.

Address PR feedback.

Disable broken tests.

Merge pull request #9727 from BruceForstall/DontBuildJitLib

Speed up JIT builds

Simplified MinOpts GC ref tracking. (#9231)

Simplified MinOpts GC ref tracking - when a method is compiled with MinOpts for targets with the 64-bit GC info encoder we mark all GC slots as untracked and we omit encoding call sites with no live tracked GC refs explicitly in the GC tables; this can be controlled using the new COMPlus_JitMinOptsTrackGCrefs environment flag.

Android: workaround absence of getcontext/setcontext (#9725)

* Android: workaround absence of getcontext/setcontext
use RtlCaptureContext and RtlRestoreContext

RyuJIT/ARM32: Fix helper kill mask and call target consuming

Reduce code in SafeHandleAddRef (#9724)

If DangerousAddRef returns rather than throws, its argument will be true, so there's no need to branch on it.

Porting the CoreMangLib/cti/system/math tests to CoreMangLib/cti/system/mathf

Faster List Add (#9539)

[Arm64] Fix genCodeForCpObj()

CORINFO_HELP_ASSIGN_BYREF is the functional equivalent to

   *dst = *src;
   ++dst;
   ++src;

Note: Description ignores barriers.

Since src and dst are incremented the registers are marked dead.

In genCodeForCpObj(), the incremented values are will be reused
and therefore must be treated as live.  Add code to explicitly
force them live for this use case.

Updating codegenxarch to support single-precision CORINFO_INTRINSIC_Sqrt

Merge pull request #9363 from tannergooding/math-intrinsics

Updating jit/valuenum to properly handle the single-precision versions of the math intrinsics.

Do not report FP restores in x86 epilogs.

The x86 unwinder neither needs nor expects to see these restores (which
right now consist solely of a `vzeroupper` instruction that is emitted
to eliminate AVX <-> SSE transition penalties), which was causing unwind
failures under GC stress. This change stops reporting these restores as
part of a function epilog.

Fixes #9452.

Speed up JIT builds

We don't need to build the static lib and crossgen static lib
versions of the JIT unless FEATURE_MERGE_JIT_AND_ENGINE is set.
And, since we currently don't set it, this means doing two
fewer JIT builds.

Merge pull request #9654 from sdmaclea/PR-ARM64-JITGC-WIP2

[WIP] [Arm64] Fix GC liveness for CodeGen::genPutArgStk()

Merge pull request #9653 from sdmaclea/PR-ARM64-JITGC-WIP1

[WIP] [Arm64] Fix GC hole in CodeGen::genCodeForCpObj()

Improve stack overflow reporting (#9650)

* Improve stack overflow reporting

This change modifies the SIGSEGV handling to use an alternate stack so that
we can safely detect and report stack overflow even in case when we are
really out of stack. Before, we were able to detect stack overflow and report
it only when JIT inserted stack probes (for functions with frames larger than
4kB) and so there was still space on the stack to run the sigsegv handler.
It brings in some additional complexity, since we need to switch to the original
stack of the thread once we figure out the sigsegv is not a stack overflow and
if we return from the hardware exception handler, we need to switch back to the
alternate stack before returning from the sigsegv handler.
Also, the alternate stack is created per thread and so we need to correctly destroy
it when a thread terminates and also install it on foreign threads that enter PAL.
This also requires creating fake stack frames to enable the libunwind to walk
the stack from the exception handler to the sigsegv location.

* Fix stack unwinding in CallDescrWorkerInternal

While testing the change to enable stack overflow handling, I've noticed that
the PROLOG_SAVE_REG_PAIR and PROLOG_SAVE_REG_PAIR_INDEXED macros are missing
.cfi_def_cfa_register fp. That resulted in inability to unwind through the
CallDescrWorkerInternal, since this function dynamically allocates stack slots
and so the default sp based frame doesn't work.

CMake configuration for cross building for Android (#9174)

* Configuration files for cross building for Android arm64

* Simplify toolchain script

* Fix trymake values for Android

Merge pull request #9712 from AndyAyersMS/FixSomeTestProjFiles

Fix jit test projects to specify debug type and optimization flags

Add test coverage for ldtoken with members (#9713)

The test coverage for `ldtoken field X` and `ldtoken method X` in the
repo is very small - this test tries to improve things a little.

In particular, it tests that we get the right reflection objects, and
that we can do this from shared and unshared code.

Merge pull request #9714 from BruceForstall/FixRegMasks

Fix non-Windows amd64 register mask initialization

Merge pull request #9702 from adiaaida/fixX86PerfTesting

Fix x86 perf testing

[Arm64] Fix GC hole in CodeGen::genCodeForCpObj()

Update tempreg liveness

[Arm64] Fix GC liveness for CodeGen::genPutArgStk()

Updating ValueNumStore to no longer upcast float to double when performing evaluations.

Separate Comparer and EqualityComparer creation logic into a non-generic class (#9640)

Fix projects that specified DebugType incorrectly

Make conditional DebugType properties uncondtional. Fix debug type
and optimize settings for _d and _do suffixed tests. Remove conditional
DebugType propery if there is also an unconditonal property.

Merge pull request #9704 from pgavlin/corefx15913

Correctly pass 12-byte GT_SIMD nodes on x86.

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

Update CoreClr, CoreFx to beta-25021-03, beta-25022-02, respectively (master)

Update CoreClr, CoreFx to beta-25021-03, beta-25022-02, respectively

Clean up ClrUnwindEx (#9482)

* Clean up ClrUnwindEx

Dropping some unnecessary conditional statements in jit\valuenum.cpp

Updating jit/valuenum to properly handle the single-precision versions of the math intrinsics.

Fix non-Windows amd64 register mask initialization

The arrays were being initialized with register numbers, not
register masks. The arrays are only used on non-Windows amd64
for homing circular incoming argument conflicts. Perhaps we
never saw this happen?

Fix jit test projects to specify debug type and optimization flags

This partially addresses the issues raised in #4124 and #4724 by explicitly
and unconditionally setting DebugType and Optimize flags for various test
project files.

The projects that were fixed are those that did not specify any debug
flag settings at all.  The fix is to set DebugType to PdbOnly and to set
Optimization to true, unless the test has an _r or _rel suffix, in which
case Optimization is set false.

This impacts 1497 projects, skipping over 311 projects flagged as needing
fixes. The projects that remain are the CodeGenBringUp set (for which we plan
to clone projects to test the various flag combinations), projects that
specify flags but do so in ways where the effective flag settings may vary
from DBG to RET, projects that specify the flags redundantly, or projects
where the flags are not consistent with the "test case name suffix" rules.

The edits were done by a tool using linq-to-xml and so some incidental
reformatting has happened.

Fix x86 perf testing

We did not use a different architecture in run-xunit-perf.cmd, so all the
x86 runs were uploading to the same architecture. This change fixes it so
x86jit32 runs upload to their own architecture. Also fixes the delayed
expansion issue for testenv.

Use CORINFO_CALLCONV to check for unmanaged calls (#9697)

`GTF_CALL_UNMANAGED` is a member of the wrong enum and it's something
that would be set on `call->gtFlags` except that at this point it isn't.
Check against the actual unmanaged calling conventions.

[x86/Linux] Fix framepointer while unwinding (#9678)

Use pCallerContext when getting frame pointer

Add a regression test.

Merge pull request #9703 from wtgodbe/CrossgenCode

Have Crossgen Failures exit in a dedicated exit routine

Correctly pass 12-byte GT_SIMD nodes on x86.

During lowering, all TYP_SIMD12 GT_SIMD nodes are retyped to TYP_SIMD16.
This is correct except in the case that the GT_SIMD node is going to be
passed by-value on x86, in which case this retyping causes us to push 16
bytes rather than 12. This change recognizes a GT_SIMD node that has
been retyped by checking the value of gtSIMDSize during LowerArg and
uses TYP_SIMD12 for the putarg instead of TYP_SIMD16 (aside from
differences in detecting retyped nodes, this is the same transformation
we already do when passing TYP_SIMD12 lclVars that are being used as
TYP_SIMD16 on x86).

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

Have Crossgen Failures exit in a dedicated exit routine

Merge pull request #9695 from jashook/up_timeout_for_arm64_gcstress

Up the timeout for gcstress arm64 jobs.

Up the timeout for gcstress arm64 jobs.

Fix compilation on macOS with case sensitive FS (#9693)

Improve string.Equals OrdinalIgnoreCase performance for matching chars (#9213)

* Improve string.Equals OrdinalIgnoreCase performance for matching chars

* Added code comment

Add --xunitOutputPath option to runtest.sh (#9646)

Clean up FixupDispatcherContext (#9685)

Use ADJUST_PC_UNWOUND_TO_CALL instead of _TARGET_XXX_ (#9686)

Merge pull request #9676 from parjong/fix/issue_9576

Fix bogus assert in flowgraph and update related ilproj

Rewrite assert using bbInTryRegions

Update related .ilproj

Fix formatting error

Comment out bogus assert in flowgraph

Merge pull request #9672 from jashook/arm64_regression

Add new regression.

Merge pull request #9671 from benaadams/revert-encoding

Revert "Fast-path for ASCII & UTF8 Encoding ASCII data (#9187)"

Merge pull request #9670 from stephentoub/cs_random

Delay Random allocation in ConcurrentStack

Add new regression.

Revert "Fast-path for ASCII & UTF8 Encoding ASCII data (#9187)"

This reverts commit 77e676be9fab34471c4ae9f16a7a51a06bd470ee.

Merge pull request #9661 from AndyAyersMS/FixTailCallStress1

JIT: tolerate GT_COMMA in tail call position

Remove a few explicit static cctors (#9669)

Merge pull request #9665 from sdmaclea/PR-ARM64-PT_KNONVOLATILE

[ARM64] Use PT_KNONVOLATILE... for cross

Delay Random allocation in ConcurrentStack

Fast-path for ASCII & UTF8 Encoding ASCII data (#9187)

[Arm64/Unix] Fix cmake race condition (#9666)

On a clean build SET(REQUIRE_LLDBPLUGIN false) was being ignored because the cmake
cache variable had not been created

Change order of cmake file to be more natural and to resolve

Merge pull request #9667 from sivarv/backout

Back-out promotion of structs containing fields of struct of single field

Backout promotion of structs containing fields of struct of single field.

[ARM64] Use PT_KNONVOLATILE... for cross

In order to enable cross compilation support, ARM64
needs to use PT_KNONVOLATILE_CONTEXT_POINTERS

Merge pull request #9657 from AndyAyersMS/FixFinallyChainMergeAssert

JIT: relax assert in fgMergeFinallyChains

Merge pull request #9636 from sivarv/shouldPromote

Refactor some of fgPromoteStruct() checks into lvaCanPromoteStructVar() and lvaShouldPromoteStructVar()

JIT: tolerate GT_COMMA in tail call position

Now that we can tail call from inlined call sites, we may see the call
embedded in a GT_COMMA, if the call's return value is ignored. This triggered
an assert the tail call transformation.

Closes #9641.

Merge pull request #9658 from chcosta/fix_coreclr_packaging

Exclude ni files from linux arm32 builds

Synchronous toolchain.cmake files with CoreFX (#9409)

Signed-off-by: Jiyoung Yun <jy910.yun@samsung.com>

Merge pull request #8843 from mikedn/decompose-cast-mul

Improve int->long cast decomposition

Merge pull request #9615 from JosephTremoulet/UpdateCompareVN

Update array length compare value numbers on CSE

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

Update CoreFx to beta-25017-02 (master)

Merge pull request #9228 from fujiy/fixcorefx4805

Add/Expose String.EndsWith(char) and String.StartsWith(char)

Merge pull request #9656 from RussKeldorph/corefx_x86

Fix corefx x86 testing and small netci.groovy fixes