Fix superpmi.py script for asmdiffs (#39948)

Disable eventpipe test under GCStress due to test failure (#39978)

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

Disable gcdump test under GCStress due to test failure (#39977)

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

Remove GetIPCEventSendBufferContinuation (#39662)

* Fix write on failure to allocate buffer

* Remove GetIPCEventSendBufferContinuation altogether

[mono] Fix building multiple os/arch combinations in the same working directory (#39970)

After adding the ICU shim code to the runtime we hit an issue when e.g. compiling for "desktop" mono in a working directory that already had Browser wasm artifacts in it.
This is due to automake putting the intermediate compilation artifacts at the same location where a file was referenced from.

This means that e.g. when using `../../../libraries/Native/Unix/System.Globalization.Native/pal_icushim.c` in the Makefile.am we'd get the intermediaries in `artifacts/obj/mono/libraries/` instead of somewhere in `artifacts/obj/mono/OSX.x64.Debug`.

Later on we'd get the following error because it was reusing the existing .o file from another architecture:

```
Undefined symbols for architecture x86_64:
  "_gPalGlobalizationNative", referenced from:
    _c_qcalls in libmonoruntimesgen.a(libmonoruntimesgen_la-native-library-qcall.o)
```

Fix this by symlinking the source files into the build directory.

Fix issue with Crossgen2 package excluding Icon (#39972)

The latest packaging targets rely on ProjectDefaults inclusion of package icon,
but this item was being removed by the Crossgen2 project's targets.

Also updating the version of packaging tools to see if we have any other failures.

Keep only Abstractions dependency on Configuration.Binder (#39461)

Add nullability annotations to files in XPath folder (#39821)

* Add nullability annotations to files in XPath folder

* Address feedback from krwq

* Make changes based on recently merged annotations

* Mark resolver parameter in SelectSingleNode as nullable

Fix JSON possible dupe key issue (#39860)

[wasm] Mark DiagnosticSource.Switches.Tests with an active issue (#39952)

[master] Update dependencies from Microsoft/vstest mono/linker dotnet/icu (#39697)

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

Fix Half comparison (#39773)

* Fix Half comparison

* Fixes the comparison to be correct when both numbers are negative
* Add relevant tests

* Add parentheses for clarity

* Add tests validating comparison between negative numbers for Single/Double as well

Fix build on some libs projects when metadata isn't available (#39960)

* Fix build on some libs projects when metadata isn't available

* Update references.targets

[wasm] Remove System.ValueTuple.Tests test suite from the exclusion list (#39957)

Annotate System.Data.{Odbc,OleDb} for nullability (#39597)

* Annotate System.Data.Odbc for nullability
* Annotate System.Data.OleDb for nullability
* Suppress nullable warnings on netcoreapp < 3.0
* Mark System.Data.DataSetExtensions as null-annotated
* Fix null-check bug in System.Data.OleDb

Mark the multifolder GC2 test as GC stress incompatible until fixed (#39942)

Disable reverseouter test under GCStress (#39937)

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

Disable processinfo for ilasm round-trip testing (#39936)

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

Disable Microsoft.Extensions.Caching.Memory.CapacityTests.DoNotAddIfSizeOverflows (#39930)

Issue: https://github.com/dotnet/runtime/issues/33993,
https://github.com/dotnet/runtime/issues/39241

Mark Assembly.CodeBase / Assembly.EscapedCodeBase as obsolete (#31127) (#39261)

Use consistent access modifiers in AsnValueReader.cs

Remove unused locals in System.Numerics.Tensors (#39578)

Fix CoreLib path resolution (#39921)

Fixes issue described in https://github.com/dotnet/runtime/pull/39891#issuecomment-663767530.

[Arm64] Don't generate hardware intrinsic method bodies (#39753)

* Don't generate hardware intrinsic method bodies on Arm64 in zapinfo.cpp

* Treat Vector64 and Vector128 methods as intrinsics on Arm64 in zapinfo.cpp

Address slowdowns in type loader performance (#39841)

Remove CoreMangLib duplicate tests (#36612)

* Cleanup duplicated Type tests

* Baseline mono test failures

Add linker annotation to LazyDebugView to avoid warnings (#39899)

In reality this is only to get rid of the warnings, the class is never instatiated by the managed code, so the annotation has no effect on including more code.
When in debugger the class is instatiated with the same T of an existing Lazy<T> which has the same annotation, so the T will always fulfill the annotation's requirements.

MacOS managed debugging perf fix (#39804)

* MacOS managed debugging perf fix

This PR adds PAL_OpenProcessMemory, PAL_ReadProcessMemory and PAL_CloseProcessMemory that
encapsulate the remote read memory functions for both MacOS and Linux.

The DBI code that reads memory using the runtime IPC messages to use these new functions
and fallback to the IPC message if the open fails.

On MacOS, this won't have any affect on VSCode performance until vsdbg/vsdbgui are signed
with the "com.apple.security.cs.debugger" entitlement.

Sample entitlements.plist file:

```
<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
<plist version="1.0">
  <dict>
    <key>com.apple.security.cs.debugger</key>
      <true/>
  </dict>
</plist>
```

Example of locally signing with entitlement command:

codesign -s <local-cert> --entitlements entitlements.plist

Fix FreeBSD build. Remove FEATURE_DATATARGET4 for macOS so managed debugging uses faster OOP unwind for HelpMethodFrames

Fix `fgUpdateChangedFlowGraph`. (#39878)

`fgComputeDoms` has an assumption that the flow graph
has no unreachable blocks. It's checked with this assert:

https://github.com/dotnet/runtime/blob/ad325b014124b1adb9306abf95fdac85e3f7f2f4/src/coreclr/src/jit/flowgraph.cpp#L2342

This assert fired when testing #39474 (`Convert Math{F}.CoreCLR methods from FCall to QCall`)
when we are updating the flow graph after inserting GC polls.

This change switches `fgUpdateChangedFlowGraph` to call `fgComputeReachability`,
which both removes unreachable blocks and calls `fgComputeDoms`.

pin-icount reported a 0.0043% throughput improvement, which is within noise level.

Fix CanCastTo behavior (#39915)

* Fix missing cases in CanCastTo

* Update issues.targets to start running the previously broken tests

Avoid using DateTime - Switch to DateTimeOffset (#39916)

Fix issue in type equivalence involving arrays (#39914)

- Fix issue where single dimensional arrays and multidimensional arrays of rank 1 are considered equivalent

Nullable: System.Xml, part 5 (XmlDocument) (#39691)

* Nullable: System.Xml, part 5 (XmlDocument)

* apply feedback

AnsiParser improvement (#39729)

Fix issues with JitStressRange (#39754)

* Add 10 when digit is [a-fA-F] in ConfigMethodRange::InitRanges in utils.cpp

* Zero out compMethodHashPrivate earlier in Compiler::compInit() in compiler.cpp

Delete redundant MSVC compiler options (#39901)

Ensure CoreCLR pkgproj include packaging props (#39906)

Since these projects don't directly consume the nupkg they didn't
automatically get props when we added it.

Fix lcl fld addr. (#39424)

* Add a repro test.

* Small ref.

Combine long chains of `addr->OperGet() == GT_* ||` with `GT_LCL_VAR_ADDR` using `OperIs` to simplify future changes.

* Add an assert that would fire in the repro test.

`emitter::emitHandleMemOp` has special logic for contained `memBase` and but the last block does not
expect a contained node. A contained node doesn't produce a register so it is not correct to use result
of `GetRegNum()` from a contained node as a valid register.
However, adding an assert to `GetRegNum()` that `!this->isContained` is a bigger task that is out of this PR.

* Assert that `LCL_FLD_ADDR` is not contained in `genPutArgStk(Split)`

We have contained `LCL_VAR_ADDR` support there but make sure that contained `LCL_FLD_ADDR` can't reach it.

* Contain `GT_LCL_FLD_ADDR` under HW_INTRINSIC.

This is an additional optimization that makes future changes simpler.

* Add contained checks.

In all these places we expect `LCL_VAR_ADDR` to be contained.
If we had gotten a `LCL_VAR_ADDR` that is not contained we would have instantiated `LCL_VAR_ADDR` twice:
in the register and the parent instruction.
The register value would have been unused.

* Support `FLD_ADDR` where `LCL_ADDR` is supported.

However, fire an assert if we think that this path is unreachable for now.

* Delete asserts in the  reachable blocks.

We have coverage for this asserts in the following tests:
hwintrinsic 478: Ssse3_ro
instr 11645: Runtime_39403
instr 1028 : Aes_ro
hwintrinsic 716: pmi of Microsoft.Diagnostics.Tracing.TraceEvent

* Review response.

* Add repro cases.

Delete the rest `assert(!"don't expect GT_LCL_FLD_ADDR");`.

* Use `GetLclOffs` from `LclVarCommon`.

* missed file.

Remove setting ICUCORE for macOS in System.Globalization.Native CMake config (#39900)

Follow-up to #39833.

We don't link against ICU on macOS but load it via `dlopen()` at runtime and compilation uses headers from homebrew.
That means `HAVE_SET_MAX_VARIABLE` and `HAVE_UDAT_STANDALONE_SHORTER_WEEKDAYS` will always be defined and we don't need the ICUCORE variable.

* PR feedback

Co-authored-by: Adeel Mujahid <adeelbm@outlook.com>

Fix GC Poll inlining. (#39881)

* Don't inline GC polls in cold basic blocks.
* Allow GC poll inlining in basic blocks with `BBF_LOOP_PREHEADER` or `BBF_RETLESS_CALL` set.

This fixes one of the assert seen in https://github.com/dotnet/runtime/pull/39474#issuecomment-662033060

Contributes to resolving #39726.

Update to unified Alpine build image version (#39903)

Update our Alpine build image so we can unify for a single image for our builds.

Fixes #2030

Jit: fix some sources of x64 chk/rel diffs (#39888)

Break IGs for perfscores the same way in chk and rel. Also, fix the xarch
peephole to work across extended IGs.

Closes #39845.

New fix for duplicate methods in the AltJit jit-diff output (#39751)

Fix ActivitySource.StartActivity when start time is provided (#39884)

Post #35606 cleanup (#39891)

* Post #35606 cleanup

Convert math intrinsics to named intrinsics (#39730)

* Convert math intrinsics to named intrinsics

* Annotate Floor and Ceiling with [Intrinsic]

[debugger] Removing unhandled_exception which was used for android. (#39377)

Trying to remove the usage of unhandled_exception function to make debugger handle with an exception when running on android.
We should ignore the try catch that is in an WRAPPER_SUBTYPE_ICALL_WRAPPER, then we can walk through all the callstack and find any try catch in managed code if it exists.

Co-authored-by: thaystg <thaystg@users.noreply.github.com>

Add CBOR property-based tests (#39828)

* Add CBOR property-based tests

* address feedback

[browser][tests] Activate System.Private.Uri.Functional.Tests (#39817)

Local:

```
info: test[0]
        Tests run: 883, Errors: 0, Failures: 0, Skipped: 1. Time: 188.7140968s
```

[runtime] Fix some more gc tracking problems in create_cattr_named/typed_arg. (#39856)

Remove leading dot check for cookie domain (#39781)

Minimal fix for domain-related cookie issues of #26141

To fully comply with RFC 6265, one should remove deprecated cookie
properties, such as Version, from public API. So only the stated
issues with leading dot were addressed now.

Also note that the leading dot was not stripped from the domain even
though RFC 6265 proposed it. This behavior was chosen because
browsers like Chrome and Edge also don't strip the leading dot.

Add ReferenceConverter entry to intrinsic type converters table (#39854)

* Add ReferenceConverter entry to intrinsic type converters table

* Add tests

Remove unused pinvokes for WinRT scenarios (#39846)

* Remove GetRestructedErrorInfo P/Invoke

* Remove win32 core memory P/Invokes

* Remove RoGetBufferMarshaler P/Invoke

* Remove RoGetActivationFactory P/Invoke

* Remove CoreComm P/Invokes

* Remove IRestrictedErrorInfo definition.

[aot] Avoid a crash in generic sharing for invalid generic instances. Fixes https://github.com/mono/mono/issues/20138. (#39869)

Co-authored-by: vargaz <vargaz@users.noreply.github.com>

Annotate TypeConverterAttribute so that the ILLinker preserves the ctor on the converter (#39144)

* Annotate TypeConverterAttribute so that the ILLinker preserves the ctor on the converter

* Address review feedback & add attributes to ref

* Use custom enum converter type

* Split tests

Implementation of CSE for GT_CNS_INT benefits ARM64 (#39096)

* Change the type of csdHashKey to size_t

* Update gtCostSz and gtCostEx for constant nodes

* Implementation of code size optimization, CSE of constant values for ARM64
Implementation of code size optimization, CSE of constant values for ARM64
We will share a single CSE for constants that differ only in their low 12 bits on ARM64

Number of shared constant low bits set in target.h  CSE_CONST_SHARED_LOW_BITS
we use 12 bits on Arm platforms and 16 bits on XArch platforms

Disable the CSE of the REG_R2R_INDIRECT_PARAM on Arm32
as it hits  Assertion failed 'candidates != candidateBit' in lsra.cpp Line: 3723

Config variable: COMPlus_JitConstCSE
// Default 0: enable the CSE of Constants, including nearby offsets. (only for ARM64)
// If 1, disable all the CSE of Constants
// If 2, enable the CSE of Constants but don't combine with nearby offsets. (only for ARM64)
// If 3, enable the CSE of Constants including nearby offsets. (all platforms)
// If 4, enable the CSE of Constants but don't combine with nearby offsets. (all platforms)
//

* Added additional Priority 0 test coverage for Floating Point optimizations

* Fix for COMPLUS_JitConstCSE=4

* Renamed config variable from COMPlus_JitDisableConstCSE to COMPlus_JitConstCSE

* Updated with Codereview feedback, removed sort from Const CSE phase

* Fix for assertionProp issue in the refTypesdynamic test

WASM: Enable Invariant.Tests (#39814)

* WASM: Enable Invariant.Tests

* Fix build and PR Feedback

WASM: enable System.ObjectModel.Tests (#39849)

Remove obsolete ActiveIssue attributes for #23972 (#39842)

Extend Linker test job timeout to 2 hours (#39847)

Remove a new assert that can fire with unsafe code and type casts (#38912)

Add @dotnet/jit-contrib to CODEOWNERS (#39862)

Write error events for service errors (#39766)

* Write error events for service errors

* Explicitly specify EventLogEntryType.Error

Ensure the size of Vector<T> takes COMPlus_EnableHWIntrinsic into account (#39368)

* Ensure the size of Vector<T> takes COMPlus_EnableHWIntrinsic into account

* Add basic logging to Runtime_34587

* Exclude InstructionSet_POPCNT and InstructionSet_POPCNT_X64 if featureSIMD is disabled

turn pauseonstart test off for Windows (#39789)

Remove outdated comment in System.CM.Composition.Tests (#39839)

We don't plan to add a .NETFramework test configuration for this library anymore.

Remove depprojs in favor of PackageReferences  (#35606)

- Remove depprojs which currently binplace external references into the RefPath folders in favor of PackageReference and PackageDownload items.
- Build all configurations by default when building an individual project (either on the CLI or inside VS) same as with the official SDK. This enables .NETFramework Test Explorer support.
- Centrally define libraries that compose the shared framework instead of in each Directory.Build.props file to be able to build the targeting pack first and consume it in the OOB libraries.
- Use ProjectReferences to reference OOB projects. Compile against the reference assembly but use the implementation assembly app-local during runtime.
- Remove OOBs from the testhost and remove the testhost folder for .NETFramework as it isn't required anymore.
- Only binplace for $(NetCoreAppCurrent) to compose a) the targeting pack, b) the runtime pack, c) the testhost, d) a full closure for the shims.
- Use Targeting Packs for OOB projects (with their implicit assembly references) but still explicitly define granular references for .NETCoreApp configurations (DisableImplicitAssemblyReferences switch). Use the implicit targeting pack references in some Microsoft.Extensions.* cases.
- Remove placeholder configurations as they aren't needed anymore with explicit P2Ps vs Targeting Pack references.
- Remove implicit assembly references (ie for .NETFramework, mscorlib)
- Remove AssemblySearchPath hacks that were introduced with b7c4cb7 as the targeting pack is now used by default.
- Reduce unnecessary .NETFramework configurations that were added to run tests in favor of the already existing ref&src configurations.
- Stop hardcoding the paths for wasm assemblies and use the returned TargetPath of the ProjectReferences.
- Addressed formatting (ItemGroups, References at the bottom of the project file, ordering of references, use LibrariesProjectRoot instead of a relative path, unnecessary AssemblyName and RootNamespace properties which are identical to the project name, ordering of tfms)
- Revert "fix clean (#33758)"

Fix CNG persisted symmetric key operations

During a code cleanup the encoding changed from UTF-16LE
to UTF-8, which is not valid for the P/Invoke.

WASM: Fix System.Net.Primitives and tests (#39748)

Add HostInformationPal and InterfaceInfoPal implementations for Browser.

In CookiePortTest.cs use example.com instead of localhost for the tests since Browser uses `localhost` as the `Environment.MachineName` and that changes the test values.

Allows the tests to run on WebAssembly:
```
System.Net.Primitives.Pal.Tests: Tests run: 60, Errors: 0, Failures: 0, Skipped: 0. Time: 0.15872s
System.Net.Primitives.Functional.Tests: Tests run: 2620, Errors: 0, Failures: 0, Skipped: 1. Time: 3.145804s
```

[wasm] Enable System.Threading.Thread.Tests test suite (#39826)

Hardcode path to ICU on macOS (#39833)

Suppress credscan false positives (#38026)

* Suppress initial cred issues

* Another bunch of supresses

* Clean up

* Another bunch of supresses

* Revert to suppression messages

* Clean up

* Apply suggestions from code review

Co-authored-by: Jeremy Barton <jbarton@microsoft.com>
* Revert passwords literals

* Fix suppression justification comment

Co-authored-by: Jan Jahoda <jajahoda@.microsoft.com>
Co-authored-by: Jeremy Barton <jbarton@microsoft.com>

Switch Windows arm64 testing to use Windows.10.Arm64v8.Open queue. (#39802)

Windows arm testing has been scaled back in #39655 so now we can start
using better machines for Windows arm64 testing. These machines were
previously used for Windows arm testing.

Fix FCall implementation collision on ARM64 (#39810)

Two FCalls, DependentHandle::nSetPrimary and MarshalNative::GCHandleInternalSet, had identical implementations, which led to a failure in ECall::GetFCallImpl ("Duplicate pImplementation entries found in reverse fcall table"). Add FCUnique() to make them different.

Disabled tests with the PlatformDetection.IsPreciseGcSupported (#39796)

Try re-enabling IBC on macOS. (#39801)

[browser][file system] GetLogicalDrives implementation (#39763)

- Directory.GetLogicalDrives threw PNSE.  Follows existing code paths.
- Add common DriveInfoInternal.Browser that is common code path for other implementations
   - Environment.GetLogicalDrives
   - DriveInfo.Drives

Fix for FileSystemTest https://github.com/dotnet/runtime/blob/master/src/libraries/System.IO.FileSystem/tests/Directory/GetLogicalDrives.cs

Fix Failing Casing Tests with Invariant Culture (#39747)

Limit arm32 Windows testing. (#39655)

Remove Windows arm32 testing from PRs. Builds are not removed.
Remove Windows arm32 jitstress testing.
Remove Windows arm32 builds from official builds.

Outerloop still has Windows arm32 builds and testing.

[runtime] Make sure newly created objects are pinned during a runtime invoke in create_cattr_typed/named_arg (). (#39774)

Hopefully helps with https://github.com/dotnet/runtime/issues/39473.

Update the JITEEVersionIdentifier (#39779)

fix CborWriter bug when writing tagged empty collections (#39786)

Add events around type load and r2r usage (#39656)

- Allows for investigation of type loader performance
  - Uses a new keyword, as this event will generate many strings which is itself fairly slow
- Changes R2R events from Verbose events to Informational events
  - These events are already controlled by a separate keyword, and don't need to also be only emitted when Verbose eventing is on

[wasm] add zone.tab to dotnet.timezones.blat  (#39731)

* [wasm] add zone.tab to dotnet.timezones.blat to allow system time zones to load
* renable GetSystemTimeZones()
* reduce size of data file by removing unnecessary timezones

Remove BundleDir/PublishDir at the end of ArchiveTests to save disk space (#39714)

Hosted build agents have not a lot of free disk space and since we don't need these dirs anymore after we created the test archive we can delete it.

[Infra] collect dumps of all corerun in tree on timeout (#38647)

Add missing space padding before exception message in log (#39534) (#39719)

MEH test failure on mono (#39784)

Factory_CleanupCycle_DisposesEligibleHandler

Fix double-writes in test projects (#39778)

* Fix double-writes in test projects

Add rules for changing InstructionSetDesc.txt (#39777)

Add runtimelab.yml for dotnet/runtimelab experiments CI and build infrastructure (#39749)

* Add runtimelab.yml for dotnet/runtimelab experiments CI and build infrastructure

* rename one of the ymls based on feedback

Allow higher iteration counts for PBES2

Fix some scenarios for DynamicRevocationTests on macOS.

[wasm] Enable System.Linq.Expressions.Tests test suite (#39772)

Use function pointers in more places (#39752)

[wasm] WasmAppBuilder write the json as json (#39660)

* Write the json as json

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

[wasm] Enable System.Xml.XmlSerializer.ReflectionOnly.Tests, System.Xml.XmlSerializer.Tests, and System.Security.Permissions.Tests test suites (#39767)

CI Integration for Cross OS DAC (#39574)

* Transfer assets from Cross OS DAC builds into a pipeline container
* Preserve additional properties in subsets' projects
* Add packaging projects for Cross OS Dac assets
* Pipeline change to package, sign, and publish the cross OS DAC

Move `GetLclOffs` method to `GenTreeLclVarCommon`. (#39623)

* Move `GetLclOffs` to `Common`.

* Start using it.

* Review response.

Use lib prefix for native library names for consistency (#39717)