Merge pull request #9495 from pgavlin/OptConfig

Add a flag to enable fine-grained optimize control.

Merge pull request #9523 from jashook/up_arm64_timeout

Up the timeout for arm64 jobs that run tests.

Merge pull request #9437 from JosephTremoulet/UpdateFormat

Update format.py to use dotnet cli rc3

[Unix] Skipped tests should report exit code 2 (#9510)

Turn segfauts into asserts in debug build (#9509)

* Prevent segfault if method table is null

* ValidateInner() check method table is not null

Up the timeout for arm64 jobs that run tests.

With the new test update arm64 jobs run too long to fit into the default 120 minutes.

Merge pull request #9501 from chcosta/mscorlib_race

Fix race condition building System.Private.CoreLib.dll and SOS.NETCore.dll

Update lsra-throughput.md

Merge pull request #9519 from CarolEidt/FixCoreFx15713

Add a test case for PR #9496

[Arm64/Unix] Remove aarch64 unsupported message (#9508)

[Arm64/Unix] Fix gdbjit support (#9512)

[Arm64] Add more GCROOTS logging (#9514)

Add a test case for PR #9496

Merge pull request #9491 from jkotas/NoInlining

Use DynamicSecurityMethod attribute to mark methods with StackCrawlMark

Merge pull request #9496 from CarolEidt/FixCoreFx15713

Fix System.Numerics.Vectors CoreFx test failures

Fix formatting.

Merge pull request #9499 from jashook/arm64_dyn_block_assert

ARM64 Modify genCodeForInitBlk assert

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

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

ARM64 Modify genCodeForInitBlk assert

Assert in codegenarm64 was incorrect for the GT_STORE_DYN_BLK oper.
If it is that oper then there will not be a temp reg used and instead
the size is computed and stored into REG_2. Add an assert to assert
this is done.

remove additional slash

Fix the release build with OPT_CONFIG and address PR feedback.

Fix race condition building System.Private.CoreLib and SOS.NETCore.dll

Create lsra-throughput.md

Use DynamicSecurityMethod attribute to mark methods with StackCrawlMark

CoreCLR does not have CAS, and so we can conveniently use it to mark methods with StackCrawlMark to
decouple it from NoInlining. The original purpose of DynamicSecurityMethod was to disable inlining
of the caller and to insert CAS security checks, so we are basically just keeping the first part.

Fixes #8102

Revert "JIT: fix confusing inline failure reason"

This reverts commit 0826f9dee6a8fdf8266523945708a684376b0280.

Merge pull request #9493 from pgavlin/ReportBytesAllocated

Report bytes allocated in the JIT time log.

Fix formatting.

Fix System.Numerics.Vectors CoreFx test failures

CoreFx Issue 15713 is due to a case of an Indir(Addr(Field(Vector3 local))) which for some reason has a MorphAddrContext of MACK_Ind. Although I haven't fully identified why that is the case, we should be conservative in this case and mark the address as do-not-enregister.
In addition, when attempting to debug this with a Checked JIT, I encountered an AV due to `GetJitTls` returning null in the `JITDUMP` calls in `getMaxIntrinsicSIMDVectorLength`. Again, I'm not sure why this would be the case but I have added guarding conditions.

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

Add a flag to enable fine-grained optimize control.

Defining this flag, `OPT_CONFIG`, enables a number of optimizer-related
configuration values. Changing these config values allows for some
interesting experiments without rebuilding the JIT (e.g. disabling the
SSA-based optimizer without forcing minopts, repeating optimizations,
etc.).

This flag is defined by default in debug builds and undefined by default
in release builds.

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

Update format.py to use dotnet cli rc3

Links taken from
https://github.com/dotnet/core/blob/master/release-notes/rc3-download.md

Modify path with prepend rather than append

To make sure we always get the intended version of these commands.

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.