this test should not be run under gcstress (#22214)

Skip rangecheck in string.EndsWith(char) (#22206)

Remove reflection-based wraper for AssemblyDependencyResolver. (#22195)

Now that AssemblyDependencyResolver is exposed in CoreFX and has propogated back to CoreCLR, we can remove the reflection-based wrapper for the unit tests for it.

Use AggressiveOptimization for intrinsics-based SpanHelpers (#22191)

* Use AggressiveOptimization for intrinsics-based SpanHelpers

* Remove from LastIndexOfAny

Reduce Task's static ctor dependencies (#22172)

* Reduce Task's static ctor dependencies

* Remove defensive check

* Revert "Remove defensive check"

This reverts commit df15ab0350e8f44fc5cbb1a18c5211cd1de11989.

Cancel CTS immediately if delay is zero. (#18098)

The intended behaviour on specifying a delay of zero is that the CancellationTokenSource is cancelled immediately - before this change, it would depend on the scheduling of the timer callback which could lead to seeing a non-cancelled token. Now, it marks itself as cancelled immediately, without invoking any callbacks (as this method is called only from within the constructor of the object, before any callbacks could have been registered.

Fixes the desktop build (#22197)

* Fixes the desktop build
warning C4311: 'reinterpret_cast': pointer truncation from 'BYTE *' to 'uint32_t'  f:\j6\codegenmirror\src\ndp\clr\src\jit\emit.cpp(5636): warning C4302: 'reinterpret_cast': truncation from 'BYTE *' to 'uint32_t'

* Fix formatting

Speedup .SequenceCompareTo(byte, ...) (#22127)

* Speedup .SequenceCompareTo(byte, ...)

* Rename jump location

* Better annotations for clarity

Fix collectible AssemblyLoadContext weak handle (#22193)

Native runtime keeps short weak handle for collectible AssemblyLoadContext
until the unload time when it gets converted to a strong one. A test
running with gc stress has recently hit a problem caused by the fact
that the weak handle is "short weak" and thus doesn't track
resurrection. Here is what happened:
* The test code has lost reference to the AssemblyLoadContext
* Finalizer was just invoked on that context due to that and so the weak
  short handle was destroyed.
* Native runtime called AssemblyLoadContext.Resolve with the now dead
  handle, the function tried to get the managed object and failed.

The fix is to change the handle type to long weak so that it doesn't get
destroyed before the finalization.

Updating MicrosoftDiagnosticsTracingTraceEventPackageVersion (#22071)

Fix argument checks in ThreadPool.RegisterWaitForSingleObject. (#6887) (#22196)

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

Rename TplEtwProvider to TplEventSource (#22194)

* Rename TplEtwProvider to TplEventSource

* Rename etwLog to log

Move Marshal to shared partition (#22146)

* Move Marshal to shared partition

* Moved code optimizations

* Cosmetic changes

* Avoid duplicate arguments checks bys shipping Encoding.UTF8.GetString

Fix SOS GCInfo for Arm/Arm64 (#22188)

Add missing DECODE_HAS_TAILCALLS flag.

Closes #22163

Fixing some issues with the decimal parser/formatter (#22070)

* Fixing the decimal formatting so that -0 prints as 0

* Fixing the decimal parser to consider digits still in the buffer as part of the zero/non-zero tail check

Disable exception handling on MacOS when DLL like DAC, etc. (#22171)

Move TaskContinuation to shared partition (#22167)

* Move TaskContinuation to shared partition

* Remove TASK_STATE_THREAD_WAS_ABORTED from Task

* Use already existing ThrowAsync

Merge pull request #14393 from mikedn/float-neg-abs

Improve `new Vector2/3/4(c1, c2, c3, c4)` codegen

Update SSE2 test

Fix importer spilling in the presence of assignment side effects

Atomic ops like GT_CMPXCHG and some HW intrinsic nodes act like assignements so impAppendStmt has to account for them. They can be top level nodes or they can appear in the RHS of a GT_ASG node that perhaps is not considered to have an assignment side effect itself.

Use .asArray/.asString syntax for scenarios and helixQueues (#22173)

Fix test build warning. (#22168)

I decided to suppress the warning for the test instead of
removing the test for test matrix completeness.

Fixes #21519.

Update PgoData to master-20190124.1 (#22181)

Fix UWP VirtualAlloc import (dotnet/corert#6882)

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

Remove obsolete thread abortion flags. (#22185)

Disable PSPSym in filter prolog for CoreRT (#22184)

Work around build failure (#22138)

Fixes https://github.com/dotnet/coreclr/issues/22103
- There were reports of build failure from https://github.com/dotnet/coreclr/pull/21292, worked around it for now with a todo

IOCompletionCallback moved to shared partition

Move IOCompletionCallback to shared partition (dotnet/corert#6875)

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

Remove no-op holder stack validation. (#22182)

Fix incorrect usage of wcscpy_s/wcscat_s. (#22153)

Fix accidental assignment inside an assert. (#22148)

Update CoreFx, PgoData to preview.19073.1, master-20190123.4, respectively (#22112)

Enable warnings-as-errors for System.Private.CoreLib (#22164)

And disable several NoWarns.

Speedup SpanHelpers.IndexOf{Any}(byte, ...) (#22118)

* Speedup SpanHelpers.IndexOf(byte)

* 128 * 2 alignment

* Move TrailingZeroCountFallback to common SpanHelpers

So it can be used by other types than byte

* Speedup SpanHelpers.IndexOfAny(byte, ...)

* Indent for support flags

* More helpers, constency in local names/formatting, feedback

* Skip bounds check in software fallback

this test should not be run under gcstress (#22175)

Implement IAsyncDisposable.ConfigureAwait (#22160)

Fix CoreRT build after moving Task.cs to shared partition

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

Merge pull request #22003 from adityamandaleeka/writebarrier_updates_arm64

Write barrier optimizations for ARM64 Windows

Add GC.KeepAlive to keep collectible ALC alive across usage. (#22133)

Disable superpmicollection in r2r jobs until https://github.com/dotnet/coreclr/issues/22027 is fixed (#22085)

Add unloadability testing tool (#22064)

* Add unloadability testing tool

This change adds the unloadability testing tool that can be used to run
coreclr tests or any other .NET core app inside of unloadable
AssemblyLoadContext, unload it after its execution completes and verify
that the unload succeeded. It has also various additional testing
options:
* memory leak testing
* running multiple iterations of the load/run/unload sequence
* optional breaking into debugger at various interesting stages (before
executing the test assembly, after executing it, on unload failure)
* delegated load when the AssemblyLoadContext that loads the test
assembly delegates loading of all the dependencies to another
AssemblyLoadContext

Move common ThreadPool code to shared CoreLib partition. (#22115)

* Move common ThreadPool code to shared CoreLib partition.

* Move numWorkingThreads counting out of shared code.

* Rename ThreadPoolGlobals.vmTpInitialized to threadPoolInitialized.
Remove exception handling from ThreadPoolWorkQueue.Dispatch.
Remove obsolete code from ThreadPoolWorkQueue.Dispatch.
Rename ThreadPool.EnsureVMInitialized to ThreadPool.EnsureInitialized.

Remove all traces of FEATURE_STACK_PROBE. (#22149)

Update parameter names in PInvoke signatures to match MSDN documentation.

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

Build fixes.

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

Fix possibly unintentional use of bit-wise or operator. (#22152)

Remove obsolete LEGACY_BACKEND ifdef

Disable OSX.1012.Amd64.Open in Azure DevOps (#22158)

Fix R2R failure in one of ObjectStackAllocation tests.

AllocateSimpleClassAndCheckTypeNoHelper fails because
the object is allocated on the heap in R2R mode. The reason for that is
that we always use a helper for isinst in R2R mode and helper calls
cause us to assume the object escapes.

We don't have to use a helper in this case (even for R2R), #22086 tracks
fixing that.

For now the fix is not to check that the object was allocated on the stack
in this test case.

Fixes #22026.

Fix for JIT\Directed\arglist\vararg test case.

TestEchoShortNoVararg was failing and disabled. The reason it was
failing was a mismatch between DllImport declaration for echo_short and
the native function: the former specified 2-byte arg and return while
the latter specified 1-byte arg and return. I fixed that and also added
a couple of test cases with 1-byte arg and 2-byte return and
vice versa.

Fixes #19704.

Don't redirect stdlib wchar functions to our test platform shims. (#22072)

* Don't redirect stdlib wchar functions to our test platform shims.

* Fix bad find-replaces.

* Remove unused overload and last wchar shadowing.

Merge pull request #22053 from BruceForstall/FixSimd12StoreLclFld

Fix SIMD12 GT_STORE_LCL_FLD

Remove variable name duplication (#22155)

Remove unused thread abortion methods. (#22147)

Move Task to shared partition (#22105)

Use Windows.10.Arm64.Open with Jenkins ONLY (#22144)

Fix SIMD12 GT_STORE_LCL_FLD

It was ignoring the local field offset, by checking for the wrong
operator.

This appeared as a recent regression, but I couldn't find any code
(including test) change that would have caused this.

The failure only happens for JitStress with remorphing stress. This
creates the STORE_LCL_FLD. Otherwise, it looks like we normally
end up with IND(LCL_FLD_ADDR). Maybe we should be morphing these
somewhere?

No x64 PMI asm diffs in corelib or the JIT\SIMD tree. With JitStress=1
there are exactly 6 asm diffs in the Vector3Interop test, which was the
one failing.

The same fix is made for both xarch and arm64 code generators.

Fixes #21935

Remove duplicate requiresSynchronization params attribute (#22143)

Don't use Ubuntu.1604.Arm64.Open in Azure DevOps (#22135)

Decouple JIT_FLAG_PREJIT from HW intrinsics (#22117)

Per dotnet/corert#6836 (comment) moving the decision whether to enable HW intrinsics to the EE side.

Restrict HW intrinsic name check to intrinsic types (#22116)

Avoids having to compare names of all types in CoreLib to see if they're HW intrinsics.

Do not throw from TraceResolver (#22126)

Fixes #22059.

Move MemoryFailPoint to shared CoreLib partition (#22104)

* Move MemoryFailPoint to shared CoreLib partition.

* Split MemoryFailPoint into Unix and Windows versions.

* Replace MemoryFailPoint.GetMemorySettings FCall with GC.GetSegmentSize to make sharing with CoreRT easier.

Fix CoreRT build breaks

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

Move TplEtwProvider to shared partition (#22120)

Remove two unnecessary ToArray calls (#22119)

We can just copy directly from the List, rather than first converting the list to an array and then copying that.

Avoid intermediate strings in some StringBuilder formatting (#22111)

* Avoid intermediate strings in some StringBuilder formatting

Several appends to string builders are first creating strings and then appending those strings, and they're not using the existing Append(primitive) overloads because they want to customize the format string or provider as part of appending the value.  This fixes a few of those cases, using another internal AppendSpanFormattable overload on StringBuilder.

* Address PR feedback

Update CoreFx to preview.19070.3 (#22061)

Move Task factory type to shared partition (#22089)

Use List<T>.ToArray() in places (#22101)

* Use ReadOnlySpan.ToArray in places

* Use List.ToArray

* Don't take .Length in to local and use as loop condition

Cleanup array related FCalls (#22097)

* Cleanup Array FCalls

* Disable outdated CoreFX tests

https://github.com/dotnet/corefx/pull/34700

Fix CoreCLR build for changes in shared partition

Fix build for ProjectN/WinRTInterop Delete superfluous AsyncCausalityTracer arguments

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

Fix build

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

Update Newtonsoft.Json (#22060)

Cleanup: Remove MethodDesc::GetLoaderAllocatorForCode (#22067)

This function was recently changed to just return the
MethodDesc::GetLoaderAllocator. This is a cleanup that removes the
function completely and replaces all of its usages.

Removed unused 'Stack Walking' code (#22039)

As fair as I can tell `ECWGCFCrawlCallBack(..)` is not wired-up anywhere (I've successfully compiled CoreCLR locally with this change under `x86` and `x64`)

Strip symbols on unix in release builds (#22083)

Fixes https://github.com/dotnet/coreclr/issues/22082

Fix calling conventions on reverse PInvoke callbacks. (#22044)

Run Azure DevOps builds against pull requests in coreclr/master (#22051)

And disable Windows_NT and Ubuntu arm32 and arm64 default triggers in Jenkins CI

SuperPMI Collect/Replay/AsmDiff tool (#21252)

This change adds superpmi.py. The tool feature three modes, collection, replay, and asmdiffs.

Collection
The collection logic is very similar to the logic in our superpmi-collect test. Mostly it just allows running a script which will run managed code and it will produce a .mch which is clean to be run against. See superpmi.md for more information on specific usage and problems.

Replay
Replay will take an existing .mch file and run the current jit over the collection. If there is no .mch file on disk, the script will download the latest collection and run against that.

AsmDiffs
superpmi.md has the latest information on what platforms support asmdiffs. So far, I have an updated OSX and Windows collection that I have run against. If there are binary diffs, the tool will automatically generate base & diff folders with the asm under each one. Future work would include automatically running jit-analyze over those locations.

In addition, the tool has the option to automatically run and diff jit dumps, I have found this to be useful to looking into diffs created, as re-running superpmi with different jits to collect this same information is somewhat tedious.

Future work
This change is in no way the end of the work needed to leverage superpmi effectively. Instead, it is a good first step. Below are some suggestions for future superpmi work:

Automated collections
Add pmi collection support
Leverage some of the new corefx work to use superpmi shim for collections of corefx runs
To be added/changed
I will unset zapdisable being set by default, it creates too much data, although it is useful it should be opt in
Will include example usage in superpmi.md.

Move AsyncCausality to shared partition (#22062)

* Move AsyncCausality to shared partition

* Set FeatureAsyncCausalityTracer property

Updating MicrosoftDiagnosticsTracingTraceEventPackageVersion

Re-enable signing of System.Private.CoreLib (#22068)

* Re-enable signing for System.Private.CoreLib

Now that https://github.com/dotnet/arcade/issues/1582 is fixed.
Addresses https://github.com/dotnet/coreclr/issues/21946.

* Use uppercase build config

For consistency with other builds. Doesn't impact functionality, but
will result in the build logs showing the bin directory with the
uppercase build config during this step.

* Override arcade version of SignTool package

Add Per-assembly Load Native Library callbacks (#21555)

Add Per-assembly Load Native Library callbacks

This Change implements the Native Library resolution
Call-backs proposed in https://github.com/dotnet/corefx/issues/32015

Update dependencies from dotnet/arcade (#22035)

* Update dependencies from https://github.com/dotnet/arcade build 20190116.1

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

* Update dependencies from https://github.com/dotnet/arcade build 20190117.6

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

Enable detection of HW intrinsics (dotnet/corert#6836)

I took the liberty of marking the types containing the hardware intrinsics as [Intrinsic] to avoid doing a name check on everything in the system module. It feels like we should take advantage of this attribute in CoreCLR too.

This doesn't actually enable the support because RyuJIT unconditionally disables HW intrinsics for prejit (both CoreRT and CPAOT are considered prejit). We might want to do something about this on the RyuJIT side to address that (for CPAOT, to be able to pregenerate best code possible ahead of time, assuming a fixed ISA; and for CoreRT without JIT, where the concerns about AVX-SSE penalty don't apply). See conversation in dotnet/coreclr#21603

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

Fix defines in TaskAwaiter (dotnet/corert#6846)

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

Handle complex constrained calls with default interface methods (#21978)

This adds handling for the interface dispatch corner case where:

* We have a constrained callsite to a method on a generic interface in shared code
* The callsite cannot be statically resolved because the result of dispatch depends on the generic context
* At runtime, the dispatch resolves to a default interface method

This would require us to have infrastructure to build "boxing thunks" - thunks that would box their first argument before dispatching to the default interface method implementation.

Since this is a corner case and the fix is actually quite involved, we're making the runtime just throw in this situation.

The test is written so that it should pass both if the runtime chooses to throw, or if the runtime makes the boxing thunk (we're not hardcoding the implementation limitation).

Use correct resolution scope when checking default implementations (#22036)

We need to use the module of the interface.

Fixes #22021.

Add comments on headers that gets parsed by Mono linker for whoever touches this next (#22054)

* Add comments on headers that gets parsed by Mono linker for whoever touches this next

* add more comments to the targets file

Move Environment to shared CoreLib (dotnet/corefx#34654)

Rather than having Environment partially live in corefx and call into an EnvironmentAugments type in CoreLib that in turn calls into an Environment type in CoreLib, we're just moving Environment to live in CoreLib.

To start that, this PR moves Environment and its dependencies from their current locations into the shared CoreLib.  Those changes will mirror over to coreclr.  After that, I'll fix it up to work in CoreLib. And once those changes are built and available back to corefx, I'll update System.Runtime.Extensions to just use the functionality from CoreLib and delete remaining unnecessary code from corefx.

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

Unify xplatform.h and platformdefines.h (#21389)

* Move most of xplatform.h into platformdefines.h and clean up the interop CMake files.

* Fix build-break off Windows.

* Fix typo.

* Fix another break off-Windows.

* Fix signed/unsigned mismatch

Unblock Azure DevOps-as-CI (#22042)

* Disable JIT.Methodical/doublearray/dblarray3_cs_do

* Disable readytorun.r2rdump/R2RDumpTest

* Disable RedHat.6.Amd64.Open queue

* Rename Creator coreclr/pulls -> coreclr-pulls

* IsExternal is deprecated as of https://github.com/dotnet/arcade build 20190115.6

* Increase timeout for all test jobs

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

Update BuildTools, CoreClr, CoreFx, PgoData to preview1-03617-02, preview-27316-02, preview.19066.1, master-20190116.5, respectively (master)

Remove ToString from RuntimeAssembly.GetManifestResourceStream (#22012)

* Remove ToString from RuntimeAssembly.GetManifestResourceStream

A small allocation we can avoid with the span-based string.Concat.

* Address PR feedback

Update BuildTools, CoreClr, CoreFx, PgoData to preview1-03617-02, preview-27316-02, preview.19066.1, master-20190116.5, respectively

Split ContractsBCL into separate classes and move them to shared CoreLib (#22006)

* Split up ContractsBCL into separate classes.

* Move remaining Contract, ContractHelper code to shared partition.

* Split ContractHelper to allow custom debugging experience in CoreRT.

* Simplify ContractHelper.InternalContractFailed.

* Use resource loading code from CoreRT.

* Remove unused parameter.

Clean up a few minor Span-related uses (#22038)

Some unnecessary Span to ReadOnlySpan casts.
Some places where new Span(...).Slice(...) can be replaced with AsSpan(...).
Etc.