Merge pull request #12011 from pgavlin/FixTestNugetRuntimeId

Fix TestNugetRuntimeId.

First steps for generating nuget packages for ARM64/Android. (#10286)

* First steps for generating nuget packages for ARM64/Android. Adds Android RIDS and related conditionals to appropriate files

* Additional changes for Android package build

* Revert addition of Android to supported packages

* Allow Android .nuget package build

Adding runtime.Android files for package generation
Adding TODO for reliable generation of __DistroRid for Android

* Added/corrected logic for detecting __DistroRid at commandline
Removed hack and TODO

* Removed newline

* Revert unintentional change to osx rid

* Removed commenting of __DistroRid initialization

* Latest fixes for __DistroRid cmdline detection

Removed __DistroRid initialization (clobbers value set at cmdline)
Removed echo test of __DistroRid in generate nuget pkg function
(Hopefully) fixed cross-build os-release detection

* Removed android.21-arm from supported RIDs

Removed arm32 from supported RIDs until test build can be done or CI for arm32 added (?)

* Fixes for package generation for android

Modified build-android-rootfs.sh so it would generate file w/platform info
Reverted all prior changes to initTargetDistroRid, added new android platform detection

* Fixed conditional for Android

* Added platform generation for Android portable build, if ever needed

* Added information about android_platform

Added help info about android_platform file
Added TODO for dynamic generation of RID based on $__ApiLevel and $__BuildArch

* Syntax cleanup for build.sh

Removed extra space in initTargetDistroRid() logic

* Recommended review changes

removed Android-specific runtime files for packaging
added conditional to have Android use Linux runtime files

* Updated topic branch from master, resolving SPC.ni.dll file nuget pkg issue

* ericstj review changes

Fixed accidental fedora version change
Fixed indenting

* Removed TODO comment; logic already in place, added in earlier commit

* Removed Android OfficialBuildRID

Merge pull request #12172 from pgavlin/gh12154

Don't dump an uninitialized bitset in LSRA.

Merge pull request #12148 from mikedn/oper-dump

Make JIT dumps more readable

Don't dump an uninitialized bitset in LSRA.

LSRA was unconditionally dumping `fpCalleeSaveCandidateVars`, which is
not initialized if we are not enregistering any lclVars. This was
causing an assertion with dumping enabled.

[Local GC] Add GetThread to the GC/EE interface (#12036)

Merge pull request #12005 from wateret/wip-armel-lsra

[RyuJIT/armel] Support putting floating-point args

Describe how to use daily by setting RuntimeFrameworkVersion and publishing self-contained. (#12056)

* Describe how to use daily by setting RuntimeFrameworkVersion and publishing self-contained.

[Arm64] stlr for JIT_WriteBarrier

[Arm64] Use ish domain JIT_WriteBarrier

Merge pull request #12145 from AndyAyersMS/UpdateBenchmarks

Modify iteration behavior some benchmarks

Update CoreClr to preview1-25408-03 (#12161)

Merge pull request dotnet/corert#3832 from dotnet/nmirror

Merge nmirror to master

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

Fix linker GUARD option (#11492)

I have found that the /GUARD:CF option set in the root CMakeFiles.txt
was not being propagated to the Visual C++ project files generated
by cmake because of a wrong casing. The CMakeFiles.txt used /guard:cf
but it works correctly only if it is all upper case.

Merge pull request #12102 from tmds/any-linux

build.sh: fall back to portable Linux when Linux host has unknown rid

Update CoreClr, CoreFx to preview1-25408-01, preview1-25408-01, respectively (#12157)

Merge pull request #12155 from pgavlin/LclVarChecks

Add lclVar semantic checks to CheckLIR.

Format code.

[RyuJIT/armel] Support putting floating-point args

- Make Lowering and LSRA be aware of armel argument push convention.
- Implement codegen for GT_COPY that was newly created from `LowerArg()`.
- Only `float` type is supported. (`double` is not supported yet.)

Fix #11928

Address PR feedback.

Merge pull request #12142 from BruceForstall/ReduceNoway

Convert some very common noway_assert to simple assert

Merge pull request #12087 from sdmaclea/PR-JIT-AcquireRelease-SimpleCases

[Arm64] Pr jit acquire release simple cases

Add lclVar semantic checks to CheckLIR.

Specifically, ensure that an unaliasable lclVar is not redefined between the
point at which a use appears in linear order and the point at which that
use is consumed by its user. This ensures that it is always safe to treat
a lclVar use as happening at the user (rather than at the lclVar node).

delete BLOCKSET_INIT_NOCOPY and BLOCKSET_INIT. (#12144)

BLOCKSET_INIT was unused.
BLOCKSET_INIT_NOCOPY was replaced by the implicit constructor.

rename temp variables with meaningful names. (#12023)

It was hard to follow the logic.

Only restore optdata for release builds (#12136)

Avoid restoring optdata during a non-release build to minimize work done
by build.sh/cmd. Fix #12126

Simplify genRegArgNext() (#12124)

* Simplify genRegArgNext()

There is no need to check the register type before calling REG_NEXT.

* Rewrite using `switch`

* Fix genRegArgNext for Linux x64

Make JIT dumps more readable

Replace all uses of NodeName with OpName so we get proper names instead of symbols (e.g. ADD instead of +).  Names stand out better, especially in JIT dumps where we use various symbols to draw tree lines.

Modify iteration behavior some benchmarks

This change updates the iteration behavior for the following benchmarks, so that
they generally run for about 1 second per iteration:
* Inline/InlineGCStruct
* Inline/NoThrowInline
* Span/IndexerBench/Indexer4
* Functions/MathTests/*

It also removes the length 1, 10, and 1000 variants of the Span/SpanBench
tests, leaving just the length 100 variants. Analysis has shown that there
is no need to test varying lengths here. Iteration counts for these tests
have been left as is and will be fixed subsequently.

Partially addresses #11654.

Merge pull request #12138 from pgavlin/CheckLIRCodegen

Run the LIR checker before codegen.

Merge pull request #12139 from pgavlin/coredistoolsfix

Define `__cdecl` if necessary in coredistools.h.

Convert some very common noway_assert to simple assert

With these few changes, I measured a JIT instruction count reduction
of 0.37% of SuperPMI over the tests, and 0.17% for MinOpts.

Related to #10421

Merge pull request #12109 from briansull/ibc-tuning

Fixed the loading of ReadyToRun tuning native image

Define `__cdecl` if necessary in coredistools.h.

This fixes a build break for CoreDisTools on Linux.

Run the LIR checker before codegen.

Just what it says on the tin.

Merge pull request #12122 from BruceForstall/CleanupEffectiveVal

Remove use of gtEffectiveVal in RyuJIT codegen

Merge pull request #11994 from CarolEidt/Throughput

Streamline LSRA for non-enregistered lclVars

Update CoreClr, CoreFx to preview1-25407-02, preview1-25407-02, respectively (#12133)

Fix FreeBSD build

Fix portable build sos plugin problems (#12125)

* Fix portable build sos plugin problems.

Removing the explicit reference to liblldb. Since the lldb program has
already loaded this lib, our will now load regardless of the distro
and version of lldb.

Issue #12098.

* Fix OSX build.

build{,-test}.sh: revert and add list of unsupported rids

Update CoreClr, CoreFx to preview1-25406-05, preview1-25406-02, respectively (#11951)

Fix build breakage on high-processsor machines (#12114)

The build calls `make -j $NumProc`. This breaks on machines with a high
number of procesors (such as 25).

To reproduce this on any machine, edit build.sh and change

    buildTool install -j $NumProc

to

    buildTool install -j 100

The error trace looks like this:

    In file included from coreclr/bin/obj/Linux.x64.Debug/Generated/eventpipe/dotnetruntime.cpp:12:
    In file included from coreclr/src/vm/common.h:306:
    In file included from coreclr/src/vm/eepolicy.h:15:
    In file included from coreclr/src/vm/vars.hpp:70:
    In file included from coreclr/src/vm/eeprofinterfaces.h:19:
    In file included from coreclr/src/inc/profilepriv.h:128:
    In file included from coreclr/src/inc/profilepriv.inl:18:
    In file included from coreclr/src/vm/eetoprofinterfaceimpl.h:23:
    coreclr/src/inc/eventtracebase.h:306:10: fatal error: 'clretwallmain.h' file not found

It turns out that the eventpipe code has a dependency on
clretwallmain.h. src/CMakeLists.txt points out that clretwallmain.h is a
generated file. But there's no actual dependency between the eventpipe
target and the header file(s).

Fix that by fixing the generator script to insert an explicit dependency
between 'eventpipe' module and 'GeneratedEventingFiles' module.

Restore logic for when to use satellite resource lookup under AppX (#12117)

Remove unused cache of Assembly flags while I was on it

More formatting

Formatting

Streamline LSRA for non-enregistered lclVars

- Don't create any lclVar intervals if no lclVars are ever enregistered.
- Don't create intervals for tracked lclVars that are not register candidates.
- Don't create any `RegToVarMap`s when no lclVars are ever enregistered.
- Don't do edge resolution if no lclVars are enregistered.
- Eliminate non-candidate lclVars from the sets used by LSRA.

Also:
- Change VarToRegMap to use regNumberSmall.
- Enhance `COMPLus_JitLsraStats` and `COMPlus_JitTimeLogCsv` dumping.

Remove use of gtEffectiveVal in RyuJIT codegen

This is not required in RyuJIT at the codegen phase: COMMA and
value-producing NOPs are no longer in place.

Merge pull request #12113 from JosephTremoulet/NoTailStress

Stop extraneously setting tail call stress

Merge pull request #12119 from wtgodbe/StableVersion

Fix error with Stable Version

Fix error with Stable Version

Merge pull request #12079 from hseok-oh/ryujit/fix_11850

[RyuJIT/ARM32] Fix assertion when struct argument passing less than 4 byte

Merge pull request #11897 from wtgodbe/TestCorrectVersion

Test packages from current build in pipeline

Fix for CoreRT issue https://github.com/dotnet/corert/issues/3672 (armel tizen) (#12104)

Stop extraneously setting tail call stress

Tests GitHub_10780 and GitHub_11689 are repro cases for issues that don't
have anything to do with tail calls; the bit of code in their csproj's
that sets COMPlus_TailCallStress was an accidental copy-paste inclusion
from GitHub_11408.  Remove that extra code.

Test packages from current build in pipeline

Merge pull request #12069 from CarolEidt/CrossGenMinOpts

Support COMPlus_JITMinOpts for crossgen

Fixed the loading of ReadyToRun tuning native image

Corrected the new ifdef that prevented the loading of tuning/profiling ReadyToRun native images
Commented out a costly precondition check, it is always true at all callsites.

Rename CMAKE_ENABLE_CODE_COVERAGE to CLR_CMAKE_ENABLE_CODE_COVERAGE (#12106)

new JitEE interface method: expandRawHandleIntrinsic (#12071)

Create a new JitEE method expandRawHandleIntrinsic. Delete old isDelegateCreationAllowed and getAddrModuleDomainID.

Support CORINFO_INTRINSIC_GetRawHandle.
Change JITEEVersionIdentifier.
fix MEASURE_CLRAPI_CALLS: add resolveVirtualMethod and expandRawHandleIntrinsic there.

Fix a bad merge for Windows standalone GC build (#12097)

Fix crash due to lack of PSPSym in the CoreRT ABI (#12082)

Signed-off-by: Petr Bred <bredpetr@gmail.com>

build.sh: fall back to portable linux when Linux host has unknown rid

[Local GC] Add TrapReturningThreads to the GC/EE interface (#12067)

TypeForwardedFrom attributes on coretypes for NetFX serialization (#12059)

* TypeForwardedFrom attributes on coretypes for NetFX serialization

* ValueTuple added

* Removed StubEnvironment typeforwards

* Modified FQAN for some special type forwards

* Support desktop to core serialization with InternalsVisibleTo for mscorlib

* Remove typeforwards from nested types

* Removed unused file

Merge pull request dotnet/corert#3793 from dotnet/nmirror

Merge nmirror to master

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

Disable AppDomainLeaks configuration option for now until it is removed (#12095)

Merge pull request #11205 from mskvortsov/FixFloatingLdSt

[RyuJIT/ARM32] Workaround vstr/vldr addressing modes limitations

Merge pull request #12091 from pgavlin/TailCallStress11689

Disable tail call stress in GH_11689 if ZapDisable is enabled.

Merge pull request #11884 from mikedn/arm-cmp-setcc

[WIP] Lower long compares that are materialized into a register for ARM32

Merge pull request #12072 from ellismg/update-buildtools

Update BuildTools

Do not set GTF_IND_UNALIGNED on byte sized types

[Arm64] Respond to review comments

Merge pull request #12085 from dotnet/Clean_up_properties_sent_for_Publish_builds

Make all publish legs of CoreCLR uniform and match existing ones.

Disable tail call stress in GH_11689 if ZapDisable is enabled.

Tail call stress does not mix well with ZapDisable due to the issues
described in #11408.

[Arm64] Handle exclusive decode group in insEncodeDatasizeLS()

[Arm64] Use Store Release when practical for genCodeForStoreInd

[Arm64] Use Load Acquire when practical for genCodeForIndir

Make all publish legs of CoreCLR uniform and match existing ones.  Previously Release - Publish symbols was in the CoreFX-style format and the other three were not.  This unifies them.

Inline YieldProcessor on Linux (#12058)

More ldar/stlr support

[Arm64] Add emitInsTargetRegSize support for ldar/stlr

Merge pull request #12025 from hseok-oh/ryujit/remorph_filedlist

Skip argTable assignment for split struct when remorphing

Update morph.cpp

Fix an excess allocation of internal registers

Forbid making lea contained for vstr/vldr

Modify genCreateAddrMode to make it behave the same way as on arm64

Adapt shared genLeaInstruction for arm32

Remove arm32's genLeaInstruction

Move arm64's genLeaInstruction into codegenarmarch.cpp

Revert commit dfca51c6

[RyuJIT/ARM32] Struct argument passing less than 4 byte

Fix assertion when struct argument is less than 4 byte and promoted

Revert and remove isReMorph argument

- Revert commit that is removing nextSlotNum update in remorphing phase
- Remove isReMorph argument in SplitArg by using argsComplete

Additional fix for ETW failure in 32 process on 64 bit machines. (#12022)

* Addtional fix for ETW failure in 32 process on 64 bit machines.

The commit associated with Pull Request
    Fix failures in ETW logging on 4GB aware 32 bit processes #11941

Was incorrect.   Some long were changed to ulong, but the critical casting an IntPtr to ulong still does
sign extention and not zero extension.   Need to cast to a void* first.  This fixes this.

Merge pull request #12068 from omajid/master

Fix typo in CRL_CMAKE_WARNINGS_ARE_ERRORS

Remove relocations from SECTION_MethodDesc for ngened images for fields accessed from jit code for Linux ARM (#11963)

Call sched_yield directly for Sleep(0) (#12063)

Related to PR #12057

Support COMPlus_JITMinOpts for crossgen

In order to better enable throughput measurement and analysis,
we would like to be able to specify COMPlus_JITMinOpts with crossgen.

Fix typo in CRL_CMAKE_WARNINGS_ARE_ERRORS

Update BuildTools

This picks up some changes to how the tool runtime is published (we
now publish it as a portable app so we have less random .so's lying
around).