Merge branch 'master' of https://github.com/dotnet/coreclr

Remove adiaaida from list of arm64 users (#14710)

Add janvorli to the list of arm64 users (#14718)

Implemented TailCall ELT hook for arm32 Linux (#14713)

* Added TailCall ELT hook for arm32 Linux

* fixed review

Make EventSourcException serializable in corert scenarios (#14716)

Merge pull request #14709 from wtgodbe/TransportFeed

Restore from transport feed in test build

Merge pull request #14667 from hseok-oh/ryujit/ftc_callee_retbuf

Fix callee argument count bug in fgCanFastTailCall

Restore from transport feed in test build

Move to DotNetCore-Build agent pool (#14693)

[Local GC] Move IsGCThread and IsGCSpecialThread to GCToEEInterface (#14685)

* [Local GC] Move IsGCThread and IsGCSpecialThread to GCToEEInterface

* Fix the windows and sample builds

Update BuildTools, CoreClr, PgoData to prerelease-02126-01, preview1-25826-01, master-20171026-0034, respectively (#14704)

Port EventSource Nuget Package Changes (#14669)

* Add missing resource strings.

* Bring back code changes required to build the nuget package.

Fix source/line info on Windows for Windows PDBs. (#14696)

Now attempts to load the diasymreader from the coreclr module path.

Issue #21079

Update CoreFx, PgoData to preview1-25825-08, master-20171025-1102, respectively (#14694)

Merge pull request #14695 from wtgodbe/DisableVerify

Disable VerifyDependencies target in package restore

Merge pull request #14697 from BruceForstall/FixCorefxTesting

Fix CoreFx testing

Fix CoreFx testing

Currently, build.cmd will pass through all arguments starting with
an argument it doesn't recognize. Since it doesn't process/recognize
'-priority', make sure this is the last argument passed.

Disable VerifyDependencies target in package restore

Merge pull request #14138 from jashook/arm64_32_multiarch_documentation

Add multiarch arm64(32) Documentation

Add multiarch arm64(32) Documentation

Helps setup running armhf code on an arm64 kernal.

Merge pull request #14668 from hqueue/arm/reenable_armel_ci

[ARM] Re-enable armel CI

Merge pull request #14691 from adiaaida/fixenforcepgooption

Fix enforcepgo option in netci.groovy

Marking Memory struct as readonly, fixing corefx#23809 (#14684)

[ARM64/Windows] optimal value of CopyThreshold for Windows after performance analysis (#14674)

Fix enforcepgo option in netci.groovy

The windows release builds are broken because they call build with
enforcepgo instead of -enforcepgo. This change fixes the parameter.

Merge pull request #14588 from hqueue/arm/ryujit/issue_14374_pr1

[RyuJIT/ARM32] Fix to find a free temp double register correctly

[RyuJIT/ARM32] Fix to find a free temp double register correctly

When finding a free temporary double register to resolve conflicting edges,
we have to consider both two float registers consisting a double register.

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

Apply review feedback

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

delete unused local variables from jit sources (#14679)

Update BuildTools, CoreClr, CoreFx to prerelease-02124-02, preview1-25824-51, preview1-25824-02, respectively (#14671)

fix lvaStressLclFldCB (#14676)

Don't create trees with the same ID.

Merge pull request #14658 from mikedn/legacy-reg-track

Ifdef out legacy RegTracker code

Merge pull request #14619 from briansull/emitter-cleanup

Cleanup unused emitter arguments

Delete dead code (#14673)

- assemblyusagelog
- compatibilityflags
- xmlparser

Merge pull request #14662 from wtgodbe/FeedTaskPackage

Use new feed tasks package & auto-update version

Use new feed tasks package & auto-update version

Ifdef out legacy RegTracker code

Cleanup unused emitter arguments
Removed unused idClsCookie from struct instrDescDebugInfo
Cleaned up several ifdefs

Update BuildTools, CoreClr, CoreFx to prerelease-02120-09, preview1-25822-02, preview1-25824-01, respectively (#14565)

[ARM] Re-enable armel CI

Because armel uses different implementation compared to armlb,
let's reenable armel CI.

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

Search/replace _DEBUG => DEBUG

Search/replace Debug.Assert(false, => Debug.Fail(

Cleanup BCLDebug

- Delete correctness, performance and safe handle stacktrace logging
- Change remaining BCLDebug.Assert uses to Debug.Assert

Fix callee argument count bug in fgCanFastTailCall

Fix bug when callee uses return buffer:
return buffer argument is already generated in callee->gtCallArgs on import phase
So we should not increment calleeArgRegCount explicitly.

Report registers as dead in GCInfo before the RhpPInvoke helper. (#14664)

* move killGCRefs to the compiler

It makes it accessible from codegen.

* use killGCRefs in codeGen

It terminates live of GCRefs before RhpPInvoke.

* small ref

profiler changes for tiered compilation (#14612)

Add new apis for profiler to use with tiered jitting.

Merge pull request #14647 from BruceForstall/AddArm64FrameLayoutDoc

Add original ARM64 JIT frame layout design document

Merge branch 'master' of https://github.com/dotnet/coreclr

Cleanup VsDevCmd.bat usage and improve build system messages to ease diagnostics

Fix for #12609 - add option to disable tiered compilation for profilers  (#14643)

add option to disable tiered compilation for profilers

Fix RID extraction in packages and test build for Alpine (#14656)

* Fix RID extraction in packages and test build for Alpine

The host RID extraction in build-packages.sh and build-test.sh
was not matching the one in build.sh

* Reflect feedback

Merge pull request #14649 from mikedn/float-absneg

Always use XORPS/ANDPS for FP NEG/ABS

[Arm64] SIMD simple defines (#14628)

* [Arm64] SIMD simple defines

* Fix #else

[Arm64] SIMD lsra (#14631)

* [Arm64] SIMD lsra

* Respond to feedback

[Arm64] SIMD ins_Load (#14636)

* [Arm64] SIMD ins_Load

* Respond to feedback

Merge pull request #14528 from jashook/pri1_rework

Pri0 and pri1 rework

Merge pull request #14639 from sdmaclea/PR-ARM64-SIMD-bogus-assert

[Arm64] Fix bogus assert

**Changes all outerloop jobs to explicitly use pri1 builds of the tests. It also rebrands PR Triggered pr triggered jobs to "innerloop" and will run pri0 tests.**

The change also includes netci.groovy cleanup. The list of Pr triggered jobs will change from:

Old PR Triggers (17):

```
CentOS7.1 x64 Debug Build and Test (debug_centos7.1_prtest)
CentOS7.1 x64 Release Priority 1 Build and Test (x64_release_centos7.1_pri1_flow_prtest)
OSX10.12 x64 Checked Build and Test (checked_osx10.12_flow_prtest)
Tizen armel Cross Debug Build (armel_cross_debug_tizen_prtest)
Tizen armel Cross Release Build (armel_cross_release_tizen_prtest)
Ubuntu arm64 Cross Debug Build (arm64_debug_small_page_size_prtest)
Ubuntu armlb Cross Release Build (armlb_cross_release_ubuntu_prtest)
Ubuntu x64 Checked Build and Test (checked_ubuntu_flow_prtest)
Ubuntu x64 Formatting (x64_ubuntu_formatting_prtest)
Ubuntu16.04 armlb Cross Debug Build (armlb_cross_debug_ubuntu16.04_prtest)
Windows_NT arm Cross Checked Build and Test (arm_cross_checked_windows_nt_prtest)
Windows_NT arm64 Cross Debug Build (arm64_cross_debug_windows_nt_prtest)
Windows_NT armlb Cross Checked Build and Test (armlb_cross_checked_windows_nt_prtest)
Windows_NT x64 Debug Build and Test (debug_windows_nt_prtest)
Windows_NT x64 Formatting (x64_windows_nt_formatting_prtest)
Windows_NT x64 Release Priority 1 Build and Test (x64_release_windows_nt_pri1_prtest)
Windows_NT x86 Checked Build and Test (x86_checked_windows_nt_prtest)
```

New PR Triggers (15):

```
CentOS7.1 x64 Checked Innerloop Build and Test (checked_centos7.1_flow_prtest)
CentOS7.1 x64 Debug Innerloop Build (debug_centos7.1_prtest)
OSX10.12 x64 Checked Innerloop Build and Test (checked_osx10.12_flow_prtest)
Ubuntu arm64 Debug Build (arm64_debug_small_page_size_prtest)
Ubuntu armlb Innerloop Cross Debug Build (armlb_cross_debug_ubuntu_prtest)
Ubuntu x64 Checked Innerloop Build and Test (checked_ubuntu_flow_prtest)
Ubuntu x64 Innerloop Formatting (x64_ubuntu_formatting_prtest)
Ubuntu16.04 armlb Innerloop Cross Debug Build (armlb_cross_debug_ubuntu16.04_prtest)
Windows_NT arm Cross Checked Innerloop Build and Test (arm_cross_checked_windows_nt_prtest)
Windows_NT arm64 Cross Checked Innerloop Build and Test (arm64_cross_checked_windows_nt_prtest)
Windows_NT arm64 Cross Debug Innerloop Build (arm64_cross_debug_windows_nt_prtest)
Windows_NT armlb Cross Checked Innerloop Build and Test (armlb_cross_checked_windows_nt_prtest)
Windows_NT x64 Checked Innerloop Build and Test (checked_windows_nt_prtest)
Windows_NT x64 Innerloop Formatting (x64_windows_nt_formatting_prtest)
Windows_NT x86 Checked Innerloop Build and Test (x86_checked_windows_nt_prtest)
```

PR Trigger Change summary:

```
-- Added Jobs --
CentOS7.1 x64 Checked Innerloop Build and Test (checked_centos7.1_flow_prtest)
Ubuntu armlb Cross Innerloop Debug Build (armlb_cross_debug_ubuntu_prtest)
Windows_NT arm64 Cross Checked Innerloop Build and Test (arm64_cross_checked_windows_nt_prtest)
Windows_NT x64 Checked Innerloop Build and Test (checked_windows_nt_prtest)

-- Removed Jobs --
CentOS7.1 x64 Release Priority 1 Build and Test (x64_release_centos7.1_pri1_flow_prtest)
Tizen armel Cross Debug Build (armel_cross_debug_tizen_prtest)
Tizen armel Cross Release Build (armel_cross_release_tizen_prtest)
Ubuntu armlb Cross Release Build (armlb_cross_release_ubuntu_prtest)
Windows_NT x64 Debug Build and Test (debug_windows_nt_prtest)
Windows_NT x64 Release Priority 1 Build and Test (x64_release_windows_nt_pri1_prtest)

-- Changed Jobs --
CentOS7.1 x64 Debug Build and Test -> CentOS7.1 x64 Debug Innerloop Build (debug_centos7.1_prtest)
OSX10.12 x64 Checked Build and Test -> OSX10.12 x64 Checked Innerloop Build and Test (checked_osx10.12_flow_prtest)
Ubuntu arm64 Cross Debug Build -> Ubuntu arm64 Cross Debug Innerloop Build
Ubuntu x64 Checked Build and Test -> Ubuntu x64 Checked Innerloop Build and Test (checked_ubuntu_flow_prtest)
Ubuntu x64 Formatting -> Ubuntu x64 Innerloop Formatting (x64_ubuntu_formatting_prtest)
Ubuntu16.04 armlb Cross Debug Build -> Ubuntu16.04 armlb Cross Debug Innerloop Build (armlb_cross_debug_ubuntu16.04_prtest)
Windows_NT arm Cross Checked Build and Test -> Windows_NT arm Cross Checked Innerloop Build and Test (arm_cross_checked_windows_nt_prtest)
Windows_NT arm64 Cross Debug Build -> Windows_NT arm64 Cross Debug Innerloop Build (arm64_cross_debug_windows_nt_prtest)
Windows_NT armlb Cross Checked Build and Test -> Windows_NT armlb Cross Checked Innerloop Build and Test (armlb_cross_checked_windows_nt_prtest)
Windows_NT x64 Formatting -> Windows_NT x64 Innerloop Formatting (x64_windows_nt_formatting_prtest)
Windows_NT x86 Checked Build and Test -> Windows_NT x86 Checked Innerloop Build and Test (x86_checked_windows_nt_prtest)
```

Merge pull request #14598 from jashook/fix_build_test_sh_osx

Add build-test osx support

JIT: defer constant-return merging for debug codegen (#14642)

If we merge constant returns into a common point we may lose track of sequence
points. So inhibit this when we are generating debuggable code.

Fixes #14339.

Merge pull request #14621 from briansull/fix-hash

Fix gtHashValue to properly hash all the bits when we have a 64-bit item

Remove low value purpose comments (#14653)

Most of the purpose comments removed referred to the struct as a "class" which is confusing. In addition, these value types are simple enough they don't warrant a full purpose description. Resolves #13479.

Hide post exception stack frames (#14652)

Respond to feedback

Enable build pipeline for Alpine Linux 3.6 (#14587)

Always use XORPS/ANDPS for FP NEG/ABS

They do the same thing but they're one byte shorter. Saves a call to genGetInsForOper too.

Add original ARM64 JIT frame layout design document

Merge pull request #14632 from sdmaclea/PR-ARM64-SIMD-SIMD.CPP

[Arm64] SIMD simd.cpp

Merge pull request #14630 from sdmaclea/PR-ARM64-SIMD-LOWERING

[Arm64] Basic SIMD lowering

Merge pull request #14627 from sdmaclea/PR-ARM64-SIMD-EMITTERS

[Arm64] Add SIMD emitters

Merge pull request #14638 from BruceForstall/FixSimdMinOpts

Allow GT_CALL as BYREF operand for SIMD intrinsics

Update profiling API status (#14644)

We've been making progress testing APIs and fixing issues. The description in this doc had gotten well out of date.

Merge pull request #14620 from CarolEidt/RegSelectFix

Fix Register selection refactor bugs

Revert "Reenable PGO on Linux Release builds" (#14645)

Merge pull request #14625 from BruceForstall/FixDynBlkAssert

Remove incorrect assert for DynBlk codegen

Rename MemoryHandle PinnedPointer to Pointer and add property HasPointer. (#14604)

remove duplicate line

[Arm64] Fix bogus assert

Allow GT_CALL as BYREF operand for SIMD intrinsics

This is an extension of #13965: in the MinOpts case, we don't
have GT_RET_EXPR -- we have GT_CALL instead.

We now see IR like:
```
[000429] --C-G--N----              |  |  \--*  BLK(32)   simd32
[000413] --C-G-------              |  |     \--*  CALL      byref  System.Runtime.CompilerServices.Unsafe.AsRef
[000410] ------------              |  |        |     /--*  CNS_INT   int    2
[000411] ------------              |  |        |  /--*  MUL       int
[000409] ------------              |  |        |  |  \--*  CAST      int <- int
[000408] ------------              |  |        |  |     \--*  CNS_INT   int    16 Vector<T>.Count
[000412] ------------ arg0         |  |        \--*  ADD       int
[000407] ------------              |  |           \--*  LCL_VAR   int    V01 arg1
```

Whereas in the optimizing case, we see:
```
[000060] ------------              *  STMT      void  (IL   ???...  ???)
[000058] I-C-G-------              \--*  CALL      byref  System.Runtime.CompilerServices.Unsafe.AsRef (exactContextHnd=0x028FAFF8)
[000055] ------------                 |     /--*  CNS_INT   int    2
[000056] ------------                 |  /--*  MUL       int
[000054] ------------                 |  |  \--*  CAST      int <- int
[000053] ------------                 |  |     \--*  CNS_INT   int    16 Vector<T>.Count
[000057] ------------ arg0            \--*  ADD       int
[000052] ------------                    \--*  LCL_VAR   int    V01 arg1

...

[000076] --C----N----              |  |  \--*  BLK(32)   simd32
[000065] --C---------              |  |     \--*  RET_EXPR  byref (inl return from call [000058])
```

Fixes #14301

[Arm64] SIMD simd.cpp

[Arm64] Basic SIMD lowering

[Arm64] Add SIMD emitters

[Arm64] Add more SIMD instructions

Remove incorrect assert for DynBlk codegen

The assert specified a particular register for the `size` argument
to the memset helper call. However, it preceded the genConsumeBlockOp()
function which ensured that would be the case.

Fixes #14544

Disable clang format around an area where it is incorrect

Fix the gtHashValue to properly hash all the bits when we have a 64-bit item

[RyuJit/arm32] Do nothing if double is on stack. (#14603)

Fix Register selection refactor bugs

Fix #14617
Fix #14618

Merge pull request #14606 from CarolEidt/Fix14591

LSRA Arm64 consistent reg sets

Merge pull request #14609 from hseok-oh/ryujit/fix_14377_linux

[RyuJIT/ARM32] Remove NYI: struct return from multi-reg GT_CALL

[RyuJIT/ARM32] Fast tail call: code generation (#14445)

* Codegen for fast tail call

Codegen call and epilog for fast tail call

* Implementation for GT_START_NONGC

This implementation removes two NYI_ARM
Code generation for GT_START_NONGC which is used to prevent GC in fast tail call

* Define fast tail call target register and mask on ARMARCH

Define REG_FASTTAILCALL_TARGET and RBM_FASTTAILCALL_TARGET on ARMARCH
Modify lsra init and codegen to use these definition

* Merge genFnEpilog

Merge genFnEpilog for ARM32 and ARM64

* Fix bug in getFirstArgWithStackSlot

Fix bug in getFirstArgWithStackSlot: AMD64 and X86

[Local GC] Add a Standalone GC loader design document (#14435)

* Add a Standalone GC loader design document

* First round of feedback:
        1. Remove some stuff from the preamble that doesn't belong in a
        design document
        2. Simplify the three-variable loading approach to just use one
        variable that contains the path of a GC to load.
        3. Clean up the wording in a few places.
        4. Remove the "state machine" section and reference to state
        machines in general.

Merge branch 'master' of https://github.com/dotnet/coreclr

Update comments in TieredCompilationManager (#14610)

We've continued plugging away and the status in the comment was out-of-date.

[RyuJIT/ARM32] Remove NYI: struct return from multi-reg GT_CALL

Remove useless NYI

Merge pull request #14600 from BruceForstall/FixArmBadGCInfoForCpObj

Fix ARM bad GC info for CpObj

Merge pull request #14602 from benaadams/jit-formatting

Fix jit fromatting

LSRA Arm64 consistent reg sets

tryAllocateFreeReg() uses the RegOrder array to iterate over the available registers. This needs to be consistent with the available registers of the given type. Otherwise, allocateBusyReg() will assert when it finds a free register that should have been allocated in tryAllocateFreeReg().
Fix #14591

Fix jit fromatting

Fix ARM bad GC info for CpObj

In the case where the dst lives on the stack, after the first
gcref/byref was copied, we never set the type back to non-GC
for subsequent copies using the same temp register.