Merge pull request #11361 from mikedn/imp-strlen-except

Add missing GTF_EXCEPT when importing string length intrinsic

Fix incorrect metadata on SOS.NETCore.dll (#11395) again (#11415)

There was a simple msbuild property.

Issue #11381

Add missing GTF_EXCEPT when importing string length intrinsic

Move UnmanagedMemoryStream and related types to shared CoreLib partition (#11409)

Add new test for COMPlus_TailcallStress=1 bug, currently disabled (#11410)

Add new test for COMPlus_TailcallStress=1 bug, currently disabled

Merge pull request #11401 from wtgodbe/DeleteNis

Rename all xxx.ni.dll's to xxx.dll for Helix Crossgen runs

Update contributing-workflow.md

Refactor arm32/arm64 per-node codegen into functions (#11377)

These new functions can sometimes be shared, as they are identical or
virtually so.

In addition, the driver function genCodeForTreeNode() has been
made common.

Rename all xxx.ni.dll's to xxx.dll for Helix Crossgen runs

Fix incorrect metadata on SOS.NETCore.dll (#11395)

Issue #11381

Update first-class-structs.md

Update first-class-structs.md

Remove UnmanagedMemoryStream overflow checks. (#11376)

* Remove UnmanagedMemoryStream overflow checks.
* Catch negative UMS PositionPointers
* Switch UMS PositionPointer exception order

Fix ProjectN build break (dotnet/corert#3517)

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

System.IO.Path.GetPathRoot string.Empty or whitespace character string should throw ArgumentException (#11387)

* GetPathRoot string.Empty and string whitespace throws ArgumentException

* throw in next line

* Refined method description

* Unix impl adjusted

* Indentation

* pr feedback

* Delete Path.Unix.cs

* Adding param to ArgumentException

Reenable MapViewOfFile PAL test1 (#11393)

This change reenables the test that was having problems on systems when the /tmp
is created on tmpfs file system which doesn't support direct access (unbuffered).
The fix is to explicitly use /var/tmp instead (and /data/local/tmp/ on Android).

Remove stale section on managed debugging. (#11392)

Merge pull request #11370 from CarolEidt/ILOffsetDump

Dump GT_IL_OFFSET value in hex

Document Linux libnuma dependency (#11367)

Disable symbol package MyGet publish on release/* (#11372)

Delete moved file

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

Merge nmirror to master

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

Update CoreClr, CoreFx to preview2-25303-03, preview2-25303-04, respectively (#11354)

Createdump fixes and cleanup. (#11368)

Fixed "with heap" option to actually include all the heaps. The way /proc/$pid/maps
was parsed missed most of the RW data/heap regions.

Added -u/--full full core dump options.

Removed the useless m_memStatus variable in enummem.cpp.

[Arm64/Unix] Revise JIT_MemSet (#11217)

* [Arm64/Unix] Revise JIT_MemSet

Use DC ZVA
Use stp
Correctly handle short set lengths
Simplify code & pseudo code
Use uint*_t to make pseudo code more readable

Adding System\Runtime\CompilerServices\IsByRefLikeAttribute type (#11369)

described in: https://github.com/dotnet/corefx/issues/19212

Fix some static analysis warnings

Most fixes are just addressing the use of (signed) enum-typed variables
as array indices.  Casting to unsigned allows us to cheaply include the
lower bound checks in the existing upper bound checks.  I would prefer
to force the underlying types of enumerations to be unsigned, but that
is a relatively new C++ feature and could have broader consequences than
I want to risk at this point.

The one other fix to asmparse.cpp just suppresses a warning that, while
technically valid, is not causing a real problem.  Perhaps this will get
better if/when #2305 is addressed.

Dump GT_IL_OFFSET value in hex

Remove COMCoverage::nativeCoverBlock (#11364)

Remove this unused function and the corresponding references in
ecalllist.h

Fix Binder NI to IL fallback (#11341)

* Fix Binder NI to IL fallback

Always try NI first

* Fix Windows warning as error

* [Arm64/Unix] Revise PR per feedback

* More cleanup

Place precompiled SPC.dll in native assets folder. (#11359)

Place precompiled SPC.dll in native assets folder.

Fix load from (#11342)

* Add LoadFrom dependencies to the LoadFrom list

* Add a comment

Implement Type.IsTypeDefinition property on CoreCLR (#11355)

This api was just approved.

https://github.com/dotnet/corefx/issues/17345

Revert "[ARM32/Linux] Copy tests.zip only for CI test and use x86 unittest"

This reverts commit 2ec8b5c2cc6d68ca02eb743e679997a8f11c90aa.

Merge pull request #11362 from DrewScoggins/FixSmoketest

Fix triggering of perf correctness runs

Fix triggering of perf correctness runs

When I made the change to have the correctness runs fire in a PR without
the trigger phrase I accidentally did it for all PR's not just for the
smoketest runs.  This change fixes that.

Merge pull request #11180 from DrewScoggins/AddCorrectnessRuns

Add smoketest for performance tests

Add smoketest for performance tests

This adds a run that will fire on the set of Azure VMs and will only run
for one interation.  This along with the changes to fail a performance
run if a test fails will allow us to start gating checkins on the
performance tests still running.

Merge pull request #11340 from ericstj/fixCrossTargetFolder

Fix folder name for cross-targeting assets

[ARM32/Linux] Copy tests.zip only for CI test and use x86 unittest

- Copy tests.zip only from x86 Windows build for CI test
- Use x86 unittest instead of x64: 32bit architecture

Enable Marshal.SecureStringToBSTR and Marshal.ZeroFreeBSTR on Unix (#11234)

Merge pull request #11336 from wtgodbe/TestingOnLinux

Update Unix test instructions

Update Unix test instructions

Fix folder name for cross-targeting assets

Enable LoadFrom to pickup assemblies next to requesting assembly (#11333)

* Enable LoadFrom to pickup assemblies from next to requestor

* Make early out explicit

Merge pull request #11329 from BruceForstall/RefactorArmCodegen

Refactor arm32/arm64 codegen

Update CoreFx to preview2-25302-03 (#11331)

Surgically remove unsupported tailcall cases

Rather than disabling tailcall tests entirely, surgically remove the
failing cases so the remaining cases can be enabled.

Update CoreClr, CoreFx to preview2-25302-02, preview2-25302-01, respectively (#11326)

Added missing icu headers for Android (#11325)

urename.h and ustring.h contain the u_stncpy and its definition.
If they are not in the list of icushim.h headers, Android build fails.

Fix assert

Fix rgTail corruption in rare scavenging cases [Desktop port]. (#11327)

Partial revert "Move a few more files to shared CoreLib partition" (#11328)

Revert change from Hashtable to Dictionary

Formatting

Refactor GT_NEG/GT_NOT

Refactor compares

Refactor GT_PUTARG_REG

Refactor GT_SWAP

Refactor GT_RETURNTRAP

Refactor GT_STOREIND

Refactor GT_LCL_VAR and GT_STORE_LCL_VAR

Refactor GT_STORE_LCL_FLD

Refactor ARM/ARM64 GT_LCL_FLD codegen

Merge pull request #11323 from dotnet-bot/master-UpdateDependencies

Update CoreClr to preview2-25301-03 (master)

Merge pull request #11067 from sjsinju/model_kill_set

[WIP][RyuJIT/ARM32] Implement the model kill for CORINFO_HELP_ASSGN_BYREF helper

Update CoreClr to preview2-25301-03

Merge pull request #11304 from pgavlin/GH11273

Fix #11273 by adjusting the fix for VSO 406163.

Move a few more files to shared CoreLib partition

[ARM] Update rootfs for Tizen armel (#11202)

* [ARM] Update cross rootfs for armel Tizen

Update cross rootfs for armel Tizen

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>
* [ARM] Update cmake file and CI for armel Tizen

Update CI and cmake file for armel Tizen, because rootfs for Tizen is updated.
And rootfs for Tizen is embedded in Docker image for CI as Ubuntu for arm does.

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

Fix current publish build warnings (#11317)

* Make VSTS Drop metadata distinct

* Remove PublishLogs artifact publish: no matches

Merge pull request #11309 from dotnet-bot/master-UpdateDependencies

Update CoreClr, CoreFx to preview2-25301-01, preview2-25301-02, respectively (master)

Fetch secondary handle only if primary is non-null

The GC is not tracking the secondary handle once primary gets cleared.

Fixes #11270

Avoid unnecessary out arguments in DependentHandle FCalls

Make the signatures closer to what they are in CoreRT

Merge pull request #11262 from pgavlin/NoCompatJit

Remove support for the x86 compat JIT from .NET Core.

Format code and fix a comment.

Add symbol publish (index/archive) build leg (#11300)

* Add symbol publish (index/archive) build leg

* Use BuildType property name in symbol publish leg

Merge pull request #11275 from CarolEidt/Fix11141

Avoid over-constrained delayed-use register case

Update CoreClr, CoreFx to preview2-25301-01, preview2-25301-02, respectively

Rename internal field of primitive type structs and make it private (#11312)

Reduce diffs with CoreRT and conform to coding conventions

Merge pull request #11297 from DrewScoggins/AddPerfRunInstr

Add instructions for running local perf tests

Merge pull request #11272 from wtgodbe/CrossgenMove

Ensure we run the Crossgened test executable in R2R tests

Bring back Marshal.Read/Write overloads that takes object (#11251)

* Bring back desktop version of Marshal.Read/Write APIs that takes object and add a test

Update CoreClr to preview2-25229-01 (#11303)

Update runtest.sh

Update if statement

Fix #11273 by adjusting the fix for VSO 406163.

Issue #11273 revealed an issue with the fix for VSO 406163: as of today,
the compiler models the incoming exception object in a handler by
looking for `GT_CATCH_ARG` in the first block of a handler and if found
marking the register that contains the exception object as holding a GC
pointer. Unfortunately, this breaks down under the transform that the
original fix (#11134) was performing for single-block filters. That
transformation was from this:

```
    filter start: BBN (BBJ_EHFILTERRET)
```

to this:

```
    filter start: BBN + 1 (BBJ_ALWAYS -> BBN)
                  BBN + 2 (jumpKind)
                  BBN     (BBJ_EHFILTERRET)
```

After this transform, `BBN` still contains the `GT_CATCH_ARG`, but the
register containing the exception object must be live through `BBN + 1`
as well as live in to `BBN`.

There are a number of possible solutions to this problem (e.g. modeling
the liveness of the exception object in some fashion or splitting the
single-block filter either after the catch arg or before the filter
ret), but many of them are complicated by the fact that this
transformation needs to operate on both HIR and LIR and must be be able
to run in a variety of locations, including after registers have been
allocated. Instead, this change takes the approach of ensuring that no
single-block filter regions exist on platforms that target the JIT32 GC
encoder by inserting a spill of the exception object to a local var in
its own block at the start of a filter.

Ensure we run the Crossgened test executable in R2R tests

Correctly enumerate loaded assemblies in AppDomain (#11294)

* Correctly enumerate loaded assemblies in AppDomain

* Remove unrequired attribute

Fix typo

Add instructions for running local perf tests

Update CoreClr, CoreFx to preview2-25228-02, preview2-25228-02, respectively (#11283)

Enable interpreter build (#11252)

Enable FEATURE_INTERPRETER build

Merge pull request #11268 from sdmaclea/PR-ARM64-two-gc-attributes

[Arm64] Support two GC attributes in pair forms

Avoid over-constrained delayed-use register case

When a node has a delayed-use register (either due to an RMW def or due to an internalRegDelayFree),
if its result has a fixed-use, don't propagate the register constraint to the def, because there's
no associated fixed-reference to ensure that it will not be used by the delay-free register.

Fix #11141

Fix R2R issue with virtual method resolving to non-generic base (#11289)

Merge pull request dotnet/corert#3446 from jkotas/nmirror-merge

Merge nmirror to master

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

Add assert

Remove comment

Update CoreClr, CoreFx to preview2-25227-03, preview2-25227-02, respectively (#11249)

Fix lsra memory consumption (#11233)

* Rename ClearD to OldStyleClearD

* create fast ClearD function.

The new ClearD reuses existing buffer and assumes that epoch wasn't
changed.

* Reuse existing buffer for the predSet in lsra.

* Use fast ClearD in LSRA.

* Use fast ClearD in liveness

* Fix the comment

* delete the test from issues

Fix issues exposed by building with MSVC and /permissive-. (#11269)

Using full-qualified name to declare members inside class is ill-formed and incorrectly allowed by MSVC.
MSVC now gives error when /permissive- is used.