Streamlined MinOpts GC ref tracking. (#9869)

Merge pull request #9831 from WinCPP/LazyInitializerNewOverloads

CoreFx #16274 Implementation - New overloads of LazyInitializer.EnsureInitialized for reference types

[x86/linux] Add IsIPinVirtualStub() on x86/linux (#9844)

* [x86/linux] Add IsIPinVirtualStub() on x86/linux #9691

To pass Loader.classloader.methodoverriding.regressions.549411.exploit
test failure on x86/linux. This patch is from #5542.

[Arm64/Unix] Revert fix stack unwinding in CallDescrWorkerInternal (#9866)

[x86/Linux] Use ESTABLISHER_FRAME_POINTER_IS_CALLER_SP (#9814)

* [x86/Linux] Introduce ESTABLISHER_FRAME_ADDRESS_IS_CALLER_SP

[x86/Linux] Correct GetStackParameterSize on Funclet (#9849)

Merge pull request #9870 from dotnet-bot/master-UpdateDependencies

Update CoreClr to beta-25101-01 (master)

Corefx #16274 - LazyInitializer.EnsureInitialized overload for reference
types

LazyInitializer.EnsureInitialized overload for reference types that does
not take boolean for tracking initialization status of the reference
type object. Null check on the object parameter is sufficient to whether
or not it has been initialized.

Globalization merge (#9835)

* Merging the Globalization code in coreclr

This change is merging the globalization code used for Linux and Windows. here Is come notes about this merge:
- Mainly picked most of the code from the corefx folder as it is well formatted and cleaned up before and then tweaked the implementation to work on Windows
- Tries to reduce the dependencies from Globalization code on the VM. this will give flexibility to work on the code without worrying the VM
- Provided the missing implementation as I removed many internal calls to the VM. This also help in having .Net Native have complete implementation for .net standard 2.0
- We may need to have some small tweaks when merging this globalization code to corert
-   We'll need to do some clean up for the code which not used anymore in the VM

* Fix ifdef's

* Fix field name in linux build

* Fix field name in linux compilation

* Misc cleanup & return randomized hashing

* Fix missing method for Linux

* one more minor fix

* Fix Linux build

Move files that are identical with CoreRT under mscorlib\shared (#9861)

Update CoreClr to beta-25101-01

Merge pull request #9865 from alexperovich/moveToShared

Add more shared sources

Merge pull request #9609 from pgavlin/HandleContainedNodesEarlier

Handle contained nodes earlier in buildRefPositionsForNode.

Merge pull request #9862 from mikedn/ge-vnf-gt

Fix broken GT_GE - VNF_GT_UN mapping

Merge pull request #9825 from mskvortsov/ryujit-arm32-pinvoke

[RyuJIT/ARM32] P/Invoke support

Handle contained nodes earlier in buildRefPositionsForNode.

These nodes never produce ref positions. Handling them earlier in this function
avoids a rather large amount of unnecessary work (e.g. this decreases overall
compile time by about 0.5% for `crossgen System.Private.CoreLib`).

Merge pull request #9794 from pgavlin/PutArgRegInit

Refactor register arg handling in TreeNodeInfoInit.

Add more shared sources

Merge pull request #9729 from rahku/hijackfix

add FP return values as part of HijackFrame

Merge pull request #9857 from jashook/arm64_inner_structs

Add the correct offset for finding nested structs.

RyuJIT/ARM32: Enable P/Invoke lowering.

Fix mismatch in the cmake script endif condition (#9859)

This change fixes a mismatch in the cmake script endif condition introduced
by #9809.

Merge pull request #9781 from dotnet-bot/master-UpdateDependencies

Update CoreClr, CoreFx to beta-25028-02, stable, respectively (master)

Fix broken GT_GE - VNF_GT_UN mapping

Merge pull request #9852 from mskvortsov/ryujit-arm32-label

[RyuJIT/ARM32] enable CG for GT_LABEL nodes

Remove some unused benchmarks (#9843)

Add the correct offset for finding nested structs.

As is the base offset of a nested struct that was a lclField was
being dropped.

Merge pull request #9796 from pgavlin/FixOurGetThreadCycles

Fix _our_GetThreadCycles for clang/xarch.

Fix cross component builderror - CMakeLists.txt for PAL (#9819)

define & use PAL_ARCH_SOURCES_DIR in PAL CMakeLists.txt to build assembly based on platform architecture.
root/CMakeLists.txt defines ARCH_SOURCES_DIR based on target architecture.

RyuJIT/ARM32: enable CG for GT_LABEL nodes

Disable DBGIPC_TRANSPORT_DI and DBGIPC_TRANSPORT_VM on Android (#9809)

* Disable DBGIPC_TRANSPORT_DI and DBGIPC_TRANSPORT_VM on Android

Use BIT64 instead of _WIN64 for LONG helpers (#9845)

Update CoreClr, CoreFx to beta-25028-02, stable, respectively

[x86/linux] Implement BackPatchWorkerAsmStub (#9690)

* [x86/linux] Implement BackPatchWorkerAsmStub

This implementation is ported from BackPatchWorkerAsmStub in
vm/i386/virtualcallstubcpu.hpp

Merge pull request #9774 from sandreenko/coreRT-fat-pointers-fix

coreRT fat pointer hidden argument fix for calls with retBufArg

Replace INSIDE_CLR with CORECLR (#9841)

Move Used interop sources into shared folder (#9840)

CC fix

Some tests for devirtualization (#9834)

Use flags in exceptionhandling.cpp (#9823)

* Add USE_FUNCLET_CALL_HELPER

* Use STACK_RANGE_BOUNDS_ARE_CALLER_SP

[x86/Linux] Fix GetCurrentSP (#9838)

coreRT fat pointer hidden argument fix for calls with retBufArg

Merge pull request #9837 from adityamandaleeka/remove_useless_message

Remove useless message from runtest output.

Remove useless message from runtest output.

Don't generate a runtime lookup for static base if not needed (#9833)

`fgInitThisClass` makes sure that the type initializer for the type that
owns the method being compiled has run. On CoreCLR, the fallthrough code
somehow ends up not doing a runtime lookup if the owning type is not
canonical. We need to avoid it on CoreRT as well.

Merge pull request #9795 from pgavlin/RefactorIsContained

Refactor isContained().

README.md: More idiomatic Markdown usage (#9799)

- In essence, use backticks where applicable.
- Also, changed some incorrect case (`Build.cmd` vs `build.cmd` which is the canonical name; if you have your git repo on a case-sensitive filesystem like I do, this makes a real difference - you can't run it with the incorrect case).
- Fixed naming of Git and CMake to use their official capitalization.

Improve BitConverter.ToIntNN methods (#9803)

* Improve BitConverter.ToIntNN methods

- Avoid duplicate argument validation
- Fix failed optimization attempt: (uint)i < a.Length needs to be (uint) < (uint)a.Length otherwise it gets promoted to long and that's bad for 32 bit architectures
- Workaround JIT limitation: fixed variables cannot be enregistered
- Use Unsafe.ReadUnaligned to avoid alignment handling complications
- Use Unsafe.As instead of fixed as it generates shorter code
- Don't forward unsigned method to signed methods. The JIT doesn't want to inline the signed methods even though the generated code isn't significantly larger

Use Unsafe.ReadUnaligned/WriteUnaligned in UMA (#9829)

Add Unsafe.ReadUnaligned/WriteUnaligned (#9808)

Use WIN64EXCEPTIONS to declare EE_PERSONALITY_ROUTINE helpers (#9821)

[x86/Linux] Clean up unused NakedThrowHelper stub (#9824)

[x86/Linux] 16-byte aligned Funclet Helpers (#9810)

Clean up PInvokeStubForHost and PInvokeStubForHostInner (#9815)

Clean up nested WIN64EXCEPTIONS-ifdef (#9817)

[x86/Linux] Use _X86_ instead of _TARGET_X86_ (#9816)

[x86/Linux] Clean up PopSEHRecords (#9812)

[x86/Linux] Clean up CallRtlUnwind (#9813)

[x86/Linux] Clean up ARRAY STUB(s) (#9811)

Delete a bit of dead code (#9801)

Merge pull request #9789 from mikedn/le-intrinsic

Treat BitConverter.IsLittleEndian as an intrinsic

Merge pull request #9782 from mellinoe/ilasm-pathlength-fix

Increase the length of allowed parameters in ilasm

Update BitConverter from CoreFX (#9788)

* Update BitConverter from CoreFX

Related to #9701

* Use ThrowHelper

Better ArraySegment .ctor tests (#9800)

[ARM/CI] Fix arm32 CI git check failure (#9798)

Because layout in corefx binary has been changed, it causes arm CI failure.
And skip arm32 tests because test is not working recently.

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>

[ARM/CI] Update arm CI to sync up with corefx change (#9797)

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>

Fix _our_GetThreadCycles for clang/xarch.

The `"=A"` constraint in `asm volatile("rdtsc" : "=A"(cycles))` does not work as expected
on AMD64. Instead, the assembly must use an output constraint for both `eax` and `edx`
(i.e. `=a"(lo), "=d"(hi)`).

Refactor register arg handling in TreeNodeInfoInit.

The handling for outgoing register arguments in TreeNodeInfoInitCall was
unnecessarily complex and difficult to follow. In particular, the processing
of FIELD_LIST arguments performed a fair amount of unnecessary work,
contained code to deal with impossible IR patterns, and did not share any
code for handling the component PUTARG_REG nodes with the normal PUTARG_REG
case. This changes resolves all three of these issues and fixes an ordering
issue in lowering that was causing PInvoke prologs to remain unlowered.

Refactor isContained().

This changes isContained to terminate early if it is not possible for the node in question
to ever be contained. There are many nodes that are statically non-containable: any node
that is not a value may not be contained along with a small number of value nodes. To avoid
the need for a `switch` to identify the latter, their node kinds have been marked with a
new flag, `GTK_NOCONTAIN`. The checks for identifying non-containable nodes are encapsulated
withing a new function, `canBeContained`.

Merge pull request #9791 from dotnet/revert-9231-MinOptsSimplifiedGCtracking

Revert "Simplified MinOpts GC ref tracking."

Revert "Simplified MinOpts GC ref tracking."

Fix GC hole when exception filter throws unhandled exception (#9785)

The extra Unix specific piece of code in the StackFrameIterator::Filter that
handles the difference in the exception stack unwinding on Unix was not
skipping exception trackers belonging to filter clauses. But that was not
right, since filter funclet stack frames behave the same way on Windows and
Unix. They can be present on the stack when we reach their parent frame if
the filter hasn't finished running yet or they can be gone if the filter
completed running, either succesfully or with unhandled exception.

This change adds skipping of filter funclet related exception trackers at
that place so that the common code processes them.

This fixes the GC hole mentioned in the title that was discovered when
running some tests with GCStress mode 2.

Treat BitConverter.IsLittleEndian as an intrinsic

Remove  `this == null` checks from string.Equals (#9768)

Move S.Buffers and S.IO sources shared with corert into shared folder (#9778)

Merge pull request #9756 from AndyAyersMS/RefCountGenericContext

JIT: keep ref count for runtime generic context lookups

Use clock_gettime(CLOCK_REALTIME) in DateTime.UtcNow (#9772)

gettimeofday() is currently used in DateTime.UtcNow.  Depending on the system, it may have accuracy and precision in line with either clock_gettime(CLOCK_REALTIME) or clock_gettime(CLOCK_REALTIME_COARSE), but in returning its data it's necessarily limited by struct timeval's tv_usec field (microseconds), whereas clock_gettime uses struct timespec which has tv_nsec (nanoseconds).  This commit switches to use clock_gettime(CLOCK_REALTIME) to gain the additional precision (though the FILETIME through which the data is returned only supports 100-nsec precision).  On my machine, there was no measurable impact to throughput.

Merge pull request #9598 from ahsonkhan/OptimizeSpanClear

Optimize span clear

Increase the length of allowed parameters in ilasm.

The way this code is structured prevents parameters longer than 128 characters
from being passed. This change simply increases that to 1024 characters, although
a more robust change could be made in the future to account for platform differences.

Remove optimizedtls.cpp (#9777)

Remove NakedThrowHelper and ifdef-out its callers (#9771)

This change removes NakedThrowHelper function for Unix since it was not used.
It also ifdefs out its upstream callers.

review feedback

Merge pull request #9336 from mikedn/const-fold

Fix incorrect cast in gtFoldExprConst

Merge pull request #9769 from ianhays/filelock_osx

Remove OSX FileStream Lock/Unlock

Remove getcontext and setcontext usage (#9759)

It turns out that the getcontext and setcontext that I have used in my stack
oveflow reporting change are not present e.g. on Alpine Linux or on Android.
So I am replacing their usage with RtlCaptureContext and RtlRestoreContext
instead.

I have also found that the addition of the .cfi_adjust_cfa_offset to the
PROLOG_SAVE_REG_PAIR has broken unwinding of all helpers that use
PROLOG_WITH_TRANSITION_BLOCK, because the PROLOG_STACK_ALLOC macro
updates the CFA offset. So I am fixing that by removing the CFA offset
updating from the PROLOG_STACK_ALLOC and adding explicit one to the
ARM64 CallSignalHandlerWrapper.

Merge pull request #9732 from hseok-oh/fix_9442

Fix code generation for array length check in legacy JIT

Enable DevDiv_377155

Fix incorrect cast in gtFoldExprConst

The fact that an operation is unsigned affects the operation itself but that doesn't mean that the result of the operation is also unsigned. Constants are stored as ssize_t and the node type is TYP_INT so the result has to be sign extended, not zero extended.

Otherwise code that uses the return of IconValue() without first narrowing it to int will behave incorrectly. Such code does exists, even gtFoldExprConst does this when folding shift operations:

uint a = uint.MaxValue;
uint b = 0;
int r = (int)checked(a + b);
Console.WriteLine((r >> 2).ToString("X"));

The above code prints 3FFFFFFF instead of the expected FFFFFFFF.

This also makes gtFoldExprConst consistent with ValueNumStore::EvalFuncForConstantArgs which evaluates TYP_INT nodes as int and then casts to ssize_t.

Update FileLock/Unlock PNSE message for OSX

Avoid unnecessary branch in CultureAwareComparer (#9767)

* Avoid unnecessary branch in CultureAwareComparer

Determine the CompareOptions up-front instead of during each operation.

Merge pull request #9749 from wtgodbe/XunitCopyStuff

Use -f instead of -n when copying from Core_Root in runtest.sh

Remove OSX FileStream Lock/Unlock

OSX doesn't support usage of both fcntl and flock. Since we're already using one in FileShare for the entire file, we cannot enable partial file locking like we do on other Unix platforms. The alternative is to throw a PNSE and suggest using FileShare on the whole file instead.

Merge pull request #9766 from discostu105/patch-1

fix typo in sos_md.h

[Local GC] Move workstation GC DAC globals to a struct shared between GC and DAC (#9255)

* [Local GC] Move workstation GC DAC globals to a struct shared between
the GC and the DAC

* (Some) code review feedback and bug fixes for issues found while debugging on OSX

* Address some code review feedback:
 1. Make g_gcDacGlobals a pointer and dacvar on the VM side, so
    that publishing the GC dac vars is done atomically (through
    a pointer assignment). This fixes a race that Noah noticed.
 2. Remove the requirement for the GC's generation class struct
    to be known at compile-time, by using a dacvar as the size
    of the generation class at run-time (for pointer arithmetic)
 3. Move all DAC-interesting fields to be at the start of GC
    internal classes, so that the DAC does not need to know the
    size or exact layout of the class past the fields it cares
    about.

* Split the definition of the size of several arrays across the SOS/DAC and GC/DAC interfaces, and add static asserts that they are the same

* Repair the Windows Release build

* Implement the GC DAC scheme for Server GC and eliminate the duplicate GC dac vars

* Some work

* Decouple use of the GC generation table from a write barrier by having the EE store a copy of the global during initialization

* Actually make it work with server GC

* Checkpoint

* Checkpoint where everything works

* Code cleanup

* Fix debugger test failures

* Additional code cleanup

* Address code review feedback by adding a static assert and standardizing the way that we iterate over the generation table

* Repair the Windows x86 build

* Revert "Decouple use of the GC generation table from a write barrier by having the EE store a copy of the global during initialization"

This reverts commit 573f61a16b4fa8c2fc4c568c0b968a921230f31c.

* Revert "Repair the Windows x86 build"

This reverts commit 188c22d87e1d65abf00ab8fa28f46ad607a9028f.

* Partial revert, move `generation_table` back the global namespace for a single-proc allocation helper

* Fix a debugger test failure

* Repair crash dump scenarios

fix typo in sos_md.h

[Linux/ARM,ARM64] Fix undefined FixContextHandler issue (#9755)

Fix segmentation fault when using ZapImage::PrintStats() (#9731)

Currently, ZapImage::PrintStats is not used anywhere. But I want to get detailed informations about Native Image(ni.dll) components.
So I just used ZapImage::PrintStats when called ZapImage::SaveImage. After that I met the segmentation fault likes #9680.

I found the reason for this issue is accessing the null pointer to get sizes.

typo in CMakeLists.txt (#9763)

Merge pull request #9738 from stephentoub/ordinalignorecase_hashing

Tweak OrdinalComparer for OrdinalIgnoreCase

[x86/Linux] Initial patch for EH funclet (#9601)

- Generate a simple EH funclet frame and support SP-based stack unwinding for funclets.
- Introduce assembly helpers : CallEHFunclet and CallEHFilterFunclet