[RyuJIT/ARM32] Fix comment and ifdef condition

Fix comment for elif and endif
Apply fix for RyuJIT/ARM32 only (not for LegacyJIT/ARM32)

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.

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

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

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>

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

Merge pull request #12064 from rartemev/fix_bash_script_issues

Fix path in bash script that runs test

fix procCount assertion (#12065)

Tiny typo in TaskCompletionSource xml comment (#12066)

Add support for not breaking *nix build on warnings (#12039)

Add a build flag to make -Werror optional and let the build continue
even in the presence of warnings.

This option is very useful for anyone compiling with a different
(version of the) compiler. A different (version of the) compiler may
produce a different set of warnings and a piece of code that compiles
without warnings may emit warnings with a different (version of the)
compiler.

Resolves https://github.com/dotnet/coreclr/issues/8586

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

Merge nmirror to master

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

Merge pull request #11699 from swgillespie/local-gc-branch-merge

Integration from dev/local-gc into master

attempt to fix r2r test failures

Merge pull request #11803 from BruceForstall/FixRuntest

Update runtest.cmd help to reflect reality

fix SleepEx(0,false) on Linux (#12057)

Return 0 on success from GC Reliability Framework scripts (#11990)

* Return 0 on success from GC Reliability Framework scripts

* Don't save standard out, Jenkins already does this

Merge pull request #11957 from sjsinju/spill_temp_size

[RyuJit/ARM32] Adjust alignment for TYP_DOUBLE in tmpPreAllocateTemps

Merge pull request #12003 from sjsinju/reg1_reg2_planB

[RyuJit/ARM32] Fix wrong behavior of UINT->CHAR casting

Add support for flat vtables in RyuJIT (#12041)

CoreRT needs one less indirection to get to the virtual method.

On CoreCLR a virtual method call resolution goes like:

```
this -> MethodTable -> chunk(?) -> slot
```

On CoreRT, it's more flat:

```
this -> EEType -> slot
```

Fix a bug in LoadNativeStringResource to honor the contract properly when the buffer is not big enough (#12051)

* Return last error properly

* adjust pcwchUsed if ERROR_INSUFFICIENT_BUFFER

* return hr

* ERROR_INSUFFICIENT_BUFFER is a win32 error....

Invoke jitShutdown during ProcessShutdownWork (#12054)

Delete long compare support from ARM32 codegen

Enable long relop lowering for ARM32

Add ARM32 support for GT_SETCC/GT_CMP/GTF_SET_FLAGS

[x64/Linux] Fix FEATURE_INTERPRETER Release build (#12046)

Fix formatting and bug

Fix formatting
Fix bug: DEBUG flag usage