Fix EnumMemberRefs always returning NULL (#54805)

[mono][debugger] Insert and search seq_point in the same MonoJitMemoryManager (#54757)

* [mono][debugger] Fixing side effect of #48483

When starting XAML Hot Reload, in transform.c the seq_point was saved using jit_mm_for_method: https://github.com/thaystg/runtime/blob/abccfadbf570033efee8ac9a6992f6f7ee51f474/src/mono/mono/mini/interp/transform.c#L9760
When trying to search was trying on get_default_jit_mm.

* Fixing everywhere that uses seq_points.

* Not used.

* Revert "Not used."

This reverts commit fd7f2061b40436300abdcc919705b56af53f4ca0.

* Revert "Fixing everywhere that uses seq_points."

This reverts commit 6faffb8c92a1c9be17d8c53d5f735075b9212220.

* Revert "[mono][debugger] Fixing side effect of #48483"

This reverts commit 44601d310bd5ca1929604af5e0b96436b416ced8.

* Always use get_default_jit_mm for seq_points.

* Adding a test case and fixing wasm debugging on ALC.

* Moving implementation to avoid changing line number

* Update src/mono/wasm/debugger/DebuggerTestSuite/Tests.cs

Co-authored-by: Larry Ewing <lewing@microsoft.com>
* Update src/mono/wasm/debugger/DebuggerTestSuite/Tests.cs

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

Avoid per-XmlSchemaValidator PositionInfo allocation (#54356)

It can easily use a static singleton, as the instance is immutable and exists purely to provide a "I don't have line info" / (0,0) value if anyone queries for position.

[mono] Intrinsify inequality comparision between runtime types (#54602)

This is needed because, when inlining op_Inequality, the information on whether the arguments are runtime types gets lost. We should track that information instead when inlining, but it is a more invasive change.

Use Empty instead of null for DCS deserializing namespace. (#54670)

Fix missing entrypoints and add build-time validation (#54785)

* Fix missing entrypoints and add build-time validation

* Fix missing include

* Update src/libraries/Native/Unix/System.Security.Cryptography.Native.Apple/CMakeLists.txt

Co-authored-by: Jan Kotas <jkotas@microsoft.com>

Do not timeout HostingListener when debugging (#54791)

Do not timeout waiting for the host to start when running with
the debugger attached.

Addresses dotnet/aspnetcore#33886.

Fix MacOS build on 11.x SDK and Catalyst build (#54506)

* Fix MacOS build on 11.x SDK and Catalyst build

The configure.cmake was not getting the minimum supported OS version because
it was being set via set_compile_options and the config functions can only
get options from CMAKE_XXX_FLAGS.

* Add comment explaining why we set the macOS options via CMAKE_XXX_FLAGS

Make DependentHandle public (#54246)

* Move DependentHandle to System.Runtime

* Update DependentHandle APIs to follow review

* Make DependentHandle type public

* Update DependentHandle on Mono runtime

* Add allocation checks to DependentHandle APIs

This avoids throwing ExecutionEngineException-s if one of the public APIs is called on a non-allocated DependentHandle instance

* Add more unit tests for new public DependentHandle APIs

* Add faster, unsafe internal APIs versions to DependentHandle

* Naming improvements to Ephemeron type

The ephemeron type is checked in the Mono runtime in "object.c" as follows:

m_class_get_image (klass) == mono_defaults.corlib && !strcmp ("Ephemeron", m_class_get_name (klass))

As such, the namespace it belongs to in the managed runtime doesn't matter: the VM will just check
that the type name matches, and that the type is in fact defined in corelib. This means we can
just move it to System.Runtime without worrying about it being properly managed in the VM.
Additionally, the type is defined in "sgen-mono.c" as follows:

typedef struct {
    GCObject* key;
    GCObject* value;
} Ephemeron;

So as long as the layout matches the one of the type defined in C# (which it does), we're also free
to rename the fields to better follow the naming guidelines, and the VM will have no issues with it.

* Code style tweaks, improved nullability annotations

* Remove incorrect DependentHandle comment on Mono

* Add default Dispose test for DependentHandle

Co-authored-by: Stephen Toub <stoub@microsoft.com>
* Fix race condition in DependentHandle on Mono

* Optimize DependentHandle.nGetPrimary on CoreCLR

Removed internal call, same optimization as GCHandle

* Small IL codegen improvement in DependentHandle.nGetPrimary

* Simplify comments, add #ifdef for using directive

* Minor code style tweaks

* Change nGetPrimaryAndSecondary to nGetSecondary

* Minor code refactoring to DependentHandle on Mono

* Rename DependentHandle FCalls

* Remove DependentHandle.UnsafeGetTargetAndDependent

* Remove DependentHandle.GetTargetAndDependent

* Fix FCall path for internal DependentHandle APIs

* Add more DependentHandle unit tests

* Reintroduce DependentHandle.GetTargetAndDependent()

This fixes a bug due to a race condition in ConditionalWeakTable<K, V>, which relies on this method which atomically retrieves both target and dependent with respect to target being set to null concurrently by other threads. This also exposes the same API publically to allow consumers to potentially implement custom conditional weak tables in the same manner.

* Minor IL tweaks to produce smaller IR in the JIT

* Add DependentHandle.StopTracking() API

This also fixes two potential GC holes when setting DependentHandle.Target (see conversation from https://github.com/dotnet/runtime/pull/54246#issuecomment-863285327 onwards)

* Rename InternalSetTarget to StopTracking, remove redundant param

* Remove FCUnique from InternalStopTracking

This was added in https://github.com/dotnet/runtime/pull/39810 to avoid a collision with MarshalNative::GCHandleInternalSet, as the two FCalls had identical implementations and their entry points were not unique. This should no longer be needed after 099fc478551f46cc54e7a18a32d9a9ac73727c73, as that changed both the signature and the implementation of this FCall.

* Update API surface to match approved specs from API review

* Update DependentHandle XML docs

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

More file stream options tests (#53982)

* add more tests

* StreamWriter and StreamReader require FileStreamOptions with necessary access

* allow for bufferSize == 0

* extend the test

* refactor the tests

* Revert "allow for bufferSize == 0"

This reverts commit f1259be18a3446c1b8939883f484cc28347c74cf.

Fix BigInteger hex string parsing bug (#54251) (#54262)

Fix WaitForNextTickAsync_CanceledWaitThenWaitAgain_Succeeds test (#54775)

There's a race condition in the test between the timer firing and cancellation being requested.  It repros more on Linux because there's a smaller quantum on Linux than on Windows.

Expose underlying unix file descriptor in SafeMemoryMappedFileHandle (#53538)

* expose unix file descriptor safe memory map

* store handle after DangerousAddRef

* add test verifying handle matches filestream handle

* Update src/libraries/System.IO.MemoryMappedFiles/src/Microsoft/Win32/SafeMemoryMappedFileHandle.Unix.cs

Co-authored-by: Stephen Toub <stoub@microsoft.com>
* Update src/libraries/System.IO.MemoryMappedFiles/src/Microsoft/Win32/SafeMemoryMappedFileHandle.Unix.cs

Co-authored-by: Stephen Toub <stoub@microsoft.com>
* Update src/libraries/System.IO.MemoryMappedFiles/src/Microsoft/Win32/SafeMemoryMappedFileHandle.Unix.cs

Co-authored-by: Stephen Toub <stoub@microsoft.com>
* Update src/libraries/System.IO.MemoryMappedFiles/src/Microsoft/Win32/SafeMemoryMappedFileHandle.Unix.cs

Co-authored-by: Stephen Toub <stoub@microsoft.com>
* Update src/libraries/System.IO.MemoryMappedFiles/tests/MemoryMappedFile.CreateFromFile.Tests.cs

Co-authored-by: Stephen Toub <stoub@microsoft.com>
* Dispose MemoryMappedFile in MapHandleMatchesFileStreamHandle test

* Update src/libraries/System.IO.MemoryMappedFiles/src/Microsoft/Win32/SafeMemoryMappedFileHandle.Unix.cs

Co-authored-by: Stephen Toub <stoub@microsoft.com>
* Update MemoryMappedFile.CreateFromFile.Tests.cs

* Move mmf in test to using block

* Fix test

Co-authored-by: Danny Friar <danny@Dannys-MacBook-Pro.local>
Co-authored-by: Stephen Toub <stoub@microsoft.com>

Use an untyped `Task` instead of a `Task<bool>` in `TimerQueueTimer`. (#54455)

add pre-cancellation check and enable ReadWriteAsync_PrecanceledOperations_ThrowsCancellationException for QUIC (#54540)

[interp] Don't cprop between vt vars of different sizes (#54734)

Via unsafe code, getting the lower Vector128 from a Vector256 ends up as a move of `sizeof (Vector128)` from a Vector256 var. However, the destination var is not a valid copy of the source var, having a different type

Revert "Fix 54025 (#54070)" (#54728)

This reverts commit b2fe6678282503374aeea7ab5f468f7a3cc85a2f.

Add metadatatoken override to SymbolMethod (#54656)

* Add metadatatoken override to SymbolMethod

* Add test

Co-authored-by: Jan Kotas <jkotas@microsoft.com>

minor optimization: move creation of X509 Subject Summary into catch block on MacOS. (#54774)

Extract fgMorph(Init/Copy)Block into their own classes. (#53983)

* Rewrite fgMorph(Copy/Init)Block.

* fix a last squash error.

* fix nits

Clean up some DI code (#54732)

- Remove empty CreateInstanceCallSite file
- Removed ServiceScopeFactoryCallsite and used ConstantCallsite instead.
- Added fast path if the value is already cached on the callsite.

Miscellaneous MetadataUpdater cleanups (#54751)

Move remote server redirect and decompression tests to HttpClientHandlerTest.RemoteServer.cs (#54738)

* move remote redirect and decompression tests to HttpClientHandler_RemoteServerTest

Co-authored-by: Geoffrey Kizer <geoffrek@windows.microsoft.com>

Fix gc hole in IO thread pool (#54769)

HTTP3: Re-enable cookie and cancellation tests (#54727)

* fix an issue with GOAWAY handling and enable cookie tests

* disable ConnectTimeout test for HTTP3

* fix Expect 100 continue handling in HTTP3

* add and use GenericLoopbackServer.SendPartialResponseHeadersAsync

* enable cancellation tests for HTTP3

* disable failing interop tests

Co-authored-by: Geoffrey Kizer <geoffrek@windows.microsoft.com>

Clean up JSON source gen APIs (#54527)

* Clean up JSON source gen APIs

* Address review feedback

* Simplify GenerateX computation

Fix handling of static virtual method implementation checking (#54710)

- It turns out that GetMethodDescFromMemberDefOrRefOrSpec and FindOrCreateAssociatedMethodDesc are not safe to use on a MemberRef whent  the associated MethodTable is not fully loaded.
- Instead only use that feature when working with a MethodDef or a fully loaded type, and when working with a not fully loaded type, use MemberLoader::FindMethod instead.
- When running the resolution algorithm for doing constraint validation, it also is not necessary to fully resolve to the exact correct MethodDesc, which as that process uses FindOrCreateAssociatedMethodDesc needs to be avoided.
- The above was not evident as in many cases, the validation algorithm did not run as it was misplaced and located directly before the call to SetIsFullyLoaded. That code path is only followed if the type is able to fully load without circular dependencies. (Test case CuriouslyRecurringGenericWithUnimplementedMethod added to cover that scenario)
- In addition, while investigating these issues, I realized we were lacking checks that the constraints on the impl and decl method were not checked at during type load, but that work was instead deferred to dispatch time. Along with the constraint check there was also a set of accessibility checks that had been missed that are common to all MethodImpl handling. Fix by adding tweaking the logic to share most of that code.

Fix thread safety of _typeRefsInCompilationModuleSet (#54760)

Tweak classcompat loader to skip MethodImpls associated with static methods (#54658)

Fix BackoutJitData (#54711)

* Fix BackoutJitData

The RemoveJitData that the BackoutJitData calls requires the code
header to be copied to the final location. This change fixes it.

I've also found that in one of my previous changes, I've accidentally
enabled jitting into a scratch buffer by default by adding the
FEATURE_WXORX define unconditionally. So I am removing it in this change
for non Apple Silicon, it will be replaced by a dynamic check whether W^X
is enabled in the final W^X change.

[mono][jit] Refactor the generic sharing code a bit. NFC. (#54705)

* [jit] Refactor the emit_rgctx code a bit.

* Add an enum to specify the way the rgctx is accessed from a gshared method.
Use it to simplify some logic.

Revert "[wasm] browser http response stream could be seekable (#54603)" (#54742)

This reverts commit c139d00640485e8c231c54d1573dc02d7c9e8daf.

[wasm] browser http response stream could be seekable (#54603)

- browser http response stream could be seekable
- test WebAssemblyEnableStreamingResponse

Override more Stream members on System.IO.Compression streams (#54518)

[main] Update dependencies from 9 repositories (#54543)

Fix CQ regression & correctness bug in morphing of long muls (#53566)

* Add a test covering GTF_MUL_64RSLT transform

* Disable the test on Mono

* Add genActualTypeIsInt helper

* Add some gtFlags helpers

* Prepare decomposition for new long muls

* Update gtIsValid64RsltMul

To understand the new format for long muls.

* Rework morphing of long muls

Previously, morph was looking for the exact pattern of
MUL(CAST(long <- int), CAST(long <- int)) when assessing
candidacy of GT_MUL for being marked with "GTF_MUL_64RSLT" and
emitted as "long mul".

This worked fine, until the importer was changed to fold
all casts with constant operands. This broke the pattern
matching and thus all MULs in the form of (long)value * 10
started being emitted as helper calls.

This change updates morph to understand the new folded casts
and in general updates the "format" of long mul from
"CAST * CAST" to "CAST * (CAST | CONST)".

In the process, new helper functions have been introduced, to
avoid bloating fgMorphSmpOp with the new sizeable logic.

Recognition of overflowing cases has been upgraded, and a correctness
bug, where "checked((long)uint.MaxValue * (long)uint.MaxValue)"
was wrongly treated as non-overflowing, fixed.

Additionally, the logic to emit intermediate NOPs has been
changed to instead always skip morphing the casts themselves,
even when remorphing.

* Add the script to generate the longmul test

The test itself has been regenerated using it and
there were no diffs, as expected.

Eliminate chained casts to small types (#52561)

We can take advantage of the implicit zero/sign-extension for
small integer types and eliminate some casts.

Optimize `CAST(int <- long)` on 32 bit targets (#53040)

* Optimize CAST(int <- long) on 32 bit targets

* Revert "Optimize CAST(int <- long) on 32 bit targets"

Revert the implementation in lowering

* Optimize CAST(int <- long) on 32 bit targets

Move the code from lowering to long decomposition.

* Fixed the "Arguments" note for DecomposeNode

* Added the function header

For OptimizeCastFromDecomposedLong.

* Remove the TODO comment

While correct, it has questionable value.

* Add a more detailed dump output

* Do not try to optimize checked casts

It is easy to get it wrong. Let the frontend handle this.

* Do not depend on tree order

Previous version of the code assumed that there could be
no nodes between the cast and its operand. That is not
a correct assumption to make in LIR.

Move the metadata update related APIs to the MetadataUpdater class (#54590)

Move the metadata update related APIs to the MetadataUpdater class

The old APIs AssemblyExtensions.ApplyUpdate() and AssemblyExtensions.GetApplyUpdateCapabilities() will be removed
after all the references to them have been changed to the new APIs.

Add the new IsSupported API described in issue https://github.com/dotnet/runtime/issues/51159.

Change the tests to use the MetadataUpdater APIs.

Fix the ApplyUpdate qcalls and icalls

Add the ILLink substitutions for MetadataUpdater.IsSupported property

Change the old APIs to call the new ones

Update mono's MetadataUpdater.IsSupported property

Update feature switch doc

Fixed the argument checking in coreclr's MetadataUpdater.ApplyUpdate().

S.N.Quic made private (#54610)

Note this will need reaction on ASP.NET side.

Disallow promotion of HVA structs when their fields of TYP_SIMD8 were promoted as plain structs (#54694)

[wasm][debugger] Detect initial status of pause on exceptions. (#54040)

* Detect initial status of pause on exceptions.

* Changing what @radical suggested.

* Changing more things.

* Test case created.
I could not test the pause on "all" exceptions because if I enable the pause on caught exceptions and reload the page it will stop in a lot of exceptions other then the one that I inserted in AttachToTarget.

* Adding a test for Reload page with ALL set.

* Fixing merge conflicts.

* setting icordebug = false.

* Removing unrelated change.

defMAC construction up a scope to make it live long enough. (#54702)

Fixes #54649

Fix MonoCrossAOT.UnixFilePermissions for wasm (#54693)

* Fix MonoCrossAOT.UnixFilePermissions for wasm

We introduced Microsoft.NETCore.App.MonoCrossAOT.UnixFilePermissions.xml in https://github.com/dotnet/runtime/pull/54501 to make sure the right permissions are set when installing AOT compiler workload packs.  We hardcoded the list to include mono-aot-cross, llc, and opt.  However, in wasm's case they only have mono-aot-cross.

This change makes the xml file a template and only includes mono-aot-cross for browser and all three for the other configurations.

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

* Use property instead of item for the file nodes

* Ankit suggestion

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

Mark AssemblyDependencyResolver as unsupported on Browser and mobile (#54601)

* Mark AssemblyDependencyResolver as unsupported on Browser and mobile

* Apply AlekseyK's workaround to fix android CI builds

* Add an annotation for maccatalyst

[mono] Implement Rdm and Dp (#49737)

See https://github.com/dotnet/runtime/issues/42322 and
https://github.com/dotnet/runtime/issues/42280.

Tested manually on an arm64 Linux VM running on an M1 Mac Mini. Does not
enable RDM or DP when AOT-compiling the associated runtime tests; our CI
hardware doesn't support these extensions yet.

Pass the default ALC gchandle to the v3 preload hook (#54686)

* Pass the default ALC gchandle to the v3 preload hook

Fixes crashes early during startup when the gchandle is then used by the hooks
to call mono API functions.  The default ALC exists, but its managed object
doesn't, so the gchandle target is null.  The mono APIs detect the the special
gchandle and resolve it to the default ALC

Example crash
https://gist.github.com/grendello/b4ab24587a055725cc5e1416b86ad7ca

* [alc] Assert that we never see a null target from a managed ALC gchandle

Add ActivitySource support to DiagnosticsHandler (#54437)

* Add ActivitySource support to DiagnosticsHandler

* Use ActivitySource as another enabled condition

* Make IsGloballyEnabled a property

* Simplify enabled check

* Revert using the exception filter

* Update HTTP ILLink.Substitutions.xml

Add CustomAttributes ApplyUpdate Capability (#54619)

* Add CustomAttributes ApplyUpdate Capability

* Rename CustomAttributes ChangeCustomAttributes per #54284

Support synchronous Stream-based serialization methods (#54632)

Add args descriptions for VNF_MapStore and VNF_MapSelect (#54108)

Ensure relocation blocks are 4-byte aligned (#54668)

[mono][wasm] Reenable the remove finally pass. (#54450)

* [mono][wasm] Reenable the remove finally pass.

* [mono][wasm] Run methods which need stack walks using the interpreter instead of pushing an LMF.

* Disable vtype based sharing for vtypes with explicit sizes.

Remove CoffeeFlux from CODEOWNERS (#54696)

[wasm][debugger] Fixing debugger tests errors (#54664)

* Fixing new debugger tests errors.
Errno is not being assigned anymore.

DebuggerTests.SetVariableValueTests

* Passing icordbg flag as false.

* Adding new tests.

Add non-LLVM iOS size scenario (#54585)

* Add non-LLVM iOS size scenario

* add category

* remove parens

* variables in the wrong place.

* move setting configs to before it is written

Fix instruction hex display (#54675)

[QUIC] Add Windows version check to QUIC initialization (#54488)

Add Windows version check to QUIC initialization, log min supported and current Windows version

Fix Activator.CreateInstance(GetType(), ...) related ILLink warnings in (#54680)

System.Data.Common

Disable MacCatalyst arm64 PR test runs on staging pipeline (#54678)

We don't have enough capacity right now on Helix to handle the load.

[WASM] Fix async/await in config loading (#54652)

* Fixed config issue

* Updated Hot Reload test

Fix for heap_use_after_free flagged by sanitizer (#54679)

Copy of https://github.com/mono/mono/pull/21120

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

[wasm] Bump emscripten to 2.0.23 (#53603)

Bumps emscripten to 2.0.23

The Browser AOT tests now use `-Wl,-lto-O0` option to reduce memory usage of `wasm-ld` tool, which was in some cases going over avaiable 8GB on helix machines.

* Revert "Add ActiveIssue to the MemoryMappedFiles tests"

This reverts commit ec1ae530606ef1061680600fc046226cc1c4cbc3.

* Revert "Add ActiveIssue attr to the FileSystem tests"

This reverts commit 356b3ff2a703980ac01b9df697a594e8c341c436.

* Bump emscripten version to 2.0.23

* Use newer docker images with 2.0.23

* Update docs

* Use 2.0.23 emscripten nuget packages

* Revert "Revert "Add ActiveIssue attr to the FileSystem tests""

This reverts commit eb2f9548b08c114b359fab8d867ba50de098fe48.

The fix is not present in 2.0.23

* Revert "Revert "Add ActiveIssue to the MemoryMappedFiles tests""

This reverts commit 8be39f583499a8d8451034c65260a785330b0795.

The fix is not present in 2.0.23

* Increase timeout for AOT tests

* Add description of emscripten bump to README

* Try to get information about resources

* Get all limits

* Escape & chars

* Reduce platform matrix

* Lets try one more build with doubled timeout

* Revert "Lets try one more build with doubled timeout"

This reverts commit 67dd7754bb79218b2c6b687034162d041715093e.

* Try -Wl,-O0 on CI

To be sure it behaves the same as in local build

* Use -Wl,-lto-O0 do lower link time optimization

It looks like it reduces the memory load a lot

* Set EmccLinkOptimizationFlag for AOT tests

And reset the default value

* Escape commas

* Revert "Reduce platform matrix"

This reverts commit fec0e557208eb165824e75cd57b895a74d164de4.

* Remove resource info retrieval

* Bump emsdk versions

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

Fix compiler references when building inside VS (#54614)

If for a source project a contract project exists, then the contract project's TargetPath should be passed to the compiler. This is handled by the SDK by default when `ProduceReferenceAssembly` is true. As dotnet/runtime doesn't use the `ProduceReferenceAssembly` feature yet, a custom target adds the necessary `ReferenceAssembly` metadata to the `TargetPathWithTargetPlatformMoniker` item which then is transformed to references for the compiler. That works fine on the CLI as the `GetTargetPathWithTargetPlatformMoniker` target runs after the ProjectReference to the ContractProject is resolved and its target path is available.
Inside VS the target ordering is different and the `ResolvedMatchingContract` item was empty as the ProjectReference to the contract wasn't yet resolved. The fix for that is to add a dependency onto the `ResolveProjectReferences` target to guarantee that the `ResolvedMatchingContract` item is populated in time.

Noticed this when the build of System.ComponentModel.Composition.Registration failed because the implementation assembly of System.ComponentModel.Composition was passed to the compiler instead of the reference assembly.

process more TLS frames at one when available (#50815)

* process more TLS frames when available

* add SslStream.Implementation.cs

* remove extra comment

* add back DefaultRequestHeaders_SentUnparsed

* feedback from review

* fix winhttp

* fix linker test

* feedback from review

* feedback from review

* feedback from review

* make EnsureFullTlsFrame async

* feedback from review

Add PeriodicTimer (#53899)

UdpClient with span support (#53429)

Add API from #864

exclude fragile tests (#54671)

get last error before calling a method that might fail as well (#54667)

[FileStream] add tests for device and UNC paths (#54545)

* add a test for unseekable device by using a path to named pipe

* add a test for seekable device by using DeviceID instead of drive letter

* add a test for a UNC file path (local file share)

Fix sporadic double fd close (#54660)

Fix https://github.com/dotnet/runtime/issues/54589

Remove Version.Clone from AssemblyName.Clone (#54621)

Version is immutable.

[wasm] Enable fixed libraries tests (#54641)

Fixed by https://github.com/dotnet/runtime/pull/52705 and
https://github.com/dotnet/runtime/pull/52707

[wasm] Fix blazor/aot builds (#54651)

`dotnet\packs\Microsoft.NET.Runtime.WebAssembly.Sdk\6.0.0-preview.7.21321.15\Sdk\WasmApp.Native.targets(342,5): error : Could not find AOT cross compiler at $(_MonoAotCrossCompilerPath)=`

Make sure this is set for the aot path.

[mono][wasm] Fix compilation error on wasm (#54659)

Fix telemetry for Socket connects to Dns endpoints (#54071)

[wasm] Build static components; include hot_reload in runtime (#54568)

* [wasm] Build static components; include hot_reload in runtime

   Workaround until https://github.com/dotnet/runtime/issues/54565 is fixed
   Build the runtime always with support for hot_reload, and without diagnostics_tracing

* Update wasm.proj

* Add a browser functional test for hot reload

   Just check that the capabilities are non-empty which is a good proxy for hot reload being enabled in the runtime.

* Turn off trimming for hot reload functional test

* Disable test on browser AOT

* fix whitespace

Co-authored-by: Thays Grazia <thaystg@gmail.com>

[wasm][debugger] Reuse debugger-agent on wasm debugger (#52300)

* Trying to reuse debugger-agent on wasm debugger. This will remove a lot of code that does the same thing on mini-wasm-debugger.

* Replace remove_breakpoint and clear_all_breakpoints with the ones on debugger-agent and remove unused code.

* Stepping and callstack using debugger-agent.

* Remove more code.

* Get frame values using debugger-agent.

* Working valuetypes and call function on valuetypes.
make -C src/mono/wasm/ run-debugger-tests TEST_FILTER=DebuggerTests.SteppingTests.InspectValueTypeMethodArgsWhileStepping is working without use_cfo.

* Failed:   316, Passed:   175

* Failed:   307, Passed:   184, Skipped:     0, Total:   491

* Failed:   280, Passed:   211

* Failed:   277, Passed:   214

* Implemented boxed value.
Failed:   271, Passed:   220

* Implementing get properties on objects.
Implementing handling error on debugger-agent.

* Implementing callfunctionon object.
Failed:   248, Passed:   243

* Implementing get pointer values.
Failed:   243, Passed:   248

* Fixing pointer values and implement call on function with pointers.
Failed:   226, Passed:   265

* Reimplement call function on, and implement set values.
Failed:   192, Passed:   299

* Failed:   192, Passed:   299

* Fixing valuetype with null values.

Failed:   184, Passed:   307

* Implemented Evaluate expressions, conditional breakpoints, all breakpoints tests are passing.
Failed:   172, Passed:   319

* Fixing evaluate with value type.
Failed:   156, Passed:   335

* Trim part and add cache.
Failed:   148, Passed:   343

* Fixing evaluate expression.
Failed:    99, Passed:   392

* GetPropertiesTests working.

Failed:    53, Passed:   438

* Passing delegate tests.
Failed:    31, Passed:   460

* Removing unused code.

* Implementing exception handler.
Failed:    30, Passed:   461

* Fixing cfo returning array.
Removing more code.
Removing 2 tests that tests functions which does not exist anymore.
Failed:    18, Passed:   471

* Fix CallFunctionOn returning primitive types and null.

Failed:     9, Passed:   480

* Failed:     7, Passed:   482

* Fixing some tests.
Failed:     2, Passed:   488

* Removing a lot of code.
Failed:     4, Passed:   485

* 0 ERRORS!

* Removing more code.
No errors.

* Fixing added tests.

* Return javascript callstack after managed callstack.
Step out from managed code return to native wasm or javascript.
Adding debug info to Wasm.Browser.Sample to help testing debugger with sample.

* Change what Ankit suggested.
Clear cache with valuetypes and pointers after resume or step.

* Fixing suggestions.

* Fix error on wasm build.

* Apply suggestions from code review

Co-authored-by: Larry Ewing <lewing@microsoft.com>
* Changing what was suggested by @lewing.

* Fix pointer tests.

* Refactoring CreateJObjectForVariableValue

* Changing what @lewing suggested.

* Apply suggestions from code review

Co-authored-by: Larry Ewing <lewing@microsoft.com>
* Update src/mono/wasm/debugger/BrowserDebugProxy/MonoProxy.cs

Co-authored-by: Larry Ewing <lewing@microsoft.com>
* Update src/mono/wasm/debugger/BrowserDebugProxy/MonoSDBHelper.cs

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

* Trying to fix CI.

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

Put Crossgen2 in sync with #54235 (#54438)

Move System.Object serialization to ObjectConverter (#54436)

* move System.Object serialization to ObjectConverter

* simply customized object converter test

Move setting fHasVirtualStaticMethods out of sanity check section (#54574)

[wasm] Compile .bc->.o in parallel, before passing to the linker (#54053)

Change PathInternal.IsCaseSensitive to a constant (#54340)

* Return constants in PathInternal.GetIsCaseSensitive() on mobile platforms

   In particular on Android probing using I/O is slow and contributes to slow app startup.

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

* Implement Path.IsCaseSensitive as PathInternal.IsCaseSensitive

   Also Path.StringComparison => PathInternal.StringComparison

* Add test for PathInternal.IsCaseSensitive

   Move GetIsCaseSensitiveByProbing to FileSystemTest

* Drop PathInternal.s_isCaseSensitive cache field

* Delete Path.IsCaseSensitive and Path.StringComparison

   update callers to use PathInternal.IsCaseSensitive and PathInternal.StringComparison

* Remove catch clause from GetIsCaseSensitiveByProbing

* Mark new test [OuterLoop]

* Apply suggestions from code review

Co-authored-by: Stephen Toub <stoub@microsoft.com>
Co-authored-by: Jan Kotas <jkotas@microsoft.com>
Co-authored-by: Adam Sitnik <adam.sitnik@gmail.com>
Co-authored-by: Stephen Toub <stoub@microsoft.com>

Make mono_polling_required a public symbol (#54592)

Resolves build failure with the Android aot+llvm functional test

Respect EventSource::IsSupported setting in more codepaths (#51977)

Root ComActivator for hosting (#54524)

* rooting ComActivator that is needed for hosting

* FB

Add ILLink annotations to S.D.Common related to DbConnectionStringBuilder (#54280)

* Add ILLink annotations to S.D.Common related to DbConnectionStringBuilder

* address some feedback

* Make GetEvents() safe

* make GetProperties safe

* Mark GetProperties with RUC

Fix finalizer issue with regions (#54550)

This fixes an issue in Server GC where an item in the finalizer queue became stale due to not being relocated.

The problem was that a finalizable object was allocated on one heap, but registered in the finalizer queue of another heap (this is possible due to heap balancing). In CFinalize::UpdatePromotedGenerations, we ask for the generation of an object, and move the object to the correct section of the finalizer queue. In the error case, we obtained the wrong result for the generation of the object because it lived on another heap, and that heap hadn't set the final generation for the region containing the object yet. So we ended up moving the finalizer entry to the section corresponding to gen 2, and missed a relocation of the object occurring in a gen 1 collection afterwards.

Fix: It seems best to make sure an object is always registered for finalization on the heap it's allocated from, so the fix simply fetches the heap from the alloc context after the allocation in the case of SOH, or determines it by calling gc_heap::heap_of in the case of LOH and POH. In the case of SOH, I added an assert to ensure that the heap obtained agrees with the result of calling gc_heap::heap_of.

I also added some dprintf calls to the finalizer logic to aid in future investigations.

Add support for multi-arch install locations (#53763)

* Add support for multiple architectures inside install_locations

* Add install_location tests

* Fallback to DOTNET_ROOT on win32

Update library testing docs page to reduce confusion (#54324)

* Add warning on unmaintained testing doc page

* Update testing.md

Some example text that seems more clear to me, but only offered as a suggestion.
Feel free to adjust it or if you want to use it as-is please double check what I wrote is accurate : )

I think the useful elements are:
1. Being explicit about what workflow steps need to happen in total
2. Being explicit about which commands are covering the entire workflow and which ones are only covering a part of it
3. Show the simple "do-it-all" options first before showing more complex partial options. Glancing at the first example and blindly copying it should land in the pit of success.

Co-authored-by: Viktor Hofer <viktor.hofer@microsoft.com>

[FileStream] handle UNC and device paths (#54483)

* stop using NtCreateFile as there is no public and reliable way of mapping DOS to NT paths

Update NetAnalyzers version (#54511)

* Update NetAnalyzers version

* Add NetAnalyzers to dependency flow

Co-authored-by: Viktor Hofer <viktor.hofer@microsoft.com>

Added runtime dependency to fix the intermittent test failures (#54587)

* Added runtime dependency to hopefully fix the intermittent test failures

* addressed comments

* cleanup

* cleanup accidental spaces and tabs cleanup

* added Larry's comments

Disable failing System.Reflection.Tests.ModuleTests.GetMethods (#54564)

Tracking: https://github.com/dotnet/runtime/issues/50831

[wasm] Move AOT builds from `runtime-staging` to `runtime` (#54577)

These builds have had ~2-3 failures in the last 14 days (~90 builds).

Keep obj node for ArrayIndex. (#54584)

Disable another failing MemoryCache test (#54578)

Set DLL flag on R2R binaries (#54533)

This is required for R2R relocations to be processed by the OS loader on Windows 7.

Add `SkipEnabledCheck` on `LoggerMessageAttribute` (#54305)

* Add SkipEnabledCheck on LoggerMessageAttribute
* Make logging generator more robust with null input
* Adds LoggerMessageAttribute ctor overload
* properly identify misconfigured input

Improve performance of IsRootScope check (#54555)

- Stash a field instead of doing an equality comparison.

Fixes #54351