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

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()

[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.

Jit interface support for devirtualization

Add new method to jit interface so the jit can determine what derived
method might be called for a given base method, derived class pair.

Implement support in the VM and in other places (zap, spmi).

[x86/Linux] Fix clang 3.9 build error (#9900)

Update CoreClr to beta-25102-02

[x86/Linux] Initial filter support (#9820)

* [x86/Linux] Initial Filter Support

Remove CLRThreadpoolHosted as it always returns false. (#9899)

[x86/Linux] Clean up EXCEPTION_REGISTRATION_RECORD (#9895)

Temporarily disabling SpanBench tests that use indexer setter. (#9896)

Update links to point to valid BotR pages - One extra link that was missed out (#9878)

Use the correct class handle for fat call temporary (#9893)

`clsHnd` is the class handle of the type that owns the method for
`opcode != CEE_CALLI`. We need the class handle for the method's
return type though.

This is ending up with RyuJIT asking weird questions over JitInterface
(`getClassGClayout` of a reference type).

Also removing a line that was added in #9191 to support the old code.

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

Prepare rootfs to build libcoreclrtraceptprovider.so
by adding fix process of rootfs for ubuntu 14.04 armhf

Preallocate jump stubs for dynamic methods (#9883)

Preallocate jump stubs for dynamic methods

- This eliminates the possibility of running into an out-of-memory situation after compiling the method
- The temporary entry points block containing FixupPrecodes is extended for dynamic methods to include sufficient space for jump stubs
- When the target is too far for the FixupPrecode to encode a short relative jump, it instead does a short relative call or jump to the corresponding jump stub, which does an absolute jump to the target

Adding Span API perf tests for comparison with arrays and to monitor progress (#9790)

* wip

* Adding Span API perf tests for comparison with arrays and to monitor progress

* Reducing iteration count so tests finish in < 2 minutes

* Reducing the number of test cases

* Commenting out ref DangerousGetPinnableReference test

* Updated csproj to use latest compiler

* Tuning test cases and runtime

Cleanup the native globalization code (#9887)

* Cleanup the native globalization code

After we merged the globalization code used for Windows and Linux, this change is cleaning up the native VM globalization code

* Added a comment to the normalization code

Fix casing for newly added test (#9889)

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

Update CoreClr to beta-25101-04 (master)

Cleanup (#9892)

* Delete unused PdbTypeMatch tool
* Delete unused GetFileVersion.dll binary
* Delete bad comments

[x86/Linux] Initial fix of arguments passing in FunctionPtrTest (WIP) (#9855)

* [x86/Linux] Initial fix of incorrect arguments passing in FunctionPtrTest

Fix incremental build on Unix (#9779)

This change fixes issue with dactablerva.h being generated even if the
libcoreclr.so hasn't changed. Also, the dependency of the daccess.cpp
on this header was not honored by the build system and so even if the
dactablerva.h was regenerated with a different contents, the daccess.cpp
was not rebuilt.

[Arm64/Unix] Fix stack unwinding (#9877)

This restore the functionality lost by #9866 w/o functional
loss

Merge pull request #9868 from wtgodbe/HelixPrep

Allow user to zip tests even when not building against packages

Update CoreClr to beta-25101-04

Merge pull request #9770 from adityamandaleeka/card_bundles_unix_2

Implement second-level card tables for non-Windows platforms

Add Throughput Perf testing

This change adds scripting to run perf testing in the lab. We perform the
testing by running crossgen on a list of dlls. It then writes the data to a
csv file which is passed to other processing scripts.