Merge pull request #24232 from janvorli/fix-large-version-bubble-2

Fix crossgen with large version bubble enabled

Improve the performance of the type loader (#24177)

Improve the performance of the type loader
- Disable interface ambiguity checks in Corelib
- Improve StackingAllocator perf
  - Increase size of allocation blocks
  - Cleanup allocation blocks comletely on gc suspension
- Check for presence of generic parameters to methods in a more efficient manner
  - Querying the count of GenericParameter records requires a binary search in the GenericParams table
  - Checking the generic flag on the method signature is effectively free
- Skip unnecessary checks in EnumerateClassMethods for corelib
- Use Sanity check flag where appropriate
- Use slightly faster hashtable functions in StringLiteralMap
  - Remove pointless string literal entry hash table search
- Change stacking allocator to not be allocated for the lifetime of a thread, but instead its allocated/destroyed around the first frame on the thread that needs the allocator.
  - Allocate at most 1 stacking allocator per thread
  - The allocation is on the stack via _alloca (as managed by the ACQUIRE_STACKING_ALLOCATOR macro
  - This will put an 8KB buffer on the stack, and if there is need for a larger buffer it will allocate off of the heap.
  - Stacking allocator initial block is always present, so the allocation routine for the allocator can be somewhat simpler
  - More logic has been move the the cpp file, to reduce header bloat, and improve iteration when modifying StackingAllocator code
  - Avoid use of alloca for StackingAllocator when less than 512KB of stack is available

Fix verification of 3-byte UTF-8 sequence followed by non-ASCII byte (#24235)

Return null for unexpected HW intrinsic (#24219)

Fix #17815

Fixing 23941 (#24199)

* Fixing 23941

The issue here is that in R2R code, unlike jitted IL code, the pinvoke calls are wrapped by a pair of calls to the JIT_PInvokeBegin and JIT_PInvokeEnd helpers, which link and unlink the InlinedCallFrame to the current thread. We do not initialize and link the ICF in the method prolog, and unlink it in the epilog, like jitted code does. We do this in the JIT helpers, right before/after the pinvoke call.
When an exception is thrown, the JIT_PInvokeEnd helper will be skipped since execution will resume at the nearest valid catch block, and the ICF will remain linked to the thread, which poses a problem if the method attempts another pinvoke operation (it will try to link the ICF which is already the top frame, and we'll end up in an infinite loop). Therefore, for the R2R case, we need to pop the ICF from the chain during exception unwinding.

Nullable: Contract.cs and more interop (#24193)

* Nullable: Contract.cs and more interop

* Address PR feedback

Postponing the managed callback so that they are called after the lock acquistion

Switch to workstation GC in case of constrained CPU resources (#24194)

* Switch to workstation GC in case of constrained CPU resources

Right now, if the user sets the configuration so that the server GC is
used, the server GC will be loaded even in conditions where we know the
workstation GC would fare better. An example of such conditions is
constrained environment where there is only 1 or less CPU or with very
low memory.

This can be harmful if users deploy the same projects on different kind
of platforms: deploying to a 20+ cores server and to Azure Functions
will require largely different configurations for the runtime.

There are already multiple ways for the user to specify to use the
server GC or not:
 - setting `COMPlus_gcServer` as an environment variable
 - setting `gcServer` in the configuration file
 - setting `System.GC.Server` passed to `coreclr_initialize`

Fix https://github.com/dotnet/coreclr/issues/23949

* Address review

* Address review

Remove GCToOSInterface::GetCurrentProcessCpuLimit in favor of
GCToOSInterface::GetCurrentProcessCpuCount because the CpuLimit is taken
into account in the CpuCount again.

* Address review

Do the work in src/vm/ceemain.cpp otherwise there will be a disparity
between what the VM and the GC are running. Before, only the GC would be
aware of the switch from server to workstation GC, but not the VM.

Added optional/default parameters for StreamWriter/StreamReader (#24056)

* Added optional/default parameters for StreamWriter/StreamReader

* Disabled outdated test

* Changed default encoding to UTF8NoBOM

* Made encoding parameter nullable

Unify WIN32_FIND_DATA (dotnet/corefx#37158)

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

Add Medium GC Profiling Mode & ICorProfilerInfo::GetObjectReferences (#24156)

More multi-appdomain support removal (#23921)

More multi-appdomain support removal
- Remove DomainLocalBlock
- Remove DomainModule
- Remove FindDomain(File/Module/Assembly)
- Remove GetDomainModule
- Remove variants of GetDomain(File/Assembly) that take AppDomain parameter
- Adjust contracts to be less strict
  - GetDomainFile/Assembly no longer need a complex contract as multi-domain support does not exist
- Eliminiate uses of encoded ModuleID

Fix crossgen with large version bubble enabled

This change fixes about 1483 failing Pri-1 tests when they are
crossgen-ed with large version bubble enabled. Only one failing test
remains.

60% of the test cases were failing due to a failure to find generic
interface in an interface map due to the fact that the interface type
was being passed in canonical form.
The remaining 40% were tests using constrained calls. We were missing
the ENCODE_METHOD_SIG_Constrained in the method flags.

Merge pull request #24226 from BruceForstall/Disable24224

Disable mandelbrot-7 against #24224

Merge pull request #24227 from BruceForstall/Disable24173

Disable _il_dbgu_fld under GCStress against #24173

Fixing issue where LTTng probe check disables GC events (#24198)

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

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

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

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

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

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

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

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

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

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

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

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

Disable _il_dbgu_fld under GCStress against #24173

Disable mandelbrot-7 against #24224

Fix ProjectN build breaks (dotnet/corert#7343)

Porting changes from the non-WinRT files.

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

Merge pull request #24222 from BruceForstall/AddCorefxExclusions

Expand corefx exclusions

Expand corefx exclusions

Update IBCMerge and ensure that it runs when in a release build and ibcoptimize is set. (#24205)

* Update IBCMerge and ensure that it runs when in a release build and ibcoptimize is set.

* Add major rollforward for ibcmerge.

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

Update CoreClr to preview6-27624-71 (master)

Merge pull request #24218 from BruceForstall/FixLinuxX64CorefxTesting

Remove extraneous percent character in Linux/x64 corefx testing command

Remove extraneous percent character in Linux/x64 corefx testing command

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

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

Update CoreClr to preview6-27624-71

Match managed DetermineLibraryNameVariations implementation to native DetermineLibNameVariations implementation. (#24203)

* Match managed DetermineLibraryNameVariations implementation to native DetermineLibNameVariations implementation.

* Update AssemblyDependencyResolver.cs

* Use ordinal comparison instead of culture-specific comparison

Delete unnecessary static and update GCSample to VS2019 (#24204)

Merge pull request #24201 from BruceForstall/AllowFineGrainedCorefxTestExclusions

Enable fine-grained and universal exclusions for CoreFX tests

Update CoreClr to preview6-27623-73 (#24200)

Revert "Dummy commit" (#24197)

Enable fine-grained and universal exclusions for CoreFX tests

One problem we've had in coreclr Jenkins when running corefx
tests (in the coreclr outerloop runs -- the ones using the
run-corefx-tests.py harness) is the inability to exclude
failing tests for x86 or x64, and the inability to have
fine-grained exclusions. This has led to problems like
https://github.com/dotnet/coreclr/issues/22442 where
virtually all Windows x86 and x64 corefx runs fail due to
timeouts. And issues like https://github.com/dotnet/coreclr/issues/24159
where all corefx test legs fail, and we have no control
over exclusions to get them all passing again quickly.

Now that the corefx RunTests.cmd/sh wrapper scripts
parse named arguments, including a response file that
is passed on to xunit, and also that the corefx used
xunit has support for this response file as well as
fine-grained exclusions (per-method/per-class/per-namespace),
we can take advantage of it.

This change adds a single, global, corefx xunit exclusion
response file, that will be used for all platforms. Since this
run-corefx-tests.py mechanism is not expected to live much longer,
this seems sufficient.

Fix failures caused by ICU regression (#24190)

Fixes https://github.com/dotnet/corefx/issues/37098

.NET Core depends on ICU when running on Linux/OSX. Recently some people raised some failure on the framework stack. After investigation we found a regression in ICU which is the root cause of this failure. The regression is, when calling ICU to get some date patterns/properties, in some cases ICU return error code U_MISSING_RESOURCE_ERROR. Although the framework code written to fallback to some invariant values at that time, but we had some wrong line of code which assumed we never fail and trying to access the returned value without checking. That cause the framework to throw NullReferenceException.

The fix here is to make the framework resilient against such cases and continue to run nicely. I have contact ICU support members and I learned there is similar issue tracked in ICU repo https://unicode-org.atlassian.net/browse/ICU-20558

Merge pull request #23932 from briansull/zero-offset-map

Fixes for tracking struct field sequences

Fix crash after unload with server GC (#24196)

When server GC is on, the DomainAssembly::EnumStaticGCRefs is called to
enumerate GC statics. For collectible assemblies, the GC statics are
stored in managed arrays and so there is no need for a special
enumeration of the GC statics. Attempt to do that was causing a crash
due to the fact that the corresponding DomainLocalModule::m_pGCStatics
is always NULL for collectible assemblies and we were trying to access
it.

Nullable : System.Diagnostics.Eventing (Directory) (#24192)

*  Eventing directory done

* adding comment and fixing unix build

Add test coverage for static members on interfaces (#23928)

This has been supported since v1 but we don't have much test coverage.

C# is going to enable interfaces to have static members and static constructors so we better start testing it.

Merge pull request #24195 from Anipik/temp

Dummy commit

Dummy commit

Added additional comment explaining ChnageOper(GT_LCL_FLD) and NotAField

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

- Microsoft.NETCore.Platforms - 3.0.0-preview6.19222.12
- Microsoft.Private.CoreFx.NETCoreApp - 4.6.0-preview6.19222.12

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

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

Update CoreClr to preview6-27623-71 (#24152)

Fix handling of negative numbers in ThreadPool.SetMin/MaxThreads (#24163)

* Fix handling of negative numbers in ThreadPool.SetMin/MaxThreads

* Fix disabled test name

Implement APIs for some threading metrics (CoreRT) (dotnet/corert#7066)

* Implement APIs for some threading metrics (CoreRT)

- API review: https://github.com/dotnet/corefx/issues/35500
- May depend on https://github.com/dotnet/coreclr/pull/22754

* Use thread-locals for counting, use finalizer instead of runtime to detect thread exit

* Don't let the count properties throw OOM

* Remove some flushes

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

Merge pull request #24176 from BruceForstall/Fix24175

Disable 2 GC tests under HeapVerify for arm64

Merge pull request #24174 from BruceForstall/Fix24172

Disable _il_relthread-race test under GCStress

Disable 2 GC tests under HeapVerify for arm64

They fail due to timeout with `COMPlus_HeapVerify=1` on arm64.

They were already being disabled for HeapVerify for x86 and arm32.

Fixes #24175

Disable _il_relthread-race test under GCStress

It takes too long, and times out in CI.

Fixes #24172

Merge pull request #24168 from wtgodbe/p6

Update branding in master to preview6

Update branding in master to preview6

Fix `CORE_LIBRARIES` doc

Correct iterator (#24160)

Disable failing IJW tests against #23358

Remove create assembly name (#24154)

* Remove RuntimeAssembly.CreateAssemblyName

Fixes #24135

CreateAssemblyName was not compatible with AssemblyLoadContext isolation.

Assembly.Load(string)
Assembly.LoadWithPartialName(string)
Activator.CreateInstance(...)

* Remove unused AssemblyNameNative::Init arguments

* Temp disable corefx CreateInstanceAssemblyResolve

Fix Arm64 UpperVector save/restore (#24043)

* Fix Arm64 UpperVector save/restore

Change the general handling of end-of-block restores so that we always have a RefPosition on which to allocate the register needed on Arm64.

Fix #23885

Fix AssemblyName cache hash and key (#24138)

* Add ContextualReflection LoadWithPartialName case

* Remove unnecessary MethodImplOptions.NoInlining

* Remove m_assembly warning

* Fix AssemblyName hash function

* AssemblyNative::Load fix stackMark usage

Do not use the stackMark if (ptrLoadContextBinder != NULL)

* Temporarily disable DefaultContextOverrideTPA

Test is failing due to a logic error.
Fix is pending in https://github.com/dotnet/corefx/pull/37071

Fixed link to .NET Core SDK (#24147)

Update BuildTools, CoreClr to preview4-03917-01, preview5-27618-71, respectively (master) (#24060)

* Update BuildTools, CoreClr to preview4-03917-01, preview5-27618-71, respectively

* Fix build errors

* Disable warning with pragma and linked issue

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

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

Update dependencies from https://github.com/dotnet/corefx build 20190419.11 (#24141)

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

Eventpipe Crst ordering fix (#24101)

* Adding PendingTypeLoadEntry to Crst order for EventPipeCrst

* update header file

Tighten asserts (#24124)

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

- Microsoft.NETCore.App - 3.0.0-preview5-27620-01

Merge pull request #24131 from sandreenko/addAReproTest

Add a repro test for #24114.

Fix Timezone whitespace (#24137)

The thread triggering finalizer shutdown events should be in an alertable state. (#24133)

Pulling in CoreFX changes and fixing up Sse2.StoreLow to be Sse2.StoreScalar (#24123)

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

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

* Removing the Sse2.StoreLow(long/ulong*, ...) methods that have been replaced

* Moving the Sse2.StoreLow(long/ulong*, ...) tests to be Sse2.StoreScalar tests

* Fixing some existing usages of Sse2.StoreLow(long/ulong*, ...)

Implement APIs for some threading metrics (CoreCLR) (#24113)

Implement APIs for some threading metrics (CoreCLR)

API review: https://github.com/dotnet/corefx/issues/35500

Add a repro test.

Merge pull request #24126 from sandreenko/fixStmtBug

Fix bug in impGetStructAddr.

Fix `impGetStructAddr`.

Code review feedback

Merge pull request #24127 from sandreenko/fixArmUbuntuCoreFXTesting

Fix Arm32 Ubuntu CoreFX script.

Fixes for Zero Offset field sequence tracking
 - A GT_LCL_VAR may have a zeroOffset field
 - Add an assert to prevent building field sequences with duplicates
 - Fix fgMorphField when we have a zero offset field
Improve fgAddFieldSeqForZeroOffset
 - Add JItDump info
 - Handle GT_LCL_FLD

Changing the sign of an int constant also remove any field sequence information.

Added method header comment for fgAddFieldSeqForZeroOffset

Changed when we call fgAddFieldSeqForZeroOffset to be before the call to fgMorphSmpOp.

Prefer calling fgAddFieldSeqForZeroOffset() to GetZeroOffsetFieldMap()->Set()

Fix Arm32 Ubuntu CoreFX script.

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

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

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

* Disable outdated test

Large Pages on Linux & macOS (#24098)

Exclude failing tests on all platforms/architectures (#24121)

Fix project metadata in test (#24119)

* Fix project metadata in test

Remove commit triggered linux_arm ReadyToRun jobs

We don't have the capacity to run these on every commit

[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).