change SafeSortHandle to be CriticalSortHandle to avoid expensive ref counting

CoreRT change

Ensure gen0_max_size to be initially >= gen0_min_size

Otherwise, gen0_min_size is eventually capped by gen0_max_size, which makes it impossible to raise gen0 size above the default max sizes for gen0.
This is required for some scenarios (CppCodeGen, WASM) in CoreRT.

Multiple CoreRT changes

Fix casts

Fix Redhawk defines

UNREFERENCED_PARAMETER

Port typo fixes from CoreRT

JIT: use stress mode tail call validation info for implicit tail calls (#25093)

If a call site fails tail call stress validation, don't consider it for
implicit tail calling either.

In normal jitting we defer this level of validation until impImportCall
to avoid duplicating work.

This avoids an assert when we have invalid IL.

Fixes #25027.

Fix context restore for ARM/ARM64 debugging (#25089)

Fall back to CpuId if failed to get cache size from OS (#24989)

* Fall back to CpuId if failed to get cache size from OS

It's possible for GetLogicalProcessorCacheSizeFromOS() to fail;
this happens on alpine linux where it compiles to just `return 0;`.

As a fallback, we can get the cache size from CpuId. Previously that
was specific to x86; this PR preserves the behavior that we never call
GetLogicalProcessorCacheSizeFromOS on x86.

CpuId only works on x86 and amd64; on other systems we may still return
0 from here. Then GC defaults to a cache size of only 0.25MB.

Note: Removed the code in an `#ifdef _WIN64` that was nested inside of
`#if defined (_TARGET_X86_)`. Presuming that is dead code.

* Fix exception handler

Fix ForeignThreadExceptions test (#25074)

This test creates a native thread on which it calls a managed callback. It uses
std::thread to create the thread. The problem is that on MUSL based Linux distros,
the default stack size for secondary threads is 80kB, which is not enough for this
test.
Since std::thread has no way to set the thread stack size, the fix is to use
pthreads on Unix and keep the std::thread for Windows only.

Improve corefx testing (#24962)

CoreFX tests are now used to test CoreCLR on Windows/x64 and Linux/x64.
PR testing includes normal and no-tiered modes. CI testing will include
all JIT stress modes that apply to CoreCLR testing. Many tests have been
disabled to allow the jobs to (mostly) run error-free.

Support for other platforms requires fixes to the testhost generation.

Compilation fixes (#25084)

Track timer count and add Timer.ActiveCount (#25061)

* Track timer count and add Timer.Count

Part of https://github.com/dotnet/corefx/issues/38422

* Use lock to prevent tearing

* Add assert

* Rename Count to ActiveCount

* Fix build

Use the per session configuration to invoke the EventPipeProvider callbacks (#25088)

* Use the per session configuration to invoke the EventPipeProvider callbacks

* Avoid making the callback if the session does not have the provider

Delete dead code (#25066)

R2R ilstubs (#24823)

* Basic support for precompiled pinvoke stubs

* Generate R2R file with multiple references to same IL stub (one per method which the IL stub is associated with)
* Not all il stubs are p/invokes. Don't fail when they aren't.
* Consistently use IsDynamicScope and GetModule to avoid unsafe memory access in IL stub compilation paths
* Enable full p/invoke il stubs when compiling System.Private.Corelib
* Disable IL Stub generation in crossgen for ARM32.
- The cross bitness logic is not correct for IL Stub generation

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

- Microsoft.NETCore.App - 3.0.0-preview7-27811-01

Make GetSessionProvider const (#25085)

Refactor one-time ETL rundown event firing (#24993)

- Moved the `RuntimeInformation` and `TieredCompilation/Settings` events to fire on rundown on attach/detach

Update IEquatable/IComparable<T> for nullability (#25053)

We need to revise our use of nullable for `IComparable<T>` and `IEquatable<T>`:
- For reference types that implement `IComparable<T>`, the implementation should use `IComparable<T?>`, e.g. `String : IComparable<String?>`, so that the type can be used in methods with a `where T : IComparable<T>` constraint: `IComparable<T>`'s contravariance then allows both `T` and `T?` to work as generic type arguments.
- For reference types that implement `IEquatable<T>`, we can't take advantage of such contravariance, as by design `IEquatable<T>` is invariant (it would be a breaking change to make it contravariant, and even if it wasn't doing so is of questionable design given that equality and inheritance don't mix well).  Given the current language design, we have little option but to explicitly make any reference type implementations of `IEquatable<T>` be oblivious, and also make all `where T : IEquatable<T>` constraints be oblivious.  Otherwise, regardless nullable annotations, a developer will get a warning for using either or both of `T`/`T?` with any given method employing such a constraint.

Update dependencies from https://dev.azure.com/dnceng/internal/_git/dotnet-optimization build 20190610.3 (#25079)

- optimization.IBC.CoreCLR - 99.99.99-master-20190610.3
- optimization.PGO.CoreCLR - 99.99.99-master-20190610.3

Update dependencies from https://github.com/dotnet/corefx build 20190611.1 (#25077)

- Microsoft.Bcl.AsyncInterfaces - 1.0.0-preview7.19311.1
- Microsoft.NETCore.Platforms - 3.0.0-preview7.19311.1
- Microsoft.Private.CoreFx.NETCoreApp - 4.6.0-preview7.19311.1

Update dependencies from https://github.com/dotnet/arcade build 20190610.24 (#25076)

- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19310.24
- Microsoft.DotNet.Build.Tasks.Feed - 2.2.0-beta.19310.24
- Microsoft.DotNet.Build.Tasks.Packaging - 1.0.0-beta.19310.24
- Microsoft.DotNet.Helix.Sdk - 2.0.0-beta.19310.24

Enable lab support for coreclr tests in unloadable context (#25064)

Add optimization tiers to the Linux perf maps for perfcollect (#24967)

Add optimization tiers to the Linux perf maps for perfcollect

Fixes https://github.com/dotnet/coreclr/issues/23222:
- It looks like module unloads are currently not taken into account. Once they would be taken into account, Although we have method JIT events from `lttng` with the code address and optimization tier, samples can only be associated with method JIT events by associating the time range when the module is loaded with times of samples, and the event times from `lttng` would not necessarily correspond with times from samples taken by `perf`.
- Updated to include the optimization tier in the perf map for each jitted or R2R method code address
- Refactored common code between eventtrace and perfmap for getting jit tiers

Switch P/Invokes into libcoreclr for the PAL to QCalls (#25055)

* Switch PInvokes into libcoreclr to QCalls

* Register QCalls in runtime. Remove pal exports from libcoreclr

* Alphabetize new ecalllist.h entries

* Remove W suffixes from native side of QCall

* Interop signatures are in nested classes (which doesn't show up here) in the root namespace. So we don't want to pass a namespace in here

* Convert the interop test in R2R to use a local native library instead of calling into the CoreCLR PAL via DllImport.

fix generic attribute bug when generic attribute has parameters and the parameters (#25054)

* fix generic attribute bug

* add missing test cases

Fix inline tracking (#25062)

Inline tracking didn't correctly place the right key into the
inline tracking map for generics. The logic places the instantiated
methoddesc into the tracking map, and then attempts to find it by
looking up the uninstantiated form. This would generally be a silent
failure, but in unloadable assembly testing we have an assert that the
CrossLoaderAllocatorHash holds as keys items from a matching
LoaderAllocator.

This issue also results in an assert `key->GetLoaderAllocator() ==
m_pLoaderAllocator` in CrossLoaderAllocatorHash when the code is
running inside of a collectible AssemblyLoadContext.

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

* Update dependencies from https://github.com/dotnet/core-setup build 20190605.02

- Microsoft.NETCore.App - 3.0.0-preview7-27805-02

* Update dependencies from https://github.com/dotnet/core-setup build 20190606.03

- Microsoft.NETCore.App - 3.0.0-preview7-27806-03

* Update dependencies from https://github.com/dotnet/core-setup build 20190608.01

- Microsoft.NETCore.App - 3.0.0-preview7-27808-01

* Update dependencies from https://github.com/dotnet/core-setup build 20190609.01

- Microsoft.NETCore.App - 3.0.0-preview7-27809-01

* Update dependencies from https://github.com/dotnet/core-setup build 20190610.02

- Microsoft.NETCore.App - 3.0.0-preview7-27810-02

Use pread() to read DOS/NT headers when mapping PE files (#25059)

Shaves off two syscalls per managed assembly load.

Add source-build hook for dotnet install (#24929)

* Add source-build hook for dotnet install

* Fix batch syntax

* Address feedback

* Use the correct returned variables:

unix: _InitializeDotNetCli
windows: DOTNET_TOOL_DIR

* Remove changes to dotnet.cmd

* Do not export, just use _InitializeDotNetCli

Make Debug.Print(string) behavior consistent with .NET Framework (#25036)

*     Debug.Print should call Debug.WriteLine
    - to behave similar to .NET Framework

* temporarily disable tests
- Waiting for coreclr/fx changes to propagate

Add support for the NetTrace EventPipe file format

Right now the new format is not on by default, but it can be enabled using COMPlus_EventPipeNetTraceFormat = 1 for testing purposes. The plan to have a follow up PR that will add shipping configuration mechanisms and change the default setting.

See the documentation in the PerfView repo for more details about the format. At a glance the goal is to create a format that is more efficient to produce, has a smaller on disk size, and offers enhanced functionality in a few areas:
a) 64 bit thread id support
b) Detection of dropped events via sequence numbers
c) Better support for extracting subsets of the file

Together with the change there was also some refactoring of the EventPipeBufferManager and EventPipeThread.

This change addresses (at least in part) the following issues:
#19688, #23414, #24188, #20751, #20555, #21827, #24852, #25046

[master] Update dependencies from dnceng/internal/dotnet-optimization (#25023)

* Update dependencies from https://dev.azure.com/dnceng/internal/_git/dotnet-optimization build 20190606.3

- optimization.IBC.CoreCLR - 99.99.99-master-20190606.3
- optimization.PGO.CoreCLR - 99.99.99-master-20190606.3

* Update dependencies from https://dev.azure.com/dnceng/internal/_git/dotnet-optimization build 20190610.1

- optimization.IBC.CoreCLR - 99.99.99-master-20190610.1
- optimization.PGO.CoreCLR - 99.99.99-master-20190610.1

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

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

- Microsoft.Bcl.AsyncInterfaces - 1.0.0-preview7.19306.9
- Microsoft.NETCore.Platforms - 3.0.0-preview7.19306.9
- Microsoft.Private.CoreFx.NETCoreApp - 4.6.0-preview7.19306.9

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

- Microsoft.Bcl.AsyncInterfaces - 1.0.0-preview7.19307.10
- Microsoft.NETCore.Platforms - 3.0.0-preview7.19307.10
- Microsoft.Private.CoreFx.NETCoreApp - 4.6.0-preview7.19307.10

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

- Microsoft.Bcl.AsyncInterfaces - 1.0.0-preview7.19309.1
- Microsoft.NETCore.Platforms - 3.0.0-preview7.19309.1
- Microsoft.Private.CoreFx.NETCoreApp - 4.6.0-preview7.19309.1

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

- Microsoft.Bcl.AsyncInterfaces - 1.0.0-preview7.19309.5
- Microsoft.NETCore.Platforms - 3.0.0-preview7.19309.5
- Microsoft.Private.CoreFx.NETCoreApp - 4.6.0-preview7.19309.5

Update dependencies from https://github.com/dotnet/arcade build 20190609.1 (#25051)

- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19309.1
- Microsoft.DotNet.Build.Tasks.Feed - 2.2.0-beta.19309.1
- Microsoft.DotNet.Build.Tasks.Packaging - 1.0.0-beta.19309.1
- Microsoft.DotNet.Helix.Sdk - 2.0.0-beta.19309.1

Use SIZE_T for SectionSize in RegisterFrozenSegment (#25044)

Fix field offset computation for large version bubble (#25029)

There was a discrepancy in field offset calculations at crossgen time
and at runtime in some rare cases due to the alignment of a derived
class offset.
The issue happened due to MethodTableBuilder::NeedsAlignedBaseOffset not
taking into account the fact that the module of the parent and child
class can both be in large version bubble.

We also had a bug in the PEDecoder::GetNativeManifestMetadata. When it
was called for regular crossgened image without large version bubble, it
left the pDir uninitialized due to the fact that there was no
READYTORUN_SECTION_MANIFEST_METADATA. And then it tried to dereference
that.

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

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

- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19306.12
- Microsoft.DotNet.Build.Tasks.Feed - 2.2.0-beta.19306.12
- Microsoft.DotNet.Build.Tasks.Packaging - 1.0.0-beta.19306.12
- Microsoft.DotNet.Helix.Sdk - 2.0.0-beta.19306.12

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

- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19307.12
- Microsoft.DotNet.Build.Tasks.Feed - 2.2.0-beta.19307.12
- Microsoft.DotNet.Build.Tasks.Packaging - 1.0.0-beta.19307.12
- Microsoft.DotNet.Helix.Sdk - 2.0.0-beta.19307.12

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

- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19307.62
- Microsoft.DotNet.Build.Tasks.Feed - 2.2.0-beta.19307.62
- Microsoft.DotNet.Build.Tasks.Packaging - 1.0.0-beta.19307.62
- Microsoft.DotNet.Helix.Sdk - 2.0.0-beta.19307.62

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

- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19308.1
- Microsoft.DotNet.Build.Tasks.Feed - 2.2.0-beta.19308.1
- Microsoft.DotNet.Build.Tasks.Packaging - 1.0.0-beta.19308.1
- Microsoft.DotNet.Helix.Sdk - 2.0.0-beta.19308.1

Fix `EventPipe::Disable` threading issues. (#24896)

`EventPipe::WriteEvent*` does not use the `EventPipe::Crst` which created race conditions with other methods such as EventPipe::Disable.

don't acquire lock if the collator has been already created (#24973)

* don't acquire lock if the collator has been already created

* use atomic_compare_exchange_strong instead of __sync_bool_compare_and_swap

* don't try to workaround clang 5.0 bug, just use __atomic_compare_exchange_n

Move Marshal class to S.P.CoreLib (dotnet/corert#7502)

After this change, CoreRT only uses S.P.Interop to host the interop hashtable readers. Those can't move to CoreLib because they call into the type loader.

I didn't touch ProjectN's version of Marshal and I left the `PInvokeMarshal` class in place to help with that. I guess we'll need to make a call at some point on what to do with S.P.Interop since without MCG the code in it is kind of useless.

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

Fix issue in NamedPipeServerStream.GetImpersonationUserName() (#37676)

Fix for issue where NamedPipeServerStream.GetImpersonationUserName() throws unexpected InvalidationOperationException("The operation completed successfully") when impersonation is not allowed, when in fact it should simply return an empty string.

The fix will call GetNamedPipeHandleStateW once. If it fails, it will try to load sspicli.dll if it was not loaded, and will reattempt the call to GetNamedPipeHandleStateW.

This fix is only for Win32: UAP does not allow calling LoadLibraryEx, and Windows Nano says denies access to GetNamedPipeHandleStateW even after attempting to load sspicli.dll.

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

Fix a race condition in EventPipe (#25025)

Fix redist

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

Fix AppDomain issues (#25019)

Populate AppDomain.CurrentDomain when invoking events
Make AppDomain method implemented in AssemblyLoadContext internal

Mention `COMPlus_AltJitNgen=*` for crossgen

Merge pull request #24007 from mikedn/enable-w4

Compile JIT sources with /W4

Explicitly set permission on the socket file created by DiagnosticServer (#24988)

* Explicitly set permission on the socket file created by DiagnosticServer

* Add an asserte and remove execute permission

* Use umask to set permission instead of fchmod

* use fchmod for linux, ifdef out osx specific permission sets

* Set correct permission bit in call to umask

* Forgot to recover umask on one of the error paths

Do not delete polymorphic objects without a virtual destructor (#23705)

SEI CERT C++ Coding Standard says:

> Do not delete an object of derived class type through a pointer to its
> base class type that has a non-virtual destructor. Instead, the base
> class should be defined with a virtual destructor. Deleting an object
> through a pointer to a type without a virtual destructor results in
> undefined behavior.

See https://wiki.sei.cmu.edu/confluence/display/cplusplus/OOP52-CPP.+Do+not+delete+a+polymorphic+object+without+a+virtual+destructor

Clang generally warns about this, but we disabled the warning via
-Wno-delete-non-virtual-dtor. This commit re-enables the warning and
fixes up all the code that hits the warning.

Add docs for rejit on attach and the new profiler attach mechanism (#24868)

Reduce indirect function calls (#24980)

* Reduce indirect dispatch in hot paths in metadata

* Remove allocation indirections in utilcode for coreclr
- we no longer have a multi-dll distribution of code that needs to share heaps

* Remove unused code in GetCLRFunction

* Remove virtual dispatch around impl/decl methodtable access in MethodData

Address nullable feedback for System.Runtime and System.Runtime.Extensions (#25017)

* Address last nullable feedback from System.Runtime

* Address nullable feedback for System.Runtime.Extensions

* PR Feedback

Fix blr printing (#24979)

A blr instruction may have a result that is not a pointer size, but its target register is always EA_PTRSIZE.

Fix #24940

Add [NotNull] to syncLock parameter of EnsureInitialized (#25012)

Add DisplayUnits property to DiagnosticCounter (#24981)

Marking Vector128<T>.Count and Vector256<T>.Count as [Intrinsic] (#24991)

* Marking Vector128<T>.Count and Vector256<T>.Count as [Intrinsic]

* Fixing NI_Vector128_Count and NI_Vector256_Count to use clsHnd when getting the simdSize and baseType

* Applying the formatting patch.

* Changing some comments to just be "vector element count".

* Fixing impBaseIntrinsic to set the baseType so Vector128_Count and Vector256_Count don't return nullptr

Use CMake's C# support to build DacTableGen instead of manually invoking csc.exe ourselves. (#24342)

* Use CMake's C# support to build DacTableGen instead of manually invoking csc.exe ourselves.

* Fix x86 failures.

* Disable DAC generation when building with NMake Makefiles and issue an error since the CMake C# support is VS-only. We don't actually support building with NMake (only configure) so this is ok.

* Clean up rest of the macro=1's

PR Feedback.

* Fix Visual Studio generator matching.

* Explicitly specify anycpu32bitpreferred for DacTableGen so the ARM64 build doesn't accidentally make it 64-bit

* Fix bad merge

Move COM object lifetime control tests to CoreCLR. (#24990)

* Add COM object lifetime control tests to CoreCLR.

* Combine cleanup control test into NETClientPrimitives.

Fix initializing BuildTools when the Platform environment variable is set (#24998)

* Make build-test.cmd exit if init-tools.cmd fails

* Clear Platform variable before initializing BuildTools

Merge pull request #25011 from dotnet/revert-24987-ReenableStackTraceTest

Revert "Reenable StackTracePreserveTests."

Revert "Reenable StackTracePreserveTests."

Clear syncblock early when `VERIFY_HEAP && DEBUG` to prevent verification asserts. (#24992)

Fixes:#24879

Reenable StackTracePreserveTests. (#24987)

I was not able to repro any failures, maybe they were fixed already. Lets see what ci thinks.

Exclude tests that do not work without tailcalls on arm32 Unix. (#24995)

Delete unnecessary security transparency attributes (#25007)

Shortenning /Platform_Assemblies_Paths to just /p for convenience. Leaving the original switch name for compatibility reasons. (#24986)

Add R2R outerloop test mode without any stress modes (#24961)

* Add R2R outerloop test mode without any stress modes

PInvoke stubs for Unix platforms (#24834)

Enable pinvoke stub inlining on Unix
Exclude x86 Unix platforms from inlining pinvoke stubs (limited support)

Update dependencies from https://dev.azure.com/dnceng/internal/_git/dotnet-optimization build 20190606.1 (#25006)

- optimization.IBC.CoreCLR - 99.99.99-master-20190606.1
- optimization.PGO.CoreCLR - 99.99.99-master-20190606.1

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

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

- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19305.13
- Microsoft.DotNet.Build.Tasks.Feed - 2.2.0-beta.19305.13
- Microsoft.DotNet.Build.Tasks.Packaging - 1.0.0-beta.19305.13
- Microsoft.DotNet.Helix.Sdk - 2.0.0-beta.19305.13

Update dependencies from https://github.com/dotnet/corefx build 20190606.1 (#25004)

- Microsoft.Bcl.AsyncInterfaces - 1.0.0-preview7.19306.1
- Microsoft.NETCore.Platforms - 3.0.0-preview7.19306.1
- Microsoft.Private.CoreFx.NETCoreApp - 4.6.0-preview7.19306.1

Fix tmp dir in configure.cmake (#25003)

The ONE_SHARED_MAPPING_PER_FILEREGION_PER_PROCESS check was using a temp
path that had some non-existent components. While this works fine on Linux,
it fails to create the temp file on OSX.
The fix is to use temp dir in the CMake's output dir.

Delete TwoKeywords test. (#24982)

Merge two equal exclusion groups for arm32 All OS. (#24984)

There were two:
<!-- Arm32 All OS -->
    <ItemGroup Condition="'$(XunitTestBinBase)' != '' and ('$(BuildArch)' == 'arm' or '$(AltJitArch)' == 'arm')">
and
<!-- arm32 All OS specific excludes -->
    <ItemGroup Condition="'$(XunitTestBinBase)' != '' and  ('$(BuildArch)' == 'arm' or '$(AltJitArch)' == 'arm')">

Less work for OptimalMaxSpinWaitsPerSpinIteration fast-path (#21702)

APIs to allow profilers to use DoStackSnapShot on Linux (#24968)

Fix GCStress coverage for multi reg returns. (#24826)

* Extract ReplaceInstrAfterCall.

* Avoid GCStress when return multireg with pointers.

Determinate when we need to protect the second register and do not cause GCStress in such cases.

* Add a repro test.

* Reenable MethodImplOptionsTests.

* Extract IsGcCoveregeInterruptInstruction.

That changes how we do checks for arm32 in `IsGcCoverageInterrupt`.

* Tolerate direct call to JIT_RareDisableHelper.

x86 ILStubClass:IL_STUB_PInvoke(byref,ref,int,byref):int generates it like:
Generating: N119 (  4,  7) [000118] ------------              *  RETURNTRAP int    REG NA
IN0021:        cmp      dword ptr [0F9BF9F8H], 0
New Basic Block BB10 [0009] created.
IN0022:        je       L_M6496_BB10
                                                        Call: GCvars=00000001 {V01}, gcrefRegs=00000000 {}, byrefRegs=00000000 {}
IN0023:        call     CORINFO_HELP_STOP_FOR_GC

* Support GC stress protect return 1/2/both Unix x64.

* Fix arm64.

Do not insert GC Stress instrucitons when we can't determinate the exact return kind.

* Fix review1.

* Fix review2.

* Change the test as Andy suggested.

* Fix some typos.

* Replace all SLOT with PBYTE.

* Disable assert that can fail because of multithreading.

Reenable StackTracePreserveTests.

I was not able to repro any failures, maybe they were fixed already. Lets see what ci thinks.

Fix PAL_GetLogicalProcessorCacheSizeFromOS on mac (#24777)

* Fix PAL_GetLogicalProcessorCacheSizeFromOS on mac

In a previous PR
(https://github.com/dotnet/coreclr/commit/ed52a006c01a582d4d34add40c318d6f324b99ba#diff-8447e54277bb962d167a77bb260760d7R1879),
GetCacheSizePerLogicalCpu was changed to no longer rely on cpuid on
amd64 systems; instead it uses GetLogicalProcessorCacheSizeFromOS().
Unfortunately that function consisted of a number of `#if`s, none of
which were active on macs, and we just returned 0. This caused us to
default to a gen0size of only 0.25MB, causing many GCs.

Fixed by adding a new case that uses `sysctlbyname`.

Fix #24658

* Fixes from code review

* Check for function sysctlbyname instead of header

Shortenning /Platform_Assemblies_Paths to just /p for convenience. Leaving the original switch name for compatibility reasons.

Zero initReg in genSetGSSecurityCookie (#24371)

Fix an issue that is reproduced when

1) a register is used in the prolog for initializing GSSecurityCookie (i.e. the register contains a random non-zero value) and
2) the same register holds a must-init GC variable in the next basic block and
3) the variable is live at the beginning of this basic block.

The register was not zeroed at the end of the prolog and this was causing segmentation fault during GC.Collect().

Merge pull request #24937 from safern/FixNullableAnnotationsApiReview

Fix some nullable annotations from API Review

Cleanup block stores and test for 24846 (#24950)

* Cleanup block stores and test for 24846

Fix zero-length assert/bad codegen for initblk.
Remove redundant assertions in codegen and those that don't directly relate to codegen requirements.
Eliminate redundant LEA that was being generated by `genCodeForCpBlk`.
Rename `genCodeFor[Cp|Init]Blk` to `genCodeFor[Cp|Init]BlkHelper` to parallel the other forms.
Fix the test case for #24846.

Add test verifying reliable SafeHandle unmarshalling (#24959)

* Add test ensuring we correctly unmarshal a SafeHandle even when the stub throws an exception.

* PR Feedback.

Use Modern CMake features instead of CMAKE_CXX_FLAGS (#24861)

* Convert C++ standard settings and warning options from CMAKE_<LANG>_FLAGS to Modern CMake isms.

* More $<COMPILE_LANGUAGE> generator expressions instead of CMAKE_CXX_FLAGS.

* Use $<COMPILE_LANGUAGE:CXX> for all -fpermissive usage

* Fix generator expression that generates multiple flags

* Fix invalid use of CMAKE_CXX_FLAGS instead of CMAKE_C_FLAGS.

* Treat AppleClang as though it is Clang (match pre-3.0 behavior).

* Update our build system to understand that AppleClang is distinct from Clang and remove CMP0025 policy setting.

* PR Feedback.

Disable corefx tests

Small fixes around AST nodes. (#24957)

* Fix MEASURE_NODE_SIZE and naming mistakes.

* The additional fields were deleted in #14582 (~1.5 years ago).

* Fix  GT_INDEX_ADDR def.

We created them as `new (this, GT_INDEX_ADDR) GenTreeIndexAddr` but used smaller `GenTreeIndex` as nessecary size.

* Use LargeOpOpcode instead of GT_CALL.

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

- Microsoft.NETCore.App - 3.0.0-preview7-27804-03

Update dependencies from https://github.com/dotnet/corefx build 20190605.1 (#24970)

- Microsoft.Bcl.AsyncInterfaces - 1.0.0-preview7.19305.1
- Microsoft.NETCore.Platforms - 3.0.0-preview7.19305.1
- Microsoft.Private.CoreFx.NETCoreApp - 4.6.0-preview7.19305.1

Resolve Windows 10 Kits dir via the environment variable. (#24960)

* Resolve Windows 10 Kits dir via environment variable.

Update dependencies from https://dev.azure.com/dnceng/internal/_git/dotnet-optimization build 20190605.1 (#24972)

- optimization.IBC.CoreCLR - 99.99.99-master-20190605.1
- optimization.PGO.CoreCLR - 99.99.99-master-20190605.1

signedness (#24878)

Remove stale Task async extension methods from TaskExtensions (#24958)

Remove Collection<T> range APIs (#24938)

Synchronize managed and unmanaged Variant fields (#24951)

* Synchronize managed and unmanaged Variant fields

Fixes #24948

More PR Feedback

Make sure we can enable auto-trace when the root level define is added (#24955)