Merge pull request #13036 from rartemev/tests_build_fix_rest

Made possible to build all tests on Linux as well.

Merge pull request #13035 from pgavlin/VSO468732

Treat byref-typed uses of int-typed lclVars as type int in LB.

Completelly fix build of tests on Linux

Enable Ubuntu's 1710 BuilsTests

Enable Ubuntu's 1710 BuilsTests

Treat byref-typed uses of int-typed lclVars as type int in LB.

This is consistent with the behvaior of both JIT32 and RyuJIT. This
resolves an assertion originating from the following scenario:

1. The input IL contains a lclVar of type `Foo*`, which the JIT imports as
   `TYP_I_IMPL` (which is `TYP_INT` in this case).
2. This lclVar is used as the `this` argument to a number of method calls.
   This is legal as per ECMA-335 section III.3.19 ("Correct CIL also allows
   a native int to be passed as a byref (&); in which case following the
   store the value will be tracked by garbage collection.")
3. All of the method calls to which this lclVar is passed as a byref are
   inlined. This produces many uses of the lclVar as a byref (i.e. we see
   nodes like `lclVar V06 byref` even though V06's varDsc has type int).
4. The lclVar is assigned a register `r`. At its first appearance--which is
   the first occasion in which it is loaded into this register--it is used
   as `TYP_BYREF`. When the code generator processes this appearance, it
   first sets the appropriate bit for `r` in `gcInfo.gcRegByrefSetCur`
   (`gcInfo.gcMarkRegPtrVal`, called by `genCodeForTree_REG_VAR1`) and then
   sets the appropriate bit for `r` in `regSet.rsMaskVars`
   (`genUpdateLife`).
5. The lclVar is used as `TYP_INT` as the operand to a `GT_RETURN` node.
   When the code generator processes this appearance, it attempts to update
   the GC-ness of `r` by calling `gcInfo.gcMarkRegPtrVal` (again via
   `genCodeForTree_REG_VAR1`). This function, though, explicitly excludes
   registers that contain live variables from its update, so `r` remains in
   `gcInfo.gcRegByrefSetCur` after this call. After calling
   `gcMarkRegPtrVal`, `genCodeForTree_REG_VAR1` calls `genUpdateLife`,
   which removes the the lclVar from `regSet.rsMaskVars`.
6. An assertion intended to verify that the only registers that are live
   after processing a statement are registers that contain lclVars fires,
   as `gcRegByrefSetCur` still contains `r`.

Fixes VSO 468732.

Cleanup GC *_STAT bitrot (#12772)

* Cleanup GC *_STAT bitrot

* Fixup per review

Revert "Fixes for issue 12982 Android cross-build fails when compiling cee_wks (#12986)" (#13031)

This reverts commit 2e7326df470bea696699cc7c812ee56dd0b470b9.

Change after correct version of the file

Merge branch 'master' into enableUbuntu1710-1

Merge pull request #13030 from wtgodbe/AddWindowsJobs

Add Helix jobs for Nano, Win7, Win8

Enable Ubuntu's 1710 BuilsTests

After adding Helix Queue for Ubuntu1710 we are enabling BuildTest for this SKU.

This completes: https://github.com/dotnet/core-eng/issues/1307

Add Helix jobs for Nano, Win7, Win8

Enable BuildTest in Windows Server Core

Enable BuildTest in Windows Server Core

Merge pull request #13011 from BruceForstall/Fix12886

Handle error case when looking for reg pair

typo (#13026)

PerfMap instrument stubs (#12437)

* PerfMap instrument stubs

* Perfmap fix segfault

Ignore specified signal when using perf (#12436)

* Ignore SIGRTMAX when using perf

Add a do nothing signal handler to SIGRTMAX
for profiling purposes.  Allows synchronization
point in injection into the profile

* Add COMPlus_PerfMapIgnoreSignal

* Fix nits per review

Merge pull request #13017 from benaadams/valuetask

ValueTask .ctor throw to Helper

Merge pull request #13005 from AndyAyersMS/FixPDepSimd12Issue

Don't map P-DEP SIMD12 local vars to SIMD16 on x64

Fixes for issue 12982 Android cross-build fails when compiling cee_wks (#12986)

* Fixes for issue 12982 Android cross-build fails when compiling cee_wks

Added conditional (__ANDROID__) to volatile.h
Added compile option to android/arm64/toolchain.cmake

* Changed ANDROID conditional to compiler and __atomic_load

Changed conditional to exclude volatile to __CLANG__ and __has_builtin(__atomic_load)

* Fixed Clang conditional

* Recommended changes to volatile.h and toolchain.cmake

* Added separate conditional for detecting builtin __atomic_load

Deleting the non-shared implementation of `System.Single` and `System.Double`.

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

Merge pull request #12193 from noahfalk/tiered_jitting_m2

ValueTask .ctor throw to Helper

Merge pull request #13012 from pgavlin/VSO468726

Fix `fgOrderBlockOps`.

Add the runtime code versioning feature

This makes tiered compilation work properly with profiler ReJIT, and positions the runtime to integrate other versioning related features together in the future. See the newly added code-versioning design-doc in this commit for more information.

Breaking changes for profilers: See code-versioning-profiler-breaking-changes.md for more details.

Addressing Will's comments

Merge pull request #12991 from hseok-oh/ryujit/fix_11783_2

[RyuJIT/ARM32] Morphing 1-field HFA struct argument and fix assertion

Enable BuildTest in Windows Server Core

Windows Server Core was just created so now enabling work to be processed in this image

Merge pull request #12996 from hseok-oh/ryujit/fix_12626

[RyuJIT/ARM32] Fix stack overflow on codegen phase

Fix `fgOrderBlockOps`.

This function was incorrect when the target was a `DYN_BLK` with
`gtEvalSizeFirst` set to `true`.

Fixes VSO 468726.

Simplify SHM-allocator (#12815)

* Simplify SHM-allocator

* Remove SHMNULL and NULLSharedID

Handle error case when looking for reg pair

When performing unsigned up-cast to long, if needReg was computed
with exactly 2 bits, then rsFindRegPairNo() is called to find the
corresponding reg pair. This can fail if one of the needReg bits
is in the reserved set (rsMaskResvd), as was the case in this test
on ARM for R10.

Add logic to handle this error return. The same logic already existed
in the signed cast case.

Fixes #12886

Create Windows PDBs from Portable PDBs during symbol archive (#12796)

* Update BuildTools to 2.0.0-prerelease-01812-02

* Use "GetAllSymbolFilesToPublish" target

"UnzipSymbolPackagesForPublish" only unzips. "GetAllSymbolFilesToPublish" is a new target that unzips then generates Windows PDBs. This allows us to index Windows PDBs on symweb even when not archiving.

Don't call AssemblyResolve event for CoreLib resources (#12999)

* Don't call AssemblyResolve event for CoreLib resources

Part of fix for #12668:
- CoreLib resource lookup should not raise the AssemblyResolve event because a misbehaving handler could cause an infinite recursion check and fail-fast to be triggered when the resource is not found, as the issue would repeat when reporting that error
- A handler could misbehave by returning an assembly that does not match the requested identity or by throwing

* Address feedback

Merge pull request #12943 from jashook/fix_8093

For LEGACY_BACKEND ARM32 do not overwrite 1 element HFA type

Merge pull request #13004 from stephentoub/move_valuetask

Move ValueTask and friends to corelib

Merge pull request #12995 from 303248153/170724_fix_comment

[JIT] Fix comment in optimizer.cpp:optOptimizeBools

Merge pull request #12975 from alpencolt/ryu-arm-casts-nyi

[RyuJIT/ARM32] Removed int <-> float casts related NYIs.

Merge pull request #12963 from BruceForstall/Fix12904

Fix a case where a reg is spilled before being freed, leading to assert

Merge pull request #12998 from github-john-doe/master

typo

Don't map P-DEP SIMD12 local vars to SIMD16 on x64

P-DEP local vars are logically independent locals, but physically embeded
in some structure with fixed layout. So they cannot be made larger.

We already had safeguards for ths in place for x86 so extend these to kick
in for x64 too.

Also update Lowering's checker to account for the fact that some SIMD12s
can persist in x64.

Fixes #12950.

A few minor perf tweaks

- Update ValueTask to use IsCompletedSynchronously: IsCompletedSynchronously is faster/slimmer than Status == RanToCompletion.
- Remove a few readonly's causing unnecessary struct copies

typo

Add ValueTask and friends to corelib

Files are copied without modification from corefx.

We will subsequently add these to the System.Runtime contract and specialize the System.Threading.Tasks.Extensions package to forward to System.Runtime on netcoreapp.

[RyuJIT/ARM32] Fix stack overflow on codegen phase

Fix stack overflow on codegen phase
Array size problem for reference map in genPutArgStk()

fix comment in optimizer.cpp:optOptimizeBools

[RyuJIT/ARM32] Morphing 1-field HFA struct argument and fix assertion

- Morphing 1-field HFA struct argument as local field
- Fix assertion on legacy JIT

Update android build script with correct package versions (#12985)

Merge pull request #12979 from pgavlin/GitHub12935

Lock the the dest reg in `inst_RV_TT` if necessary.

Merge pull request #12984 from dotnet/revert-12951-ryujit/fix_11783

Revert "[RyuJIT/ARM32] Morphing 1-field HFA struct argument"

Revert "[RyuJIT/ARM32] Morphing 1-field HFA struct argument"

[RyuJIT/ARM32] Removed int <-> float casts related NYIs.

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

Merge nmirror to master

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

Lock the the dest reg in `inst_RV_TT` if necessary.

For ARM32, `inst_RV_TT` may need to load the referenced lclVar into a
temporary register before emitting the requested instruction. The
chosen register *must not* be the instruction's destination register.
The existing code attempted to ensure this by masking the dest register
from the set of pickable registers, but this is not sufficient:
`rsPickReg` must return a valid register and will happily attempt to
spill and return the destination register even if it is not in the
recommended set. To prevent this, this change locks and unlocks the
destination register if necessary (i.e. if it was not locked upon
entry).

Fixes #12935.

Merge pull request #12951 from hseok-oh/ryujit/fix_11783

[RyuJIT/ARM32] Morphing 1-field HFA struct argument

Updated the path to built test binaries in examples

Merge pull request #12974 from pgavlin/GitHub12909

Disable promotion of custom-layout structs for legacy backend.

Merge pull request #12946 from BruceForstall/FixArmTempEstimation

Fix ARM32 spill temp allocation

Added IsPathFullyQualified() method to Path class (#12965)

Disable promotion of custom-layout structs for legacy backend.

These kinds of structs--particularly structs with holes--are not handled
correctly by argument morphing on ARM.

Promotion of these kinds of structs was already disabled for the x86
legacy backend; this change disables the optimization  for all legacy
backend targets.

Fixes #12909.

Associate last 2 RVAStatics tests with known issue

Merge pull request #12852 from jyoungyun/ryujit/rewrite_the_struct_args_to_be_passed_by_value_on_stack

[RyuJIT/ARM32] Rewrite the stack args to be passed by value on stack

Merge pull request #12968 from wateret/fix-gentree-creation

[RyuJIT/armel] Fix incorrect GenTree creation

Add CGroup CFS CPU limit support (#12797)

[RyuJIT/armel] Fix incorrect GenTree creation

When we generate a node for GT_PUTARG_REG, it should call gtNewPutArgReg().

Moving 'System.Single' and 'System.Double' to the 'System.Private.CoreLib.Shared' project. (#12967)

[Local GC] Refactor usage of GC-internal functions in the DAC (#12458)

* Refactor usage of GC-internal functions in the DAC

* Fix one debugger test failure, one to go

[RyuJIT/ARM32] Morphing 1-field HFA struct argument

Morphing 1-field HFA struct argument as local field

[RyuJIT/ARM32] Remove unnecessary GT_ADDR and GT_LCL_VAR_ADDR related code

Fix resources lookup to use satellite assembly resouce lookup inside AppX when the assembly is Private.Corelib (#12954)

Merge pull request #12866 from erozenfeld/12761ILRegressionTest

Add regression tests for #12761.

Fix a case where a reg is spilled before being freed, leading to an assert

In genRangeCheck(), under stress, the call to the range check helper
might spill the index register. This leads to an assert in rsMarkRegFree():
`rsIsTreeInReg(reg, tree)`. So, instead of calling rsMarkRegFree,
call genReleaseReg(), which handles this case.

This unfortunately reloads the spilled tree, which seems unnecessary, but it
otherwise frees the register fine.

One subtlety that is not obvious to me: the current code calls
`rsMarkRegFree(node->gtRegNum, node)` but `genRangeCheck()`
calls `rsMarkRegFree(genRegMask(node->gtRegNum))`. I'm not 100% sure
these are equivalent.

Fixes #12904.

Merge pull request #12961 from jashook/up_arm64_arm_timeouts

Up arm timeout to arm64 timeout

Up arm timeout to arm64 timeout

Merge pull request #12960 from jashook/stop_arm64_default_jobs

Add the correct job name and regex to arm64 ubuntu flow jobs

Merge pull request #12955 from stephentoub/tryderegister

Fix corefx break due to missing internal method

Merge pull request #10496 from sergign60/jitdrop

Jitted Code Pitching Feature implemented

Add the correct job name and regex to arm64 ubuntu flow jobs

This will keep the arm64 ubuntu jobs from being default auto triggered.

For lb arm do not overwrite 1 element hfa type

On arm64 we will treat 1 element HFA types as the primitive
type that is contained. On Legacy Backend codegenlegacy will
incorrectly assign these two types. Instead keep the struct
type through the IR correctly to do a blockop.

Fix IndexOutOfRangeException in PayloadNames property (#12932)

If you pass an null payload using Write<T>, and in the EventListener then call PayloadNames, it
will throw an IndexOutOfRangeException.   It should just return null.   This fixes this.

Merge pull request #12945 from pgavlin/FixLstFile

Fix a couple errors in the ARM32 lst file.

Fix corefx break due to missing internal method

System.Runtime.WindowsRuntime uses CancellationTokenRegistration.TryDeregister instead of Dispose to avoid a potential deadlock due to blocking waiting for an in-progress cancellation to complete.  Putting back TryDeregister to unblock corefx until a corresponding public API is exposed for this.

Merge pull request #12901 from jashook/arm64_ubuntu_changes

Fix auto starting arm64 ubuntu jobs

[RyuJIT/ARM32] Remove unnecessary isHfaRegArg check routine

[RyuJIT/ARM32] Add a case wrapping GT_OBJ on struct arguments

If the struct arguments pass by value on the stack, it should be wrapped in GT_OBJ.

[RyuJIT/ARM32] Rewrite the stack args to be passed by value on stack

Fix auto starting arm64 ubuntu jobs

Remove section headers writing to createdump (#12887)

Issue #12853

Fix a couple errors in the ARM32 lst file.

- Remove an entry for a test that has been deleted
- Disable ForeignThreadExceptions against its active bug

Fix ARM32 spill temp allocation

For ARM32 only, lvaAssignVirtualFrameOffsetsToLocals() forced temps
to be allocated high in the frame, with the comment:

```
// On ARM, SP based offsets use smaller encoding. Since temps are relatively
// rarer than lcl usage, allocate them farther from SP.
```

However, lvaFrameAddress(), when estimating the maximum frame offset for a
temp before final frame layout, did not consider this: it always assumed
temps were low on the frame. In addition, lvaTempsHaveLargerOffsetThanVars()
always returns `false` for ARM.

This caused an asserts with JitStress when we had an SP frame with large
locals, and estimated a small offset to a TEMP before final frame layout
but required a very large offset after final frame layout -- an offset that
could not be encoded by the chosen instruction. (These very large non-FP
frames might also exist only because of stress, such as GS stress.)

To fix this, simply remove the optimization, for both RyuJIT and LEGACY_BACKEND.

There is a very small code size regression (242 bytes over 42M) in asm diffs on internal
framework assemblies.

Fixes #12903

Fix Platform reporting parameters for build pipeline (#12900)

Merge pull request #12938 from pgavlin/TagGCTests

Disable some unreliable GC tests under JIT stress.

Disable some unreliable GC tests under JIT stress.

These tests depend on the JIT not extending lifetimes. This dependency
is frequently unsatisfied when running under JIT stress modes (e.g.
JITStress=1/2 or JITStressRegs=1).

Contributes to #12920, #12923, #12922, and #12921.

Fix CultureInfo.Parent to avoid throwing and catching (#12916)

Merge pull request #12930 from pgavlin/TagGH11408_ARM32

Tag the GitHub_11408 test with its bug number.

Merge pull request #12928 from pgavlin/TagVerify01Small

Tag the verify01_small test with its bug number.

Merge pull request #12925 from pgavlin/FixScenarioReturnCode

Fix the SustainedLowLatency/scenario test.

Merge pull request #12934 from pgavlin/TagVarargTests

Tag two vararg-related tests with their bug number for ARM.

Merge pull request #12931 from jashook/fix_machine_arm32_checked

Use the correct machine for arm32 checked jobs

Tag two vararg-related tests with their bug number for ARM.

These tests are not supported.

Contributes to #12914.