Merge pull request #10040 from adiaaida/perfjobtriggers

Change perf test trigger phases

Change perf test trigger phases

Currently the trigger phases for perf testing use an underscore between OS
and architecture. This is inconsistent with our other trigger phrases.
This change changes the underscore to whitespace.

Merge pull request #10013 from BruceForstall/CiTailcallStress

Add COMPlus_TailcallStress=1 testing

Merge pull request #10017 from sivarv/morphAssertFix

Morph assert fix.

[x86/Linux] Calling convention mismatch inside Vector3Interop tests (#10028)

* [x86/Linux] Fix Calling Convention on JIT.SIMD.Vector3Interop tests

[x86/Linux] Calling convention mismatch inside systemvbrinup test (#10029)

* [x86/Linux] Fix calling convention mismatch inside JIT.Methodical.struct.systemvbringup test

Remove CORECLR_GDBJIT length check in gdbjit (#10016)

realted issue: https://github.com/dotnet/coreclr/issues/10003

Fix wrong gcc inline assembly code. (#10024)

It was coded as "rbx" instead of "ebx" register in clobber list position.

Fix assert in portable pdb reader running the clrstack command (#10011)

Issue #9969

Move two sharable classes into the shared partition. (#10009)

The corert side of this was commited by
  https://github.com/dotnet/corert/pull/2915#event-990346611

Merge pull request #10006 from wtgodbe/parallelWrapperGen

Zip all test wrappers in parallel

Remove TRACK_CXX_EXCEPTION_CODE_HACK and related code. (#10018)

Port Windows PGO support to master (#9985)

The bulk of this PR is a cherry-pick of commit fa02660 that shipped in
release/1.1.0, updating the build system support for PGO to support
consuming PGDs properly during release builds on Windows.

Also included are the following new changes:
* Skip restore of opdata if the requisite project.json is missing
* If the optdata package restore fails, fail the build.
* Add new build option: 'skiprestoreoptdata'

Note: This change doesn't by itself enable PGO in master yet, because
training data (optdata packages) for master don't exist on myget yet.
However, with these changes, the only step remaining to enable PGO
optimizations is to add a project.json referencing the correct optdata
package.

Merge pull request #10021 from adityamandaleeka/fix_broken_ci_badges

Fix broken CI badges

Add new JIT stress modes to the CI system; clean up netci.groovy duplication

1. Add JIT tailcall stress: COMPlus_TailcallStress=1
2. Add "SSE2 only": COMPlus_EnableAVX=0, COMPlus_EnableSSE3_4=0
3. Add new JIT stress regs mode: COMPlus_JitStressRegs=0x1000
   a. Do this plain, with JitStress=2, with CoreFX, and in R2R testing
4. Fix R2R JitStressRegs10 and 80 to actually test 10 and 80! (previously, they
   were passing "x10" and "x80", which get parsed as zero.
5. Delete tons of duplicate code from netci.groovy, and use a table/function
   to determine test display names.

Fixes #8626, #6129, #10004

Fix broken badges.

Add shared changes from corert (#10010)

Morph assert fix.

Merge pull request #10005 from stephentoub/taskextensions

Add TaskExtensions to CoreLib

Add scripts for building for arm Android (#9961)

* Add scripts for building for arm Android

* Use arm soft float for bringup

* Use NDK r14

* Set HAVE_PT_REGS to false, as they are not available on Android

* _aligned_alloc is not available on clang

* Detect Visual C++ instead of clang

* Detect Android instead of clang

[x86/linux] Fix "Undefined Reference" error in CLR release build. (#10000)

Replace "IncCantStopCount()" and "DecCantStopCount()" functions with
nothing(#define IncCantStopCount()) to call in methods of UTSemReadWrite
on x86/Linux.

Fix incremental build of mscordaccore (#9982)

* Fix incremental build of mscordaccore

This change sets up proper dependency for the mscordaccore build and stops
using PRE_LINK command which is executed always.
The way it solves the problem is that it creates an object library from the
mscordac.cpp and then lets the mscordaccore.exp be generated from that object
and finally use both that object library and the mscordaccore.exp while building
mscordaccore.

Merge pull request #9992 from adiaaida/fixjit32testing

Fix jit32 perf and throughput testing

Add TaskExtensions to CoreLib

Our TaskExtensions.Unwrap implementation in corefx is more allocation-heavy than the implementation we had in desktop and we could have in CoreLib, where CreateUnwrapPromise is available.  By moving this implementation down to CoreLib, the code:
```C#
Task<Task> toUnwrap = ...;
Task t = toUnwrap.Unwrap();
```
incurs 1 allocation instead of 4.

Fix jit32 perf and throughput testing

For jit32, we need to download the compatjit to make sure we are using a
jit32 build, not legacy backend (which is what is built when we run a
regular build out of coreclr). This change updates the groovy scripting to
pull down the jit nuget package that contains compatjit.

Until the packaging is fixed (#9991), we are defaulting to the last
package that packaged compatjit along with clrjit.

Merge pull request #9996 from BruceForstall/FixCallFunFormatting

Un-clang-format-horrible-ify emitIns_Call() and genEmitCall()

Zip all test wrappers in parallel

[x86/Linux] CDECL as default P/Invoke Calling Convetion (#9977)

* [x86/Linux] CDECL as default P/Invoke Calling Convetion

Enhance ICU version scan (#9993)

The scanning that tries to find locally installed ICU version was not
considering the case when only ICU library files with major version
component were present (like libicuuc.so.52). This is a rare case that
I've seen so far only when people built and installed ICU manually.
But it is worth to add checks for such case as a fallback, which is
what this change does.
I have also added checks for minorVersion and subVersion being -1 when
trying to detect the symbol versioning used. While the pre-existing code
would still work fine, it would unnecessarily try nonsensical version.

[x86/Linux] Fix incorrect FP values (#9975)

* [x86/Linux] Fix incorrect FP values

Restoring FP by CallerSP could be incorrect for nested EH.
Now that we have #9820, we can use current context for 1st pass of EH.

Fix #9848

Merge pull request #9995 from BruceForstall/FixNetciForceRelocs

Add missing "break" in a forcerelocs case

Harden resolveVirtualMethod against invalid types (#9989)

Split resolveVirtualMethod into a wrapper/helper to allow early
returns as failure reasons are discovered.

Before devirtualizing, ensure that the derived class claimed by the jit
is really a subclass of the class that introduces the virtual method.
Fail early if this is not the case.

Also add early out for interface call devirtualization. The jit currently
does not ask for this but asking for it should not trigger an assert.

Closes #9945 (along with #9959).

[x86/Linux] Check Stack Alignment in StubRareEnable/StubRareDisableTHROW (#9925)

Merge pull request #9986 from wtgodbe/XunitWrapperGranularity

Create Xunit wrappers for 2nd-level test directories instead of top-level

Fix ICU detection in System.Globalization.Native build (#9990)

The config.cmake was not getting the ICU library paths it requires to
build a code that detects presence of the ucol_setMaxVariable.
I have excluded the libraries detection from the CMakeLists.txt
some time ago since they are not used in the build itself anymore,
but I have not realized that they are needed for this detection
script that's included from the CMakeLists.txt.

Un-clang-format-horrible-ify emitIns_Call() and genEmitCall()

Add missing "break" in a forcerelocs case

Merge pull request #9991 from adiaaida/fixx86NugetPackageCreation

Change PackagePlatform to Platform for x86 jit pkg

Change PackagePlatform to Platform for x86 jit pkg

PackagePlatform was removed in 8dcf9665feabee339029c79b9d86dd24c71905d1,
but the x86 jit package props file was never updated to not use that
property. This change updates the props file to use Platform instead of
PackagePlatform to make sure we package compatjit.dll for x86.

Create Xunit wrappers for 2nd-level test directories instead of top-level

Implementation for Dictionary.TryAdd (#9923)

* Add Dictionary.TryAdd

* Fix the conflicts for real

* Remove branch from Add(TKey, TValue)

* Tweak naming

* Remove a slow assert

* PR feedback

Merge pull request #9398 from mikedn/switch-cast

Change switch value widening

Add stripSymbols argument; refactor symbol package build (#9664)

* Add argument to explicitly enable symbol stripping

* Refactor symbol package build

* Remove unnecessary native file distinctions

* Remove unnecessary WindowsSymbolFile step

[x86/Linux] Stack align aware unwinder (#9928)

Delete a few more unused hosting methods (#9973)

Add 3PN for GCBench (#9971)

Add 3PN entry for Livermore Loops coded in C (#9972)

Remove hosting methods that always return false (#9930)

* Remove CLRIoCompletionHosted as it always returns false.
* Remove CLRSyncHosted as it always returns false.
* Remove CLRMemoryHosted as it always returns false.
* Remove CLRTaskHosted as it always returns false.
* Remove CLRAssemblyHosted, CLRGCHosted,and CLRSecurityHosted.
They are not called.
* Remove IsThreadPoolHosted as it always returns false.
* Remove EnterRuntime and LeaveRuntime, as they do nothing.
* Add back calls to RevertIfImpersonated and GCX_PREEMP.

I accidentally deleted the call to RevertIfImpersonated instead of just
removing an extra parameter.

When I removed the HR_LEAVE_RUNTIME_HOLDER macro from windowsruntime.h,
I not only removed a LeaveRuntimeHolder but also a GCX_PREEMP. So I added
it back. The holder and GCX_PREEMP where only inserted when the
FEATURE_LEAVE_RUNTIME_HOLDER macro was defined. Since it is always defined,
I removed it. Also as I understand it, you would always want to have a
GCX_PREEMP before calling into the Windows API as not to block the GC,
so I'm not sure why you would want to disable it.

Delete IsNonW8PFrameworkAPI checks (#9964)

Dead code in CoreCLR

Add 3PN for Burgers equation (#9966)

- Thanks to @taumuon for licensing the test, per our request!
- License is MIT.

JIT: change codgen for dup to not lose type information

The jit has traditionally optimized most cases of a

   zz; dup; stloc.x

sequence into

   zz; stloc.x; ldloc.x

with a comment noting that this makes the resulting compiler IR
more amenable to CSE. Here zz is any IL operation that pushes
a value on the stack. However if zz's result is a ref type, the
dup'ed value produced by zz may be a subtype of the type of the
local x, so this transformation can lose type information.

In the past the loss of type information hasn't mattered, but now if
zz subsequently is used as the this object in a virtual call, the jit
may attempt devirtualization and ask an ill-posed question to the VM,
claiming the type of the this object is an impossible supertype.

This changes modifies the transformation by introducing a new local temp y
that has the type of zz. The result is now

   zz; stloc.y; ldloc.y; ldloc.y; stloc.x

with the net result that a ldloc.y is left on the stack with the proper
type. As an optimization, if zz is ldloc.z then the expansion is simply

   zz (ldloc.z); ldloc.z; stloc.y

and if zz is a simple "cheap" constant (say ldnull) then the constant
expression is duplicated:

   zz (ldnull); ldnull; stloc.y

This resolves the jit side of #9945. Further work is needed on the VM
side since in unverifiable IL there may be other cases where the jit
can present an impossible type.

Diffs from this were minor and in many cases small improvements.

Remove ComVisible as per #9371 (#9962)

When lazy was merged, this change was lost.

Remove unused bbAssertionKill (#9957)

Update glossary.md (#9951)

* Added the definitions for AOT, UTC and UWP.
* Made R2R as the initialism with 'Read-to-Run' its abbreviation.
* Fixed a typo: contributers -> contributors.
* Format as Term-Description markdown table.

Extending optimized JIT helpers to Buffer.MemoryCopy (#9786)

Move System.Lazy to shared CoreLib partition (#9955)

Merge pull request #9954 from tannergooding/fcall

Updating `COMSingle` in `classlibnative` to use the appropriate FCALL declarations.

Remove unreferenced file (#9952)

Alternative System.Lazy implementation (#8963)

* Remove use of lock for PublicationOnly

This involves a little bit of trickery. In avoiding the lock we need to
go from m_implementation as PublicationOnly to null in a single step. We
can't do this though without another object, so we reuse out Lazy
object, which basically just spins its wheels until the value is ready.
This shouldn't be a very common scenerio (i.e. hitting that code path)
but it is possible.

* Minimize additional object creation

Storing factory in Lazy object, and passing Lazy as a parameter to
ILazyItem functions. This means that None & PublicationOnly now need no
creation of secondary object.

* Remove Func<T> for default constructor invoking

Fixing startup performance concerns

* Moved non-generic functionality out of Lazy

...and into a helper class

* Expression-bodied functions

Fixing the FCALL 'common gotchas' comment to clarify that it applies to all types that are not passable via register in __fastcall.

Updating `COMSingle` in `classlibnative` to use the appropriate FCALL Declarations.

[x86/Linux] Clean up pContext access (#9933)

* [x86/Linux] Clean up pContext access

* Fix ARM/Windows build error

[x86/Linux] Revise VirtualMethodFixupStub (#9929)

* [x86/Linux] Revise VirtualMethodFixupStub

Clean up Normalization code (#9941)

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

Update CoreClr to beta-25103-03 (master)

Support local stack unwinding on Android (#9940)

Merge pull request #9773 from mikedn/user-range-check

Generate OAK_NO_THROW assertions from (uint)i < (uint)a.len checks

Add 3PN entries (#9943)

* Fix spelling

* Add 3PN for SciMark

* Add ByteMark 3PN entry

* Remove Microsoft copyright

* Add 3PN for Richards benchmark

Merge pull request #9873 from mskvortsov/ryujit-arm32-long

[RyuJIT/ARM32][WIP] Enable CG for long nodes

Add/improve assertion propagation comments

Update CoreClr to beta-25103-03

Generate OAK_NO_THROW assertions from (uint)i < (uint)a.len checks

If "(uint)i < (uint)a.len" is true then we know that "i" is always within bounds and so "a[i]" does not need a range check.

Refactor JTRUE handling during assertion gen/prop

- Pull JTRUE related code out of the inner loop of assertion gen to keep the loop small and simple
- Ignore JTRUE in the inner loop of assertion prop, there's nothing to propagate to or from it
- Avoid some unnecessary bitset clears and copies

Add tests for (uint)i < (uint)a.len generated assertions

Merge pull request #9897 from hseok-oh/patch_rootfs

[ARM32/Linux] Prepare rootfs to build libcoreclrtraceptprovider.so

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

Update CoreClr to beta-25103-02 (master)

Use CoreCLROverridePath property for CoreFx tests

Remove file copy hack and use the new msbuild property to allow
overriding the runtime when running CoreFx tests.

Fix license header for test (GitHub_1323) (#9931)

RyuJIT/ARM32: enable codegen for long nodes.

Update CoreClr to beta-25103-02

Update CoreClr to beta-25103-01 (#9927)

Merge shared changes from corert and use C# shared project to enable VS editing (#9922)

Merge pull request #9915 from BruceForstall/IncBuild1

Fix AsmConstants.inc to be incrementally rebuilt as necessary

Merge pull request #9921 from adityamandaleeka/rt_dump_changes

Fix error in dumpling text file cleanup

Fix AsmConstants.inc to be incrementally rebuilt as necessary

Previously, it was rebuilt on every build.

Merge pull request #9917 from BruceForstall/BuildCrossAltJit

Build cross-compile altjits for arm32 and arm64

Fix error in dumpling text file cleanup.

Build cross-compile altjits for arm32 and arm64

On x86, build an x86-host, arm32-target altjit.
On amd64, build an amd64-host, arm64-target altjit.

Each are named protononjit.dll, using our existing naming
convention (proto for "prototype", nonjit meaning the code
won't be executed).

Use these via:
```
set COMPlus_AltJit=*
set COMPlus_AltJitName=protononjit.dll
```
Of course, instead of `*`, you can use any function name.

They are very useful for debugging the JIT in a "nicer" environment
than the full native environment, especially if the full native
environment isn't complete, or if the tools (such as debuggers, editors)
are better in the cross environment than the native environment.

These are only built on Windows. It might be possible to build them
on Linux as well, but that will require some more build work.

[x86/Linux] Add STDCALL for reverse P/Invoke delegates (#9909)

[Linux][GDB-JIT] Fix lldb type displaying issues (#9906)

* Make array data have fixed size of 1 element

* Add bounds info for multidimentional arrays

* Fix array element size for value types

* Fix ref types duplication in DWARF types cache map

* Add typedef for reference types

Example: make MyClass a typedef for __MyClass *

* Use type name with namespace in DWARF

* Fix duplication of typedef statements in DWARF

* Fix memory leak in ByteTypeInfo::DumpStrings

* Add typedefs for build-in types

Merge pull request #9916 from dotnet-bot/from-tfs

Merge changes from TFS

Add src\jit\protononjit directory to git mirror in preparation for future GitHub work

[tfs-changeset: 1649472]

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

Update CoreClr to beta-25102-03 (master)

Merge pull request #9775 from adiaaida/addThroughputPerfTesting

Add Throughput Perf testing

Fix LDVIRTFTN case for GVM target methods (#9890)

* Fix LDVIRTFTN case for GVM target methods

Update windows-instructions.md

Removing ref T Span.GetItem and updating Indexer to return ref (#9787)

Update CoreClr to beta-25102-03

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

Update CoreClr to beta-25102-02 (master)

Devirtualize calls in some simple cases

Devirtualize calls where the this object type at a call site
is a subtype of the type described at the call site. Currently learns
types from local and arg references and a subset of other operators.

Will devirtualize if either the class or method is final (sealed in C#),
or if the type is known exactly (eg from a newobj).

Devirtualization is run twice, once during importation, and again in a
limited way after inlinining. Calls devirtualized during importation are
are subsequently eligible for inlining. Calls devirtualized during inlining
currently cannot be inlined.