Update to latest dependencies

Use preview3 dependencies

Merge commit 'fae2a56cd032d9b68c58fd4dc960160b7013842a' into masterMerge

Remove accidentally added Windows.10.Nano.Amd64 from Windows_x86 in platform-matrix.yml (#22723)

Add CoreCLR blobfeed to NuGet.config (#22699)

CoreCLR packages are not found for versions created after #21947 split the feeds and we stopped publishing to MyGet. This adds the new CoreCLR feed.

Fix dblarray tests and reenable for arm32 (#22718)

Run the allocate/gen-check sequence under a forbid GC region, and enable the
tests for arm32.

Closes #22015.

Use ThrowHelper in ManualResetValueTaskSourceCore (#22714)

Just move the helper being used to ThrowHelper.

Use Build.DefinitionName for Creator when submitting tests to Helix (#22692)

Fix tizen rootfs building (#22715)

Move tizen-release package from base to unified repo

Fix bug introduced during cleanup in JapaneseLunisolarCalendar (#22717)

Allow Task.WaitAny completion to run synchronously (#21245)

Abstract deprecated and selectany for GCC and remove LLVM'ism where possible (#22662)

* Abstract selectany

* Fix initializer element is not constant

src/corefx/System.Globalization.Native/pal_icushim.c:58:34: error: initializer element is not constant
 static const int MaxICUVersion = MinICUVersion + 20;

* Enable ms extensions

* Apply LLVM patterns to GCC

* Remove deprecated function

* Fix const conversion error

src/corefx/System.Globalization.Native/pal_calendarData.c:390:16: warning: passing argument 1 of ‘ures_close_ptr’ discards ‘const’ qualifier from pointer target type [-Wdiscarded-qualifiers]
ures_close(erasResBundle);

src/corefx/System.Globalization.Native/pal_calendarData.c:419:22: warning: initialization discards ‘const’ qualifier from pointer target type [-Wdiscarded-qualifiers]
char* name = GetCalendarName(calendarId);

* Remove old compiler option

Fix race condition in ManualResetValueTaskSourceCore (#22711)

* Fix race condition in ManualResetValueTaskSourceCore

ManualResetValueTaskSourceCore.GetStatus is used from ValueTaskAwaiter.IsCompleted.  If GetStatus returns Success too early, then IsCompleted will also return true too early, which will result in GetResult being called too early.  This doesn't happen today when an MRVTSC is used sequentially.  But if an instance is pooled by the object its stored on getting put back into a pool as part of a call to the wrapper's GetResult, then we can end up in the following situation:
- Thread 1 starts an await on an instance containing an MRVTSC.  It calls IsCompleted.
- Thread 2 starts to complete that instance, getting as far as calling SignalCompletion and setting _completed to true but not yet setting _continuation.
- Thread 1 sees _completed == true and returns true from IsCompleted.  It calls GetResult, and the wrapper extracts the result from the instance, resets it, and puts it back into the pool.
- Thread 3 takes the object out of the pool and starts using it.
- Thread 2 continues SignalCompletion on that instance, and sets _continuation to the sentinel.
- Now Thread 3's instance's _continuation is the sentinel when it should be null.  If it calls SignalCompletion, it'll erroneously find that _continuation is not null and will queue _continuation/_continuationState for execution, resulting in the sentinel getting executed.  If it calls OnCompleted, it'll find that the _continuation is not null, and will queue the supplied continuation/state to execute immediately even though the operation may not yet actually be done.

The fix is simply to check not just _completed but also _continuation.  The operation is considered pending if either _completed is false, meaning SignalCompletion has not yet been called, or if _continuation is still null, meaning it's neither been set to the supplied delegate nor to the sentinel.  We can't just rely on _completed for the above outlined reasons, and we can't just rely on _continuation because it can be non-null if OnCompleted was called to hook up a callback (if we only cared about the await pattern, then we could just check _continuation and wouldn't need _completed, but we also need to support non-await access).

* Address PR feedback

Fix CLRDataCreateInstance export. This was breaking SOS. (#22701)

Fix CLRDataCreateInstance export. This was breaking SOS.

Added the right cmake magic so the function is exported properly.

Add ManagedRuntimeEventSource (#22515)

* Adding a dummy ManagedRuntimeEventSource

* Add ManagedRuntimeEventSource

* Renaming ManagedRuntimeEventSource to RuntimeEventSource

* some cleanup

* use Condition instead of ifdefing out the whole file

* some cleanups

* remove unused namespaces

* Move m_RuntimeEventSource to RuntimeEventSource from EventPipeController

* Adding some TODO items

* address more pr comments

* Use GUID to initialize RuntimeEventSource

* Exclude BasicEventSourceTests from CoreFX test CI leg

* Addressing rest of the PR feedback

* dont dispose counters on disable command to avoid races

Early out from vectored exception handler on uninitialized threads (#22702)

Fixes #22698

Use out arguments in UmAlQuraCalendar (#22704)

Cleanup CoreLib makefiles (#22706)

Adding tannergooding to the arm64Users list. (#22665)

Merge pull request #22656 from briansull/fix_arm64_assert

ARM64: Correct the assert to add R8 as a potential valid arg register

Fix formatting

Use RunContinuationsAsynchronously in SemaphoreSlim.WaitAsync (#22686)

SemaphoreSlim.Release shouldn't invoke arbitrary continuations as part of its invocation, so when it dequeues a task waiter and goes to complete it, rather than just calling TrySetResult, it queues the task to the thread pool to have TrySetResult invoked there.  Now that we have RunContinuationsAsynchronously, though, we can just use that functionality instead.  This has two main benefits:
1. We avoid queueing a work item entirely if there are no continuations from the task.  This might happen, for example, if the semaphore is released so quickly after waiting on it that the WaitAsync caller hasn't yet hooked up a continuation, in which case the await on the WaitAsync task will just see IsCompleted as true and continue running.
2. We avoid queueing a work item when the task represents a synchronous Wait, which happens if there's already a pending WaitAsync when Wait goes to block.  The main benefit here is avoiding potential thread pool starvation, if threads in the pool are blocked in such Waits, and previously another thread pool thread would have been needed to run the queued work item to complete the synchronous Wait.

Correct the assert to add R8 as a potential valid register
Also fix the calls to RemoveElemD to use lvVarIndex
Re-enable the interop tests that were failing on ARM64

Cleanup Calendars code (#22582)

Fixing extension method overload resolution for some S.R.I.Vector methods (#22696)

Update dependencies from https://github.com/dotnet/core-setup build 20190218.4 (#22689)

This change updates the following dependencies
- Microsoft.NETCore.App - 3.0.0-preview4-27418-4

Use testGroup for defining test jobs (#22608)

This adds a mapping between testGroup string and values of the following parameters:

* scenarios
* priority
* timeoutTotalInMinutes
* timeoutPerTestCollectionInMinutes
* timeoutPerTestInMinutes

and uses testGroup on a level of azure-pipeline.yml to set these values.

Enable telemetry for official builds

Optimize Vector128/256<T>.Get/WithElement (#22353)

Adding an update-dependencies.cmd script, like CoreFX has. (#22672)

Fail loading types with default interface methods for fragile ngen (#22603)

We are not generating the right data structures for this. Since fragile crossgen is no longer a mainstream scenario, it's not worth the effort to add default interface methods support for it.

This avoids `!"Precode::GetPrecodeFromEntryPoint: Unexpected code in precode"` and similar issues.

Adjust gc stress compatibility settings for two tests (#22633)

* enable eventsourcetrace for gc stress. Closes #17188.
* disable GitHub_12224 for gc stress (relies on timeout). Closes #19086.

Use correct version for Microsoft.NETCore.Platforms (#22683)

Remove resetting of unmanaged thread locale from thread pool APIs (#22682)

The cost of doing this is showing up in the TechEmpower profiles. There is a lot of thread locale state in unmanaged libraries and OS APIs that the threadpool friendly code has to be aware off. It does not make sense to keep reseting just the Locale.

Update dependencies from https://github.com/dotnet/corefx build 20190218.1 (#22678)

This change updates the following dependencies
- Microsoft.NETCore.Platforms - 3.0.0-preview4.19118.1
- Microsoft.Private.CoreFx.NETCoreApp - 4.6.0-preview4.19118.1

Fix constrained call corner cases (#22464)

Fixes #22423. I'm still unclear on how JitStress manages to get itself into that situation, but I was able to write a repro that triggers the `!pMDAfterConstraintResolution->IsInterface()` assert using regular IL, so we need to handle that either way. The repro for that is constrained3.il.

While figuring out the repro, I wrote a bunch of other test code and found another bug (constrained2), where we would box in a situations that doesn't require boxing (canonically ambiguous situation where there's a suitable default interface implementation and a valuetype implementation of the constrained method that does not requires boxing once we no longer deal with __Canon).

Fix warning causing error in warning as error (Windows) (#22660)

BOOL -> bool conversion

Update dependencies from https://github.com/dotnet/arcade build 20190215.1 (#22669)

This change updates the following dependencies
- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19115.1
- Microsoft.DotNet.Helix.Sdk - 2.0.0-beta.19115.1

Update dependencies from https://github.com/dotnet/corefx build 20190217.1 (#22670)

This change updates the following dependencies
- Microsoft.NETCore.Platforms - 3.0.0-preview4.19117.1
- Microsoft.Private.CoreFx.NETCoreApp - 4.6.0-preview4.19117.1

Fix Integer Rounding issue and duplicate definition of __deref in sal.h (#22645)

* Fix large integer truncation

pal/pal.h:4988:43:
warning: large integer implicitly truncated to unsigned type
[-Woverflow]
     static const SIZE_T NoTargetFrameSp = SIZE_MAX;
                                                ^~~~~~~~

* Fix compilation error with GCC

/usr/include/c++/7.3.0/debug/functions.h:452:15: error: expected unqualified-id before ')' token
       __deref();
               ^
/usr/include/c++/7.3.0/debug/functions.h:455:34: error: expected primary-expression before '<' token
       typename = decltype(__deref<_It>() < __deref<_It>())>
                                  ^
/usr/include/c++/7.3.0/debug/functions.h:455:38: error: expected primary-expression before '>' token
       typename = decltype(__deref<_It>() < __deref<_It>())>
                                      ^
/usr/include/c++/7.3.0/debug/functions.h:455:40: error: expected primary-expression before ')' token
       typename = decltype(__deref<_It>() < __deref<_It>())>
                                        ^

Delete RtlMoveMemory and tests (#22668)

Fix the build of Loader/NativeLib test (#22648)

The native test assets were not build correctly on Unix platforms.
The native library exports were generated as private symbols by clang.

This change fixes the export declaration so that the global symbols are
correctly generated for exported symbols.

Fixes #22549

Only import the RoslynPropsFile if it exists. (#22657)

Fix visibility and signed comparison issues for GCC (#22586)

* Fix visibility and signed comparison issues for GCC

* Fix hidden _CLRDataCreateInstance warned by macOS

* Fix indentation in vswprintf/test1

* Change void* to PVOID in implementation files

Avoid pulling in Lazy<T> during startup (#22659)

Environment constructor runs on every startup path. Remove dependency on Lazy<T> from it.

Merge pull request #22528 from mikedn/idx-addr

Fix genCodeForIndexAddr

[master] Update dependencies from dotnet/corefx (#22595)

* Update dependencies from https://github.com/dotnet/corefx build 20190213.23

This change updates the following dependencies
- Microsoft.NETCore.Platforms - 3.0.0-preview4.19113.23
- Microsoft.Private.CoreFx.NETCoreApp - 4.6.0-preview4.19113.23

* Update dependencies from https://github.com/dotnet/corefx build 20190215.1

This change updates the following dependencies
- Microsoft.NETCore.Platforms - 3.0.0-preview4.19115.1
- Microsoft.Private.CoreFx.NETCoreApp - 4.6.0-preview4.19115.1

* Fixing up the tests dependent on the S.R.I.Vector API changes

* Regenerating the tests dependent on the S.R.I.Vector API changes

* Disabling an outdated CoreFX test.

Set the 'PLATFORM_OSX' constant during OSX build (#22639)

* Set the 'PLATFORM_OSX' constant during OSX build

* Remove PLATFORM_OSX define from SPCL

Fix bug in FindDispatchSlot usage

Update BuildTools, IbcData, PgoData to 3.0.0-preview1-03715-01, master-20190208.1, master-20190208.1, respectively (master) (#22647)

Updating some local functions to be 'static' (captureless). (#22635)

JIT: modify how jit determines when to update a type (#22618)

For single-def locals, the type of a reference seen at the assignment to the
local may be a more specific type than the local's declared type. If so the jit
would prefer to use the assignment type to describe the local's value, as this
will lead to better optimization. For instance in
```
    object x = "a string";  // only assignment to x
```
the jit can optimize better if it models the type of `x` as `string`.

Instead of relying on `mergeClasses` plus some jit-side screening to decide if
the assignment type is a more specific type, implement a new jit interface
method `isMoreSpecificType` that tries to answer this question more directly.

Added a test case with type equivalence that hit asserts.

Closes #22583.

Disable arm64 contracts. (#22605)

* Fix ifdef for `g_DbgSuppressAllocationAsserts`.

It needs to be defined under `FEATURE_INTEROP_DEBUGGING`.

* Unify contracts disabling for UNIX/ARM.

It also disables some parts that were disabled for ARM, but not for UNIX.
It fixes the difference that we see between ARM32 and ARM64 Unix Checked.

* Disable contracts on arm64.

FIxes the difference that we see between windows arm32 and arm64 in checked/debug builds.

* Revert CoreFX arm64 timeout change.

That is no longer necessary.

* Reenable arm64 corefx jobs.

They should not fail with timeouts anymore.

* Don't allow `g_DbgSuppressAllocationAsserts` defnition in release builds.

* delete AssertAllocationAllowed

Port profiler howtos from David Broman's blog to the documentation folder (#22363)

Publish version to dotnet/versions

Fix UpdatePublishedVersions.ps1

Merge pull request #22643 from wtgodbe/Signing30

Revert signing to fixed version (#22609)

Remove redundant triggers from YAML file

Revert signing to fixed version (#22609)

SPC signing got broken when #22544 unpinned the SignTool version. Pin the tool version again until issue gets fixed.

@jashook

Widen index to a temporary register

Delete references to System.Memory package that does not exist anymore (#22626)

Disable failing corefx test (#22625)

Related to https://github.com/dotnet/corefx/issues/35351

Manually update the dependencies.props and move S.P.Corelib to use LangVersion=8.0 (#22452)

* Update BuildTools to preview1-03713-01 (master)

* Updating CoreCLR to use LangVersion=8.0

* Moving the Windows scripts to default to `dotnet msbuild` for managed components

* Setting UseSharedCompilation=true

* Changing some additional callsites that were using msbuild to use dotnet msbuild

* Revert packages.builds to use Desktop msbuild on Windows

* Fixing runtest.cmd to always set DotNetCli

Merge pull request #22611 from dotnet/mmitche-patch-1

Enable PR/CI for 3.0

Reserve fast dictionary slots for Tier1 code (#22619)

Disable use of fast dictionary slots for R2R images when tiered JITing is enabled.

Fixes #22400

Update dependencies from https://github.com/dotnet/arcade build 20190214.4 (#22621)

This change updates the following dependencies
- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19114.4
- Microsoft.DotNet.Helix.Sdk - 2.0.0-beta.19114.4

Remove CoreLib's src\Interop\Windows directory (#22622)

It contains only one file with an interop signature that's not actually being used. Deleting the dup.

Fix CoreFX build breaks

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

Fixing the number formatting code to only recognize 'R' for double/float. (#22613)

Move Reflection.Emit Tokens to shared partition (#22602)

Missed optimization from #22497

Remove unnecessary ProjectReferences to CoreCLRTestLibrary (#22539)

Remove the ProjectReferences to CoreCLRTestLibrary.csproj in the project files where CoreCLRTestLibrary is not actually being used.

Each ProjectReference item is ended up in PrepareProjectReferences target that creates unnecessary items.

Replace multi-loaderallocator hash implementation in MethodDescBackpatchInfo (#22285)

* GCHeapHash
- Hashtable implementation for runtime use
- Implementation written in C++
- Data storage in managed heap memory
- Based on SHash design, but using managed memory

CrossLoaderAllocatorHash
- Hash for c++ Pointer to C++ pointer where the lifetimes are controlled by different loader allocators
 - Support for add/remove/visit all entries of 1 key/visit all entries/ remove all entries of 1 key
- Supports holding data which is unmanaged, but data items themselves can be of any size (key/value are templated types)

* Swap MethodDescBackpatchInfo to use the CrossLoaderAllocatorHash

* The MethodDescBackpatchCrst needs to be around an allocation
- Adjust the Crst so that it can safely be used around code which allocates
- Required moving its use out from within the EESuspend logic used in rejit

Remove unnecessary fields from WaitHandle (#22593)

* Remove unnecessary fields from WaitHandle

* Drop volatile from _waitHandle

* Address race condition in unmanaged code

* Remove unused hasThreadAffinity parameters

* Fix FCDECL argument counts

Fix check for memory containment safety. (#22563)

This change ensures that if an operand can produce an exception
and any instructions executed after the operand evaluation but before
the operand's parent can also produce an exception, the operand
shouldn't be contained. The reason is that in this case operand
containment may reorder exceptions.

With `strict` set to true the containment is blocked here:
https://github.com/dotnet/coreclr/blob/d27fff3f65193dd71c6197e9876101f496bbd28b/src/jit/sideeffects.cpp#L485-L488

Also, make the check for ordering side-effect interference less
conservative.

Fixes #22556.

Make branch explicit

Enable PR/CO for 3.0

Merge pull request #22610 from mmitche/bring-3.0-up-to-date

Bring release/3.0 up to date

Merge remote-tracking branch 'upstream/master' into bring-3.0-up-to-date

Revert signing to fixed version (#22609)

SPC signing got broken when #22544 unpinned the SignTool version. Pin the tool version again until issue gets fixed.

@jashook

Consolidate implementation of Rotate and PopCount (#22584)

* Perf: BitOps.LeadingZeroCount

* Remove redundant MSIL cast, conv.u8

* Use local functions for SoftwareFallback

* Target BIT32/64

Adjust corert for shared SafeHandle and EventRegistrationToken

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

Fix netfx build break in Microsoft.Diagnostics.Tracing.EventSource.Redist

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

Merge pull request #22607 from wtgodbe/30branding

Update branding in versions.props to preview3

Move FrameworkEventSource to shared partition (#22507)

* Move FrameworkEventSource to shared partition

and remove unused code

* Remove more unused code

* Remove CA8001 suppress message

Update branding in versions.props to preview3

Gnuport cleanup 3 (#22581)

* Reorder extern "C" and visibility statements

In file included from coreclr/src/pal/inc/pal.h:70:0,
coreclr/src/pal/inc/pal_mstypes.h:30:25: error: expected unqualified-id
before string constant
 #define EXTERN_C extern "C"
                          ^
coreclr/src/pal/inc/rt/palrt.h:216:30: note:
  in expansion of macro 'EXTERN_C'
   #define STDAPI_(type)        EXTERN_C type STDAPICALLTYPE

* Need double quotes around single quotes in error statements

fix error statements

Can't have ' character in error statement

* Abstract __declspec(noinline)

Generate compiler independent macros for noinline request.

* Remove unused DBG_NOINLINE_X86__RET_INLINE

* Abstract __declspec(nothrow)

Add Rune creation API from UTF-16 surrogate pair (#22590)

Also brings in some perf improvements to existing char and UnicodeUtility APIs

Update dependencies from https://github.com/dotnet/arcade build 20190213.1 (#22594)

This change updates the following dependencies
- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19113.1
- Microsoft.DotNet.Helix.Sdk - 2.0.0-beta.19113.1

Add coreclr-outerloop build definition jobs (#22578)

Don't use $(_HelixSource) and $(_HelixType) in test-job.yml (#22589)

Annotate PlatformID as a moved type

This should have been annotated along with the rest in dotnet/corert#6928 but I missed it.

[tfs-changeset: 1733685]

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

Improving an assert in fgInsertInlineeBlocks

Use dependency flow for core-setup and corefx dependencies (#22544)

* Remove old use of signing package used to work around SPC signing now that arcade has bubbled up the version.
* Move dependency update for CoreFX and Core-setup dependencies to BAR/Darc/Maestro++ flow
* Update property references in tests.
* Version bump for CoreFX and Core-Setup dependencies.
* Update test Github_20958 to use revised System.Range constructor as surface area changed with #22331.

Merge pull request #22327 from BrianBohe/refactoring_genCodeForBBList

Moving structures initialization before generating code for blocks to…

Remove Marshaling MDA (#22579)

Remove the "marshaling" MDA. In .NET Framework it was off by default and in .NET Core it was commented out in the boilerplate file. Additionally, from what I can tell only half of its functionality was ever implemented in .NET Core.

Contributes to #22538.

Remove some CAS cruft (#22576)

* Update comment on SuppressUnmanagedCodeSecurityAttribute.
* Remove unused declsec.h header.

Perf: Consolidate implementation of LeadingZeroCount (#22497)

* Perf: BitOps.LeadingZeroCount

* CR fix

* CR fixes

* Optimization

* Revert

* Confirmed fix

* Simplify

* Return int

* Fixes

* Simplify

* CR fixes

* CR fixes

* Fix unit failure on Linux-musl x64 Debug

* CR fixes

* CR fixes

* Cleanup

Fix comment placement. (#22580)

Moving structures initialization before generating code for blocks to other function