Merge pull request #9085 from mskvortsov/ryujit-arm32-eh

[RyuJIT/ARM32] EH, switch implementation and various updates

Create lsra-throughput.md

Merge pull request #9493 from pgavlin/ReportBytesAllocated

Report bytes allocated in the JIT time log.

JIT: enable implicit tail calls from inlined code (#9405)

Inlines of calls from implicit tail call sites should allow recognition
of inlinee implicit tail call sites.

The jit recognizes implicit tail call sites during importation,
but the inlinee compiler instance did not have compTailCallOpt set
and so never recognized these instances. Fix this and update the logic
to detect the transitively implicit tail calls.

Now that these sites are recognized, morph needs a fix to tunnel through
repeated casts for tail calls, since each level of inlining might add a
cast. All these casts should be identical.

Note under R2R tail calls are not yet recognized (see ZapInfo::canTailCall).

Enable only under FEATURE_TAILCALL_OPT_SHARED_RETURN since the
inline tail call sites are not likely to be in BBJ_RETURN blocks.

Closes #9349.

Use `%Iu` instead of `%I64u`.

Report bytes allocated in the JIT time log.

This adds a new column, "Total Bytes Allocated", to the JIT time log.
This column reports the total number of bytes requested from the host by
the JIT's arena allocator.

This change also enables `FEATURE_JIT_TIMER` by default (which only
affects JIT32).

[Linux/ARM] Fix cross-architecture component build error: disable unused code (#8866)

* [ARM32/Linux] Fix cross-architecture component build error: unused function comparing CONTEXT with T_CONTEXT.
- change build script: remove duplicate test build in cross-component build

Hide VirtualUnwindXXX for DACESS_COMPILE (#9407)

Unify GetControlPC/GetRegdisplaySP/SetRegdisplaySP (#9444)

* Unify GetControlPC

* Unify GetRegdisplaySP/SetRegdisplaySP

Remove CER metadata (#9487)

GcHandle Perf Tweaks (#9473)

* GcHandle Perf Tweaks

Revert earlier CAS changes to Eventing/** and put under !if !CORECLR (#9478)

* Revert earlier CAS changes to Eventing/** and put under !if !CORECLR

Merge pull request #9455 from sivarv/structPromotion

Increase field count limit to 3 for promoting a struct with no field accesses.

Merge pull request #9456 from adiaaida/fixDesktopBreak

Replace binary literal with hex

Merge pull request #9476 from jkotas/revert-9313

Revert "Delete mscorlib from packages (#9313)"

Merge pull request #9470 from AndyAyersMS/FixNoinlineMessage

JIT: fix confusing inline failure reason

Optimize Span.Fill (#9441)

Increase the field count limit to 3 for promoting a struct with no filed access.

Revert "Delete mscorlib from packages (#9313)"

This reverts commit 4fe623c9235e4df42be810d09ca7f282956e3cec.

# Conflicts:
# src/.nuget/Microsoft.NETCore.Runtime.CoreCLR/debian/Microsoft.NETCore.Runtime.CoreCLR.pkgproj

Merge pull request #9451 from JosephTremoulet/UpdateValnum

Update conservative value numbers during CSE

Merge pull request #9293 from guhuro/master

Add k-nucleotide to BenchmarkGames

Add k-nucleotide to BenchmarkGames

Changed .gitattributes to set line endings to windows ones for the input
files of this test.
Added two different tests, one with and the other without parallelism.
Used FileStream instead of BufferedStream, since the latter does not
exist in netstandard 1.4

[x86/Linux] Port SWCB_GetExecutionState (#9436)

JIT: fix confusing inline failure reason

When attempting to inline a method from the core library, the inline
will fail if the method being inlined has a noinline callee; the
presumption being that the callee expects to be able to find it's
immediate caller on the stack (this may or not may be the case, see #8102).

Update the failure message for this case to hopefully better explain
why the inline doesn't happen.

Replace binary literal with hex

The desktop build breaks with the binary literal used in the fat pointer
mask. Replace it with a hex equivalent.

Update conservative value numbers during CSE

When a CSE candidate's defs all share the same conservative value number,
its uses can be updated to share that conservative value number as well
when CSE is performed, because we are removing any reloads that may have
been the cause of the divergence.  Performing this update can improve
subsequent range check elimination when the CSE use is array length or
index in a bounds check.

Merge pull request #9413 from jashook/fix_internal_build_warning

Fix warning from an unsigned/signed comparison

Inlinable ArraySegment ctor (#9433)

Rename __reserved to __clr_reserved to avoid a conflict on Android (#9402)

* Rename __reserved to __clr_reserved to avoid a conflict on Android

[x86/Linux] Fix UMThunkStub stack alignment (#9365)

Fixes out going call in UMThunkStub to be 16 byte stack aligned

Address the feedback.

RyuJIT/ARM32: more of verbosity for NYI tracking

RyuJIT/ARM32: misc tiny updates

RyuJIT/ARM32: update lea lowering

RyuJIT/ARM32: enable fallback to legacy JIT

RyuJIT/ARM32: enable DecomposeLongs phase

RyuJIT/ARM32: fix consume order and clone emitInsTernary

RyuJIT/ARM32: refactor emitInsMov, update st.lclFld/Var codegen

RyuJIT/ARM32: fix node info for range check

RyuJIT/ARM32: enable call lowering

RyuJIT/ARM32: enable switch and jump table codegen

RyuJIT/ARM32: enable EH code generation

Merge pull request #9416 from wtgodbe/Suse13

Remove OpenSuse13 from CI

Clean up GetAddrOfSecurityObject (#9423)

Clean up EECodeManager::GetInstance (#9424)

* Clean up EECodeManager::GetInstance

* Use !CROSSGEN_COMPILE instead of CROSSGEN_COMPILE

Clean up EECodeManager::GetGSCookieAddr (#9425)

Clean up EECodeManager::GetReturnKind (#9426)

Clean up EECodeManager::GetFunctionSize (#9427)

Clean up EECodeManager::GetParamContextType (#9428)

Clean up EECodeManager::GetExactGenericsToken (#9429)

Clean up EECodeManager::GetParamTypeArg (#9430)

[x86/Linux] Revise TAStackCrawlCallBackWorker (#9432)

Merge pull request #9431 from BruceForstall/Fix9374

Generate better code for x86 SIMD get[i] intrinsic

Merge pull request #9191 from sandreenko/CORERT-fat-call-transformation

CoreRT fat call transformation

[x86/Linux] Fix Calling Convertion Mismatch inside ilasm/ildasm (#9410)

Delete unnecessary domain sanity check (#9420)

Generate better code for x86 SIMD get[i] intrinsic

For get[i] intrinsic, with LONG base type and constant index, we can
compute the decomposed indices statically.

Fixes #9374

Merge pull request #9392 from BruceForstall/FixGtGetOp2

Add gtGetPossibleOp2()

[ARM32/Linux] Fix definition and CMake for cross-architecture build (#9262)

- Fix ABI flag based on target
- Exclude corefx for cross-architecture component

Span Slice method AggressiveInlining (#9404)

CoreRT calli transformation

Transform managed calli when compiling for CoreRT.

Add gtGetOp2IfPresent()

It was noticed that many uses of gtGetOp2() fully expect op2 to
exist and be non-nullptr, and the use of gtGetOp2() was simply
for style/readability purposes. However, it has a cost, as it
checks whether the tree is binary first, and returns nullptr if
not. For most cases, this is unecessary and expensive.

Introduce a new gtGetOp2IfPresent() function that captures the
previous behavior: checking if the tree node is binary before
returning op2, or returning nullptr otherwise. gtGetOp2() is
changed to simply return gtOp2 directly, without any checking
in non-DEBUG builds. It can be used if you know the op2 exists.

Most uses of gtGetOp2() were left alone (and hence get the new
behavior). The ones that need the old behavior were renamed
gtGetOp2IfPresent().

Mostly newer code is affected, as older code would generally
access gtOp2 directly, whereas newer code started using gtGetOp2()
for style and symmetry, especially after gtGetOp1() was introduced.

Merge pull request #9382 from BruceForstall/CallFinallyCleanup

Remove unnecessary parameter to genCallFinally()

Merge pull request #9386 from BruceForstall/FixSwitchTable

Fix compare/branch switch mode expansion

Filter RefEmit modules in SendEventsForNgenMethods (#9418)

* Filter RefEmit modules in SendEventsForNgenMethods

Fixes #6427

Merge pull request #9417 from wtgodbe/publishNativeBins

Publish test native bins from correct directory

Publish test native bins from correct directory

Remove OpenSuse13 from CI

Fix warning from an unsigned/signed comparison

This fixes a desktop build break.

Merge pull request #9169 from JosephTremoulet/UntrackedMem

Value-number byref loads

Value-number `ByrefExposed` memory and loads

Teach value numbering to keep track of the current value number for
`ByrefExposed` memory, updating it appropriately at relevant stores.
Introduce a new VN operator `ByrefExposedLoad`, used for loads of
address-exposed locals and loads by indirs that don't have more specific
value numbers, which is a function of the current `ByrefExposed` memory VN
and the pointer VN.  This allows loop hoisting and CSE to recognize
redundant loads via byrefs when there are no intervening stores.

Fixes #7903.

Define `MemoryKind::ByrefExposed`

Add a new `MemoryKind` to represent "any memory that byrefs may
reference", called `ByrefExposed`.  All definition points of `GcHeap`
become also definition points of `ByrefExposed`, and additionally, so do
definitions of address-exposed locals.

Because it is a common case (currently happening in 90% of methods in
System.Private.CoreLib) that a method has no definitions of
address-exposed locals, have liveness detect this condition and set the
new `byrefStatesMatchGcHeapStates` flag accordingly; when the states
match, the same `MemoryPhi`s, defnums, and `PerSsaData` are then shared
between the two memory kinds, to avoid excess allocations/processing.

This change defines `ByrefExposed` memory and its def/use points, and
builds SSA for it, but no optimizations make use of it, so this is a
no-diff change.

Introduce `MemoryKind` abstraction

Re-cast the notion of "heap" (in liveness, SSA, and value-numbering) as
one of potentially many `MemoryKind`s, called `GcHeap`.  Update names,
comments, data structures, and signatures as appropriate to parameterize
relevant data/methods over `MemoryKind`.  This change is a no-diff
refactoring, and currently `GcHeap` is the only `MemoryKind`.  Generally,
codepaths which will generically need to process all `MemoryKinds`s
(initializing, dumping, dataflow propagation) now iterate over all
`MemoryKinds`, and codepaths which are sensitive to the semantics of the
specific `MemoryKind` (def/use identification in liveness and value
numbering) are changed to specifically operate on `MemoryKind::GcHeap`.

One notable exception is that `lvMemoryPerSsaData` and `CountForMemoryDef`
are *not* parameterized over `MemoryKind`; there's a single "space" of SSA
defnums for memory defs (though the same tree can incur different defs for
different memory kinds [in which case their defnums will differ]), to
facilitate subsequently sharing SSA nodes across memory kinds when
appropriate.

Refactor fgCurHeapVN and HeapSsaMap updates

Consolodate the code for updating the current heap VN and HeapSsaMap
entries into new helper method `recordHeapStore`, to make it easier to
evolve that code.  Change `fgValueNumberArrIndexAssign` to return the new
heap VN to allow callers to pass it down into `recordHeapStore`
accordingly.

This is just a refactoring; no change to compiler behavior.

Merge pull request #9329 from JosephTremoulet/LateArgVN

Get call arg value numbers from late args

Clean up EnumGcRefs (#9412)

Merge pull request #9408 from jashook/disable_checked_arm64

Disable checked arm64 jobs until jobs are stable

[x86/Linux] Use SP as StackMarker (#9389)

Disable checked arm64 jobs until jobs are stable

Jobs currently are expected to run red until a lstFile update goes in. Therefore,
disable the job as a PR trigger until it is clean.

Add ComVisible(true) back to several special types (#9403)

It is required by WinRT/COM interop. Fixes #9401.

Remove more CAS (#9390)

* Remove PermissionSet

* Remove HostProtectionAttribute

* Remove PermissionState

* Remove S.Security.Permissions

* Remove IPrincipal

* Fix native side

* Remove model.xml again

Fix field type for ByReference<T> and TypedReference (#9284)

Fix field type for `ByReference<T>` and `TypedReference`

Fixes #7894

Add scripts & docs which support creating an Android toolchain on Linux (#9175)

* Add scripts & docs which support creating an Android toolchain on Linux

Merge pull request #9385 from jashook/lower_block_store_arm64

Call LowerBlockStore TreeNodeInfoInitBlockStore

Design portable tail calls with helpers (#8596)

* Design portable tail calls with helpers

Currently we don't handle non-trivial cases of tail calls on Unix due to the non-portable and
complex way it is implemented. This pull request describes a design of a portable way of
handling these cases so that it can be used uniformly over all our jitted and AOT (ahead of time)
platforms.

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

Update CoreClr, CoreFx to beta-25007-01, beta-25007-01, respectively (master)

[x86/Linux] Clean up EHContext-related code (#9395)

Remove bclrewriter (#9351)

* Remove bclrewriter

* Dump crossgen error to Jenkins log

Clean up methods for USE_GC_INFO_DECODER (#9394)

Move GC/Coverage/delete_next_card_table to the Long GC playlist (#9376)

Fix a typo in TailCallFrame::UpdateRegDisplay log (#9381)

Merge pull request #8849 from seanshpark/fixstackalign

[x86/Linux] Stack align 16 bytes for JIT code

[ARM32/Linux] Fix calling convention - stdcall in crossgen (#9364)

* [ARM32/Linux] Fix calling convention - sdcall in crossgen

- Fix calling convention:
not use stdcall for ENCODEMODULE_CALLBACK & DEFINETOKEN_CALLBACK in linux/x86
x86-host/arm32-target crossgen & x86 crossgen
build error in clang 3.9: not allowed calling convention casting

Update CoreClr, CoreFx to beta-25007-01, beta-25007-01, respectively

Fix compare/branch switch mode expansion

It appears we never hit this case due to a very low
minSwitchTabJumpCnt number that almost always forces
us to use jump tables. This was found in the RyuJIT/ARM32
bring-up, because ARM uses a higher number.

Also, add a stress mode to force using the compare/branch expansion
mode more frequently, during stress.

Call LowerBlockStore TreeNodeInfoInitBlockStore

For arm64 call LowerBlockStore before TreeNodeInfoInitBlockStore
as done on other architectures.

Remove unnecessary parameter to genCallFinally()

Merge pull request #9377 from RussKeldorph/defaultprivate

Default-trigger ARM64 testing on certain users' PRs

[x86/Linux] Stack align 16 bytes for JIT code

Change JIT code to align stack in 16 byte used in modern compiler

Default-trigger ARM64 testing on certain users PRs

Re-using the logic that allows certain users to request ARM64 testing to
default-trigger ARM64 testing on their PRs.  Ideally, we'd be able to
separate the list of people who have permission to trigger testing from
those that get testing by default, but that is harder than it seems.