Add a brief comment to the new test.

Preserve VNs in fgMorphIntoHelperCall.

This method is used to transform a node into a semantically-equivalent
helper call, and should preserve any value numbers that are present on
the the node.

Merge pull request #10922 from adiaaida/fixXUnitPerf

Fix run-xunit-perf.cmd for local use

[x86/Linux] Use appropriate FCALL macro for SafeBuffer methods (#10935)

Remove distro specific RIDs from SOS.NETCore's project.json. (#10925)

Ensure Environment.ExitCode works correctly after app domain unloaded. (#10842)

* Ensure Environment.ExitCode works correctly after app domain unloaded.

This PR addresses 2 problems of Environment.ExitCode:

1. Can't get correct exit code of main function.
2. Can't set %errorlevel%.

Details can be found on #6206

Fix #6206

Update CoreClr to preview1-25213-02 (#10937)

Merge pull request #10875 from rahku/fix_x86_gcstress

modify lastAvAddress when adjusting context for Stub

[ARM32/Linux] Build libcoreclrtraceptprovider.so (#10406)

* Build libcoreclrtraceptprovider.so for ARM32 and modify cmake

- Build libcoreclrtraceptprovider.so for ARM32
- Include libcoreclrtraceptprovider.so in nuget package
- Don't build libcoreclrtraceptprovider.so for
Tizen: lttng is not supported in Tizen
- Exclude packaging libcoreclrtraceptprovider.so based on os-name
excluded for tizen

* Remove libcoreclrtraceptprovider.so before test for ARM32/Linux

Remove libcoreclrtraceptprovider.so before test for ARM32/Linux in CI
CI test environment using docker with qemu has some problem to use lttng library

Create ELF core dump on coreclr exceptions and signals. (#10902)

* Create ELF core dump on coreclr exceptions and signals.

When coreclr aborts because of a unhandled managed exception or hardware signal, the
"createdump" utility is launched if the "COMPlus_DbgEnableMiniDump" environment
variable is set to "1".

This utility uses the DAC enumerate memory interface to determine what memory regions
should be written for the managed state plus all the normal Linux memory regions for
the native state like threads, stacks, DSO, etc.

There still needs some to configure some of the "policies" like the name/location of
the core dump and the type of minidump (MiniDumpNormal, MiniDumpWithFullAuxiliaryState,
MiniDumpWithPrivateReadWriteMemory, etc.).

Currently only supported on Ubuntu Linux.

Make createdump smaller by using PAL in DAC module.

Add platform (x64 & arm64) specific optimized jit helpers for statics access (#10912)

Merge pull request #10930 from rahku/marshalfix

HelperMethodFrame missing at certain places in method MarshalNative::PtrToStructureHelper

HelperMethodFrame missing at certain places in method MarshalNative::PtrToStructureHelper

Merge pull request #10914 from hqueue/jit/remove_unused_fuction

Remove unused function in BasicBlock

Merge changes from TFS (#10926)

* Fix desktop build break

[tfs-changeset: 1654125]

* JIT formatting

Merge pull request #10917 from stephentoub/fix_badges

Fix build badges in OfficialAndDailyBuilds.md

[x86/Linux] Fix ResumeEsp before resume (#10749)

* [x86/Linux] Correctly unwind FCALL frames

* Fix Up ResumeEsp inside ProcessCLRException

* Fix Context Before PC Update

[x86/Linux] 16-byte aligned BackPatchWorkerAsmStub (#10759)

* [x86/Linux] 16-byte aligned BackPatchWorkerAsmStub

* Use fixed alignment padding

* Do NOT push ebx (no longer required)

Merge pull request #10905 from pgavlin/VSO406158

Fix a number of reordering issues in shift decomposition.

[x86/Linux] 16-byte aligned DelayLoad_MethodCall (#10760)

Merge pull request #10706 from sdmaclea/PR-ARM64-GC-EmitInsLdStOp

[Arm64] Fix GC Holes in emitInsLoadStoreOp

Merge pull request #10923 from wtgodbe/ImproveReporting

Improve reporting parameters for pipebuild

Merge pull request #10911 from adityamandaleeka/handle_table_local_gc_hide_internals

[Local GC] Hide handle table internals from VM side

Improve reporting parameters for pipebuild

Fix closed delegate deserialization to account for extension methods (#10901)

Functional fix for #9597:
- The code was incorrectly assuming that the serialized target type info refers to the target object's type; it actually refers to the method's reflected type, which may not be the same type when it's an extension method

Merge pull request #10243 from sdmaclea/PR-CALLI-LOWERING-ALLOWED

Allow target of calli to be lowered to ldc.i4

Fix run-xunit-perf.cmd for local use

run-xunit-perf.cmd  specifies the full path to the benchmark using
%WORKSPACE%, which a user may not set locally. This change modifies it to
use %CORECLR_ROOT%.

NS2.0 AssemblyName.ReferenceMatchesDefinition() (#10919)

This api was broken from its debut in 2.0 (and yet somehow
scored a 2.9% on .NET api catalog)

 https://connect.microsoft.com/VisualStudio/feedback/details/752902/assemblyname-referencematchesdefinition-returns-true-even-though-the-parameters-dont-match

In 2012, the MSDN docs were changed to match its broken behavior:

   Return Value
   Type: System.Boolean
   true if the simple assembly names are the same; otherwise, false.

   Remarks

   The comparison depends only on the simple assembly name. It ignores version, culture, and public key token.

A more accurate description would be:

   This api takes the simple names of the ref and def.
   Then it parses the simple names *as if they were*
   full-blown assembly names with optional Version, Culture
   and PKT (!)

   Then it applies a RefToDef test to that name using a test
   that's quite separate from the one used by the actual
   CoreCLR loader (so any similarities between
   it and the actual binding rules is purely coincidental.)

   Oh, and that code assumes the Def name came from trusted
   sources so if it's has a null Culture property, it NULL-AV's.

We could replicate this behavior on CoreRT (relying on
its own AssemblyBinder) but the more I progressed, the
more the venture failed to pass the laugh test (though
at least we got a nice AssemblyBinder cleanup
as a side effect.)

So what we'll do is make the api behavior match the
MSDN description (and likely come close enough to the compat bar)
but in a more straightforward fashion.

(The RefMatchesDef routine is used on one case - to
match assembly refs against a list of friend-access
assembly defs. So it's not quite dead code though
it probably should be.)

Merge pull request #10904 from BruceForstall/ArmMergeCodegenFuncs

Merge similar arm32/arm64 codegen functions

[x86/Linux] Funclet-based synchronization (#10791)

* [x86/Linux] Funclet-based synchronization

* Fix x86/Windows build error

* Revise per feedback

* Fix format error

Merge pull request #10852 from mskvortsov/ryujit-arm32-ldst-offset

[RyuJIT/ARM32] Encodable offsets for ld/st

Format code.

Update CoreClr to preview1-25212-04 (#10879)

Fix resource string name

Delete moved files

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

Change default clang version to clang3.9 in cross build (#10889)

If we use clang3.9 version, we will use O3 option for optimization.
Otherwise, use O1 option as before.

Fix build badges in OfficialAndDailyBuilds.md

- None of the badge links were correct, resulting in every badge always showing
as "Building" regardless of actual status
- The OSX badge links would 404.
- We were still using shields.io badges, which have been unreliable.

I've fixed all of these.

RyuJIT/ARM32: encodable offsets for ld/st

Remove unused function in BasicBlock

lastTopLevelStmt() is declared in BasicBlock,
but not defined and unused throughout JIT.

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

Rename overloaded "table" to "store" for VM side concept.

Fix CI tizen build error (#10909)

Fix CI tizen build error caused by chmod

modify lastAvAddress when adjusting context for Stub

Hide knowledge of handle table buckets and home heaps from VM side.

Delete Ref_ContainHandle and ContainsOBJECTHANDLE.

Add GetNewHandleTable and GetGlobalHandleTable to interface.

Remove GetHandleTableForHandle and add CreateDuplicateHandle.

Replace GetHandleTableContext with GetHandleContext.

Merge pull request #10882 from adityamandaleeka/double_init_appdomain_creation

Remove double initialization of variables in app domain creation

Merge pull request #10877 from adityamandaleeka/handle_table_local_gc_nullchecks

[Local GC] Move handle null checking code to VM side

Move files to shared CoreLib partition (#10885)

Merge pull request #10816 from adityamandaleeka/handle_table_local_gc_init

[Local GC] Move handle destruction code to IGCHandleTable

Merge pull request #10907 from wtgodbe/BuildTypeReporting2

Pass null instead of empty for PB_BuildType

Pass null instead of empty for PB_BuildType

Merge pull request #10906 from wtgodbe/BuildTypeReporting

Pass PB_BuildType as a Pipeline ReportingParameter

Pass PB_BuildType as a Pipeline ReportingParameter

Fix a number of reordering issues in shift decomposition.

Shift decompostition contained a fair number of IR rewrites that were
unintentionally reordering nodes. This change fixes these instances s.t.
they no longer reorder code.

Merge pull request #10899 from wtgodbe/PipelineReporting

Add last set of reportingParameters to pipelines.json

Merge similar arm32/arm64 codegen functions

Add last set of reportingParameters to pipelines.json

Merge pull request #10867 from AndyAyersMS/FixSingleDefBug

Jit: fix issue with single-def type propagation

CoreFx #15622 Dictionary<TKey, TValue>.Remove(TKey, out TValue) (#10203)

Merge pull request #10880 from BruceForstall/ArmCommon

Create common files for ARM32 and ARM64 common code

Merge pull request #10876 from pgavlin/VSO406156

Fix a few errors in cast decomposition.

Move a few more types from CoreFX to CoreLib (#10893)

Reducing shards between CoreCLR and CoreRT shards

Merge pull request #10894 from dotnet/Add_Build_Type

Add some BuildType and Subtype info to CoreCLR build pipeline definition

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

Merge nmirror to master

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

Create common files for ARM32 and ARM64 common code

Create codegenarmarch.cpp, lowerarmarch.cpp, lsraarmarch.cpp.

For codegenarmarch.cpp, I copied over a bunch of functions that
are almost identical between arm and arm64. I left the functions
intact (didn't change anything), so created one for ARM32 and
one for ARM64, unless they were truly identical. A follow-up
will be to actually merge the functions.

For lowerarmarch.cpp and lsraarmarch.cpp, I did the function
merge for nearly (or completely) identical functions, since
there were so many that were so similar.

Add some BuildType and Subtype info to CoreCLR build pipeline definition

Merge pull request #10886 from hseok-oh/ci/fix_ci_error

Fix CI test error: permission error

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

Merge nmirror to master

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

Remove sudo in netci.groovy

Overwrite permission in coreoverlay generation

fix runtest.sh to overwrite corerun's permission in coreoverlay generation

Fix netci.groovy

Fix netci.groovy instead of test script to avoid rebase

Fix CI test error: permission error

Fix CoreCLR/CoreFX copy order: permission problem

Do not spill eeStack after ldtoken opcode. (#10215)

Do not spill eeStack if the last opcode was ldtoken.

Remove double initialization of variables in app domain creation.

Fix some static methods on ThreadPool to wait for initialization (#10869)

Fix for part of #10521

Merge pull request #10782 from parjong/fix/x86_funclet_unwinding

[x86/Linux] Use GCInfo for funclet unwinding

Move handle null checks to VM side.

Update CoreClr to preview1-25210-04 (#10862)

Merge pull request #10871 from pgavlin/PipelineCompatJit

Package compatjit.dll in pipeline builds.

Merge pull request #10866 from pgavlin/VSO405852

Fix VSO bug 405852.

Fix a few errors in cast decomposition.

The cursor was not guaranteed to be in the correct location after cast
decomposition due to the location in which the high result was typically
placed. This mispositioning can cause re-decomposition, which can lead
to asserts e.g. when encountering a GT_LONG node that was left behind by
a previously-decomposed node. This change ensures that the high result
immediately follows the cast node so that the cursor will be in the
correct location following decomposition of the cast.

empty array (#10841)

Make EmptyArray private in favor of Array.Empty<T>

Merge pull request #10814 from rartemev/ilasm_reference_resolver_fix

Added 'netstandard' corelib name into ilasm reference resolver

Merge pull request #10808 from sdmaclea/PR-ARM64-enable-initblk-unroll

[Arm64] enable initblk loop unrolling

Extract genInterruptible setting as a separate region

Use hasPartialArgInfo instead of isPartialArgInfo

add test with deep execution tree without stores. (#10835)

Add a test that should fail with stack overflow if the compiler doesn't spill trees from time to time.

Package compatjit.dll in pipeline builds.

Just what it says on the tin.

Format code.

Move handle destruction code to IGCHandleTable.

Conflicts:
src/vm/gchandletableutilities.h

Fix VSO bug 405852.

This bug was due to a mismatch between the semantics of the FP modulus
operator as specified by ECMA-335 and as implemented by value numbering.

ECMA-335 requires the following behavior for FP modulus:
- If the divisor is 0 or the dividend is not finite, the result is NaN
- If the divisor is not finite and is not NaN, the result is the
  dividend.
Value numbering, however, simply used `fmod`, which does not have these
semantics. This change implements the required semantics in VN.

Add back UnmanagedType.LPUTF8Str (#10859)

CoreLib Build System portability updates (#10839)

Updates to corelib project file for target specific includes.  Intended
to ease maintainability of existing targets and clarify necessary work
for new targets.

Fix contract to allow call into GarbageCollect() to throw (#10856)

I did not intend to mark this function as NOTHROW, and there is no reason to expect NOTHROW here. Other functions that call into GarbageCollect() allow THROWS.

Merge pull request #10823 from adityamandaleeka/move_variable_handle_creation

Refactor variable strength handle creation

Add empty System.Runtime.CompilerServices.RuntimeFeature.IsSupported() API (#10779)

Added netstandard corelib into ilasm reference resolver

Fix perf penalty for creating dictionaries the easy way. (#10855)

We went to the trouble of mitigating the perf hit
introduced by using randomized string hashing.

I assume that said perf benefit wasn't meant to be denied
to the 90% case where people use the nullary
constructor to create the Dictionary...