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

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

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

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

Add arm32 GCStress excludes

Merge pull request #13390 from stephentoub/avtmbcreate

Remove overhead from AsyncValueTaskMethodBuilder.Create

Small perf infrastructure changes (#13325)

- Move xUnit Perf Api output files to a folder `Logs` under sandbox
- Made necessary changes to `perf.groovy` to archive artifacts
- Added a switch -outputdir to the run-xunit-perf* scripts
- Enable running all CoreClr scenarios (JitBench & IlLink) despite failure
- Move sandbox to bin folder
- Use the default output dir: `.\bin\sandbox`

Remove overhead from AsyncValueTaskMethodBuilder.Create

AsyncValueTaskMethodBuilder.Create wraps an AsyncTaskMethodBuilder and thus initializes its _methodBuilder field to AsyncTaskMethodBuilder.Create.  But even though AsyncTaskMethodBuilder.Create just returns default(AsyncTaskMethodBuilder), a non-trivial amount of code is being generated into AsyncValueTaskMethodBuilder.Create, even though it's really just returning default(AsyncValueTaskMethodBuilder).  So while the "right" thing to do is to call Create, this commit just skips it and adds a comment that if AsyncTaskMethodBuilder.Create ever becomes more than a nop, various callers would need to be updated as well (though I doubt we'd ever change it).

This reduces the overhead of a nop ValueTask-returning async method by ~30%.

Fix formatting

Get sort version dynamically and expose ICU version from native component (#13382)

* Support ICU Version

* Add the assert

Fix genIsValidDoubleReg not found

[RyuJIT/ARM32] Add additional unassignToInterval if the currentInterval is TYP_DOUBLE

Implement out of context stack unwinder (#13302)

Implement out of context stack unwinder

Decode the eh frame info found in the in-memory module image
and pass it back to the remote libunwind8 to do the unwind.

Added remote-unwind.cpp for all the out of context unwind code.

Added an all managed threads option -all the "clrstack" (sos ClrStack).

The IDebugDataTarget4 feature needs to be enabled for OS X.

Add libunwind license notice to third party notices file.

Add enforcepgo to official builds (#13375)

Merge pull request #13029 from mikedn/disp-bbid

Dump basic block ids instead of pointers

Use dashes for build options (#13353)

* Fix IndexOutOfRangeException in PayloadNames property

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.

* To be consistant with the other build.cmd files use - for options

* Update docs to track change in option convention

* Updated Unix script to use - for options.

* Fix build break

* Fix Whitespace (review feedback)

* Another fix

Allocate FileMappingImmutableData::szFileName and CFileProcessLocalData::unix_filename strings dynamically (#13374)

Merge pull request #13366 from mikedn/imp-init-array

Do not use GT_INDEX in impInitializeArrayIntrinsic

JIT: fix CHK/REL diff from fgRetypeImplicitByRefArgs (#13372)

Make sure some important code is not under DEBUG.

Fixes #13358.

Skip null checks for implicit byref argument references (#13355)

```
struct S
{
    public int _a;
    public int _b;
    public int _c;
    public int _d;
};

static void foo(S s)
{
    bar(ref s._b);
}
```

Before:

```
sub      rsp, 40
cmp      dword ptr [rcx], ecx // unnecessary
add      rcx, 4
call     My:bar(byref)
nop
add      rsp, 40
ret
```

After:
```
sub      rsp, 40
add      rcx, 4
call     My:bar(byref)
nop
add      rsp, 40
ret
```

Add full URL to the FileVersion information created a build time (#13352)

* Tag dlls with the URL of the github commit

* Fix name of BuildVersion file for non-official builds

This fix does something very safe (only overrides the name of the buildVersionFile with
a name from the OfficialBuildId variable if that variable is non-empty.

This is importnat because otherwise is that the override ALWAYS kicks in and sometimes
corrupts the name, which has the effect of throwning the information in the BuildVersionFile
(namely the version number) out

* Update to latest build tools.

Merge pull request #13310 from shimingsg/v-shige/add-perftc-0810

add perf test cases(GCSmall, GCHandle, LOHSmooth) to coreclr

Merge pull request #13334 from shimingsg/v-shige/add-perftc-0811

add perf test cases(MicroBench, MemoryPressure, etc) to coreclr

Merge pull request #13362 from shimingsg/v-shige/add-perftc-0814-2

add perf test cases(ServerSpin, ToBoxOrNot, etc.) to coreclr

Merge pull request #13328 from hseok-oh/ryujit/tmp_high_cost

[RyuJIT/ARM32] TmpVar for expensive struct argument

Fix race condition for some threading tests using GC.KeepAlive (#13351)

* Fix race condition for some threading tests using GC.KeepAlive

* Removing unnecessary change to null array test

* Removing extra space

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

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

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

[RyuJIT/ARM32] Fix regression

Dump basic block ids instead of pointers

[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"

Do not use GT_INDEX in impInitializeArrayIntrinsic

We know that array size is larger than 0 so we do not need a range check.  This range check survives until the RangeCheck phase so it creates more work of the phases before it. It is also very unlikely to enable other optimizations, subsequent `a[0]` range checks can still be eliminated because the array length is known and constant.

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

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>

add perf test cases to coreclr

remove line includes CopyToOutputDirectory

replace p/invokes into kernel32 with cross-platform implementations

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)

add perf test cases to coreclr

[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()

[RyuJIT/ARM32] TmpVar for expensive struct argument

Assign expensive argument to tmpVar on morphing phase

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