Update WCF dependencies on the compat pack (#59764)

Fix retry when downloading clang tools (#59806)

Invoke-WebRequest throws an exception when the download fails, so we
should use try-catch instead of status code to check failure. We can use
the generic Retry function from tools.ps1 to do this.

In addition pass -PassThru to avoid leaving a corrupted download file in
case the download fails. This will buffer the download and write it once
at the end.

Fix #57196

Several improvements to regex source generator (#59660)

* Fix comparison in EmitMultiChar

For strings of length 2 we weren't using the optimized path of reading/comparing a single uint.

* Remove duplicative length checks for some concatenations

When we have a sequence like `abc[def]ghi`, today we'll emit a length check of 3 for the `abc` multi, a length check of 1 for the `[def]` set, and a length check of 3 for the `ghi`, but we can instead emit a single length check for 7.

This also fixes a small bug where I used a > instead of a >= in a length comparison, the net effect of which was using two comparisons instead of getting away with one for a two character string.

And added some debug-only output that helped me diagnosing why things were working the way they were.

* Special-case regexes where Go can be a nop

For an expression like "abc", if FindFirstChar returns true, it's found and fully validated the match.  There's no benefit to comparing it again in Go; instead, Go can simply perform the capture.

* Clean up generated Boyer-Moore code

Fix NRT warnings in source generated code (#59755)

* Fix NRT warnings in source generated code

* address feedback

* address style

Increase timeout for ConsoleLifetimeExitTests.EnsureEnvironmentExitDoesntHang (#59811)

* Increase timeout for ConsoleLifetimeExitTests.EnsureEnvironmentExitDoesntHang

* missed the comment

fixes to get Quic working on macOS (#59728)

* fixes to get Quic working on macOS

* Apply suggestions from code review

Co-authored-by: Marie Píchová <11718369+ManickaP@users.noreply.github.com>
Co-authored-by: Marie Píchová <11718369+ManickaP@users.noreply.github.com>

Fix regression in Regex lowercasing in parser (#59434)

For better or worse, Regex matches char by char, without any attention paid to surrogate pairs and the like.  In .NET Core 3.0, a change was made to the regex parser that used ToLower on a whole string rather than calling ToLower on each individual char, but this then leads to inconsistencies with how the matching is performed, making it so that things that used to match no longer do.

Leverage `cwde` and `cdqe` for sign extensions on xarch (#56172)

* Add CWDE support to the emitter and codegen

It is a shorter form of a sign extending mov from eax to eax.
Use it in codegen for casts.

* Fix printing for CWDE and CDQ

Their size determines what they do.

* Move the optimization to emit_Mov

This allows us to catch more cases.

[wasm][debugger] Fix reusing buffer for debugger (#59773)

* Fixing resuing buffer on main

* Adding comment as suggested by @lambdageek

Build one MonoTargetsTasks assembly for Microsoft.NET.Runtime.MonoTargets.Sdk pack (#59720)

* Build one MonoTargetsTasks assembly for Mono.NET.Runtime.MonoTargets.Sdk pack

   Instead of 3 assemblies for some tiny tasks.

   Building a single assembly also ensures that everything in the pack depends on and bundles identical versions of dependent assemblies.  For example on net472, a single version of System.Text.Json with a single fixed version shared by both the JsonToItemsTaskFactory task factory and the RuntimeConfigParser task.

   Related to https://github.com/dotnet/runtime/issues/59619

* Update testing and wasm to use MonoTargetsTasks.dll

* Move assembly stripper to a separate assembly

   Build it with nullability checking disabled.

   Build the main MonoTargetsTasks assembly with nullability enabled

* Don't include msbuild assemblies in MonoTargets package

* re-enable analyzers for the MonoTargetsTasks assembly

* Include *.dll in the MonoTargetsTasks nupkg

   In particular, pick up all the dependent assemblies for the net472 build

* Update various READMEs

* remove unneeded target

* Switch ILStrip to use the same Parallel.ForEach logic as MonoAOTCompiler

* Remove unneeded NoWarn warnings

* rearrange package references in MonoTargetsTask.csproj for readabilty

* More ItemGroup reformatting

* fix nits

* Use SystemThreadingTasksExtensionsVersion instead of hardcoding

Co-authored-by: Ankit Jain <radical@gmail.com>

Remove ILStrip temporarily to retain ActiveIssue attributes in release mobile device configuration (#59762)

* Remove ILStrip temporarily to retain ActiveIssue attributes in release mobile device configuration

* Add Android Active Issue

Co-authored-by: Mitchell Hwang <mitchell.hwang@microsoft.com>

CI matrix update: add Ubuntu 21.04 (#59578)

CI matrix: add openSUSE 15.2 (#59574)

CI matrix change: add Windows.Server.Core.20H2 (#59572)

* Add Windows.Server.Core.20H2 to CI matrix

* add windows server 20H2

* Remove 20H2 from PR

* Remove 20H2 from PR run

Fix assert for chain of ASG nodes after tail-calls (#59708)

Fix debugger-tests that were broken after https://github.com/dotnet/runtime/pull/59153 (#59795)

Update how-to-debug-dump template (#59798)

* Update how-to-debug-dump template

* Address CR feedback

* Update windows instructions

* Use backslashes on Windows

Rev Alpine WithNode from 3.9 to 3.13 (#57324)

* Rev Alpine WithNode from 3.9 to 3.13
Also,
* update helix queues from 3.12 to 3.13
* add 3.14 helix queues
* update performance job to install cargo from main branch

* Add Alpine 3.13 in rolling CI

* Add missing closing paren

[interp] Fix unboxing during delegate invoke (#59749)

We need to unbox only if the called method is defined on a valuetype

CI matrix change: debian+centos+sles (#58988)

* CI matrix change: debian+centos+sles

* Adjust the CI matrix based on latest comments

Fix hfa field count to handle more cases under LLVM. (#59751)

Current way of calculating fields for hfa types was not correct,
incorrectly describe some hfa types in LLVM.

One scenario where it breaks is when using explicit struct layout
having several fields starting at the same offset. Our current
implementation would count each field meaning that the struct
of floats or doubles described to LLVM would not match the size
of allocated object, since it would count fields starting
on the same offset twice, causing issue in p/invokes on platforms
where the platform ABI push value types on to callers stack.

An example:

StructLayout(LayoutKind.Explicit)]
public struct Vector2
{
  private unsafe struct F2
  {
    public fixed float data[2];
  }

  [FieldOffset(0)]
  private F2 V;

  [FieldOffset(0)]
  public float V1;

  [FieldOffset(4)]
  public float V2;
}

The old implementation would describe the above struct like this to LLVM:

type { float, float, float }

incorrectly adding an additional float, creating a 12 byte struct instead
of 8 bytes.

Any construction using

private fixed float x[12];

would also incorrectly be calcualted as one field instead of 12.

The way mini_type_is_hfa works is that it will make sure the type is
only made up of floats or doubles and not a combination of them or any
other type. Instead of using the field count calculated through this
schema (that sees all field types, but inaccurately count them in some
scenarios), we could use types value size (since its all floats or
all doubles) and then get number of fields based on type size
(float or double). Implementation of mono_class_value_size will correctly
handle size in cases where we have explicit layout as above and shared
with how none LLVM AOT:ed code views the same type.

[main] Update dependencies from dotnet/linker (#59257)

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
Co-authored-by: vitek-karas <vitek.karas@microsoft.com>

[wasm] Build improvements based on feedback (#59391)

- Fix for a incremental build issue
- Change native build defaults for `Debug` config, to make it faster (thanks @SteveSandersonMS, @mattleibow)

- MonoAOTCompiler:
  - Skip unmanaged assemblies, and emit a warning
- PInvokeTableGenerator:
  - Fix to not fail in presence of native variadic functions, and pinvokes with function pointers (eg. used by sqlite)

Co-authored-by: Larry Ewing <lewing@microsoft.com>

Fix an occasional small perf issue in `ThreadLocal` constructor/dispose (#59300)

- Replaced the linear search for a free ID with a pair of collections that operate in O(1) time for insertion and removal
- See https://github.com/dotnet/runtime/issues/59145#issuecomment-922162343 for more information
- Fixes https://github.com/dotnet/runtime/issues/59145

Add new analyzers to CodeAnalysis.globalconfig files (#59763)

Fix issue with incorrect handling of optimized method layouts (#59781)

- The sort order of the RuntimeFunctions table (among others) was not properly handled when an optimized method layout was applied
  - While this bug applied to non-composite builds, it was easily reproduced with a composite build
- Fix lack of test infrastructure around non-standard method layouts.
  - Add new random method sorting order
  - Use new random sorting order in crossgen2 test passes

Fixes #59089

Partially disable test (#59760)

Fix for GitHub issue #59432 and the corresponding regression test (#59663)

Turns out there was a subtle bug in the interface slot map population
that got triggered by a suitable combination of non-virtual and
virtual interface methods.

Thanks

Tomas

Avoid resetting gen0 bricks for background_sweep (#59106)

Do not enregister Return SP check (#59759)

Retry installer Process.Start if ETXTBSY (#59765)

Caused by #58964

Fixes #53587

Fix several issues with ELT profiler hooks and tests to cover these cases (#54912)

* Add ELT profiler tests for HFA/HVA arguments/return values on Arm64.

* Fix an issue with `ProfileArgIterator::GetNextArgAddr` and `ProfileArgIterator::GetReturnBufferAddr` not properly handling HFA/HVAs on Arm64.

* Add tests to cover different ways of returning a struct on Linux x64 and macOS x64 (i.e. one register, two registers, stack).

* Fix an issue with `ProfileArgIterator::GetReturnBufferAddr` not properly handling struct returned in registers on Linux x64 and macOS x64.

* Fix an issue with not passing an actual value of `rdx` down to a callback in `ProfileLeaveNaked` and `ProfileTailcallNaked` helpers.

fix few more QUIC tests that can hang  (#59682)

* fix few more tests that can hang

* update

Treat 'abstract' properties the same as 'virtual' for src-gen (#59707)

Src gen: use GetBestTypeByMetadataName instead of GetTypeByMetadataName (#59092)

Use cmakedefine in eglib config (#59623)

Both `config.h.in` and `eglib-config.h.cmake.in` are now configured via
cmake. However, only `config.h.in` is using `cmakedefine*` and
`eglib-config.h` was having no effect on introspection result.

[wasm][debugger] Fix debugger after ts changes (#59702)

* Debugger was broken.

* Reverting usage of local_eval.

* Addressing @lambdageek suggestion.

* Addressing @kg comment.

* Fix spacing before parentheses.

Add initial codespaces support for dotnet/runtime (#59723)

* Add initial codespaces support for dotnet/runtime

* Enable codespaces-prebuild

[mono] Increase the number of preallocated trampolines for Mono FullAOT runtime test runs (#59737)

I have no justification for these numbers; they were pulled from
MonoAOTCompiler.props. It might be a good idea to port the "infinite
trampoline" hack to non-Apple platforms.

Add PEM export methods to certificates

This allows exporting a certificate directly to a X.509 PEM string
(`-----BEGIN CERTIFICATE-----\n...\n-----END CERTIFICATE-----`), and cert collections
to either a PKCS#7 SignedData with no signers PEM ("BEGIN PKCS7"), or to a X.509
multi-PEM (including the empty string and a single PEM for the 0 and 1 length collections).

Acknowledge community triagers in area-owners.md (#59748)

* Update area-owners.md

* address feedback

* Update docs/area-owners.md

Co-authored-by: Stephen Toub <stoub@microsoft.com>
* Update area-owners.md

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

Arm: Include BBJ_ALWAYS blocks in dominance calculation (#59376)

* Include BBJ_ALWAYS in inverse post ordering for computing reachability

* Add test case

* Add BBJ_ALWAYS in fgAlways list

* jit format

Prefer readonlyspan properties over readonly array fields (#59670)

* Fix violations in System.IO.Compression

* Fix System.Private.DataContractSerialization

* Fix System.Net.HttpListener

* Fix System.Text.Encoding.CodePages

* Revert changes to .editorconfig

* Apply suggestions from code review

Co-authored-by: Günther Foidl <gue@korporal.at>
Co-authored-by: Günther Foidl <gue@korporal.at>

delay locking in `ConcurrentDictionary.GrowTable` until necessary (#59656)

[wasm][debugger] Reusing buffer to avoid allocate every debugger message (#59480)

* Reusing buffer to avoid allocate every debugger message.

* Fixing what was suggested by @radical.

* cleaning code.

* Changing what was suggested by @radical

* Implementing again after merge

* Fix spacing before parentheses.

* Addressing @pavelsavara comments.

* Fix _debugger_buffer type.

Revise descriptions of VS workloads, and add missing ones (#59698)

Revise descriptions of VS workloads, and add missing ones

Use new cross-build images for arm/arm64 builds (#59534)

Append _FILE_OFFSET_BITS=64 to CMAKE_REQUIRED_DEFINITIONS (#59504)

After setting CMAKE_REQUIRED_DEFINITIONS in PAL introspection, we need
to append this definition.

Integrate changes from NativeAOT branch (#59510)

* Move TryResolveConstraintMethodApprox to compiler

* Integrate changes from NativeAOT branch

[mono][jit] Eliminate some AOTCONST instructions if they are not used. (#59699)

Antigen: Change Antigen's helixqueue to public (#59657)

* Use public helix queue

* Remove helix queue setup from python

* Remove unnecessary comments

* TOREVERT: Download specific artifacts

* change creator to dotnet-bot

* set Creator

* run only for windows x64

* Publish antigen build logs

* put in quotes

* use correct script for helix queue

* Set some helix variables:

* Use Helix properties in proj

* Include the display name

* Revert specific build

* Add continueOnError to display warnings

Deduplicate epilogs in Type.operator== (#59711)

Only run source-index on main branch. (#59725)

Universal arm crossgen compiler (#58279)

Stop generating OS specific compilers for the arm and arm64 architectures for use in crossgen2
- As it turns out the OS differences here are relatively minor and fairly easy to handle with runtime switches.
- However, when an exact target is known we can generate better code by having the various if statements evaluate to a constant true/false. (This is done for all runtime jit builds)
- This has been validated to produce binary identical output for generating System.Private.CoreLib.dll on the various architectures.
- This work reduces the distribution size of crossgen2, and also reduces the amount of time it takes to build everything (Most notably, the Mac build times in PR's are reduced by 5-7 minutes, but improvements are visible across all of/architecture pairs)

Revert bind single elements to array in configuration binder (#59716)

Update HttpConnectionBase.cs (#59722)

Turn on XHarness telemetry for mono runtime tests (#59646)

Fix Time Zone when running with Globalization Invariant Mode (#59710)

Add Compiler record to PDB files in Crossgen2 (#59686)

Fix incorrect tracking of sign bit (#59525)

* Fix incorrect tracking of sign bit

* Perform the 1 check earlier

* Add comment on checking special pattern

* Simplify fix

* Update the test to target 32 bit right shift
as the logic has been restricted to that

* Remove stray newline

Check Options validations length instead of null (#58157)

The `OptionsFactory` constructors ensure that the member variable
`_validations` is never null. Yet, the `Create` method checks whether
`_validations` is null before running them.

Check that the length of `_validations` is larger than zero instead of
checking for null. This eliminates a `List<>` allocation when there are
no validations.

JIT: allow suppressing some dumps or parts of dumps (#59299)

Another bit of code that I've had lying around that might be generally useful.
* Optionally suppress Tier0 compilation dumps
* Optionally suppress dumps from inline compiler phases

Setting `JitDumpTier0=0` prevents you from getting two dumps when you specify
dumping a method by name or hash, when TC is enabled.

Dependency Injection SAVE_ASSEMBLIES code clean up (#59684)

A simple clean up of the SAVE_ASSEMBLIES symbol code in DI.
I am also adding the service type name to the class name because
it helps a lot when manual inspecting MSIL code for large service trees.

Removing unused CallSiteKind items from Dependency Injection (#59687)

Removing Scope, Transient and Singleton from enum CallSiteKind.
Their CallSites were removed by 8e8650f9

Prevent cache from being resized (#59607)

Fix race condition in ServiceProcess tests (#59676)

Reduce allocations in VersionConverter (#55564)

* Add more test cases for VersionConverter

* Reduce allocations in VersionConverter.Read method

* Remove allocations from VersionConverter.Write method

* Parse Version from chars, instead of parsing individual components. Assert successful formatting on write.

* Allow escaping, disallow leading and trailing whitespaces

* Hopefully fix tests for .NET Framework, add few more test cases that should fail

* Assume whitespaces can be only on the start, or only at the end, add one more test case that should fail

* Remove redundant return statements

* Elaborate on comment regarding leading and trailing whitespaces. Copy it to .NetStandard2.0 target.

* Add another test case.

Update MonoVM diagnostics-tracing doc with new features and scenarios. (#58620)

Remove redundant checks (#59310)

* Remove useless if-else branch

* Try to change the absolute expiration only if the relative has value

Dependency Injection support for ValueTypes as Services (#59625)

* Dependency Injection support for ValueTypes as Services

The only that was thing missing to support ValueTypes as Services
was box the result when visiting constructors call sites.

I am also fixing several tests that were building the ServiceProvider
using the default engine, instead of relying on each specific engine.

fix #42160

* addressing pr feedback regarding tests

Using optimized ILGenerator.Emit function in Dependency Injection (#59680)

We were doing some local optimizations with MSIL’s Stloc and Ldloc,
but the very same ones are already present in an override of the
ILGenerator.Emit function, so we can remove them.

This PR also fixes the use of Ldloca that was generating additional
nop operations.

fix #59624

[mono][jit] Eliminate some type comparisons in gshared methods. (#59543)

* [mono][jit] Make MONO_INS_HAS_NO_SIDE_EFFECT () into a function.

* [mono][jit] Eliminate some type comparisons in gshared methods.

Comparisons of the form
```
        internal static void ThrowForUnsupportedIntrinsicsVectorBaseType<T>() where T : struct
        {
            if (typeof(T) != typeof(byte) && typeof(T) != typeof(sbyte) &&
            ...
        }
```
are very common in BCL code. They are compiled to:
```
    IL_0001:  ldtoken    !!T
    IL_0006:  call       [System.Runtime]System.Type [System.Runtime]System.Type::GetTypeFromHandle([System.Runtime]System.RuntimeTypeHandle)
    IL_000b:  ldtoken    [System.Runtime]System.Byte
    IL_0010:  call       [System.Runtime]System.Type [System.Runtime]System.Type::GetTypeFromHandle([System.Runtime]System.RuntimeTypeHandle)
    IL_0015:  call       bool [System.Runtime]System.Type::op_Inequality([System.Runtime]System.Type,
                                                                         [System.Runtime]System.Type)
    IL_001a:  brfalse    IL_015a
```

In gshared code, !!T is usually constrained to be a reference, or to have a primitive base type.
Use this fact to eliminate some of the Type::op_Equality/op_Inequality () calls.

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

Disabling Match_ExcessPrefix until the attribute is fixed

Fewer AddressExposedReason::TOO_CONSERVATIVE  (#59429)

Delete some too_conservative addrExposed reasons.

Remove log message when closing browser while debugging. (#59664)

Skip ExportPkcs7_Empty test on mobile platforms

* Skip ExportPkcs7_Empty test on mobile platforms

The test made it through and was failing on CI.  PKCS#7 export is not supported on mobile platforms.

* PKCS7_Empty was failing on android b/c 0 length was not allowed.

Co-authored-by: Steve Pfister <steve.pfister@microsoft.com>

Localized file check-in by OneLocBuild Task (#59662)

Add checking SpeechRecognitionEngine Culture to GrammarBuilder test (#59516)

* Add checking SpeechRecognitionEngine Culture to GrammarBuilder test.

* Update GrammarTests.cs

Co-authored-by: Dan Moseley <danmose@microsoft.com>

forward-port from `release/6.0-rc2`: MonoAOTCompiler: detect when nothing has changed, a… (#59597)

…nd skip any precompiling (#58979)

Refactor to allow fast-path

implement fast-path for MonoAOTCompiler when nothing has changed

re-enable some tests that got disabled by mistake

(cherry picked from commit 9fd17b1)

Change loop cloning condition blocks flow graph (#59233)

Currently, the loop choice condition blocks are created in a way that creates
a confusing flow graph. Restructure them to be simpler, and lay the path for
future work.

The simpler layout (and, hopefully, code and logic to create them) also
lays the groundwork to potentially put the slow path loop in the loop table,
if desired, as it creates a "standard" loop header layout.

While the flow graph has an extra block or two, there are almost no diffs,
as subsequent phases clean up any extras.

There are a few diffs where the new layout enables some downstream optimization
phases that were stymied by the old layout, but mostly in cases where we leave
around dead code slow cloned blocks (a known issue). Also, there are a couple
trivial CSE changes.

Add superpmi.exe option to break to the debugger on exception. (#59234)

If you use `-box` (similar to pre-existing `-boa`, `-boe`), then
an exception with invoke a `DebugBreak`, which will stop in the
debugger when an exception is thrown during replay. This is helpful
when debugging (without needing to configure the debugger to catch
a specific exception).

Add InstallLocation component to host (#59537)

Write SPMI download progress in MB (#59647)

Add argument check (#59634)

update area-owners to account for pod change (#59635)

[mono] Improve Makefile's dotnet.js dependencies (#59603)

* [mono] Improve Makefile's dotnet.js dependencies

* Pass _MSBUILD_WASM_BUILD_ARGS properly

* And add $(MSBUILD_ARGS) as well

[wasm] windows: Fail the build if provisioning fails (#59560)

[draft][wasm] introduce typescript modules (#59392)

- refactor wasm javascript as typescript
- use nodeJs from EMSDK
- use rollup.js for packing code as ES6 IFFE for emcc consumption
- incremental build:
Co-authored-by: Radek Doulik <radekdoulik@gmail.com>
- types:
Co-authored-by: Daniel Genkin <genkind@gmail.com>
* linker feedback

Pipeline to run Exploratory fuzzing tool to catch codegen issues (#59489)

* Exploratory tool pipeline

Add pipeline to run exploratory tools and return back the result

* Use public Antigen repo

* Add RunDuration

* Rename some of the files to antigen

* TOREVERT: download specific artifacts

* Trigger nightly and on PRs

* Adjust the duration

* Exit on fail for Antigen

* Rename yml files

* Add Antigen.md

* Temporarily run for 10 minutes

* Remove non-x64 build download and helix workaround

* Add back HelixResultsDestinationDir

* Change to net6.0

* extra ,

* Antigen is still on net5.0

* Default framework net6.0

* Run for 3 hours

* Run full build

* fix space

* Make the schedule weekly Sun@6am

* Reduce duration to 2 hours

Remove invalid test exclusions. (#59608)

These exclusions are pointing to non-existing files. So they are not doing anything.

FreeBSD12 version cleanup and process and thread enhancements (#59339)

* Set FreeBSD 12 as minimum version

This is a small clean-up to remove reliance on FreeBSD versions
prior to 12 and more specifically 12.2 which is the lowest currently
supported version of FreeBSD. 11.4 becomes unsupported as of
October 2021. See https://www.freebsd.org/releases/

* Add functionality for FreeBSD process and threads

Added support for naming native threads, the process disambiguation
key and detecting if the process is running under a debugger.

Changed references to thr_self to pthread_getthreadid_np already
used elsewhere - it returns the same value.

JIT: Convert Interlocked intrinsics to NamedIntrinsics (#55170)

[tests] Fix PlatformDetection.IsMonoInterpreter for mobile (#59587)

* [tests] Correctly detect the Mono interpreter

Checking for MONO_ENV_OPTIONS is not reliable for the mobile test runners
because they use the embedding API to enable the interpreter, rather than
setting an environment option.

Instead detect the interpreter by checking the values of the
IsDynamicCodeSupported and IsDynamicCodeCompiled runtime features.

* [tests] Run System.Threading.Tasks.Dataflow.Tests on Apple platforms

* Only check RuntimeFeature on netcoreapp.  Otherwise return false

these tests are not running on mono if they're running on .NET Framework

enable client certificate test for QUIC on Linux (#59544)

[main] [Release/6.0] Throw error if assembly version for ref pack assemblies  doesnot match assembly version shipped in 6.0.0 (#59583)

* throw errow if assembly version in the ref pack doesnot match last released version

* Fix the condition

* Update eng/Versions.props

Co-authored-by: Viktor Hofer <viktor.hofer@microsoft.com>
Co-authored-by: Anirudh Agnihotry <anagniho@microsoft.com>
Co-authored-by: Anirudh Agnihotry <anirudhagnihotry098@gmail.com>
Co-authored-by: Viktor Hofer <viktor.hofer@microsoft.com>

Delete unused file (#59585)

Some more cleanups in Assembly/Binder area (#59288)

* Reducing BINDER_SPACE::Assembly closer to what it represents

* make GCC happy

* Removed `Binder[AddRef/Release]PEImage`. Can just use `AddRef/Release`

* moved IsValidArchitecture helper to bindercommon

* Removed the notion of PEModule - it does not exist

* some more unused code

* fix for GCC

* associate System lib with DefaultBinder - makes things simpler

* PR feedback

Add a note about rerunning the license/cla check (#59440)

Fix behavior ObjectCollection for single item contains (#59547)

* Fix behavior ObjectCollection for single item contains
* Address PR feedback

Fix ArrayPool leak with JsonDocument (#59540)