Remove superfluous 'const' qualifier from trivial return types (#20652)

The 'const' used in this context has no meaning

Re-Enable Windows tests with JitOptSensitive flag (#20697)

* Re-Enable windows with JitOptSensitive flag

The check in batch targets would keep us from running jitOptimiztionSensitiveTests
with COMPlus_TieredCompilation=0.

* Address pr feedback

Fix DevDiv issue with JitStressRegs=0x3 on arm32. (#20485)

* Fix the issue.

* Fix the comment.

* update the comment

Fix Windows x86 EH for exception from UMThunkPrestub (#20704)

The exception handling was poping an explicit frame in
UMThunkPrestubHandler down the call chain from CallRtlUnwindSafe, but it
was not updating the tct.pBottomFrame after returning from that function
back to CPFH_RealFirstPassHandler. And the call to COMPlusAfterUnwind
then called UnwindFrames starting at the old frame that was already
removed from the chain.

Work around XUnit FileLoadException issues (#20644)

In order to mitigate System.IO.FileLoadException we built our own xunit.console.dll with ConcurrentDictionary used for DependencyContextAssemblyCache.managedAssemblyCache instead of Dictionary and use this instead of the one pulled from NuGet.

Merge pull request #20638 from fiigii/movetest

Move SSE/SSE2/AVX Load* intrinsic tests to the template framework

Merge pull request #20710 from BruceForstall/DisableNonPRJobsMaster

Disable non-PR triggers in dev/unix_test_workflow branch

Disable non-PR triggers in dev/unix_test_workflow branch

Delete redundant framework API tests (#20693)

This is the last batch of deleting tests that are obviously redundant. The remaining framework API tests in CoreCLR can be dealt with as needed if it ever becomes a problem.

Fixes #12782

Add support for large number of argumets in CCW call on Arm64 (#20670)

* Fix CCW with large numbers of arguments on arm64
- Correctly align the stack in COMToCLRDispatchHelper
- Add tests for naturally 16 byte aligned stack growth, and non-aligned growth
* New many arguments COM test
* Add support for the IL stub ETW diagnostic event

Merge pull request #20703 from MattGal/bsd-pipebuild-tweaks

Remove DotNetBootstrapCliTarPath from variables

Fix printing in `CodeGen::siEndScope`. (#20698)

Fixed small typo in a comment (#20695)

fixed a simple typo, "co-" should contain a dash in this case.

fix detection for LLDB.h and make sos mandatory on FreeBSD (#20551)

* fix detection for LLDB.h and make sos mandatory on FreeBSD

* feedback from review

* feedback from review

* add llvm40 as well for completness

* fix OR condition

Remove DotNetBootstrapCliTarPath from variables as it was getting forcibly upper-cased, making it useless. Instead, use the workaround of setting it explicitly before sync.sh.

Merge pull request #20699 from MattGal/bsd-pipebuild-tweaks

Add FreeBSD to official build pipelines

add -portablebuild=false to build.sh (affects naming)

Fix `JitStressRegs=0x1` issue with `RefTypeUpperVectorSaveDef`. (#20623)

* Add a repro test.

* Fix the issue.

* Add comments

Fixing build.sh args

Add freebsd flavor to CoreCLR pipebuild

JIT: Fix call flag propagation for GenTreeArrElem (#20660)

Closes #20651.

Also fix up some "near miss" cases for GenTreeField and GenTreeBoundsCheck,
where we get lucky and the importer currently splits trees with temps so the
currently ignored child nodes have no interesting side effects.

Revise GenTreeField a bit to pull more of the initialization work into the
constructor. Add a missing R2R field propagation for field nodes in GtClone
(evidently also never hit in practice).

Merge pull request #20641 from briansull/vso_707552

Fix VSO 707552

Add the execution policy when running powershell in size on disk benchmarks (#20680)

Since moving to the helix queue, we have been failing because the execution policy on the vms is not set to unrestricted. To get around this, we need to explicitly pass -ExecutionPolicy Bypass.

Disable failing on Windows/ARM Interop/COM tests (#20675)

* Interop/COM/NETClients/Primitives/NETClientPrimitives
* Interop/COM/NativeClients/Primitives
* Interop/COM/NETClients/IDispatch/NETClientIDispatch
* Interop/COM/NETClients/Aggregation/NETClientAggregation

Corrected Typos and grammar (#20658)

* Corrected some typos and grammar as per English language guidelines.

* Reverted some of the changes

* Reverted back some changes Phase 2

Remove unnecessary Pack directive to fix build break (#20681)

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

Merge pull request #20679 from BruceForstall/DumpTempReason

Dump the "reason" for a compiler temp

Dump the "reason" for a compiler temp

Compiler temps are created with a "reason" that is dumped in JitDump.
Save the reason and display it in the local variable table dump.
This is useful when trying to find a particular temp and see what
code has been generated using it.

JIT: streamline temp usage for returns (#20640)

If the jit decides it needs a return spill temp, and the return value
has already been spilled to a single-def temp, re-use the existing
for the return temp rather than creating a new one.

In conjunction with #20553 this allows late devirtualization for calls where
the object in the virtual call is the result of an inline that provides
a better type, and the objected formerly reached the call via one or more
intermediate temps.

Closes #15873.

Fix VSO 707552
[RyuJIT][ilgen] Assertion failed 'exp->gtOper != GT_COUNT'
Use nullptr instead of NULL

Remove outdated GC perf test framework (#20673)

Some cleanup of the System.Number class (#20619)

* Formatting Number.Formatting.cs and Number.Parsing.cs

* Removing some duplicated parsing code by having the Parse method call TryParse

* Moving two constants from NumberBuffer to Dragon4

* Rename FloatPrecision to SinglePrecision

* Updating the casing of the NumberBuffer fields

* Updating NumberBuffer to allow taking a custom-sized digit buffer.

* Updating the various NumberBufferLength constants to be the exact needed lengths

* Fixing DoubleNumberBufferLength and SingleNumberBufferLength to account for the rounding digit.

* Fixing TryParseNumber to use the correct maxDigCount

* Ensure the TryParseSingle out result is assigned on success

Fix typo in linux-performance-tracing (#20667)

precomppiled -> precompiled

Unhelpful error when Activator.CreateInstance can't find default constructor  (#20491)

Update CoreClr, CoreSetup, IbcData, PgoData to preview1-27028-04, preview1-27029-01, master-20181029-0053, master-20181029-0054, respectively (#20662)

Update CoreClr, CoreSetup, IbcData, PgoData to preview1-27028-03, preview1-27028-01, master-20181028-0058, master-20181028-0055, respectively (#20653)

Delete redundant CoreMangLib decimal tests (#20649)

Fixes failing test obsoleted by #20638 and contributes to #12782

Delete redundant CoreMangLib decimal tests  \85

Update BuildTools, CoreClr, CoreSetup, IbcData, PgoData to preview1-03326-01, preview1-27027-01, preview1-27027-01, master-20181027-0055, master-20181027-0036, respectively (#20559)

Add IAsyncDisposable implementation to Timer (#20646)

Implement IAsyncDisposable on CancellationTokenRegistration (#20645)

Remove dead RESOURCE_SATELLITE_CONFIG (#20639)

Merge pull request #20621 from AaronRobinsonMSFT/block_tests_WindowsNano

Add a run time check for Windows.Nano to all COM related tests.

Allow ArrayPool.Shared devirtualization (redux) (#20637)

Move SSE/SSE2/AVX Load* intrinsic tests to the template framework

New Decimal.Remainder implementation (#20305)

* New Decimal.Remainder implementation

* Update CoreFX.issues.json

Implement interface dispatch to COM for ARM64 (#20618)

* Implement interface dispatch to COM for ARM64
- Implement missing stub (GenericComPlusCallStub)

* Re-enable IDispatch test for Arm64 (Fix issue #20580 )

JIT: refactor how we do late devirtualization (#20553)

Change late devirtualization to run in a postorder callback during
the tree search to update GT_RET_EXPRs, instead of shoehorning it into
the preorder callback.

This allows the jit to reconsider all calls for late devirtualization,
not just calls that are parents of particular GT_RET_EXPRs. The jit will
take advantage of this in subsequent work that does more aggressive
bottup-up type sharpening.

Reconsidering all calls for late devirt instead of just a select subset
incurs around a 0.5% throughput impact.

To mitigate this, short-circult the tree walk when we no longer see a
GTF_CALL flag. This prevents us from walking through CALL and RET_EXPR
free subtrees.

To make this work we have to make sure all node types propagate flags from
their children. This required some updates to a few node constructors.

There is also an odd quirk in the preorder callback where it may overwrite
the parent node (from GT_ASG to GT_BLK or GT_NOP). If the overwrite creates
a GT_NOP, the postorder callback may see a null tree. Tolerate this for now
by checking for that case in the postorder callback.

Also take advantage of `gtRetExprVal` to tunnel through GT_RET_EXPR chains
to the final tree to avoid needing multiple overwrites when propagating
the return expression back into the IR.

With these mitigations this change has minimal throughput impact.

fixes to build properly on FreeBSD (#20588)

* fixes to build properly on FreeBSD

* remove ulimit from freebsd branch

Add missing include guard in header file. (#20635)

Remove some dead code related to cross-appdomain exceptions. (#20634)

Merge pull request #20628 from stephentoub/asyncinterfaces

Add interfaces/helper types for async iterators

Working towards clean-up comments /**/ in cs (#20609)

* Working towards cleaning comments /**/

Cleaningup /**/ style comments - Following https://docs.microsoft.com/en-us/dotnet/csharp/programming-guide/xmldoc/recommended-tags-for-documentation-comments

#20330

* Updating tags

-Updating some <sumary> to <remarks>
-Adding missing tags

* Fixing PR comments

-Comment does not belong to the public description

* Hiding comment from public description.

* Update //***** to the prevalent // style.

Optimize FormattingHelpers.CountHexDigits using Lzcnt.LeadingZeroCount (#19006)

* Optimize CountHexDigits using Lzcnt

* shift bits instead

* use Lzcnt only on 64 bit

* Optimize Buffer.Utilities.SelectBucketIndex()

* #if HAS_INTRINSICS

* remove HAS_INTRINSICS

* remove HAS_INTRINSICS

Add ConfigureAwait extension method for IAsyncEnumerable

Add ManualResetValueTaskSourceCore

Add AsyncIteratorMethodBuilder

Add IAsyncEnumerable and IAsyncEnumerator

Add IAsyncDisposable

Merge pull request #20579 from fiigii/broadcast

 Implement AVX2.BroadcastScalarToVector128/256

Merge pull request #20611 from BruceForstall/DisableB152292

Disable b152292 for arm32 also

Add a runtime check for Windows.Nano to all COM related tests.

Merge pull request #20527 from BruceForstall/FixPspSymMove

Stop attempting to move PSPSym during localloc for arm32/arm64

Stop attempting to move PSPSym during localloc for arm32/arm64

For arm64, this was creating a data corruption possibility, as the
calculations were done wrong.

In neither case was it necessary, as in the main function we always
access the PSPSym FP-relative, so when writing back the PSPSym, we
were always writing to exactly the same slot; nothing was actually
moving.

I can't recall or figure out why we were moving it in the first place,
or what might have changed.

Added a new variant of the localloc eh tests that has outgoing arguments,
to exercise the case where we need to re-establish the outgoing argument
space after the localloc.

Merge pull request #20617 from AaronRobinsonMSFT/windows_nano_detection

Add support for tests to detect Windows Nano at runtime.

Disable b152292 for arm32 also

Add DumpDelegate sos command (#20591)

* Add DumpDelegate sos command

Finding out what method a delegate points to today with SOS is tedious.  This PR adds a DumpDelegate command that simplifies it: give it the delegate address, and it outputs a line for each delegate that makes up the input delegate, where each line includes the target object, the method descriptor, and a friendly name.

* Address PR feedback

And fix some warnings that were showing up in CI but not locally.

Add support for tests to detect Windows Nano at runtime.

Remove debug-only file writers. (#20612)

Add test cases for AVX2.BroadcastScalarToVector128/256 and move Avx.Broadcast* tests to the template

Implement AVX2.BroadcastScalarToVector128/256

Merge pull request #20614 from AaronRobinsonMSFT/coreclr/20601

Disable `IDispatch` test on Windows.Nano

Skipping CI verification since this is a non-code change and only impacts nightly runs.

Disable IDispatch test on Windows.Nano

Disable ExecuteInDefaultAppDomain test on *nix

ExecInDefAppDom add InjectCode test

Add test for ICLRRuntimeHost::ExecuteInDefaultAppDomain

Add IID_ICLRRuntimeHost QueryInterface support

Restore ExecuteInDefaultAppDomain implementation

Restore deleted ExecuteInDefaultAppDomain implentation
Enable for CoreCLR
Cleanup for CoreCLR

Add EventPipe Config File Option MultiFileSec (#20548)

Avoid comparing capacity in StringBuilder.Equals (#20567)

* Avoid comparing capacity in StringBuilder.Equals

* Disabled corefx StringBuilder test

Fallback to old behavior when marshaling a non-blittable fixed buffer (#20575)

* Throw an exception only when attempting to marshal nonblittable fixed buffers to/from native instead of on all marshalling operations.

* Assert in debug and checked builds when trying to marshal a fixed buffer of a non-blittable type.

JIT: ensure float folding is done using float precision (#20578)

Cast float folded values back to float before assigning to the double
the jit uses for storing FP literal constants.

Fixes #20561.

No diffs on Core; exposing the original bug requires building RyuJit with
an older x86 C++ compiler that uses x87 floating point.

Fix comment Style (#20604)

Use IThreadPoolWorkItem in ConcurrentExclusiveSchedulerPair (#20513)

ConcurrentExclusiveSchedulerPair exposes two TaskSchedulers, one for concurrent processing and one for exclusive processing, such that any tasks scheduled are executed in a corresponding fashion.  CESP does that by itself scheduling workers to an underlying scheduler, and those workers in turn process the queued tasks.  Today this always queues those workers as Tasks, but if the underlying scheduler is TaskScheduler.Default (aka the thread pool), we can avoid per-worker allocations by using the new UnsafeQueueUserWorkItem overload that takes an IThreadPoolWorkItem, and just queueing the same worker object over and over and over.

JIT: emit debug info for locals for minopts/tier0 (#20466)

The jit no longer tracks locals when running at minopts. But untracked
local debug emission was tied to `cmpDebugCode. Change untracked local
debug emission to instead check if there are no tracked locals.

Fixes #20421.

Note we could improve things further in the case where there is a mixture
of tracked and untracked locals. I have opened #20465 for that as my first
attempts here had that ambition but ran into problems.

Avoid implicit long casts in uint compares (#20511)

Avoid having DateTime.TryParse throwing (#20587)

* Avoid having DateTime.TryParse throwing

There are some cases during the parsing which will require adding ticks to the DateTime object we are creating during the parsing.
DateTime.AddTicks can throw which will make DateTime.TryParse fail with throwing instead of just returning false.

The fix here is to avoid throwing in the first place and let DateTime.TryParse return false on such failures.

* Remove the invalid link

Add IsCollectible property to Assembly and necessary backing functions (#20574)

Get the real update for Unicode 11 data (#20589)

In the PR #20529 we have used the old existing data when we moved the file CharunicodeInfoData.cs to shared folder.
This change is to restore the actual updated file.

Exclude IDispatch test for Windows Arm64 (#20584)

Disable for now while failures are investigated. See #20580.

Remove support for nonexistent UnmanagedFunctionPointerAttribute.PreserveSig. (#20396)

UnmanagedFunctionPointer doesn't have a PreserveSig attribute on framework or core, so remove the unused, confusing, and inaccurate support from the runtime.

Run IBCMerge step for Linux on Windows (#20496)

This change does the following:

* Move the IBCOptimize step out of the Crossgen section and into the CoreLib build section of build.cmd
* Adds -ibconly which will skip building System.Private.CoreLib and only run the ibcmerge step
* Adds crossgenonly and partialngen flags to build.sh

These three changes facilitate our ability to apply IBC data to Linux assemblies on Windows and the perform the crossgen step on Linux, which will be our flow for official builds when we want to apply IBC data since IBCMerge cannot run on non-Windows platforms

Update Unicode data to version 11 (#20529)

* Update Unicode data to version 11

* Move CharUnicodeInfoData.cs to the hsared folder

* Delete un-needed file

* Disable the failed test

Move ILLink and SizeOnDisk testing to Helix Queue (#20569)

Expose EnableISA knobs in release build (#20501)

Merge pull request #20497 from AaronRobinsonMSFT/additional_com_tests

Support for IDispatch

Delete duplicate CoreFX tests (#20532)

Contributes to https://github.com/dotnet/coreclr/issues/12782

Initialize `StructReturnType` for `gtNewRefCOMfield`. (#20056)

The issue could be repro only on .Net Framework, because this helper is only needed for remoting, that .Net Core doesn't have.

JIT: recover types from helper calls and more (#20447)

The jit needs to recover class handles in order to devirtualize and
do other type-based optimizations. This change allows the jit to find
the type for more trees: in particular, helper calls, intrinsics, and
expanded static field accesses.

Also, annotate a few methods to control jit optimization

We don't want to optimize special methods that are used to inform crossgen
about desirable generic instantiations. `CommonlyUsedGenericInstantiations`
was already annotated but `CommonlyUsedWinRTRedirectedInterfaceStubs` wasn't.

And because `RuntimeType` is sealed calls through types are now often
devirtualized. Name lookups on types are frequent, especially on error paths.
The method `GetCachedName` looks like an attractive inline but simply expands
into a larger sequence of two other calls. So block it from being inlined.

Use the same version of Xunit as in corefx (2.4.1 prerelease) (#20506)

From https://xunit.github.io/releases/2.4.1-pre-4059

**xUnit.net 2.4.1 Pre-Release (build 4059)**

Core framework
* BUG: Fixed an issue with xunit.abstractions not correctly installing with older versions of Visual Studio (pre-2017).
* BUG: Fixed an issue with ByRef types with .NET Core 2.1.
* BUG: Removed an unnecessary first chance exception related to attempting to load configuration files.
* BUG: Fixed an issue where synchronous and asynchronous disposal of test classes were running in parallel. The documented behavior is that asynchronous disposal (via IAsyncLifetime) runs to completion before synchronous disposal (via IDisposable).

Console runner
* BUG: Fixed an issue with XML element order with the NUnit report.

MSBuild runner
* BUG: Fixed an issue with XML element order with the NUnit report.

Merge pull request #20537 from BruceForstall/RemoveLocAllocSP

Remove the LocAllocSP slot for non-x86 platforms