Enable cross OS DBI build (#35021)

* Enable cross OS DBI build

* Fix .gitignore

* Fix Cross OS DBI compilation issues

* Review feedback

* Cleanup dummy/twowaypipe.cpp

Add wasm for browser RID (#34940)

Disable IPGlobalProperties_TcpActiveConnections_Succeed test on Mono/Windows (#35113)

Fixes #35111

Update VS minimum dependency (#35114)

With the net5.0 TFM change and the nuget static graph restore, we require VS >= 16.6 Preview2.

Adds support for generating Mono named NETCore.App Runtime Packs (#34980)

Starting with osx, we will produce a uniquely named runtime pack with Mono.  The intent is to give the sdk an easier way to identify the mono runtime.

We will now publish Microsoft.NETCore.App.Mono.Runtime.<rid>.nupg

Throw DirectoryNotFoundException on Image.Save with bad directory (#34998)

* Provide better exception when saving to non-existent directory

When saving an image to a non-existent directory, throw a DirectoryNotFoundException
vs a System.Runtime.InteropServices.ExternalException.

Fix #31367

* Change exception message to be more intuitive

Use: The directory {0} of the filename {1} does not exist.
instead of: The target directory {0} of the targetPath {1} does not exist.

Fix #31367

* Fix test on unix by using built path.

We were using a hardcoded path with backslashes. That didn't match the built path for the Assert.

* Make Test conditional as it requires GDI+

* Put directory check closer to where it's used to ensure that we
capture it from all Save variations.

* Remove trailing whitespace

* Update test to ensure it doesn't run on the .NET Framework

* Change CheckIfDirectoryExists to ThrowIfDirectoryDoesntExist
to match naming conventions for throw-style methods.

Fix #31367

* Fixes typo in test name

Fix #31367

Fix assert while logging typeload failures (#34985)

* Fix assert while logging typeload failures

* Update src/coreclr/src/vm/clsload.cpp

keeping getTypeToken for `ELEMENT_TYPE_CLASS`

Co-Authored-By: Simon Nattress <nattress@gmail.com>
Co-authored-by: Simon Nattress <nattress@gmail.com>

Set fetch depth to 5 for stress tests on NanoServer (#35109)

Increases git fetch depth for Windows NanoServer to avoid checkout failures

Fixes #33428

Move and lint ECMA-335 Augments spec  (#35095)

* Move specs and rename Ecma-335-Issues.md

* Fix typos

* MD009/no-trailing-spaces

* MD012/no-multiple-blanks

* MD007/ul-indent

* MD031/blanks-around-fences

* MD032/blanks-around-lists

* MD040/fenced-code-language

* MD030/list-marker-space

* Simplify markdown code span syntax

* Update title and intro

* Fix broken markdown code span

* Link addendum doc from dotnet-standards.md

Remove IReadyToRunMethodCodeNode interface (#35085)

`IReadyToRunMethodCodeNode` was useful in the CoreRT repo where we had multiple kinds of method code nodes and needed a common way of setting related method data from the JIT. It's not useful in crossgen2 since we only have `MethodWithGCInfo` representing code nodes.

Minor changes - Update specific words in mono code base to be PoliCheck/TSA compliant (#35053)

* Updating some comments to align with PoliCheck rules

* one additional change

* Update src/mono/mono/mini/mini-wasm-debugger.c based on pr suggestion

Co-Authored-By: Stephen Toub <stoub@microsoft.com>
* Update w32process-unix.c based on PR suggestion

Co-authored-by: Stephen Toub <stoub@microsoft.com>

updated output folder from 'aforementioned' to the folder where the single library output is found. (#34340)

fix runntinf tests with targetFramework property and -f flag (#35016)

Add IR support for multireg lclVars & intrinsics (#34822)

* Add IR support for multireg lclVars & intrinsics

* PR Feedback

[mono] iOS Test runner (#34976)

Update Microsoft.NET.HostModel to reference System.Text.Json 5.0 (#35024)

Microsoft.Extensions.DependencyModel and Microsoft.NET.HostModel are both used on .NET Framework's MSBuild by dotnet/sdk's Microsoft.NET.Build.Tasks assembly. However, they are referencing differnet versions of System.Text.Json, which is causing file load errors on .NET Framework.

Fixing this by updating Microsoft.NET.HostModel to use a 5.0 version.

Fix #35006

Resetting _waitForCompletion to null was removed from DuplexContent (#35010)

`_waitForCompletion` gets reset to null in `DuplexContent.Complete` and `Fail` which leads to race condition with `SerializeToStreamAsync`. This reset seems redundant since `_waitForCompletion` is always unconditionally initialized on each call to `SerializeToStreamAsync`. This PR removes that reset logic.

Fixes #33141

Merge pull request #35058 from janvorli/fix-vsd-stub-nullref-handling

Fix VSD stub null reference handling on amd64

[aot] Fix the support for separate aot data files. Reorder the MONO_AOT_TABLE constants so they match the ordering in MonoAotFileInfo. (#35064)

Co-authored-by: vargaz <vargaz@users.noreply.github.com>

Fix an incorrect CSE case with struct retyping. (#34676)

* Fix old printing issues.

* add a repro test that doesn't require crossgen2.

* Check that all `ADDR` sources are marked as DONT_CSE.

* Fix text(no size) diffs found by SPMI.

* Add a question/comment.

* Update src/coreclr/src/jit/compiler.hpp

Co-Authored-By: Eugene Rozenfeld <erozen@microsoft.com>
* Delete an optimization that was not proved to be correct.

* Delete useless code.

* Revert "Fix text(no size) diffs found by SPMI."

This reverts commit cd24383210b9ed22f7ea3640005217021f9fbd63.

Fix for PInvoke behavior in composite mode (#35027)

Thanks to Fadi's help I finally identified the root problem with
Crossgen2 PInvokes in composite mode: we weren't setting the
READYTORUN_FLAG_NONSHARED_PINVOKE_STUBS flag on the component
assembly R2R headers and that's what the runtime is looking at.

I believe it's both easier and more general to fix this in the compiler
as it comes for free whereas redirecting the runtime to always look
at the composite header would requite at least an additional
indirection. Also we could theoretically use different settings for
different component assemblies even though my impression from Fadi's
detailed explanation was that we basically always want to use
non-shared PInvokes in Crossgen2.

Thanks

Tomas

fix endless loop in GetActiveUdpListeners (#35078)

Merge pull request #34933 from monojenkins/sync-pr-19535-from-mono

[wasm][debugger] Show signature for delegate, or target, if available…

Fix Http.Json serialization performance by using static options (#35040)

Get the IUnknown identity of COM instance for global ComWrappers (#35032)

JIT: fix recursive inline checking (#35020)

We were not setting `m_Code` in the root inline context, and so were
sometimes allowing one level of recursive inlining.

[jit] Add support for fconv.ovf.*.un opcodes (#34949)

* [jit] Handle fconv_ovf_un opcodes

We handle them the same way as non _un versions, by doing a float to int64 conversion followed by a simple integer conversion to the final size.

* [jit] Remove no longer needed emulation

We no longer need to emulate fconv_ovf_u8_un and rconv_ovf_u8_un since we reuse the mechanism for non _un versions (which uses just the emulation for fconv_ovf_u8 and rconv_ovf_u8)

Partly reverts https://github.com/mono/mono/commit/de5dca22649ae581085863b0935f94eff4483e55.

* Enable some mono tests

* [jit] Properly handle conv.r.un applied to r4

Fixes src/coreclr/tests/src/JIT/Regression/VS-ia64-JIT/V2.0-Beta2/b309576/bug2.il which was silently failing before

PR Feedback

Allow known values for HttpHeaderType.Custom in KnownHeaders (#35003)

Add OSPlatform.Browser and Architecture.Wasm (#34781)

[wasm][debugger] Show signature for delegate, or target, if available…

… (#19505)

* [wasm][debugger] Show signature for delegate, or target, if available

- As object properties, we return a `Target` which has the signature of
the delegate target.

Fixes https://github.com/mono/mono/issues/19382

* [wasm][debugger] Some tidying up

* [wasm][debugger] Remove unused `sig_desc`

* [wasm][debugger] Simplify code, avoid extra allocations

.. as suggested by @lewing

(cherry picked from commit 4eca12e794764da1a0646e77af5b8c60a2e2767a)

Remove explicit null check for VSD from JIT for 64 bit targets

Fix missing semicolon

Make the -Wsometimes-uninitialized into an error. (#34965)

Fix swapped Unix and Windows instruction codes

[wasm][debugger] Fix typo and silence output (#35041)

Co-authored-by: lewing <lewing@users.noreply.github.com>

[debugger] Remove assert when the DebuggerTypeProxy cannot be instantiated (#34966)

Remove the assert because we don't need to stop the debug if we cannot create the object when calling the constructor using invoke_method, if it's a DebuggerTypeProxy, we are not able to show the value of the DebuggerTypeProxy, but we still can show the Raw View, if it's not the case of DebuggerTypeProxy, we just cannot evaluate this expression, but we can continue the debug session.

Fix mono/mono#19492

Co-authored-by: thaystg <thaystg@users.noreply.github.com>

Creating a runtime pack artifacts directory in the libraries build (#34662)

This change populates a runtime pack directory structure in artifcats/bin.  This is so that we can support running libraries tests for iOS and Android.

Add regression test

Remove double lookups for virtual stubs

Make the ajdustment unconditional for all architectures

Fix null reference handling in VSD stubs on amd64

When null "this" pointer is passed to the VSD dispatch and resolve
stubs on amd64, it is not converted to NullReferenceException and it
generates AccessViolationException instead, tearing down the process.
This happens due to the fact that AdjustContextForVirtualStub always
returns FALSE on amd64. It behaves correctly on ARM, x86 and ARM64.
This issue has been present at least since .NET Framework 4.5.

This change fixes it by implementing the AdjustContextForVirtualStub for
amd64 too. I've also found that we were doing this adjustment for ARM
and x86 only at some places while ARM64 needs it too. So I've made it
unconditional for all architectures at all places.

improve TLS perf on macOS (#32338)

* improve TLS perf on macOS

* feedback from review

* feedback from review

* remove unnecesary locking

Co-authored-by: Tomas Weinfurt <furt@Shining.local>

Prevent 'in' folder from being created in output directory (#34979)

Additional nullable annotation on IPAddress (#34698)

* Additional nullable annotation on IPAddress.

* TryParse for System.Net.Http

* TryParse for System.Runtime.Numerics

* CoreLib TryParse nullable annotations

* Update Runtime reference source.

* NotNullWhen for Guid format.

* NotNullWhen for DateTime{Offset} format parameters

* Correct format parameter nullability for TimeSpan.

* Update reference source.

Co-authored-by: Stephen Toub <stoub@microsoft.com>

Pass $(officialBuildIdArg) during build phase of Mono (Closes: #34993) (#35002)

mono.proj: use --build-id=sha1 (#34939)

To mirror what the coreclr/libraries builds are using

Remove unnecessary MemoryStream allocation from ByteArrayContent (#35005)

Constructing a ByteArrayContent (or anything that derives from it, like StringContent) or a ReadOnlyMemoryContent  proactively allocates a MemoryStream that it stores as the _bufferedContent on the base HttpContent.  But the 99.99% use case for ByteArrayContent is as a request payload, and this _bufferedContent isn't used for that.  The only time this would be used is if a developer created a ByteArrayContent and then explicitly got the stream from it, e.g. via GetStreamAsync, in which case we're no worse off by doing the creation lazily.  This PR just deletes the unnecessary code / allocation.

Add IR support for multireg lclVars & intrinsics

Fix some flag uses (#34969)

* Fix some flag uses

Script changes enabling composite R2R build with shared framework (#34661)

Add FreeBSD-specific SONAME versions to portable OpenSSL probing.

The version of OpenSSL installed will depend on the version of FreeBSD and
whether OpenSSL has been updated from the FreeBSD Ports collection.

The order of attempted loading is:
    libssl.so.11 (ports)
    libssl.so.111 (base FreeBSD using OpenSSL 1.1.1)
    libssl.so.8 (base FreeBSD using OpenSSL 1.0.2)

Close timeout increased to 20 seconds (#34999)

CloseOutputAsync_Cancel_Success tests are failing due to 2 reasons:
1. `Assert` expecting the state to be `Aborted`, finds it's `Open` - most frequent
2. Connection gets forcibly terminated on a connect attempt

However, the second failure type is not specific to this test because it affects many tests in different types during the same time. Thus, it seems to be an infra issue.
This PR fixes only the first failure type by increasing Close operation timeout.
Fixes #1725

Simplify KnownHeaders lookup in SocketsHttpHandler (#34974)

* Simplify KnownHeaders lookup

* Update src/libraries/System.Net.Http/src/System/Net/Http/Headers/KnownHeaders.cs

Co-Authored-By: David Shulman <david.shulman@microsoft.com>
Co-authored-by: David Shulman <david.shulman@microsoft.com>

[metadata] Identify when a field is overlapped by a non-object field (#34745)

- Fix class loader to identify when the class has a field overlapped by a non-object field.
- Fix method-to-ir to forward the TypeLoad error.
- Fix transform to forward the TypeLoad error.
- Fix unit test that has a class with a field that is overlapped by a non-object field.

Fixes https://github.com/dotnet/runtime/issues/34388

Co-authored-by: thaystg <thaystg@users.noreply.github.com>

Fix Span length in TryFormatUInt64Slow (#34840)

* Make the Span length right

* Adding a regression test for the span problem

* Avoid dependency on machine locale

* Using customFormat

Adds AssertContentsAgainstJsonNet to avoid unnecessary calls (#34931)

* Adds AssertContentsAgainstJsonNet to avoid unnecessary calls

Fixes #32351

* Remove temporary hack comments

Fix pal_io.c code based on code review suggestions (#34970)

These suggestions were provided by @lpereira on
https://github.com/dotnet/corefx/pull/42900

Fix section reference (#34992)

Add wasm cross compiler to the mono build. (#34935) (#34972)

System.Net.Http ref sources regenerated. (#34961)

Fix missing array ctor(int32,int32) test failures. (#34764)

According to ECMA, VES should add the following methods to arrays:

* A constructor that takes a sequence of int32 arguments, one for each dimension of the array, that specify the number of elements in each dimension beginning with the first dimension. A lower bound of zero is assumed.

* A constructor that takes twice as many int32 arguments as there are dimensions of the array. These arguments occur in pairs—one pair per dimension—with the first argument of each pair specifying the lower bound for that dimension, and the second argument specifying the total number of elements in that dimension.

* Get.

* Set.

* Address.

In Mono's case we only added the `ctor (int32, int32)` for rank 1 arrays if they were "jagged" array. Array with rank 1-4 executed an optimized code path always using the one int32 per rank constructor.

This doesn't work in cases where the array is constructed using both a length and a lower bound pattern, like:

`newobj instance void int32[10000...10005]::.ctor(int32, int32)`

this would trigger a method not found exception, since its not a jagged array, Mono wouldn't add the `ctor(int32, int32)` constructor for this type.

Fix will make sure we follow ECMA's definition of this and fix code to use right instance methods of arrays. Code also adds an assert to trap if a jagged array is created through this code path
(shouldn't happen) since current implementation would have handled it as
a 2 ranked array.

Fix also adds several tests into iltests.il allocating different variations of arrays using lower/upper bounds and validating length, rank, upper/lower bounds of allocated arrays.

Co-authored-by: lateralusX <lateralusX@users.noreply.github.com>

Remove outdated CoreFx.Private.TestUtilities.Unicode references (#34981)

* Remove outdated CoreFx.Private.TestUtilities.Unicode references

* Update System.Globalization.Extensions.Nls.Tests dependencies

* Add System.Runtime.Nls.Tests to System.Runtime.sln

* Add missing .sln entry

Fix PrinterSettings.SupportsColor to use the right PInvoke (#34651)

* Fix PrinterSettings.SupportsColor to use the right PInvoke

* PR Fedback

* Fix test

* PR Feedbak

Fix NullReference on JsonExtensionData (#34569)

* Fix NullReference on JsonExtensionData

Fix NullReferenceException on serialization when JsonExtensionData dictionary has a custom JsonConverter.

Fix #32903

* Add test for converters declared through attribute

* Ensure custom serialization is not used for reading JsonExtensionData

* Cover all the combinations in tests

Sync cgroup v2 in libraries with coreclr (#34665)

This commit brings in two changes from coreclr to libraries:

1. https://github.com/dotnet/runtime/pull/980

   "Fix named cgroup handling in docker"

   This fixes getting cgroup information for named cgroups inside containers.

2. https://github.com/dotnet/runtime/pull/34334

   "Add cgroup v2 support to coreclr"

   This is essentially the same change pushed to corefx (now libraries)
   to add cgroupv2 support, but this newer coreclr change has one major
   difference: it determines whether the system is using cgroup v1 or
   cgroup v2 once, and then explicitly uses that (only). This avoids
   issues on systems where both cgroup v1 and v2 are enabled, (but only
   one is being used by default).

HttpClientMiniStressTest: Disable parallelism (#34941)

* Update HttpClientMiniStressTest.cs

Currently, it's possible for 4 different instances of SingleClient_ManyGets_Sync to run in parallel, each using up to 2 x 5000 sockets, totaling in 40,000 sockets. way beyond the original intent of testing 5000 client instances. since this additional stress is random in nature, I believe it is not intentional.
This change make sure that those tests will not run in parallel, this avoids some issues I've encountered in some systems.

* HttpClientMiniStressTest: Disable Parallelism: Used CollectionAttribute on the base class

Update issues.targets to baseline recent Crossgen2 bugs (#34917)

I have also expanded the timeout from 150 to 200 minutes as
my several recent attempts to run the runtime-coreclr crossgen2
pipeline timed out on Windows.

Thanks

Tomas

Call `SetParentAssembly` when binding from native manifest (#34924)

In large version bubble mode, Crossgen2 uses the native manifest table to encode assembly references. When a referenced module is decoded and an `AssemblySpec` is created, set the parent assembly for the bind.

Fixes Loader\binding\tracing\BinderTracingTest.Basic regression, which checks parent assembly is correctly set in the bind result.

Fix for runtime constraint checking for default interface methods (#34889)

* pass in InstantiationContext to check constraints.

* add a test

Improve crossgen HWIntrinsics handling (#34849)

* improve crossgen HWIntrinsics handling

* use fIsPlatformSubArchitecture for all x64 checks

Use ICU on Windows when available (#34645)

* Use ICU when available on Windows

* Fix System.Globalization tests

* Fix System.Globalization.Extensions tests

* Fix System.Runtime tests

* Add System.Globalization.Nls.Tests to test UseNls runtime switch

* Add System.Globalization.Extensions.Nls.Tests to force them to run on NLS

* Add System.Runtime.Nls.Tests.csproj to test Nls behavior

* Fix left over from GlobalizationTestMode

* PR Feedback

* PR Feedback 2

* Fix mono build

* Fix failing tests

* Fix typo

* Fix bad merge in projitems for tvOS

* PR Feedback 3

* PR Feedback 4

Add some more common response headers to KnownHeaders (#34922)

Fix an uninitialized variable error introduced by #34877. (#34951)

Revert "Add wasm cross compiler to the mono build. (#34935)" (#34968)

This reverts commit b134994b40eaecd98051362f076be4e23f904944.

Add JsonIncludeAttribute & support for non-public accessors (#34675)

* Add JsonIncludeAttribute & support for non-public property accessors

* Address review feedback

* Remove extra reflection call for non-public properties

use curl with OpenSSL backend on 16.04-cross based containers (#34909)

* use curl with OpenSSL backend

* update docs

Add wasm cross compiler to the mono build. (#34935)

* Add wasm cross compiler to the mono build.

* Address comments.

* Fix typo

Co-authored-by: Alexander Köplinger <alex.koeplinger@outlook.com>

Single-File: Add a test case with Resource Assemblies (#34736)

Add a test to verify that resolution of deps.json files directly from bundle succeeds when there are resource assemblies.

Better codegen / perf form Math.Max and Math.Min (#33851)

* Forced inline of Math.Min/Max double/float

* Math.Min split in fast and slow

* Math.Min another try

* Added more tests to Math

* Picked variant "MinReorder"

[mono] Add iOS app builder project (#34563)

* Add iosAppBuilderTasks project

* Escape spaces in ProjectName

* Remove add_definitions(-DDEVICE)

* Fix typoe Buillder -> Builder, also build the task as part of mono runtime

* Rewrite HelloiOS sample (to use the task)

* Fix build

* Clean up

* Add license headers

* Clean up

* dont build app if provisionig is not set

* Add ExcludeFromAppDir input argument, remove stopwatch

* Drop DOTNET_SYSTEM_GLOBALIZATION_INVARIANT

* Address feedback

* Validate input

* fix build

* Disable GenerateXcodeProject by default

* move to mono/msbuild

* Add OutputDirectory property

* Fix sample

* Minor improvements

* fix build issues

Fix infrequent/random crashes on Windows x64 due to use of GC forwarded objects. (#34694)

Hard to repro and very infrequent crash. Have been analyzing a couple of crash dumps from retail devices getting different crashes related to vtable "corruption" on Windows x64. After some deeper analysis it turns out the object instance has been forwarded by GC (object vtable pointers lowest bit set to 1), but object still holds tagged vtable. This will then cause misaligned reads, getting back random values and pointers from vtable on next object access.

After some further analyzing it turns out that LLVM codegen and some specific generic vt arrays lowering can cause optimized mem copies using XMM registers. I have also identified scenarios where vt copies gets lowered into a c-runtime memcpy that in turn uses XMM registers as an optimization. Since Windows x64 currently don't include XMM registers in context, any references in XMM registers will not be visible and pinned by GC, meaning that they will point to potentially
forwarded objects after completing GC, restarting threads, leading to these infrequent random crashes.

Fix includes xmm0-xmm15 into MonoContext on Windows x64, making sure GC will see all references that could be held in those registers, regardless if getting into those registers due to LLVM optimization or other native code, like memcpy.

Co-authored-by: lateralusX <lateralusX@users.noreply.github.com>

Merge pull request #34754 from monojenkins/sync-pr-19490-from-mono

[wasm][debugger] Improve some valuetype visualization

[wasm] Disable the AOT cache check on wasm as well. (#34880)

[runtime] Try to load assemblies from bundles on netcore before trying other methods, the ALC might not be initialized yet. (#34877)

Port Native libraries to SunOS (#34867)

Update ownership of Buffers and Drawing (#34928)

Update ELT Hooks - tail calls.md (#34926)

Issue incorrect exception message (#34416)

* Created unit test to trigger incorrect error message.  This also covers a previously uncovered branch.

* Added string resource for the fractionDigits error, and changed the error message being thrown when the derived type has a larger fractionDigits value than the parent type.

enables test (#34911)

[mono] Implement Sse41, Pclmulqdq, Aes and enable Sse41 intrinsics  (#34866)

* Implement Sse41, Pclmulqdq and Aes, enable Sse42.

Reimplement System.Net.Http's ObjectCollection<T> to reduce allocation / interface dispatch (#34902)

The current implementation of `ObjectCollection<T>` wraps a `List<T>` and derives from `Collection<T>`.  This means that every `ObjectCollection<T>` allocated also involves an extra `List<T>` object (and its array if items are added to it) as well as multiple levels of indirection on each operation.

We can instead just implement `ObjectCollection<T>` directly.  Since most uses end up with just a single object contained (e.g. for a `MedaTypeHeaderValue`'s `CharSet`), and since it only ever stores `T`s that are non-null classes, we can use the items field to be either a `T` or a `T[]`, optimizing for the case where a single element is stored.  In implementing it directly, we then also avoid the extra `List<T>` object allocation, as well as the interface dispatch that results from going through the `IList<T>` interface.

Code cleanup (#34862)

made these #define's always defined -

INTERIOR_POINTERS
MARK_ARRAY (replace it with BACKGROUND_GC)
SEG_MAPPING_TABLE
GROWABLE_SEG_MAPPING_TABLE

got rid of code associated with GC_STATS, recursive_gc_sync and the following configs:
GCStressMix, GCStressStep, GCStressMaxFGCsPerBGC, GCMixLog, GCStressStart and GCStressStartAtJit.

got rid of all the unnecessary UNREFERENCED_PARAMETER
got rid of MAYBE_UNUSED_VAR

for joins, got rid of all the extra pairs of checking for #ifdef MULTIPLE_HEAPS and made code styling consistent

got rid of various other pieces of unused code and completely pointless comments.

note there is one functional change: fixed a memory leak by actually doing seg_table->delete_old_slots
when FEATURE_BASICFREEZE is defined; previously this could be leaking memory if there were more than 600
segments (which is very rare for ro segs) and it only leaks a small amount memory in that case.

Co-authored-by: Maoni Stephens <maonis@ntdev.microsoft.com>

Fix HostPolicyMock (corehost_error_writer_fn was moved) (#34903)

fix: distinct from streams (#34918)

Add logic for JsonIgnoreCondition[Never|WhenNull] on properties to win over global IgnoreReadOnlyValues (#34672)

Add support to pass subset without arg specifier in Unix (#34844)

* Add support to pass subset without arg specifier in Unix

* PR Feedback

* Fix regex expression to match whole word

Fix bug in test regarding GC'd delegate instance. (#34901)

Console.Unix: avoid deadlock between LazyInitializer and Console.Out (#34297)

* Console.Unix: avoid deadlock between LazyInitializer and Console.Out

* Add assert for InternalSyncObject

* Console: use Interlocked for lazy initialization

* Cleanup unused 'using'

* Limit locking to event handlers

* Fix AllowNull -> NotNull

* Localize LazyInitializer.EnsureInitialized logic

* Remove lazy initialize delegates

* Use static local functions for initializing

* Remove unused EnsureInitializedDisposableCore

* Remove unused arg from EnsureInitializedStdInReader

* Also EnsureInitialize when not redirected

* Dispose Console Streams with their Reader/Writer

* Fix s_out -> s_error

* leaveOpen streams, fix Encoding set race

* Fix races between Encoding en initializers of In/Out/Error

* Remove EnsureInitialized

* Cleanup

* Rename static local initializer functions to EnsureInitialized

* Use lock in Input/OutputEncoding

* Rename InternalSyncObject to s_syncObject

* Cleanup space

* Fix _isStdErrRedirected -> _isStdInRedirected

* Remove space

* Improve comments

* Fix potential CancelKeyPress deadlock

* Apply suggestions from code review

Co-Authored-By: Stephen Toub <stoub@microsoft.com>
* Remove unnecessary null forgiving

* Add comment about why we're not Disposing StdInReader

* Add Volatile.Writes for Volatile.Read fields

* More Volatile.Writes

* Add back lock to SetIn

Co-authored-by: Stephen Toub <stoub@microsoft.com>

Append TryGetValue to ImmutableHashSet`1.Builder and ImmutableSortedSet`1.Builder (#34869)

* Append TryGetValue method.

Append method to builder of ImmutableHashSet and ImmutableSortedSet.

Implements #28160.

* Small cleanup.