Fix wrong property use in resolveContract.targets (#60868)

Helps with https://github.com/dotnet/runtime/issues/59390

Add inline target (callsite/callee) to inline output messages (#60661)

e.g.,
```
Inlines into 06000005 [via ExtendedDefaultPolicy] MainApp:Main():int
  [1 IL=0005 TR=000008 06000004] [callee: below ALWAYS_INLINE size] MainApp:TestNewInline_A_Inline(int):int
    [2 IL=0003 TR=000032 06000003] [call site: profitable inline] MainApp:TestNewInline_B_Inline(int):int
      [3 IL=0003 TR=000045 06000002] [call site: profitable inline] MainApp:TestNewInline_C_Inline(int):int
        [4 IL=0003 TR=000058 06000001] [callee: below ALWAYS_INLINE size] MainApp:TestNewInline_D_Inline(int):int
          [0 IL=0004 TR=000075 06000095] [FAILED: callee: noinline per IL/cached result] System.Console:WriteLine(int)
        [0 IL=0011 TR=000066 06000095] [FAILED: callee: noinline per IL/cached result] System.Console:WriteLine(int)
      [0 IL=0011 TR=000053 06000095] [FAILED: callee: noinline per IL/cached result] System.Console:WriteLine(int)
    [0 IL=0009 TR=000038 06000095] [FAILED: callee: noinline per IL/cached result] System.Console:WriteLine(int)
  [0 IL=0032 TR=000025 0600009A] [FAILED: call site: within catch region] System.Console:WriteLine(System.String)
  [0 IL=0027 TR=000024 06000454] [FAILED: call site: target not direct] System.Exception:get_Message():System.String:this
```

Don't cache commandline in coreclr diagnostics server (#60275)

[DllImportGenerator] Stop flagging methods with well-known unsupported types for conversion (#60841)

* Make analyzer stop flagging methods with well-known unsupported types for conversion
* Make fixer put attribute arguments in preferred order

Update compatibility package and unpin compiler version (#60847)

* Update compatibility package and unpin compiler version

Process: throw ArgumentNullException when ArgumentList contains a null. (#59562)

* Process.Start: validate ArgumentList doesn't contain null.

* Add a custom message.

* PR feedback

Fix DGML debug tests for Regex NonBacktracking (#60853)

Use preferred region from PAL for JIT reloc hints (#60747)

We currently have two schemes that try to ensure jitted code and statics
end up close to each other and close to coreclr. In Windows, we have
USE_UPPER_ADDRESS that lazily will reserve memory for loader heaps that
is close to coreclr. For PAL, we eagerly reserve a large chunk of memory
nearby during start up. However for PAL we were not using this region to
report back to the JIT that addresses in this range can use rip-relative
addressing. Add this support.

I have also cleaned up some of the code around USE_UPPER_ADDRESS: I have
renamed it to USE_LAZY_PREFERRED_RANGE and removed some dead code.

[main] Update dependencies from dotnet/runtime dnceng/internal/dotnet-optimization dotnet/arcade dotnet/roslyn-analyzers dotnet/msquic dotnet/xharness (#60767)

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

Use SDK-selected runtime in IL SDK (#60400)

* Use SDK-selected runtime in IL SDK

* Disable ProduceReferenceAssembly in IL SDK

* Reorder to Import before the override

Up retry for ETXTBSY (#60741)

* Up retry for ETXTBSY

Still seeing occasional failure in Alpine
(https://dev.azure.com/dnceng/public/_build/results?buildId=1434759) so
raising the retry time.

* Linearly scale wait time

[DllImportGenerator] Skip unit tests that hit exceptions due to roslyn-sdk#590 (#60848)

[metadata] Break up mono_metadata_parse_type_internal (#60725)

* [metadata] Break up mono_metadata_parse_type_internal

Separate the type allocation and cleanup logic from the main decoding logic and
from the cached instance lookup.

Also free the allocated memory, if any, in case of decoding errors.

Implement System.Array functions used by MD arrays as intrinsics (#60816)

Specifically, implement:
- GetLength
- GetLowerBound
- GetUpperBound

These are implemented as intrinsics for MD arrays only, when given
a constant dimension argument that is less than the array rank (that is,
in range).

Thus, exceptional cases fall back to the IL code which throws an exception
as appropriate.

Example x64 codegen for 2-D `int[,]` array (instead of call or inlined call):

GetLength(1):
```
mov      r9d, dword ptr [rcx+20]
```

GetLowerBound(1):
```
mov      r9d, dword ptr [rcx+28]
```

GetUpperBound(1):
```
mov      r9d, dword ptr [rcx+28]
mov      r10d, dword ptr [rcx+20]
lea      r10d, [r9+r10-1]
```

Refactor Regex FindFirstChar (#60822)

* Refactor Regexcompiler/Emitter.FindFirstChar

These have become unwieldy, with multiple large code blocks for finding the first character at which to perform a full match, and we'll be adding more soon.  This simply splits each of those out into their own method.  No functional changes.

* Use switch in RegexInterpreter.FindFirstChar

We don't need to re-evaluate which mechanism to use on every FindFirstChar call... we can do so once in the ctor and then just switch on that in the actual FindFirstChar call.

Use SDK's .NETFramework refpack version (#60555)

Use the MicrosoftNETFrameworkReferenceAssembliesLatestPackageVersion property which is exposed by the SDK instead of the one exposed by Arcade to make sure that we are always targeting the latest version.

Skip more failing DllImport tests (#60836)

Follow-up on https://github.com/dotnet/runtime/pull/60773

Initial support for partially jitted methods (#60791)

Introduce a new form of patchpoint for rarely-executed blocks in a method
(typically, blocks with throws). Instead of jitting the code in those blocks,
add code to call a new patchpoint helper (similar to the existing one) which
finds or creates the jitted code for the method beyond this point, and then
transitions control to that code.

Suppressing codegen for exception throws in this way gives similar benefits
to the throw helper transformation, without needing to modify source. This
currently works only in Tier0, but in principle could be extended to handle
optimized code.

fix the summarize bug (#60834)

Don't trim PNSE assemblies and enable trimming for inbox assets on all .NETCoreApp versions (#58345)

* Don't trim PNSE assemblies

Don't trim platform not supported assemblies, since even in "library" trimming mode, the trimmer might remove some non-public members if they are not called within the assembly. In the case of PNSE assemblies this is most likely always the case.

Print "callsites by likely classes" chart in 'dotnet-pgo dump' command (#59306)

Swap using the provided indexes (#55331)

SwapElements is passed two indexes, however, it ignored the second index
and instead used `index + 1`. It just so happened that the caller used
`index + 1` as the second index, but that might not have been the case.

Add HttpClient.PatchAsJsonAsync() apis (#60672)

* feat: add PatchAsJsonAsync methods

* fix: add new api to ref file

* feat: support net46 and netstandard2.0

* refactor: use preprocesser directives to switch between different PatchAsync implementations

* test: validate http method also when validating request

* refactor: implmente extension methods for netstandard that match netcoreapp implmentations

* Update src/libraries/System.Net.Http.Json/src/System/Net/Http/Json/HttpClientJsonExtensions.Patch.cs

Co-authored-by: Eirik Tsarpalis <eirik.tsarpalis@gmail.com>
* refactor: split net standard PatchAsync implementation to another file

* docs: add xml docs for public PatchAsJsonAsync APIs

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

[mono][interp] Allow interp->jit transitions for generic methods. (#60461)

[mono][jit] Reorganize the creation of the initlocals bblock. (#60804)

This is needed because the gsharedvt init code was emitted into it,
and that code might contain branches in llvmonly mode, so
emit_rgctx_fetch_inline () could not be enabled for it. This
caused gsharedvt methods to always emit a slow jit icall in
the gsharedvt init code.

Add ObjectDisposedException.Throw for object instance and type (#58684)

Integrate changes in shared files from feature/NativeAOT (#60811)

Optimize PersianCalendar.AddMonths calculations (#60778)

More stack overflow avoidance in regex NonBacktracking (#60786)

* Add stackoverflow avoidance to SymbolicRegexNode

Remove unused Restrict methods instead of adding avoidance.

Co-authored-by: Stephen Toub <stoub@microsoft.com>
* Add more stack overflow avoidance in regex

This follows a full audit of the NonBacktracking code.

* Apply suggestions from code review

Co-authored-by: Stephen Toub <stoub@microsoft.com>

Add a COMPlus variable to throw runtime exception on JIT asserts (#60631)

Add COMPlus_JitThrowOnAssertionFailure: if nonzero, the runtime will
throw an InvalidProgramException with the assertion message if a JIT
assert is hit during compilation. Previously this would bring down the
process. Use this from the Fuzzlyn pipeline scripts to reduce examples
that hit JIT asserts automatically, since this can now be done much more
efficiently.

JIT: some enhancements to redundant branch opts (#60732)

Handle cases where the dominating compare is the reverse of the compare
we're trying to optimize. For example, if `(x > y)` dominates `(y <= x)`
we may be able to optimize the dominated compare.

Addresses aspects of #48115.

Fix typo (#60797)

Use XHarnessAppBundleToTest to create helix work items when runtime tests run on iOSSimulator (#58870)

2709 runtime tests were enabled to run with iOSSimulator x64.

Co-authored-by: Přemek Vysoký <premek.vysoky@microsoft.com>

Fix typo in comment: NoPropogateInherit -> NoPropagateInherit (#60717)

Maui File location and target runtime updates (#60784)

* Update the file location and android runtimes. The Maui android architecture targets are being updated to include all android targets.

* Updated the targets and props copy location.

* Add in the correct depends on for the maui build.

* The initial file move did not actually delete the folder being moved, delete it.

Use Math.CopySign in CalendricalCalculationsHelper (#60776)

API Implementation: Add Indexer to Vectors (#60517)

* Add indexer for Vector2

* Add indexer for Vector3

* Add indexer for Vector4

* Add indexer for Quaternion

* Add indexer for Matrix3x2

* Add indexer for Matrix4x4

* Add indexer for Vector64<T>, Vector128<T> and Vector256<T>

* Implement better codegen

* Replace switch-tree by if-else-construct and add add Debug.Assert call to indexer of Matrix3x2

* More performant indexer implementation for Vector64<T>

* Improve performance of Indexers for Vector64<T>, Vector128<T> and Vector256<T>

* Improve code readability

* Easier index implementation for Vector64<T>, Vector128<T> and Vector256<T>

* Implement slimmer index implementation for Matrix3x2<T> and Matrix4x4<T>

* Implement GetElement/SetElement on various types

* Usage of Vector2 instead of Vector4

* Implement GetElement/SetElement for Vector2 and Vector3

* Remove setters

* Made Count const in Vector2, Vector3 and Vector4

* Implement GetElement/WithElement Extensions for Quaternion, Vector2, Vector3 and Vector4

* Setter implementation for Matrix3x2 and Matrix4x4

* Fix Quaternion

* Clone vectors in WithElement of QuaternionExtensions, Vector2Extensions, Vector3Extensions, Vector4Extensions

* Make QuaternionExtensions, Vector2Extensions, Vector3Extensions and Vector4Extensions internal

* Replace Extension Methods with normal static methods in Quaternion, Vector2, Vector3 and Vector4

* Make WithElement public and SetElementUnsafe internal in Quaternion, Vector2, Vector3 and Vector4

* Add methods and indexers to references

* Fix Matrix3x2 and Matrix4x4

* Update that the setter of Quaternion, Vector2, Vector3 and Vector4 sets `this` to the result of WithElement

* Simplify SetElementUnsafe in Vector2, Vector3 and Vector4

* Simplify SetElementUnsafe in Quaternion

* Add test cases for Matrix3x2, Matrix4x4, Quaternion, Vector2, Vector3 and Vector4

* Add indexer to reference for Vector64<T>, Vector128<T> and Vector256<T>

* Fix solution file

* Make WithElement in Quaternion, Vector2, Vector3 and Vector4 internal

* Fix project file of System.Private.CoreLib.Shared

* Fix indentation in System.Numerics.Vectors reference

* Code improvements

* Fix solution file

* Remove unsafe from reference

* Add Tests for Vector64<T>, Vector128<T> and Vector256<T>

* Create Test project for System.Runtime.Intrinsics and move vector tests from System.Runtime to System.Runtime.Intrinsics

* Fix test cases

* recreate System.Runtime.sln

* Fix System.Runtime.sln and remove old Vector Tests

* Implement usage of ThrowHelper for Matrix3x2 and Matrix4x4

* Replace Assert.All over array by Assert.Equal in Vector128Tests

* Replace Assert.All over array by Assert.Equal in Vector256Tests

* Replace Assert.All over array by Assert.Equal in Vector64Tests

* Fix indexer tests for Vector64<T>

* Fix indexer tests for Vector128<T>

* Fix indexer tests for Vector256<T>

* Fix indexer of Matrix3x2

* Fix indexer of Matrix4x4

Improve IndexOfAny(byte,byte,byte) (#40894)

Co-authored-by: Jeff Handley <jeff.handley@microsoft.com>

Disable tests that are failing with JIT asserts on ARM64 (#60773)

Remove xunit-performance dependency in tests (#60711)

* Remove dependence on obsolete xunit-performance

We use BenchmarkDotNet in the dotnet/performance repo for
benchmarking now. These copies of the benchmarks are for
correctness testing, and for ease of debugging.

* Make benchmarks look more like the dotnet/performance versions

[wasm][debugger] Fix loading a non wasm page and then returning to a wasm page. (#60688)

* Fix loading a non wasm page and then returning to a wasm page.

* Adding non-wasm-page.html

* fixing other method

* addressing @radical comments

* adding last empty line

Remove unnecessary initialization of variables in generated code (#60748)

Turn off reloc generation for handles (#60721)

Due to #60712 we cannot unconditionally generate lea for handles, since
the runtime does not expect us to generate ask for reloc hints for
arbitrary constants. In addition, for many Intel CPUs it looks like
rip-relative lea has greater latency than mov with an 8-byte immediate.

Instead of reverting the change I'm just turning it off here, since the
change also unified a couple of functions to simplify the handling.

Fix #60626
Fix #60627
Fix #60629

Add general multi-dimensional array test (#60749)

This is similar to MDArray2, but uses the GetLowerBound/GetUpperBound
APIs, and has a test case that uses non-zero lower bounds.
It's in the benchmarks folder as it can be used as a benchmark
to compare against MDArray2 behavior.

Simplify runtime test projects (#60724)

* CLRTestPriority of zero is the default, so don't specify it in test project files

* Remove default CLRTestKind of BuildAndRun from test project files

[main] Update dependencies from 8 repositories (#60618)

[main] Update dependencies from 8 repositories

Disable additional llvmfullaot Linux_arm64 CI lane (#60730)

Missed excluding this configuration in https://github.com/dotnet/runtime/pull/60576

Co-authored-by: Steve Pfister <steve.pfister@microsoft.com>

[wasm] Remove exclusions to bad paths (#60744)

Add limited support for backtracking Regex single char loops to simplified code gen (#60385)

* Add limited support for backtracking Regex single char loops to simplified code gen

In .NET 5, we added simpler compiled code gen for regexes that didn't entail backtracking (or that had only very constrained backtracking, such as a top-level alternation).  In our corpus of ~90K regular expressions, that code generator is employed for ~40% of them.  The primary purpose of adding that code generator initially was performance, as it was able to avoid lots of the expense that original code generator had, especially for simple regexes.  However, with the source generator, it's much more valuable to use this code gen as the generated code is human-readable and really helps to understand how the regex is operating, is much more easily debugged, etc.

This change allows the simplified code gen to be used even if there are backtracking single-character loops in the regex, as long as those loops are in a top-level concatenation (or a simple grouping structure like a capture).  This increases the percentage of expressions in our corpus that will use the simplified code gen to ~65%.

Once we have the simplified loop code gen, it's also a lot easier to add in vectorization of searching for the next location to back off to based on a literal that comes immediately after the loop (e.g. "abc.*def").  This adds support into both RegexOptions.Compiled and the source generator to use LastIndexOf in that case.

The change also entailed adding/updating a few recursive functions.  The plan has been to adopt the same model as in System.Linq.Expressions, Roslyn, and elsewhere, where we fork processing to continue on a secondary thread, rather than trying to enforce some max depth or rewrite as iterative, so I've done that as part of this change as well.

* Address PR feedback

* Clean up partial classes in SourceGenRegexAsync test helper

Fix GCC warnings in test build (#60734)

* fix warnings

* Trigger GCC leg when src/tests is modified

* Add src/native in default paths

Co-authored-by: Adeel Mujahid <3840695+am11@users.noreply.github.com>

Updated tests regarding issue #48119 (#52994)

* added tests for parsing decimal/double/half/single with . at different positions

* adjusted string length for testing . at or after max digit count for single/half

* Fix test cases with decimal separators to work in localized runs

Co-authored-by: Jeff Handley <jeff.handley@microsoft.com>

Disable a couple of pen tests on Windows (#60746)

[llvmonly] Optimize delegate invocation. (#60593)

* [llvmonly] Optimize delegate invocation.

* Unify the initialization of delegates in llvmonly mode in mini_llvmonly_init_delegate ().
* Pass a MonoDelegateTrampInfo structure to init_delegate (), use it to
  cache various data.
* Invoke delegates similarly to the non-llvmonly case, by making an indirect
  call through del->invoke_impl, which is initialized in init_delegate ().
  This also fixes some special cases like bound delegates, since init_delegate ()
  has access to the delegate instance, so it can use the appropriate wrapper.

* Compute a cached MonoDelegateTrampInfo for delegates created from the interpreter as well.

* Fix interp execution of delegate invokes, the invoke wrapper depends on the delegate instance, so it needs to be created in interp_entry () not in transform_method ().

Add direct SDK installer links (#53061)

These are most of the time more useful than just the runtime ones.

I could not find the equivalent `.sha512` links for the checksums for the SDK installers, though :(

Fix intermittent failure in DllImportGenerator unit tests when resolving assemblies (#60722)

Reduce time for Regex tests (#60660)

The introduction of the source generator tests significantly increased the time to execute the regex test suite.  A large chunk of this time is spent for each individual pattern creating an assembly load context, spinning up Roslyn to generate and then compile the code, and then tearing everything down.  We can significantly speed things up if, for the theories that generate a lot of individual tests, we instead batch the compilation to generate and compile the code for many expressions at the same time.

This PR does so and more than halves the execution time for the regex test suite on my machine.  I only added batching for the two largest theories that were contributing the most overhead; we can do so for others should it prove warranted, but it looks like there's diminishing returns here.

I also moved a couple of tests to outerloop.

Temporarily Allow Use of OpenSSL 1.0.2 In Perf Pipelines (#60728)

* Set CRYPTOGRAPHY_ALLOW_OPENSSL_102

Use getexepath in mono to deduplicate related code (#60449)

* Use getexepath in mono to deduplicate related code

* Strip trailing spaces in checked files

* Address CR feedback

Stop adding unnecessary variable when marshalling UTF-16 string by value (#60708)

Disable DllimportGenerator unit tests on ARM32 (#60718)

They're crashing with JIT asserts on main, so disable them for now to get a clean rolling build.

Add copies of binarytrees-6 and fannkuch-redux-9 (#60700)

* Add copies of binarytrees-6 and fannkuch-redux-9

https://github.com/dotnet/performance/pull/1453 added these to dotnet/performance.
Add copies to dotnet/runtime, configured to run and check output like
their neighbors.

Addresses #60578

* Disable binarytrees-6 on wasm

Several tweaks to RegexGenerator (the tool, not the emitted code) (#60667)

- Fixed a bunch of nullable reference types warnings (they're suppressed as a netstandard2.0 project, but I temporarily and locally compiled it for netcoreapp current to get clean)
- Fixed the comparer used to aid in avoiding recomputing regexes when typing
- Added a hook to start exploring code gen for NonBacktracking

Rerun mobile target jobs when it fails for 4 known scenarios (#60475)

* Rerun jobs when it fails for 4 known scenarios

* Add exitcode to retry files

* Add app name to retry files

* Fix callsite

Avoid int[] allocation in ResourceReader when reading a decimal (#60408)

Fix dependency of LLVM AOT jobs on CoreCLR product for CORE_ROOT (#60681)

There's a dependency in this job on CoreCLR's product for the coreroot creation. However, there was no explicit dep and it just raced sometimes.

add test for profiler inlining (#60225)

Update docs for PipeReader.TryRead (#52237)

* Update docs for PipeReader.TryRead

As per https://github.com/dotnet/runtime/issues/30161#issuecomment-831035626, add some documentation to `PipeReader.TryRead` to better document some unclear behavior.

The unclear behavior technically resides in the `StreamPipeReader.TryRead` implementation, which is internal, so I added this as a `<remark>` to the public `PipeReader.TryRead` method instead.

* remove doc reference to internal StreamPipeReader

The StreamPipeReader <see .. /> reference is an internal type, so it should not be mentioned in the public documentation.

* Format remarks as markdown and use !IMPORTANT

Co-authored-by: Jeff Handley <jeff.handley@microsoft.com>

Cleanup workloads for testing logic (#60652)

* Cleanup workloads for testing logic

* PR Feedback

Disable failing DllImportGenerator.Tests.BooleanTests.ValidateBoolIsMarshalledAsExpected test for Mono (#60692)

Skip building interop test native assets for Windows-only tests on non-Windows (#60696)

[wasm] Change dotnet.wasm link optimization (#60633)

* [wasm] Change dotnet.wasm link optimization

Context: #60349

When we switched to Emscripten 2.0.21, we stopped using deprecated
--llvm-opts 2 option. This caused unwanted performance degradation.
Switching to -O2 gets us similar performance as with --llvm-opts.

Blazor wasm test app Time to first UI benchmark times:

    branch/commit        link option    time    dotnet.wasm size  .br size
    --------------------+------------+--------+------------------+---------
    main f40247                  -Oz    484ms    2,430,259         849,844
    main f40247                  -O2    458ms    2,472,851         852,332
    main f40247                  -O3    445ms    2,556,188         856,617

* [wasm] Use '-O3' for link step, in WasmApp.Native.targets too

* Change to -O2

I left -O3 here by mistake after testing

Co-authored-by: Ankit Jain <radical@gmail.com>

[Android] Skip/tweak some System.Runtime tests (#60569)

Fixes #37088

Fixes #50925

Tweak workload build to use _GenerateMsiVersionString target (#60634)

Replaces GenerateVersions that wasn't quite accurate enough for what we need

Fix setting creation date for OSX-like platforms (#49555)

* Fix setting creation date for OSX

Fix for #39132

* Fix setting creation date for OSX - fix for setattrlist

This commit adds a fallback for setattrlist failing.
It can sometimes fail on specific volume types, so this is a workaround to use the old behaviour on unsupported filesystems.

Fix for #39132

* Fix setting creation date for OSX - fix for caching when set time & possible stack overflow

This commit adds _fileStatusInitialized = -1; which resets the cache for the file status after setting a time value to the file using the setattrlist api version so that it will not simply return the last cached time.
It also fixes SetCreationTime_OtherUnix method so that it calls SetLastWriteTime_OtherUnix directly instead of SetLastWriteTime because otherwise you'll get stuck in an infinite loop on OSX when the fallback is theoretically needed.

Fix for #39132

* Fix setting creation date for OSX - changes for PR 1

Fix the behaviour of SetLastWriteTime (OSX) when setting lastwritetime before creationtime.
Use Invalidate() instead of _fileStatusInitialized = -1, and only when appropriate as per PR #49555.
Add SettingUpdatesPropertiesAfterAnother test to test things such as setting lastwritetime to before creationtime.
Rename the added _OtherUnix methods to _StandardUnixImpl, a more logical name given their purpose (#49555)

Fix for #39132 and issues in #49555.

* Fix setting creation date for OSX - changes for PR 2

Added a new test to test the behaviour of the file time functions on symlinks as per standard Windows behaviour, and also what I use in my code today. It makes sure that the times are set on the symlink itself, not the target. It is likely to fail on many unix platforms, so these will be given the [ActiveIssue] attribute when we discover which ones.
Make access time be set using setattrlist on OSX to fix following symlinks issue that would've failed the new test.
Fix and update wording of comment in SetAccessOrWriteTime.
Add T CreateSymlinkToItem(T item) to BaseGetSetTimes<T> (and implementation in inheritors) for the new test.
Made the SettingUpdatesPropertiesAfterAnother test skip on browser since it only, in effect, has one date attribute.

Fix for #39132 and issues in #49555.

* Fix setting creation date for OSX - changes for PR 3

Revert change in last commit: Make access time be set using setattrlist on OSX to fix following symlinks issue that would've failed the new test. It is now using the other API as otherwise some file watcher tests fail.
Set AT_SYMLINK_NOFOLLOW in pal_time.c to not follow symlinks for access time on OSX, or any times on other unix systems.

* Fix setting creation date for OSX - fix test for Windows and browser + update comments

Fix the SettingUpdatesPropertiesOnSymlink test for Windows by setting FILE_FLAG_OPEN_REPARSE_POINT in the CreateFile used for setting times.
Disable the SettingUpdatesPropertiesOnSymlink test for Browser as it can't use symlinks with the current API in the test project.
Update comments.
Add FILE_FLAG_OPEN_REPARSE_POINT.

Fix for #39132 and issues in #49555.

* Fix setting creation date for OSX - make modification time use normal unix APIs

Only implement creation time setting with setattrlist.
Here is why: eg. the old code doesn't account for setting creation time to 2002, then setting modification time to 2001, and then setting access time (since access time didn't have the creation date check).
I've split up SetAccessOrWriteTime into 3 parts - SetAccessOrWriteTime, SetAccessOrWriteTime_StandardUnixImpl, and SetAccessOrWriteTimeImpl (this last one contains the logic of the method without the things like refreshing the times so it's not called twice on OSX). In this process I replaced the _fileStatusInitialized = -1 calls with Invalidate() calls. And to make sure that if the creation time is needed to be set by SetAccessOrWriteTime, then we don't get in an infinite loop in case setattrlist fails, so we call SetAccessOrWriteTime_StandardUnixImpl.

* Fix setting creation date for OSX - hotfix: Remove ATTR_CMN_MODTIME

Adding this commit now to avoid 2x CI.
It was meant to be in the last one, but I forgot to save the file.

Fix for #39132 and issues in #49555.

* Fix setting creation date for OSX - revert windows changes & simplify code

Revert the changes for Windows & disable the SettingUpdatesPropertiesOnSymlink test on Windows.
Remove unnecessary setting to 0 on the attrList variable.
Remove the unnecessary MarshalAs attributes in the AttrList type.

Fix for #39132 and issues in #49555.

* Fix setting creation date for OSX - hotfix: change to CULong for marshalling with setattrlist

Use CULong as per https://github.com/dotnet/runtime/pull/49555#discussion_r599986776

Fix for #39132 and issues in #49555.

* Fix setting creation date for OSX - hotfix: change to CULong for marshalling with setattrlist 2

Use CULong as per #49555 (comment)
Forgot to save the other file's changes

Fix for #39132 and issues in #49555.

* Fix setting creation date for OSX - consolidate unix nanosecond calculation

Consolidate unix nanosecond calculation into UnixTimeSecondsToNanoseconds as per https://github.com/dotnet/runtime/pull/49555#discussion_r600004629.

Fix for #39132 and issues in #49555.

* Use InvalidateCaches instead of Invalidate and EnsureCachesInitialized instead of EnsureStatInitialized

* Fixes for reviews in PR #49555

Add an identifying field to the TimeFunction tuple and a function to get a TimeFunction from the name.
This function is then used so that SettingUpdatesPropertiesAfterAnother can be written as a Theory, and is a lot more readable.
Fix some comments that were incorrect.

* Hotfix for missing parameter

* Fix naming

* Revert changes re SettingUpdatesPropertiesAfterAnother (mainly) and revert using a Theory

Reversion as per https://github.com/dotnet/runtime/pull/49555#discussion_r633895912
This change can be unreverted if that's what's decided.

* Fix errors of #49555 due to missing variables

* Remove the parameters that were meant to be removed

* Finish merge

* Remove duplicate file

* Add custom error handling for setattrlist

Add custom error handling for setattrlist as per https://github.com/dotnet/runtime/pull/49555#discussion_r633788259.

ENOTDIR means it is not a directory, so directory not found seems logical to me, I think the other errors can be dealt with by an IOException, happy to change this.

* Remove symlink specific code from this PR

Removes the symlink specific code as per https://github.com/dotnet/runtime/pull/49555#issuecomment-904413721.
This will be reverted (and modified) as per https://github.com/dotnet/runtime/pull/49555#issuecomment-904413721 into #52639.

* Remove custom ENOTDIR handling, enable tests for other OSX-like platforms + move items in project file

• Removed ENOTDIR handling as per https://github.com/dotnet/runtime/pull/49555#discussion_r705012504
• Enabled tests for other OSX-like platforms since the code was included on them (IsOSXLike is presumably defined at src/libraries/Common/tests/TestUtilities/System/PlatformDetection.Unix.cs). If it fails on any of these platforms, they'll be fully explicitly excluded everywhere for this PR or fixed.
• Move item in project file (System.Private.CoreLib.Shared.projitems) as per https://github.com/dotnet/runtime/pull/49555#discussion_r705032218

* Rename files according to review

Rename files according to https://github.com/dotnet/runtime/pull/49555#discussion_r705073538 (and nearby comments).

* Change names of functions to improve readability and refactor code

This commit changes and refactors the core functions of this PR to have clearer names and to create a more logical flow of the code. It also updates some comments and fixes some oversights. Changes are based on the discussion https://github.com/dotnet/runtime/pull/49555#discussion_r705180391 (read whole conversation).

* Fix the wrongly named variable and add the missing 'unsafe' keyword

Fix the wrongly named variable and add the missing 'unsafe' keyword

* Better comments and minor improvements

Changes according to https://github.com/dotnet/runtime/pull/49555#pullrequestreview-753354257
• Use explicit type `IEnumerable<TimeFunction>` instead of var
• Use PlatformDetection rather than OperatingSystem
• Move the InvalidateCaches code for creation time setting to avoid unnecessary checks
• Update explanatory comments to be more useful

* Add additional tests and improvements

- Use tuple destruction to swap variables
- Add SettingOneTimeMaintainsOtherDefaultTimes and AllDefaultTimesAreSame test as per https://github.com/dotnet/runtime/pull/49555#discussion_r712787328
- Change TFM(/s) as per https://github.com/dotnet/runtime/pull/49555#discussion_r717042126
- Avoid unnecessary call to `UnixTimeToDateTimeOffset` as per https://github.com/dotnet/runtime/pull/49555#discussion_r712606875
- Use InvalidOperationException as per https://github.com/dotnet/runtime/pull/49555#discussion_r717040240

* Fix typos

* Revert to <TargetFrameworks>$(NetCoreAppCurrent)-windows;$(NetCoreAppCurrent)-Unix;$(NetCoreAppCurrent)-Browser</TargetFrameworks>

Revert to using <TargetFrameworks>$(NetCoreAppCurrent)-windows;$(NetCoreAppCurrent)-Unix;$(NetCoreAppCurrent)-Browser</TargetFrameworks> since <TargetFramework>$(NetCoreAppCurrent)</TargetFramework> failed; I wouldn't be surprised if this failed too.

* Remove SettingOneTimeMaintainsOtherDefaultTimes and AllDefaultTimesAreSame tests

Remove SettingOneTimeMaintainsOtherDefaultTimes and AllDefaultTimesAreSame tests since they don't work on windows because it is apparently an invalid time. Additionally, the function SettingOneTimeMaintainsOtherDefaultTimes wasn't working and needed to be fixed.

* Fix: setattrlist can only return 0 or -1

As per https://github.com/dotnet/runtime/pull/49555#discussion_r731451808, use GetLastErrorInfo to get the error when setattrlist indicates that there is one.

Co-authored-by: David Cantú <dacantu@microsoft.com>
* Update code comment for SettingUpdatesPropertiesAfterAnother test

The comment is now clearer and grammatically correct :)

* Update code comment for SettingUpdatesPropertiesAfterAnother test

Describe how utimensat change more dates than would be desired

* Update comment for SettingUpdatesPropertiesAfterAnother for consistency

Update for consistency with the comment in FileStatus.Unix.cs

* Fixes for compilation and update to comment

• Fix trailing spaces and incorrect capitalisation in FileStatus.SetTimes.OSX.cs
• Add more info to the comment in the SettingUpdatesPropertiesAfterAnother test

* Update FileStatus.SetTimes.OSX.cs

Use the Error property

* Move comments and add explicit types to SettingUpdatesPropertiesAfterAnother test

Move comments and add explicit types to SettingUpdatesPropertiesAfterAnother test

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

Fallback to x64 directory if running x64 process on win-arm64 apphost (#59890)

* Append x64 to simulations on win-arm64

* PR feedback

* Update trace and comments

* Use LoadLibraryExW
No need unloading kernel32 dll
Remove unnecessary variable
Remove unnecessary else-if branch

* Remove is_running_in_wow64 from unix pal

* Move is_emulating_x64 to pal
Use sysctlbyname on macOS

* Update ifdef condition

Disable SetOut.SetOutReadToEnd (#60678)

Fix "fix rid processing on macOS" (#60668)

The `else if (major == 12)` is dead code, since the previous if `if (major > 11)` would be true for `major == 12`. Judging by the comment and code, it looks like the intention of this `else if` statement was to match `major == 11`.

issue 56835 make ordered dictionary kv collection implement i list (#60060)

* issue-56835 Initial commit: make OrderedDictionaryKeyValueCollection implements IList.

* issue-56835 Use GetReadOnlyMessage() to give detailed (Keys/Values) readonly exception message.

* Fix comment: reuse existing wording to describe not supported error message.

* Add tests for IList implementation of KeyValueCollection.

* issue-56835: rename isKeysProperty to testKeysProperty.

* issue-56835: Fix comment: reduce code size; use Assert.IsAssignableFrom().

* Fix comment: remove small OrderedDictionaryKeyValueCollection ctor; Ensure initializing Hashtable field before construct keyed OrderedDictionaryKeyValueCollection.

* Fix comment: remove field '_isKeys'.

Make unit tests for DllImportGenerator build/run in CI (#60598)

[DllImportGenerator] Don't generate a full stub body when the body does nothing (#60601)

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

Loop Alignment support for Arm64 (#60135)

* Enable FEATURE_LOOP_ALIGN for Arm64

* basic loop alignment for arm64

* misc changes

* perf score should account for align

* some fixes

* updated some asserts

* jit format

* Fix test cases

* Misc changes

* jit format

* Review comments

Delete AssemblyLoadContext30Extensions test (#60670)

[tests] In test wrapper scripts if -debug is gdb or lldb, add separator before corerun and args (#60416)

* [tests] If -debug is LLDB, add separator before corerun and args

   For runtime test shell scripts, if a debugger is specified with `-debug=/usr/bin/lddb` or the like, add a separator so that `lldb` doesn't try to interpret `-p` as a PID argument.

   Also update mono workflow doc

* Also support gdb as the debugger

Port InformationVersion fix to Mono corelib (#60614)

https://github.com/dotnet/runtime/pull/60572 for Mono's corelib.

Fix condition for dependOnEvaluatePaths for runtime-community pipeline (#60552)

It was missed where we set a default value for that variable.
Also removed the "Evaluate paths" step from runtime-richnav.yml since it doesn't use changed path conditions (otherwise we'd need to add it to variables.yml too).

[HTTP/3] Fixed stress (#60364)

* Fixed stress

* update nanoserver docker image

* Revert "update nanoserver docker image"

This reverts commit 3b3fa4f67265c037646c1642dcd22b5e3231d1c2.

* always pull new image

* Latest msquic

* whitespace

* upper case

* removed image pull madness

Share TypedReference managed code between CoreCLR and Mono  (#60636)

* [mono] Rename common TypedReference fields to the same names as CoreCLR

* Share TypedReference managed code between CoreCLR and Mono

The field layout is different. Although both have `_value` as a pointer to the
value (and this PR updates mono to use a `ByReference<byte>` for that pointer,
same as CoreCLR), and a `_type` which is a type handle (`MonoClass` in mono on
the native side), in addition, mono also has a RuntimeTypeHandle (`MonoType` in
mono on the native side) that is used to help the JIT.

But the managed code is identical and only uses `_value` and `_type`.

Fix Regex outerloop tests for RegexOptions.NonBacktracking (#60647)

* Fix Regex outerloop tests for RegexOptions.NonBacktracking

As part of this, fixes a thread-safety issue in SymbolicRegexRunner.  While the SymbolicRegexMatcher is thread-safe, the runner instance itself is not.

It also enables a bunch of outerloop tests for NonBacktracking and source generators that we'd previously missed enabling.

Add documentation on how VN numbers memory (#60476)

Ensure outgoing arg space for stack cookie fail fast (#60615)

The stack cookie check might call a fail fast helper, so make sure to
bump the outgoing arg size as otherwise the stack traces logged can be
corrupted.

Try net7.0 (#60671)

Fix a few more ASAN-exposed issues (#60262)

Re-enable tests that are no longer failing (#60609)

[mono][llvmonly] Optimize virtual calls/rgctx fetches in gsharedvt methods. (#60491)

* [mono][llvmonly] Optimize virtual calls in gsharedvt methods.

Previously, these were all implemented using a slowpath to a JIT
icall. Instead, use the normal calling sequence and add a gsharedvt
out wrapper at the end if needed.

* Optimize rgctx fetches made from gsharedvt methods.

* Implement a fastpath for mrgctx fetches.
* Load some entries from the gsharedvt info struct instead of doing an rgctx fetch.

Improve FileProviders.Physical tests to reuse existing helper types (#58155)

* Replace DisposableFileSystem with FileCleanupTestBase and TestDirectory

* Fix get directory name

Co-authored-by: Jeff Handley <jeff.handley@microsoft.com>

Move 2 Drawing APIs that are not implemented in netfx to netcoreapp (#60599)

* Move 2 Drawing APIs that are not implemented in netfx to netcoreapp3.1 or later (#60371)

* Don't use ifdefs on ref file

Update IntelliSense package (#60487)

* Update IntelliSense package

* Fixed package version

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

Unified request/response logging for H/2 and H/3 (#60418)

DotNetHost packages are not created during source-build (#60575)

The subset `host.pkg` is not producing the DotNetHost* NuGet packages. This is because the "Pack" target is getting invoked on the pkgprojs, but pkgprojs expect the "Build" target to be called. Since the "Pack" target is overriden in the Directory.Build.targets to be empty, no one is calling the "Build" target on the pkgprojs.

In an official build, a later subset `packs.tests` comes through and builds `Microsoft.DotNet.CoreSetup.Packaging.Tests.csproj`, which explicitly calls "Build" on the pkgprojs. So official builds still get these packages produced.

In source-build, we are no longer building the `packs.tests` subset, so the packages are not produced.

Update mac installers location string (#60499)