Decrease backlog cleanup inactivity threshold (#77288)

Decreases the inactivity threshold for `backlog-cleanup-candidate` automation from 5 years to 4.5 years. This change will directly impact [approximately 100 inactive issues in our backlog](https://issuesof.net/?q=repo%3Aruntime%20is%3Aopen%20is%3Aissue%20updated%3A%3C2018-04-20).

Disable DisasmChecks if DOTNET_JitForceControlFlowGuard is set (#77467)

* Disable DisasmChecks if DOTNET_JitForceControlFlowGuard is set

* Update src/tests/Common/CLRTest.Jit.targets

Co-authored-by: Jakob Botsch Nielsen <Jakob.botsch.nielsen@gmail.com>
Co-authored-by: Jakob Botsch Nielsen <Jakob.botsch.nielsen@gmail.com>

statically link gc_pal on windows (#77468)

* build coreclr using gc_pal on windows

Also removes NUMA related code from coreclr PAL

* CR feedback

Clean up converter initialization logic. (#77290)

Detect specifical character in EnumConverter.cs (#76873)

* add specifical char flag and fail earlier before invoke naming policy

* add unit test

* improve unit test and special char check

* fix format error

* fix unit test

* take comments and improve unit test

* use string.IndexOfAny

* throw exception in constructor and improve unit test

* add custome exception message

* take comments

* Update src/libraries/System.Text.Json/tests/System.Text.Json.FSharp.Tests/EnumTests.fs

* Update src/libraries/System.Text.Json/tests/System.Text.Json.FSharp.Tests/EnumTests.fs

* Update src/libraries/System.Text.Json/tests/System.Text.Json.FSharp.Tests/EnumTests.fs

* Update src/libraries/System.Text.Json/src/System/Text/Json/Serialization/Converters/Value/EnumConverter.cs

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

PGO: Add instrumented tiers (#70941)

Co-authored-by: Koundinya Veluri <kouvel@users.noreply.github.com>
Co-authored-by: Andy Ayers <andya@microsoft.com>

Handle GT_IND in optVNConstantPropCurStmt (#77105)

Add README.md to System.Text.Json (#77435)

Resolve tvOS/iOS FileSystem test failures (#74927)

* Use SetCurrentDirectory on tests which use an unrooted path on
non-Windows out of the app's working directory, which is readonly on
some platforms.

Closes: #67853

* Just blank out mkfifo tests

Audit MemoryExtensions.IndexOf variants (#75754)

* Add missing char specialisation to LastIndexOf(Span, ROS)

This method is updated to match LastIndexOf(ROS, ROS)

* Forward LastIndexOfAny(Span, ROS) to (ROS, ROS) version

This is consistent with IndexOfAny(Span, ROS)

* Add missing special cases for {Last}IndexOfAny<byte>(ROS, ROS)

Add {Last}IndexOfAny{Except}InRange (#76803)

* Add {Last}IndexOfAny{Exception}InRange

* Address PR feedback

* Address PR feedback

[tests] Skip NamedPipeTest_CurrentUserOnly on iOSlike and LinuxBionic (#77471)

* [tests] Disable NamedPipeTest_CurrentUserOnly on iOS/tvOS

* [tests] Disable NamedPipeTest_CurrentUserOnly on Linux Bionic

Refactor Arm64 ConvertToXXX instrinsics for amd64 (#77211)

Remove the invalid assert (#77462)

Adding EVEX encoding pathways for emitOutputRRR(). (#75934)

Adding flag to turn on EVEX encoding.

Implement Array.Initialize in C# (#77336)

* Implement Array.Initialize in C#

* Add Mono implementation of Initialize

* Fix test failure

* Implement for NativeAOT

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

statically linking GC PAL on linux (#76985)

* statically linking GC PAL

The GC PAL will be used for both coreclr and standalone GC on linux

* fixing arm64 and nativeaot build breaks

* macos build break and reducing renaming.

* trying to remove numa support from PAL

* one more rename to resolve MacOS break

* delete pal numa code.

* Adding missing madvise in GC PAL

* added missing MADV_DONTDUMP calls.

* CR feedback

* undo (long long) cast in GetMemoryStatus

* only invoke madvise on success.

Add runtime-coreclr pgostress YAML file (#77446)

Add readme files to the globalization libraries (#77436)

JIT: implement tail merging (#77103)

Add a phase that looks for common tail statements in a block's
predecessors and merges them.

Run it both before and after morph.

Also
* add range enable config and overall `JitEnableTailMerge` config
* add indir flag checking to `GenTree::Compare`
* remove an apparently unnecessary assert from loop recognition.

Closes #8795.
Closes #76872.

README for extensions libraries (#77413)

Move usages of `COMPlus_` var to `DOTNET_` fallback (#77239)

Fold static readonly struct X = <zero> to a jit-time const, e.g. Guid.Empty (#77354)

Disable SIMD intrinsics for Vector128/64 on Arm64 (#77406)

Fix documentation for INumberBase IsComplex and IsImaginary (#77330)

* Fix documentation for INumberBase IsComplex and IsImaginary

* Update src/libraries/System.Private.CoreLib/src/System/Numerics/INumberBase.cs

Fix OCSP test responder to omit fractional seconds

[mono][tests] Fix FullAOT runtime generic methods test  (#77079)

* get vtable method slot by using its declaring

SPMI: Implement missing NAOT support (and fix garbage values passed to JIT in expandRawHandleIntrinsic) (#76996)

Implement expandRawHandleIntrinsic in SPMI

Also fix NAOT passing garbage back to JIT.

Fix #76925

Enable multi-reg MUL_LONG/CALL nodes on 32 bit (#76263)

Add startIndex argument to getStringLiteral JIT/EE interface method (#77423)

* Add startIndex argument to getStringLiteral JIT/EE interface method

Avoid copying up to 1kB string just to get a single char out of it.

* Format patch

removed duplicate conditional compilation (#77428)

Remove custom Utf8 encoder implementation. (#77400)

Co-authored-by: kasperk81 <kasperk81 kasperk81 83082615+kasperk81@users.noreply.github.com>

[mono] Avoid checking for exceptions after calls to Math icalls. (#77347)

* [mono] Avoid checking for exceptions after calls to Math icalls.

Also add a way to turn off exception checking for arbitrary icalls.

* Fix wasm support.

[wasm] Use latest stable version of chrome for testing (#77133)

* Use the latest `stable` chrome version by default
* Install these by default on codespaces

## Details

- Added a new task `GetChromeVersions` to get the latest version
- This is applicable to library tests, debugger tests, and Wasm.Build.Tests
- The channel used for getting the "latest" version is `$(ChromeChannel)=stable`
- And they are installed in `artifacts/bin/chrome`, and `artifacts/bin/chromedriver`
- A newer version is fetched only `$(ChromeVersionCheckThresholdDays)=3` days after the last version check
- This is enabled by default on containers
- And the behavior can be disabled with `$(InstallChromeForTests)=false`.

Add null check before calling Path.GetFullPath (#76696)

Consolidate the `GUID` to string logic (#77394)

* Consolidate the GUID to string logic

Replace obvious uses of GUID to string
logic with a new constant for buffer length.

SPMI: More information on CI failures (#77361)

This enhances parts of the SPMI process with more information on failures.

Also fixes a bug I introduced when I added tpdiffs: when asmdiffs fails, it was not marking the entire pipeline as failing.

Fix reported PriorityQueue issue. (#77229)

* Fix #77212.

* Remove unneeded using statement.

Detriplicate internal SingleProducerSingleConsumerQueue (#76932)

System.Private.CoreLib, System.Threading.Tasks.Dataflow, and System.Threading.Channels have all ended up with their own copy of SingleProducerSingleConsumerQueue and its associated helpers.  This consolidates them down to a single shared copy.

There's no functional change here, just deleting duplicates and moving things around.

Fix copy propagation dump (#77389)

Fold primitive-typed access to promoted structs in local morph and forbid mismatched struct assignments (#76766)

* GenTree::GetLayout field FIELDs

* Do not create invalid IR when replacing promoted fields

Instead, create constructions like "OBJ(ADDR(LCL_VAR))"
that block morphing can recognize on its own.

* Delete lvFieldHnd

* GenTree::GetLayout - CALL/COMMA

* GenTreeHWIntrinsic::GetLayout

* Add an assert that LHS and RHS match

* Fix assert violation

And simplify the code...

* Delete RetypedAsScalarFieldsMap

Without handle equality, the assert no longer holds, for example:
```cs
private StructWithInt Problem(StructWithInt b, StructWithInt a)
{
    a = ((StructWithStructWithInt*)&b)->StructWithInt;
    return a;
}
```

* Delete a bit of code

Dead / unncessary.

No diffs.

* Fold promoted locals in local morph

* Support GT_IND in MorphStructField

This is significantly simpler than moving the promotion logic
to post-order because we don't need to fiddle with the ref
counting process and complexities of intermediate states, e. g.
"IND<float>(ADDR(FIELD<int>(ADDR(LCL_VAR<struct>))))" can be
naturally turned into "BITCAST<float>(LCL_VAR<int>)" like this.

* More code removal

* Move "fgMorphStructField" to local morph

* TP tuning

With this, we have only a very small regression:

Base: 1297463478, Diff: 1297750893, +0.0222%
LocalAddressVisitor::MorphStructField : 345099  : NA       : 31.52% : +0.0266%
LocalAddressVisitor::PreOrderVisit    : 224684  : +3.53%   : 20.52% : +0.0173%
memset                                : 88591   : +0.99%   : 8.09%  : +0.0068%
Compiler::fgMorphSmpOp                : -19170  : -0.06%   : 1.75%  : -0.0015%
Compiler::fgMorphStructField          : -367474 : -100.00% : 33.56% : -0.0283%

(This is with a dummy field on LclVarDsc)

Losing the assert does not seem worrysome as we have an identical one in "fgMorphField".

[wasm] Add `IMemoryView` as exported type (#77371)

- Add IMemoryView as exported type.
- IMemoryView extends IDisposable.

Fix comment (#77226)

Update def flags in gtCloneExpr (#77378)

Limit to cases where are actually performing replacements to mitigate TP impact.

Revert "Override ReadAsync and WriteAsync methods on ConsoleStream. (#71971)" (#77360)

This reverts commit 8cb4e934757b959bc1b38aecaa764e21a47ec85e.

SPMI: Fix repPrintObjectDescription and repGetStringLiteral (#77372)

repPrintObjectDescription was not null-terminating when the map was
null. There is SPMI error trap on the JIT side so we can just make it
fail on missing data.

repGetStringLiteral should not succeed on missing data, it is used for
optimization purposes.

SmtpClientTest Assertion Fail Fix (#76361)

* Eliminate temporary -1 on Socket.Unix.cs

Add ReadOnly{Observable}Collection/Dictionary.Empty (#76764)

* Add ReadOnly{Observable}Collection/Dictionary.Empty

* Revert XamlLoadPermission.cs change to fix downlevel build

Temporary instrumentation for #76280 (#77365)

[Android][tests] Replace hardcoded /tmp path (#77291)

Akin to #73067

[mono][interp] Small improvements to get span item (#77331)

* [mono][interp] Improve GETITEM_SPAN intrinsic

Field offsets are known at compile time, they don't need to be included in the code stream.

* [mono][interp] Add version of getitemspan that doesn't use indirection

* [mono][interp] Add general MonoSpanOfVoid struct and use it in interp opcodes

Set DNER for all local fields (#77341)

Not just the promoted ones.

Delete removed tests from list (#77344)

Update Value of TwoDigitYearMax to 2049 (#76848)

Co-authored-by: Dan Moseley <danmose@microsoft.com>

JIT: Use metadata names for SIMD intrinsic method recognition (#76786)

* JIT: Use metadata names for SIMD intrinsic method recognition

eeGetMethodName is only expected to be used for diagnostics.

* Reorder checks in impSIMDIntrinsic

* Suggested refactoring

* Avoid unnecessary metadata lookups in getMethodNameFromMetadata

* Fix nested lookup

* Fix repGetMethodNameFromMetadata

Remove some unnecessary path manipulation helpers (#77301)

Rename COMPlus_ to DOTNET_ (#77321)

Fix `CacheEntryExtensions.SetValue()`'s signature to allow `null` values (#77305)

Sync shared code from aspnetcore (#77323)

[tests][Android] ReAdd Android library test jobs (#77280)

Android library tests were not running on CI in runtime-extra-platforms.
This PR brings back the jobs that ran library tests on Android device/simulator removed in #76328
They are updated to follow the similar changes made to branch out runtime-extra-platforms in the aforementioned PR.

JIT: Clean up inliner substitution (#77176)

* Stop using GenTree::ReplaceWith
* GenTreeRetExpr now directly contains its substitution members, which
  is a tree either from the inline candidate, a local (due to spill
  temps) or the call itself (due to inliner failure). In the former
  case, it also contains the BasicBlock the child tree comes from to
  ensure that its flags can be propagated once and for all during
  substitution.
* Make various members strongly typed, e.g.
  GenTreeRetExpr::gtInlineCandidate is now a GenTreeCall*,
  InlineCandidateInfo::retExpr is now a GenTreeRetExpr*. This is
  possible since we are no longer using GenTree::ReplaceWith.

Fix #75827

[wasm] Misc fixes (#76942)

* [wasm] Fix Wasm.Build.Tests with stable 7.0.0

Set the manifest package version to use (`7.0.0` instead of `7.0.0-ci`)
early.

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

* Misc fixes based on review feedback

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

* Update WorkloadBuildTasksDir to remove 'publish' in the path

* workloads-testing.targets: ignore symbols.nupkgs when looking for runtime packs

* Fix WBT to have the correct version for WORKLOAD_PACKS_VER

* [wasm] Emit an error if PackageVersionForWorkloadManifests property is not set

* workloads-testing: Get the PackageVersion at build time

.. because it is not set at evaluation time. So, get it in a target, and
move the WorkloadIds to a target too.

* Ensure the version is set before generating the runscript for WBT

Simplify DynamicMethod arg validation (#77277)

Filter out addresses that are not in bookkeeping range during background promote (#77067)

[ppc64le] Implementation of mono_arch_get_delegate_virtual_invoke_impl method for ppc64le architecture (#77308)

* [ppc64le] Fixed thunk address 8 byte alignment issue

* Fixed FSharp crash issue

* [ppc64le] Implementation of mono_arch_get_delegate_virtual_invoke_impl method for ppc64le architecture

Fix GCShadow for regions (#77011)

Remove _fdopen from PAL (#77272)

This function is never used.

Clearing all cards for free regions (#77224)

Migrate host.tests from Newtonsoft to STJ (#76901)

[mono][jit] Disable inlining for constrained calls on gshared parameters. (#77274)

The concrete method cannot be looked up so 'cmethod' is still the interface method.

Related: https://github.com/dotnet/runtime/pull/76724.

Fix link (#77297)

Fix the __int64 to be long long on macOS only (#77268)

It turned out that my recent change to redefine __int64 on Unix is
actually correct for macOS only. For other 64 bit Unixes, the __int64
should be defined as long.

This change fixes it.

Fix recurisve `Monitor.TryEnter` to not return false due to a condition (#77243)

A race condition when the object header is transitioning to a sync block can cause a concurrent recursive `TryEnter` to return false.

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

Remove uses of PAL's `swprintf_s` from mono. (#77284)

* Remove uses of PAL's swprintf_s from mono.

Add new version of the ContentionStart event (#72627)

* Add new version of the ContentionStart event and a LockCreated event

The new ContentionStart includes a lock ID, associated object ID, and the holding thread ID. The LockCreated event includes a lock ID and associated object ID.

Co-authored-by: Koundinya Veluri <kouvel@microsoft.com>

Don't invoke EnsureInstructionSetFlagsAreValid all the time in JIT (#77237)

Fix dotnet-pgo (#77259)

* Fix dotnet-pgo dump

* Fix merge compare-mibc and create-mibc too

* A couple more fixes

Adds `ActivatorUtilities.CreateFactory<T>` API (#77185)

Modify ARM64 thunktemplates.S to make it work with GNU ld (#77275)

Currently the ld linker links incorrectly the compiled thunktemplates.S
for ARM64 and puts wrong offsets into all the LDR instructions. Only
the LLVM lld linker works correctly.

I have looked into it and it turns out the assembler was putting in
relocations that the linker then replaced by fixed offsets. That's
where the GNU ld made the mistake.
I have figured out a slight modification of expressing the addresses
in the data page referenced by those LDR instructions so that the
assembler doesn't produce any relocs and so the linker cannot
cause any problems.

Fix a signed/unsigned comparison warning (#77230)

Trim file type bits from mode header (#77151)

* Trim file type bits from mode header

* Define and use ValidUnixFileModes

* Rev System.Formats.Tar.TestData version

* Move ValidUnixFileModes to shared helpers

Port stackoverflow fix from Roslyn to SourceGenerator PolyFill (#76955)

Add [DebuggerStepThrough] to MulticastDelegate constructors. (#77132)

Adding [DebuggerStepThrough] to these constructors allows debuggers to
avoid stepping into them and stopping regardless of Just My Code
settings. This is desirable for debuggers as there is little benefit to
stopping in these constructors as they are primarily infrastructure
inserted by the compiler.

Fixes issue when monitoring a process launched via the same command line (#76965)

* Initial fix for null reference exception

* Took away lazy and put back private constructor

* Added parent property and made handler thread safe

Update System.Speech to Eric/Jeff (#77142)

Make sure all heap segment fields are populated in the DAC (#77035)

Update .vsconfig file (#77189)

* Update .vsconfig; remove outdated and add new/missing components

* Update .vsconfig to sort the components

* Remove Blend component from .vsconfig

Remove more swprintf_s uses (#77228)

* Remove the swprintf_s export

* Convert paths to UTF-8 for flow graph diagnostics in JIT.

[wasm] IDBFS test

Added snake and kebab naming policies to JSON serializer (#69613)

* Added snake and kebab naming policies to JSON serializer

* Code styling issues

* Explicit types

* Fixed range slicing issue

* Fixed tests

* Forgotten conversion in tests

* Fixed docs

Co-authored-by: Daniel Stockhammer <daniel@stockhammer.it>
* Used nameof instead of hardcoded names in source generator

* Updated public API

* Fixed kebab case lower policy

* Added tests for long inputs

* Performance improvements

* Made ConvertName sealed

Co-authored-by: Eirik Tsarpalis <eirik.tsarpalis@gmail.com>
* Explicit variable type

* Clear only a dirty part of the buffer

* Fixed exception on slicing more that exists

* Better variable name

* End-to-end serialization tests

Co-authored-by: Daniel Stockhammer <daniel@stockhammer.it>
Co-authored-by: Eirik Tsarpalis <eirik.tsarpalis@gmail.com>

[wasm] Template nits 2 (#77205)

* Drop duplicated description from json definition.
* Update readme in both templates.

Update dependencies from https://github.com/dotnet/linker build 20221018.2 (#77214)

Microsoft.NET.ILLink.Tasks
 From Version 7.0.100-1.22514.1 -> To Version 7.0.100-1.22518.2

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

JIT: Support delegate GDV guards in loop cloning (#75140)

Add support for cloning loops based on delegate GDV guards. Mark delegate address loads as invariant to allow VN and CSE of them.

Remove exceptions on indirs in guards after loop cloning, which otherwise prevents RBO from optimizing them away if the loop is inside an EH handler (e.g. foreach loop).

[mono][interp] Don't use mov.vt when storing into primitive type fields  (#77221)

* [mono][interp] Rename opcodes to better suggest that they are doing a conversion

* [mono][interp] Don't use mov.vt when storing into primitive type fields

Use the faster mov.x opcodes instead.

[wasm][debugger] Remove non-deterministic signals from debugger tests. (#77038)

* Easy fix.

* Non wasm page loading detection.

* This is a good test, Sleep was redundant.

* This comment can be removed.

* Applied radical's idea to simplify the logic.

[wasm] few JS interop tests (#77218)

* new JS interop tests
Co-authored-by: Marek Fišera <mara@neptuo.com>
Co-authored-by: Ankit Jain <radical@gmail.com>

Add missing tests for NamedPipes (#72956)

* Add missing tests for NamedPipes

* Address feedback

* Move event wait to happen one line before Connect.

* Bring back previous version of test

* Reword SkipTestException message

* Move NamedPipeTest.CurrentUserOnly.cs to compile for all platforms

* Address feedback

* Remove IsRemoteExecutorSupportedAndOnUnixAndSuperUser

* Keep RemoteExecutor.IsSupported

[JIT] X64 - Three instruction replacement sequence for multiply in certain cases (#76981)

* Using 3 instruction sequence for x64 multiply

* Do not do this in morph. Do it in codegen now.

* Fixing codegen

* Only allow values under 127 and do not skip mov - correctness testing

* Try to fix tests

* cleanup

* Moving to Lowering

* Quick fix

* Fully works in lowering now

* Account for all ints

* Take into account codegen opts

* Minor cleanup

* Minor cleanup

* Fixed test

* Added int multiply disasm checks. Fixed SuperFileCheck namespace bug. Made SuperFileCheck anchors more likely to match.

* Update comments

* Update comments

* Update comments

* Update comments

* Formatting

* Fixing build

* Fixing build again

* minor rename

* Feedback. Removed use of FULL-LINE as it is more readable not strictly necessary. Forgot to add an additional instruction to a disasm test.

* Formatting

Add IR support for modeling multiple SSA definitions as one store (#76139)

* Change how we track use SSA numbers for partial defs

Store them in the SSA descriptor instead of a map.

Memory (x64): +0.18%
PIN    (x64): -0.04%

The memory regression is not small...

* SsaNumInfo support

* Support in SSA, VN and CopyProp

For now, just the "CanBeReplacedWithItsField" case.
This enables some nice simplifications, even as the
general case gets more complex.

Two quirks have been added to attain zero diffs.

* Support in dumping

* TP tuning

Gets us back 0.05% on the PIN counter. Hard to believe but true.

* More TP tuning

Another 0.025%.

Fold 'static readonly single-field-struct' to constants (#77102)

Co-authored-by: Andy Ayers <andya@microsoft.com>