[Arm64/Unix] Use portable MP optimized new/alloc (#12333)

[Arm64] Fix RecycledListsWrapper cache guard (#13397)

Merge pull request #12216 from ellismg/fix-warnings

Fix warnings when building SOS.NETCore.csproj

[Local GC] Enable building the GC as a shared library (#13137)

* [Local GC] Build the GC using system headers

* Disable features to get the GC to build

* Get rid of the separate 'GC PAL' build

* Remove unused stuff

* Don't build gcenv.os.cpp when linking in a standalone gc

* Stub out CPUGroupInfo and NumaNodeInfo

* Stub out IsGCThread and friends

* Build the GC as a shared library :tada:

* Build, link, and run! :tada:

* Fix standalone GC build break

* Fixes where the GC's MethodTable and VM's MethodTable classes disagree

* Integrate a standalone GC into the CoreCLR build system (so it gets copied to the output folder). Re-enable some ifdef-ed out includes that are required for a non-standalone build of the GC.

* Bring changes to Unix and fix the Unix build. Implement some compiler intrinsic wrappers and alignment functions expected by the GC.

* Fix the Windows build

* 1. Code review feedback: use standard types for BitScanForward and
   BitScanForward64
2. Delete FEATURE_COM stuff from the build system, not needed for this
PR
3. Fix the Unix build

* Fix the Windows x86 build - the _BitScanForward64 intrinsic is not available when targeting 32-bit platforms

* Remove a number of things from gcenv.base.h that are not used

* Remove a linker workaround now that we are not linking a standalone GC into the runtime

* Remove dead code, make the lack of GC_PROFILING for standalone gc not break profiling on the non-standalone build

* Code review feedback - use add_library_clr and other cmake-related fixes

* Fix include indentation

* Remove some extraneous parameters to cmake functions (cmake is remarkably lenient...)

Merge pull request #13461 from JosephTremoulet/TodoUpdate

Update JitOptimizerTodoAssessment.md

Follow-up: Add enforce PGO to official builds (#13437)

* Add variables to pipeline to enable enforcepgo for Release

* Address review

Merge pull request #13487 from mikedn/switch-temp-type

Fix incorrect switch temp lcl type

Delete some dead PAL code (#13469)

* Delete some dead PAL code

GetComputerName and GetUserName are not used.

* HAVE_GETPWUID_R can now be deleted

Remove empty try's (#13493)

* Remove empty try's

* Remove some dead comments

* more

Undo a few JIT layout workarounds (#13505)

Remove some `goto`s that were added to work around #9692 (poor code
layout for loop exit paths) -- the JIT's layout decisions were improved
in #13314, and these particular `goto`s are no longer needed; crossgen
of System.Private.CoreLib now produces the same machine code with or
without this change.

Part of #13466.

ThreadPool minimalistic thread request cap (#13431)

Cap ThreadPool thread requests minimalistically

- Only request an additional thread on dequeue if enqueue request was suppressed, and if there are any work items left in the queues
- ThreadPool pad hot cache line
- Add ideas from #13408

fix JitConfig initialize and destoy (#13477)

fix JitConfig initialize and destoy.

Merge pull request #13496 from justinvp/version_parse

Avoid unnecessary intermediate Version allocation

Fixing wcsstr function to account for cases where search string is longer than remaining target string. (#13504)

Use CreateFile2FromApp if available (#13410)

* Use CreateFile2FromApp if available

This is needed for F5 UAP brokering scenarios.

Remove check for handle type. Overzealous, doesn't apply to UAP scenarios.

* Address feedback

- Use CreateFile2 if Win8 or higher
- Always use CreateFile2FromApp if Appx
- Make DisableMediaInsertionPrompt a struct
- Push CreateFile2FromApp down to avoid JIT

* More feedback

Always use FromApp for WinRT.

* Conditionalize IsWindows8OrAbove check

* Always use FromApp if in AppX

* Address more feedback

- Fix Unix build by moving LoadString to shared
- Break out CREATEFILE2 struct
- Condition CreateFile2FromApp usage

* Move include

Clean up VS experience for s.p.corelib.csproj (#13481)

* Remove bogus assembly warnings in VS

* Fix warnings about sharing files within the project tree

* Remove bogus lines

* Remove lines showing up as bogus source files

* Remove overridden target

* feedback

Merge pull request #13482 from rartemev/issue_12469

Fixed misconception between FP register allocator and RyuJIT's CSE phase

Merge pull request #13424 from stephentoub/versionspan

Add span-based Version methods

Support docker cgroup limits (#13488)

* Fix cgroup mountinfo parsing

The parsing would find the wrong '-' in lines like this:
354 347 0:28 /system.slice/docker-654dd7b6b8bbfe1739ae3309b471e95ccc82b3a3f56b7879f0a811d68b5c4e1d.scope /sys/fs/cgroup/cpuacct,cpu ro,nosuid,nodev,noexec,relatime - cgroup cgroup rw,cpuacct,cpu

* cgroup: don't append cgroup relative path for reading docker limits

Merge pull request #13497 from noahfalk/tiered_jit_fix_tests

Enable by-request tiered compilation testing in CI

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

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

Merge pull request #12489 from jashook/unix_x64_and_arm64_fasttailcall_fix

[Unix x64|Arm64] Correct canfastTailCall decisions

Update CoreClr, CoreFx to preview2-25621-02, preview2-25621-02, respectively

Fixing assert in groovy

Enable by-request tiered compilation testing in CI

Avoid unnecessary intermediate Version allocation

`Version..ctor(string)` is implemented by calling `Version.Parse`, which
allocates an intermediate `Version` instance. Avoid the unnecessary
intermediate allocation by using `Version.Parse` directly.

Fix incorrect switch temp lcl type

Merge pull request #12892 from pgavlin/LivenessDCE

Improve DCE in LIR liveness.

delete unused return argument (#13468)

Rename System.Threading.Thread.m_ManagedThreadId to System.Threading.Thread._ManagedThreadId to align with CoreRT implementation (#13473)

Rename System.Threading.Thread.m_ManagedThreadId to System.Threading.Thread._ManagedThreadId to align with CoreRT implementation

Fixed misconception between FP register allocator and RyuJIT's CSE phase

Merge pull request #13392 from noahfalk/tiered_jit_fix_tests

Make unit tests run clean with tiered compilation

Merge pull request #13472 from dotnet/revert-13464-reimport_rhel6_test_change

Revert "Re-port test project changes from release/2.0.0 to master"

`perf.groovy` updates. (#13465)

- Increase how long performance artifacts are kept.
- Added an extra step to move files to be archived into a different folder.

Improve DCE in liveness.

In particular:
- Rather than only removing dead code as part of dead store removal,
  remove all side-effect-free nodes that either do not produce a value
  or produce a value that is unused.
- When optimizing, set `fgStmtRemoved` in order to indicate that tracked
  lclVar uses or defs have been removed and liveness may need to be
  recalculated. Previously this flag was only set upon eliminating a
  dead store.

JIT: extend gc descriptor for ppp quirked local struct (#13438)

When we expand the size of a local for the ppp quirk, also expand
the size of its gc descriptor, and mark the new "fields" as non-gc.

Fixes #13407.

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

Update CoreClr, CoreFx to preview2-25618-04, preview2-25618-02, respectively (master)

Revert "Re-port test project changes from release/2.0.0 to master (#13464)"

This reverts commit 7a386da8fc7067978a880445555942ccc6343be8.

Re-port test project changes from release/2.0.0 to master (#13464)

Re-port test project changes from release/2.0.0 to master

Mention Byref Store/Load VN and Helper Call Kills

These are two more areas that have received recent scrutiny.

Merge pull request #13314 from JosephTremoulet/loops

Lay out loop bodies contiguously

Categorize optimization ToDos

Update JitOptimizerTodoAssessment

Update notes on struct promotion, loop optimization, if-conversion, and
mulshift.  Add notes about switch lowering and write barriers.

Merge pull request #13459 from adiaaida/updatePgoCounts

Update optdata in master to version 20170817-0120

Update optdata in master to version 20170817-0120

Merge pull request #13456 from adiaaida/revertPGO

Revert "Update optdata in master to version 20170817-0120"

Revert "Update optdata in master to version 20170817-0120"

This reverts commit 694eb48ae21ae79ca745fa3448be87d144ff912d.

Merge pull request #13455 from adiaaida/updatePgoCounts

Update optdata in master to version 20170817-0120

Introduce COMPlus_GDBJitElfDump (#13448)

* Add COMPlus_GDBJitElfDump

* Fix Release build error

* Add flags in EEConfig

Update optdata in master to version 20170817-0120

Merge pull request #13400 from sergign60/arm_build

[armel tizen] Fix for CoreRT issue https://github.com/dotnet/coreclr/issues/13195

[Unix x64|Arm64] Correct canfastTailCall decisions

This will change how the fastTailCall decision is made for x64 unix and arm64.
Before this change the decision was based on the amount of incoming and outgoing
caller arguments like on Windows. This was incorrect on Unix x64 and Arm64
because one argument does not translate to one register or one stack slot use.

Before this change structs on Arm64 and Amd64 Unix could
pessimize when we could fastTailCall if they were engregisterable
and took more than one register.

This change also fixes several cases when determining to fastTailCall. It fixes
 #12479 and will cause a no fastTailCalls decisions for case #12468.

In addition this change adds several regression cases for #12479 and #12468. It
includes more logging ofr fastTailCall decisions, including a new COMPlus
variable named COMPlus_JitReportFastTailCallDecisions, which can be toggled with
COMPlus_JitReportFastTailCallDecisions=1.

Update CoreClr, CoreFx to preview2-25618-04, preview2-25618-02, respectively

Fix for CoreRT issue https://github.com/dotnet/coreclr/issues/13195

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

Merge nmirror to master

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

Typo (#13444)

Merge pull request #13445 from hseok-oh/ryujit/fix_13421

[RyuJIT/ARM32] Set carry bit: neg operation

Merge pull request #13442 from BruceForstall/EnableSuperPmiCoreDisAsmDiffs2

Enable SuperPMI asm diffs using CoreDisTools

Port fixes in DotNet-CoreClr-Trusted-Linux.json to unblock coreclr (#13446)

* Port fixes in DotNet-CoreClr-Trusted-Linux.json to unblock coreclr official run.

Port this commit:
https://github.com/dotnet/coreclr/commit/66d0fed853589e19336b6486f05f9218019ca4c9

* Remove extra new line at end of the file

Remove extra new line at end of the file

* Remove end line

Remove end line

Split fgComputeLifeLocal's subcases into separate functions.

There are three cases that are relevant here:
- Defs of tracked locals
- Uses of tracked locals
- Appearances of untracked locals

Each of these cases has been moved into its own function.

[RyuJIT/ARM32] Set carry bit: neg operation

Set carry bit when we generate ARM32 code for neg operation.

Enable SuperPMI asm diffs using CoreDisTools

Change SuperPMI to dynamically load coredistools.dll on demand.

Merge pull request #12632 from rartemev/issue_12464

Fix legacy backend crash on assert #12464

Migrate Git clone from GitHub to VSO (#13430)

* Migrate Git clone from GitHub to VSO

Migrate Git clone from GitHub to VSO

* Correct var name

Correct var name

* Remove PB_ as they are unnecessary and confusing.

Remove PB_ as they are unnecessary and confusing.

* Add dependent vars.

Add dependent vars.

* Fix repo name.

Fix repo name.

Fix more contract violations

Add perf test

Refactor loop identification into a class

This is mainly done to increase readability, as `optFindNaturalLoops`
had grown excessively large.  It also facilitates re-using code to fix
up fallthrough, and skipping past CallFinally/BBJ_ALWAYS pairs rather
than aborting once they're found.

Lay out loop bodies contiguously

Rearrange basic blocks during loop identification so that loop bodies
are kept contiguous when possible.  Blocks in the lexical range of the
loop which do not participate in the flow cycle (which typically
correspond to code associated with early exits using `break` or
`return`) are moved out below the loop when possible without breaking EH
region nesting.  The target insertion point, when possible, is chosen to
be the first spot below the loop that will not break up fall-through.

Layout can significantly affect the performance of loops, particularly
small search loops, by avoiding the taken branch on the hot path,
improving the locality of the code fetched while iterating the loop, and
potentially aiding loop stream detection.

Resolves #9692.

Fix legacy backend crash

Revert test part "Enable RedHat 6 in coreclr master (#13315)" to unblock CI.

This reverts commit b2b5fef27a4cb68571549f2e19660f39bd76467b.

Revert PR 13375 (#13436)

Merge pull request #13073 from rartemev/add_minopts_scenario

Added perf and perf_minopts scenarios for legacy backend

Add minopts scenario for Legacy backend

Merge pull request #13426 from JosephTremoulet/ConservativeConstant

Use conservative VN in CSE candidate const check

Enable RedHat 6 in coreclr master (#13315)

* Enable RedHat 6 in coreclr master

Enable RedHat 6 in coreclr master.

This is identical as the approved PR to enable RedHat 6 in coreclr
release/2.0.0:
https://github.com/dotnet/coreclr/pull/13301

* Correct Rid to match /src/.nuget/dirs.props

Correct Rid to match /src/.nuget/dirs.props

* Update dockertag and add logic to detect RHEL6 in init-tools.sh

Update dockertag and add logic to detect RHEL6 in init-tools.sh

* Port changes from Release/2.0.0 to fix coreclr RHEL 6 official runs.

Port changes from Release/2.0.0 to fix coreclr RHEL 6 official runs.

* Remove empty quotes.

Remove empty quotes.

Merge pull request #13419 from wateret/fix-arm-lea-needless-mov

[RyuJIT/arm32] Fix LEA codegen

Merge pull request #13198 from CarolEidt/LowerContain

Move containment analysis to 1st phase of Lowering

Support GDBJIT on NI/IL_STUBS (#13417)

* Support GDBJIT on NI/IL_STUBS

* Move tls_isSymReaderInProgress into gdbjit.cpp

Merge pull request #13100 from sjsinju/currentInterval2

[RyuJIT/ARM32] Add additional unassigning if the currentInterval has TYP_DOUBLE

Use conservative VN in CSE candidate const check

The check in CSE is supposed to leave code alone that constant prop
(done by VN-based Assertion Prop) is going to handle, but since that
constant prop code only propagates based on conservative VN, the check
in CSE needs to likewise use conservative VN to determine what to skip,
or else neither phase will eliminate the redundancy.

Fixes #6234.

Merge pull request #13406 from ericstj/updateDependencies

Update dependencies

Add span-based Version methods

The span-based {Try}Parse is allocation-free, and as the string-based {Try}Parse uses it, it's also now allocation-free, rather than allocating a string[] from a split and a string for each component in the version.

Fix AssemblyName(string) constructor's version parsing (#13373)

Fix AssemblyName(string) constructor's version parsing

Functional fix for https://github.com/dotnet/corefx/issues/22663
- Allow fewer version components
- Match .NET Framework behavior in several cases. Major and minor version must be specified for the version to be used.
- Used zero for unspecified build/revision. This is different from .NET Framework but the loader also behaves differently. Details are in code comments.

[RyuJIT/arm32] Fix LEA codegen

Do not generate `mov` when src and dst are the same
when we generate code for LEA node.

Fix #13418

Merge pull request #13409 from stephentoub/encoder_span

Add span-based overloads to Encoder/Decoder

Fixes for code review feedback

Update coreclr tests to netcoreapp2.1

`archiveSettings.addFiles` uses glob instead of path. (#13403)

- Files were not being archived after last changes.

Add span-based overloads to Encoder/Decoder

Added BinaryWriter Span-based APIs (#13378)

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

Fixed Android AMD64 build. (#13402)

Introduce FEATURE_GDBJIT_FRAME feature (#13283)

Add Span-based {Try}Parse methods to primitive types (#13389)

* Add Span-based {Try}Parse methods to primitive types

Adds Parse and TryParse methods to Boolean, Byte, Double, Int16, Int32, Int64, SByte, Single, UInt16, UInt32, UInt64, and Decimal.

* Address PR feedback

- Make delegation between overloads as consistent as possible across the primitive types: Boolean, SByte/Byte, Int16/UInt16, and Single/Double were doing it one way, whereas Decimal, Int32/UInt32, and Int64/UInt64 were doing it another way (most of this inconsistency was preexisting this PR, but my previous commit doubled-down on the inconsistency).  Changed the former to be like the latter.

Move Containment Analysis to Lowering

- Ensure that the only changes to child nodes are the specification of srcCandidates
(i.e. the register requirements of the parent/consumer)
- Since this now occurs before redoing dataflow, we don't have complete information on
what lclVars are containable. To mitigate this:
  - In `TreeNodeInfoInit()`, if a lclVar is marked `isRegOptional` but is now untracked,
  change it to be contained.
  - Mark decomposed/promoted fields of long params as `lvDoNotEnregister`
 - Ensure that any new code added after `Lowering` is analyzed as necessary.

- When a value becomes unused, it is no longer contained. (This is needed now because liveness runs after containment analysis)
- Add dumping of nodes deleted as unused.
- Need to set byteable regs, as needed on contained RMW memory ops.

Merge pull request #13208 from jashook/add_gc_stress_arm_metadata

Add arm32 GCStress excludes

Merge pull request #13377 from jashook/arm64_alignment

Restrict alignment assert to only if volatile

Restrict alignment assert to only if volatile

Fix AsyncValueTaskMethodBuilder.Create for corert (#13396)

I changed AsyncValueTaskMethodBuilder.Create to avoid calling the nop AsyncTaskMethodBuilder.Create, but it turns out on corert that AsyncTaskMethodBuilder.Create actually does additional work if the debugger is attached, so we should still call it there.

Don't run arm64 on every commit (#13384)

We cannot keep up currently

@jashook PTAL