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

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

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.

LinkBench: Report reduction as a ratio.

Report the linker size reduction metric as a ratio
(linked filesize / unlinked filesize) instead of as a percentage.

Merge pull request #11423 from jashook/arm64_live_tests

Windows Arm64 - Build/Run tests in CI

Windows Arm64 - Build/Run tests in CI

Build tests with arm64 runs and upload/run them to the arm64 test
machines.

Handle ADC in x86 unwindLazyState (#11527)

When we add PGO instrumentation to coreclr, the instrumentation injects
PUSH, MOV, ADD, ADC, POP into the prolog to mark the function entry. This
causes a crash because we do not currently handle ADC instructions in
unwindLazyState. This change adds support for ADC, which uses the normal
DecodeRM code we already have in place.

Add back private BadImageFormatException constructor called by the VM (#11523)

Also removed rethrowing of an exception inside AssemblyName.GetAssemblyName that sometimes resulted into two identical exceptions getting nested into each other.

Fixes #11499

Fix AssemblyName version (#11505)

* Fix AssemblyName.FullName

* Use canonicalized version instead of Version.ToString

[Arm64] Fixup stlr instrs

[Arm64] Add emitter support for ldar/stlr

[Arm64] Fix instGen_Set_Reg_To_Imm per review

Merge pull request #11437 from mjsabby/eventpipetoprofilerapi

Add dynamic method unload event to Profiler API

This change has the following improvements to LinkBench

1. Move the testing to the latest version of ILLink package
   (for all benchmarks except for Roslyn, which is still on netcoreapp1.1)
2. Support for LinkBench to run individual benchmarks in the suite --
   helps local testing.
3. Support for running without clean, cloning, build etc --
   for local runs during continuous development.
4. Support to obtain the correct version(s) of DotNet CLI required to run
   the benchmarks
5. Get-Crossgen.ps1 is required to crossgen MusicStore binaries (before/after linking).
   Since it was recently dropped from JitBench repo, it is added to linkBench as an asset.
6. Add the --perf:runid parameter to generate output file names suitable
   to upload the results to BenchView.

Remove eventpipe stuff left from prev commit

add prebuilt file

Remove EventPipeProfiler due to time constraints

Allow multiple SetData calls on AppDomain (#11496)

* Allow multiple SetData calls on AppDomain
* Remove restriction from GetData and remove now-dead code Locate
* Remove LoaderOptimization check since it is not used on CoreCLR

Re-Factor EventSource to Support Writing to EventPipe (#11435)

Re-Factor EventSource to Support Writing to EventPipe.

Merge pull request #11478 from sdmaclea/PR-ARM64-9711

[Arm64] Support GTF_IND_VOLATILE

Add version.txt to the coreclr transport package (#11497)

* Add version.txt to the coreclr transport package

* PR Feedback

Merge pull request #11448 from DrewScoggins/AddPriority

Launch perf process with high priority

Renable building frankenjit

[tfs-changeset: 1657926]

[ARM/Linux] Enable build with FEATURE_DBGIPC=0 (#11254)

* [ARM/Linux] Enable build with FEATURE_DBGIPC=0

* Replace HAVE_ALIGNED_MALLOC with SUPPORT_LOCAL_DEBUGGING