Moving more Tfm specific properties to the targetFramework.props file (#34349)

* follow up

* other occurences

* fixing compiler services failure

Hardware instruction set support for crossgen2 (#33274)

- Add support for the --instruction-set parameter as described in #226 .
NOTE: As the abi for Vector parameters is not yet stable, support for the --instruction-set parameter is only enabled if --inputbubble is also enabled. Parallel work to stabilize the abi is in progress, but is not complete.
ALSO NOTE: The names of the instruction sets are shared with mono, and don't follow the names in issue #226
- Add concept of baseline instruction set support to R2R file format
- Can be applied at a per method level or at the entire R2R file level
  - R2RDump support for dumping the extra data
- Refactor how support for hardware intrinsics beyond SSE2 support are handled in crossgen2
- Add feature to the JIT to detect which hardware features are actually used
  - Tell the JIT unconditionally that SSE42+Lzcnt+Popcnt+Pclmulqdq are supported
  - But if support beyond the --instruction-set specified baseline is used, notate the method with a per-method instruction set support fixup.
  - This enables usage of many intrinsics in corelib with greater efficiency than today
  - This enables usage of SSE42 and below intrinsics safely in non-CoreLib code. Use of higher level intrinsics in non CoreLib code will generate code which does not use the higher level intrinsic, and note that the method's code should not be used in the presence of hardware which does support greater CPU capabilities.
  - In the future a logical enhancement of this work would be to generate multiple bodies of code to handle these more complex cases.
  - In combination with the --instruction-set argument, if Avx2 is enabled, then the logic gracefully adds a dependency on Avx2 capability and Vector<T> becomes useable by crossgen'd code.

Add profiler tests to the runtime repo (#34411)

Ports GCBasic, ReJIT, and EventPipe tests as well as adds a test that verifies profilers can call MetaDataGetDispenser.

Enable nullable reference types for mono's Corelib (#34400)

* Enable nullable reference types for mono's Corelib

Corelib had nullable reference types enabled, but all warnings were suppressed.  This fixes the thousands of warnings that were being hidden and fully enables nullable reference types for mono's corelib, as it is for coreclr's corelib.

I copied over the public API annotations from coreclr's corelib and tried to annotate the mono source as accurately as possible.  However, I'm 100% sure there are issues, both places I misexpressed the contract of an internal API and places where I suppresed warnings with `!` where it likely could actually null ref.  Along the way, when there was an obvious null ref where coreclr was doing a null check and providing different behavior, I fixed it or at least added a comment, but I expect there are more issues lurking.

* Fix iOS build for GetSpecialFolder

* Address PR feedback

Socket.Windows: support ConnectAsync(SocketAsyncEventArgs) for UDP, and Unix sockets (#33674)

Socket.Windows: support ConnectAsync(SocketAsyncEventArgs) for non-stream protocols

[CBOR] Add a CborReader.SkipValue() method (#34477)

* Add a CborReader.SkipValue() method

* revert reader state format error changes

* refactor skip from recursive to imperative

* use non-allocating skips for string data items

* add a test for extremely nested values

* remove blank line

* consolidate SkipString() methods

Disable iOS_arm builds on official builds too (#34504)

It started failing with the same issue as https://github.com/dotnet/runtime/issues/34465

Crossgen2 fixes to enable composite build with shared framework (#34431)

These changes involve targeted fixes to issues seen due to the new
build mode and generic signature encoding fixes identified in
my offline investigation with JanV. While I locally see the tests
passing with just a handful of failures, I still seem unable to
make the tests run in the lab; I keep investigating that but
I believe it's useful to merge this delta in to let us parallelize
follow-up efforts.

Thanks

Tomas

Parallel Builds PRv2 (#34460)

* simplify libraries build dependencies.

* FIx depends for tests

* Fix artifact name for webAssembly

Add armel arch in help message (#34491)

Delete stale CPU cache size detection (#34488)

Fixes #34478

Port CoreCLR's TypeNameBuilder to C#, and use it in Mono too (#33701)

* Port CoreCLR's TypeNameBuilder to C#, and use it in Mono too

Mono's System.Reflection.Emit creates type names that fail to be normalized or shaped in all ways that CoreCLR does.

Port CoreCLR's mixed-mode thread-unsafe implementation to thread-safe C#, and start using it in Mono for names in TypeBuilder.

Fixes issues with e.g. null-delimited type names being passed to different Reflection.Emit builders. Contributes to https://github.com/dotnet/runtime/issues/2389.

Fix WriteLargeJsonToStreamWithoutFlushing test (#34489)

Don't build `System.Utf8String.Experimental.Tests.csproj` tests for mono (#34445)

Fix subsetCategory is just one `-` not two `--` (#34498)

Otherwise fails with:
```
MSBUILD : error MSB1001: Unknown switch.
Switch: --subsetCategory
```
on Windows

[sgen] Implement api for allocating pinned arrays (#34398)

* [sgen] Add support for allocating pinned array

These objects will end up allocated either in LOS (which is by design not moved around) or into special major blocks where compaction doesn't take place.

* [sgen] Be consistent with coreclr on GC api

* Enable GC tests for pinned array allocation

* Skip large array allocation test on mono

[mono][sre] Add an expected attribute on underlying fields of enum classes (#34212)

Follow CoreCLR's example and add the RTSpecialName type attribute to any class field with the 'value__' name when , used as the value store for enumeration class instances.

Second part of the "RTSpecialName fix" that started with https://github.com/dotnet/runtime/pull/33389 - before, our behavior was to add the RTSpecialName immediately, as soon as the EnumBuilder was constructed.
Contributes to https://github.com/dotnet/runtime/issues/2389

Fix two issues detected by Valgrind (#34462)

* Fix two issues detected by Valgrind

When I have used Valgrind to investigate a memory corruption issue recently,
I've noticed that it has also reported two cases when a conditional jump
was using an uninitialized variable as one of the inputs to the condition.

This change fixes these.

Disable iOS arm PR builds (#34467)

* Disable iOS arm PR builds

Infrastructure issue: https://github.com/dotnet/runtime/issues/34465

* Add tracking issue and disable installer iOS arm build too

Remove -runtime version suffix hack for M.E. (#34464)

Arm64: Add xtn and xtn2 intrinsics codegen, api and tests. (#33108)

Remove @jeremykuhne from area owners (#34483)

cc @ericstj

Enable warn as error for libraries (#34457)

This enables warn as error for the libraries jobs now that our builds
are warning free.

Sync changes from ASP.NET Core dynamic HPack (#34247)

Experimental pgo tool (#34433)

* Experimental pgo tool
- Experiment and rough design going forward for providing profile data to the compiler

* Update build so dotnet-pgo is built

Add build configuration to generate runtime packs for iOS (#34050)

This adds support for generating Microsoft.NETCore.App.Runtime packs for ios-x64, ios-arm, and ios-arm64.

[meta] Add mono_type_get_name_full to public API (#34436)

This is needed by Xamarin.Android to be able to round-trip with names fetched via reflection, as the only public API for this (mono_type_get_name) uses the IL format instead.

fix iOS sample build (#34455)

[loader] Add append option to loader hook installation functions (#34284)

[wasm][debugger] Add support for visualizing valuetypes (#33604)

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

Merge pull request #34166 from mangod9/master

Add support for building S.P.CoreLib in a separate job

Update WASM image (#34421)

Move to an Ubuntu image which has the `en_US.UTF-8` locale available.
This should remove the spurious warning that we're seeing during
PR / CI

closes #34280

Fix large version bubble field offset computation (#34401)

This change fixes two bugs in field offset computation where the results
that crossgen2 was getting was different from what runtime computes. In
both cases, the problem was caused by alignment of a derived class being
done differently.
The first issue was happening for the case when the base and derived
classes are in different assemblies. Runtime detect if two assemblies
are in the same version bubble using the native manifest metadata table
containing a list of assemblies that was supposed to contain all
assemblies that the assembly being compiled was found to reference.
However, it contained only assemblies that were not in the original
assembly reference list, e.g. ones pulled in by inlining. So runtime
wasn't getting the same view on what's in the bubble.
The second issue happened for the case when both the base and derived
class were from the same assembly, but one of the ancestor classes had a
field of a value class type that was from another assembly and could be
transitively decomposed to fields of types from the same assembly or types
like primitive types, object, pointer or enums. The alignment of a derived
class members is determined based on that and runtime decision is to
align if there is any type from another assembly in the type hierarchy
of a class or in fields of any ancestors.
For example, the decision would be different for the following scenario:
Assembly A:
struct AA
{
    int a;
}
Assembly B:
class B1
{
    AA aa;
}
class B2 : B1
{
    int x;
}
Here crossgen2 would not align the first member but runtime would. So the
layout of B2 produced by crossgen2 would be:
```
Offset  Field
0       MethodTable
8       a
12      x
```
Layout produced by the runtime would be
```
Offset  Field
0       MethodTable
8       a
16      x
```

The fix for the first issue is to put all referenced assemblies into the
native manifest metadata.
The fix for the second issue is to stop decomposing members of value
classes once we hit a value class that's from another module.

R2RDump fixes for parsing composite images (#34429)

1. Fixed several inconsistencies w.r.t. signature context in
recursive descent into generic signatures based on offline
investigation with JanV.

2. Don't crash the dump upon invalid UTF8 characters.

Thanks

Tomas

Fix ordering of the instructions (#34447)

Command for opening Visual Studio works from root folder,
but not inside library folder as was implied by ordering in the instructions.

Enhance PAL initialization order (#34308)

* Enhance PAL initialization order

The CGroups initialization has _ASSERTE that is not asserting because
debugging support was being initialized after the CGroups initialization.

This change moves the debugging support initialization to the earliest
possible point. It also modifies it so that it doesn't depend on the
PAL env functions and uses getenv instead. And finally, initialization
of the critical section for debug printf is moved to the very end
of DBG_init_channels just as a little cleanup.

Fix crossgen2 delegates to static generic methods (#34370)

In crossgen2 we convert all methods to their canon versions in
CreateMethodEntrypointNodeHelper, which causes a problem for
delegates set to generic static methods. When these methods are called
via a delegate, at runtime it gets a hidden generic argument that
is a MethodDesc of a canon version of the static method instead of the
specific instantiation.

I've investigated why we convert all methods to their canon versions in
the CreateMethodEntrypointNodeHelper and I've found a PR by @trylek
in CoreRT repo that has introduced it. I've discussed that with him, but
he wasn't able to figure out why the change was made and it seems like
it was mostly accidental or fixing some problem that no longer exists.

Removing this conversion fixes the problem and I have verified that it
doesn't introduce new ones by running both pri 1 CoreCLR tests and the
libraries tests. It has also made the diff between code generated by
crossgen1 and crossgen2 e.g. for System.Private.CoreLib.dll smaller.

Initial Mono Android CI (#34125)

removed some unused corelib configs.

Move StrongNameKeyId=Open default value down after arcade is imported (#34428)

Add test for native library load through AssemblyLoadContext.LoadUnmanagedDll (#34425)

* Reorganize the NativeLibrary tests to be in one root folder and build/use the same native library for testing

* Add test for AssemblyLoadContext.LoadUnmanagedDll

Delete Ubuntu 14.04 conditions (#34361)

Ubuntu 14.04 is no longer supported.

Eliminate dead configuration code (#34285)

* Eliminate dead configuration code

* Remove more dead code

* Eliminating EEConfig::GetConfiguration_DontUse_

* Eliminate EEConfig::GetConfigValueCallback

* Eliminate ConfigSearch

Fix run-test-job pretest build

Fix write behind allocated memory in thread name setting (#34424)

The code in CorUnix::InternalSetThreadDescription is writing behind
the end of the allocated memory in case the name is shorter than 16
characters. That is causing memory heap corruption.

typo fix :(

PR Comments

Build managed part of crossgen2 as platform independent (AnyCPU) (#33041)

Report correct max depth in exception message (#34410)

Co-authored-by: Thomas Levesque <thomaslevesque@users.noreply.github.com>

update perf jitdump for ppc64/ppc64le and s390x (#34373)

Add missing constants to handle ppc64/ppc64le and s390x platforms.

Related: https://github.com/mono/mono/commit/68a8b9290958ee5e9b21d8ff6b73a707e2504d39
Fixes: https://github.com/mono/mono/issues/19402
Co-authored-by: sharkcz <sharkcz@users.noreply.github.com>

Perf followup for Pinned Object Heap (#34215)

* free list bucket skipping

* couple more tweaks

* More size buckets in POH free lists

* static_data_table

* PR feedback

* off-by-one error in BASE_LOH_ALIST_BITS

Ensure generic parameter constraint type is included when building image (#34346)

sets.

This fixes: https://issuetracker.unity3d.com/issues/editor-crash-when-entering-and-exiting-playmode-for-a-few-times-with-microsoft-maps-sdk

@joncham and I were working on this we hit the two asserts that I've removed. We were wondering if they were still needed?

<!--
Thank you for your Pull Request!

If you are new to contributing to Mono, please try to do your best at conforming to our coding guidelines http://www.mono-project.com/community/contributing/coding-guidelines/ but don't worry if you get something wrong. One of the project members will help you to get things landed.

Does your pull request fix any of the existing issues? Please use the following format: Fixes #issue-number
-->

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

[wasm] Throw an exception when trying to start a thread instead of asserting. (#34347)

<!--
Thank you for your Pull Request!

If you are new to contributing to Mono, please try to do your best at conforming to our coding guidelines http://www.mono-project.com/community/contributing/coding-guidelines/ but don't worry if you get something wrong. One of the project members will help you to get things landed.

Does your pull request fix any of the existing issues? Please use the following format: Fixes #issue-number
-->

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

[mono] Fix finalizer thread init in iOS sample (#34405)

Starting with https://github.com/mono/mono/pull/16907 , the runtime ends in GC-Safe state (mode) after mono_jit_init_version() is called. mono_gc_init_finalizer_thread() expects the GC to not already be in safe mode.

Update ownership of area-System.ServiceModel (#34408)

Stephen Bonikowsky is no longer on the WCF team, updating ownership.

Revert "Revert "Switch reverse PInvoke to the NativeCallable plan (#34251)" (#34306)" (#34315)

This reverts commit 94c4b5248bbf7f3219ca4af7960832d9acf218da.

Disable test on windows and enable on other platforms (#34331)

* Disable test on windows and enable on other platforms

* Remove mac from test

Call into ETW methods in ConcurrentBag only when logging is enabled (#34392)

Merge remote-tracking branch 'upstream/master'

[mono] Fix setMArch, getArchName() doesn't work as expected (#34368)

Turns out `llvm::Triple(llvm::sys::getDefaultTargetTriple()).getArchName()` doesn't work as expected on all targets so the only option is to set arch via defines.

NOTE: it accepts "x86-64", not "x86_64

Remove duplicated .gitignore patterns (#34281)

Fix build break in coreclr where -os is not supported in Windows (#34358)

[mono] fix SIMD instructions' availability (#34319)

[mono] Fix LLVM JIT build on arm64 host (#34332)

[mono] Set MArch in llvm-jit (#34350)

Disable Bitmap round-tripping tests for old libgdiplus versions (#34354)

* Fix filename typo

* Disable Bitmap round-tripping tests for old libgdiplus versions

Old libgdiplus versions have uninitialized stack variable bug that makes Bitmap round-tripping unreliable. The bug causes Bitmap.Flags, Bitmap.HorizontalResolution and Bitmap.VerticalResolution properties to be set to bogus values if the stack happens to contain certain bit patterns. This bug was fixed for libgdiplus 6 by https://github.com/mono/libgdiplus/commit/81e45a1d5a3ac3cf035bcc3fabb2859818b6cc04#diff-c96a8261ecb168c12b44248208da21c0R118.

* Fix and simplify gdiplus library loading

Fix System.Data.SqlClient baseline (#34352)

* Fix System.Data.SqlClient version

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

* Update Microsoft.Windows.Compatibility.pkgproj

Disable test based on 34178 (#34179)

* Disable test based on 34178

* Try with */*

* Fix diff

remove upload unsigned artifacts step from official build

Fixing intellisense in vscode for src and ref projects (#34025)

* fixing intel for ref and src

* fixing ref and src intell

* renaming props file

* remove extra property

* fixing the intellisense in tests

* add extra line

* addressing feedback

* revert shim path

* fix allconfig restore

* adding comment

[FileSystem] ReAdd check for same device (#34343)

Co-authored-by: Mitchell Hwang <mihw@microsoft.com>

libraries test should depend on full product

This fix is similar to artifact download logic done in build-base.yml

Fix ProfileTailCallNaked so it doesn't write xmm1 past the end of its stack (#33908)

* Fix ProfileTailCallNaked so it doesn't write xmm1 past the end of its stack, and do some documentation/refactoring.

* code review feedback

add support for FreeBSD cross-compilation (#34000)

* add support for FreeBSD cross-compilation

* feedback from review

* update detection

* add openssl to docs

* feedback from reviews

* feedback from reviews

* final cleanup

* feedback from reviews

* simplify detection of route.h

* correct __CrossBuild detection

* use TargetOS for installer

update brancing to preview4 (#34338)

Initial proposal for P/Invokes via Source Generators  (#33742)

* Initial proposal for P/Invokes via Source Generators.

NativeCallable bugfixes (#34335)

- Fix race condition in JIT_ReversePInvokeEnter
- Disable R2R for x86 on all platforms

[debugger] Adding static to ss_req_acquire and ss_req_count (#34327)

Adding static to ss_req_acquire and ss_req_count
Fixes mono/mono#19389

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

Add corelib to other pipelines.

Apply formatting patch (#34333)

Cache parameterized ctor delegates in class info rather than converter (#34248)

* Cache parameterized ctor delegates in class info rather than converter

* Address review feedback - move delegate assignment to start of deserialization

* Address review feedback - nullability

Guard against deserializing types with open generics (#34163)

* Guard against deserializing types with open generics

* Include serialization tests

* Address review feedback

* Address review feedback

Check for COM registration functions on base types of registered type (#34307)

Remove inactive JsonElement test that depends on JSON path being set on DOM exceptions (#34167)

Adding corelib job to coreclr=and=Libraries.

[mono] Implement Environment.GetFolderPath on iOS (#34022)

* Implement Environment.GetFolderPath on iOS

* Address feedback

* Move GetFolderPathCore to Environment.Unix.GetFolderPathCore.cs

* Fix build issue

* Address feedback

* cache all special directories

* Fix build issue

* remove a whitespace

* Fix UserProfile issue

* undo changes in GetEnvironmentVariableCore

* Update Environment.Unix.Mono.cs

* Extract to InternalGetEnvironmentVariable

* Fix build issue

* Return emtpy string if underlying native function returns null

* Add nullability

Update Microsoft.CodeAnalysis.FxCopAnalyzers to 3.0.0-beta3.final (#34312)

Expression.Field should validate field name for null (#34301)

Small change to correctly check for null in one of the overloads of Expression.Field.

[mono] Use $(HostArch) for LLVM packages (#34322)

* Use HostArch for LLVM packages

* define HostArch for mono

Initial perf jitdump implementation (#33893)

Fixes mono/mono#17024

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

[runtime] Fix a problem with PR mono/mono#19361. (#34299)

<!--
Thank you for your Pull Request!

If you are new to contributing to Mono, please try to do your best at conforming to our coding guidelines http://www.mono-project.com/community/contributing/coding-guidelines/ but don't worry if you get something wrong. One of the project members will help you to get things landed.

Does your pull request fix any of the existing issues? Please use the following format: Fixes #issue-number
-->

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

fix -allConfigurations build

Streamline C heap allocations (#34289)

- Remove unnecessary layers in heap allocation APIs
- Delete HeapCreate/Alloc/Free/Destroy from the PAL

Remove TargetOS renaming under installer (#34313)

Merge pull request #34208 from maryamariyan/inspection

Extensions commit mop-up

Change PublicKeyToken for M.E. src and test projects
- Updates InternalsVisibleTo files as well

- Setup Logging.EventSource.Tests
- Fix three failing EventSourceLoggerTest tests
- Delete file: IntegrationTesting csproj not needed.
- Rename folder test -> tests (Logging/tests/DI.Common/tests)
- Cleanup other csproj

    fix compile issue

Moving runtime restore from \restore\dirs.proj

Moved runtime restore + pretest.proj to teh tests.proj build.

Condition before/after import in IL.Sdk for pre5.0 (#34278)

With this, the IL SDK will continue to work when it comes to either single or multitargeting without restoring the projects in multi-targeting mode. This helps to not introduce a breaking change for 3.x SDKs.

Revert "Switch reverse PInvoke to the NativeCallable plan (#34251)" (#34306)

This reverts commit 4e30ff033fbd915c5a39df8901d43c601deeeeb4.

Restore TargetArchitecture default (#34295)

Fixes #34292