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.

Enable R2R jitstress jobs for x86

Allow conversions between enum and underlying type

Fixes https://github.com/dotnet/corefx/issues/13816

Disallow conversions between Object* and Object in Array.Copy

Fixes https://github.com/dotnet/coreclr/issues/10646

Unify fetching LoadContext for Dynamic Assemblies (#11264)

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

These changes remove support for the x86 compat JIT from the build, the
runtime, and the various perf/test scripts.

Fixes #10733, #10734.

[Arm64] Use ldp for mismatched GC types in genPutArgStk

[Arm64] Support two GC attributes in pair forms

Also use in genCodeForCpObj

[Arm64] Clean up genPutArgStk (#11239)

* [Arm64] Clean up genPutArgStk

Use ldp/stp in more case
Simplify code by eliminating most addrReg conflict cases

* Fix formatting

Turn on unwind information at informational level rather than Info level. (#11257)

This is bascically a bug fix.   Quite a while ago we wanted to be able to collect stacks (which needed JIT names, and unwind information)
without collecting all verbose CLR events (in particular the GC allocation Tick events).   We did this in most places but missed this one.

Remove unused TreeNodeInfo::isHelperCallWithKills field (#11259)

Remove unused TreeNodeInfo::isHelperCallWithKills field

Merge pull request #11224 from parjong/fix/arm_unaligned_ldobj

[ARM/Linux] Honor unaligned prefix for ldobj instruction

[WIP] Enable binding to CoreLib as NI image (#11040)

Enable binding to CoreLib as NI image

Merge pull request #11255 from mskvortsov/FixPutArgReg

[RyuJIT/ARM32] Update putarg_reg codegen

Merge pull request #11258 from mskvortsov/FixConsumeReporting

Do not report out-of-order consuming if a node wasn't consumed at all

Fix a double-free encountered when failing to initialize a handle table bucket (#11244)

* Fix a double-free encountered when failing to initialize a handle table bucket

* Don't free m_bucket in the destructor of HandleTableBucketHolder

Fix unittest `LPSTRTest` (#11096)

* Fix unittest `LPSTRTest`

The test case Interop.StringMarshalling.LPSTR.LPSTRTest
always returned exit code 100(pass) even if it actually failed.
This implements `ExitTest()` in the same manner with `LPTSTRTest`.

* Fix unittest `LPSTRTest` native

Merge pull request #11219 from sdmaclea/PR-ARM64-CpObj-Use-LDP-STP

[Arm64] Use ldp/stp in genCodeForCpObj()

Merge pull request #11243 from sdmaclea/PR-ARM64-LDP-ALLOW-GCREFS

[Arm64] Update emitOutputInstr to support GC refs in ldp forms

Do not report out-of-order consuming if node wasn't consumed at all

[x86/Linux] Disable USE_CALLER_SP_IN_FUNCLET in EH (#11247)

RyuJIT/ARM32: Update GT_PUTARG_REG codegen.

[Arm64/Unix] Revise JIT_MemCpy (#11143)

* [Arm64/Unix] Revise JIT_MemCpy

Use ldp/stp
Correctly handle short copy lengths
Simplify code & pseudo code
Use uint*_t to make pseudo code more readable

Fix building with only VS2017 installed. (#11182)

The fix was originally made in https://github.com/dotnet/corefx/pull/18700

Merge pull request #11240 from wtgodbe/SourceBranch

Use SourceBranch from pipebuild to define Helix Branch

Fix to respect readonly prefix for multidimensional array Address method (#11245)

Fixes #9414
- Added missing 'else' back from regression in https://github.com/dotnet/coreclr/commit/97b4ff0b438261ba11b357008630076054a6f25d#diff-edc46b80f57431489a82311948a8234dL6424, which has the effect of passing null to the Address method's instParam to indicate the readonly prefix in order to bypass the type check

Fix filename display in exception messages (#11238)

Update CoreClr, CoreFx to preview2-25226-02, preview2-25226-02, respectively (#11127)

Delete moved file

Move DllNotFoundException, EntryPointNotFoundException to CoreLib (#3415)

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

Fix timer callback time drift (#11220)

Fixes #6408:
- The change to S.T.Timer is the actual fix, but fixed the timer firing on the thread pool timer thread similarly as well

Fix comment

[Arm64] Update emitOutputInstr to support GC refs in ldp forms

Merge pull request #11235 from pgavlin/VSO424019

Require a frame pointer in methods with throw blocks for sources that…

Use SourceBranch from pipebuild to define Helix Branch