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>

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.

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

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.

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.

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

[Arm64] Cleanup/fix InitBlkUnroll

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

[Arm64] enable initblk loop unrolling

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.