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

[mono] Add link dependencies for dotnet.js (#59470)

This fixes incremental build in cases where the js libraries are changed
or the response file.

Add execute bit to postinstall script (#59552)

JIT: Return multiple likely classes in getLikelyClass (for better GDV) (#58984)

Co-authored-by: Jakob Botsch Nielsen <Jakob.botsch.nielsen@gmail.com>
Co-authored-by: Bruce Forstall <brucefo@microsoft.com>
Co-authored-by: Andy Ayers <andya@microsoft.com>

Improve ReliabilityFramework (#59384)

Fix PKCS7 export quirks.

On macOS, exporting an empty X509Certificate2Collection to PKCS7 would
produce an exception. Instead, we return a pre-encoded empty PKCS7
result.

On Windows, the PKCS7 export would include trailing zeros in PKCS7
export. This occured because the first calls to CertSaveStore would
return a size that is at least big enough, but not exact. This trims the
result to the actual amount written on the second call.

disable System.SpanTests.SpanTests.FillNativeBytes (#59563)

Remove unused emitter call types and improve INS_tail_i_jmp handling (#59495)

The size estimation for INS_tail_i_jmp instructions was working by
accident: we were always regarding these as requiring rex.w
prefixes, even when they did not. Fix the logic so that rex.w is not
considered required for 8-byte tail jumps, but add it manually for the
forms where it is really necessary.

There were also a lot of unused emitter call types that I have removed.

Rewrite Enum.HasFlags and Enum.Equals in C# (#59514)

Fix krb5 library SO name in the gcc api shim (#59526)

The library name used in the shim is a name of the build time library which
is usually installed only for development purposes. We should use
libgssapi_krb5.so.2 which is the underlying runtime library.

Update area owners to include new owners of S.DS.* (#59357)

@jay98014 @grubioe @BRDPM

Add a no-cache backed CLRConfig lookup mechanism (#59513)

* Add a no cache backed CLRConfig lookup mechanism

Remove remaining locations that had a hardcoded COMPlus_ prefix.
The new no cache CLRConfig is usable from within the PAL.

Issue 59415 (#59497)

* Undo shift-back with AVX register for block copy, use GPR for remainder. Results in better perf

* Cleanup

* Use GPR instead of SIMD for block init remainder

* JIT formatting

* More JIT formatting

* Fixed local jit-format.bat, formatting should be correct now

* Using older jit-format.bat on backup machine

* Revert "Using older jit-format.bat on backup machine"

This reverts commit b0e081714cab85aa9fd5cec53375745fa5c68fb2.

* Revert "Fixed local jit-format.bat, formatting should be correct now"

This reverts commit 39c3f2ff22e8c7abfa7108b2506ecb10c249c320.

* Revert "More JIT formatting"

This reverts commit 237ce3122294201abbcfd2c52cbb1ebccc4f4462.

* Revert "JIT formatting"

This reverts commit 3191bd4075fc1c93ac9eeb0af08e367702d3d804.

* Undo weird JIT formatting, add note for YMM->XMM step down

Move clang-format download into dotnet/runtime and add docs for setting up auto-formatting in the repository (#59374)

Remove unused crossIBC.ps1 script (#59530)

Revert "Fix incorrect handling of character range and capitalization … (#59425)

* Minimal change

* Nit

* Use the new Regex engine infra properly

Prevent racing on installer shared props/targets (#59276)

[docs] macOS: add note about Xcode command line tools (#59477)

File preallocationSize: align Windows and Unix behavior. (#59338)

* File preallocationSize: align Windows and Unix behavior.

This aligns Windows and Unix behavior of preallocationSize for the
intended use-case of specifing the size of a file that will be written.

For this use-case, the expected FileAccess is Write, and the file should be
a new one (FileMode.Create*) or a truncated file (FileMode.Truncate).
Specifing a preallocationSize for other modes, or non-writable files throws ArgumentException.

The opened file will have a length of zero, and is ready to be written to by the user.

If the requested size cannot be allocated, an IOException is thrown.

When the OS/filesystem does not support pre-allocating, preallocationSize is ignored.

* fix pal_io preprocessor checks

* pal_io more fixes

* ctor_options_as.Windows.cs: fix compilation

* Update tests

* tests: use preallocationSize from all public APIs

* pal_io: add back FreeBSD, fix OSX

* tests: check allocated is zero when preallocation is not supported.

* Only throw for not enough space errors

* Fix compilation

* Add some more tests

* Fix ExtendedPathsAreSupported test

* Apply suggestions from code review

Co-authored-by: David Cantú <dacantu@microsoft.com>
* Update System.Private.CoreLib Strings.resx

* PR feedback

* Remove GetPathToNonExistingFile

* Fix compilation

* Skip checking allocated size on mobile platforms.

Co-authored-by: David Cantú <dacantu@microsoft.com>

Fix QuotedNumbers_Deserialize test (#59399)

Fix underestimation of temps size (#58969)

It was possible for us to underestimate the size of temps required. This
happened because we used the state of the frame layout to check if the
size of temps was computed and that check was wrong. This could lead us
to make different decisions about which registers needed to be saved in
the prolog and epilog on ARM32. In particular, if the frame size was
around the size where a stack probe is necessary, this could be
possible.

There are also other comments that suggest that this could result in
failure while encoding instructions that reference the stack locals.

Fix #58293

Disable more SendPacketsElement tests on Win11 (#59359)

[mono] Build system quality of life tweaks (#59511)

On a Linux machine that has a copy of LLVM installed in /usr, along with
a copy of ICU also installed in /usr and registered with `pkg-config`,
`ICU_LDFLAGS` will be populated with `-L/usr/lib`, which will later
cause this flag to be prepended to several targets' linker flags,
causing the linker to fail when attempting to link against the global
copy of LLVM rather than the copy of LLVM supplied by the user.

Work around this by using absolute paths to each LLVM static archive
file.

Additionally, prepend the LLVM include directory to the list of include
directories for every target defined in mono/mini/CMakeLists.txt,
instead of relying on `CMAKE_C_FLAGS` and `CMAKE_CXX_FLAGS` for this,
for improved support in IDEs.

[mono] Remove support for llvm versions < 9.0. (#59512)

Move Type equality operators to shared CoreLib partition (#59499)

XmlSerializer support for IsDynamicCodeSupported=false (#59386)

* XmlSerializer support for IsDynamicCodeSupported=false

Add more checks to XmlSerializer to check the SerializationMode. Don't try to use Reflection.Emit if the SerializationMode is ReflectionOnly.

These changes were ported from
* https://github.com/dotnet/runtimelab/pull/593
* https://github.com/dotnet/runtimelab/pull/600

Fix #59167

* Fix a bug in XmlSerializer.CanDeserialize when in ReflectionOnly mode.

* Port UAP code for CanDeserialize

* PR feedback

* Add a linker test to ensure linker option '--enable-opt sealer' works when IsDynamicCodeSupported==false.

Setup retry for System.Net.* tests (#57370)

The Helix SDK in arcade supports test retries in Helix by reading this file, "test-configuration.json" if it's present.

This sets up any test who's assembly name starts with System.Net.* to have 3 attempts (2 reruns) locally.

Fix xmldoc of TypeGenerationSpec (#59361)

* Fix xmldoc of TypeGenerationSpec

* Apply suggestions from code review

Co-authored-by: Carlos Sanchez <1175054+carlossanlop@users.noreply.github.com>
Co-authored-by: Carlos Sanchez <1175054+carlossanlop@users.noreply.github.com>

Fix buffering bugs in TextFieldParser (#59487)

Three bugs being fixed here:
1. The SlideCursorToStartOfBuffer method is incorrectly assuming that the buffer is filled to its end (that m_Buffer.Length = m_CharsRead).  As a result, two things happen.  First, we copy more data than is needed to the temporary buffer; that's just unnecessary but not harmful.  But second, when it issues a read to fill the remainder of the buffer, it does so at the wrong position, both leaving zeros in the buffer that end up getting parsed as real data and losing real data from the end.
2. IncreaseBufferSize assumes m_CharsRead = m_Buffer.Length, which may not be the case.  As with (1), this can result in it reading into the wrong part of the array and leaving garbage that gets processed.
3. IncreaseBufferSize is always increasing the buffer size, even if only a small portion of the buffer is used.  This can then result in the whole operation failing when it repeatedly increases the buffer size and ends up failing due to ticking over the max buffer threshold.

(1) doesn't exist in .NET Framework 4.8 but (2) and (3) do.  (1) is easily triggered by a stream that sporadically doesn't produce all the data requested; (2) and (3) end up needing a stream that frequently produces much less than requested.  This PR fixes all three.

Update FileStatus field names in HostModel tests (#59482)

In https://github.com/dotnet/runtime/commit/6a81cfa4e84ab55c26abb25d82ed7355c134a602, we changed `FileSystem._fileStatus._fileStatus.Mode` to `FileSystem._fileStatus._fileCache.Mode` and https://github.com/dotnet/runtime/commit/159a01acabc79bfeb77d8c0b28de2d5fdb7773d0 adapted the change in test code during the infra update. This PR is updating field and type names along with exception message, that reflects current state.

Mark time trigger (#58261)

[main] [Release/6.0] Add uploading of intermediate package assets (#59490)

* add uploading of intermediate package assets

* dont push or sign sourcebuild packages in normal leg to avoid duplication

* add comments

Co-authored-by: Anirudh Agnihotry <anagniho@microsoft.com>

Set *isMapped to false to satisfy 'out' semantics (#59430)

CordbModule::IsMappedLayout was not setting the out
parameter *isMapped to false before returning. The
function could therefore have a successful return code
(S_FALSE) without satisfying the constraint that out
parameters be set before return, resulting in undefined
behavior for code that merely checks for a successful
return, rather than S_OK.

This fix updates the function to set *isMapped to false
as soon as it has been verified. It also updates the
documentation of ICorDebugModule4::IsMappedLayout
to indicate that the value of pIsMapped cannot be assumed
to be valid when S_FALSE is returned.

Fixes #59393

Integrate changes from NativeAOT branch (#59478)

Add initial regex source generator (#59186)

* Add RegexGeneratorAttribute

Adds the new RegexGenerator attribute that's a signal to the regex
generator to generate code for the specified regex.

* Implement RegexGenerator

Add a source generator for generating C# code for Regex.  This is
primarily a port of RegexCompiler.cs, generating C# code instead of
MSIL.

* Add generator test project

Adds tests dedicated to the mechanics of the source generator, e.g. that
appropriate diagnostics are issued for improper use of RegexGenerator.

* Integrate source generator into many regex tests

Start integrating the source generator into the regex test suite, so
that many existing tests also validate the generated code.

* Address PR feedback

* Improve cachability of source generator

Changing the generator to not collect all regexes together means we don't need to reprocess/regenerate all regexes every time any one of them is changed.

* Use closest matching ctor from GetRegexAsync

To better test the appropriate ctor usage.

* Improve a few of the polyfills

* Address PR feedback

* Fully qualify types and remove unnecessary $s

Also fixed one place where the IL we were generating wasn't as good as the reflection emit code.

* Suppress a couple more warnings

* Fix stray !

* Fix TODO about unrolling multi comparisons

Also clean up generated code in a few places to make it more readable / concise.

* Update resources per PR feedback

* Add more tests, clean up parser code, and allow instance/interface methods

* Fix test suppression on mobile

Split install_location to file-per-architecture (#59404)

On non-Windows we used install_location file with multiple lines which contained locations per-architecture (and the default).

This changes the host to use separate file for each architecture:
* `install_location_<arch>` for the architecture specific
* `install_location` for the "legacy" file

This is a much easier format for installers to handle (and it also simplifies the apphost code a little bit).

Adapted tests to the new behavior. Removed some tests which don't make sense anymore. Some small test cleanup to deduplicate code.

Changed the test-only env. variable to point to the directory with configuration files instead of to the configuration file itself.

Set install_location arch specific (#59441)

Also don't write to paths when running translated (rosetta)

Delete Type.GetTypeHandleInternal (#59459)

Never-overriden virtual method forwarding to virtual property.

Retrieve dumps from stress pipeline (#58860)

* Removed parts of stress to make the test loop faster.

* Dump collection and artifacts publish

* Chown dumps

* Removed Windows build for tests

* Bringing windows on par with linux

* Fixed windows paths.

* reverted testing changes

* feedback

* Update eng/pipelines/libraries/stress/http.yml

Co-authored-by: Santiago Fernandez Madero <safern@microsoft.com>
* feedback renames

* feedback newlines

Co-authored-by: Santiago Fernandez Madero <safern@microsoft.com>

Cleanup reflection RuntimeType checks (#59457)

* Delete IsRuntimeImplemented helper method

No longer necessary

* Use compact syntax for RuntimeType checks

[wasm][debugger] Fix concurrent access to cache. (#59412)

* Fix concurrent access to cache.

* Fixing test case.

* Changing what was suggested by @radical.

Fix brick table issue due to race condition in adjust_limit_clr (#59411)

With regions, we had observed AVs due to 0 brick table entries.

As it turns out, this is due to a race condition in adjust_limit_clr where ephemeral_region may change before we check it to determine whether we need to set bricks.

The fix is simply to fetch ephemeral_region into a local before we release the more space lock.

This PR also contains the code checking the brick table for 0 entries - I propose leaving this code enabled even in retail until we are confident the race condition mentioned above is the only cause.

CI matrix change: ubuntu (#58987)

* Remove ubuntu 16.04

* Remove another occurence

CI matrix change: drop OSX 10.13 (#59084)

* Drop OSX 10.13 from CI matrix

* Drop OSX 10.14 from PR  CI matrix

We should not call distribute_free_regions at the end of background GC - then it may run concurrently with user threads allocating and getting regions from the free lists. (#59413)

[wasm] Enable System.Text.Json.SourceGeneration.Tests (#59396)

Add notes about not calling src-gen APIs directly and make them non-browsable (#59385)

Big-endian fixes: PDB creation by ilasm (#59422)

* Fix byte order of debug directory fields

* Fix byte order of PDB stream data (PdbHeap::SetData)

update xunit version (#59275)

Delete a legacy local for arm32. (#59380)

Arm: Use correct form of store instruction when data is float (#59187)

* use the correct src type for str operation

* Add test case

* Proper fix

* Extend to arm64

* remove unnecessary property from csproj

* Update the test case

* restore the assert

* jit format

* Use ins_StoreFromSrc

* jit format

Fixing memory corruption. (#59419)

First round of converting System.Data.OleDb to COMWrappers (#54822)

* First round of converting System.Data.OleDb to COMWrappers
This change just remove warning from ILLink and do not make attempt
to convert other COM object creation to use ComWrappers

Also I have to add target net5.0-windows where ComWrappers would be
used. Same concerns regarding testing as in
https://github.com/dotnet/runtime/pull/54636

Two calls inside OleDbDataAdapter call GetErrorInfo but do not do anything with results. That's probably mistake from previous refactoring. These parts does not touched and left as is. Need advice on that specific parts how to proceed.

* Update src/libraries/System.Data.OleDb/src/OleDbComWrappers.cs

Co-authored-by: Aaron Robinson <arobins@microsoft.com>
* Update src/libraries/System.Data.OleDb/src/OleDbComWrappers.cs

Co-authored-by: Aaron Robinson <arobins@microsoft.com>
* Apply PR recommendations

* Make closer to original

* One more change

* Apply PR feedback

* Update src/libraries/System.Data.OleDb/src/System.Data.OleDb.csproj

Co-authored-by: Viktor Hofer <viktor.hofer@microsoft.com>
* Update src/libraries/System.Data.OleDb/src/System.Data.OleDb.csproj

Co-authored-by: Viktor Hofer <viktor.hofer@microsoft.com>
* Fix compilation errors

* Fix errors

* Consolidate code

* Fix compilation warnings

* Fix compilation error

* Fix compilation error

* Address PR feedback

* Update PR feedback

* Address PR feedback

* Direct cast

* Ask wrapper to implement IDisposable

* Update src/libraries/System.Data.OleDb/src/OleDbConnection.COMWrappers.cs

Co-authored-by: Eric Erhardt <eric.erhardt@microsoft.com>
* Update src/libraries/System.Data.OleDb/src/OleDbComWrappers.cs

Co-authored-by: Eric Erhardt <eric.erhardt@microsoft.com>
* Update src/libraries/System.Data.OleDb/src/OleDbComWrappers.cs

Co-authored-by: Eric Erhardt <eric.erhardt@microsoft.com>
* Update src/libraries/System.Data.OleDb/src/DbPropSet.COMWrappers.cs

Co-authored-by: Eric Erhardt <eric.erhardt@microsoft.com>
* Restore changes in OnInfoMessage

* Consolidate different part of code

* Remove junk (probably from merge)

* Found that last commit was a mess

* Remove no longer needed annotations

* Apply suggestions from code review

Co-authored-by: Aaron Robinson <arobins@microsoft.com>
* Update src/libraries/System.Data.OleDb/src/OleDbComWrappers.cs

Co-authored-by: Aaron Robinson <arobins@microsoft.com>
* Free BSTR

* Remove reference to no longer used files in this PR

* Add data

* Fix compilation errors

* Last blow to compiler

Co-authored-by: Aaron Robinson <arobins@microsoft.com>
Co-authored-by: Viktor Hofer <viktor.hofer@microsoft.com>
Co-authored-by: Eric Erhardt <eric.erhardt@microsoft.com>

[HTTP/3] Fix NullReferenceException on Timeout when streaming request body (#59204)

Brings the logic for canceling streaming request body from HTTP/2 to HTTP/3.

The problem was that cancellation token was not applied to SendContentAsync, which resulted in Http3WriteStream not being disposed properly.

Fixes #57619

Fix issues where the module index GUID table isn't properly computed (#59375)

* Fix issues where the module index GUID table isn't properly computed
- Since there are two possible places where the module indices must be up to date, ensure that they are finalized before either Node writes its data
- Ensure that module indices are precomputed when needed from inlinee data and the InstanceEntryPointTableNode

[unix x86] Fix GCMemoryInfoData size assertion (#59417)

Update Unloadability documentation (#59383)

Fix gtCloneExpr when cloning during R2R compilation a GT_ALLOCOBJ node (#59395)

* Fix gtCloneExpr when cloning during R2R compilation a GT_ALLOCOBJ node
- Without this fix cloned expressions with allocations will fail
- This is most common in profile guided code around devirtualization, but I believe it can occur in other where gtCloneExpr is used
- Symptom of the failure is a compilation failure during crossgen2

[main] [Release/6.0]fix the package version for non-shipping csproj packages (#59388)

* fix the package version for non-shipping package

* use version prefix

Co-authored-by: Anirudh Agnihotry <anagniho@microsoft.com>

[x86/Linux] Fix SIGSEGV on Debugger.Break() during debugging. (#59350)

SIGSEGV occur in src/vm/i386/cgenx86.cpp `HelperMethodFrame::UpdateRegDisplay()` at line `pRD->pCurrentContext->Eip = pRD->ControlPC = m_MachState.GetRetAddr();`, since `pRD->pCurrentContext` is NULL.
The point of issue - wrong code block compilation in DebuggerWalkStack() in case of Linux x86.

<EnablePreviewFeatures> for HttpStress (#59410)

Check if External Object Context is still active after a possible GC (#59285)

* Handle a possible GC after allocating an array by iterating over the hash again.

Co-authored-by: Elinor Fung <elfung@microsoft.com>

Fix crossgen for FreeBSD by adding FreeBSD RID. (#59331)

* Fix crossgen for FreeBSD by adding FreeBSD RID.

Fix implemented as described by @Thefrank in this thread:
https://github.com/dotnet/runtime/issues/14537#issuecomment-922546527

* Add FreeBSD platform-support to crossgen2.

Allow contained indirections in tailcalls on x64 (#58686)

This adds support for contained indirections in tailcalls on x64. The
significant diff of this change is refactoring to be able to reuse the
code for generating call instructions when generating tailcalls as well.
Other than that, the main change is to allow contained indirs in
lowering and to ensure LSRA uses volatile registers for the addressing
mode so that the registers are not overridden by the epilog sequence.

To be sure we insert rex. prefix correctly I also refactored the emitter
to base the decision on a new instruction (INS_i_jmp) and emit it when
necessary. The rex. prefix is necessary because the unwinder uses it to
determine that a tail jmp is non-local and thus part of the epilog.

Finally, unlike the OS unwinder our unwinder needs to compute the size
of the epilog. This computation was wrong for jmp [foo] for addressing
modes/rip-relative addressing, so fix this as well. Presumably that has
not been a problem before since we did not generate these instructions
in managed code (although native code may have had these instructions --
not sure if we use that unwinder for native code).

Fix invalid parsing of not unicode Uri (#59252)

* Fix parsing of not unicode Uri (#56482).

* Changing test for reproducing problem with Uri creation by Unicode (#59252).

MmQuicListener Comment (#59365)

Fix building GC on Android (#58703)

[mono][interp] Don't assert when encountering invalid IL opcode (#59334)

* [mono][interp] Don't assert when encountering invalid IL opcode

Throw exception instead.

* [tests] Re-enable tests

[mono][wasm] Rework the handling of GC references in AOTed code. (#59352)

Previously, variables holding GC refs were marked volatile so they
were loaded/stored to the C stack on every access. Since the stack
is conservatively scanned, all the objects pointed to by it are pinned,
so there is no need to load them on every access. Instead of
marking them as volatile, allocate a 'gc pinning' area on the stack,
and store ref variables to it after they are assigned. This improves
code size and performance.

[wasm][debugger] Reverting the old behavior of scope id numeration (#59368)

* Keeping the old behavior of scope id what we have before start using debugger-agent.

* Fix compilation.

* Fix compilation.

Delete duplicate runtime tests (#58676)

Microsoft.VisualBasic.Core.Tests fail tests with NaN conversion (#59321)

* Microsoft.VisualBasic.Core.Tests fails tests with NaN conversion

* Update src/libraries/Microsoft.VisualBasic.Core/tests/SingleTypeTests.cs

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

Change logic to not break private runs (#59356)

The previous change here broke private runs as it required the channel map
to have an entry matching the name of the runtime branch you were testing.
This fix now defaults the channel to main for private runs.

Suppress SendPacketsElement_FileStreamMultiPartMixed_MultipleFileStreams_Success test on windows 11 (#59203)

Ignore MSBuild_Logs/ directory (#59323)

This is a new directory used by msbuild for logging since https://github.com/dotnet/msbuild/commit/cdb5077c451180ab38161e0b5e70f5448e70355b.

Remove some unnecessary allocations / copies on startup (#59270)

Delete dead code (#56865)

* Delete dead QMARK optimization

It is checking for both sides of GT_COLON being identical,
but QMARKs are only created in some very limited circumstances
today, and certainly not with identical then and else clauses.

No diffs.

* Delete dead code from fgSetTreeSeqHelper

QMARKs are rationalized before it is ever called.
Likewise, GT_INDs with "a special structure" are a thing of the past.

No diffs.

* Delete unused variable

* Delete unused impAssignSmallStructTypeToVar

* Delete unused gtIsVtableRef & getObjectHandleNodeFromAllocation

* Delete unused IsTreeAlwaysHoistable

* IsAddWithI32Const

* Delete unused Jit config values

* Delete unused fgEstimateCallStackSize

* Delete unused parameters in fgMorphStmts

* Delete unused StripExOp

Refactor subrange assertion generation and propagation (#55186)

* Refactor subrange generation

Move it into its own function.

No diffs.

* Stop generating assertions for TYP_U/INT ranges

They are not useful: they do not tell us anything about the value.
They can be harmful: code operating on the assertions can wrongly
remove a checked cast because of an assertion for a non-checked one.

* Add the header for optTryExtractSubrangeAssertion

* Refactor subrange assertions

This change completely refactors the way subrange assertions are
generated and consumed. It rationalizes and generalizes the model
for subrange assertions which are generated and propagated for casts,
makes AssertionDsc::Range into an actual abstraction, eliminates bugs
related to wrong assertions generated for TYP_BOOL (without CQ loss),
enables propagation for checked casts to non-small types both on 64
and 32 bit, all the while making the AssertionDsc structure smaller.

There are quite a few moving parts, but at the heart of this change
are following items:

1. Make ranges symbolic, i. e. made up of symbolic upper and lower
   bounds, represented by the new SymbolicIntegerValue enum class.
   This is necessary to not regress memory consumption and throughput
   of operations on 32 bit while handling long subranges.
2. Split the concept of "input" and "output" for a cast node.
   To properly handle checked cast and make the code "obviously correct",
   it is necessary to split the two values, one the cast node produces
   and one it consumes, apart. The old code only handles the "easy" case
   of the "input" and "output" ranges matching (such is the case for small
   types), the new code must be, and is, robust in reasoning about, e. g.
   CAST_OVF(uint <- ulong) vs CAST_OVF(int <- long). The ranges that
   represent these values are consistently in the "signed" format, making
   handling them easy.
3. Tweak the propagation code to not remove representation-changing casts,
   make it explicit and obvious in what cases (and why) it retypes the
   local under the cast.

As a side-effect of the above changes, TYP_BOOL is now treated as TYP_UBYTE
in propagation, as it should be. Actually, the old code didn't remove
redundant casts to TYP_BOOL, the new code does, which is also where the
bulk of the positive diffs come from.

The old code also generated "useless" ranges like [INT_MIN..INT_MAX], the new
code only generates ranges that could be used in subsequent propagation.

Also, some miscellaneous cleanup, such as adding function headers, was performed.

* Use correct ranges for global subrange assertions

In global assertion propagation, tentative assertions are
created for casts, with the expectation that proving them
via implication will allow for removing the cast.

The old code did this through the common path with local
assertions, however, this cannot be done the same way anymore
because the two types of assertions are fundamnetally different.

In local assertion propagation, subranges are derived from ASGs,
where the cast is on the RHS and the local - LHS, so the "output"
range for the cast must be used - local is assigned value the cast
**produces**.

In contrast, global subrange assertions say that the value the
cast **consumes** is within the correct bounds for the cast to
become a no-op or non-overflowing.

Because of this fundamnetal distinction, I chose to make global
assertions for casts in a new method instead of plumbing it
through "optCreateAssertion". Note that it would be possible to
do it via "optCreateAssertion" as well - global propagation
always passes OAK_SUBRANGE to it, while local - OAK_EQUAL, but
I decided against that option for the reason that it seems
unintuitive to me that it would work the way it would. These
tentative assertions are a very special case, so I think it is
better to treat them that way. This leads to some minor code
duplication, but I think that's ok.

No diffs for this commit, though that's expected as "proving"
the generated assertion is quite rare.

* Refactor AssertionDsc::Range into its own class

Make IntegralRange the owner of the logic for various
methods relating to it and decouple it from AssertionDsc.

IntegralRange will be useful in the future for other code
that deals with cast removal, e. g. in fgMorphCast.

Also some miscellaneous formatting fixups.

* Change the implementation of SymbolicToRealValue

Using a static array results in smaller and more
efficient code on all compilers.

* FP & GC castees in IntegralRange::ForCastOutput

This change is for completeness, since "fgMorphCast" decomposes
casts from FP types to small types today and as such assertions
are generated for them anyway.

No diffs for this commit as expected.

* Add the TYP_BOOL quirk

To avoid regressions.

* Fix the input ranges for non-overflowing casts

They did not handle "long" cases and as such were too conservative.

* Add missing function headers

* Remove a TODO now that BOX import has been fixed

* Use gtEffectiveVal & ChangeType to simplify code

* Fix build break

* Add a more detailed comment

* Tweak the comment

[mono][interp] Add checks for endfinally opcode (#59332)

* [interp] Add checks for endfinally opcode

Throw invalid IL if endfinally is not within a finally block.

* [tests] Re-enable test

Use correct NumberStyle when parsing double and float (#59205)

* User correct NumberStyle when parsing double and float

IParseable<float> and IParseable<double> would fail when parsing
a decimal point number due to incorrect NumberStyles parameter.

E.g. IParseable<double>.TryParse would fail as opposed to
IParseable<double>.Parse because the former explicitly states to use
NumberStyles.Integer

* Adding tests covering parse for float/double/half

* Delete MSBuild_pid-37116_6904092b1c9a481da960c9e28eba2a27.failure.txt

Co-authored-by: Tanner Gooding <tagoo@outlook.com>

[main] Update dependencies from 4 repositories (#58999)

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
Co-authored-by: Premek Vysoky <premek.vysoky@microsoft.com>

Use PowerShell in Android arm64 XHarness Helix payloads (#59340)

cmake list separator (#59317)

Bring XmlReader{Async} to one behavior (#51736)

* Replace XmlUntypedStringConverter to XmlUntypedConverter

* Add more test

* add more tests

* remove unssoprt test

* feedback review

Co-authored-by: kronic <kronic@softland.ru>

Fix indentation in api prososal template (#59318)

5 spaces were used when 4 is the convention.

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