Update CoreClr, CoreFx, PgoData to preview1-26116-03, preview1-26116-03, master-20180116-0045, respectively (#15859)

Fix mismatch between new and free in numa.cpp (#15870)

* Fix mismatch between new and free in numa.cpp

One of the allocations in the numa.cpp uses new to allocate an array,
but it incorrectly uses free to free the memory. This change fixes it.

Fix Android build (#15868)

There were the following issues:
* The current versions of packages available at the termux site were obsolete.
* The libintl.h was removed from the Android SDK, so the dgettext and bindtextdomain
  are not available. But they were dummy implementations before anyways, so we can remove
  their usage on Android.
* The detection of the ucol_setMaxVariable needs to be done using the
  check_cxx_symbol_exists instead of check_symbol_exists, since on Android, the
  ICU libraries now depend on C++ runtime.
* The SIZE_T_MAX is already defined in Android headers, so the definition in cgroup.cpp was
  colliding with it.
* The pthread_condattr_setclock detection was using pthread library, but on Android, it is
  located in the "c" library instead. So it was not being detected.

Delete BasicSpanTest.cs (#15860)

It is redundant with CoreFX Span tests now

Update CoreClr, CoreFx, PgoData to preview1-26114-02, preview1-26114-03, master-20180114-0044, respectively (#15856)

Merge pull request #15855 from jkotas/revert

Revert "Remove relocations for MethodTable::m_pParentMethodTable for Linux ARM"

Update CoreClr, CoreFx, PgoData to preview1-26113-02, preview1-26113-03, master-20180113-0045, respectively (#15853)

Revert "Remove relocations for MethodTable::m_pParentMethodTable for Linux ARM"

This reverts commit cf1fb9e17fc8b6ee849edab5a696d0ec5c6eadd2.

Mark legacy ByRef-like types as ref in sources (#15846)

Delete special casing in the type loader

Merge pull request #15850 from BruceForstall/FastGCStress

Do fast GC stress on some extreme tests

Merge pull request #15822 from BruceForstall/FixWriteBarrierHelperKills

Fix ARM GCStress hole with byref write barrier helper

Added Append(Stringbuilder,int,int) and Append(StringBuilder) overlaod apis (#15786)

Added Append(Stringbuilder,int,int) and Append(StringBuilder) overlaod apis

Do fast GC stress on some extreme tests

Set COMPlus_FastGCStress=1 to avoid GC in the
`CoreCLR!JIT_Stelem_Ref => CoreCLR!ArrayStoreCheck` path.

Merge pull request #15804 from tannergooding/hwintrin-containment

Adding basic containment support to the x86 HWIntrinsics

updated docs: how to run against local core clr build (#15841)

* updated docs: how to run against local core clr build

* split the running docs into 3 files to make it simpler to understand

LoaderHeap: remove LHF_ZEROINIT option.

This option was used for UMEntryThunkCode::Poison. Now we use own free list
to store freed thunks and don't return allocated memory to the LoaderHeap.
So reused thunks are always uninitialized.

dllimportcallback: remove code for CallbackOnCollectedDelegate MDA

UMEntryThunk: store freed thunks into FIFO free list

Use free list to delay reusing deleted thunks. It improves
collected delegate calls diagnostic.

Improve UMEntryThunkCode::Poison method.

Improve UMEntryThunkCode::Poison to produce diagnostic message
when collected delegate was called.

Update CoreClr, CoreFx, PgoData to preview1-26112-01, preview1-26112-01, master-20180112-0034, respectively (#15837)

Fixing the hwintrin codgen containment checks

Adding asserts that values are as expected for certain containment checks

Adding basic containment support to the x86 HWIntrinsics

Merge pull request #15815 from jashook/fix_helix_publish

Fix overwriting os/arch when publishing to helix

Fix overwriting os/arch when publishing to helix

Convert run-xunit-perf to py script (#15568)

* Convert run-xunit-perf to python script

This change merges the two run-xunit-perf scripts (.sh and .cmd) into
one unified python script and updates the pipeline job to use the
pythong script. This change also updates the linux jobs to use the new
build-tests.sh generatelayoutonly command so that we don't need to pull
down corefx from the cloud anymore. The unified python script enables us
to more easily update both linux scripting and windows scripting at the
same time so that one does not lag behind the other (such as when we add
new configurations or options like slicing). This change also turns
linux testing back on by default for PRs.

Merge pull request #15835 from BruceForstall/FixGCKeepAlive

Fix GC.KeepAlive test case

Merge pull request #15832 from BruceForstall/DisableNoGCInStress

Disable NoGC test in GCStress test runs

Update BuildTools, CoreClr, CoreFx, PgoData to prerelease-02411-04, preview1-26111-02, preview1-26111-02, master-20180111-0041, respectively (#15807)

[Local GC] Introduce standalone eventing design document (#15570)

* Typing

* First draft

* Update the doc based on feedback

* Next iteration based on feedback

* Iteration feedback

Fix GC.KeepAlive test case

In the test, as written, a GC could sneak in between the GC.KeepAlive()
call and the subsequent line that checks whether the finalizer
has run, especially in GC stress modes. Simply move the GC.KeepAlive()
call down.

Disable NoGC test in GCStress test runs

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 non-Windows compile error

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 ARM GCStress hole with byref write barrier helper

When unrolling a STOREOBJ, we can generate multiple consecutive
byref helper calls. This helper has a unique calling convention
where the dst and src addresses are modified by adding pointer
size to their original value (thus allowing consecutive helper
calls without reloading the dst/src addresses). So, for liveness
purposes, the helper call kills the dst/src values. However, for
GC purposes, it does not, as the registers still contain byref
pointers. We were, in the ARM case, reporting the r0/r1 registers
dead after the first call, so a GC didn't update them, and a
second call updated garbage.

In fixing this, I cleaned up the helper call kill handling a bit.
I also fixed and improved RyuJIT/x86 write barrier kill modeling.

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.