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.

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

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

Fix and re-enable some GC finalizer tests (#11216)

* Fix and re-enable some GC finalizer tests

* Make the Usage test a little more robust

[Arm64] CpObj ldp/stp review feedback

Rework asserts
Add comments
Format

Merge pull request #11178 from sbomer/ilrt_factoring

Refactor ildasm round trip logic

Require a frame pointer in methods with throw blocks for sources that have mismatched stack depths.

This requirement effectively disables double-aligned frames (which are
effectively a special case of ESP frames). Without this change it is
possible for the JIT to generate a throw block with sources that have
mismatched offsets, which can lead to GC holes.

Fixes VSO 424019.

Merge pull request #11218 from AndyAyersMS/FixReturnTypeMismatch

JIT: fix return type mismatch inlining block

Lookup System.Runtime and mscorlib references by Assembly name

Rather than by alias.

Fixes #10595

Merge pull request #11229 from wtgodbe/CrossgenFix

Exit on failure of HelixPrep.proj

Exit on failure of HelixPrep.proj

Refactor ildasm round trip logic

This change factors the Ildasm round trip logic so that it should not
require a separate test build. The ildasmroundtrip flag is no longer
passed to build-test.cmd; instead the wrapper scripts are always
generated with a check for an environment variable that can be set from
runtests.

Also remove separate test build for ilrt in the ci scripts

Enable build with clang 4.0 (#11226)

This change enables build with clang 4.0 and fixes a bunch of new errors that
the stricter compiler was reporting.

Fix thread wait to be resilient to real time clock changes on Unix (#11213)

This change fixes a problem that happens when a thread uses a timed wait
and the real time clock change in the system (e.g. due to the user action,
time server sync, leap second, etc). In such case, the wait may end up waiting
for shorter or longer period of time.
It fixes the issue for Thread.Sleep and timed wait on a process local synchronization
primitives by using CLOCK_MONOTONIC instead of CLOCK_REALTIME for waiting on
cond vars.
Unfortunately, the waits on cross process synchronication primitives like
named mutexes always use CLOCK_REALTIME, so this fix works on process local primitives
only.

[x86/Linux] Stack alignment check in Hijack Helpers (#11221)

[x86/Linux] Use appropriate FCALL macro for DateMarshaler__ConvertToNative (#11194)

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

Merge nmirror to master

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

Comment explaining use of FlushProcessWriteBuffers in StompWriteBarrier. (#10924)

Fix few clang analyzer warnings (#11185)

This change fixes two issues reported by the clang analyzer. In both cases, uninitialized
variable value was used in an error code path.

[ARM/Linux] Honor unaligned prefix for ldobj instruction

Merge pull request #11107 from BruceForstall/NumSuccImprovement

Num succ improvement

Merge pull request #11204 from BruceForstall/FixArmAltjitGcinfo

Fix GCInfo library used by ARM/ARM64 protononjit builds

Merge pull request #11215 from wtgodbe/ActuallyDoCrossgen

Set RunCrossGen=true for non-Windows R2R runs

[Arm64] Use ldp/stp in genCodeForCpObj()

Move DynamicHelperFrame::UpdateRegDisplay into cgenx86.cpp (#11193)

* Move DynamicHelperFrame::UpdateRegDisplay into cgenx86.cpp

* Fix x86/Windows link error

JIT: fix return type mismatch inlining block

If caller passes a directly substitutable expression whose
type is compatible with but not the same as the callee argument type,
and that argument is the callee's return value, the jit may
reject inlining the method because of a return type mismatch.

Fix is to retype the expressions we use in the callee's body
as the callee's type.

Closes #11211.

Set RunCrossGen=true for non-Windows R2R runs

Merge pull request #11134 from pgavlin/VSO406163

Fix fgNewBBinRegion when inserting into filters.

Merge pull request #11110 from rartemev/issue_9880

Made behavior to conform to ECMA for 'tail.call, pop, ret' case

[RyuJIT/ARM32] Move genJumpKindsForTree into common part (#11103)

* Move genJumpKindsForTree() into codegenarmarch.cpp.

* Edit genJumpKindsForTree().

* Introduce genCodeForJumpTrue() for both ArmArch and XArch.

* Fix GT_JTRUE codegen for ARM32 by using genCodeForJumpTrue().

Fix GCInfo library used by ARM/ARM64 protononjit builds

Otherwise, using the normal GCInfo library, the altjits will
assert creating GC info.

Create new "cross-target" builds of the GCInfo library.

Merge pull request #11118 from rartemev/issue_10094

Fixed netci.groove scripts to make CentOS and Ubuntu R2R jitstressregs builds worked.

Made behavior to conform to ECMA for 'tail.call, pop, ret' case

Merge pull request #11206 from wtgodbe/ContainerName

Update R2R Container Names

Update R2R Container Names

Testing. DO NOT MERGE ME

Allow FEATURE_INTERPRETER to be enabled by command-line option (#11188)

* Allow FEATURE_INTERPRETER to be enabled by command-line option

* Set FEATURE_INTERPRETER as 0

Merge pull request #11187 from wtgodbe/NonWindowsR2R

Enable running R2R tests in Helix on non-Windows

Enable running R2R tests in Helix on non-Windows

Merge pull request #11203 from mskvortsov/FixLclFld

[RyuJIT/ARM32] Update lclFld codegen