Merge pull request #14717 from CarolEidt/Fix14624

Fix FixedReg assert

Don't wake up other GC threads when we decide to not proceed with the GC (#14719)

Enable Portable Pdb generation (#14721)

When https://github.com/dotnet/corefx/pull/24025 goes through to change coreFX over to use portable PDBs we want to do the same for System.private.Corelib.pdb in CoreCLR.   This change does this.

Delete dead code (#14703)

Revise class hierarchy for some EqualityComparer<T> subclasses (#14723)

Make GenericEqualityComparer<T> a final class and reparent
NonRandomizedStringEqualityComparer.

Also add a note to the jit sources indicating why we can't be more aggressive
about devirtualizing `get_Default` if we have a shared type (in particular
__Canon).

Merge pull request #14715 from jashook/14667_regression_test

Add regression test for double counting retbuf

Revert "Move to DotNetCore-Build agent pool (#14693)" (#14731)

This reverts commit 4103b6276cd5c3d497cd4a20de14bda6a98495cb.

Merge pull request #14515 from 4creators/master

Fix Windows arm target cross build: detect prerelease VS2017, stay in repo root

[Local GC] Implement loader protocol for a standalone GC (#14663)

* First cut of the load protocol

* Implement for non-standalone GC

* Initial working implementation

* First steps towards not using GetProcAddress when not using a standalone GC

* Factor out loading routines into standalone and non-standalone cases

* Remove the FEATURE_STANDALONE_GC_ONLY build

* Code cleanup and comments

* Comments for the version numbers

* Use more appropriate type for config string

* add GC_LOAD_STATUS_BEFORE_START to disambiguate failures before the start of the load and failures at the beginning of the load

* FEATURE_STANDALONE_GC on by default

* Implement YieldProcessor and MemoryBarrier for arm and arm64

* Remove missed FEATURE_STANDALONE_GC feature check

Address code review feedback

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

Merge pull request #14640 from briansull/ind-icon

Added gtNewIndOfIconHandleNode

Merge pull request #14678 from fiigii/vex

[RyuJIT] Decoupling SIMD levels from instruction sets

Add regression test for double counting retbuf

This relates to #14667.

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)

Fixes for zero diffs

Merge pull request #14709 from wtgodbe/TransportFeed

Restore from transport feed in test build

Fix FixedReg assert

With JitStressRegs (various values) it is possible to have RefPositions that have a single candidate, even though they are not really a fixed reference to the given register.

Fix #14624

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

Fix callee argument count bug in fgCanFastTailCall

Changes for Zero Asm Diffs

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

Decoupling SIMD levels from instruction sets

Added gtNewIndOfIconHandleNode
Renamed GTF_ICON_CIDMID_HDL to GTF_ICON_MID_HDL
Added a standard function header comment for gtNewIndOfIconHandleNode
Made compiletimeHandle a required argument instead of an option argument for gtNewIconEmbHndNode
Added new isInvariant arg to gtNewIndOfIconHandleNode, it will set GTF_IND_INVARIANT when true
Converted IBC Block Counts to use new gtNewIndOfIconHandleNode method
Simplified gtNewStringLiteralNode

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