Merge pull request #11207 from JosephTremoulet/PreserveVN

Preserve value numbers in gtSetEvalOrder

Enable GDBJIT feature in Tizen armel CI (#11708)

Tizen basically requires GDBJIT enabled libraries.
Enable GDBJIT feature to prevent related build fail issues in advance.

[GDBJIT] Fix incomplete type build error (#11706)

Update CoreFx to preview2-25319-01 (#11736)

Merge pull request #11724 from wtgodbe/FixJITCheckProjects

Fix & Re-enable JIT CheckProjects test

Merge pull request #11728 from wtgodbe/RemoveWarnings

Get rid of double-import of Buildversion.targets

Get rid of double-import of Buildversion.targets

Merge pull request #11697 from DrewScoggins/FixRestoreIssue

Fix Performance Runs

Fix & Re-enable JIT CheckProjects test

Fix Externally reported issue with are ETW logging messages (#11722)

Some of the string messages associated with our event are invalid in obvious ways.
External users found this during a review.   Trivial Fix.

Disable Musicstore from ILLink test

MusicStore is currently a very flaky scenario so for now I am disabling
the MusicStore test of ILLink.  We will add this back when we move this
scenario test over to its own leg.

pal/configure.cmake: add missing HAVE_SCHED_AFFINITY (#11711)

Enable pow for arm/armel in tryrun (#11703)

This will fix CoreFX System.Tests.MathFTests.Pow() failures

Use relevant define for resumable frame SP handling (and update ifdef condition) (#11253)

* Use relevant define instead of explicit _TARGET_XXX_

m_pvResumableFrameTargetSP-related implementations in the stack walker
are explicitly ifdefed by _TARGET_XXX_.

This commit introduces relevant define (RECORD_RESUMABLE_FRAME_SP) and
uses it to ifdef relevant implementations.

This commit does not introduce any behavior changes.

* Update ifdef condition

[x86/Linux] Port 'DacUnwindStackFrame' (#11666)

* [x86/Linux] Implement 'DacUnwindStackFrame'

* Update ContextPointers using ContextRecord

* An attempt to fix x86/Windows build error

[x86/Linux] Fix undefined references on SOS (#11701)

Finish deleting dead CAS code from CoreLib (#11436)

Fixes #9321 and deletes CleanupToDoList.cs

Delete unmanaged security implementation

Merge pull request #11690 from wtgodbe/IlTestsFix

Fix & re-enable failing Loader IL tests

The createdump utility needs RPATH (#11688)

To find the mscordaccore reference when run as a standalone utility. Works fine when launched from coreclr.

Issue #11687

Fix Performance Runs

This fixes the performance runs to work with the new 2.0 build tools
package that we just took.

Merge pull request #11660 from CarolEidt/LSRACleanup

LSRA Cleanup

merger takes MC file names as UTF16 names. (#11506)

There were problems with:
1. Arabick and Japanese symbols in CLR_SH;
2. Long MC names.

We didn't see these problem before, because we moved all MC files and
renamed them with short numbers names.
Now if we want to:
1. avoid  MC files copying;
2. save information that is encoded in this names for spmi diff between
different collections
we want this fix.

Fix & re-enable failing Loader IL tests

Move master to 2.1 Preview1 (#11662)

Merge pull request #11684 from pgavlin/FixPushdBalance

Remove a couple mistaken calls to popd.

Remove a couple mistaken calls to popd.

These calls were causing individual test runs to fail if other
directories were on the stack.

Fix CreateDump-related undefined reference on non-AMD64/Linux platforms (#11635)

Move some local declaration into exceptionhandling.cpp (#11668)

* Locally declare HandlerFn in exceptionhandling.cpp
* Move ClrUnwindEx forward declaration

Remove unused CLRVectoredExceptionHandlerShim forward declarations (#11667)

Switch multicast delegate stub on Windows x64 to use stubs-as-il (#11624)

Fixes #11611. The old hand generated assembly path did not work well for structs passed by reference.

Merge branch 'master' of https://github.com/dotnet/coreclr

correct link to Using Your Build

Merge pull request #11675 from dotnet/revert-11671-PerfTrigger

Revert "Make Perf Tests by Comment only"

Revert "Make Perf Tests by Comment only"

Merge pull request #11671 from dotnet/PerfTrigger

Make Perf Tests by Comment only

Make Perf Tests by Comment only

Merge pull request #11669 from dotnet/FailingTestList

Fix Formatting in TestsFailingOutsideOfWindows.txt

Fix Formatting in TestsFailingOutsideOfWindows.txt

Merge pull request #11620 from wtgodbe/BuildToolsUpdate

Update Coreclr to 2.0.0 buildtools

Disable opt-sensitive tests under tailcall stress and balance pushd. (#11658)

Just what it says on the tin.

Allow provider deletion to be deferred until after tracing is stopped. (#11651)

Update Coreclr to 2.0.0 buildtools

LSRA Cleanup

Remove an unused method, and clean up the comments for `checkLastUses`.

Update CoreClr to preview2-25316-03 (#11650)

Merge pull request #9354 from tannergooding/math

Adding perf tests for the single precision math functions in System.MathF.

Add LinkBench to BenchView

1) Add LinkBench to the xunit-perf benchmark runs, as a scenario test
2) Some improvements to run-xunit-perf to
   - Handle tests that perform size measurement
   - Parametrization of test group
   - Tests running multiple scenarios

CoreFX testing is currently disabled because it fails in the lab
due to a path length limitation.

Updated the JIT/Performance/CodeQuality/Math/Functions single-precision tests to have the correct expected results.

JIT/Performance/CodeQuality/Math/Functions tests to use the appropriate precision when printing failure messages.

Updating the JIT/Performance/CodeQuality/Math/Functions tests to target netcoreapp1.1 instead of netstandard1.4

Adding perf tests for the single precision math functions in System.MathF.

Disable tail call stress in GH_10780 if ZapDisable is enabled. (#11649)

* Disable tail call stress in GH_10780 if ZapDisable is enabled.

Tail call stress does not mix well with ZapDisable due to the issues
described in #11408.

Fixes #11648.

* Add a missing semicolon.

Merge pull request #11037 from jashook/arm64_test_update

Turn on tests which are now passing.

Update Corelib to use SetThreadErrorMode (#11625)

See #19738. Note that the enumerable is only used on Unix- but updated
it anyway. Conditioned the set/unset for ! Unix. SetErrorMode in the
PAL is a no-op, want to use the shared interop.

[x86/Linux] Remove GetFrameFromRedirectedStubStackFrame stub (#11638)

Merge pull request #11639 from brianrob/fix_corefx

Fix EventSource Test Breaks in CoreFX

Fix EventSource tests.

Move PopSEHRecords into excepx86.cpp (#11626)

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

Merge nmirror to master

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

Update CoreClr, CoreFx to preview2-25316-01, preview2-25316-01, respectively (#11622)

Remove an unnecessary binary file

Remove a binary file that was unintentionally checked in.

Turn on tests which are now passing.

This also adds metadata for all tests which are to be excluded
because they do not work with core.

Use full move for byte registers stores in jumps. (#11570)

* Use the 4-byte move for jump spilling.
The other types of spilling already use the proper move:
1) for lsra spilling it always use 4-byte move because we allocate 4-byte slots;
2) for other types it sets needsByteReg and lsra chooses correct register;

We do not apply the second approach to fixing this issue because jmp doesn't
have real uses, that can keep this requirement on. Also, it creates more strict
restrictions, that we need.

Write EventSource events with metadata to EventPipe (#11577)

* Fix EventPipe initialization.

* rebase code with master, add prototype of event source metadata

* fix define event

* add null ptr checking

* move WriteEvent metadata serialization to managed side

* Fix enabled keywords for COMPlus_PerformanceTracing=1.

* put parameter name into metadata, use WriteToBuffer

* Write event metadata to the file.

* allocate buffer in EventPipeEvent ctor

* prevent double free

* refactor code

* AddEvent params reorder

Fix issue #11574. (#11579)

This bug was yet another case of a transformation being performed during
remorphing that preserved a tree's value number without preserving its
associated semantics. This fix disables the problematic transformations
in morph.

Update License Info (#11617)

Throw PNSE in ThreadPoolBoundHandle.BindHandle on Unixes (#11613)

Throw PNSE in ThreadPoolBoundHandle.BindHandle on Unixes

Functional fix for https://github.com/dotnet/corefx/issues/19785

Update CoreClr, CoreFx to preview2-25315-02, preview2-25315-01, respectively (#11596)

Remove unused code from excepx86.cpp (#11602)

Allow coniguration of sampling rate. (#11595)

EventPipe: Prepend variable length field sizes (#11600)

* Write the size in bytes of the stack before the stack contents.

* Specify the size of the metadata payload explicitly in the file.

Fix minidump hang under shipping lldb 3.9.1 (#11591)

Issue: #11590

Update CoreClr, CoreFx to preview2-25313-04, preview2-25313-02, respectively (#11575)

Merge pull request #11593 from brianrob/eventsource_unix

Disable ETW calls on UNIX

Disable ETW calls on UNIX.

Fix String.IndexOf with empty string input (#11587)

Enable Symbolic Rundown for EventPipe (#11582)

Merge pull request #11584 from dotnet/revert-11500-from-tfs

Revert "Merge changes from TFS"

Revert "Merge changes from TFS"

Put the serialization marker under its own IFDEF. (#11568)

Don't allow specification of needStack to EventPipeProvider when creating an event. (#11571)

Update CoreClr, CoreFx to preview2-25312-01, preview2-25312-03, respectively (#11544)

Only call EventActivityIdControl ETW function on Windows. (#11549)

Delete non-shared version and switch over to shared version.

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

Merge nmirror to master

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

Merge pull request #10972 from hqueue/arm/ryujit/lsra

[Ryujit/ARM32] LSRA support for arm32 floating-point register allocation

Merge pull request #11500 from dotnet-bot/from-tfs

Merge changes from TFS

EventPipe Circular Buffer Support and Ability to Start/Stop Tracing (#11507)

Preserve value numbers in gtSetEvalOrder

There was already code to preserve the value number during global CSE, but
value numbers remain relevant through assertion prop, so this changes the
code to simply always pass the PRESERVE_VN flag.

Merge pull request #11536 from jashook/green_arm64_ci

[Arm64] Disable new test regressions

[Arm64] Disable new test regressions

Disable tests based on #11533, #11534.

Re-morph lclVar nodes after simplifying (ind (addr (lclVar))). (#11531)

Morph simplifies (ind (addr (lclVar))) if the type of the load matches
the type of the lclVar node. This simplification is not valid, however,
if the lclVar must be normalized upon load: in that case, the lclVar
node must be normalized appropriately as part of the transformation.
This change fixes the simplification to perform this normalization if
necessary by calling `fgMorphLclVar` on the result if the result is a
lclVar node.

Fixes issue #11508.

Update CoreFx to preview2-25311-04 (#11532)

Merge pull request #11528 from sdmaclea/PR-ARM64-Add-ldar-stlr-instrs

[Arm64] Add emitter support for ldar/stlr

Merge pull request #11468 from sdmaclea/PR-ARM64-3092

[Arm64] Use movn in instGen_Set_Reg_To_Imm

Removes unneeded Serializable attributes from enums and delegates. They can be serialized without attributes and this will remove noise from future serialization changes. (#11535)

[RyuJIT/ARM32] Fix GT_NEG decomposition (#11495)

RyuJIT/ARM32: Fix GT_NEG decomposition.

[RyuJIT/ARM32] Implement for GT_STORE_OBJ (#10721)

* Implement lowering for GT_STORE_OBJ

In #10657, I commented that the messages for NYI were printed about GT_STORE_OBJ on running the CodeGenBringUpTests.

'Lowering::LowerBlockStore(GenTreeBlk* blkNode)' method implementation is just copied.
but after lowering phase, in code generation, codegenarm.cpp, below would be used.

```cpp
blkNode->gtBlkOpKind = GenTreeBlk::BlkOpKindUnroll;
```
```cpp
blkNode->gtBlkOpKind = GenTreeBlk::BlkOpKindHelper;
```

```cpp
void CodeGen::genCodeForStoreBlk(GenTreeBlk* blkOp)
{
    if (blkOp->gtBlkOpGcUnsafe)
    {
        getEmitter()->emitDisableGC();
    }
    bool isCopyBlk = blkOp->OperIsCopyBlkOp();

    switch (blkOp->gtBlkOpKind)
    {
        case GenTreeBlk::BlkOpKindHelper:
            if (isCopyBlk)
            {
                genCodeForCpBlk(blkOp);
            }
            else
            {
                genCodeForInitBlk(blkOp);
            }
            break;
        case GenTreeBlk::BlkOpKindUnroll:
            if (isCopyBlk)
            {
                genCodeForCpBlkUnroll(blkOp);
            }
            else
            {
                genCodeForInitBlkUnroll(blkOp);
            }
            break;
        default:
            unreached();
    }
    if (blkOp->gtBlkOpGcUnsafe)
    {
        getEmitter()->emitEnableGC();
    }
}
```

'genCodeForCpBlk' and 'genCodeForInitBlk' are implemented in ARM/ARM64 by MEMCPY/MEMSET
but 'genCodeForCpBlkUnroll' and 'genCodeForInitBlkUnroll' are not implemented both ARM and ARM64.

Therefore those need to implement.

* Implement NYI : GT_STORE_OBJ is needed of write barriers implementation

It was copied from ARM64 and removed codes related with gc write barrier which doesn't support on ARM.

* Implement CodeGen::genCodeForCpObj

* Refactor some codes

* Use INS_OPTS_LDST_POST_INC option for post-indexing

When structure is copied, the results of asm codes have been strange.

IN0013: 000048 ldr r3, [r1+4]
IN0014: 00004A str r3, [r0+4]
IN0015: 00004C ldr r3, [r1+4]
IN0016: 00004E str r3, [r0+4]

It needs that the index would increment or the post-indexing.

So I use INS_OPTS_LDST_POST_INC option for post-indexing when the instruction is emitted.

* Fix conflicts

* Fix conflicts and Apply #11219

I want to merge genCodeForCpObj to codegenarmarch.cpp but the function has modified in #11219.

So I decided to keep the code divided now.

In future, If modifying the function on ARM is also needed, it would be able to modify.

* Fix conflicts

* Remove NYI

* Fix genCountBits assertion

Comment per review

Fix build.

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

[RyuJIT/ARM32] Fix assertion in 'unreached' when using emitIns_R_L (#11486)

* [RyuJIT/ARM32] Fix assertion in 'unreached'

Now, All of CodeGenBringUpTests have failed.

message : Assertion failed 'unreached' in 'DomainNeutralILStubClass:IL_STUB_PInvoke(ref)'

I found the problem was caused by #11377 that 'genCodeForTreeNode' merged to codegenarmarch.cpp.