Merge pull request #11629 from pentp/Math-perf

Optimized Math.Sign

Update CI to follow up corefx changes (#12384)

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

.clang format from jit (#12377)

Merge pull request #12313 from kyulee1/st

Remove static for function pointer

Don't close passed in FileStream handles in constructor (#12253)

* Don't close passed in FileStream handles in constructor

As the finalizer will run when throwing we need to be
careful to make sure we don't put the passed-in handle
in the member field until we're sure we've succeeeded.

Additionally we need to make sure any wrapping SafeHandle
we create on a passed in IntPtr isn't collected.

* Move platform specific assertion and add comment.

* Comment- fix mistake in CanSeekCore

Fixing CompareInfo AssemblyMode.Full serialization between Core <--> Desktop (#12365)

* Fixing CompareInfo AssemblyMode.Full serialization

* Adding reason why culture field is introduced

Merge pull request #12055 from mikedn/lower-long-relops

[WIP] Simplify long relop lowering

ILLink Performance Tests: Enable MusicStore

Enable the Musicstore tests, since the failure in JitBench is
now fixed.

[RyuJIT/ARM32] Update LinearScan::resolveLocalRef() for ARM32 (#12194)

Update LinearScan::resolveLocalRef()
- Always update two float register consisting a double register
- Use helper function for ARM32 double register
- Use stronger assertion condition

Introduce updateAssignedInterval() to update assigned interval
of ARM32 register considering register type.

Merge pull request #12364 from adiaaida/removeSudoThroughput

Remove sudo from throughput run

Update CoreClr, CoreFx to preview1-25419-02, preview1-25419-02, respectively (#12342)

Remove sudo from throughput run

Sudo is not necessary for running throughput testing, so remove sudo so
we don't run into requests for passwords if the ubuntu machines are not
set up correctly.

Cleanup most code in StringBuilder (#10156)

* Cleanup most code in StringBuilder

* Respond to PR feedback

* Document StringBuilder's fields

* VerifyClassInvariant -> AssertInvariants

* Fix always-true assert

* Respond to PR feedback from @stephentoub

Merge pull request #12335 from mikedn/assert-prop-alloc

Remove unnecessary bitvec allocation in assertion prop

Further improve perf of Char.IsWhiteSpace (#12353)

IsWhiteSpaceLatin1 is not being inlined.  By changing a range check done with two comparison operations to instead be done with a subtraction and a single comparison, the code is shortened to not only be less expensive but also then get inlined into IsWhiteSpace and then further into String.IsNullOrWhiteSpace.  The net result is a measurable throughput improvement for IsNullOrWhiteSpace.

Merge pull request dotnet/corert#3911 from jkotas/nmirror-merge

Merge nmirror to master

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

Add regression test for issue #12224 (#12331)

This change adds regression test for EH getting stuck in an infinite
loop when NullReferenceException happens inside a handler of another
NullReferenceException.

Remove some dead argument checks for unsigned primitives for less than zero (#12349)

* Remove dead comparisons of unsigned primitive less than zero

* Delete bad assertition that is consistently hit using the Debug coreclr using the tests

Fix chained hardware exception handling on Unix (#12344)

There is an issue when hardware exception occurs while handling another hardware exception. In such case,
the exception unwinding ends up in an infinite loop. It is caused by the kernel reusing the same location
for signal handler context.

The fix is to use a windows style context local variable in the common_signal_handler that contains
the right context - it is the original signal context converted to windows style context.

Throw ArgumentNullException on Unix calling Marshal.IsComObject(null) (#12348)

Fix perf.groovy

Fix perf.groovy so that ILLink perf tests can be triggered.

LoadFromResolve event handler should return null incase of any load error (#12329)

Fix CompareInfo.Version on Windows (#12330)

We call Windows to get the sort version. We call Windows passing the structure NlsVersionInfoEx but Windows expect we have to initialize the size of this structure which we are missing.

Security build JSONs. (#12279)

Update CoreClr, CoreFx to preview1-25416-04, preview1-25416-02, respectively (#12319)

Add path to exception messages (#12322)

* Remove dead code

* Arg_PathIllegal

* IO_PathTooLong

* Remove no longer necessary path sanitization

* Argument_PathFormatNotSupported

* Arg_PathIllegalUNC

* Arg_InvalidSearchPattern

* Dead strings

* Missing nameof

Remove unnecessary bitvec allocation in assertion prop

optVnNonNullPropCurStmt is called for each node and every time it allocates an empty BitVec because optNonNullAssertionProp_Call and optAssertionProp_Ind require it, they are also used in other code paths that do provide a non-empty BitVec.

Merge pull request #12203 from wtgodbe/Timing

Fix timing issues in some baseservices tests

Merge pull request #12321 from BruceForstall/FixMeasureSize2

Convert size measurements for nodes to unsigned __int64

Remove unused source causing restore error

[ARM] Set local variables as addr-exposed (for LEGACY_BACKEND) (#12237)

Disable Portable RID build for Tizen (#12292)

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

Fix chained hardware exception handling on Unix (#12316)

There is an issue when hardware exception occurs while handling another hardware exception. In such case,
the exception unwinding ends up in an infinite loop. It is caused by the kernel reusing the same location
for signal handler context.

The fix is to use the windows style context local variable in the common_signal_handler that contains
the right context - it is the original signal context converted to windows style context.

Merge pull request #12320 from BruceForstall/CompactGenTreeCall2

Compact the size of GenTreeCall nodes

Merge pull request #12185 from hqueue/arm/ryujit/lsra_11851_resolve_edges

[RyuJIT/ARM32] Select proper temp register when resolving edges

Compact the size of GenTreeCall nodes

The data stored in GenTreeCall under `#ifdef FEATURE_MULTIREG_RET`
(defined for x86 (RyuJIT), amd64/Linux, arm32, arm64) was poorly
packed. Improving this reduces x86 GenTreeCall node size from 124 to 112 bytes.
Since GenTreeCall is the largest node, it also reduces the size of all
"large" nodes.

Convert size measurements for nodes to unsigned __int64

I found these numbers overflowing a size_t on x86 with a long
SuperPMI run.

Merge pull request #12298 from dotnet/revert-12227-PR-12197

Revert "[Arm64] JIT_WriteBarrier optimization"

[x86/Linux] Enable gcMarkFilterVarsPinned (WIN64EXCEPTIONS) (#11281)

* [x86/Linux] Enable gcMarkFilterVarsPinned

Do gcMarkFilterVarsPinned() for WIN64EXCEPTIONS

* [x86/Linux] GCInfo : Force this pointer untracked

`this` pointer is now always untracked so we can use pinned flag in tracked lifetimes.
This allows us to make the refs(inside filter) pinned to prevent from double-relocation.

* [x86/Linux] GCInfo : fix comment and formatting

* [x86/Linux] GCInfo : Update

- Force "this" pointer untracked only when "this" is generic context
- Style fixes

* [x86/Linux] GCInfo : remove this_OFFSET_FLAG

Update CoreFx to preview1-25415-03 (#12305)

Remove static for function pointer

qsort with static compare functions crashes with ALTJIT.
It appears due to _guard_check_icall_fptr in Windows 10.
Simply dropping static keyword works.

Merge pull request #12307 from stephentoub/iswhitespace

Small cleanup to Char.IsWhiteSpaceLatin1

Merge pull request #12311 from jashook/arm64_unix_ci

Add the arm64 unix CI

Fix timing issues in some baseservices tests

Merge branch 'master' into arm64_unix_ci

Add the arm64 unix CI

This will enable unix arm64 testing in CI. Currently there are no
default tirggers; however, anyone will be able to request the jobs.

Merge pull request #12310 from dotnet/revert-11743-arm64_unix_ci

Revert "Add the arm64 unix CI"

Revert "Add the arm64 unix CI"

Merge pull request #11743 from jashook/arm64_unix_ci

Add the arm64 unix CI

Merge pull request #12297 from stephentoub/getdatepart

Avoid duplicated computations with DateTime.GetDatePart

Small cleanup to Char.IsWhiteSpaceLatin1

For some reason this is resulting in measurably better throughput, in particular for non-whitespace chars, so much so that it's visible in microbenchmarks against string.IsNullOrWhiteSpace.

If nothing else, it's cleaner.

Merge pull request dotnet/corert#3890 from jkotas/nmirror-merge

Merge nmirror to master

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

Merge pull request #12047 from rartemev/issue_4122_2

Fixed determintaion of need to make code that contains tail.call loop fully interuptible

[Local GC] Fix a number of handle table interface violations (#12277)

* Fix a smattering of GC handle table interface violations

Stub out a few other handle violations

* [Local GC] Add SetExtraInfoForHandle onto GC handle manager interface

* [Local GC] Changes uses of HndGetHandleADIndex to GetHandleContext

* Add HandleGetType to GC handle table interface

1) Change IGCHandleManager methods that take "int" to represent the type of a handle to HandleType, and fix callers to cast to HandleType
2) Add HandleFetchType to IGCHandleManager, returning a HandleType
3) Fix uses of handtablepriv's HandleFetchType and remove the GC directory from the include path

* 9 -> HNDTYPE_WEAK_WINRT, 0 -> HNDTYPE_WEAK_SHORT in assert

Merge branch 'master' into arm64_unix_ci

Merge pull request #12287 from briansull/nowarn-SIMD

Crossgen - Disable the Target-dependent SIMD vector types warning

Merge pull request #12251 from kyulee1/override

Missing override keyword to DoPhase

Add sdmaclea to arm64Users

Update CoreClr, CoreFx to preview1-25415-03, preview1-25415-01, respectively (#12284)

Revert "[Arm64] JIT_WriteBarrier optimization (#12227)"

This reverts commit b5914c8d1b20be898b8982a4dfcf9d8e9046b2ec.

Avoid duplicated compuations with DateTime.GetDatePart

- Remove duplicate code in GregorianCalendar. GetDatePart is identical to the implementation in DateTime.  Just reuse it via the Year/Month/Day properties. Also avoids an unnecessary virtual call.
- Then copy GetDatePart into a version that returns year/month/day rather than just one of them.
- And use that in several places throughout corelib where all three are retrieved, avoiding redoing all of the calculations three times.

[RyuJIT/ARM32] Reserve a double register for both TYP_DOUBLE and TYP_FLOAT

First, try to reserve a double temp register for both TYP_DOUBLE and TYP_FLOAT.
If fails, try to reserve a float temp register for only TYP_FLOAT.

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

[RyuJIT/ARM32] Fix typo

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

[Arm64] JIT_WriteBarrier optimization (#12227)

* [Arm64] JIT_WriteBarrier optimization

Add skipEphemeralCheck if gcServer=1

Copy all gc parameters into a literal pool which
fits in single cache line

Add dmb ishst on gcHeap changes

Reduce unpredictable branches

Remove unneeded instructiuons

Fixes #11374
Fixes #12197

* [Arm64] fix typo in asm

* Fixup asm

* [Arm64] Revise per comments

Nits - Whitespace, Labels to PascalCase
Fix state change order
Add Suspend/Resume to fix race
Fix errors in Windows arm64 assembly

* [Arm64] Remove SuspendEE/RestartEE in StompWriteBarrierResize

* Use Volatile<> for g_highest_address

* Revise synchronization per comments

Remove unused forward declaration (#12288)

Fix build break

Download from Azure task does not retain blob prefix when downloading blobs.

Crossgen - Disable the Target-dependent SIMD vector types warning

When a SIMD type is referenced during CEEPreloader instead of loading it
we throw a type load exception with a unique resource ID and message.
This commit changed the ICorCompileDataStore interface to allow us to pass
the unique resource ID over to the zapper from the VM.

The Zapper can then demote certain resource ID warnings down to the
lowest informational level, so that they aren't printed out  during crossgen.
With this commit we currently are demoting IDS_EE_SIMD_NGEN_DISALLOWED

Update CoreClr to preview1-25414-05 (#12282)

Update buildtools to 2.0.0-prerelease-01714-04 (#12280)

Fix NullReferenceException thrown when using CurrentCulture (#12281)

CurrentCulture used the static field s_userDefaultCulture which used to store the current culture. We always ensure this field is initialized all the time. When we introduced the v2.0 added APIs, we have added ClearCachedData(). This method was setting s_userDefaultCulture to null which can cause any other thread to throw this NullReferenceException if it is using s_userDefaultCulture. Here is example of such stack

System.NullReferenceException : Object reference not set to an instance of an object.
Stack Trace :
   at System.Globalization.NumberFormatInfo.get_CurrentInfo()
   at System.Double.ToString(String format, IFormatProvider provider)

Renaming serializable field (#12273)

Remove the warning about Implicit boxing during crossgen (#12274)

Merge pull request #12272 from jashook/add_documentation_for_building_tests_on_unix

Add documentation for building test(s) on Unix

Add documentation for building test(s) on Unix

This also includes steps on how to build individual tests.

Fix a COMDouble::Round() issue (#12210)

* Fix a COMDouble::Round() issue
fixes https://github.com/dotnet/coreclr/issues/12137

* Add a couple tests for Math.Round(double)
based on https://github.com/dotnet/coreclr/issues/12137

* Use G17 format specifier for printing double values

Update CoreClr to preview1-25414-04 (#12270)

Add `-PortableBuild=false` option in Tizen CI build (#12241)

Additional to fix #12104 for CoreRT issue #3672 (armel tizen) (#12244)

* Additional to fix #12104 for CoreRT issue #3672 (armel tizen)

* Fix for unregistered info.compLvFrameListRoot

Fix gethashcode for ordinal comparer (#12267)

Fix GetHashCode for ordinal comparer

Rewrite of OpCodeGen.pl script to generate correctly IL OpCodes (#12040)

* Rewrite of OpCodeGen.pl script to generate correctly IL OpCodes related C# code: FlowControl.cs, OpCodes.cs, OpCodeTypes.cs, OperandType.cs, StackBehaviour.cs from opcode.def file

* Update OpcodeType.cs to OpCodeType.cs in System.Private.CorLib.csproj

Improve the packing of RefPosition (#12263)

The LSRA RefPosition type was not packing well. In particular,
bitfields don't pack unless they have the same base type.

Move all the data member declarations to the top, and make them adjacent,
so it is easier to see them all together. Also, change all bitfields
to "unsigned char" base type.

This reduces the size from 36 to 28 bytes for x86, and 48 to 40 bytes
for x64 (release, on Windows).

[Arn64/Unix] Revise Volatile.T barriers (#12156)

* [Arm64] Revise Volatile<T> barriers

* Remove VolateStore from CopyValueClassUnchecked

* Replace MemoryBarrier() in CopyValueClassUnchecked

Keep previous cmake arguments when using buildstandalonegc (#12108)

Currently, using buildstandalonegc makes ./build.sh discard any
previous values of __cmakearg set explicitly or implicitly. Preserve
those arguments.

Without this fix, the following invocations behave differently:

    $ ./build.sh ignorewarnings buildstandalonegc verbose

    $ ./build.sh buildstandalonegc ignorewarnings verbose

Update CoreClr, CoreFx to preview1-25414-02, preview1-25414-01, respectively (#12258)

Remove serializable attribute from OrdinalIgnoreCaseComparer (#12215)

* Remove serializable attribute from OrdinalIgnoreCaseComparer

* Introducing OrdinalCaseSensitiveComparer to serialize correctly with netfx

* Updated PR feedback

Merge pull request #12255 from briansull/concat-fix

Removed the unused vararg version of String Concat

Removed the unused vararg version of String Concat

The __arglist version of varargs is not supported on CoreCLR

Fixes issue #12146 - crossgen warning - Vararg calling convention not supported.

Added a new test case for String Concat
Added <DebugType>None
Fixed src\JIT\Directed\Misc\gettype csproj as well

Skip PGO if clang is too old, and add warnings for skipped PGO (#12248)

Clang/LLVM cannot merge profile data generated by a newer version of
itself. Training currently requires 3.6, but we don't want to completely
break the build for 3.5. I'm adding a version check to allow non-PGO
release builds on 3.5, along with a warning message. I'm also updating
the LTO detection to print a warning message if it causes PGO to be
disabled.

Added the checked CORE_ROOT path to this

Merge pull request #12092 from wtgodbe/CoreDisTools

Explicitly download correct version of CoreDisTools package

Specified where the failing tests are listed.

Updated the path for the test results html file

Fixed typo

Updated for checked build testing and corrected some paths

Missing override keyword to DoPhase

Merge pull request #12105 from alpencolt/ryu-arm-softfp-pass-ret

[RyuJIT/armel] Fix for correct passing and returning float values.

[RyuJIT/ARM32] Fix assertion when using invalid imm for INS_add (#12162)

* [RyuJIT/ARM32] Fix assertion when using invalid imm for INS_add

`JIT/Regression/CLR-x86-JIT/V1-M12-Beta2/b54611/b54611/b54611.sh`

Assertion failed '!"Instruction cannot be encoded" has been occurred.

INS_add couldn't process with the invalid imm of -273 on this test.
So in this PR, If it got the value that cannot be processed, moving to register that after changing status would be ran so that it can be processed.

* Modify 'Lowering::IsContainableImmed' to work normally.

insFlags when the function is called was always INS_FLAGS_DONT_CARE.
In this case, if the imm is such as -273, the result would be false.

The correct flags have to be set.

* add ifdef for the flags variable

Merge pull request #12243 from hqueue/arm/ryujit/issue_11842

[RyuJIT/ARM32] Enable HasMultiRegRetVal() for struct

Make ActivityID guids process specific. (#11726)

This is a port a change from the Nuget version of this library.
Bascially it hashs in the process ID into the activity ID GUID.  This make collisions across processes unlikely.

Merge pull request #12235 from BruceForstall/FixSuperPmiCollect

Fix SuperPMI collection script to allow passing arch/flavor args