[master] Update dependencies from dotnet/core-setup (#24059)

Microsoft.NETCore.App - 3.0.0-preview5-27618-16

Move CLR_ADDITIONAL_COMPILER_OPTIONS to be set after default options (#24044)

CLR_ADDITIONAL_COMPILER_OPTIONS was set before visibility settings and
thus adding additional compiler options would be overwritten.

Fix build errors in Interop.Errors

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

Disable warning in ValueStringBuilder.ToString

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

Block usage of default interfaces feature in COM scenarios (#23970)

* Block default interface usage in COM scenarios

Add additional test cases and move some CCW functions to a more
consolidated layout.

Look for IBC data outside Windows (#24062)

#23363 added a crossplat implementation of `GetWin32Resource`, but forgot to enable places that call it.

Nullable: couple of missing #nullable enable (#24094)

Nullable: System.Collections.Concurrent (#24054)

Fix performance regression on glibc on Linux (#24099)

Move R2R-specific code to be outside FEATURE_PREJIT (#24075)

This refactoring is preparation for disabling fragile NGen support in the runtime. It keeps fragile-NGen specific code under FEATURE_PREJIT and moves the code required to support R2R to be outside FEATURE_PREJIT.

The eventual goal is to compile the runtime without FEATURE_PREJIT defined to avoid fragile-NGen specific overhead.

Adding Sse2.StoreScalar(long*, V128) which will replace Sse2.StoreLow(long*, V128) (#24102)

Don't return value types as const (#24103)

These types are value types; they are returned by copying. Marking the
return type as const has no effect: since the value is copied, the
caller can modify it anyway.

This resolves all the -Wignored-qualifiers warnings produced by clang 7
(when those warnings are explicitly enabled).

Merge pull request #24105 from BruceForstall/ReduceLinuxArm32PeriodicJobs

Reduce the number of Linux arm32 periodic jobs triggered

Allow UpperVector Save and Restore without base type (#24080)

Fixes an issue with crossgen on Arm64/linux

Reduce the number of Linux arm32 periodic jobs triggered

Only the corefx jobs are left. The rest should be done in AzDO,
and scheduling both AzDO and Jenkins is causing too much work
for the few machines we have.

Fix RVA field placement order (#24047)

* Static RVA fields should be placed sequentially after the qsort operation to not break Managed C++ binaries in R2R

* Add regression test

Disable native-client tests on IL round-trip (#24041)

If IPC connections fails, then remove active session. (#24082)

Merge pull request #24071 from AndyAyersMS/Fix24055

JIT: fix arm64 crossgen simd issue

Add a destructor for sos::GCHeap (#24072)

The constructor allocates memory via new[]. It should be freed up via
delete[] by a destructor to avoid a memory leak.

Merge pull request #23853 from jkoritzinsky/cmake-ninja-fix

Fix CMake warning about clrjit.exports.in file when using Ninja generator

Merge pull request #24066 from janvorli/put-back-cpu-limits

Put back the CPU limiting in GC

Skip type sanity checks for CoreLib (#24074)

Fixed ChunkEnumerator.Current NRE (#24076)

* Fixed ChunkEnumerator.Current NRE

* Added Nullable Flow analysis annotation comment

* Changed TODO-NULLABLE comment to link to appropriate issue

Merge pull request #23934 from franksinankaya/gcc_cleanup_18

Integer Conversion Issues from GCC

Merge pull request #23762 from dotnet/NullableFeature

Nullable feature into master

another similar fix

Removing the unnecessary friend relationship

Merge pull request #24067 from briansull/desktop-fix2

Fix for Desktop build break

Added comment on #endif

JIT: fix arm64 crossgen simd issue

Need to check `supportSIMDType()` and not `featureSIMD` in rationalize.

Fixes #24055.

JIT: ignore pinning of non-gc types (#24010)

We may see pin modifiers on locals that are not gc types.
Ignore these modifiers.

Closes #23950.

Also added a missing copyright header on an unrelated test.

Fix StringBuilder annotations and debug.asserts (#24069)

Runtime counters naming change and setting default rates.  (#24052)

* Renaming runtime counters and giving them DisplayNames

* fix build err

* Adding default DisplayRateTimeScale to runtime counters

Restrict morph add rearrangement. (#23984)

Morph has transformations
((x + const) + y) => ((x + y) + const)
and
((x + const1) + (y + const2)) => ((x + y) + (const1 + const2))

If x or y is a GC pointer and one of the int operands may be negative,
this may result in a byref temp that points outside of the ref object.
If the code is in a non-interruptible region and a GC happens, the byref
won't be updated.

This change disallows the transformations if one of the non-const
operands is a GC pointer.

Fixes #23792.

Merge pull request #24065 from sandreenko/fixCoreFXScripts

Add --runtime-path to CoreFX test runner.

Fix for Desktop build break

Add lvIsImplicitByRef information to lvaSetStruct (#19223)

Before implicit byrefs were tracked by setting lvIsParam and lvIsTemp.
This change explicitly adds a flag for implicitByRef instead of overloading.
In addition, it fixes the decision to copy an implicitByRef for arm64 varargs.

Temporarily bump weight on byref params to match old behavior and avoid codegen
diffs.

Re-enabled various tests and parts of tests.

Closes #20046
Closes #19860

Put back the CPU limiting in GC

The CPU limiting was accidentally removed during refactoring of the CPU
groups support in GC. This change puts them back.

Add --runtime-path to CoreFX test runner.

Update the types consumers of Index and Range (#24036)

* Update the types consumers of Index and Range

The C# compiler now can automatically support the indexers with Index and Range parameters on any type meet the conditions:
- The type exposes Length or Count property which returning integer.
- The type has indexer of integer parameter.
- The type has a Slice method which takes 2 integers

The change here is to remove the methods and indexers of the types which we previously added it as the compiler will automatically provide those.

* Remove the JIT failing test

Exclude ToString_NullFrame_ThrowsNullReferenceException test from nullable change

Fix a ZeroMemory call in methodcontext.cpp (#23843)

The argument is of type `GetArgTypeValue` and so the `sizeof()` should be
done on `GetArgTypeValue`, not on the different type `GetArgType`.

Nit: Update comment (#23896)

Update CoreClr to preview5-27616-71 (#24020)

Update dependencies from https://github.com/dotnet/corefx build 20190415.16 (#24030)

- Microsoft.NETCore.Platforms - 3.0.0-preview5.19215.16
- Microsoft.Private.CoreFx.NETCoreApp - 4.6.0-preview5.19215.16

Update dependencies from https://github.com/dotnet/arcade build 20190415.12 (#24029)

- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19215.12
- Microsoft.DotNet.Helix.Sdk - 2.0.0-beta.19215.12

Remove limits on IVT public key lengths. (#24042)

Changing Math.Max, Math.Min, Math.MaxMagnitude, and Math.MinMagnitude to propagate NaN inputs (#24039)

Fix incorrect EHCount calculation (#24048)

Fix incorrect thread affinitization (#24045)

The PAL_SetCurrentThreadAffinity was incorrectly adding the specified processor
to the current thread affinity set instead of setting the affinity to only
the processor specified.
It was causing significant performance hit in aspnet benchmarks on machines with
many cores.
This change crept in when I was refactoring the related code while removing
CPU groups emulation.

[EventPipe] Minor bug fixes, and remove redundant/unused code. (#23956)

- Update MicrosoftDiagnosticsTracingTraceEventPackageVersion
- Delete connection if the request is unknown/unhandled
- Adding missing error handling.
  - Provider names must be defined.
- Some renaming, error handling, and build warnings.
- Removing test code, and merge EventPipe::Enable functions.
- Remove commented/non-used lines.

Merge pull request #24037 from BruceForstall/ReduceStackRequirement

Reduce stack requirements of test

Merge pull request #24002 from sandreenko/fixDesktopFailure

Another surgery fix for genPutArgStk to fix desktop build.

Fix if condition in AssemblyLoadContext

Merge pull request #23947 from sandreenko/reenableSystem.ComponentModel.Composition.Tests

Reenable System.ComponentModel.Composition.Tests arm64 Windows.

Reduce stack requirements of test

Fixes #24021

Use lvaGetDesc.

Nullable: Addition COM types (#24032)

Somehow these slipped through previous rounds of annotating.

Merge remote-tracking branch 'dotnet/master' into NullableFeature

Downgrade xUnit package version (#23944)

* Use the same version of xUnit that arcade uses (2.4.1-pre.build.4059)

Arm64 vector ABI (#23675)

* Support for Arm64 Vector ABI

Extend HFA support to support vectors as well as floating point types.
This requires that the JIT recognize vector types even during crossgen,
so that the ABI is supported consistently.

Also, fix and re-enable the disabled Arm64 Simd tests.

Fix #16022

Test remove ContextualReflectionProxy (#24006)

Update ContextualReflection doc (#24018)

* Update ContextualReflection doc

* PR Feedback

Use delete [] on array types (#24027)

Calling delete on types allocated with new[] leads to undefined
behaviour.

Move EventSource to shared CoreLib partition (#23925)

* Move most of EventSource to shared CoreLib partition

* Reenable some code that can be shared with Microsoft.Diagnostics.Tracing.EventSource.Redist in CoreFX

* Remove unused functions

* Fix placement of #endregion

* Clean up P/Invoke prototypes

* Move remaining code from EventSource_CoreCLR to shared partition (guarded by !CORERT)

* Remove more unnecessary marshalling attributes

* Move remaining CoreRT/ProjectN code to shared partition

* Avoid using Windows errors on non-Windows platforms

Merge pull request #24009 from sandreenko/fixLsraBlockEpochCheck

Fix lsraBlockEpoch check.

Use NewArrayHolder for array types in src/debug (#24013)

Using a NewHolder with array types means that when the holder is ready
to release the  memory, it ends up invoking `delete` (instead of
`delete[]`) on that array. This is an undefined behaviour.

Use NewArrayHolder isntead to fix this.

Use NewArrayHolder for array types (#24017)

This touches all the code outside of src/debug/.

Using a NewHolder with array types means that when the holder is ready
to release the memory, it ends up invoking `delete` (instead of
`delete[]`) on that array. This is an undefined behaviour.

Use NewArrayHolder instead to fix this.

Conversion issues

Fix an issue with literal 0xFFFFFFFFFFFFF000LL being treated as unsigned long long int (#24011)

Update clr-configuration-knobs and cleanups (#24016)

Fix incorrect bit match pattern in UTF-16 validation (#24015)

Fold ResourceSet::CommonInit into constructor (#23993)

Fix lsraBlockEpoch check.

Initialize IsolatedComponentLoadContext.Name (#24001)

Add "partial" to MarshalAsAttribute (#24014)

Delete unused YieldProcessorScalingFactor from GC (#23994)

Remove redundant UnderlyingSystemType calls (#24000)

UnderlyingSystemType is virtual

Check for null methodhandle before checking if the method is in the same version bubble (#23940)

Fix precompiling CoreRoot overlay with crossgen for Windows arm/arm64 (#23946)

Opt COM methods out of the new Windows instance-method handling. (#23974)

* Opt COM methods out of the new Windows instance-method handling.

* Add test for an HResult "struct" returned from a COM method.

* Update ErrorMarshalTesting.cs

* Update "is member function" check on the ilmarshalers.h side to only consider thiscall.

Improved performance of System.Reflection.Emit API (#23833)

* Improved performance of System.Reflection.Emit API

* Moved heap allocation to stack

* Eliminating conditional branch by casting sbyte to byte

* Reduced heap allocation of ScopeTree

* Removed redundant `unsafe` from function signature

Merge pull request #23997 from safern/merge-master-nullable

Merge master into nullable feature

Fix build errors from merge conflicts and compiler update

Get rid of varNumInp.

Another surgery fix for genPutArgStk to fix desktop build.

Merge pull request #23981 from VSadov/arm32fix22422

Adjust plug_size_to_fit to consider large alignment on ARM32

Merge pull request #23996 from BruceForstall/Fix64BitAlignment

Fix test for 64-bit platforms

Fix test for 64-bit platforms

Object type in structs apparently must be 8 byte aligned.

Fixes #23986

[master] Update dependencies from dotnet/arcade (#23977)

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

- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19214.2
- Microsoft.DotNet.Helix.Sdk - 2.0.0-beta.19214.2

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

- Microsoft.NETCore.App - 3.0.0-preview5-27614-10

Merge remote-tracking branch 'dotnet/master' into merge-master-nullable

Update CoreClr to preview5-27615-71 (#23991)

Update dependencies from https://github.com/dotnet/corefx build 20190414.16 (#23989)

- Microsoft.NETCore.Platforms - 3.0.0-preview5.19214.16
- Microsoft.Private.CoreFx.NETCoreApp - 4.6.0-preview5.19214.16

improve entropy (#23591)

Update CoreClr to preview5-27614-72 (#23985)

Fix a few internal nullable annotations and comments (#23987)

Fix typo in environment variable name - profiler loading doc (#23983)

Nullable: System.Runtime.InteropServices.CustomMarshalers/WindowsRuntime (#23930)