Fix handling process affinity mask on Windows 11 (#69548)

Refine wording in RequiresDynamicCode annotation in JsonStringEnumConverter. (#73161)

* Refine wording in RequiresDynamicCode annotation in JsonStringEnumConverter.

* Address feedback

* Remove string constant.

* address feedback

Disable SourceGen unit tests in non-release runtime builds (#73181)

Remove buffer copying from Utf8JsonReader Deserialize methods. (#73092)

Delete duplicate unwind info files (#73129)

* Delete duplicate unwind info files

* Add after prefix

* Add RISC-V machine value in remote unwind

Add more tests that have been enabled by recent changes (#73175)

Descriptor parser support (#72975)

Add DescriptorMarker support from linker, this allows the NativeAOT compiler to read embedded xml files in an assembly called ILLink.Descriptors.xml. Then this file will be processed and used to add members to the compilation as roots. Is a very similar functionality to rd.xml with the difference that this format is Cecil based and embedded in the assemblies. Several of the libraries already insert this xml into the assembly to be trimming safe. This PR allows reading those XML files.
Add node to represent loading an assembly embedded file and checking for its dependencies
Verify for embedded files every time a module constructor is called
Support for events and properties in the XML reading
Add nodes to the dependency graph instead of marking logic
Add tests for embedded descriptor file in the NativeAOT smoke tests infra

Revert "Consolidate importer spilling code V2 (#72744)" (#73146)

This reverts commit cea17b2ac2886a154aa813bcfa2fac386ed9f10a.

Fix typos (#73122)

Remove all pinvoke source generator attributes in aggresive trimming mode (#72952)

Use UnrelatedR2RCode flag when --compilebubblegenerics is specified (#71798)

* Use UnrelatedR2RCode flag when --compilebubblegenerics is specified

#73041 exclude Microsoft.CSharp.Tests for tvOS (#73061)

The vast majority of tests are failing because it's trying to JIT and that is not supported on devices.

[eventpipe] Use MonoImage:module_name when filename is NULL (#73018)

* [eventpipe] Use MonoImage:module_name when available, if MonoImage:filename is null

   When assemblies are bundled (for example in WASM), the filename field
   is set to NULL. The module name comes from the metadata Module table and is something like
   "Foo.dll" even if the assembly is loaded from a bundle or from
   Assembly.Load(byte[]), whereas image->name might be "data-0x00abcdef" for byte loads

Remove ActiveIssue tag in ThreadPoolBoundHandle_IntegrationTests.netcoreapp.cs test (#72973)

Reduce the cost of adding many unnamed parameters to an OdbcCommand. (#66869)

Fix #28693

Localized file check-in by OneLocBuild Task: Build definition ID 679: Build ID 1912601 (#73091)

Fixed a WKS specific issue by not clobbering the heap_segment_saved_allocated (#72923)

* Fixed the WKS issue by not clobbering the heap_segment_saved_allocated

* Addressed feedback:

Update the asp.net spmi collection script (#73125)

* update the sets of complus flags to reflect that OSR is now enabled by default
* add some new scenarios like orchard

This increases the number of collected MCs from ~68K to ~120K.

Make IHostEnvironment.ApplicationName non-nullable (#72945)

* Make IHostEnvironment.ApplicationName non-nullable

It is not intended that normal execution has a null ApplicationName, so don't make callers need to check for null. When an ApplicationName is not available, it is set to string.Empty.

Fix #68512

* Make the other string properties in HostingEnvironment default to string.Empty.

Don't hardcode /tmp in named pipe tests, it's not for us on Android (#73067)

Closes: #49873

Remove 1024 character limit on ODBC connection strings (#73149)

Fixes #63630

Remove cctor from X500NameEncoder (#73115)

[Unix] Console.ReadKey rewrite (#72193)

* split TermInfo related logic into separate files, so they can be referenced from test project

* move the key mapping logic to a separate class, do NOT change it

* simple tests for ASCII characters

* test cases for xterm and xterm-256color

* add PuTTY

* add Windows Terminal data

* add more PuTTY test cases

* add mappings for single characters

* add mappings for 3 characters

* fix test cases where pressing Ctrl/Alt/Shift + other key produces two bytes which represent single UTF8 character by design

* handle more complex sequences (with modifiers)

* handle edge cases properly

* add tmux test data

* add rxvt-unicode test data and implementation for rxvt modifiers

* switch to the new implementation and allow the users to differentiate between Enter (\r) and Ctrl+Enter (\n) like we do on Windows

* add way more test cases and fix identified issues

* add more SCO mappings

* Numeric Keypad

* add Tmux 256 color test cases (TERM=screen-256color)

* fix issue discovered by adding more tmux test cases: ^[OM should be mapped to Enter

* address code review feedback

* add .NET 6 compat switch

* Apply suggestions from code review

Co-authored-by: David Cantú <dacantu@microsoft.com>
* address code review feedback

Co-authored-by: David Cantú <dacantu@microsoft.com>

Add more DataFlow tests from linker to NativeAOT (#72777)

This adds several dataflow tests from linker - only those which are passing with the fixes in this change. More will be added later.

Fixes:
- Correctly handle Requires attributes in generic types
- Recognize `ref Type` as interesting for data flow
- Fix generic parameter propagation for compiler generated types
- Treat multi-dimensional arrays as "Unknown" values (to match linker behavior)
- Improve origin of warnings generated from attributes

[mono][interp] Improve tiering performance (#70649)

* [mono][interp] Don't allocate some vars as execution stack

It is not really true and it serves no purpose here.

* [mono][interp] Print code before any optimizations take place

Fix bitrotten mono_interp_print_td_code. Don't print IL_SEQ_POINT opcodes since they are too noisy.

* [mono][interp] Use td->optimized directly in more places

* [mono][interp] Add dummy MINT_LDNULL instruction

We were pushing local that wasn't defined by any instruction, potentially confusing the var offset allocator.

* [mono][interp] Add fast offset allocator, to be used by unoptimized code

This is the old offset allocation scheme that we were using originally, before the var offset allocator was added. Vars have the same offset as they would have in IL code, based on their position on the execution stack at the moment when they are pushed. Whenever we push/pop on the execution stack we keep track of the used stack size. Every var pushed on the execution stack will therefore have this stack_offset remembered. Once the entire IL code is traversed and we have all the global locals allocated, the real offset of the execution stack locals can be determined. It is computed as the originally determined stack_offset added with the offset of the execution stack start (size of the global locals space).

With this offset allocator, calls no longer need to store all the call args sregs and the return_offset is always the same as call_args_offset. This is because all vars are directly placed in the right position and no optimizations can move them around. The offset of the return value will therefore be also the offset where all the args are placed.

The limitation with this way of allocating offsets is that we run into the same problems with opcodes that don't have typical stack usage (use values, pop them, store result). This happens with newobj opcodes. The opcode receives the params, and then it needs to call a ctor with these same params and a newly allocated this object. Since we can't use a var offset allocation pass to compute the offset ideally, the newobj opcodes in the case of unoptimized code must move these params around on the stack, in order to make room for `this`.

* [mono][interp] Add dreg to all calls in unoptimized code

All calls need to have a dreg (a dummy one if it is void call), in order for unoptimized offset allocator to determine the offset of the call. In unoptimized code, the offset of the first argument is always the same as the offset of the return, if any.

* [mono][interp] Fix issue with passing of exvars

Unoptimized code can't use a global local directly (like the exvar), it must first be pushed to a new var on the execution stack. Add a mov instruction when we start executing the basic block for a handler.

Add localloc empty stack verification (#73109)

* Add localloc empty stack verification
* Update error message
* Add more tests
* Fix tests

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

Add some QuicListener tests (#73072)

Link issue to the RDC attribute on DispatchProxy (#73137)

Fix assert in NativeAOT when handling data flow on compiler generated code. (#73085)

The problem occurs when an entire type/assembly is preserved through explicit rooting (command line, rd.xml, ...). If such type contains a local function (for example) which is only called from a branch which is going to be removed by constant-prop/branch removal the internal tracking of compiler generated methods will see this local function as orphaned (not belonging to any user method). This leads to a case where we will try to run data flow on the local function - but that should never happen for compiler generated methods directly -> assert.

The fix is (just like in the linker), to never run data flow on compiler generated methods directly - they should only run data flow as part of their respective user method.

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

Enable more libraries tests (#73104)

Update CLRConfig knob description (#73133)

Add TPN entry for Date/Time processing algorithm (#73128)

Add documentation for hot/cold splitting (#73029)

* Add documentation for hot/cold splitting

* Address feedback

* Improve phrasing

30% optimization of `DateTime.GetDate()`/`.Year/.Month/.Day/.DayOfYear` by 'Euclidean affine functions' (#72712)

* Optimize DateTime.GetDate()/.GetDatePart() by 'Euclidean affine functions'

* Code style

* Refactoring: avoid switch and 'var' keyword; add March1BasedDayOfNewYear constant

* Fix build

* Add [MethodImpl(MethodImplOptions.AggressiveInlining)] to GetDatePart()

* revert relevant comments from original code

* Inline calculations into properties: Year, Month, Day, DayOfYear instead of Common GetDatePart() function

* Fix indentation

* align comment

JIT: have jit tail call stress avoid creating loops in some cases (#73114)

Don't create a loop via tail call stress if the method didn't already have a loop and could have patchpoints.

Fixes #73090.

[Android] Don't exclude .gz files from assets.zip (#73077)

Closes: #36845

[GC] Handle objects from frozen segments in GCHeap::IsEphemeral (#73110)

Remove undefined behavior for address of null reference (#73113)

Implement test checking whether CET is active (#71509)

As part of Control Flow Enforcement Technology (CET) testing we need
to make sure that CET is actually active on the execution machines;
otherwise subtle infra changes could easily regress the testing by
inadvertently deactivating CET without anyone noticing. This change
introduces an initial CET availability test for this purpose.

Thanks

Tomas

Add retry to symlink test (#73042)

Move System.ValueType.s_seed into System.Runtime.CompilerServices.RuntimeHelpers (#73080)

PR https://github.com/dotnet/runtime/pull/69723 added a 's_seed' field to
System.ValueType. This causes the debugger to show all structs as having
static members. This PR moves the field and the function that used it into RuntimeHelpers.

Annotate System.Linq.Queryable with RDC (#72899)

Annotates for RequiresDynamicCode, which represents AOT safety.

Appropriately free temporary values on EC import

EC import from parameters calls into a series of set-routines that neither
self-describe their ownership of the reference counted inputs (`set0` vs `set1`)
nor document them.

Looking at code usage examples, and underlying code, they are all logically
`set1`-style routines, in that they do not take ownership of the inputs, which
means we need to free the BN values on both success and failure.  This assertion
was verified by an intermediate change that reset all numbers to 1
(e.g. `BN_one(qxBn)`) and watched for test failures.  When none of our tests
failed with that change, it moved to the freeing form.

Verified with importing the cross-product of:
{ secp256r1, sect163k1 }
x
{ named curve, explicit parameters }
x
{ q + d, q only, d only }

Disable NJulianRuleTest on Android (#73030)

Co-authored-by: Ilona Tomkowicz <32700855+ilonatommy@users.noreply.github.com>

Add EH filters for generic catch(T) blocks (#72721)

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

Add stateful marshaller support to the CustomMarshallerAttributeAnalyzer (#72888)

Update ConvertToLibraryImport analyzer and fixer to limit false-positives/code breaks (#72965)

Fix incremental XxHash32/XxHash64 (#69185)

Fix crc32 issues found in #62692 (#73005)

Refactor root-level serialization logic and polymorphic value handling. (#72789)

* Refactor root-level serialization logic and polymorphic value handling.

* Address feedback

* Do not consult metadata LRU cache in JsonResumableConverter bridging logic.

* Use secondary LRU cache when resolving root-level polymorphic types.

* Add test coverage for root-level polymorphic values in JsonTypeInfo<T> JsonSerializer methods.

* Change caching strategy for root-level polymorphic values.

* Update src/libraries/System.Text.Json/src/System/Text/Json/Serialization/JsonSerializer.Helpers.cs

* Remove commented out code

recursive generic fix (#73071)

[wasm] `JSImport` generator `AllowUnsafeBlocks` diagnostic (#72996)

- added `JSImportAttribute requires unsafe code` + unit test
- fixed Diagnostic IDs + reservation
- moved System.Runtime.InteropServices.JavaScript.UnitTests to subfolder

Define __cpuid{ex} only when there's no builtin one (#73065)

* Define __cpuid{ex} only when there's no builtin one

Fix clang 15 RC1 build: `error: definition of builtin function '__cpuid'`

* Add clang-15 autodetection

Localized file check-in by OneLocBuild Task: Build definition ID 679: Build ID 1910260 (#73028)

re-enable tests for 66421 (#73052)

[wasm-ep] diagnostic server IPC fixups (#72859)

* [wasm-ep] Trim trailing nul when parsing IPC command strings

The protocol spec says there's a length and a nul terminator, but sending that literal UTF-16 nul to ep_enable_2 means we use an incorrect name when looking for the provider.

* [wasm-ep] Fix formating of event pipe provider strings

   convert the 64-bit "keywords" mask into a hex string properly by padding both halves with zeroes after calling the JS Number.toString() method

* use Array splice instead of String substring

* Address review feedback

Fix UnsupportedOSPlatformAttribute usage in System.Net.NetworkInformation.cs (#72792)

* Test attribute on IsDnsEligible

* Add more attributes

* Icmp statistics

* IPGlobalProperties

* IPInterfaceProperties

* IPInterfaceStatistics

* Replace unsupportedOS attribute by SupportedOS attribute

* IPv4/6InterfaceProperties

[NativeAOT] Fix assert in Thread.SpinWait(0) (#73033)

* do not inline long wait

Make JSON support required properties (#72937)

* Throw exception when 'required' keyword used but ctor does not have 'SetsRequiredMembers'

* Support required keyword (and internal IsRequired)

* remove local variable (forgot to press Save All)

* revert added namespaces in two files

* Apply feedback

* apply second round of feedback (minus HashSet optimization)

* Use BitArray rather than HashSet

* simplify validation code

* Truncate the message if too long, flip meaning of bit array bits

* remove default value, add description for BitArray

* for compiler attributes check for full type name rather than using typeof on net7.0

Unblock tests blocked on 70012 (#73044)

Fixing the per object hard limit support for regions (#72229)

Implementation of opcode OP_GC_SAFE_POINT for ppc64le (#72997)

* Avoid transformation from multiplication to left shift in case of 64 bit value

* Fixed System.Collections.Concurrent.Tests timeout issue

Run more test on NativeAOT Linux (#72832)

Get more libraries tests running with NativeAOT (#72917)

* MetadataLoadContext tests need more work for single file testing in general. Filed a separate issue.
* S.R.Intrinsics is now clean. Running it on NativeAOT found good JIT bugs (#72081, #72158). Promoting to a smoke test.
* S.Drawing.Primitives was testing DataContractSerializer for whatever reason. Marked that as aggresive trimming incompatible. This wouldn't be covered by #72910.
* Made S.R.InteropServices.ComDisabled fully pass by fixing exceptions we throw.
* A couple tests won't ever make sense to run on NativeAOT so removed them from triage area.

JIT: Disqualify methods with modifiable `this` from OSR. (#72877)

Closes #72845.

Remove some debug code (#72985)

* Remove stale debug code

Remove the FreeBuildDebugBreak() and DebBreak() API.
Remove BreakOnRetailAssert config knob.
Remove DBGIGNORE logic.
Remove LaunchJITDebugger() API from utilcode.
Remove x86 StackOverflow break config
Remove INTERNAL_DebugBreakOnAssert config knob
Remove unused debug macros and constants
Remove TerminateOnAssert() and DebugBreakOnAssert() logic.
Remove unused NGen bind config knobs
Remove ValidateObject() from rspriv.

Add support for specifying Kerberos package on Linux/macOS (in addition to NTLM and Negotiate) (#72024)

[wasm] reduce legacy interop usage (#72021)

[wasm][debugger] Fix assert when passing an invalid array to runtime. (#73006)

Check if it's an array before calling the array functions.

Fix #65742

Do not embed result type in VN for scalar hw intrinsics (#72895)

* do not embed result type for scalar hw intrinsics

* Check the simdSize

* oversight fix

Avoid promoting in sweep_in_plan when we already decided not to promote (#72998)

Update the CustomMarshallerAttributeFixer to support adding missing methods for stateless marshaller shapes (#72869)

Added remaining APIs to complete NativeMemory set (#72960)

* Implemented Copy method

* Added Fill implementation

* Review feedback

* Fixed tests naming convention

* Added tests for Fill method

* More test cases for overlapped memory copy

* Removed unused MemSet native function

Use the common path for lowering single-reg SIMD returns (#72944)

* Use the common path for lowering single-reg SIMD returns

This handles type mismatches that morph can create.

* Avoid regressions from RA copies

Lowering will now retype non-reg-candidate LCL_VAR nodes to LCL_FLD
ones in more circumstances, and this pessimizes LSRA a bit.

Apply the same fix as used by the LCL_VAR case. This turns out to
be a good CQ improvement on its own as well.

* Add a test

* Add more tests

* Fix CNS_INT handling

* Fix another similar issue

And remove the duplicative special handling of HWI/SIMD nodes.

Annoating Net.Quic & Net.Security libraries to be AOT safe (#72791)

* Annoating Net.Quic & Net.Security libraries to be AOT safe

* changing the justification msg as per FB

* workaround for issue 72833

HTTP/3 Made alt-used show only on alt-svc redirect (#72669)

* #56876 issue test

* Implemented Alt-Used compliance

* resolved code review

* changed variable name

* removed white spaces

* Update src/libraries/System.Net.Http/tests/FunctionalTests/HttpClientHandlerTest.Http3.cs

Co-authored-by: Natalia Kondratyeva <knatalia@microsoft.com>
* Update src/libraries/System.Net.Http/tests/FunctionalTests/HttpClientHandlerTest.Http3.cs

Co-authored-by: Natalia Kondratyeva <knatalia@microsoft.com>
Co-authored-by: iuliaco <t-icorcodel@microsoft.com>
Co-authored-by: Natalia Kondratyeva <knatalia@microsoft.com>

Fix `ldelem[a]/stelem/cpobj/ldobj/stobj <pointer type>` import (#71679)

[mono][jit] Use MONO_CLASS_IS_SIMD () in more places to fix running with -O=-simd. (#72982)

Fix race when receiving HEADERS and RST_STREAM in rapid succession. (#72932)

* Fix race when receiving HEADERS and RST_STREAM in rapid succession.

* Improve test

Enable tests disabled by issue 36882 (#72931)

JIT: Skip Min/Max optimization when SSE2 is not enabled (#72991)

Fix #71928

[libs][mono] Enable diagnostics_tracing runtime component in Android build and reenable System.Diagnostics.Tracing (#72955)

[wasm][debugger] Added support for getting members of static structures. (#69542)

* Added info flow about being static + including static.

* Added static testcases to Browsable, changed names to more suitable.

* Fixed tests affected by adding static members to GetProperties reply.

* Removed whitespaces I did not intend to correct.

* Test require full name after merge with main.

* Added @radical's sugestions.

* Fixed tests: Private attr is detected even when it's not set.

* Better way of checking attr flags.

Fix SslStress docker build (#72940)

Fix evaluate expression of mutidimensional array. (#72964)

Localized file check-in by OneLocBuild Task: Build definition ID 679: Build ID 1908187 (#72980)

[wasm] Wasm.Build.Tests: Use Chrome on windows, instead of V8 (#72983)

* [wasm] Wasm.Build.Tests: Use Chrome on windows, instead of V8
* Wasm.Build.Tests: Completely disable running tests with V8 on windows

#53624 enable test suites in maccatalyst (#72445)

* #53624 enable test suites in maccatalyst and update ntrampolines

[wasm] remove support for CommonJS (#72865)

Implement ControlledExecution API (#71661)

Add extra Module.ModuleHandle test (#72875)

* Add extra Module.ModuleHandle test

* Update src/libraries/System.Runtime/tests/System/Reflection/ModuleTests.cs

Co-authored-by: Jan Kotas <jkotas@microsoft.com>
* Fix mono implementation

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

Minor cleanup in MethodDesc::GetPrecompiledCode (#72963)

Bump timeout on ARM64 NativeAOT legs (#72974)

[wasm][wbt] Test that `dotnet.js` could be run from any current directory (#69441)

In #72275 we fixed locateFile to deal with relative file path or URL. We could also handle absolute path/URL. This PR is adding tests to prove it works.

Add memory fences to ByRefAssignRef on Unix ARM64 (#72919)

Disable tests that wait for compaction (#72954)

Rename [RegexGenerator] to [GeneratedRegex] (#72434)

Co-authored-by: Jose Perez Rodriguez <joperezr@microsoft.com>

Fixed HWIntrinsic reflection bug with `NextCallReturnAddress` intrinsic (#72505)

* Adding more asserts

* Another assert

* Force add a label

* Extra check

* Remove cookie set

* Added check for compHasNextCallRetAddr for hw intrinsics

* Added additional checks

* Conslidate compHasNextCallRetAddr

* Remove extra assert

* Fixing some logic

* Remove opt check

* Reverting back to original nextcallretaddr logic

* Fix formatting

* Fix formatting

Remove a few hardcoded instances of "net7.0" (#72950)

Follow-up to https://github.com/dotnet/runtime/pull/71129.

The xunit runner package defines a property `XunitConsoleNetCore21AppPath` which (despite the name) points to the xunit.console.dll, we can use that instead.

The properties for the workloads/installer tasks aren't needed either since the packages already define them.

Add SqlTypes apis for SqlClient (#72724)

* SqlMoney changes

* SqlDecimal changes

* SqlBinary changes

* add CLS compliance attributes