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.

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)

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

Update CoreFx to preview1-25210-01 (#10850)

Implement slicing APIs for ArraySegment<T> (#9926)

Incorrect parameter order found by PVS-Studio (#10497)

* Incorrect parameter order found by PVS-Studio
I used PVS-Studio static analyzer to check this project. I would like to suggest a variant of the way to fix the error, detected with the help of V3066 diagnostic https://www.viva64.com/en/w/V3066/

Get rid of "#if FEATURE_RANDOMIZED_STRING_HASHING" in shared partition. (#10851)

Since CoreCLR defines it (for now) and CoreRT doesn't, let's not have
"#if's" on this in shared code. It just creates confusion.

The only ones are in StringComparer.cs and fortunately, they are quite
expendable as...

1. It's only purpose is serialize Hashtables and Dictionaries in
   a way that CLR 1.1 can deserialize them. We don't care about that in
   .NET Core.

2. In CoreCLR as of today, this infrastructure is not changing the serialization
   blob one iota since all the GetEqualityComparerForSerialization() methods
   return "this."

So this removes the appearance of downgrading the comparers to non-randomized
versions for serialization purposes (and yes, we could go on and remove
all the other "let's do it for Everett and RTM" stuff in Hashtable.cs, but
I want to keep this PR on topic. Hashtable.cs is not going to be imported
into CoreRT so it's low priority for me.)

Updating CODE_OWNERS.TXT (#10834)

Added Nuget package implementation owners

Forbid lea without index and base. (#10771)

Delete FEATURE_READYTORUN define for CoreLib build (#10840)

This define does not have much value for the CoreLib build, and it was missing on arm64.
Also deleted dead props and targets files left over from the legacy build

[x86/Linux] Set ResumeEsp as Caller Sp when unwond to native frame (#10838)

[x86/Linux] 16-byte aligned ResolveWorkerChainLookupAsmStub (#10754)

* Fixed issue, when structures with many fields had wrong size in dwarf (#10797)

Move files to shared CoreLib partition (#10837)

Move variable handle creation.

[Local GC] Move knowledge of the free object method table outside of the GC (#10755)

* [Local GC] Move knowledge of the free object method table outside of the GC

* Revert "[Local GC] Move knowledge of the free object method table outside of the GC"

This reverts commit 02979e5eb12dd44edb2603e16411dd7d7ffc377d.

* Address code review feedback: leave ArrayBase alone, as it should be part of the binary contract between the EE and the GC

* Correct a bad merge

Move files to shared CoreLib partition (#10830)

Update VS2017 'Individual Components' install info (#10695)

A few small changes to make it easier to compare selections in the
installer UI against the list in the doc:

 - Re-order the list to match the order presented in the installer UI
 - Place the individual items under groupings matching the installer
 - Change casing to match installer UI

Delete reference to model.xml since it no longer exists (#10827)

Fix the broken link in contributing.md (#10828)

The List.cs link is broken in contributing.md. This PR updates the link to List.cs accordingly.

Fix #10822

Merge pull request #10776 from sdmaclea/PR-ARM64-CpBlkUnroll

[Arm64] Enable cpblk loop unrolling

Merge pull request #10736 from sdmaclea/PR-ARM64-GC-genLeaInstruction

[Arm64] Fix GC Hole in genLeaInstruction

Fix corerun issue when loaded from PATH (#10745)

Use a monotonic clock for test timing (#10812)

Fixes #8348

Handle SIGTERM on OSX (#10723)

* Handle SIGTERM on OSX

Fixes #7394

* Address feedback

Exit gracefully when running ildasm -nat against unsupported ni file. (#10821)

ildasm -nat only works for fragile native images (ie CoreLib only by default).
Running ildasm -nat against unsupported native images would cause a segmentation fault. This PR checks pNativeHeader->Signature == CORCOMPILE_SIGNATURE and end the program with a meaningful error message.

fix #10247

Merge pull request #10811 from BruceForstall/RemoveRELOC_SUPPORT

Remove RELOC_SUPPORT define

[Local GC] Fix some interface violations (#10817)

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

Merge nmirror to master

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

Find & replace Exception.SetErrorCode with Exception.HResult setter (#10810)

Merge pull request #10757 from adityamandaleeka/handle_table_local_gc_init

[Local GC] Move handle creation to IGCHandleTable

Move OBJECTHANDLE definition to gcinterface.h

Add creation of special handle types to IGCHandleTable.

Add System.Runtime.CompilerServices.ReadOnlyAttribute (#10777)

Temporarily disable symweb indexing (#10807)

Remove RELOC_SUPPORT define

It's always defined, is always expected to be defined, and the build
doesn't work without it.

Also remove unused `SECURITY_CHECK` and `VERIFY_IMPORTER` defines.

Strip out unused reg code. (#10741)

* Strip out unused reg code.

Removing the unused code to better be able to audit our CoreLib usage and to prepare for targeted code sharing with CoreFX. Wraps key usages in using statements.

* Fix static init and remove Contract

Merge pull request #10806 from wtgodbe/portableMac

Publish portable OSX binaries into separate container in pipeline

Delete moved file

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

Merge nmirror to master

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

Using AsSpan since we no longer have slice on string. Renaming Tests. (#10758)

Merge pull request #10799 from sdmaclea/PR-Fix-lvRegister-assert

Correct assert w.r.t meaning of lvRegister

Publish portable OSX binaries into separate container in pipeline

Move public ThreadAbortException back to CoreLib (#10801)

Update CoreClr, CoreFx to preview1-25207-02, preview1-25205-01, respectively (#10787)

Address #10799 comments

Merge pull request #10656 from hseok-oh/ryujit/fix_10654

[RyuJIT/ARM32] [ReadyToRun] Fix target register for invocation to Thunk

Merge pull request #10781 from rartemev/ilasm_path_length_fix

Changes for #9757 issue. Handle long parameters in the right way.

Merge pull request #10704 from wtgodbe/BuildDefs

Initial commit of checked in build definitions for CoreCLR

[Arm64] Use ldp/stp in CpBlkUnroll

Merge pull request #10798 from adiaaida/fixjit32Perf

Fix jit32 Perf testing

Delete duplicate files and fix build breaks

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

Merge nmirror to master

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

Address review feedback

Correct assert w.r.t meaning of lvRegister

Per recomendation of Pat Gavlin in #9898

Fixes #9898

Fix jit32 Perf testing

In the CI, we were moving compatjit.dll into the Products directory
instead of the core_root directory, so the jit32 compatjit.dll was not
being picked up by the build. This change moves the nuget logic to before
we run runtest.cmd generatelayoutonly, so the jit32 compatjit.dll will be
moved to the core_root directory along with everything else.

Handle unsupported varargs tests on Unix gracefully. (#10794)

Some of mcc_i* tests caused segmentation faults on Unix. This commit make these tests exit by throwing a System.EntryPointNotFoundException exception instead of causing a segmentation fault.

Fix #9530

Bring back DefaultDllImportSearchPathsAttribute (#10786)

Add reporting swap file size to GlobalMemoryStatusEx on Unix (#10700)

The swap file size reporting is added using the Linux, OSX and FreeBSD
means. Also a new PAL test was added to exercise the function.

Delete duplicate files and fix build breaks

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

Merge nmirror to master

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

Call custom parameterless constructor on structs through Activator (#10778)

Fixes #6843
- Disabled caching struct types that have custom parameterless constructors in `ActivatorCache`
- Removed some things relevant to security, which don't apply to CoreCLR

remove fedora 23 (#10725)

* remove fedora 23

* Fedora rename

NS2.0 Prepare FileNotFoundException for move to shared partition. (#10775)

* String->string - get rid of the big comment block.

* Fields into autoprops.

* FullName => ToString() to avoid MME problems on N.

* Remove SecurityException guard around FusionLog

* Split off the parts we don't want to share.

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

Merge nmirror to master

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

Merge pull request #10784 from briansull/ibc-shutdown-fix

Fix shutdown A/V issue when IBC logging with the Music Store app

Devirtualizable default encoding (#10783)

don't switch to COOP GC when logging CCW ref count (#10756)

Use _TARGET_ARMARCH_

Chagne _TARGET_ARM_ and _TARGET_ARM64_ to _TARGET_ARMARCH_

Fix shutdown A/V issue when IBC logging with the Music Store app

This app continues to run managed code on other threads after the Main thread exits and enters shutdown.
We now block IBC logging during the time that we are writing out the IBC data.

Update ILLink build scripts

Update ILLINK build commands to reflect current CoreBuild of ILLinker.
Also remove the line that sets DoNotEmbedDescriptors,
since it is no longer necessary.

Correct fix for long ilasm params

Fix formating

Merge pull request #10768 from mskvortsov/ryujit-arm32-lea

[RyuJIT/ARM32] Update genLeaInstruction

[Arm64] Enable cpblk loop unrolling

Add views for the jitstress and illink folders (#10773)

Initial commit of checked in build definitions for CoreCLR

Make Encoding.Default return UTF8 without BOM emission (#10747)

* Make Encoding.Default return UTF8 without BOM emission

* Remove SL reference in the code

* Remove the static cache

Merge pull request #10697 from sdmaclea/PR-FIX-RUNTEST

Do not use declare -A

Merge pull request #9086 from mikedn/simd-cast

Add Vector.op_Explicit tests

Revert EncodingForwarder for the sake of the Perf (#10740)

* Revert EncodingForwarder for the sake of the Perf

* Add missing optimization

[x86/Linux] PAL test in CI build (#10688)

* [x86/Linux] PAL test in CI build

Add PAL test in CI x86/Linux build
Change docker image: including rootfs

* Fix netci.groovy

Add PAL test's result

[ARM/CI] Update image for armel Tizen CI (#10730)

Update image for armel CI to sync with Tizen rootfs

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

RyuJIT/ARM32: update GT_LEA codegen

Merge pull request #10742 from BruceForstall/RemoveUnusedDefine

Remove unused PREDICT_REG_RER_INDIRECT_PARAM define

Merge pull request #10752 from BruceForstall/FixBBFformatting

Clean up block.h formatting

Clean up and prepare FileLoadException.cs for move to Shared Partition. (#10764)

* String => string

* Replace fields with autoprops.

* Turn Type.FullName call to Type.ToString() in ToString().

Avoids MissingMetadataExceptions on CoreRT.

Assuming metadata exists, ToString() and FullName()
still generates the same string except in the weird
case where an app subclasses FileLoadException
with a generic exception type - in which case,
the ToString() version is actually more readable.

* Get rid of useless try-catch around FusionLog calls.

FusionLog is now just a normal property and an answer
to a trivia question - it won't throw SecurityException.

* Get rid of low-value SetMessageField() middleman.

* Split off the stuff we don't want to share.

* The complementary carwash...

* Don't conditionalize FusionLog serialization.

Modify THUNK_PARAM generated in SaveWork

- Use parameter r4 to pass Indirection from code generated by R2R
- Define REG_R2R_INDIRECT_PARAM in ARM32 to merge with ARM64 routine

[x86/Linux] Clean up ZapUnwindData (#10750)

Add Vector.op_Explicit tests

[Local GC] Fix a bad merge (#10748)

Move handle table stuff out of gcheaputilities.

replace the linear time algorithm from lclvars with the constant one. (#10401)

add new block flag DOMINATED_BY_NOT_NORMAL_ENTRY

It allows to answer the question: "Is block dominated by non normal
entry" - in O(1).

gtLdftnResolvedToken is no longer necessary, because token is saved on the stack. (#10738)

The was an error in the assert because I checked pointers that were
obvious different.
The actual values could not be different because they are created from
the same token in the same place (importer.cpp DO_LDFTN:
impMethodPointer saved resolvedToken as gtLdftnResolvedToken ,
CORINFO_RESOLVED_TOKEN* heapToken = impAllocateToken(resolvedToken);
impPushOnStack(op1, typeInfo(heapToken)); saved the token on the stack.

[x86/Linux] 16-byte aligned HelperMethodFrameRestoreState (#10720)

* [x86/Linux] 16-byte aligned HelperMethodFrameRestoreState

* Use STACK_ALIGN_PADDING and update comment

Clean up block.h formatting

Hide a few things from clang-format