Copy additional GenTreeBoundsChk fields in gtCloneExpr

We were previously losing the gtStkDepth field, which led to
an assert with the STRESS_CLONE_EXPR stress mode with array
bounds checks in function argument lists that had already
pushed stack arguments.

Merge pull request #13190 from jyoungyun/ryujit/use_the_regtype_of_refposition_instead_of_its_type_in_the_gt_putarg_split_case

[ARM32/RyuJIT] Use regtype instead of node's type in PUTARG_SPLIT case

add the license header to the C# file

Fix issue #13161.

This issue was a crash when calling R2R-compiled code from JITted code
on Windows/ARM32. The caller had a live value in `r4` that was scribbled
over by the callee. This change adds the necessary call to
`regSet.rsTrackRegTrash` which ensures that `r4` is saved/restored in
the prolog/epilog when it is used for R2R indirect calls.

Fixed issue #13282. (#13291)

Changed the open process token from TOKEN_READ to TOKEN_QUERY.

Merge pull request #13234 from janvorli/add-rhel6-official-rid

Add RHEL 6 official RID

Merge pull request #13268 from dotnet-maestro-bot/master-UpdateDependencies

Update CoreClr, CoreFx to preview2-25609-02, preview2-25609-01, respectively (master)

Add fail-fast check after lvaGrabTemp in impImportBlockCode (#13271)

Fix #13231.

* responce Brian's comment.

Merge pull request #12802 from pgavlin/VSO462269

Fix VSO 462269.

typo (#13287)

Start at running ILLink to find dead code in Corelib (#13266)

* Remove some dead code

* Remove from native side

* More

* Remove nls consts

* Fix break

* Fix break

Update ryujit-tutorial.md

Format the dump info for getting the flow graph

Fix VSO 462269.

When decomposing a long compare on 32-bit platforms, the operands to the
decomposed compare must be sign- or zero-extended appropriately.

Update CoreClr, CoreFx to preview2-25609-02, preview2-25609-01, respectively

Merge pull request #13270 from stephentoub/refactor_taskwait

Refactor large Task NoOptimization bodies into own methods

add perf test case GCLarge

[RyuJIT/arm32] Fix Regression

Fix regression caused by #13023

Fix #13281

Ensure scratch BB in tail recursion -> loop xform

Our loop detection requires a predecessor outside the loop, so without
this the loops often don't get optimized as such.

Merge pull request #13276 from dotnet-bot/from-tfs

Merge changes from TFS

Cache both string and its length during GDBJIT setup (#13221)

Add Encoding Span-based APIs (#13269)

* Add Encoding Span-based APIs

* Address PR feedback

Fix fo CoreCLR cofde formating error
Add extra spacing

[tfs-changeset: 1669566]

Fix possibly uninitialized warning

[tfs-changeset: 1669561]

Change Task.WaitAll/Any overloads to call WaitAll/AnyCore directly

Avoid multiple unnecessary levels of NoOptimization frames.

Add StringBuilder Span-based APIs (#13163)

* Add StringBuilder Span-based APIs

* Factorized ThreadSafeCopy methods

The char[] based one was called only by CopyTo so it wasn't needed anymore. Replaced by a Span based version

* Fix possible overflow

Merge pull request #12168 from gbalykov/remove-relocations-readonly

Partially remove relocations from SECTION_Readonly

Add build-time check to enforce PGO compilation for applicable DLL files (#13258)

* Added PGO execution verification script

Added a script which can help verify whether or not PGO was used
to compile PE files passed to it

* Added target to enforce PGO compilation

* Remove OS checking for enforcepgo

Use logf() instead of JitDump() (#13265)

* Use logf() instead of JitDump()

This allows more JitDump output to get routed through the CLR
logging facility.

* Formatting

Improve performance of string.IndexOfAny for 2 & 3 char searches (#13219)

Merge pull request #13261 from pgavlin/VSO475427

Fix VSO 475427.

Merge pull request #13263 from BruceForstall/FixDesktopX86SPMIBuild

Fix SuperPMI x86 desktop build

Merge pull request #13259 from pgavlin/VSO469210

Fix VSO 469210.

Format code.

Add smoke test for event pipe on Linux (#13240)

* Add smoke test for eventpipe

* Clean up some code from a different version

* Use more descriptive project and namespace names

Refactor large Task NoOptimization bodies into own methods

The methods apparently need to be marked as [MethodImpl(MethodImplOptions.NoOptimization)] to aid the Visual Studio debugger in picking out these methods and their arguments, but it's causing a few chunky functions to have no optimizations on the whole body.  Separating these bodies out.

Merge pull request #13260 from pgavlin/VSO471670

Fix VSO 471670.

[ARM32/RyuJIT] Set the source candidates for PutArgSplit nodes

The Use RefPosition of PutArgSplit should use the same register with the Def RefPosition.

[ARM32/RyuJIT] Use regtype instead of node's type in PUTARG_SPLIT case

The size of TYP_STRUCT in definition is 1 and it is lower than sizeof(int).
It means TYP_STRUCT should use the node's type instead of its type.
In PUTARG_SPLIT case, it wil use its regtype of child nodes.

Update CoreClr, CoreFx to preview2-25608-01, preview2-25608-01, respectively (#13262)

Merge pull request #13023 from wateret/fix-12994

[RyuJIT/armel] Make RefPosition arg regs fixed

Remove some dead code (#13257)

Fix SuperPMI x86 desktop build

Fix VSO 475427.

In addition to repredicting if the frame size is large enough to require
that we reserve `r10`, we also need to throw out any previous
prediction. If we do not, we may decide to reuse it if uses less stack
space than the reprediction.

Fix VSO 471670.

If we remove a NOP during rationalize that is unused, we need to ensure
that its operand is also marked as an unused value.

Fix VSO 469210.

This bug arises when ARM32/Legacy backend generates an RMW operator
whose source operand is an addressing mode. In that case, `sched_AM`
needs to choose a register to use for the value that will be loaded from
memory. Originally it was doing so via `rsPickReg`, but that function
may spill registers that are in use, including the register that is
supposed to be modified by the RMW operator. All other points in
`sched_AM` that need to allocate a register use `rsPickFreeReg` in order
to avoid issues like this. This change updates the use of `rsPickReg` to
instead use `rsPickFreeReg`.

Merge pull request #13250 from JosephTremoulet/CaptureHead

Stop rejecting loops with backward exits

Swaping the order of how scenarios are run. (#13255)

- Currently ILLink scenario is broken and the x64 scenario leg does not run JitBench
  By swaping the scenario it will let JitBench run while ILLink is being fixed.

Merge pull request #13236 from briansull/arm-issue

Updated fix for issue #13093

Fix Region DisplayName (#13249)

This change fix the logic when getting the display name from the OS and when we use the native name instead

Cleanup code access security from the unmanaged runtime (#13241)

Stop rejecting loops with backward exits

Remove some convoluted code from `optFindNaturalLoops` that has the effect
of rejecting loops with exits that are (lexically) backwards branches.
The comments in the removed code indicate that the intent was to identify
loops which include blocks lexically before `top`, but since `top` is
defined as `head->bbNext`, any such loop was doomed to be rejected by the
code (also being removed) that rejects a loop that "captures head".

When a loop does have a branch back to a block prior to `top`, the
subsequent code that identifies loop exits will identify it as such, and
the subsequent code that rejects predecessors other than `head` of loop
blocks ensures that the backwards branch truly was an exit.

This change still leaves `top` and `first` as separate concepts in the
loop table, to allow for subsequently improvements to loop detection to
identify such loops, though today they're always the same block.

Removed the legacy JIT32 assert regarding 4-byte alignment inArenaAllocator::allocateMemory
Immediately after this assert we roundUp to an pointer size allocation amount.

Stopped using two implementation of ArenaAllocator::allocateMemory
Instead we add ifdef DEBUG parts to the common version
Move the implementation of ArenaAllocator::allocateMemory to the header file

Fixed a couple of allocations in assertionprop to use CMF_AssertionProp so that we correctly
attribute which phase uses the memory being allocated.
Added range check for the array writes in Compiler::optMapComplementary

Removed the AlignUp to 4-byte in operator new and new[] in compiler.hpp

fix callsignalhandlerwrapper(arm) (#13114)

issue: #13099
1. add unwind information
2. add free_stack

Update CoreClr, CoreFx to preview2-25607-01, preview2-25607-02, respectively (#13133)

Use attribute names without namespaces in a few places (#13248)

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

Reduce size of array enumerators for x86 (#13157)

Use placement new to initialize ELF Header (instead of memcpy) (#13218)

Enable string format tests on Linux. (#13191)

Modified string format tests so that it can be supported on Linux.

Fix #11655

Update buildtools to 2.0.0-prerelease-01903-01. (#13228)

Review fixes

Move files to shared partition (#13244)

Update linux-instructions.md (#13246)

On Ubuntu a configuration failure occurs when running `cmake` through `build.sh` on `corefx` asking to install `libkrb5-dev` or its equivalent.

[RyuJIT/armel] Fix ifdef condition and Formatting Error

[RyuJIT/armel] arg regs are always TYP_INT

[RyuJIT/armel] Allocate one more temp for call address

[RyuJIT/armel] Each reg gets own SpillFlag for MultiRegOp

[RyuJIT/armel] Make RefPosition arg regs fixed

Passing double arg regs via `r0:r1` or `r2:r3`,
each RefPosition's RegMask should have only one exact register
so the candidate register can be fixed.

e.g.)

```
<RefPosition #4099 @2625 RefTypeUse <Ivl:1237> BB35 regmask=[r0-r1] last>
<RefPosition #4100 @2625 RefTypeUse <Ivl:1238> BB35 regmask=[r0-r1] last>
```

to be:

```
<RefPosition #4099 @2625 RefTypeUse <Ivl:1237> BB35 regmask=[r0] last fixed>
<RefPosition #4100 @2625 RefTypeUse <Ivl:1238> BB35 regmask=[r1] last fixed>
```

Fix #12994

Merge pull request #13227 from BruceForstall/Revert13186

Revert "Fixes issue #13093"

typo (dotnet/corert#4274)

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

typo (#13229)

align the segment size to power of 2 and perf fix on getting the seg mapping table indices (#13076)

seg size fixes

Revert "Fixes issue #13093"

This reverts commit 0d390cceb7ab0c8a271afce315fe4456211fdc6c.

Merge pull request #13197 from BredPet/master

ARM32: fix switch jump table generation bug

Merge pull request dotnet/corert#4271 from dotnet/nmirror

Merge nmirror to master

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

ARM32: fix switch jump table generation bug
- force using an inlined jumping instead switch table generation
- see https://github.com/dotnet/coreclr/issues/13194

Signed-off-by: Petr Bred <bredpetr@gmail.com>

Merge pull request #13143 from shimingsg/v-shige/add-nogc

Add NoGC test to coreclr

Add RHEL 6 official RID

Merge pull request #13186 from briansull/alloc-align-fixes

Fixes issue #13093

Fixes issue #13093

Removed the legacy JIT32 assert regarding 4-byte alignment inArenaAllocator::allocateMemory
Immediately after this assert we roundUp to an pointer size allocation amount.

Fixed a couple of allocations in assertionprop to use CMF_AssertionProp so that we correctly
attribute which phase uses the memory being allocated.
Added range check for the array writes in Compiler::optMapComplementary

Removed the now unnecessary AlignUp to 4-byte in operator new and new[] in compiler.hpp

Drop the first iteration value of JitBench. (#13211)

typo (#13196)

* typo

* typo

Merge pull request #13145 from hqueue/arm/ryujit/issue_13118

[Ryujit/ARM32] Update allocateBusyReg() in LSRA for ARM32

Merge pull request #13166 from YongseopKim/ryujit/arm32/add_handling_secure_delegate_inv

[RyuJIT/ARM32] Adds code to handle SecureDelegateInvoke

[RyuJIT/ARM32] Adds code to handle SecureDelegateInvoke

- Adds code to handle SecureDelegateInvoke on morph phase like legacy's handling.
- Set R4 to proper addr for indirect call of vsd
- This is like legacy's doing from #6500.
- Fix #12993

Apply review feedback

- Sink a local variable into the innermost block
- Fix format for if-else statement by adding braces

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>

Merge pull request #13154 from christianscheuer/fix-readme

Fix reference to System.Threading.Thread class in README.md

Merge pull request #13182 from adiaaida/addMinOptsTP

Add JitMinOpts throughput jobs

Add JitMinOpts throughput jobs

This change adds an opt_level throughput option so that we can run
throughput jobs in minopts mode. This also adds jobs to the lab for min
opts runs

JIT: refactor logic for removing box upstream side effects (#13169)

Introduce a new method `gtTryRemoveBoxUpstreamEffects` to capture
the logic for cleaning up after a dead value type box into a utility.
Call this from `gtFoldExprSpecial` and now also from the inliner when
it finds a dead box argument.

Also remove the useless `fgExpandInline` field and associated
`fgIsInlining` method, as the jit never set the field to true.

No diffs from the refactoring. The inliner case kicks in in 3 places
in the test tree.

Closes #13136.

Add JitBench scenario testing. (#11400)

Merge pull request #13158 from parjong/fix/INTERPERTER_GDBJIT_Conflict

Resolve FEATURE_GDBJIT/FEATURE_INTERPRETER conflict

Merge pull request #13155 from CarolEidt/UModTreeNodeInfo

Set Register Requirements for Mod/Div

Merge pull request #13128 from JosephTremoulet/Mulshift

Emit power-of-two constant multiply as shift

Merge pull request #11456 from CarolEidt/EHWriteThruDoc

Add an alternate algorithm done during LSRA

[Arm64] GetLargestOnDieCacheSize (#13071)

Look for cache info in /sys/.../cache/index*/size

If that fails estimate cache size based on processor count

[RyuJIT/ARM32] Update allocateBusyReg() with a helper for ARM32

- Add a helper `isRegInUse()`
- Update `allocateBusyReg()` using `isRegInUse()` for ARM32

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>

[RyuJIT/ARM32] Unassign a double register properly in allocateBusyReg()

- Introduce a helper, `unassignDoublePhysReg()` for ARM32
- Update `allocateBusyReg()` for ARM32 using `unassignDoublePhysReg()`

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>

Remove unused extern declarations (#13159)

[RyuJIT/ARM32] Update allocateBusy() to test spill candiates with new helpers

- Introduce a helper `canSpillReg` for all targets
- Introduce a helper `canSpillDoubleReg` for ARM32
- Remove a obsolute helper `canSpillThisReg`
- Update allocateBusy() to test spill candiates with new helpers

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>

PAL enable Thread affinity support (#12936)