Put default interface support behind a COMPlus switch (#15233)

Merge pull request #15022 from MichalStrehovsky/master-mere

Merge master to dev/defaultintf

Merge branch 'master' into master-mere

Merge pull request #15017 from BruceForstall/DisableArm64Tests

Disable arm64 tests that fail in Debug CI runs

Add object null checks in Memory<T> APIs to support default (#14816)

* Add object null checks in Memory<T> APIs to support default

* Changing Empty property to return default value.

* Update use of object null checks

* Addressing PR feedback.

* Removing typeof char check.

* Fix typo

Fix CppCodeGen break with latest XCode (dotnet/corert#4926)

Latest XCode errors on ordered comparison of pointer with integer

Also, fixed some unnecessary always-true comparisons that the C++ compiler emitted warnings for.

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

Merge pull request #14929 from sdmaclea/PR-ARM64-SIMD-genStructReturn

[Arm64] SIMD genStructReturn

Merge pull request #14934 from sdmaclea/PR-ARM64-SIMD-genMultiRegCallStoreToLocal

[Arm64] SIMD genMultiRegCallStoreToLocal

Merge pull request #14975 from sdmaclea/PR-SIMD-genSIMDIntrinsicInit-0

[ARM64] genSIMDIntrinsicInit contain 0

Merge pull request #14976 from sdmaclea/PR-SIMD-genCodeForStoreLclVar-0

[ARM64] SIMD genCodeForStoreLclVar zero init

Update BuildToolsVersion.txt

Adding Memory CopyTo APIs (#15010)

Merge pull request #15018 from dotnet-bot/from-tfs

Merge changes from TFS

Merge pull request #14944 from sdmaclea/PR-ARM64-SIMD-Partial-Callee-Save-SIMD12

[ARM64] SIMD Partial Callee Save SIMD12

Disable arm64 tests that fail in Debug CI runs

Addresses #15016, until these can be investigated.

Merge pull request #14953 from mikedn/ssa-mem-track

Track actual SSA memory usage

Merge pull request #15013 from BruceForstall/DisableTests

Disable ARM tests

Merge pull request #14970 from jashook/better_smarty_archiving

Better smarty archiving

Disable VectorConvert_r/VectorConvert_ro for ARM

The are now failing due to #14665.

Enable long-running or resource-intensive GC tests

Specifically,
```
foregroundgc
lohfragmentation
536168
```

These are already disabled for arm64 and in issues.targets and the
various non-Windows *.txt files.

Merge pull request #14988 from briansull/fix-523532

Fix VSO 523532 - Possible mismatch in size on GT_EQ created for switch expansion

Merge pull request #14921 from BruceForstall/EnableRyuJitArm32Crossgen

Enable RyuJIT/arm32 crossgen compilation of S.P.C.dll

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

Merge nmirror to master

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

Merge pull request #14984 from jashook/add_new_publish_vars

Use new publish vars

Use new publish vars

Better smarty archiving

fix DevDiv521437

[tfs-changeset: 1680966]

check that VM set pAddr == nullptr (#15003)

When VM returns addr != nullptr from getHelperFtn, it must set pAddr to nullptr.

Enable RyuJIT/arm32 crossgen compilation of S.P.C.dll

Currently, an ARM build and CI test run uses the legacy backend
arm32 JIT. This change enables a build of RyuJIT/arm32 as a
cross-targeting protojit.dll (altjit) in the cross tools (placed
next to the x86 hosted crossgen.exe in the build), and uses that
RyuJIT/arm32 cross-compiler to build System.Private.CoreLib.dll.
The CI system when testing RyuJIT/arm32 will use this to test
the crossgen'ed System.Private.CoreLib.dll built by RyuJIT/arm32
for all "arm" architecture jobs.

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

Update BuildTools, CoreClr, CoreFx, PgoData to prerelease-02210-05, preview1-25913-02, preview1-25913-02, master-20171110-0018, respectively (master)

Update BuildTools, CoreClr, CoreFx, PgoData to prerelease-02210-05, preview1-25913-02, preview1-25913-02, master-20171110-0018, respectively

Fix sempahore spinning fix (#15002)

A recent fix around SemaphoreSlim's spinning has led to it spinning even when spinning isn't necessary, resulting in measurable overheads when trying to acquire a semaphore that has count available.  This fixes that by putting back the check to avoid spinning if count is known to be available.

Port https://github.com/dotnet/coreclr/pull/13148 to master (#14999)

Updating the xUnit Performance Api with bug fixes to the benchmark scenarios (#14986)

This newer version fixes collecting data on x86 where:
1. The Jit generate methods at runtime and that are not associated with any loaded module
2. Modules are loaded/unloaded multiple times
3. This newer version of the Api added some breaking changes to the Api interface

Allow non-abstract non-public interface methods in ILAsm (#14998)

This is to support the default interface method feature in C#.
See https://github.com/dotnet/csharplang/blob/5971e753a20a32e5f471a5439c88e667e32413f0/proposals/default-interface-methods.md

Remove unnecessary Exception._exceptionMethodString field (#14992)

Fix implementation of GCToOSInterface::VirtualCommit to ditch the pages (#14979)

The GC depends on the pages to be zeroed out when they are mapped in again

Move '-all' from filename to args in Mac build (#14990)

Fix VSO 523532 - Possible mismatch in size on GT_EQ created for switch expansion

Disable GC test requires much bigger amount of memory than arm32 machine may ever have (#14981)

Merge pull request #14973 from sdmaclea/PR-ARM64-VECTOR-LD-REG-SIZE

[Arm64] Do not change vector load register size

Run clean -all instead of just -b (#14978)

[Arm64] Do not change vector load register size

[ARM64] SIMD genCodeForStoreLclVar zero init

[ARM64] genSIMDIntrinsicInit contain 0

Merge pull request #14974 from jcagme/master

Removing retry parameters since they are not used anymore

Removing retry parameters since they are not used anymore

Added UnixEpoch to DateTime and DateTimeOffset. (#14648)

* Added UnixEpoch to DateTime and DateTimeOffset.

* Use ticks instead.

* Use DateTime.TicksTo1970

* Consolidated UnixEpochTicks

Separate Profile=On and Off for perf pipeline (#14963)

* Separate Profile=On and Off for perf pipeline

In pipeline jobs, we can separate these two runs into separate runs.
Doing so decreases the time spent running windows jobs since we are not
running them in sequence. Only run Profile=On scenarios for PRs. This
should reduce the time spent running the PR job.

This change also adds the xunit max iteration parameters.

[ARM64] SIMD Partial Callee Save SIMD12

Merge pull request #14708 from mikedn/rc-ssa

Change RangeCheck to use the available SSA data

Fix build break when VS2015 is not installed. (#14948)

* Fix build break when VS2015 is not installed.

If you ONLY install VS2017 (now the common scneario)
and build coreclr, it will fail  with COM with a Class_Not_Registered error

This is because a build-tools package is depending on an old COM object
that has gone way in VS2017.

While we sort getting a proper fix, this makes the work-around (which
is registering the DLL by hand) discoverable.

See https://github.com/dotnet/coreclr/issues/11305 for details.

* typo

Fix clean.cmd to work if no parameters are passed (#14969)

Merge pull request #14967 from jashook/small_netci_fixes

Fix arm(64) jobs

Fix arm(64) jobs

[Arm64] SIMD genMultiRegCallStoreToLocal

[Arm64] SIMD genStructReturn

Merge pull request #14961 from jashook/arm64_windows_nt_rework

Rework Arm64 Windows jobs to flow jobs

Change RangeCheck to use the available SSA data

The necessary information (the assignment node and the block where it appears) can be obtained from the SSA data, there is no need to build a map of SSA definitions. Doing that consumes almost half of the time taken by RangeCheck and a sizeable chunk of memory.

[Local GC] Unify background GC thread and server GC thread creation (#14821)

* Initial cut, ignoring thread affinity

* Integrate thread affinity

* Affinity for standalone Windows

* Add 'specialness' and the thread name as arguments to CreateThread

* First crack at unified implementation

* Set priority for server GC threads

* Remove unused parameter

* Address code review feedback and remove some dead code that broke the clang build

* Use char* on the interface instead of wchar_t (doesn't play well cross-platform)

* Rename IsGCSpecialThread -> CurrentThreadWasCreatedByGC

* Code review feedback and fix up the build

* rename CurrentThreadWasCreatedByGC -> WasCurrentThreadCreatedByGC

* Fix a contract violation when converting string encodings

* Thread::CreateUtilityThread returns a thread that is not suspended - restarting a non-suspended thread is incorrect

* CreateUnsuspendableThread -> CreateNonSuspendableThread

Merge pull request #14882 from 4creators/cleanall

[Infrastructure] Improve clean.{cmd|sh} by adding git clean -xdf cleanup step, unifying implementation with corefx and adding VBCScompiler.exe kill logic and

Update CoreCLR package to the freshly published version (#14962)

Rework Arm64 Windows jobs to flow jobs

This drops our dependency on the internal Arm64CI

Merge pull request dotnet/corefx#25092 from brianrob/missing_define

Add PLATFORM_WINDOWS define for MDT EventSource package

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

Remove Linux perf from pr pipeline job (#14943)

* Remove Linux perf from pr pipeline job

We don't have a lot of linux perf machines and the sheer number of PR
jobs is causing them to be overloaded, so the perf leg is taking too
long. Disable them for prs for now until we can increase capacity.

* Disable baseline jobs

Restore #13148 (#14958)

Use ILAsm from this branch. This was lost in merges because the property got renamed.

We should really just port the ILAsm part of the feature to master...

Merge pull request #14954 from dotnet/revert-14722-arm64_windows_nt_rework

Revert "Rework Arm64 Windows jobs to flow jobs"

Revert "Rework Arm64 Windows jobs to flow jobs"

Track actual SSA memory usage

Currently SsaBuilder use comp->getAllocator() in a lot of places so a lot of the memory it allocates lands in CMK_Generic instead of CMK_SSA.

Same for CopyProp phase.

Merge pull request #14722 from jashook/arm64_windows_nt_rework

Rework Arm64 Windows jobs to flow jobs

Rework Arm64 Windows jobs to flow jobs

This drops our dependency on the internal Arm64CI

[RyuJIT/arm32] Fix GC hole in address mode generation requiring temps (#14949)

* [RyuJIT/arm32] Fix GC hole in address mode generation requiring temps

In cases requiring a temp register to compute a portion of an LEA,
where the partial computation involves a GCREF/BYREF base register,
the temp register must be marked as a BYREF, so it can get
updated during a GC.

This fixes a case like this:
```
add r2, r0, r1 LSL 2
ldr r0, [r2+24]
```

where `r2` needs to be marked BYREF.

Related to #14856.

* Formatting

Merge pull request #14906 from stephentoub/add_memory_string

Add string support to ReadOnlyMemory<char>

Merge pull request #14918 from BruceForstall/NoLegacyUnsafeCpBlk

Don't allow unsafe CpBlk for legacy backend

Merge pull request #14935 from sdmaclea/PR-ARM64-SIMD-genCodeForLclFld

[ARM64] SIMD genCodeForLclFld

Add foreach support to Span<T> and ReadOnlySpan<T> (#14922)

Adds a ref struct Enumerator, and a corresponding GetEnumerator method to each.

Merge pull request #14946 from MichalStrehovsky/master-merge

Merge master to dev/defaultintf

Merge branch 'master' into master-merge

Merge pull request #14912 from briansull/add-b124443

Added missing JIT test V1-M15-SP2/b124443/b124443.cs

Adding placeholder Span debugger proxy (#14749)

* Adding placeholder Span debugger proxy.

* Remove unnecessary unsafe keyword.

Small number of bug fixes. (#14900)

- Added stability prefix to the scenario benchmark (JitBench)
- Specify output directory to the `run-xunit-perf.cmd` script and avoid the extra step to xcopy files to the archive folder.
- Added a command line parser class to the illink scenario, and changed its behavior where it used to fail when a new command line option passed to xUnit was not recognized.
- Save the output log of the tests into the sandbox-logs folder.
- Updating the label of the machine pool used by the illink scenario

Merge pull request #14942 from jcagme/master

Adding IsPublic to CreateAzureContainer

Adding IsPublic to CreateAzureContainer

Change Sym publish to use UnzipSymbolPackagesForPublish (#14941)

Add missing license header (#14936)

Increase retry attempts in publish (#14940)

Pass in property SkipCreateWindowsPdbsFromPortablePdbs to fix (#14939)

Remove RelativePath from Publish (#14938)

Added missing test JIT/Regression/CLR-x86-JIT/V1-M15-SP2/b124443/b124443.cs
Also added an IL version of this test for regression repro on CoreCLR

Merge pull request #14928 from sdmaclea/PR-ARM64-SIMD-genPutArgStk

[Arm64] SIMD genPutArgStk

Merge pull request #14937 from jcagme/master

Updating BuildTools version

Updating BuildTools version

[ARM64] SIMD genCodeForLclFld

Merge pull request #14861 from sdmaclea/PR-ARM64-SIMD-genFnPrologCalleeRegArgs

[Arm64] SIMD genFnPrologCalleeRegArgs

Remove ConfigurationGroup from Publish Symbols (#14931)

Update clean.sh script to reflect changes introduced in clean.cmd, unify messages and functionality

JIT: make suitably optimistic prejit inline assessments (#14850)

When prejitting, the jit assesses whether each root method is a potential
inline candidate for any possible caller. Methods deemed un-inlinable in any
caller are marked in the runtime as "noinline" to save the jit some work
later on when it sees calls to these methods.

This assessment was too conservative and led to prejit-ordering dependences
for inlines. It also meant that prejitting was missing some inlines that
would have happened had we not done the prejit root assessment.

This change removes some of the prejit observation blockers. These mostly
will enable more prejit methods to become candidates. We also now track when
a method argument reaches a test.

When we are assessing profitability for a prejit root, assume the call site
best possible case.

Also, update the inline XML to capture the prejit assessments.

This increases the number of inlines considered and performed when prejitting
and will also lead to slightly more inlining when jitting. However we do not
expect a large througput impact when jitting -- the main impact of this change
is that inlining when prejitting is now just as aggressive as inlining when
jitting, and the decisions no longer depend on the order in which methods are
prejitted.

Closes #14441.
Closes #3482.

[Arm64] SIMD genPutArgStk

Add MemoryMarshal.AsMemory

Add string support to ReadOnlyMemory<char>