Merge pull request #13364 from hqueue/arm/ryujit/issue_13363_regression_of_13190

[RyuJIT/ARM32] Fix regression

[Linux/ARM] Fix managed breakpoints (#13316)

* [Linux/ARM] Fix managed breakpoints

This commit introduces the following changes in order to enable
ICorDebug-based debuggers to use breakpoints on ARM Linux:

* Use 0xde01 as breakpoint instruction on ARM Linux.
  ARM reference recommends to use 0xdefe as a breakpoint instruction,
  but Linux kernel generates SIGILL for this instruction.
  The 0xde01 instruction causes the kernel to generate SIGTRAP.

* Fix SIGTRAP handling on ARM Linux.
  Unlike x86, when SIGTRAP happens on ARM Linux, the PC points
  at the break instruction. But the rest of the code expects
  that it points to an instruction after the break,
  so we adjust the PC at the start of HandleHardwareException().

* Enable ARM single stepping for PAL.
  Handle single stepping for PAL path the same way as for non-PAL path.
  Also enable ArmSingleStepper executable buffer by allocating it
  from system global loader executable heap.

* Hande ARM single step only when debugger is attached, fix comments and code style

* Pass existing Thread object to HandleArmSingleStep

Merge pull request #13350 from BruceForstall/FixVSO478352

Fix double-indirect p/invoke call generation

Added SetThreadDescription to set the unmanaged thread name (#12593)

* Added SetThreadDescription to set the unmanaged thread name as well when a managed thread name was set.
This will show up in future debuggers which know how to read that information or in ETW traces in the Thread Name column.

* use printf  instead of wprintf which exists on all platforms.

* Removed printf
Ensure that GetProceAddress is only called once to when the method is not present.
Potential perf hit should be negligible since setting a thread name can only happen once per managed thread.

* - Moved SetThreadName code to winfix.cpp as proposed
- Finalizer and threadpool threads get their name
- GCToEEInterface::CreateBackgroundThread is also named
- but regular GC threads have no name because when I included utilcode.h things did break apart.

* Fix for data race in g_pfnSetThreadDescription

* Fix string literals on unix builds.

* Fixed nits
Settled thread name on ".NET Core ThreadPool"

Increase timeout and log the result for a couple of WaitAny tests that expect AbandonedMutexException (#13298)

[RyuJIT/ARM32] Fix regression

- This fixes regression cuased by PR #13190
- Restore code for resolving registers for multi register operator

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

Typo (#13360)

Update CoreClr, CoreFx to preview2-25614-01, preview2-25614-01, respectively (#13359)

Finish moving Guid to shared partition

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

Update CoreClr, CoreFx to preview2-25613-02, preview2-25613-02, respectively (#13300)

Follow-up: Add enforcepgo for release Windows_NT default release build (#13319)

* Add enforcepgo for release Windows_NT default release build

* Restrict enforment of PGO to PGO supported builds

* Expand the usage of enforcpgo

* Fix break in the if-else control flow

Merge pull request #13332 from wateret/fix-ryuarm-hfa

[RyuJIT/armel] Fix PutArgReg BuildRefPositions

Move files to shared CoreLib partition (#13342)

Fix double-indirect p/invoke call generation

When the signature to getAddressOfPInvokeTarget() was changed,
the double-indirect CT_USER_FUNC case was accidentally changed
to pass a non-null `addr` to emitIns_Call(). This leads to asserts
when ngen'ing (in my test, ngen of System.dll on desktop using `LEGACY_BACKEND`
cross-compiling arm32 altjit). For some reason, x86 is more permissive
in the emitIns_Call() assert for this case, but doesn't use the
non-null addr.

Fixes VSO 478352

Fix bug where whitespace breaks arg proccessing (#13320)

* Fix bug where whitespace breaks arg proccessing

* Fix whitespace proccessing bug in buil-text.cmd

Merge pull request #13338 from jashook/log_crossgen_command

Log the crossgen command in build.sh.

Changes that didn't make it to PR#12982 (#13055)

* Changes that didn't make it to PR#12982

* Recommended changes to volatile.h

Added two templates to cast away volatility due to clang 3.8 changes

* Update runtime.Linux.Microsoft.NETCore.Runtime.CoreCLR.props

Fixed erroneous extra Platform conditional line

Log the crossgen command in build.sh.

This makes it easier to copy paste the commnand if it fails.

Fixed CoreRT issue https://github.com/dotnet/coreclr/issues/13195 (#13335)

[RyuJIT/armel] Fix PutArgReg BuildRefPositions

We should fix registers for GT_PUTARG_REG, not GT_COPY or GT_MUL_LONG.
And merge the `if` statement with GT_PUTARG_SPLIT case.

Merge pull request #13306 from hseok-oh/ryujit/fix_13156

Fix lclVar move node type inserted on lsra resolve phase

Fix genSpillVar

Fix genSpillVar() instead of insertMove()

Fix the race condition in mutex test using another manual reset event (#13308)

* Fix the race condition by using a manual reset event instead of gc.keepalive

* Addressing PR comment

Merge pull request #13285 from shimingsg/v-shige/add-perftc-GCLarge

add perf test case GCLarge

Merge pull request #13299 from pgavlin/GH13161

Fix issue #13161.

Implement BitConverter Span-based APIs (#13278)

Finished first checkbox of issue 22355: Implement in
System.Private.CoreLib in coreclr

Merge pull request #13304 from BruceForstall/FixVSO471672

Copy additional GenTreeBoundsChk fields in gtCloneExpr

JIT: modify box/unbox/isinst/castclass expansions for fast jitting (#13188)

When the jit is generating code in debug/minopts/rare-block modes,
we'd prefer it to generate code more quickly and worry less about overall
generated code performance.

Generally speaking smaller intermediate and final code should correlate
well with faster jitting.

This change alters the expansions of box, unbox, isinst, and castclass when
generating code for minopts, debug, or in rarely run blocks. In such modes
the jit estimates whether an inline sequence or general helper call would
result in more compact code, and then chooses the smaller sequence.

This reduces generated code size around 2.5% in a variety of scenarios,
and roughly translates to a 1.5% improvement in time spent jitting.

Similar strategies can be applied to other complex operations during
importation. That work is forthcoming.

Revert "Add enforcepgo for release Windows_NT default release build (#13275)" (#13318)

This reverts commit b07a921524ec9e8c8f5e482707f9fc1b9b86d95e.

Merge pull request #13307 from wateret/cleanup-armel-lsracond

[RyuJIT/armel] Cleanup with OperIsMultiRegOp()

Merge pull request #13309 from wateret/arm-getregcount

[RyuJIT/armel] Introduce GetRegCount() for MultiRegOp

Add enforcepgo for release Windows_NT default release build (#13275)

* Add enforcepgo for release Windows_NT default release build

* Restrict enforment of PGO to PGO supported builds

* Expand the usage of enforcpgo

* Fix break in the if-else control flow

Merge pull request #13286 from JosephTremoulet/TailLoopScratch

Ensure scratch BB in tail recursion -> loop xform

[RyuJIT/armel] Cleanup with OperIsMultiRegOp()

Use OperIsMultiRegOp() instead of separated conditions

remove empty line

[RyuJIT/armel] Introduce GetRegCount() for MultiRegOp

Abstract getting reg count with method GetRegCount()
Added the case when the reg count is 0.

Merge pull request #13284 from wateret/fix-13281

[RyuJIT/arm32] Fix Regression

Fix lclVar move node type inserted in lsra phase

When we try to insert lclVal move node (load, store) on lsra phase,
fix type as TYP_INT when size is smaller than TYP_INT

LinkBench: Fix Roslyn job

This change fixes a build-break in the LinkBench job, because
of a failure in the Roslyn Build.
With this change, Roslyn Benchmark is built using MsBuild (similar to
Roslyn's Build.cmd) instead of directly calling dotnet publish.

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)