Release the stream once the response is sent (#90722)

Co-authored-by: ManickaP <mapichov@microsoft.com>

Merge pull request #90668 from dotnet-maestro-bot/merge/release/8.0-rc1-to-release/8.0

[automated] Merge branch 'release/8.0-rc1' => 'release/8.0'

Update dependencies from https://github.com/dotnet/arcade build 20230815.4 (#90665)

Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.Build.Tasks.Archives , Microsoft.DotNet.Build.Tasks.Feed , Microsoft.DotNet.Build.Tasks.Installers , Microsoft.DotNet.Build.Tasks.Packaging , Microsoft.DotNet.Build.Tasks.TargetFramework , Microsoft.DotNet.Build.Tasks.Templating , Microsoft.DotNet.Build.Tasks.Workloads , Microsoft.DotNet.CodeAnalysis , Microsoft.DotNet.GenAPI , Microsoft.DotNet.GenFacades , Microsoft.DotNet.Helix.Sdk , Microsoft.DotNet.PackageTesting , Microsoft.DotNet.RemoteExecutor , Microsoft.DotNet.SharedFramework.Sdk , Microsoft.DotNet.VersionTools.Tasks , Microsoft.DotNet.XUnitConsoleRunner , Microsoft.DotNet.XUnitExtensions
 From Version 8.0.0-beta.23411.1 -> To Version 8.0.0-beta.23415.4

Dependency coherency updates

Microsoft.DotNet.XliffTasks
 From Version 1.0.0-beta.23408.1 -> To Version 1.0.0-beta.23415.1 (parent: Microsoft.DotNet.Arcade.Sdk

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
Co-authored-by: Carlos Sánchez López <1175054+carlossanlop@users.noreply.github.com>

Do not set fetch depth on source-build (#90704)

Co-authored-by: Matt Mitchell <mmitche@microsoft.com>

Do not set fetch depth on source-build (#90703)

[release/8.0] Add doc for the new invoker classes (#90636)

* Add doc new invoker classes

* Fix cref typo

* Misc errors

* Remove arg reference in 0-arg ctor summary

---------

Co-authored-by: Steve Harter <steveharter@users.noreply.github.com>

[mono] Add a --path command line argument as an alternative to MONO_PATH. (#90678)

Can be specified multiple times.

Co-authored-by: Zoltan Varga <vargaz@gmail.com>

[release/8.0-rc1] Bump package version for net7 to 7.0.11 (#90651)

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

[release/8.0-rc1] [mono] Fix InlineArray tests on android platforms (#90679)

* Enable inline array tests

* Run android jobs only

* Add verbose logging

* Revert runtime changes

* Remove android-arm64 job

* Rename common assembly and revert CI changes

* Rename assembly name

---------

Co-authored-by: Milos Kotlar <kotlarmilos@gmail.com>

Fix gcroot SOS command on arm/arm64 (#90658)

Faulted in DAC because the HelperMethodFrame's REGDISPLAY CurrentContextPointers were not initialized correctly.

Fixes issue https://github.com/dotnet/diagnostics/issues/3726

Co-authored-by: Mike McLaughlin <mikem@microsoft.com>

[release/8.0-rc1] Add missing type forwards (#90673)

* Add missing type forwards

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

IDispatchImplAttribute, IDispatchImplType and SetWin32ContextInIDispatchAttribute were removed with https://github.com/dotnet/runtime/commit/9f1dd1aa499a882453b34d4e8810626423fbfed8 and https://github.com/dotnet/runtime/commit/26a91ad6ac02393c6c7463500c030ec07803c5e6

Those dropped types weren't flagged because APICompat only validates reference assemblies. We have three implementation only shim assemblies: mscorlib, System and System.Data. I verified that no other type forwards were lost between .NET 7 and .NET 8.

* Update mscorlib.cs

---------

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

Fix NullableAttribute illink test failures (#90680)

When we started building with preview 7 in
5549f72da3b3cea9e74bb81cfbb0f2d27731fc42, NullableAttribute in
these testcases started to use the attribute definition from the
framework, instead of generating it into the code. This broke the
`--used-attrs-only` optimization because `skip` assemblies (the
default for the framework in these testcases) are treated as if
all types in them are kept, for the purposes of the
`--used-attrs-only` optimization. (The optimization removes
attribute instances unless the attribute type is preserved for
some other reason).

It's not clear what the intended behavior of `--used-attrs-only`
is for `skip` assemblies, and the discussion in
https://github.com/dotnet/linker/issues/952 indicates that it's
considered experimental, so this fixes the failures by using the
`link` action. This represents a more realistic scenario since
`skip` is mainly used in testing to avoid linking the framework
in every testcase.

Co-authored-by: Sven Boemer <sbomer@gmail.com>

Fix problems with `Assembler::ReflectionNotation()` (#90655)

Do more parameter validation, correct buffer size calculations
for security-enhanced CRT functions, and avoid memcpy
buffer overrun of locale.

Co-authored-by: Bruce Forstall <brucefo@microsoft.com>

[ppc64le] performance improvements while branching (#90656)

[release/8.0-rc1] Unify with logging category matching (#90642)

* Unify with logging category matching

* -1 => 0

* Weird overlapping match

---------

Co-authored-by: Chris R <Tratcher@outlook.com>

Merge pull request #90628 from dotnet-maestro-bot/merge/release/8.0-rc1-to-release/8.0

[automated] Merge branch 'release/8.0-rc1' => 'release/8.0'

[release/8.0-rc1] Annotate System.Linq.Expressions with RequiresDynamicCode (#90616)

* Annotate System.Linq.Expressions with RequiresDynamicCode

All this ended up with an RUC on Expression.Compile due to new arrays.
I could potentially silence this warning with a feature flag, but it is a
real risk, and one that users could maybe work around if alterted to the problem.

* Remove suppression that's no longer needed

* Fix tests for new feature switch behavior

* Update src/libraries/System.Linq.Expressions/src/System/Linq/Expressions/Expression.cs

Co-authored-by: Eric Erhardt <eric.erhardt@microsoft.com>
* Typo in Throws call

* Fix assert for net framework

* Respond to PR comments

* Add suppression

* Typo

* Update src/libraries/System.Linq.Expressions/src/Resources/Strings.resx

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

---------

Co-authored-by: Andy Gocke <andy@commentout.net>
Co-authored-by: Andy Gocke <angocke@microsoft.com>
Co-authored-by: Eric Erhardt <eric.erhardt@microsoft.com>
Co-authored-by: Stephen Toub <stoub@microsoft.com>

[release/8.0-rc1] Fix support for non-public default constructors using JsonIncludeAttribute (#90615)

* Fix support for non-public constructors using JsonIncludeAttribute

* Address feedback.

---------

Co-authored-by: Eirik Tsarpalis <eirik.tsarpalis@gmail.com>

[release/8.0] Return false from ComWrappers.Try... methods (#90635)

* Return false from ComWrappers.Try... methods

Return false from ComWrappers.TryGetComInstance/TryGetObject instead of
throwing PNSE. It saves callers from needing to protect against PNSE.

Fix #90311

* More efficient IsWindows check

---------

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

Update dependencies from https://github.com/dotnet/emsdk build 20230815.5 (#90649)

Microsoft.NET.Workload.Emscripten.Current.Manifest-8.0.100.Transport
 From Version 8.0.0-rc.1.23411.2 -> To Version 8.0.0-rc.1.23415.5

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

Merge branch 'release/8.0' into merge/release/8.0-rc1-to-release/8.0

[release/8.0-rc1] [workloads] Put quotes around GetFileName call when collecting telemetry (#90606)

* [workloads] Put quotes around GetFileName call when collecting telemetry

There are cases where `%(ReferencePath)` is empty, so prevent exceptions by quoting the value.

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

* Feedback

---------

Co-authored-by: Steve Pfister <steve.pfister@microsoft.com>
Co-authored-by: Steve Pfister <steveisok@users.noreply.github.com>

[release/8.0-rc1] [mono][ios] Revert LLVM mode for apple mobile tests (#90624)

* Revert LLVM mode for apple mobile tests

* Add comment

---------

Co-authored-by: Milos Kotlar <kotlarmilos@gmail.com>

[wasm] Pin chrome version for testing to 115.0.579.x (#90644)

Hybrid globalization tests are breaking with 116.0.5845.0

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

Fix condition on mono-wasm workloads (#90646)

Co-authored-by: Juan Sebastian Hoyos Ayala <juan.hoyos@microsoft.com>

[release/8.0-rc1] Setup internal feeds in internal builds (#90638)

Co-authored-by: Juan Sebastian Hoyos Ayala <juan.hoyos@microsoft.com>

[release/8.0-rc1] Fix ToFrozenDictionary(selectors) on empty sources (#90599)

* Fix ToFrozenDictionary(selectors) on empty sources

As part of optimizing construction, the empty check when using ToFrozenDictionary taking selector delegates was skipped, leading to later failures when an empty source is used.

* Address PR feedback

---------

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

[release/8.0] Bump to RC2 (#90557)

* [release/8.0] Bump to RC2

Bump the base release/8.0 branch to target RC2.

* Set prerelease to 0 in configureplatform.make

[NativeAOT] Natvis visualization for threadstatic variables. (#90473)

* Slow path

* keep an index node for every type

* encode the offset

* Support inline storage in natvis

* store typeManager in TLS root

* test for module

* root param

* tweak comments.

* make junk MOV`s unreachable dead code

* Keep index nodes for inlined threadstatics unconditionally, remove "deadcode MOV" hack

* Update src/coreclr/nativeaot/Runtime/thread.h

---------

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

Fix VersionWithType Crossgen2 check for parameterized types (#90501)

As Egor Bogatov discovered in

https://github.com/dotnet/runtime/pull/90441

Crossgen2 is unable to compile a method taking RuntimeTypeMethod[]
parameter because Crossgen2 decides that the array doesn't version
with the module being compiled (System.Private.CoreLib in this
case). This change modifies the relevant logic so that, for
parameterized types, we decide on versioning based on the
parameter type.

Thanks

Tomas

support SslStream SSLKEYLOGFILE in debug builds (#90429)

* support SslStream SSLKEYLOGFILE in debug builds

* compat

* feedback

HttpClientFactory Prevent throwing if ILogger couldn't be created (#90503)

* HttpClientFactory stop cleanup timer on dispose

* Remove finalizer

* Tweak ThrowIfDisposed

* Revert some changes

* Fix

Release setup (#90529)

* Remove schedule to move us back to rolling

* Add rc1 explict branch name

* Comment out section

improve handling of LLA in Quic (#90455)

* QUIC LLA

* update

* linux

* line

* update

* Update src/libraries/Common/src/System/Net/IPEndPointExtensions.cs

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

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

Rename GeneratedClearManagedSource to correct name (#90537)

DCS work nicely with collectible AssemblyLoadContext (#90437)

* Add tests to check for ALC goodness with DCS.

* Add context-aware collections for caching/indexing DataContracts.

* Refine the context-aware collection types.

* Use lock to prevent running expensive delegate more than once.

* Use double-check pattern to prevent 'create' delegate from running multiple times.

* Upon further review, GetValue/GetOrAdd already do this check before running the 'create' delegate. Docs suggest this is intentional and will remain the case.

[libs][mono] Prevent static constructor from referencing `Internal.Runtime.Augments.DynamicDelegateAugments` in build scenarios without linking (#90519)

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

---------

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

Do not OCSP staple invalid OCSP responses

[Blazor] Custom ICU loading from runtime path (#90236)

* Loading a user-created custom file from absolute path.
* Issue 2 solved: files from runtime pack work when abs path is used.
* Cover not found file scenario.

* Appied @maraf's suggestion.
Co-authored-by: Ankit Jain <radical@gmail.com>

Removed redist entries from PlatformManifestFileEntry (#90542)

Put HasNativeCodeReJITAware into GetFunctionAddress (#90049)

* Remove calls to GetFunctionAddress with Null address and clarify method names

[mono][debugger]Adding more information about the array type (#89304)

* Adding more information about the array type.

* Adding more changes

* Update src/mono/mono/component/debugger-agent.c

* Only add this info for icordbg.

---------

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

Make TimeProvider Delay Continuations Synchronously (#90489)

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

Use '==' instead of 'Equals()' to test type equality in ServiceIdentifier (#90334)

Update DNNE version (#90524)

* Update Versions.props

Revert "Fix perf jobs param (#90215)" (#90535)

This reverts commit df5f3e6ea27bcab2c52fadf71e2961de15b9d17c.

[wasm][debugger] Support calling `get_Item` passing decimal, float or double as parameter. (#90260)

* Support calling get_Item using decimal, float and double as parameter.

* Update src/mono/wasm/debugger/BrowserDebugProxy/SignatureTypeProvider.cs

Co-authored-by: Ankit Jain <radical@gmail.com>
* Addressing some comments.

* fix compilation

* Fix wrong cast.

---------

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

[wasm][debugger] Do not run the test with justMyCode enabled in debug mode. (#90532)

* Do not run the test with justMyCode enabled in debug mode.

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

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

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

[wasm] Respect `AppendRuntimeIdentifierToOutputPath`, and `UseArtifactsOutput` (#90365)

* [wasm] Remove all instances of `ConditionalTheory IsUsingWorkloads`

.. because we run tests in two modes:

1. with workloads - all the tests *without* `no-workload` category are
   run (thus the default)
2. no workloads - all the tests *with* `no-workload` category are run
   (thus need explicitly attributes)

* Allow custom bin framework dirs for asserting bundles

* [wasm] Respect `AppendRuntimeIdentifierToOutputPath`, and

.. `UseArtifactsOutput`.

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

Fix deadlock in EventPipeEventDispatcher (#90053)

[mono] Enable callconv and classloader runtime tests (#90421)

* Enable callconv and classloader runtime tests

* Don't remove merged tests for apple mobile platforms

JIT: Unspill normalize-on-load variables using exact small type (#90318)

The existing logic would sometimes unspill using the type of the local
that is being unspilled. This type is often wider than the exact small
type in the LclVarDsc, since NOL locals are normally expanded as
CAST(<small type>, LCL_VAR<int>).

This causes problems since morph will in some cases avoid inserting
normalization for NOL locals when it has a subrange assertion available.
This optimization relies on the backend always ensuring that the local
will be normalized as part of unspilling and args homing.

Size-wise regressions are expected on xarch since the encoding of the
normalizing load is larger. However, as we have seen before, using wide
loads can cause significant store-forwarding stalls which can have large
negative impact on performance, so overall there is an expected perf
benefit of using the small loads (in addition to the correctness fix).

Fix #90219

Fix ilasm casting (#90450)

* Fix 64-bit cast

* Fix a few more cases of casting `strlen`

Fix output to use `%zd` for size_t values.

Implement Edit and Continue on Linux and MacOS (#90349)

* Add EnC support for Linux and Mac

---------

Co-authored-by: Jakob Botsch Nielsen <jakob.botsch.nielsen@gmail.com>
Co-authored-by: Jan Vorlicek <janvorli@microsoft.com>
Co-authored-by: Tom McDonald <tommcdon@hotmail.com>
Co-authored-by: Jan Kotas <jkotas@microsoft.com>

[NativeAOT] Use TCP transport for EventPipe on mobile platforms (#90315)

* [NativeAOT] Use TCP transport for EventPipe on mobile platforms

* Update src/coreclr/nativeaot/Runtime/eventpipe/CMakeLists.txt

---------

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

[wasm] Cache the output of `mono-aot-cross --print-icall-table` (#90476)

Currently, this is invoked to get the runtime icall table for native builds, even if it is not needed by ManagedtoNativeGenerator (for example when outputs are up-to-date). This will change only with the cross compiler, so it can be cached.

Suggested by @vargaz .

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

[wasm] Incremental build improvements (#90194)

- [wasm] Sanity check emcc versions only when using a custom EMSDK_PATH
- [wasm] Emit a message about compiling native assets only when actually doing so
- [wasm] ConvertDllsToWebcil: avoid doing unncessary conversion, and other work
- [wasm] ManagedToNativeGenerator: add support for incremental runs (time goes from 1.6s to <900ms)

This improves build times ~30% for no-op rebuilds.

*With* native reference | Old | New | Difference | %
-- | -- | -- | -- | --
First build | 8.01s | 8.13s | +0.12s |  
4th rebuild | 1.91s | 1.35s | -0.56s | 29%
rebuild with only cs change | 2.90s | 2.48s | -0.42s | 14%

*Without* native reference | Old | New | Difference | %
-- | -- | -- | -- | --
First build | 5.34s | 5.29 | -0.05s |  
4th rebuild | 1.78s | 1.20s | -0.58s | 32%
rebuild with only cs change | 1.96s | 1.45s | -0.51s | 26%

- These were run for blazor projects
- These were clean builds, with no `dotnet` process running
- And nuget cache cleared of `*webassembly.pack` which is the relevant package here.
- The native reference was a simple `factorial.c` from https://github.com/dotnet/runtime/issues/60245

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

With feedback from @kg .

[wasm][debugger]Evaluate a sum between an object and a string (#90175)

* Support sum an object and a string

* Update src/mono/wasm/debugger/BrowserDebugProxy/EvaluateExpression.cs

Co-authored-by: Ankit Jain <radical@gmail.com>
* Apply suggestions from code review

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

* Adding more tests as suggested by @radical.

---------

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

Support invoke of function pointer type arg (#90270)

Switch DateTimeKind from Unspecified to Local (#90149)

Co-authored-by: Maksim Golev <mgolev@htc-cs.ru>

Ensure that constant folding for long->float is handled correctly (#90325)

* Ensure that constant folding for long->float is handled correctly

* Adding a regression test ensuring long->float conversions are correctly handled

* Print failure info for test

* Ensure we continue doing the incorrect 2-step conversion for 32-bit, to match codegen

* Fix build failure

* Ensure test project uses process isolation

[mono] Fix Vector<T>.IsSupported intrinsic (#90023)

Fixes https://github.com/dotnet/runtime/issues/88983
---------

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

Handle side-effects in impBoxPatternMatch (#90496)

Disable InlineArrayValid/Invalid tests on Android (#90414)

[mono] Resolve failing tests on apple mobile platforms (#90515)

* Don't run arm64 jobs

* Disable failing jobs

* Fix failing calendar test

Change how toggle refs are reported in gcdump (#90467)

Improve error message in the new `JsonNode.GetElementIndex()` and `GetPropertyName()` methods. (#90486)

[mono][debugger] Do not use `select(2)` on debugger-agent (#90293)

* Do not use select on debugger-agent

* use pool on APPLE os

[wasm][debugger] Symbolicate function names on debug callstack (#90348)

* Symbolicate function names on debug callstack

* Addressing @radical suggestion

* Addressing @radical suggestions.

* cleanup

---------

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

Update dependencies from https://github.com/dotnet/roslyn-analyzers build 20230812.1 (#90484)

Microsoft.CodeAnalysis.Analyzers , Microsoft.CodeAnalysis.NetAnalyzers
 From Version 3.11.0-beta1.23407.1 -> To Version 3.11.0-beta1.23412.1

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

[DependencyInjection] Avoid useless reflection call in ActivatorUtilities (#90378)

Add NonGC documentation (#90206)

[QUIC] Merge 7.0 fix (#90228)

* Merge pull request #90173 from vseanreesermsft/internal-merge-7.0-2023-08-08-1042

Merging internal commits for release/7.0

* Consume MsQuic release package on Win

---------

Co-authored-by: Carlos Sánchez López <1175054+carlossanlop@users.noreply.github.com>
Co-authored-by: Natalia Kondratyeva <knatalia@microsoft.com>

[mono][arm64] Fixed passing/receiving hfa structures with fixed buffers. (#90502)

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

Enable nullable on Hosting.Systemd (#90495)

- Passing null into the hosting lifetimes' loggerFactory parameters would result in a NRE

Contributes to #90400

Fixing COM Aggregation and XAML reference tracking on .NET AOT (#90283)

* Add initial changes for COM Aggregation and XAML reference tracking in AOT

* Fix dependency list to avoid allocations

* Fix issues with finalizer races and issues with allocating memory during GC

* Add COM weak reference support

* PR feedback on separating reference tracker object fields.

Co-authored-by: Aaron Robinson <arobins@microsoft.com>
Co-authored-by: Jan Kotas <jkotas@microsoft.com>
Co-authored-by: Jeremy Koritzinsky <jkoritzinsky@gmail.com>

Fix TimeProvider Test (#90448)

Disable OneLocBuild in runtime-official.yml (#90480)

It is blocking the official build: https://github.com/dotnet/runtime/issues/90466

Localized file check-in by OneLocBuild Task: Build definition ID 679: Build ID 2241323 (#89414)

* Localized file check-in by OneLocBuild Task: Build definition ID 679: Build ID 2228738

* Localized file check-in by OneLocBuild Task: Build definition ID 679: Build ID 2229389

* Localized file check-in by OneLocBuild Task: Build definition ID 679: Build ID 2230534

* Localized file check-in by OneLocBuild Task: Build definition ID 679: Build ID 2231640

* Localized file check-in by OneLocBuild Task: Build definition ID 679: Build ID 2233112

* Localized file check-in by OneLocBuild Task: Build definition ID 679: Build ID 2234217

* Localized file check-in by OneLocBuild Task: Build definition ID 679: Build ID 2235989

* Localized file check-in by OneLocBuild Task: Build definition ID 679: Build ID 2236160

* Localized file check-in by OneLocBuild Task: Build definition ID 679: Build ID 2236693

* Localized file check-in by OneLocBuild Task: Build definition ID 679: Build ID 2238073

* Localized file check-in by OneLocBuild Task: Build definition ID 679: Build ID 2240179

* Localized file check-in by OneLocBuild Task: Build definition ID 679: Build ID 2241002

* Localized file check-in by OneLocBuild Task: Build definition ID 679: Build ID 2241178

---------

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

Run trimming tests for global.json changes (#90451)

[wasm] Add more blazor tests for InvariantGlobalization (#90440)

* [wasm] build-packages make target - build all the required packages for wbt

* WBT: Add more tests for blazor+InvariantGlobalization

* Fix test

[wasm] Suggest installing the workload when publishing a blazor project (#90471)

[wasm][debugger] Disable some tests on Firefox (#90347)

* Do not run this test on Firefox

* Disable other test

[wasm] Don't build samples in parallel on CI (#90470)

Prompted by failures described in https://github.com/dotnet/dnceng/issues/450

[NativeAOT] Do not use private APIs on iOS/macOS (#90430)

* Use custom implementation of _dyld_find_unwind_sections on Apple
platforms since it's a private API and it blocks uploads to TestFlight,
iOS App Store, and Mac App Store.

* Link against local ICU libraries on iOS-like platforms

* Update src/coreclr/nativeaot/BuildIntegration/Microsoft.NETCore.Native.Unix.targets

* Add comment

* Update src/coreclr/nativeaot/Runtime/unix/UnwindHelpers.cpp

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

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

Fix condition for setting if symbol is a file in native AOT publish targets (#90438)

runtime: fix variable null check (#89920)

Fixed error when _hashtableContentsToEnumerate is null.
It appears always when using 'Length' property when variable is null.
Also reduced nesting.

[main] Update dependencies from dotnet/arcade dotnet/emsdk (#90461)

* Update dependencies from https://github.com/dotnet/arcade build 20230811.1

Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.Build.Tasks.Archives , Microsoft.DotNet.Build.Tasks.Feed , Microsoft.DotNet.Build.Tasks.Installers , Microsoft.DotNet.Build.Tasks.Packaging , Microsoft.DotNet.Build.Tasks.TargetFramework , Microsoft.DotNet.Build.Tasks.Templating , Microsoft.DotNet.Build.Tasks.Workloads , Microsoft.DotNet.CodeAnalysis , Microsoft.DotNet.GenAPI , Microsoft.DotNet.GenFacades , Microsoft.DotNet.Helix.Sdk , Microsoft.DotNet.PackageTesting , Microsoft.DotNet.RemoteExecutor , Microsoft.DotNet.SharedFramework.Sdk , Microsoft.DotNet.VersionTools.Tasks , Microsoft.DotNet.XUnitConsoleRunner , Microsoft.DotNet.XUnitExtensions
 From Version 8.0.0-beta.23409.5 -> To Version 8.0.0-beta.23411.1

* Update dependencies from https://github.com/dotnet/emsdk build 20230811.2

Microsoft.NET.Workload.Emscripten.Current.Manifest-8.0.100.Transport
 From Version 8.0.0-rc.1.23411.1 -> To Version 8.0.0-rc.1.23411.2

Dependency coherency updates

runtime.linux-arm64.Microsoft.NETCore.Runtime.ObjWriter,runtime.linux-x64.Microsoft.NETCore.Runtime.ObjWriter,runtime.linux-musl-arm64.Microsoft.NETCore.Runtime.ObjWriter,runtime.linux-musl-x64.Microsoft.NETCore.Runtime.ObjWriter,runtime.win-arm64.Microsoft.NETCore.Runtime.ObjWriter,runtime.win-x64.Microsoft.NETCore.Runtime.ObjWriter,runtime.osx-arm64.Microsoft.NETCore.Runtime.ObjWriter,runtime.osx-x64.Microsoft.NETCore.Runtime.ObjWriter,runtime.linux-arm64.Microsoft.NETCore.Runtime.JIT.Tools,runtime.linux-x64.Microsoft.NETCore.Runtime.JIT.Tools,runtime.linux-musl-arm64.Microsoft.NETCore.Runtime.JIT.Tools,runtime.linux-musl-x64.Microsoft.NETCore.Runtime.JIT.Tools,runtime.win-arm64.Microsoft.NETCore.Runtime.JIT.Tools,runtime.win-x64.Microsoft.NETCore.Runtime.JIT.Tools,runtime.osx-arm64.Microsoft.NETCore.Runtime.JIT.Tools,runtime.osx-x64.Microsoft.NETCore.Runtime.JIT.Tools,runtime.linux-arm64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk,runtime.linux-arm64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools,runtime.linux-musl-arm64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk,runtime.linux-musl-arm64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools,runtime.linux-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk,runtime.linux-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools,runtime.linux-musl-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk,runtime.linux-musl-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools,runtime.win-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk,runtime.win-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools,runtime.osx-arm64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk,runtime.osx-arm64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools,runtime.osx-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk,runtime.osx-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools
 From Version 16.0.5-alpha.1.23401.4 -> To Version 16.0.5-alpha.1.23408.1 (parent: Microsoft.NET.Workload.Emscripten.Current.Manifest-8.0.100.Transport

---------

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

[workloads] Remove experimental off of librarybuilder workload (#90410)

JIT: Prevent generic context runaway during inlining of polymorphic recursive methods (#90306)

The inliner heuristics do not take into account that inlining methods
causes type/method loading of the generic context. After #88749 this can
quickly cause significant resources to be consumed as part of inlining
when polymorphic recursion is involved (the blow-up can be exponential,
as we see in the failing test under jitstress).

This PR adds another safe-guard to the recursive inlining check in terms
of a complexity limit on the generic context of the inline candidate.

Fix #90144

Remove all use of <Nullable>annotations</Nullable> (#90401)

* Remove all use of <Nullable>annotations</Nullable>

We've inadvertently shipped multiple libraries with incorrect nullable reference type annotations because:
a) Their ref assemblies aren't actually being shipped, and/or
b) Their src projects using `<Nullable>annotations</Nullable>`

`<Nullable>annotations</Nullable>` means "I'm nullable annotated but don't validate them", which means consumers of these libraries see annotations that we haven't thoughtfully added or reviewed.

This removes all use of it and gets us back to a place where we're only shipping nullable annotations for libraries where we've done the work to ensure they're correct.  We can subsequently finish annotating these stragglers.

* Don't disable LibraryImportGenerator when nullable is disabled

* Add #nullable enable to LibraryImportAttribute file for polyfill scenarios.

* Disable LibraryImportGenerator on ILCompiler.TypeSystem.Tests assemblies.
Convert a DllImport in HostWriter that now gets the polyfill experience.

---------

Co-authored-by: Eric StJohn <ericstj@microsoft.com>
Co-authored-by: Jeremy Koritzinsky <jekoritz@microsoft.com>

Add DiagnosticDescriptorHelper to set HelpURI for generator / analyzer diagnostics (#90353)

* Add DiagnosticDescriptorHelper

Create a helper method that formats and sets the helpLink for
DiagnosticDescriptors

* Add HelpLinkUris to ILLink

* Handle different DiagnosticCategories

[workloads] Add telemetry for mobile and wasm workloads (#90208)

This change adds CLI telemetry for common mobile and wasm scenarios.

Total app size is another element we plan on tracking and that will be added in a follow up.

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

Update Versions.props (#90425)

Prefix type names with global and use static TypeSyntax and NameSyntax instead of Parse(Type)Name (#90339)

* Prefix type names with global and use static TypeSyntax and NameSyntax instead of Parse(Type)Name

Support Exposing new Metric properties using EventSource (#90417)

[wasm] Fix builds for blazor app + InvariantGlobalization=true (#90432)

The following commit caused workload to be required for a blazor app
when `InvariantGlobalization==true`, but this is not required.

```
commit 26ae09784c16dfa6707cd73eca2aeb00a28741d0
Author: Ankit Jain <radical@gmail.com>
Date:   Thu Aug 10 23:39:10 2023 -0400

    [wasm] Fix up conditions to trigger relink, and require `wasm-tools` workload (#89754)
```

And this broke some sdk tests.

Allow reference types for pinned GC.AllocateArray() (#89293)

* Relaxing constraints of System.GC.AllocateArray<T>.

It's now possible to allocate pinned and default-initialized arrays that contain references.

Fix #48703

* Adding coverage of GC.AllocateArray with ref types.

Added a test that pins a reference type array and resolves references to indices with pointer arithmetic, checking that modifications are visible back in the original array.

Test #48703

* Mono/NativeAOT relaxing of System.GC.AllocateArray<T>.

* Relaxed GC.AllocateUninitializedArray for ref types as well.

This was done by deferring reference types to GC.AllocateArray to avoid potential memory issues with the GC + uninitialized memory. The API only promises to avoid initialization if possible, anyway.
Refactored tests to parametrically exercise these new relaxed constraints.

* Simplifiy conditionals in AllocateUninitializedArray.

Relying on internal implementation zeroing refs if necessary.

* Also simplify path for NativeOAT.

Mono already piggybacks on the AllocateArray path anyway.

* Simplify pinning paths.

All conditional paths in GC.Allocate*Array now handle pinning unconditionally out of the main branch.

* Don't use `var` if type name doesn't exist explicitly on right-hand side

* PR feedback for using terse method tables and JIT intrinsics for GC array allocation that should be slightly faster.

* Changed EmbeddedValueType and added comment.

After a longer discussion, settled on a slightly augmented suggestion that isn't as controversial as the prior one.

* Fixing signature for `RhAllocateNewArray` in NativeAOT to directly use a `MethodTable*`

* Adding explicit structlayout to silence warning for EmbeddedValueType in GCTests, and improved the comment.

---------

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

Fix session-local named mutex compat issue (#90342)

- A previous change that was serviced back changed session-local named mutexes to be user-specific by restricting the permissions of the session directories and files under them, and adding the sticky bit to some directoires. A compat issue arose from that change, as the session directories have the session ID in their name and session IDs can be reused between different users. The current plan that we have discussed is to revert the change and service back the revert, which also restores the intended behavior, and offer user-specific mutexes as a new feature in a future .NET that would satisfy some user scenarios in a better way.
- This PR reverts the previous change (first commit) and restores one change from the previous change (second commit) to improve backward compatibility due to differences in permissions for session directories before and after the change
- Fixes https://github.com/dotnet/runtime/issues/80619

Using stackalloc instead of new byte[x] on Streams (#88303)

* first XOR Linq implemetation

* replace classic allocation by stack allocation for ReadByte

* replace classic allocation by stack allocation for ReadByte & WriteByte

* replace classic allocation by stack allocation for ReadByte/CopyToAsync

* add missing ConfigureAwait(false) on ReadAsync

* replace classic allocation by stack allocation for ReadByte/WriteByte

* Revert "first XOR Linq implemetation"

This reverts commit 449fddec66b3311866dc1e57c07ad7127e8039ff.

* indent fix

* rollback stackalloc on async methods

* missing char

* fix ConfigureAwait

* rollback ReadByte optim (degrade perfs)

* variablename fix

* missing comment

* rollback Stream allocations

* missing arguments

* add Write(ReadOnlySpan<byte>) method

* better initialize Span

* add CRLF

* resolve comments

* simplification call method Write

* Update src/libraries/System.Private.DataContractSerialization/src/System/Runtime/Serialization/Json/JsonEncodingStreamWrapper.cs

replace Span by ReadOnlySpan

Co-authored-by: Michał Petryka <35800402+MichalPetryka@users.noreply.github.com>
* remove trailing whitespace

* Update src/libraries/System.Private.DataContractSerialization/src/System/Runtime/Serialization/Json/JsonEncodingStreamWrapper.cs

Co-authored-by: Stephen Toub <stoub@microsoft.com>
* Update src/libraries/System.Private.DataContractSerialization/src/System/Runtime/Serialization/Json/JsonEncodingStreamWrapper.cs

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

Co-authored-by: Michał Petryka <35800402+MichalPetryka@users.noreply.github.com>
Co-authored-by: Stephen Toub <stoub@microsoft.com>