Merge pull request #14730 from fiigii/SSE4

[RyuJIT] Rename and Simplify SSE3_4 flag

[Local GC] Fix an issue where the size of ScanContext differs between EE and GC (#14747)

Fix build warning (#14751)

Update BuildTools to prerelease-02130-03 (#14750)

Merge pull request #14683 from weshaggard/MergeDevRelease200

Merge changes from dev/release/2.0.0

Update code in AwaitUnsafeOnCompleted to use interface matching

Given that the jit can now avoid boxing on some interface calls to value types,
generalize the patterns introduced in AwaitUnsafeOnCompleted in #14718 by
using interfaces instead of checking for specific types.

Also move the catch-all processing back in line as the jit can now fold the
interface tests early and not pull in the EH portion of the method unless it is
needed.

JIT: optimize calls on boxed objects

For calls with boxed this objects, invoke the unboxed entry point if available
and modify the box to copy the original value or struct to a local. Pass the
address of this local as the first argument to the unboxed entry point.

Defer for cases where the unboxed entry point requires an extra type parameter.

This may enable subsequent inlining of the method, and if the method has other
boxed parameters, may enable undoing those boxes as well.

The jit is not yet as proficient at eliminating the struct copies, but they
are present with or without this change, and some may even be required when the
methods mutate the fields of `this`.

Closes #14213.

Merge pull request #14677 from CarolEidt/Fix14626

Avoid allocating IP0 and IP1

Update dependencies to latest with new property names

Update dependency properties used in PerfHarness project

Add RoslynIncompatibleMsbuildVersion condition on Roslyn import

Due to merge conflict this condition got removed so adding it back
to allow for older msbuild versions to work (at least temporarily).

Add blob feed to the restore sources

Merge the blob feed restore source added in
https://github.com/dotnet/coreclr/pull/14709

Switch to sourcing in init-tools to enable the return statements to work

Move CoreCLRPackageVersion property useage

Also did some small clean-up:
Fix missing tag from bad merge
Fix spacing in init-tools.sh

Use restore with no --source for optdata proj feed

This lets dotnet pick up RestoreSources hierarchically rather than only use what's predefined in DotnetRestoreCommand (in dir.props).

Implement auto dependency flow Repo API (#14518)

Pass overridden packages directory to CMake

Align init-tools scripts with other source-build repos

Normalize the alpine and rhel RIDs

See: https://github.com/dotnet/core-setup/commit/c8a27076f93c3107759b8a86a02ea4f40d100b67

We need to strip the last part of the version number for alpine and rhel because
they are compatible and we don't want the specific versions.

Remove coreclr patches from source-build repo

Updates the branch to include patches that were applied in source-build:

0001-Respect-NUGET_PACKAGES-if-it-is-set.patch
0002-Unify-CLI-and-BuildTools-Versions.patch
0003-Add-clear-to-NuGet.config.patch
0004-PATCH-Use-clang-3.9-by-default.patch

Update BuildTools, CoreClr, CoreFx, PgoData to prerelease-02130-01, preview1-25829-02, preview1-25830-02, master-20171030-0047, respectively (#14714)

Rename and simplify SSE3_4 to SSE4

Fix gtNewStringLiteralNode (#14744)

- there is IAT_VALUE case from constructStringLiteral(CoreRT)

Signed-off-by: Petr Bred <bredpetr@gmail.com>

Simplify Comparer/EqualityComparers (#14737)

- Make EnumEqualityComparer sealed. This helps with JIT optimizations.
- Delete SByteEnumEqualityComparer and ShortEnumEqualityComparer because of they are not needed anymore. The optimization that they were introduced for is done by the JIT now.
- Deleted ForwardedFrom attributes on types that do not need them

Fix spelling error in comment (#14745)

delete unused functions and arguments in lower. (#14726)

* move putArg inside

* delete unused function

LowerPutArgStk with two arguments was never used.

* delete extra argument

Merge pull request #14456 from fiigii/crc32

Enable Crc32 , Popcnt, Lzcnt intrinsics

Merge pull request #14712 from BruceForstall/VS2017Arm64

Allow building arm64 using VS2017 tools

Fix for issue EventSource MultiEnable bug (#14729)

Fixes issue #14728

Merge pull request #14733 from CarolEidt/Fix14732

Fix another overly agressive assert

Formatting & typo

Update Hashtable.cs (#14741)

Fix little misspelling.

Fix another overly agressive assert

Fix #14732

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

Allow building arm64 using VS2017 tools

VS2017 now includes ARM64 compilers. Change the build scripts so
that the normal VC tools path is used for the builds for ARM64.
However, if -toolset_dir is passed, then use the specified tools,
as before.

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.

Enable Crc32 , Popcnt, Lzcnt intrinsics

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)

Avoid allocating IP0 and IP1

Revert to prior behavior.

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