fix build errors after merge

Commit migrated from https://github.com/dotnet/coreclr/commit/737907cec8b74abe3b9d3dc1c4501b29c8b7f4a4

Nullable: Comparers, Dictionary and Friends (dotnet/coreclr#23971)

* Nullable: Comparers, Dictionary and Friends

* Add object constraint to Dictionary

* Fix warning from new compiler and annotating dictionary

* PR Feedback

Commit migrated from https://github.com/dotnet/coreclr/commit/ad38bf7164ecb3ecad102645e8a8381fa015f565

Nullable for Type class (dotnet/coreclr#23489)

System.Type, System.Reflection nullability

Commit migrated from https://github.com/dotnet/coreclr/commit/56f38ea2a930b18b0c43f717f90496b44a5d9283

Nullable: System.Diagnostics.Tracing (dotnet/coreclr#24070)

* Nullable: System.Diagnostics.Tracing

* apply feedback

* fix one more comment

Commit migrated from https://github.com/dotnet/coreclr/commit/898d95a811428b0adc0c8de0b74797ea445a93fd

Fix CoreLib build breaks

Commit migrated from https://github.com/dotnet/coreclr/commit/bf657dfda9b61fdde81f871305b73730f1128ea7

Add System.Resources.Extensions (dotnet/corefxdotnet/coreclr#36906)

* Add System.Resources.Binary.Reader|Writer

* Fix ResourceWriter tests

* Test fixes and PR feedback

* More test fixes

* Add packages for System.Resources.Binary.*

* Suppress duplicate types in System.Resources.Binary.*

* Test refactoring and adding RuntimeResourceSet

It turns out me must have our own ResourceSet since the CoreLib resource set doesn't
expose a constructor that takes an IResourceReader.

I've shared the code since it does a bit of non-trivial caching.

* Don't use auto-property initializers for platfrom sensitive test data

For some reason I thought these lazy-initialized the properties but they don't.

As a result we were hitting the platform sensitive code even when we never
called the getter.  Switch to an expression instead.

* Only use Drawing converters on Windows

* Fix test failures

* Don't leak System.Private.CoreLib into resources

* Make sure RuntimeResourceSet doesn't call ResourceReader(IResourceReader)

* WIP

* Rename types in System.Resources.Extensions

Leave RuntimeResourceSet internal as it doesn't need to be public.

* Update packages

* Respond to API review feedback

Remove abstraction for ResourceReader/Writer: just reuse the source.

Remove non-essential members.

* Clean up

* Further cleanup

* Further cleanup

* Review feedback

* Ensure we have stable type names in Resources.Extensions

We don't want to use the runtime type identity when doing type checks or writing
types as this may change.  Instead, check-in hard-coded strings that match the
type identity.

Add a test case to ensure the resources we generate match what we
test the reader against in the resource manager case (also to track any
change to the binary format).

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
Commit migrated from https://github.com/dotnet/coreclr/commit/ae6c7a9a7ddfa43f30450fa4085d952a1aaf3723

Switch to non-Int Azure pool providers

Commit migrated from https://github.com/dotnet/coreclr/commit/a011bafd661919f67eff1532563ce95f2f57ce19

Typos (dotnet/coreclr#24280)

* thier -> their

* exeption -> exception

* Estbalisher -> Establisher

* neeed -> need

* neeed -> need

* neeeded -> needed

* neeeded -> needed

* facilitiate -> facilitate

* extremly -> extremely

* extry -> extra

Commit migrated from https://github.com/dotnet/coreclr/commit/2832f54a6602cd4c0dff4fa65163345ab3ad953c

Improve LOH heap balancing (dotnet/coreclr#24081)

* Improve LOH heap balancing

Previously in `balance_heaps_loh`, we would default to `org_hp` being
`acontext->get_alloc_heap()`.

Since `alloc_large_object` is an instance method, that ultimately came
from the heap instance this was called on. In `GCHeap::Alloc` that came
from `acontext->get_alloc_heap()` (this is a different acontext). That
variable is set when we allocate a small object. So the heap we were
allocating large objects on was affected by the heap we were allocating
small objects on. This isn't necessary as small object heap and large
object heaps have separate areas. In scenarios with limited memory, we
can unnecessarily run out of memory by refusing to move away from that
hea. However, we do want to ensure that the large object heap accessed
is not on a different numa node than the small object heap.

I experimented with adding a `get_loh_alloc_heap()` to acontext similar
to the SOH alloc heap, but performance tests showed that it was usually
better to just start from the home heap. The chosen policy was:

* Start searching from the home heap -- this is the one corresponding to
our processor.

* Have a low (but non-zero) preference for that heap (dd_min_size(dd) /
2), as long as we stay within the same numa node.

* Have a higher cost of switching to a different numa node. However,
this is still much less than before; it was dd_min_size(dd) * 4, now
dd_min_size(dd) * 3 / 2.

This showed big performance improvements (over 30% less time) in a
scenario with lots of LOH allocation where there were fewer allocating
threads than GC heaps. The changes were more pronounced the more we
allocated large objects vs small objects. There was usually slight
improvement (1-2%) when there were 48 constantly allocating threads and
48 heaps. The one place we did see a slight regression was in an 800MB
container and 4 allocating threads on a 48 processor machine; however,
similar tests with less memory or more threads were prone to running out
of memory or running very slow on the master branch, so we've improved
stability. Previously the gc could get lucky by having the SOH choice
happen to be a good choice for LOH, but we shouldn't be relying on it as
it failed in some container scenarios.

One more change is in joined_generation_to_condemn: If there is a memory
limit and we are about to OOM, we should always do a compacting GC. This
helps avoid the OOM and feeds into the next change.

This PR also adds a *second* balance_heaps_loh function for when there
is a memory limit and we previously failed to allocate into the chosen
heap. `balance_heaps_loh` works based on allocation budgets, whereas
`balance_heaps_loh_hard_limit_retry` works on the actual space available
at the end of the segment. Thanks to the change to
joined_generation_to_condemn the heaps should be compact, so not looking
at free space here.

* Fix uninitialized variable

* In a container, use space available instead of budget

* Fix duplicate semicolon

Commit migrated from https://github.com/dotnet/coreclr/commit/141926d90c54bb358cfe8d9eb641c88e94639a8c

Merge pull request dotnet/coreclr#24270 from janvorli/fix-gc-numa-to-heap-map-creation

Fix creation of the NUMA node to heap number map

Commit migrated from https://github.com/dotnet/coreclr/commit/4452efd309d40d4bc7fc1fa48bf1b6e615ee6755

Merge pull request dotnet/coreclr#24281 from BruceForstall/CleanCorefxRuns

Disable failing corefx tests

Commit migrated from https://github.com/dotnet/coreclr/commit/08bad50dc319c74f8f6efdd59565f3b2c3c39e12

Disable failing corefx tests

Commit migrated from https://github.com/dotnet/coreclr/commit/77c0e27c0e5d56aeb4e6c1e06f60477f61395011

Merge pull request dotnet/coreclr#24276 from briansull/revert-24229

Revert 24229

Commit migrated from https://github.com/dotnet/coreclr/commit/72e450b5d4eb6cc8fd9e99e0202b538b795350e8

Revert "Fix for IBC profile data on Linux - Build the full command line for FEATURE_PAL"

This reverts commit dotnet/coreclr@08422af33fa2026953c2e720cecf000ed4345a32.

Commit migrated from https://github.com/dotnet/coreclr/commit/1a30173f2fb4e6cf427dfbd962ab4f80a36583a3

Revert "Added method ReleaseManagedCommandLine to release memory"

This reverts commit dotnet/coreclr@1f2435763926f6068fc88a36a2b8dc49c263f271.

Commit migrated from https://github.com/dotnet/coreclr/commit/78bfc4fbd634f34870e4c11f280c36b7ed696c9f

Undo a change where we ignore the specified EventPipe sessionType.

Commit migrated from https://github.com/dotnet/coreclr/commit/c3e404425f9f03088f0e7106914190634baef619

Refactored the duplicated code into a template method pattern

Commit migrated from https://github.com/dotnet/coreclr/commit/3a559ad43022df53c0f1426c828a6c21f79dd5dd

Add config value for jit host slab cache maximum size (dotnet/coreclr#24254)

Commit migrated from https://github.com/dotnet/coreclr/commit/f3a932ad6ebcb1135562843a53476e373f7b5de3

Fix creation of the NUMA node to heap number map

The current implementation assumes that the NUMA nodes of CPUs
used for GC threads form a zero based continous range. However that
doesn't have to be true for cases when user selects only a subset of the
available CPUs for the GC heap threads using the
COMPlus_GCHeapAffinitizeMask or COMPlus_GCHeapAffinitizeRanges. The
selected CPUs may belong only to a subset of NUMA nodes that don't
necessarily start at node 0 or form a continuous range.

This change fixes the algorithm that initializes the
numa_node_to_heap_map lookup array so that it works correctly even in
such cases.

Commit migrated from https://github.com/dotnet/coreclr/commit/9733c30b27ca90d3eeb07928df6e6e5861c203bb

Merge pull request dotnet/coreclr#24229 from briansull/ibc-linux

Fixes for IBC profile data on Linux

Commit migrated from https://github.com/dotnet/coreclr/commit/03e79408df6b14d36467cfc7eb30bd699f8d3849

Helix telemetry fixes

- Enable telemetry unconditionally
- Set the `_BuildConfig` variable in `xplat-job.yml` so it is available to
  both product and test builds.
- Set the Helix `Type` parameter explicitly to distinguish product and test builds
- Set the Helix `Source` parameter for builds as well as test jobs

This includes a fix to job.yml from dotnet/arcadedotnet/coreclr#2620.

Commit migrated from https://github.com/dotnet/coreclr/commit/9957d039a2ac69bf42fac51b45abae48c1c212d0

[System.Globalization.Native] Fix small issues in CloneCollatorWithOptions and GetCollatorFromSortHandle (dotnet/coreclr#24100)

* Fix allocation size calculation when resizing array
Allocate initial array using malloc to avoid needlessly zeroing it
Reinstate optimization for empty array returned from GetCustomRules lost in dotnet/coreclr#22378

* Avoid resizing arrays in GetCustomRules by preallocating the maximum size we can consume (5648 bytes)

* Avoid creating a binary search tree for something that could be easily stored as 32-entry lookup table

* Remove obsolete comment

Commit migrated from https://github.com/dotnet/coreclr/commit/a59ead8f8b31f836ea551807eef77b77281848e2

Null-terminate buffers that strncpy writes to (dotnet/coreclr#24095)

It is possible that the string being copied is so large that strncpy
fills up the destination array and does not write any null characters to
it. That will lead to buffer overflows. Work around that by always
writing a null character at the end of the destination array.

Commit migrated from https://github.com/dotnet/coreclr/commit/dea615bfbeff6b8769e6d162aea33862e676fadb

Remove redundant tests (dotnet/coreclr#24255)

Tests are technically flaky and have been determined largely redundant with CoreFx.

Fixes dotnet/coreclr#24212

Commit migrated from https://github.com/dotnet/coreclr/commit/37957f427e324448152fb6e75795b876967cfe55

Added method ReleaseManagedCommandLine to release memory

Commit migrated from https://github.com/dotnet/coreclr/commit/1f2435763926f6068fc88a36a2b8dc49c263f271

Add IsFrozenObject Profiler API (dotnet/coreclr#24239)

Commit migrated from https://github.com/dotnet/coreclr/commit/14458204d17a47eff4323f1f8102eb222b157637

Removing EventPipe file polling (EventPipeController+Timer) (dotnet/coreclr#24225)

* Remove file polling only, and leave the COMPlus_* functionality.
* Fix bug/typo introduced with https://github.com/dotnet/coreclr/pull/21718

Commit migrated from https://github.com/dotnet/coreclr/commit/e30263455e930b63c0f311cf28d4f73a8abb504c

Fix for IBC profile data on Linux - Build the full command line for FEATURE_PAL

Added public static function GetManagedCommandLine() andSaveManagedCommandLine()
Add helper method Append_Next_Item
Moved GetManagedCommandLine and SaveManagedCommandLine to ceeload to link crossgen
Cleanup, codereview feedback

Commit migrated from https://github.com/dotnet/coreclr/commit/08422af33fa2026953c2e720cecf000ed4345a32

Merge pull request dotnet/coreclr#24242 from janvorli/fix-numa-node-for-disabled-numa

Fix NUMA node for heap when NUMA is not available

Commit migrated from https://github.com/dotnet/coreclr/commit/58a26dfea967eee422cffe487ffff4c1a2546da6

Using SList<SListElem<T>> to store the EventPipeProviderCallbackData

Commit migrated from https://github.com/dotnet/coreclr/commit/4449737e3e25d57f3fb1e4bb7aa8f16110a27538

Add IBCMerge restore source to the restore command. (dotnet/coreclr#24250)

Commit migrated from https://github.com/dotnet/coreclr/commit/bb24f926d7b0c001b75d4fcb4d64c2d7572c1d38

Nullable System.Collections.EmptyReadOnlyDictionaryInternal, System.Collections.ObjectModel.* (dotnet/coreclr#24221)

* annotating collections

* annotated collection.cs and readonlycollection.cs

* adding tkey constraint and comments

Commit migrated from https://github.com/dotnet/coreclr/commit/ebb717431b79c6d6c83a32983b5819a13eb62cef

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

Fix crossgen with large version bubble enabled

Commit migrated from https://github.com/dotnet/coreclr/commit/b5bb75ec39a501bde4089ba754bb717e1d52fe0d

Improve the performance of the type loader (dotnet/coreclr#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

Commit migrated from https://github.com/dotnet/coreclr/commit/713091f4e504b9a194dcb2cfda180baa21572c58

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

Commit migrated from https://github.com/dotnet/coreclr/commit/e3d7d212a4d5350227c1fcc6949e9a379e9162c5

Return null for unexpected HW intrinsic (dotnet/coreclr#24219)

Fix dotnet/coreclr#17815

Commit migrated from https://github.com/dotnet/coreclr/commit/9cd5658f969f503127e2a40991989976c72f866d

Fixing 23941 (dotnet/coreclr#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.

Commit migrated from https://github.com/dotnet/coreclr/commit/e966022128c67163e04be4ab2360d314d7d9c062

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

* Nullable: Contract.cs and more interop

* Address PR feedback

Commit migrated from https://github.com/dotnet/coreclr/commit/4dc324887a85bc90ef6830b2d27ba49c31058d72

Fix NUMA node for heap when NUMA is not available

The recent refactoring of the GCToOSInterface::GetProcessorForHeap has
accidentally changed the NUMA node returned in case NUMA is disabled
(either via the COMPlus_GCNumaAware or due to the fact that there is
just a single NUMA node on the system) and the CPU groups are disabled.
Before that refactoring, the code was incorrectly returning 0 as the
NUMA node when CPU groups were disabled no matter whether NUMA was
enabled or disabled. The refactoring fixed that by returning the
current CPU group number for the case when NUMA was enabled, however
it still returned incorrect value, this time GroupProcNo::NoGroup as
the NUMA node number in case NUMA was disabled.

This change fixes it by returning the current group number in this case.

Commit migrated from https://github.com/dotnet/coreclr/commit/8d93bd9080f47efd87dc5420a6e600f8dc432659

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

Commit migrated from https://github.com/dotnet/coreclr/commit/e6034d903f2608445a3f66e3694f461fad7b8b88

Switch to workstation GC in case of constrained CPU resources (dotnet/coreclr#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.

Commit migrated from https://github.com/dotnet/coreclr/commit/a9b6851ad1a9551f3f654b654b094147b50e5999

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

* Added optional/default parameters for StreamWriter/StreamReader

* Disabled outdated test

* Changed default encoding to UTF8NoBOM

* Made encoding parameter nullable

Commit migrated from https://github.com/dotnet/coreclr/commit/57a40b91ce8d340f60717b73ab5d779e9f6a53be

Unify WIN32_FIND_DATA (dotnet/corefxdotnet/coreclr#37158)

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
Commit migrated from https://github.com/dotnet/coreclr/commit/6d0ea2a67116907dc7e3e9213d3270944142e7f5

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

Commit migrated from https://github.com/dotnet/coreclr/commit/c2dacd2025145c586a5569408c1427899a3767cc

More multi-appdomain support removal (dotnet/coreclr#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

Commit migrated from https://github.com/dotnet/coreclr/commit/4a2654124ecf38300a717db79926334703abeb70

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.

Commit migrated from https://github.com/dotnet/coreclr/commit/f0ef156828d607c8eb79e9dc104a10de6b5ece9a

Merge pull request dotnet/coreclr#24226 from BruceForstall/Disable24224

Disable mandelbrot-7 against dotnet/coreclr#24224

Commit migrated from https://github.com/dotnet/coreclr/commit/e9691546d308fde4e56cc821f8c9bfda7e9a8b91

Merge pull request dotnet/coreclr#24227 from BruceForstall/Disable24173

Disable _il_dbgu_fld under GCStress against dotnet/coreclr#24173

Commit migrated from https://github.com/dotnet/coreclr/commit/efad71804278957d556930ce978dec1ce285911d

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

Commit migrated from https://github.com/dotnet/coreclr/commit/8c979b75e59815bcc1c7a9a1acdfda4e143f1a66

Disable _il_dbgu_fld under GCStress against dotnet/coreclr#24173

Commit migrated from https://github.com/dotnet/coreclr/commit/34b35a56f24d20e9aae076a67bf8481b6cb631dd

Disable mandelbrot-7 against dotnet/coreclr#24224

Commit migrated from https://github.com/dotnet/coreclr/commit/df0c12d5ec4f7832245dd5a6cbd5a9bc5f021183

Fix ProjectN build breaks (dotnet/corertdotnet/coreclr#7343)

Porting changes from the non-WinRT files.

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
Commit migrated from https://github.com/dotnet/coreclr/commit/8eb913402071a6c6c9bd56768d0fd6f68f41d745

Merge pull request dotnet/coreclr#24222 from BruceForstall/AddCorefxExclusions

Expand corefx exclusions

Commit migrated from https://github.com/dotnet/coreclr/commit/377be0e800b3a0e1ebd855de779f3af2e00d5360

Expand corefx exclusions

Commit migrated from https://github.com/dotnet/coreclr/commit/7b7eaccdab4aeb756a873f76ff76e99e8318050b

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

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

* Add major rollforward for ibcmerge.

Commit migrated from https://github.com/dotnet/coreclr/commit/78f21fa4e3bec511249b230f52b24d1b9bff5285

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

Update CoreClr to preview6-27624-71 (master)

Commit migrated from https://github.com/dotnet/coreclr/commit/8705f11f78377ad926479087a9589a7277957239

Merge pull request dotnet/coreclr#24218 from BruceForstall/FixLinuxX64CorefxTesting

Remove extraneous percent character in Linux/x64 corefx testing command

Commit migrated from https://github.com/dotnet/coreclr/commit/edfd95518ace718388d4ef2396b4233635b2d26b

Remove extraneous percent character in Linux/x64 corefx testing command

Commit migrated from https://github.com/dotnet/coreclr/commit/f3db0557c898c346c3f48da825fc2c8c522e5063

Update CoreClr to preview6-27624-71

Commit migrated from https://github.com/dotnet/coreclr/commit/cc7d630fb782f0436946fd68a16018684c8c892a

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

* Match managed DetermineLibraryNameVariations implementation to native DetermineLibNameVariations implementation.

* Update AssemblyDependencyResolver.cs

* Use ordinal comparison instead of culture-specific comparison

Commit migrated from https://github.com/dotnet/coreclr/commit/8808f4eab10d1760182590cad4964f566b9c88ca

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

Commit migrated from https://github.com/dotnet/coreclr/commit/3f21a0f6cc8e0036ad536c791d7877a35d22cd01

Merge pull request dotnet/coreclr#24201 from BruceForstall/AllowFineGrainedCorefxTestExclusions

Enable fine-grained and universal exclusions for CoreFX tests

Commit migrated from https://github.com/dotnet/coreclr/commit/22c467297d0d537c2e647ec5c5b4256ab42b18bb

Update CoreClr to preview6-27623-73 (dotnet/coreclr#24200)

Commit migrated from https://github.com/dotnet/coreclr/commit/2167c422f96027dc826fa9eb7006babb122923c8

Revert "Dummy commit" (dotnet/coreclr#24197)

Commit migrated from https://github.com/dotnet/coreclr/commit/14af2765d0a4837bd017b9716e6405a3b778bb78

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.

Commit migrated from https://github.com/dotnet/coreclr/commit/612a568e9177c0f126a6fc211a54c70d67bf8773

Fix failures caused by ICU regression (dotnet/coreclr#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

Commit migrated from https://github.com/dotnet/coreclr/commit/cbc358d9617816d38109df5a40fb677ce69c840f

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

Fixes for tracking struct field sequences

Commit migrated from https://github.com/dotnet/coreclr/commit/e7ecfecc3b8a35e645e9f2a5b5ebb10889ca956b

Fix crash after unload with server GC (dotnet/coreclr#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.

Commit migrated from https://github.com/dotnet/coreclr/commit/f0a244953fae71378dc707289773edc509288fcc

Nullable : System.Diagnostics.Eventing (Directory) (dotnet/coreclr#24192)

*  Eventing directory done

* adding comment and fixing unix build

Commit migrated from https://github.com/dotnet/coreclr/commit/8b35a2bbfbe79713505bd96b80476bafb3c40bdd

Add test coverage for static members on interfaces (dotnet/coreclr#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.

Commit migrated from https://github.com/dotnet/coreclr/commit/d5f569771f8dfda895867f5a1309afbe384ebf51

Merge pull request dotnet/coreclr#24195 from Anipik/temp

Dummy commit

Commit migrated from https://github.com/dotnet/coreclr/commit/6c09b4c6f39727abcf2780f1ae4334ca9352bef4

Dummy commit

Commit migrated from https://github.com/dotnet/coreclr/commit/67d1bfbc149db36896de845855cfe5893d75baab

Added additional comment explaining ChnageOper(GT_LCL_FLD) and NotAField

Commit migrated from https://github.com/dotnet/coreclr/commit/dbf2f74ded4310025d0ce6d74d137ed2bc64727e

Update CoreClr to preview6-27623-71 (dotnet/coreclr#24152)

Commit migrated from https://github.com/dotnet/coreclr/commit/6cde974215e97f95178a1f20886e8a5734b9d388

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

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

* Fix disabled test name

Commit migrated from https://github.com/dotnet/coreclr/commit/efe39f35933d89bb8f82472333810fddb78480b2

Implement APIs for some threading metrics (CoreRT) (dotnet/corertdotnet/coreclr#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>
Commit migrated from https://github.com/dotnet/coreclr/commit/447b6554f393f3fa02f3d697cdf7c56612a0fa1a

Merge pull request dotnet/coreclr#24176 from BruceForstall/Fix24175

Disable 2 GC tests under HeapVerify for arm64

Commit migrated from https://github.com/dotnet/coreclr/commit/c0cae0a041549edd759366eb3e732f8348b466b3

Merge pull request dotnet/coreclr#24174 from BruceForstall/Fix24172

Disable _il_relthread-race test under GCStress

Commit migrated from https://github.com/dotnet/coreclr/commit/20acddfbcb6b9437667a88b8d433d9a2cb3a34e3

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 dotnet/coreclr#24175

Commit migrated from https://github.com/dotnet/coreclr/commit/f024265c2871b70491d5524f1cdfe7805ab48e47

Disable _il_relthread-race test under GCStress

It takes too long, and times out in CI.

Fixes dotnet/coreclr#24172

Commit migrated from https://github.com/dotnet/coreclr/commit/7b7e33d5e3dbc2d199a2e4b35024df09233c9673

Merge pull request dotnet/coreclr#24168 from wtgodbe/p6

Update branding in master to preview6

Commit migrated from https://github.com/dotnet/coreclr/commit/ca96ed41a6bc5d00d220380dc74e0c7e8535fa92

Update branding in master to preview6

Commit migrated from https://github.com/dotnet/coreclr/commit/38ce75a17520789b4bf6ca1be4419fa7370d36b7

Fix `CORE_LIBRARIES` doc

Commit migrated from https://github.com/dotnet/coreclr/commit/1ae88b330ae55514fd6123a0fb9340762ab9bbb4

Correct iterator (dotnet/coreclr#24160)

Commit migrated from https://github.com/dotnet/coreclr/commit/9401fa655260453168026985d3d40d3e567ef94c

Disable failing IJW tests against dotnet/coreclr#23358

Commit migrated from https://github.com/dotnet/coreclr/commit/fc6dd7009fb953d43d0912ad0755d5566868047a

Remove create assembly name (dotnet/coreclr#24154)

* Remove RuntimeAssembly.CreateAssemblyName

Fixes dotnet/coreclr#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

Commit migrated from https://github.com/dotnet/coreclr/commit/479d7edafe8035c5e8301c8ab098835614e401e7

Fix Arm64 UpperVector save/restore (dotnet/coreclr#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 dotnet/coreclr#23885

Commit migrated from https://github.com/dotnet/coreclr/commit/a8d5756796b57d2347e31deb405db4cf1a7e055b

Fix AssemblyName cache hash and key (dotnet/coreclr#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

Commit migrated from https://github.com/dotnet/coreclr/commit/a36bc61442d89d0b5c58b0b14e7bd3bde218f24d

Fixed link to .NET Core SDK (dotnet/coreclr#24147)

Commit migrated from https://github.com/dotnet/coreclr/commit/c48969221da787abaa0c6f1477ec9c864f0455d2

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

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

* Fix build errors

* Disable warning with pragma and linked issue

Commit migrated from https://github.com/dotnet/coreclr/commit/472d840077d05336c812657353de5334e8fdccf0

Eventpipe Crst ordering fix (dotnet/coreclr#24101)

* Adding PendingTypeLoadEntry to Crst order for EventPipeCrst

* update header file

Commit migrated from https://github.com/dotnet/coreclr/commit/9a7be98147755d337c3c5c1d8972afe32adaf797

Tighten asserts (dotnet/coreclr#24124)

Commit migrated from https://github.com/dotnet/coreclr/commit/b2693453717f43bf90c47b8a87c60d92203551b7

Merge pull request dotnet/coreclr#24131 from sandreenko/addAReproTest

Add a repro test for dotnet/coreclr#24114.

Commit migrated from https://github.com/dotnet/coreclr/commit/69436a80895770a5dde4d52f05b9a72596ce2af5

Fix Timezone whitespace (dotnet/coreclr#24137)

Commit migrated from https://github.com/dotnet/coreclr/commit/bf0600c9ce4b9de06d80b7a02b33eb174b3dd64b

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

Commit migrated from https://github.com/dotnet/coreclr/commit/324344f82a6f8221d28f9015f64be06779632468

Pulling in CoreFX changes and fixing up Sse2.StoreLow to be Sse2.StoreScalar (dotnet/coreclr#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*, ...)

Commit migrated from https://github.com/dotnet/coreclr/commit/0f93182d5d8eeefe18ec50b1d4d32fd5d05333eb

Implement APIs for some threading metrics (CoreCLR) (dotnet/coreclr#24113)

Implement APIs for some threading metrics (CoreCLR)

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

Commit migrated from https://github.com/dotnet/coreclr/commit/0410c3e4fe54590eea34aead28f550539d814b98

Add a repro test.

Commit migrated from https://github.com/dotnet/coreclr/commit/83d07828c04758a3332f611cf54b9b9b5c371fb2

Merge pull request dotnet/coreclr#24126 from sandreenko/fixStmtBug

Fix bug in impGetStructAddr.

Commit migrated from https://github.com/dotnet/coreclr/commit/924d4fcc6961a76881a42a5a4a80479352708ace

Fix `impGetStructAddr`.

Commit migrated from https://github.com/dotnet/coreclr/commit/b8916d5c8be8bd9701fa36b875f10721bde8ce16