Merge pull request #11713 from mskvortsov/Ckfinite

[RyuJIT/ARM32] Ckfinite codegen

Merge pull request #11209 from erozenfeld/MorphCastNarrowing

Add missing opcodes for morph cast narrowing.

Merge pull request #11680 from mskvortsov/CleanupLongNYIs

[RyuJIT/ARM32] Clean up remaining GT_LONG NYIs

Merge pull request #11643 from hqueue/arm/ryujit/cast_smalltype2float

[RyuJIT/ARM32] Enable casting from small int to float

remove unnecessary return copy (#11133)

Merge pull request #11700 from BruceForstall/CleanupGTFlags

Un-clang-format GTF_ flags

Merge pull request #11594 from mikedn/lower-shift-mask

Removes unnecessary shift count masking

[ARM/Linux] Support unaligned struct read/write (#11290)

* [ARM/Linux] Support unaligned struct read

* Fix format error

* Support unaligned struct write

Merge pull request #11011 from mikedn/imp-runtime-lookup

Simplify runtime lookup tree

Merge pull request #11720 from wtgodbe/BuildToolsCleanup

Clean up 2.0.0 BuildTools PR

LSRA throughput tweaks (#11698)

Convert LinearScan::getVarReg() to take a tracked var index, not varNum

Change LSRA to create Intervals only for tracked variables

Change getIntervalForLocalVar() to take a tracked variable index.

Create getIntervalForLocalVarNode()

This reduces instruction count of superpmi replay over minopts tests by 0.6%.

Improve BitVecOps<>::Iter::NextElem (#11696)

* Improve BitVecOps<>::Iter::NextElem

Tweak the implementation, to reduce the number of instructions
executed in the hot path.

Also, don't pass "env" to NextElem; it can be stored by Init()
if required. For non-inlined calls, this saves setting up one
argument.

Use a `m_bsEnd` end condition. This eliminates the need to handle
short/long differently, and reduces conditions when updating
the current bits to iterate over in the long case.

Overall, pin shows this reduces instruction count of superpmi over
a minopts test run by 2.6% (NextElem is very hot).

Also, fix BitSetAsUInt64 NextElem() iterator: It should store and updated
its own bit count, and not depend on the value passed in to be the
correct latest bit count.

Merge pull request #11746 from pgavlin/gh11733

Do not reverse operands for user-call intrinsics.

Remove `GTF_REVERSE_OPS` from LIR. (#10698)

In HIR, this flag indicates that the second operand to a binary node will execute before
the node's first operand. LIR, however, no longer determines ordering via use edges, so
this flag only affects the order in which operands to a node are considered. The sole
constraint on this use ordering is that for a given node, the same ordering must be used
in liveness, LSRA, and the code generator; this is due to the correspondence between
use ordering and spill/reload/last-use ordering. As a result, the reverse ops flag is
unnecessary and rather unhelpful in LIR, causing little more than a bit of extra complexity
throughout the backend.

This change removes `GTF_REVERSE_OPS` from LIR by clearing this flag during rationalize
and verifying that it remains clear in `LIR::CheckLIR`. We could reuse this bit for an
additional backend-specific purpose in the future with a bit more work in the checker.

Format code.

Set StrongName=None for System.Private.CoreLib

We don't ship the managed version of System.Private.CoreLib so
it doesn't need a strong name signature. We always crossgen it
so it is native.

Merge pull request #11741 from mskvortsov/FixUBFX

[ARM32] Fix ubfx instruction encoding

Clean up 2.0.0 BuildTools PR

Merge pull request #10662 from helloguo/VectorConversion

Add JIT intrinsics support for vector conversion on AMD64 and x86

Merge pull request #11747 from weshaggard/UpdateSigning

Update signing for coreclr repo

Update signing for coreclr repo

Make sign.builds consistent with corefx and only add strongname
signatures for managed assemblies and not native assemblies

Do not reverse operands for user-call intrinsics.

Maintaining reversed execution order for such nodes is problematic when
these intrinsics are rewritten into calls during rationalize.

JIT: relax method attribute assertion check in devirtualization (#11730)

The jit may see `CORINFO_FLG_DONT_INLINE` change between the time it
first reads a method's attributes and the time it re-reads it, since
jitting on some other thread may have deduced the method as noinline
in between the two reads. This bit doesn't affect devirtualization
and so the bit change is harmless.

Tolerate this by masking out this flag bit when comparing attributes.

No changes to generated code.

Closes #11619.

[lldb-plugin] Include PAL headers instead of embedding it (#11705)

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.

ARM32: Fix ubfx instruction encoding.

RyuJIT/ARM32: enable ckfinite codegen.

[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

RyuJIT/ARM32: Cleanup remaining GT_LONG NYIs.

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

[RyuJIT/ARM32] Enable casting from small int to float

This just enable casting from small int to float by removing assertion.

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

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

Removes unnecessary shift count masking

The C# compiler translates code like `x << y` to `x << (y & 31)` because the behavior of IL shift operations is unspecified if the shift count is greater than or equal to the bit width of the shifted value. However, x86/x64 shift instructions do mask the shift count so `& 31` is redundant.

It's also possible that developers also mask the shift count, not knowing that the C# language specification guarantees masking. In that case we end up with 2 `and x, 31` instructions.