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

Removing the unnecessary friend relationship

Merge pull request #24067 from briansull/desktop-fix2

Fix for Desktop build break

Added comment on #endif

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)

Remove unused offset calculation (#23793)

Utf8Parser has a redundant calculation of date time offset.

Fix #21834

Remove old method names from Rune (#23938)

* Remove old method names from Rune, reenable suppressed tests

* Revert re-enablement of failing ArraySegment tests

Removed support for short plug padding in `fit`.
Best fit is only done for gen1->gen2, and thus there is no padding.

Update CoreClr to preview5-27614-71 (#23978)

Adjust plug_size_to_fit to consider large alignment om ARM32

Fix broken new lines (#23979)

These files don't render correctly on github due to broken new lines - fixing it. Only new line characters replaced, nothing else touched.

Move GCMemoryInfo to shared partition

Fix AssemblyName regression

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

Move GCMemoryInfo to shared partition

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

Update BuildTools, CoreClr to preview4-03913-01, preview5-27612-73, respectively (master) (#23916)

* Update BuildTools, CoreClr to preview4-03913-01, preview5-27612-73, respectively

* Fix nullable build errors

Delete unused AssemblyName._hashForControl and related code (#23972)

EnterContextualReflection handle null (#23953)

* EnterContextualReflection handle null

* Add ContextualReflection MockAssembly test

* Fix ContextualReflection typo

Use xUnit TestOutputHelper to get proper test output in Azure DevOps Tests Explorer (#23957)

Delete outdated comment (#23967)

Remove unnecessary ThrowIfCreated call (#23839)

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

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

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

Fix ARM's genPutArgStk codegen (#23867)

* Fix ARM's genPutArgStk codegen

When the OBJ node wraps a LCL_VAR node the code uses the type information (struct size, GC layout) from LclVarDsc. This is not always correct because the OBJ may actually have a different struct type due to type reinterpretation (e.g. Unsafe.As<X, Y>).

* Fix genPutArgStk comment

Merge pull request #23948 from BruceForstall/Fix23920

Update maximum allowed arm prolog size

Move DateTime Windows implementation to shared CoreLib (#23888)

* Move DateTime Windows implementation to shared CoreLib

* Make all the BOOL/struct marshalling explicit

* internal -> private

* Use unsafe code on all layers, remove unnecessary fixed blocks

Delete outdated comment

Update dependencies from https://github.com/dotnet/corefx build 20190412.14 (#23964)

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

Delete COMPlus_FinalizeOnShutdown (#23595)

This compat quirk is increasingly more broken since the framework is generally not compatible with, and we have not heard anybody actually using it.

Changed Environment.HasShutdownStarted to unconditionally return false. It does not make sense for it to ever return true with shutdown finalization disabled.

Nullable: src\System\Runtime\InteropServices (#23936)

* Nullable: src\System\Runtime\InteropServices

* Address PR feedback