Merge pull request #15791 from dotnetrt/dstdstsrc

Refactor emitter::IsDstDstSrcAVXInstruction and emitter::IsDstSrcSrcAVXInstruction

[Infrastructure/Win] Add C++ header files to Visual Studio clrjit project (#15775)

* Add C++ header files to Visual Studio clrjit project

* Add comment about script change impact and usage

movhlps categorized as dstdstsrc

Allow CALLEE_IS_FORCE_INLINE precedent over CALLEE_DOES_NOT_RETURN (#14586)

Allow CALLEE_IS_FORCE_INLINE precedent over CALLEE_DOES_NOT_RETURN

Keep LowLevelDictionary for CoreRT for now (#15824)

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

Improve performance for Math.Abs (#15823)

* Improve perf for Math.Abs

* Inline Math.Abs

Fix Obsolete on ReadOnlySpan Equals and GetHashCode methods (#15820)

* Fix Obsolete on ReadOnlySpan Equals and GetHashCode methods

* Delete unnecessary CLSCompliant attribute

Merge pull request #15814 from CarolEidt/FixObj

Fix change to fgMorphBlockOperand

Fix change to fgMorphBlockOperand

My recent fix broke the `DYN_BLK` case. In addition to checking that types match, we *still* need to check that the sizes match.

This was caught by desktop testing. The test cases are somewhat complex, so I created a smaller repro.

Fix incremental build by preventing nonchanging writes (#15817)

Fix incremental build by preventing non-changing writes in generated ETW source code

Merge pull request #15784 from fiigii/hwci

Add new CI mode for Intel HW intrinsics

Underlying CoreCLR support for new string slicing overloads (#15811)

* Underlying CoreCLR support for new string slicing overloads

https://github.com/dotnet/corefx/issues/25254

These add the underlying support for the fast versions of
these extension methods.

* Underlying CoreCLR support for new string slicing overloads

https://github.com/dotnet/corefx/issues/24072

These add the underlying support for the fast versions of
these extension methods.

Add privaterun properties (#15816)

Move System.Globalization.CultureData to shared CoreLib partition (#15805)

Update CoreClr, CoreFx, PgoData to preview1-26109-03, preview1-26110-01, master-20180109-0023, respectively (#15788)

Port servicing fix (#15802)

* Port the servicing fix

This is porting the servcing fix https://github.com/dotnet/coreclr/commit/8ce622d569fae9feb160399db03dc2b86577be1b  with a small change to get rid of unsafe code in CultureData per Jan recommendation

* remove un-needed lines

* rename the variable name as it is not a pointer anymore

Add new CI mode for Intel HW intrinsics

Update the document for netcoreapp2.1

Update links to nightly .NET Core SDK builds (#15799)

Merge pull request #15724 from sdmaclea/PR-ARM64-VECTOR64

[Arm64] Add Vector64 HW Intrinsic support

Adds StringBuilder.Equals(ReadOnlySpan<char>) Api (#15759)

Adds StringBuilder.Equals(ReadOnlySpan<char>)  Api

Address review feedback

fix corefx/#26230 (#15792)

Fixing https://github.com/aspnet/JitBench/issues/80 (#15764)

- Issue was fixed with this: https://github.com/aspnet/JitBench/pull/82
- Set the UseSharedCompilation=false to avoid that the `VBCSCompiler.exe` stays running.

Refactor emitter::IsDstDstSrcAVXInstruction function implementation

Update CoreClr, CoreFx, PgoData to preview1-26108-06, preview1-26109-01, master-20180108-1624, respectively (#15785)

Update BuildTools, CoreClr, CoreFx to prerelease-02408-01, preview1-26108-02, preview1-26108-02, respectively (#15763)

Fixed superpmi JIT debugging on desktop build. (#15773)

Update Intel hardware intrinsic APIs (#15601)

- add missing APIs
- fix generic names
- fix comments
- disambiguate with `using static`

JIT: improve return types in cases with spill temps (#15766)

If the jit sees that an inlinee has multiple return sites or has gc ref locals
it will choose to return the inline result via a temp. The jit was not assigning
a type to that temp and so losing track of some type information.

So, for inlinees returning ref types, initially type the return spill temp with
the declared return type of the method.

When importing we may discover that particular return sites will return more
specific types. If all discovered return sites agree, we can update the return
type for the spill temp to match the consensus improved type.

This can lead to removal of some type checks and also to devirtualization.

Addresses issues discussed in #9908 and #15743.

Merge pull request #15772 from tannergooding/hwintrin-expand

Updating the x86 HWIntrinsic importer to not inline when returning gtNewMustThrowException

Updating the x86 HWIntrinsic importer to not inline when returning gtNewMustThrowException

Move types to shared corelib partition (#15768)

- YieldAwaitable: Fixed readonly mismatch
- Comparer: Made public to fix https://github.com/dotnet/corefx/issues/25973

Cleanup Activator (#15767)

Merge pull request #15756 from briansull/fix-15671

Fix for 15671

Change just-in-time debugging registry keys. (#15722)

From "SOFTWARE\Microsoft\Windows NT\CurrentVersion\AeDebug" to "SOFTWARE\Microsoft\.NETCore\JITDebugging".

ARM r2r fix. (#15758)

Do not trash CALLEE_SAVED register r4.
Use r12 that is CALLEE_TRASH.

JIT: fix decompose long left shift for overshift cases (#15704)

Need to reduce the shift amount modulo 64 to match the helper and
`gtFoldExpr` behavior. Since reduced amount must be less than 64
we can remove handling for that case.

Also updating the arm LLSH helper.

Re-enable the test case disabled by #15567 and also enable for arm/arm64.

Closes #15566.

Proposed fix for 15671

If we are fetching an Array Length for an array ref that came from global memory
then for CSE safety we must use the conservative value number for both

Simplify and unify EnumCalendars interop (#15762)

Windows was not allocating GCHandles, switch Unix to do the same.

Merge pull request #15639 from tannergooding/expand-hwintrin

Updating Compiler::impIntrinsic to always expand hardware intrinsics.

JIT: fix issue with inline observations (#15713)

In DEBUG/CHECK builds the jit tries to keep track of failed inlines.

Because inlines can be rejected "early" (when the parent method
is being imported) as well as "late" (when their call site is encountered
by the inliner) there is a tracking mechanism to convey the early observations
that cause failures to be resurrected later on.

These observations sometimes didn't end up in the inline context, leading
to assertions when dumping methods.

Fix is to add a new way to propagate the earlier observation to the context
that bypasses some of the policy sanity checks and simply record the reason
that the inline failed.

Merge pull request #15760 from briansull/fix-budget

Fix issue where the unsigned value used for the the VN map select budget could underflow

Merge pull request #15716 from CarolEidt/RefactorMinRegCount

Refactor minRegCount and eliminate auto

Refactor minRegCount and eliminate auto

When building RefPositions in the TreeNodeInfoInit methods, it is very complex and messy to have the setting of the (debug-only) `minRegCandidateCount` spread all over. Instead, set them for all the RefPositions for a given node after they have been created.

This required the ability to create an iterator marking the end of the list, prior to creating the new ones.

In the process, eliminated the use of auto from lsra.cpp in the interest of strong typing, as well as avoiding the confusion of having things with `RefPosition` in their name that are actually iterators.

These changes originally caused failures with JitStressRegs because `getKillSetForNode()` had side effects - generating Kill `RefPosition`s for `MOD` & `DIV`. This is not actually necessary, and causes problems when the debug-only stress code was calling it *after* all the `RefPosition`s had been created. Also, this had the weird (existing) effect that duplicate kill `RefPosition`s were being generated in the stress case!

Fix issue where the unsigned value used for the the map select budget could underflow
resulting in an unlimited budget

Changed m_mapSelectBudget and budget to be a signed integer
Added an assert that ensures that the remaining budget is between [0..m_mapSelectBudget]
Change the test for running out of budge to be a <= zero test instead of an equal zero test.

Fixed the bug in the handling of phiArgs in VNForMapSelectWork
by adding a check if we exceeded our budget when processing the first phiArg.

Added a define for DEFAULT_MAP_SELECT_BUDGET

Cleanup AppDomain (#15748)

Remove dead code and various cleanup.

Detect ByRefLike types using attribute (#15745)

* Detect ByRefLike types using attribute and improve error messages for their invalid use

Fixes #11371 and #15458

fix COMPlus_JitHalt for arm32. (#15761)

Update BuildTools, CoreClr, CoreFx to prerelease-02404-02, preview1-26105-01, preview1-26105-01, respectively (#15736)

AggregateException: Use StringBuilder.AppendFormat (#15747)

Use `AppendFormat(...)` instead of `Append(string.Format(...))` and
`AppendLine()` instead of `Append(Environment.NewLine)`.

Unix: Specify Ordinal comparison in TimeZoneInfo (#15739)

The default behavior for StartsWith uses the current culture for
comparisons. These should be ordinal.

Address ReflectionTypeLoadException feedback (#15738)

* Address ReflectionTypeLoadException feedback

Address additional minor feedback from #15711:

 - Share the code for `Message` and `ToString`
 - Remove trailing `NewLine` from resulting string
 - Pass initial base string to `StringBuilder..ctor`
 - Cache the exceptions array in a local (passed-in as a parameter)
 - Move methods below properties
 - Remove trailing whitespace

Use string.Contains(char) instead of Contains(string) (#15740)

Now that string.Contains(char) exists, use it in corelib.

Adding EnsureCapacity implementation for Dictionary (#15729)

Adding EnsureCapacity implementation for Dictionary

Updating the existing HWIntrinsic tests to also test indirect calling via reflection.

Update profiling API status (#15659)

* Update profiling API status

* Fix wording for ARM ReJIT issue

Remove duplicate callback to AppDomainCreationFinished in profiler (#15691)

Change monitor test (#15727)

Relevant to https://github.com/dotnet/coreclr/issues/15592:
- Not sure what is actually happening, guessing there is an exception during TryEnter() and the exception from Exit() from the finally block is hiding the actual exception
- Moved TryEnter() to before the try block
- Fixed pass/fail condition for some similar monitor tests

Fix waitone test (#15728)

Fixes https://github.com/dotnet/coreclr/issues/15098:
- Probably a timing issue, changed to use Thread.Join() instead of sleeps

Updating Compiler::impIntrinsic to always expand hardware intrinsics.

Reset lvIsStructField when removing promoted struct vars (#15721)

Merge pull request #15726 from CarolEidt/Fix15237

Keep OBJ on mismatched struct assignment

Remove bad assert in Mutex & update error messages (#15734)

* Remove bad assert in Mutex & update error messages

* Redo error message

Merge changes from TFS (#15732)

* Remove unused CORJIT_FLAG_USE_SSE3_4

This changes the JIT-EE interface GUID.

This is the desktop side of https://github.com/dotnet/coreclr/pull/14825.

[tfs-changeset: 1685154]

* Pick up rest of desktop side changes

Mark all CoreCLR packages as non-shipping (#15725)

This indicates the packages shouldn't be shipped to the NuGet gallery.

[Arm64] Add Vector64 HW Intrinsic support

Update BuildTools, CoreClr, CoreFx to prerelease-02403-01, preview1-26103-02, preview1-26104-02, respectively (#15684)

Fix link to IdnMapping.cs (#15717)

Link to THIRD-PARTY-NOTICES is fixed (#15665)

The link to the THIRD-PARTY-NOTICES page is broken in contributing.md [https://github.com/dotnet/coreclr/blob/master/Documentation/project-docs/contributing.md]

The current link is [https://github.com/dotnet/coreclr/blob/master/THIRD-PARTY-NOTICES]

Seems it is missing a '.TXT' at the end of the URL. The correct link should be [https://github.com/dotnet/coreclr/blob/master/THIRD-PARTY-NOTICES.TXT] which is working fine.

Issue #15664

Check for null in ReflectionTypeLoadException Message/ToString (#15711)

Fix typo in docstring (#15714)

Keep OBJ on mismatched struct assignment

In a mismatched struct assignment, e.g. using Unsafe.As, we need to retain the OBJ(ADDR(lcl)) on the rhs of the assignment.

Fix #15237

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

Merge changes from TFS

Use the message that the ReflectionTypeLoadException exception was initialized with (#15708)

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

Updating the build script to work when python is installed to a path containing a space. (#15707)

Merge pull request #15614 from BruceForstall/AddPushPeriodicViews

Add more Views to Jenkins UI

Fix size-on-disk benchmark to handle new CLI behavior (#15689)

Add flag to ensure ensure the compiler server is not used so that the installation can be deleted successfully after the test. Additionally use a more robust deletion strategy to ensure files are deleted and the test does not fail spuriously due to random IO errors.

Fix formatting

Merge pull request #15602 from sdmaclea/PR-HWINT-XARCH-DEP

[Arm64] getBaseTypeAndSizeOfSIMDType

Spill the eval stack when storing to a pinned local.

The jit may not see direct interference between a pinned local and expressions that use the pinned value. So worst case it should assume that such interference exists and spill the stack when storing to a pinned local. Otherwise the jit may reorder unpins and calls that refer to pinned storage.

Fixes DevDiv 545749

[tfs-changeset: 1685045]

Pass manifest details to PushToBlobFeed (#15705)

Merge pull request #15702 from sdmaclea/PR-EOL

Normalize lsra.h line endings

Normalize lsra.h line endings

Print Inner Exceptions in ReflectionTypeLoadException ToString (#15688)

* Print Inner Exceptions in ReflectionTypeLoadException ToString

* Simplify formatting

* Remove tab

* Fix newlines

* Use AppendLine

* Use foreach

* Remove ToString

* Update string & message

* Modify Message

* Remove parens

[Arm64] getBaseTypeAndSizeOfSIMDType

Remove XARCH platform dependencies
Disable Vector256 support on Arm64 until needed

Merge pull request #15698 from mikedn/patch-1

Fix minor formatting issue in jit-call-morphing.md

Merge pull request #15699 from CarolEidt/FixFeatureSIMDDesc

Fix description of FeatureSIMD

Use Span to drop byte[1] allocations (#15680)

Fix description of FeatureSIMD

The associated dll is actually System.Numerics.Vectors.dll,
plus System.Numerics.dll on desktop.

Fix minor formatting issue in jit-call-morphing.md

Merge pull request #15692 from BruceForstall/DisableJitdiffJobs

Disable jitdiff jobs

Disable jitdiff jobs

They are failing now. Disable them until someone puts the effort
in to make them fully work.

Merge pull request #15640 from tannergooding/hwintrin-lower

Adding basic containment analysis support for hardware intrinsics.

Merge pull request #15596 from CarolEidt/RemoveMap

LSRA: Replace operandToLocationInfoMap with a list

Retry: Enable EventPipe across Unix and Windows (#15611)

* Revert "Revert "Enable EventPipe across Unix and Windows (#14772)" (#15609)"

This reverts commit 302005ca8ae14eade37ddf4ac6e900617c1c166a.

* Fix ARM build break

* Use more explicit references to resolve build failures

* Fix compat with python3

* Disable FeaturePerfTracing on Windows as it is not ready

* Disable test for incomplete functionality

* Fix test diabled patterns

* Add license header

* Use keyword types for managed code

* Add message prefix

* More precisly condition generation of eventing sources

* Remove erroneously added changes

Update BuildTools, CoreClr, CoreFx to prerelease-02402-01, preview1-26102-02, preview1-26102-01, respectively (#15678)

Give a proper VN to GT_BOX (#15666)

GT_BOX doesn't do anything so it should just get the VN of its sole operand. This allows assertion prop to see that the result of BOX is not null (since the BOX operand is the result of an object allocation which is never null).

Encoding: Don't alloc for zero length array (#13794)

Remove some unnecessary boxing (#15676)