Use ROSpan.IndexOf as the workhorse for string.IndexOf (#17284)

* Use ROSpan.IndexOf as the workhorse for string.IndexOf

* Make changes to Span.IndexOf to follow what string.IndexOf did.

* Address PR feedback.

* Use Unsafe.Read instead of Unsafe.ReadUnaligned.

* Remove special casing for count == 0

* Fix up debug assert to use vector count instead of intptr.size

* Use size of Vector<ushort> instead of Vector<byte>.Count

Fix Dictionary CopyTo regression (#17300)

Merge pull request #17303 from BruceForstall/EnableUbuntuArmGCStressJobsMaster

Enable Linux arm32 GCStress jobs

Removing 'EXPERIMENTAL' from tiered compilation env var (#17283)

Things have progressed far enough that its time to use a friendlier name. The feature still still has performance aspects that need to be investigated and improved, but I don't want to scare people off simply because it isn't as fast as it could be.
This also updates to use a newer CoreFX version for JitBench since that appeared to be broken, and updated some comments and usage of the tieredcompilation variable.

Enable Linux arm32 GCStress jobs

Non-PR jobs (cron and push) are disabled for now.

Unique-ify name of TestList.json to avoid collisions

Fixes #16840

Merge pull request #17090 from debayang/corefx_tests

Fix run-corefx-tests.py for Arm64 Linux cross build

Fix some more unwinding in R2R ARM (#17292)

There was one more change needed to make the unwinding work properly.
Pushes in some prologs were missing the unwinder annotation.
The fix is to use PROLOG_PUSH for them.
To make things in this file consistent, I've also replaced pops in
epilogs with EPILOG_POP macro and vpush / vpop with PROLOG_VPUSH /
PROLOG_VPOP, although it is not functionally necessary.

With these changes, all the EH related issues are gone.

Nit changes in API diff between .NET Core 2.0 and .NET Core 2.1 (#17288)

* 'value' to 'other': bool System.IEquatable<System.UIntPtr>.Equals(UIntPtr value);

* 'value' to 'other': bool System.IEquatable<System.IntPtr>.Equals(IntPtr value);

Update BuildTools, CoreClr, CoreFx to preview3-02627-03, preview3-26328-07, preview3-26328-06, respectively (#17248)

Merge pull request #17126 from CarolEidt/Fix545500

ARM: Fix reg resolution for doubles

Add crossbitness support to ClrJit:
  * Add FEATURE_CROSSBITNESS in crosscomponents.cmake
  * Exclude mscordaccore mscordbi sos from CLR_CROSS_COMPONENTS_LIST when FEATURE_CROSSBITNESS is defined in crosscomponents.cmake
  * Introduce target_size_t in src/jit/target.h
  * Use size_t value in genMov32RelocatableImmediate in src/jit/codegen.h src/jit/codegencommon.cpp
  * Fix definition/declaration inconsistency for emitter::emitIns_R_I in emitarm.cpp
  * Zero HiVal when GetTree::SetOper GenTreeLngCon->GetTreeIntCon in src/jit/compiler.hpp
  * Explicity specify roundUp(expr, TARGET_POINTER_SIZE)
  * Use target_size_t* target in emitOutputDataSec in src/jit/emit.cpp

Root methods used by debugger for IL linker (#17287)

Remove tests of unimplemented ISAs (#17286)

fix for logalways eventing level (#17277)

Fix run-corefx-tests.py for Arm64

Enable cross build of corefx on x64, creates runtime for arm64
and builds corefx tests.

Fix DelayLoad_MethodCall unwinding (#17275)

* Fix DelayLoad_MethodCall unwinding

Unwinding through DelayLoad_MethodCall was broken due to the overwriting
of R7 which is used as a frame pointer. That caused some managed
exceptions to cause abort with unhandled PAL_SEHException.

This change fixes the problem by using a different register.

* Fix one more spot with the same issue

Merge pull request #17161 from dotnetrt/testbuild

Improve Windows test build performance by building projects in groups - 4x less memory 326% faster build.

 Adding Memory.Pin() to eventually replace Memory.Retain(bool) (#17269)

* Adding Memory.Pin() to eventually replace Memory.Retain(bool)

* Fix copy/paste error and return default for when object is null.

* Fix XML comments.

Fix handling of FCalls in ExternalMethodFixupWorker (#17252)

When running ready to run code on ARM, the ExternalMethodFixupWorker
doesn't handle the entrypoints of FCalls correctly. It tries to handle
them as compact entrypoints, but those use a different machine code
instructions and it results in an assert in debug / checked build.

This change detects the runtime supplied calls before trying to check
for the compact entrypoint.

Fix regression in StreamWriter.Write perf for small inputs (#17251)

* Streamline StreamWriter/Reader.CheckAsyncTaskInProgress and make it inlineable

* Re-inline StreamWriter.WriteSpan

Prior to my span changes, the code in WriteCore was manually inlined into each of its call sites.  I'd refactored it out into a shared helper to reduce code duplication, but that introduced a regression for small inputs.  I'm keeping the factoring but making it AggressiveInlining to avoid that regression while still keeping the C# code reuse.

* Use NoInlining on WriteSpan call sites

In case methods get devirtualized and JIT attempts to inline them and WriteSpan.

Fix comment

Failfast windows event log (#16875)

* Add FailFast error log to Windows Event Log

* change const wchar * to lpcwstr

* Enable sending unhandled exception info to Windows Event Log

* Change log format to match console output and address PR comments

* Remove more comments

* Change the order DoReportForUnhandledException to do a safety check first before calling managed code

* Fix parameter name in header file

* Add Windows Event logging in DefaultCatchHandler and remove DoReportForUnhandledException

* Add back event reporting for ignored unhandled exception cases, fix broken UNIX builds

* Fix more broken unix builds

* Fix typo

* Address PR comments

[Arm64] Fix forcerelocs (#17264)

Enable reflection load ComImport assembly and Type.IsComObjectType (#16943)

* Enable reflection load ComImport assembly and Type.IsComObjectType

* Update Enable reflection load ComImport assembly

Fix comment in test.

Turn off PGO unprofiled warning on Linux (#17262)

This warning will break the build when new files are added to the build that do not have PGO data, meaning that release builds will be broken until a new PGO collection can be run. We should not break the build if there are not counts for new files.

restrict memory limit by taking VM into consideration (#17177)

Merge pull request #17233 from fiigii/portvectortests

Port VectorReturn test to Vector128/256<T>

Merge pull request #17255 from BruceForstall/UpdateArmWhitelist

Update arm64 whitelist

Merge pull request #17258 from dotnet/dev/unix_test_workflow

Update arm Ubuntu cross build docker image

Cleanup InternalsVisibleTo (#17185)

Remove InternalsVisibleTo for mscorlib and make sure that corefx still builds with it.
Delete global constants that were left-overs from .NET Framework build system.

Merge pull request #17246 from debayang/clr_jitstress_arm64

[ARM64] Fix UnspillReg instruction generation

Update arm64 whitelist

[Arm64] reserve for jump stubs (#17244)

Move String.Comparison.cs to shared CoreLib partition (#17247)

Fix build without libunwind installed (#17221)

* Fix build without libunwind installed

I have removed libunwind dependency in a recent commit, but
it turns out that the build was using incorrect include
paths and it was still depending on the libunwind installation
for the libunwind.h header.

This change fixes it.

* Fix clang warning

The libunwind headers contain two empty structs and clang
issues a warning since the size of such structs is
0 in C and 1 in C++.
This is benign for our purposes, since the affected
structs are not directly used by our code.

Update BuildTools, CoreClr, CoreFx to preview3-02627-01, preview3-26327-02, preview3-26327-01, respectively (#17239)

Delete MemoryExtensions.AsBytes (#17245)

Moved to MemoryMarshal

Vectorized SequenceCompareTo for Span<char> (#17237)

- This change makes the compare for very short Span strings a bit slower and for longer Span strings many times faster.
- Switch several places where it was a clear benefit to use it.
`String.CompareOrdinal(string,string)` is notable exception that I have left intact for now. It is fine tuned for current string layout, and replacing with a call of vectorized SequenceCompareTo gives mixed results.

Merge pull request #17240 from BruceForstall/RemoveTizenPRTrigger

Remove Tizen armel automatic PR triggered jobs

[ARM64] Fix UnspillReg instruction generation

Fixes the runtime issues (#16359,#15389) seen with COMPlus_JitStressRegs=0x2* on ARM64.

New chunk should be equal/larger than the one it replaces (#17219)

* The new chunk should be equal or larger than the one it is replacing. Fixes #17205

* Adding .exe.stackdump to .gitignore

Enable partially implemented Intel HW intrinsics ISAs - CoreCLR part of changes

Merge pull request #17226 from weshaggard/MergeRestoreTargets

Merge various restore tasks into one Sync target

Remove Tizen armel automatic PR triggered jobs

These are failing almost every job with "qeumu: Unsupported
syscall: 389". Issue: https://github.com/dotnet/coreclr/issues/12972.

Port VectorReturn test to Vector128/256<T>

Remove a few bogus reinterpret_cast(nullptr_t) (#17229)

reinterpret_cast cannot be used to convert nullptr_t values, so does C++ std says in a note. VC++ ignores this restriction claiming that notes are not normative. Clang enforces the restriction. Oh well...

Merge pull request #17236 from BruceForstall/UpdateArmDockerImage

Update arm Ubuntu cross build docker image

Update arm Ubuntu cross build docker image

Now, use:

microsoft/dotnet-buildtools-prereqs:ubuntu-14.04-cross-e435274-20180323032140

Merge pull request #15301 from mikedn/cast-un

Fix inconsistent handling of zero extending casts

Disable MSBuildEventLogging for optdata alias

We don't need the binclash logger for optdata so disable it
by pass a single space (note empty string means default).

The default value points at net46 which isn't what we want on
linux.

Use movsdsse2 for HW intrinsic SSE2_ConvertToDouble (#17110)

Instead of movsd, which is for GPRs. Fixes one of the stress failures
seen in #17027.

Merge pull request #17018 from dotnet/dev/unix_test_workflow

Enable arm32 Ubuntu testing in CI

Only use FEATURE_PERFTRACING and FEATURE_EVENT_TRACE features when turned on (#17232)

Fix condition

[local gc] Fix for BitScanForward64, it was ignoring the high byte check (#17142)

Update CoreFx to preview3-26326-03 (#17222)

Work around Jenkins problem with large env vars

Clear out `ghprbCommentBody` before msbuild is invoked, which can't
deal with large environment variables.

Linux arm32 testing

Add support for arm Ubuntu testing on hardware. Most JIT stress jobs are enabled.
GC stress jobs are not enabled currently (they haven't been tried yet).
Most jobs have both PR trigger jobs as well as weekly "cron" jobs.
Job frequency will be adjusted as we gain experience with the load.
Initially, there are no automatically triggered PR jobs, but I plan to add
one or more soon.

Moved existing arm Ubuntu (on emulator) and arm Tizen (on emulator)
to use new "armem" pseudo-architecture.

The arm Ubuntu jobs have the product cross-built on Ubuntu/x64 in Docker,
use tests built in a new Windows_NT x86 BuildOnly (`_bld`) job, and are run
on hardware after copying over all the correct artifacts.

Refactored lots of code related to determining if a particular combination
of os/architecture/configuration/isPR/etc. should create a job.

Refactored test/flow job creation.

runtest.sh now creates three files based on test results that can be fed
back to runtest.sh using the `--playlist` argument: coreclrtests.pass.txt,
coreclrtests.fail.txt, coreclrtests.skip.txt.

New script `runtesttilstable.sh` runs `runtest.sh` in a loop, rerunning failing
tests until no tests fail, or a maximum iteration count is reached (default: 4).
This works around existing issues with flaky test failures on Linux/arm32 hardware.

Some extraneous unused jobs have been removed, such as unused `_bld` jobs
and non-PR triggered `innerloop` jobs.

Add /p:UseRazorBuildServer=false to prevent background processes (#17152)

Optionally support using the system-installed libunwind (#17164)

This allows coreclr to link against a system-installed libunwind instead
of using the copy included with the coreclr sources.

The default is still to use the bundled version, except on macOS, where
we always use the system-installed version.

Many Linux distributions prefer to avoid bundled libraries as much as
possible. For their reasons, please see:

- https://fedoraproject.org/wiki/Bundled_Libraries
- https://wiki.debian.org/UpstreamGuide#No_inclusion_of_third_party_code
- https://wiki.gentoo.org/wiki/Why_not_bundle_dependencies

Update uses of AsBytes to use MemoryMarshal.AsBytes (dotnet/corefx#28460)

* Update uses of AsBytes to use MemoryMarshal.AsBytes

* Cleanup commented out code

* Remove AsBytes from MemoryExtensions Portable

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

Merge various restore tasks into one Sync target

There were various restore targets happening independently
when they should all be combined so the sync step can be
independent from the build step. This change merges them
together under the Sync target.

In particular this moves RestoreOptData and RestoreNETCorePlatforms
to be part of the sync step instead of being individually ran.

Pass BuildType to sync commands so optdata gets restored correctly

Fix funclet unwinding on x86 Linux (#17144)

JitBench scenarios fix (#17154)

* each web app should use a separate directory to avoid conflicts + dead code removal

* update the framework package version to get scenarios benchmarks working

Merge pull request #17151 from jashook/disable_armlb_windows_jobs

Disable armlb windows jobs

[Arm64] Remove C++ stubs obscurring assembly versions (#17172)

Don't require all-user permissions on the temp directory for named mutexes on Unix under docker (#17206)

Avoids the need for a workaround for one of the issues seen in https://github.com/dotnet/coreclr/issues/17098

build-test.sh: support Clang v4.0 to v6.0, fix always false logic condition preventing tests build (#17025)

Vectorize Span.IndexOf for T = char, similar to T = byte. (#28464) (#17218)

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

Fixed a few typos (#17217)

Update CoreFx to preview3-26326-01 (#17216)

Fixed a few typos (#17215)

* Fixed a few typos

* Fixed a few typos

Update use of AsBytes in perf tests to use MemoryMarshal.AsBytes. (#17214)

Update CoreFx to preview3-26325-01 (#17190)

Dictionary/List code clean up/formatting (C#7) (#17196)

Fix CompareTo/Equals when dealing with Empty Span or Span wrapping a null string (#17115)

* Fix CompareTo/Equals when dealing with Span wrapping a null string

* Removing unnecessary virtual keyword and address other feedback.

* Remove unnecessary/incorrect Debug.Asserts.

* Remove more unnecessary/incorrect Debug.Asserts.

Merge pull request #17201 from BruceForstall/FixDiffable

Fix a case of non-diffable JitDump

Fix Unreachable Code warning in desktop build (#17203)

Fix a case of non-diffable JitDump

Merge pull request #17195 from sdmaclea/PR-Arm64-Assert-on-NYI

[Arm64] Assert on NYI

[Arm64] Assert on NYI

Rename CancellationTokenRegistration.TryDeregister method to Unregister. (#17194)

Reflects CoreRT change https://github.com/dotnet/corert/pull/5612.

Delete some unused functions (#17193)

Merge pull request #17159 from sandreenko/changeTimeOutForAltJitjobs

increase timeout for altjit jobs

Delete unused files from src/inc (#17186)

[RyuJit/ARM] Fix lsra BuildShiftRotate (#17103)

* add repro

* delete BuildShiftRotate for arm

* fix GT_LSH_HI and GT_RSH_LO

* return the special handling for GT_LSH_HI and GT_RSH_LO

* fix the header

Update CoreFx to preview3-26323-09 (#17183)

Generate native image for System.Private.CoreLib.dll during Linux/arm build.sh (#17157)

* Add generating native image for System.Private.CoreLib.dll during Linux/arm cross build when __CrossArch/__BuildArch is x86/arm

Merge pull request #17155 from fiigii/removezero

Remove non-deterministic AVX APIs

 [arm32] Disable JIT\jit64\opt\cse\hugeexpr1 stress testing. (#17137)

It fails R2R stress testing, issue 16587

Merge pull request #17072 from CarolEidt/Fix545505

ARM: call compRsvdRegCheck later

Update DockerTag to use the image from dotnet-buildtools-prereqs-docker#38 which enables running x86_arm/crossgen

Merge pull request #17133 from sdmaclea/PR-FIX-17095

[Arm64] Fix #17095

Improve Windows test build performance by building projects in groups

Merge pull request #17136 from weshaggard/OptDataSouceBuildFixes

Fix source-build issues in optdata

increase timeout for altjit jobs

Remove non-deterministic AVX APIs

Merge pull request #17083 from BruceForstall/RemoveArmDeadCode

Remove arm32/arm64 dead code