[wasm] Enable jiterpreter traces; fix floating point compares (#81417)

This PR enables jiterpreter traces and fixes a bug in the implementation of floating point compares / floating point branches.

Address ConfigurationBinder feedback (#81384)

Co-authored-by: Eric Erhardt <eric.erhardt@microsoft.com>

Bump main branding to .NET 8.0 Preview2 (#81399)

* Bump main branding to .NET 8.0 Preview2

* Update the flow now that the version changed

* Manually bump emsdk to preview 2

* Update Versions.props

---------

Co-authored-by: carlossanlop <carlossanlop@users.noreply.github.com>
Co-authored-by: Alexander Köplinger <alex.koeplinger@outlook.com>
Co-authored-by: Larry Ewing <lewing@microsoft.com>

Remove manual DLL copy from ILCompiler build task (#79783)

Rather than manually copying DLLs from the NuGet package directory
we can just set an MSBuild property to copy package dependencies
to the build output. This is also safer since it includes all the
transitive dependencies.

Disallow setting Parent on the started Activity (#81394)

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

NativeAOT: Loop alignment support for xarch (#81206)

Encode calling convention in mangled function pointer representation (#81122)

This fixes the recently introduced issue #81106 tracking Crossgen2
crash due to not being able to distinguish two function pointers
differing only by calling convention (one of the pointers is Static
and the other is Static | UnmanagedCallingConvention).

This change also removes the issues.targets exclusion for the
new function pointer test that Fan Yang had to add to unblock
merging in the test in the first place.

Thanks

Tomas

* Delete superfluous comment per Michal's PR feedback

Rewrite SpanHelpers.IndexOfNullByte/IndexOfNullCharacter to unmanaged pointers (#81347)

[browser][MT] Calling JS functions from workers + config loading (#81273)

* calling JS functions from workers
* remove workaround
* config loading earlier

Fix nullability on `IDesignerSerializationProvider.GetSerializer` (#81203)

* Fix nullability on IDesignerSerializationProvider.GetSerializer

* Add nullability to ref IDesignerSerializationProvider interface

More frozen collection perf, test, and cleanup improvements (#81389)

* More frozen collection perf, test, and cleanup improvements

- Changed SmallIntegerFrozenDictionary/Set implementations to be based on `IComparable<>` rather than `IBinaryInteger<>`.  This enables them to be used with comparable types other than integers and also be used in pre-.NET 7 builds, as well as to simplify consuming code. Codegen isn't quite as good for the types it did work with, but it's good enough (and will hopefully improve in the future.
- Removed SmallInt32FrozenDictionary/Set implementations, which as a result of the above changes are no longer needed.
- Rewrote FrozenHashtable.Create to avoid creating so many temporary allocations as part of setting up the hashtable.  It was creating a list per bucket, which on average would be a list per element.  This was not only a lot of allocation but also a significant time sink, with benchmarks showing this can up to halve construction time.
- Sorted some .csproj includes
- A few more tests to improve code coverage. Line coverage for the Frozen namespace is up to 99.1%.

* Address PR feedback

* Apply suggestions from code review

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

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

[JIT] X64/ARM64 - `CAST` removals for small types on `ADD`, `SUB`, `MUL`, `AND`, `OR`, `XOR`, `NOT`, `NEG` (#77874)

JIT: Skip LSRA inserted nodes in gcIsWriteBarrierCandidate (#81423)

gcIsWriteBarrierCandidate is expected to return the same results during
LSRA and during codegen, so it needs to skip GT_COPY and GT_RELOAD
inserted on top of the data node.

Fix #77141
Fix #77143

Allow overriding WarningsAsErrors policy with an env variable (#81402)

* Allow overriding WarningsAsErrors policy with an env variable

Placing `WarningsAsErrors=false` in an environment variable will disable the repo policy to waste people's time with frivolous warnings in their inner dev loop.

* Update build.sh

[mono][jit] arm64 neon opcodes (#81026)

* [mono] ARM64 NEON opcode encodings, part 1.

* [mono] ARM64 NEON binary encodings, part 2.

* [mono] ARM64 NEON binary encodings, part 3.

* [mono] ARM64 NEON binary encodings, part 5.

* [mono] ARM64 NEON binary encodings, part 6.

* [mono] ARM64 NEON binary encodings, part 7.

* [mono] ARM64 NEON binary encodings, part 8.

* [mono] ARM64 NEON binary encodings, part 9.

* [mono] ARM64 NEON binary encodings, part 10.

* [mono] ARM64 NEON binary encodings, part 11.

* [mono] ARM64 NEON binary encodings, part 12.

* [mono] ARM64 NEON binary encodings, part 13.

* [mono] ARM64 NEON binary encodings, part 14.

* [mono] ARM64 NEON binary encodings, part 16.

* [mono] ARM64 NEON binary encodings, part 17.

* [mono][jit] ARM64/NEON opcode binary encodings, fixes.

Fix formatting in list-of-diagnostics.md (#81396)

* Fix formatting in list-of-diagnostics.md

I noticed the page currently renders with dangling `_` characters. This change cleans up the markdown formatting, changing the redundant `*_` to `_` and ensuring each is paried correctly.

I also improved alignment of the suppression table columns.

* Update docs/project/list-of-diagnostics.md

---------

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

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

* Update dependencies from https://github.com/dotnet/arcade build 20230127.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.23076.1 -> To Version 8.0.0-beta.23077.1

* Update dependencies from https://github.com/dotnet/arcade build 20230130.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.23076.1 -> To Version 8.0.0-beta.23080.3

---------

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

Add MCREGs to fix the build for FreeBSD/arm64 (#81397)

* Add MCREGs to fix the build for FreeBSD/arm64

* Use &&

JIT: Fix import of TrailingZeroCount on ARM64 (#81419)

Fix #81366

[wasm][debugger][libs] Documentation: How to debug library tests (#77368)

* Draft.

* DevTools as IDE instructions.

* VS Code as IDE instructions.

* Correction: `WebRoot` is not required.

* Added: expect long wait & new pic without webRoot

* Renamed as suggested by @pavelsavara.

* The first bp is not automatic, user has to set it on their own.

[wasm][libs] Fix `WasmAppHost`, and `AppBundle` for library tests (#77719)

* Removed fixed path separator + duplicated `browser-wasm`.

* Working fix.

* Fix for different `OutputPath` conventions.

* Fixing the url printed on the console.

* `ProcessQueryArguments` in test-main.js forces debug to be the 1st arg.

* Removing special case for WBT.

* We cannot skip checking for missing browser-wasm.

* Both runArgs and queryArgs are important when running tests.

* Revert changes in the shipped targets and move them to tests targets.

* Update src/mono/wasm/host/BrowserArguments.cs

Co-authored-by: Ankit Jain <radical@gmail.com>
* Fetch does not work for node<18 => get rid of json.

* Separated runtime args from app args for both hosts.

* Revert 7.0 file.

* Focusing on enabling libs debugging for browser.

* Should have been reverted with dae5d7460c3a23b108e1d141e72c3b0bcdc9d730.

* Fix CI lib tests: avoid referencing undefined properties.

* In all the cases runArgs should be first populated by defaults that can be later overwritten.

* Update src/mono/wasm/test-main.js

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

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

Enable CA1859 - Use concrete types when possible for improved performance (#80848)

Improve Http2Connection buffer management (#79484)

* Improve Http2Connection buffer management

* Add a test

* Add a few comments around buffer disposal

Improve and dedup GetNonZeroBytes (#81340)

- RsaPaddingProcessor has its own complete copy; that now appears to be an unnecessary duplication.
- If a zero byte is found, the implementation currently looks at the rest of the bytes byte by byte.  We expect 0s to be rare, so copy in bulk instead.

[Android] Fix System.IO.MemoryMappedFiles tests (#81368)

fix the space in build-job.yml (#81414)

Calculate PackageVersionNet6 off PackageVersionNet7 (#81401)

Update fgGetTopLevelQmark to handle GT_LCL_FLD (#81375)

Fully resolve WasmNativeWorkload and WasmNativeWorkloadAvailable (#81393)

Narrow PAL tests for _vsnprintf_s (#81381)

The .NET version of _vsnprintf_s defers to the
system provided printf so reducing the cases
to only what is needed for truncating validation.

Use SetupProperties to set properties related to NativeAOT and remove the top-level defaults (#79233)

Re-enable producing call stack from crashreport.json (#81387)

* Revert "Revert "Produce crashreport.json and use llvm-symbolizer to create stack trace (#77578)" (#81379)"

This reverts commit 3c76e140da9539e1fc0e0a44af0af3e01ceca920.

* Add nativeSymbols only for PRs

Inline resource strings in the compiler (#80896)

On a high level:
* When we're looking at IL to do substitutions, we additionally look for calls to `SR.SomeResourceName`. These are generated properties (generated by a piece of code in Arcade) that basically just do `GetResourceString(nameof(SomeResourceName))`. We look up what the resource string is (in the manifest resource) and replace the call with the looked up string literal.
* We also keep track of calls to `SR.GetResourceString`. Seeing this in the graph means that the optimization was defeated - someone bypassed the generated accessors. If we see one, we add dependency graph node to the graph that represent the manifest resource that has the string.
* When generating managed resources we skip over the one that has the strings unless the above dependency node is in the graph. This allows optimizing away the resource blobs if all accesses were inlined.

JIT: build pred lists before instrumentation (#81288)

Move pred list building to just before instrumentation (and just after
importation -- we are getting very close to the front of the phase list now).

The block and edge count instrumenters were both using cheap preds to keep
track of some relocated count probes. Revise this so they can use the regular
pred lists. Also rework both approaches so their `RelocateProbes` methods are
fairly similar and perhaps could be unified one day.

Contributes to #80193.

Revert "Produce crashreport.json and use llvm-symbolizer to create stack trace (#77578)" (#81379)

This reverts commit ff987dc063e954d5b88a14b28ca54c1a25f01294.

[wasm][debugger] Fix firefox tests (#81364)

* Fix firefox tests

* fix debugger tests on firefox

* fix firefox debugger tests

* remove unrelated change

* adressing radical comments

[wasm][debugger] Default null columnNumber to 0 (#81336)

* Default null columnNumber to 0

* Fix compilation

* adding test case

---------

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

Optimize the SqlGuid struct (#72549)

* Optimize the SqlGuid struct

Optimizes the SqlGuid struct according to the idea
from #51836.

* Update SQLGuid.cs

* Apply suggestions

* Update SQLGuid.cs

* Update SQLGuid.cs

* Update ref assembly

* Update System.Data.Common.cs

* Apply suggestions

* Update SQLGuid.cs

* Update SQLGuid.cs

* Update SQLGuid.cs

* Add a comment about binary serialization

* Expand on the comment for backwards-compatible binary serialization

---------

Co-authored-by: Jeff Handley <jeffhandley@users.noreply.github.com>

Remove additional null-check from ActivityCreationOptions (#81351)

[wasm] Misc fixes (#81320)

* [wasm] Catch warnings in wasm template project builds

This will help catch warnings like the ones fixed in https://github.com/dotnet/runtime/pull/77704

* [wasm][debugger] tests: fix warning

`[18:17:44] warn: Microsoft.AspNetCore.StaticFiles.StaticFileMiddleware[16] The WebRootPath was not found: /datadisks/disk1/work/B4180A14/w/992108AB/e/DebuggerTestSuite/wwwroot.  Static files may be unavailable.`

JIT: Avoid creating stores to dead fields in block morphing (#81095)

Fix #80498

Delete HeapEnableTerminationOnCorruption setting (#81343)

HeapEnableTerminationOnCorruption is enabled by default for subsystem major version 6 that we have set for our binaries. Also delete intranet link to a document that does not exist anymore.

Re-enable OneLocBuild (#81296)

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

Icu wasi package (#81345)

Update icu packages for wasi-wasm

fix fgMorphArgs for stress mode (#81346)

More thoroughly check for function pointers (#81344)

Fixes #81117.

[Android] Fix SslStream IsMutuallyAuthenticated on Android API 21-27

Add reflection path for ActivatorUtilities.CreateFactory (#81262)

Co-authored-by: Stephen Toub <stoub@microsoft.com>
Co-authored-by: Jan Kotas <jkotas@microsoft.com>
Fixes https://github.com/dotnet/runtime/issues/81258

Change Nullability annotation on `ISelectionService.PrimarySelection` (#81254)

* Change Nullability on ISelectionService.PrimarySelection

* Add nullability to ref ISelectionService interface

Disable function pointer test for crossgen due to a bug (#81286)

Eliminate a bounds check in `NumberBuffer` (#81039)

* Eliminate a bounds check in `NumberBuffer`

* Add AggressiveInlining

[main] Update dependencies from dotnet/icu dotnet/roslyn-analyzers (#81265)

* Update dependencies from https://github.com/dotnet/roslyn-analyzers build 20230126.2

Microsoft.CodeAnalysis.Analyzers , Microsoft.CodeAnalysis.NetAnalyzers
 From Version 3.3.5-beta1.23076.1 -> To Version 3.3.5-beta1.23076.2

* Update dependencies from https://github.com/dotnet/icu build 20230127.1

Microsoft.NETCore.Runtime.ICU.Transport
 From Version 8.0.0-alpha.1.23076.1 -> To Version 8.0.0-alpha.1.23077.1

---------

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

Fixing a comment attached to DEFAULT_UNROLL_LOOP_MAX_ITERATION_COUNT (#81300)

* Fixing a comment attached to DEFAULT_UNROLL_LOOP_MAX_ITERATION_COUNT

* Update src/coreclr/jit/compiler.h

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

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

Use IndexOf in RsaPaddingProcessor.FillNonZeroBytes (#81327)

Publish `UpDownCounter` and `ObservableUpDownCounter` Events for System.Diagnostics.Metrics (#81041)

Add support for publishing UpDownCounter (and Observable...) - also added in the missing testing for this.

[wasm] Update emsdk dependency to `8.0.0-alpha.1.23077.4` (#81312)

Added this with `darc add-dependency` this time, instead of a manual
edit.

Fix use-after-free in EventPipe (#81135)

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

Remove `printf` implementation (#81243)

* Remove PAL_printf and PAL_vprintf

Replace implementation of file related printfs using the
platform implementation (PAL_fprintf and PAL_vfprintf).

Remove tests for PAL_printf, PAL_vprintf, PAL_fprintf and
PAL_vfprintf since they are all now supplied by the platform.

* Remove underlying printf impl

Left _vsnprintf_s tests to ensure the _TRUNCATE
logic is validated.

Propagate cancellation tokens to TrySetCanceled in Dataflow (#80978)

* Propagate cancellation tokens to TrySetCanceled in Dataflow

When the System.Threading.Tasks.Dataflow library was originally written, CancellationTokenSource's TrySetCanceled didn't have an overload that allowed passing in the CancellationToken that was the cause of the cancellation. Now it does, and we no longer build for target platforms that lack the needed overload.  Thus we can update the library to propagate it everywhere that's relevant.  In some cases, to do this well we do need to rely on a newer CancellationToken.Register overload that accepts a delegate which accepts a token, so there's a little bit of ifdef'ing involved still.

While doing this, I also took the opportunity to sprinkle some `static`s onto lambdas, since I was already doing so for some lambdas as part of this fix.

* Fix pipelines handling of cancellation token

Flush operations were synchronously throwing an exception if cancellation was requested prior to the operation.  Cancellation exceptions should always be propagated out through the returned task.

* Fix pre-cancellation in QuicStream.WriteAsync

Cancellation exceptions should flow out through the returned task, not synchronously.

[wasm][mt] Don't call Module.removeRunDependency in workers (#81311)

Fixes a sporadic hang that depends on the order in which webworkers get initialized.

The way `removeRunDependency` works is that it decrements a counter (incremented by `addRunDependency`) and when the counter hits 0 it runs a callback that runs `run`.

The problem is that on workers there are no other dependencies, so `removeRunDependency("wasm_pre_init_essential")` ticks the counter down to 0 and calls `run`.

The second problem is that `run` on a worker short-circuits most of the logic and just calls `readyPromiseResolve`, `initRuntime` and `postMessage` and returns.

So the worker posts a message back to the main thread that it is loaded.

Then the main body of dotnet.js in the worker calls `run` a second time. And we send a second `postMessage`.

So the main thread receives 2 "loaded" message for each worker in the pool.

That messes up the counter in `instantiateWasmPThreadWorkerPool` which is waiting for every fresh worker to send just 1 message before allowing the main thread to continue.

As a result, by the time the main thread runs some user code, only half the workers have loaded.

Depending on the timing, that could mean that if the main thread attempts to start a new pthread, that pthread might be in a worker that hasn't loaded the runtime yet.  In that case, the pthread start function doesn't get called, and the main thread hangs in Mono's `create_thread()` helper waiting for the `start_info->registered` semaphore that will never be posted.

[mono][wasm] Avoid passing the icall name/c function name to mono_register_jit_icall_info () on wasm. (#81293)

It creates unused string literals.

[mono] Clean up StackTrace/StackFrame icalls. (#81303)

* Register them normally in icall-def.h instead of calling
  mono_add_internal_call ().
* Remove unused Mono.Runtime::mono_runtime_install_handlers
  icall registration.
* Use ObjectHandleOnStack to pass/return objects.
* Move some of the logic to icall.c.

[mono] Use mono_is_sre functions instead of checking class names in more places. (#81305)

Fix the error condition for objcopy (#81304)

[workloads] Add Current to the name of the workload manifest (#80401)

https://github.com/dotnet/sdk/commit/f5cb7e3c3eb631d13a1bdc6033f399cf8a1ceadb introduced a change that would make `microsoft.net.workload.mono.toolchain` an outdated id in order to work around manifest errors when older previews of the .NET 7 SDK were installed. The unfortunate side effect of this change meant the name of the runtime manifest needs to be changed.

On the main branch, the manifest id will now contain current in the name for the .net version being developed.

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

Forbid creation of non-faulting null-check nodes. (#77078)

* Forbid creation of non-faulting null-check nodes.

* Skip null check if op1 cannot be null.

* x86 bug fix for Skip gtChangeOperToNullCheck if it cannot be null.

* Fix Arm64 AVE: Do not lower indir when NOP.

* formatting fix.

* Arm64 regression bug fix that removes const bool for a variable.

* Preserve const and shorten expression.

* Fix format.

* Addressed code review feedback

* Forbid creation of non-faulting nullcheck node: Addressed feedback

* Forbid creation of non-faulting null-check nodes: Remove duplicate codes that was supposed to be deleted.

[mono][jit] Improve handling of the ldtoken+ltoken+Type::op_Equal optimization. (#81277)

Previously, only comparisons with gshared types were handled, but if gsharing
is disabled, its possible to run into cases where both ldtoken calls product
a concrete type.

Fix x86 clrgc loading (#81176)

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

Reduce the size of HttpRequestMessage and HttpRequestHeaders (#81251)

* Reduce the size of HttpRequestMessage and HttpRequestHeaders

* Assert that the request message can't transition from Disposed to non-Disposed

[wasm] Pin the sdk version to avoid a regression (#81301)

.. in manifest ordering in `IncludedWorkloadManifests.txt` caused by
https://github.com/dotnet/installer/pull/15346/commits/717f422202f5fb3494e8f79f96e3a27e83fc72c0
, which the workload causing relinking to break in WBT.

The version with the bug is : 8.0.100-alpha.1.23077.4

Revert "Fix generic parameter data flow validation in NativeAOT (#80956)" (#81259)

This reverts commit aa5e31310ef892995e7832f30acf8d78bfd0f5c2.

[browser][MT] conditional revert of wasm imports optimization (#81275)

* conditional revert of https://github.com/dotnet/runtime/pull/79466

* feedback

JIT: Switch tailcall implicit byref optimization to use last use information (#81033)

The last use information is more general in most cases (for example, the
old logic cannot allow the optimization when there are previous field
writes to the implicit byref). In addition the old logic can interact
with the new last-use copy elision optimization in an incorrect way.

Fix #81019

Change the default loop unroll limit to 4 (#80353)

* Change the default loop unroll limit to 4

* Remove verification testing for two complex tests involving both loop unrolling and large amounts of dead code elimination

Ensure various scalar cross platform helper APIs are handled directly as intrinsic (#80789)

* Ensure various scalar cross platform helper APIs are handled directly as intrinsic

* Small refactoring to lookupNamedIntrinsic and impIntrinsic to improve TP

Make System.Reflection.Emit public types abstract (#78544)

* Add abstract AssemblyBuilder

* Create managed RuntimeAssemblyBuilder

* Add RuntimeAssemblyBuilder to the build

* Boilerplate for RuntimeAssemblyBuilder

* Emit Assembly record

* Fixups after rebase

* Add temporary ReflectionEmitLoadContext

* Factor out RuntimeModuleBuilder

* Abstract TypeBuilder and EventBuilder

* Refactor PropertyBuilder

* Refactor EnumBuilder

* Refactor FieldBuilder

* Refactor ConstructorBuilder and MethodBuilder

* Move suppressions into the common file

---------

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

Target lower glibc for Linux arm64 (#80939)

This uses new cross-build images that target Ubuntu 16.04,
lowering the glibc requirements. The initial change is only for
Linux arm64, to match what we plan to do in .NET 7. The 8.0
change will let us validate the PGO instrumentation, which
happens in official builds, but not in PR jobs.

Add MemoryExtensions.Count (#80662)

* Add MemoryExtensions.Count

* add test cases

* fix implementation

* fix review comments part I

* more tests

* return 0 if value is Empty

* Address PR feedback

---------

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

Make sure we specify arguments only when the test ask for it (#80947)

Avoid using a null heap to decommit the mark array (#80640)

[wasm] Force partial trim mode for browser bench (#81284)

Before we fix json [de]serialization to work with full trim

JIT: update `fgReplaceJumpTarget` to maintain pred lists (#81246)

This is used some early phases; make it pred list aware.

Contributes to #80193.

Avoid allocations in 'Transfer-Encoding: Chunked' check (#81253)

Use IndexOf in StringBuilder.Replace(string, string, ...) (#81098)

* Use IndexOf in StringBuilder.Replace

StringBuilder.Replace(string, string, ...) currently walks character-by-character to find the next location of the oldValue.  This change adds the use of MemoryExtensions.IndexOf to search for the oldValue, and only falls back to the character-by-character walk when it nears and needs to account for a chunk boundary.  It also switches to track replacement indices using more stack space and the array pool (via ValueListBuilder) rather than less stack space and array allocations.

* Fix faulty assert

Update dependencies from https://github.com/dotnet/arcade build 20230126.1 (#81263)

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.23075.2 -> To Version 8.0.0-beta.23076.1

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

Disposed change token registration should not raise consumer (#79966)

Update analyzers.md (#79768)

* Update analyzers.md

* Apply suggestions from code review

* Update analyzers.md

* Apply suggestions from code review

Co-authored-by: Stephen Toub <stoub@microsoft.com>
* Apply suggestions from code review

---------

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

Fixed CA1854 occurrences (#80996)

[mono] Re-enable System.Threading.Tasks tests for desktop interp (#81046)

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

JIT: consolidate inlining code in fgInline.cpp (#81198)

When I reorganized and split up the flowgraph.cpp a while back I created
fgInline.cpp to hold all the inlining related methods, but didn't move
the inlining related methods from morph.cpp. Now I am moving them.

[NativeAOT] Objective-C exception propagation (#80334)

* [NativeAOT] Objective-C exception interception

* Fix Windows build

* Add back UnwoundReversePInvoke flag after re-initing.

* Switch to preemptive mode before jumping to propagation callback

Co-authored-by: Jan Kotas <jkotas@microsoft.com>
* Fix windows AMD64 offsets.

[mono][interp] Reduce redundant moves with calls (#79905)

We were marking INTERP_LOCAL_FLAG_NO_CALL_ARGS for source vars of non calls. With further optimizations, the var could end up being used only for the call but we never unset the flag. Reuse the ref count data, which is kept updated following optimizations, to achieve same thing. If cprop is not run then we always duplicate the args, but this is an unused scenario.

[mono][interp] Fix type pushed by mkrefanytype (#81261)

This instruction pushes a RuntimeTypeHandle.

[mono][aot] Emit native-to-managed wrappers for [UnmanagedCallersOnly] methods in non-full aot mode as well. (#80918)

Add additional bool-only ToFrozenDictionary/Set overloads (#81256)

API review decided to include additional overloads to enable only specifying the bool.

Mono musl support (#76500)

Co-authored-by: Alexander Köplinger <alex.koeplinger@outlook.com>
Fixes https://github.com/dotnet/runtime/issues/76805

Skip tests that need RemoteExecutor (#81231)

Make a number of improvements to the System.Text.Json source generator (#81239)

* Ensure singleton JsonTypeInfo instances are preconfigured using JsonSerializerOptions.

* Fix indentation & whitespace issues in the source generator.

* Fix #76829.

* Fix nullability annotations in GetTypeInfo gen.

* Add clarifying comment.

* Add testing for types without metadata or fast-path generation.

[mono][interp] Add @kotlarmilos as code owner (#81209)

Replace successive "ldr" and "str" instructions with "ldp" and "stp" (#77540)

* Replace successive "ldr" and "str" instructions with "ldp" and "stp"

This change serves to address the following four Github tickets:

    1. ARM64: Optimize pair of "ldr reg, [fp]" to ldp #35130
    2. ARM64: Optimize pair of "ldr reg, [reg]" to ldp #35132
    3. ARM64: Optimize pair of "str reg, [reg]" to stp #35133
    4. ARM64: Optimize pair of "str reg, [fp]" to stp  #35134

A technique was employed that involved detecting an optimisation
opportunity as instruction sequences were being generated.
The optimised instruction was then generated on top of the previous
instruction, with no second instruction generated. Thus, there were no
changes to instruction group size at “emission time” and no changes to
jump instructions.

* No longer use a temporary buffer to build the optimized instruction.

* Addressed assorted review comments.

* Now optimizes ascending locations and decending locations with
consecutive STR and LDR instructions.

* Modification to remove last instructions.

* Ongoing improvements to remove previously-emitted instruction
during ldr / str optimization.

* Stopped optimization of consecutive instructions that straddled an instruction group boundary.

* Addressed code change requests in GitHub.

* Various fixes to ldp/stp optimization

Add code to update IP mappings when an instruction is removed.

* Delete unnecessary and incorrect assert

* Diagnostic change only, to confirm whether a theory is correct or
not when chasing an error.

* Revert "Diagnostic change only, to confirm whether a theory is correct or"

This reverts commit 4b0e51e87af44cbef5b9e643f9c5bf7aff48b548.

* Do not merge. Temporarily removed calls to
"codeGen->genIPmappingUpdateForRemovedInstruction()".

Also, corrected minor bug in instruction numbering
when removing instructions during optimization.

* Modifications to better update the IP mapping table for a replaced instruction.

* Minor formatting change.

* Check for out of range offsets

* Don't optimise during prolog/epilog

* Fix windows build error

* IGF_HAS_REMOVED_INSTR is ARM64 only

* Add OptimizeLdrStr function

* Fix formatting

* Ensure local variables are tracked

* Don't peephole local variables

Co-authored-by: Bruce Forstall <brucefo@microsoft.com>
Co-authored-by: Alan Hayward <alan.hayward@arm.com>
Co-authored-by: Alan Hayward <a74nh@users.noreply.github.com>