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>

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

* add DOTNET_EnableCrashReport

* Enable JitStress and JitStressRegs by default to see some test failures

* Enable stress switches only for JIT

* fix the build errors

* Revert "Set DNER for all local fields (#77341)"

This reverts commit 4e85471628214a3e6257132cadd8f7fd33299eb0.

* Set EnableCrashReport in testenvironment.proj

* Pass --crashReport to capture for hangs

* Comment bunch of jobs to trigger

* Comment all runs except Linux_x64

* Stop running libraries job

* Fix the tag in testenvironment.proj

* Use keepnativesymbols

* Update JitStress/JitStressRegs

* Embed native_image_offset field in crashreport.json

* Add TryPrintStackTraceFromCrashReport()

* Call TryPrintStackTraceFromCrashReport() even for crashes

* Add newtonsoft in CoreCLRTestLibrary.csproj

* Fix the crash dump folder

* Add random failure in jit

* Add some logging around TryPrintStackTraceFromCrashReport

* Move the error randomization to GetLayout() that has compiler object access

* Seach for crashreport.json files

* Switch to System.Text.Json

* fix the json path

* Add some logging

* Fix the clrjit symbols

* Add a comment

* Add few more commnets

* chown

* ls -l, chmod

* Change to chown

* USER environment variable

* update runtime.yml

* fix runtime.yml

* add logging

* Add llvm-symbolizer -h

* Skip superpmi pipelines

* read contents before process exit

* Add error output also

* fix some parameters for llvm-symbolizer

* Also enable OSX_x64 runs

* Remove the debugging code:

* Revert "Revert "Set DNER for all local fields (#77341)""

This reverts commit 5cfff119dbd8dd7a925a5db6fdb33f4f6487cb3a.

* Undo JIT changes to trigger assert

* Undo some yml changes

* Review feedback

* Undo runtime.yml changes

* Add some comments

* Address review feedback

Temporarily disable eventsourceerror test for Windows ARM32 (#81244)

[JIT] - X64 - Peephole optimization to possibly skip emitting `and reg, -1` (#80468)

* Optimize 'x & -1' to a cast

* Check for small type

* Update gentree.cpp

* Use peephole optimization instead as it catches more cases

* Simplify logic

* Fix comment. Added test case.

* Update IntAnd.cs

* Check for set flags

* Check explicitly for TYP_INT

Improve -static-pie support (#81163)

Improve error for missing Python (#81202)

* add error

* Fix a case were missing VC components didn't error

* handle non empty stdout

* Remove self assignment

Remove defensive Task[] copy from non-generic Task.WhenAll (#81065)

When calling it with a `Task[]` or `List<Task>`, we'll no longer generate a temporary array; the only allocation in those cases will be for the returned Task itself, at least if it completes successfully (if multiple of the constituent tasks fail or are canceled, we'll end up with an additional list allocation beyond what we had before).  If you call it with an enumerable that's not an `ICollection<Task>`, we'll also save a `Task[]`.

[wasm] Add Vector.Normalize measurement (#81237)

JIT: build pred lists before patchpoint expansion (#81196)

Move pred list building 3 phases earlier. It now happens just after
instrumentation (or importation if we're not instrumenting) and just before
expanding patchpoints.

Revise the patchpoint, indirect call, and post importer cleanup phases
to do proper pred list maintenance.

Update the flow checker to handle cases we see when we haven't yet run
the post importer cleanup.

Contributes to #80193.

Use RunContinuationsAsynchronously for block completion tasks (#81013)

* Use RunContinuationsAsynchronously for block completion tasks

* Tweak test

Co-authored-by: Petr Onderka <gsvick@gmail.com>

Fix Configuration Binding with Instantiated Objects (#81050)

[mono][wasm] Avoid saving the result of OP_AOTCONST into the gc pin area, its not needed. (#81179)

Disable the /JIT/opt/ObjectStackAllocation/ObjectStackAllocationTests (#81195)

* Disable the /JIT/opt/ObjectStackAllocation/ObjectStackAllocationTests

Disable the test until a dotnet with a fix of the issue #81103 is used
in the runtime repo to build / run tests.

* Moving the disabling to crossgen2 specific section

Fix a variety of issues in FrozenCollections (#81194)

- Add the new ToFrozenDictionary/Set methods that take optimizedForReading and make the existing overloads cheaper for construction.  This makes construction time and allocation by default much closer to that of a normal dictionary or set while also enabling more of the implementation to be trimmed away by default.
- Fixed the use of FrozenSet/Dictionary.Empty such that we only use the singleton when its comparer matches; otherwise, we could end up handing back an instance with the wrong comparer.
- Avoided passing both a dictionary/set and a comparer down through constructors, as the dictionary/set contains the comparer and we don't want to accidentally end up in a situation where we pass through the wrong one.
- Condensed some of the construction code to be a bit less verbose
- Removed unused locals from ToFrozenDictionary (the min/max/range were no longer being used)
- Remove unnecessary array allocations from PickIntegerDictionary
- Delete the sparse range integer sets, which are rarely needed and are broken for TryGetValue
- Fix CheckUniqueAndUnfoundElements to slice down to only the relevant size prior to clearing