Remove some dead code that threw file load exception (#87290)

* Remove some dead code that threw file load exception

* Remove FUSION_E_CACHEFILE_FAILED

Fix typo in dac-notes.md (#87295)

[main] Update dependencies from 10 repositories (#87125)

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

JIT: Handle primitive remainder stores to regularly promoted fields in physical promotion (#87217)

If the remainder of a block copy is handled as a primitive, then it is
possible that the destination or source is a regularly promoted field.
In this case we do not have to DNER it.

Example. Before:

```
Processing block operation [000009] that involves replacements
  V08 (field V02.hasValue (fldOffset=0x0)) <- V16 (V00.[000..001)) (last use)
  Remainder: [004..008)
  => Remainder strategy: int at +004

Local V00 should not be enregistered because: was accessed as a local field

Local V02 should not be enregistered because: was accessed as a local field
New statement:
STMT00003 ( 0x00D[E-] ... 0x00E )
               [000090] -A---------                         ▌  COMMA     void
               [000087] DA---------                         ├──▌  STORE_LCL_VAR bool   V08 tmp4
               [000086] -----------                         │  └──▌  LCL_VAR   bool   V16 tmp12         (last use)
               [000089] UA---------                         └──▌  STORE_LCL_FLD int   (P) V02 loc1         [+4]
                                                               ▌    bool   V02.<unknown class>:hasValue (offs=0x00) -> V08 tmp4
                                                               ▌    int    V02.<unknown class>:value (offs=0x04) -> V09 tmp5
               [000088] -----------                            └──▌  LCL_FLD   int    V00 arg0         [+4]
```

After:

```
Processing block operation [000009] that involves replacements
  V08 (field V02.hasValue (fldOffset=0x0)) <- V16 (V00.[000..001)) (last use)
  Remainder: [004..008)
  => Remainder strategy: int at +004

Local V00 should not be enregistered because: was accessed as a local field
New statement:
STMT00003 ( 0x00D[E-] ... 0x00E )
               [000090] -A---------                         ▌  COMMA     void
               [000087] DA---------                         ├──▌  STORE_LCL_VAR bool   V08 tmp4
               [000086] -----------                         │  └──▌  LCL_VAR   bool   V16 tmp12         (last use)
               [000089] DA---------                         └──▌  STORE_LCL_VAR int    V09 tmp5
               [000088] -----------                            └──▌  LCL_FLD   int    V00 arg0         [+4]

```

Move DynamicCodeSupport default into aot targets (#87135)

* Move DynamicCodeSupport default into aot targets

* Clean up redundant PublishTrimmed setting

Add IHostApplicationBuilder interface (#86974)

* Add IHostApplicationBuilder and implement it in HostApplicationBuilder.

- Move ILoggingBuilder from MS.Ext.Logging to MS.Ext.Logging.Abstractions so IHostApplicationBuilder can reference it.
- Add IConfigurationManager and implement it in ConfigurationManager.
- Add CompatiibilitySuppressions. These errors are caused by ILoggingBuilder being moved to Logging.Abstractions, and ApiCompat not respecting TypeForwardedTo attributes.

Fix #85486

Fix DeserializeAsyncEnumerable generic recursion issue. (#87276)

Don't enable ILLink analyzers for unsupported TFMs (#87071)

* Don't enable ILLink analyzers for unsupported TFMs

This adds a condition to EnableSingleFileAnalyzer to prevent it from requiring
the ILLink pack that is only supported on netcoreapp3.0 and above.

Adding a condition on the TFM requires moving the setting into a targets file.

Also consolidates more of the analyzer logic into this file.

---------

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

[wasm] Fix for getelema1 crash & cpblk crash in jiterpreter (#87246)

* Fix for getelema1 crash in specific scenarios
* Check more of the zero page for nonzero data
* If we ever see garbage in the zero page disable it permanently
* Log an error if garbage appears in the zero page after startup
* Overhaul trace local variable naming and reuse
* Fix count not being initialized for cpblk in some cases

make mono_wasm_invoke_method_bound safer with exception root (#87266)

Remove calls to GetHighPrecisionTimeStamp from allocate_uoh_object that made it into main by mistake (my bad). (#87273)

JIT: Fix a couple of inequalities in promotion decomposition (#87186)

These checks were of the form `end < start` when more precisely they
can be `end <= start`. It was causing some unnecessarily conservative
writebacks to occur in some cases.

Also skip entries in the decomposition plan that are writing to the
remainder when the remainder is dead. Saw a bunch of these cases in
the regressions and realized we were missing this case.

Remove Vector<T> fallbacks for Vector128<T> (#85916)

* Remove Vector<T> fallbacks for Vector128<T>

* ifdef instead of deleting Vector<T> code paths

ifdef them to mono as they're not used by coreclr or nativeaot.  They can be deleted instead once mono's Vector128 support improves.

we don't load timezone with mono_wasm_load_data_archive anymore (#87264)

don't use GetEarlyNode in impGetSpecialIntrinsicExactReturnType (#87263)

[mono][arm64] Handle sysctlbyname failing to find system options. (#87189)

This fixes a startup crash on Big Sur:

> error: * Assertion at /Users/runner/work/1/s/src/mono/mono/utils/mono-hwcap-arm64.c:35, condition `res == 0' not met

Because sysctl can't find some of these options:

    $ sysctl hw.optional.armv8_crc32
    hw.optional.armv8_crc32: 1
    $ sysctl hw.optional.arm.FEAT_RDM
    sysctl: unknown oid 'hw.optional.arm.FEAT_RDM'
    $ sysctl hw.optional.arm.FEAT_DotProd
    sysctl: unknown oid 'hw.optional.arm.FEAT_DotProd'
    $ sysctl hw.optional.arm.FEAT_SHA1
    sysctl: unknown oid 'hw.optional.arm.FEAT_SHA1'
    $ sysctl hw.optional.arm.FEAT_SHA256
    sysctl: unknown oid 'hw.optional.arm.FEAT_SHA256'
    $ sysctl hw.optional.arm.FEAT_AES
    sysctl: unknown oid 'hw.optional.arm.FEAT_AES'

Full stack trace:

* thread #1, queue = 'com.apple.main-thread', stop reason = breakpoint 2.1
  * frame #0: 0x0000010ef37560 libmonosgen-2.0.dylib`monoeg_assertion_message
    frame #1: 0x0000010ef375cc libmonosgen-2.0.dylib`mono_assertion_message + 32
    frame #2: 0x0000010ef40d6c libmonosgen-2.0.dylib`mono_hwcap_arch_init + 544
    frame #3: 0x0000010ef54bd8 libmonosgen-2.0.dylib`mono_hwcap_init + 72
    frame #4: 0x0000010ee14dc0 libmonosgen-2.0.dylib`parse_optimizations + 52
    frame #5: 0x0000010edbed48 libmonosgen-2.0.dylib`mono_init
    frame #6: 0x0000010ee18968 libmonosgen-2.0.dylib`mono_jit_init_version
    frame #7: 0x0000010f48a300 libxamarin-dotnet-debug.dylib`xamarin_bridge_initialize + 216
    frame #8: 0x0000010f4900a4 libxamarin-dotnet-debug.dylib`xamarin_main + 376

[browser] Fix fingerprinting of relinked dotnet.js - follow up (#87051)

Address comments from #86048

[NativeAOT] Improving HelloiOS sample for NativeAOT to be testable with runtime packs (#86652)

- Enabled end-to-end testing of HelloiOS with NativeAOT runtime packs
- Update ILCompiler paths to handle PublishAotUsingRuntimePack=true
Fixes https://github.com/dotnet/runtime/issues/80911
---------
Co-authored-by: Filip Navara <navara@emclient.com>

[wasm][debugging] Enable debugging when Device Toolbar is enabled and Iphone is selected (#86920)

* Enable debugging when Device Toolbar is enabled and Iphone is selected
* unify browser detection
Co-authored-by: pavelsavara <pavel.savara@gmail.com>

Fix inline decision reporting + noinline propagation (#87115)

Ifdef out DynamicDependencies that are not necessary (#87209)

These create unnecessary reflection targets. Trimming done by NativeAOT will do the right thing and it is redundant on CoreCLR.

Setup for runs with physical promotion. (#87227)

Optimize MethodTable on both sides of `==` (#87238)

Optimizes `typeof(Foo) == typeof(Bar)` for both sides. This is necessary to get `System.Double` MethodTable from hello world again.

I tried to write a test for this but I wasn't able to come up with something in 10 minutes so I gave up. It seems to require interactions of several things to actually trigger the size bloat without this extra optimization. But this does fix the `Double` case.

Delete some dead code (#87249)

I went through RuntimeImports looking for things that looked unnecessary. The rest of this PR is basically "stuff that came out after pulling on a loose thread of a sweater".

Handle diagnostic port configuration in NativeAOT (#87088)

[wasm] Fix running debugger tests on windows (#87228)

* [wasm] Fix running debugger tests on windows

.. by making sure to escape `&` in the command line. This was introduced
in:

```
commit e96321db982c3add192381f6ea4ff0a999ee1410
Author: Ilona Tomkowicz <32700855+ilonatommy@users.noreply.github.com>
Date:   Tue Jun 6 05:59:43 2023 +0200

    [wasm][debugger] Disable failing multithreading debugger tests (#86501)
```

.. but missed CI due to the helix outage.

* Fix escaping ! on windows

WIP (#86512)

Fix FreeBSD build break (#87242)

Update ClaimsIdentity debug display (#87159)

* Update ClaimsIdentity.cs

* Update src/libraries/System.Security.Claims/src/System/Security/Claims/ClaimsIdentity.cs

Co-authored-by: Adam Sitnik <adam.sitnik@gmail.com>
---------

Co-authored-by: Adam Sitnik <adam.sitnik@gmail.com>

Simplify Random for ulong (#87219)

Use Math.BigMul instead of UInt128 since both values are always ulongs.

Implement `FlipBit` (#83400)

* Implement `ToggleBit`

* Rename to `ToggleBit`

* Minor improvement to documentation comments

[NativeAOT] Don't link Swift/CryptoKit on iOS-like platforms (#87216)

Fix reading runtime settings early in startup (#87205)

* Since runtime settings are read before we had a chance to run dehydration, they cannot go to the dehydrated section. This is a 48-byte penalty per setting on Linux. Hopefully we won't have too many of these.
* Fix the repro project to enable dehydration. When testing the original change, I used repro project where this worked because of no dehydration. We should mirror the shipping configuration.
* Add a regression test. Picked a random test as a "victim" so that we don't unnecessarily regress testing time.

Implement JsonStringEnumConverter<TEnum> (#87224)

* Fix NativeAOT support for JsonStringEnumConverter.

* Fix merge issue.

* Implement JsonStringEnumConverter<TEnum>.

* Remove leftover comments.

* Revert unnecessary suppression.

* Update compatibility suppressions.

* Add missing XML docs

* Fix capitalization.

* add local project compatibility suppressions

* apply suggestion in all location

* Fix failing test.

* Ensure both converter factories throw an appropriate exception when passed invalid inputs.

Improve codegen for AVX-512 comparisons and general handling of `TYP_MASK` (#87089)

* Add the various base kmask instructions

* Updating lowering to better handle the AVX512 comparisons that produce a mask

* Remove INS_Flags_IsMskSrcSrcEvexInstruction as its the same as INS_Flags_IsDstDstSrcAVXInstruction

* Save 1-byte on the encoding for CompareEqualMask and signed CompareGreaterThanMask

* Apply formatting patch

* Ensure kmovd is encoded correctly and kmov is disassembled correctly

* Ensure vpcmpgtq is actually handled

* Fix the definition of a couple kmask instructions

* Ensure gtNewSimdCmpOpAll node creates correct IR for simdSize == 64

* Add , that were dropped when resolving merge

* Ensure that the new hwintrinsiclistxarch entries don't assert ValueNum

Apply source build patch (#87226)

Fixes #83695

Use Unsafe.BitCast for Interlocked.{Compare}Exchange of float/double (#87166)

* Update {Compare}Exchange in coreclr to use Unsafe.As

* Remove corresponding FCalls

* Share with mono

* Remove Mono icalls

* Add AggressiveInlining

* Remove unused union type in mono

[VM] Fix potential undefined behavior (#87119)

Use pointer arithmetic instead of direct array access to avoid
compilers, specifically GCC, to discover undefined behavior and
generate unintended code when optimization is turned on.

The array involved is `pSeries->val_serie`, which is declared as
a fixed sized array of size 1. However, `index` is always a
non-negative integer, and we want to access the memory at
`-index`, which is either zero or negative.

Ensure the CPUID tests cover a couple hwintrinsic disablement edge cases (#87127)

* Ensure CPUID tests have BMI1/BMI2 use the AVX hierarchy checks, not AVX2

* Ensure CPUID tests check the DOTNET_EnableSSE3_4 environment variable

* Fix a build failure in X86Base/CpuId.cs

* Ensure that BMI1/BMI2 are checking the right bits

Allow duplicate feature switch settings (#87207)

Fixes #86186.

Tweak assert in AsyncTaskMethodBuilder (#87215)

Add timestamps to XUnitLogChecker output (#86737)

* Add timestamps to XUnitLogChecker output

* format

* Handle $ strings

* Combine two Writes into one

Fix handling of CreateScalarUnsafe for embedded broadcast (#87134)

* Adding a regression test for dotnet/runtime#87116

* Ensure IsContainableHWIntrinsicOp takes into account whether CreateScalarUnsafe is coming from memory for embedded broadcast

[wasm] Add workaround `NuGet-Migrations` issue for a missed case (#87041)

One case was missed in the earlier attempt at this, which caused some of
the tests in wasm runtime tests to not get the workaround script.

One such example - `JIT.PGO`

The other cases had:
```sh
export WasmTestAppArgs="$WasmTestAppArgs --exclusion-list=TestExclusionList.txt"
chmod +x ./RunTests.sh
(CONSOLE_TEMP_DIR="$(mktemp -d)" ; "$DOTNET_ROOT/dotnet" new console -o "$CONSOLE_TEMP_DIR" ; rm -rf "$CONSOLE_TEMP_DIR") || true
./RunTests.sh
```

.. and the broken one has:

```sh
dotnet $CORE_ROOT/xunit/xunit.console.dll profiler/unittest/profiler.unittest.XUnitWrapper.dll -parallel collections -nocolor -noshadow -xml testResults.xml -trait TestGroup=profiler.unittest
```

.. no workaround with `dotnet new ..`.

Move the CreateCastingConverter<T>() method to the base JsonConverter type. (#87211)

Make LocalizedApp test stop requiring UTF8 output encoding (#87176)

Set ApiCompatIgnoreTargetPlatformMoniker for packable projects (#87212)

Follow-up change on a bugfix/feature in APICompat: https://github.com/dotnet/sdk/commit/b8c07fee1b150d46e5c4229cf1f0d81da20393a5

Add back fixed xunit decimal, IntPtr and UIntPtr commented test cases (#87084)

Disable S.T.Json tests on NativeAOT (#87203)

We're getting OOM-killed in the CI. See #87078.

I'm also changing how we do warning suppressions. We still need to suppress most of AOT warnings because xUnit/tests are not warning clean, but the warning about generic recursion is critical for the product and should fail the build.

[wasm] Disable System.Text.Json/AOT tests (#87157)

- this was disabled earlier from highresource_aot builds, but that had a
  side-effect of enabling them for regular aot builds.

Issue: https://github.com/dotnet/runtime/issues/86164

JIT: Clean up BB successor iteration (#86839)

* Remove old code
* Use visitor with an array as part of AllSuccessorEnumerator

Cleanup scripts for PGO cleanup (dotnet/performance#3067). (#87163)

Fix creating cultures with extensions in the name (#87114)

Enable EventPipe support in MacOS (#86766)

* Enable EventPipe support in MacOS

* Fix FreeBSD build break

* FB

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

[fabricbot] Do not add needs-further-triage to issues still marked as untriaged (#87160)

update QuicConnection.ConnectAsync exceptions (#87048)

* update QuicConnection.ConnectAsync exceptions

* feedback from review

Reenabled k-nucleotide-9 test. (#87058)

Update area-owners.json to reflect change in area-owners.md (#87144)

[RISC-V][JIT] Fix arithmetics (#86996)

* [RISC-V][JIT] Fix multiplication instruction

* [RISC-V][JIT] Fix a test failure

- value_numbering_checked_arithmetic_with_constants_ro

[wasm][debugger] Disable failing multithreading debugger tests (#86501)

* Block for multithreading.

* Another failing test.

* category - case matters.

* Cleanup.

* "Result reported for unknown test case" fix. Cannot be lazy.

* Debugger has its own treading feature discovery.

* Prevent trait filter from being overwritten.

* Applied @radical's review, reverted `TEST_FILTER` removal with changes fixing the issue.

* Makefiles - full revert, fix should be in msbuild.

* Failing frequently.

* Fleaky tests.

* Now this one failed - block.

* 10 min still timeouts, set to 15.

* Revert "10 min still timeouts, set to 15."

This reverts commit 2e008d40b3006e464f3c70030e0de637d17dc426.

[fabricbot] Area pods: Exclude arch- and os- issues/prs with owners (#87040)

* [fabricbot] Area pods: Exclude arch- and os- issues/prs with owners

* Do not exclude arch-wasm from the michael/tanner pod

[wasm] Perf blazor scenarios: Use the correct wasm sdk pack (#87075)

- Make the built nuget for wasm sdk pack available when running the
  blazor scenarios.

```
/home/helixbot/work/B975098B/p/performance/src/scenarios/blazorpizza/app/BlazingPizza.Client/BlazingPizza.Client.csproj
: warning NU1603: BlazingPizza.Client depends on Microsoft.NET.Sdk.WebAssembly.Pack (>= 8.0.0-ci) but Microsoft.NET.Sdk.WebAssembly.Pack 8.0.0-ci was not found. An approximate best match of Microsoft.NET.Sdk.WebAssembly.Pack 8.0.0-preview.4.23213.11 was resolved.
```

.. otherwise it resolves to an older package from the published sources.

Superpmi collection for real-world benchmarks (#87104)

* add support to collect spmi for real-world

* rename some script names

* Do not pass partition_index for non-microbenchmarks

* fix the project names

* fix physics engine benchmarks:

* adjust physics benchmarks:

Fix IDE0001 / IDE0002 (simplify names) in corelib (#87106)

Allow the user to control the MaxVectorTBitWidth (#85551)

* Expose DOTNET_MaxVectorTBitWidth and an undocumented DOTNET_PreferredVectorBitWidth

* Ensure SPMI keeps a getMaxVectorTBitWidth implementation

* Fix the non-xarch vm build

* Remove getMaxVectorTBitWidth from the JIT/EE interface, it's no longer needed

* Move SetCpuInfo down into the EEJitManager constructor

* Remove getXarchCpuInfo in favor of passing `JIT_FLAG_VECTOR512_THROTTLING`

* Make sure CORINFO_XARCH_CPU is fully removed

* Have ENCODE_VERIFY_TYPE_LAYOUT not fail-fast for Vector<T> size differences

* Only encode types containing Vector<T> as check, not verify

* Remove changes that were extracted to separate PRs

* Ensure that the optimistic flags are a strict superset of the supported flags

* Make VectorT128/256/512 proper instruction sets and only allow one to be active at a time

* Don't allow avxvnni to be "optimistic" since that brings in avx2

* Ensure we handle HWIntrinsics being disabled

* Ensure that the Vector<T> size ISAs are covered by FromInstructionSet

* Ensure that `getMaxVectorByteLength` being 0 is handled

* Ensure NAOT startup can correctly check for the VectorT size bits

* Have BlkOpKindUnroll account for SIMD being disabled

* Ensure InstructionSet_VectorT128 is set in the fallback path for PAL_GetJitCpuCapabilityFlags

Fix a number of accessibility resolution issues in the source generator. (#87136)

Documentation for the HotColdMap section (#87023)

[nativeaot][tests] Update bundle zip path for NativeAOT sample app (#87049)

* Update bundle zip path for NativeAOT sample app

* Add shared path for Mono and Native AOT

Disable mark list optimization if we hit a per region mark list overflow (#86508)

Test passing struct on a border of 12bit offset (#86914)

Remove unnecessary suppressions in APICompat files (#87094)

* Remove unnecessary suppressions in APICompat files

- Remove unnecessary suppressions in APICompat files. This is in
  preparation for https://github.com/dotnet/sdk/pull/32964 which will
  validate the existing suppressions going forward.
- Set the required APICompat properties for the future tooling support.

* Add suppressions back for two CoreLib flavors

* Fix CoreLib suppression because of API attribute difference

Set .env variables in corerun prior to CLR init (#85220)

Fold obj.GetType to a constant (#87101)

[VM] Fix potential double free (#86207)

Use a raw char `NewArrayHolder` instead of a `NewHolder` to store
the `StackingAllocator` to prevent its destructor from being
called twice since `StackingAllocatorHolder` has already taken
care of the destruction.

Avoid bloom filter checks for IndexOfAnyExcept in ProbabilisticMap (#85203)

[browser][non-icu] `HybridGlobalization` set flag in SDK (#85245)

* Initial change.

* Filename update.

* WBT for WASM.

* Updated docs.

* Run Hybrid wbt on CI.

* Feedback + hierarchy compatible with #86255.

* Feedback. Test program code to assets.

Ensure JsonSerializerOptions always returns a JsonTypeInfo for `object`. (#87093)

* Ensure JsonSerializerOptions always returns a JsonTypeInfo for `object`.

* Address feedback.

fix(76268): Fix invalid behavior for ignored properties with "new" modifier of derived class. (#84756)

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

[main] Update dependencies from dotnet/arcade dotnet/sdk (#87091)

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

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.23281.3 -> To Version 8.0.0-beta.23302.3

* Update dependencies from https://github.com/dotnet/sdk build 20230602.9

Microsoft.DotNet.ApiCompat.Task
 From Version 8.0.100-preview.6.23302.2 -> To Version 8.0.100-preview.6.23302.9

* Update dependencies from https://github.com/dotnet/sdk build 20230603.2

Microsoft.DotNet.ApiCompat.Task
 From Version 8.0.100-preview.6.23302.2 -> To Version 8.0.100-preview.6.23303.2

---------

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

[main] Update dependencies from dnceng/internal/dotnet-optimization (#87090)

* Update dependencies from https://dev.azure.com/dnceng/internal/_git/dotnet-optimization build 20230602.4

optimization.linux-arm64.MIBC.Runtime , optimization.linux-x64.MIBC.Runtime , optimization.windows_nt-arm64.MIBC.Runtime , optimization.windows_nt-x64.MIBC.Runtime , optimization.windows_nt-x86.MIBC.Runtime , optimization.PGO.CoreCLR
 From Version 1.0.0-prerelease.23301.6 -> To Version 1.0.0-prerelease.23302.4

* Update dependencies from https://dev.azure.com/dnceng/internal/_git/dotnet-optimization build 20230603.6

optimization.linux-arm64.MIBC.Runtime , optimization.linux-x64.MIBC.Runtime , optimization.windows_nt-arm64.MIBC.Runtime , optimization.windows_nt-x64.MIBC.Runtime , optimization.windows_nt-x86.MIBC.Runtime , optimization.PGO.CoreCLR
 From Version 1.0.0-prerelease.23301.6 -> To Version 1.0.0-prerelease.23303.6

---------

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

Fix a few Parse uses to be TryParse (#87105)

[browser][non-icu] Revert `HybridGlobalization` normalization. (#87007)

* revert

* Keep perf tests but with smaller buffers

Update area-owners.md (#87121)

Do not use ValueTuple in SynchronizationContext (#87120)

This is the same issue as e.g. #84206. The compiler needs to consider delegate Invoke methods always reflectable because of trimming warning suppressions around this in the framework. But that means the compiler is going to create type loader templates for all types in the `Invoke` signature. If one of those is a `ValueTuple`, it will root the ability to runtime-create arbitrary comparers at runtime (`ValueTuple` annoying implements `Equals` using `EqualityComparer.Default`) and with it the entire type loader.

For a sample app in #82607 this causes about 200 kB of bloat to be generated (more than 10% of the app size).

Save 260k in InitValueNumStoreStatics (#85945)

It appears that the big #define for the intrinsics is causing InitValueNumStoreStatics to get big enough that C++ optimization ends up being disabled, which means a lot of constant operations aren't folded. This rewrites it as a const table. It adds some redundant information to the tables that we #include/#define in several places but currently includes many assertions that the old and new values match.

Local size change of release clrjit_win_x64_x64.dll: 2,001,920 -> 1,735,680.  InitValueNumStoreStatics (261k) is replaced by 1.2k of static data.

Resolves #85953

[NativeAOT] A few cleanups in stack walking on Windows (#86917)

Ensure the correct `typeToConvert` parameter is being passed to constructor parameter converters. (#87111)

Add and use ConfigureAwaitOptions (#87067)

* Add and use ConfigureAwaitOptions

Adds the ability to further control how awaits are performed, with the ConfigureAwaitOptions enum.  For .NET 8 this is just for `Task` / `Task<TResult>`, and for the latter, the `SuppressThrowing` option isn't supported (as it can make the TResult erroneous).

Also uses it throughout the repo to replace various custom awaiters, use of Task.Run, and places we were catching all exceptions to suppress them with awaits.  Some of these just help to clean up the code; others have / enable meaningful perf improvements.

* Address PR feedback

* Update src/libraries/System.Net.HttpListener/src/System/Net/Managed/HttpResponseStream.Managed.cs

Unify trim warnings for accessing compiler generated code (#86816)

This modifies the behavior of both the illink and ilc to match and to implement some changes we've discussed recently - https://github.com/dotnet/runtime/issues/86008.

The semantic changes (mostly as compared to previous illink behavior):
* Reflection access to a RUC virtual method will produce warning on all methods which have RUC, regardless of their relationships. Specifically if both the override and base methods are accessed, and they both have RUC, they will both generate a warning. (This effectively removes an optimization which illink had to avoid "duplicate" warnings where it tried to warn only on the base methods in this case. But this has holes - see https://github.com/dotnet/runtime/issues/86008).
* Reflection access to compiler generated code will not produce warnings even if the target has RUC on it. It proved to be really difficult to design a good experience for reporting warnings in these cases. The problem is that developer has little control over the generated code and fully reporting all warnings leads to lot of noise. The above optimization in the illink tried to solve some of this, but it's not very successful. So we're removing all of these warnings. The reasoning is that reflection access to compiler generated members is an undefined behavior (even on non-trimmed, normal CLR) - and is likely to break for example between compiler versions - and there's no diagnostics about it ignoring trimming. Trimming/AOT just makes it a little bit worse.
* AOT compiler will not implement warnings caused by reflection access to compiler generated code which has annotations either (so `IL2118`, `IL2119` and `IL2120`). The plan is to eventually remove these from illink as well.

Note that there are exceptions to the above rules, which can be described as: Accessing a token of a member in IL is considered a reflection access (because the token can be turned into a reflection object), but it is also considered a direct reference. So in that case marking is implemented as "reflection access", but diagnostics is implemented as "direct reference". What this means is that accessing a compiler generated member through its token will still produce all warnings (RUC or DAM) as for non-compiler-generated member. This is important for things like creation of `Action` from a lambda, where the lambda is a compiler generated method, but we need to produce warnings if it uses annotations for example.

Refactorings:
* Refactored code in MarkStep to move most of the diagnostics logic into `ReportWarningsForReflectionAccess` - this is to mirror the design in ilc and also to make it more in sync with the already existing `ReportWarningsForTypeHierarchyReflectionAccess`.

Test changes:
* Adapting existing tests to the new behavior
* Adding some new tests, specifically around reflection access to compiler generated code and token access to compiler generated code.
* Enables `CompilerGeneratedCodeAccessedViaReflection` for ilc

[mono] Use underlying type in RuntimeHelpers.GetSpanDataFrom (#87019)

* [mono] Use underlying type in RuntimeHelpers.GetSpanDataFrom

Make it work correctly for spans of enums

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

Note that in net8 RuntimeHelpers.CreateSpan<T> is an intrinsic, so
GetSpanDataFrom is never called directly.

But in net7 CreateSpan is not intrinsified on Mono, so the underlying
method really does get called.

* test: Print all hidden field names if we can't find the right one

[main] Update dependencies from dotnet/emsdk dotnet/cecil dotnet/sdk (#87054)

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

Microsoft.NET.Workload.Emscripten.Current.Manifest-8.0.100.Transport
 From Version 8.0.0-preview.6.23281.1 -> To Version 8.0.0-preview.6.23301.1

* Update dependencies from https://github.com/dotnet/cecil build 20230601.1

Microsoft.DotNet.Cecil
 From Version 0.11.4-alpha.23265.2 -> To Version 0.11.4-alpha.23301.1

* Update dependencies from https://github.com/dotnet/sdk build 20230602.2

Microsoft.DotNet.ApiCompat.Task
 From Version 8.0.100-preview.6.23301.17 -> To Version 8.0.100-preview.6.23302.2

---------

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

[RISC-V] Sync comment and implementation (#87033)

enable DATAS config via runtimeconfig.json (#87034)

* enable DTAS config via runtimeconfig.json

* updating config name and switching it to INT_CONFIG

Dispose of linked CTS in QuicListener (#87057)

[wasm] Remove dependency on definition of MonoClass (#87077)

The jiterpreter recently gained a dependency on the definition of MonoClass which broke debug builds. This PR changes it to use accessors that are available in both configurations.

[wasm] Disable System.Text.Json AOT tests due to crashes (#87079)

V8, and chrome crash currently when running the tests.

Issue: https://github.com/dotnet/runtime/issues/86164

Remove legacy stacksampler logic (#87076)

* Remove legacy stacksampler logic

* Remove ALLOW_SXS_JIT_NGEN

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

Remove win-arm RIDs (#87074)

Create DiagnosticInfo as an incremental Diagnostic replacement type. (#86908)

Creates DiagnosticInfo record to replace Diagnostic in the incremental pipeline to work around dotnet/roslyn#68291