Merge pull request #17246 from debayang/clr_jitstress_arm64

[ARM64] Fix UnspillReg instruction generation

[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.

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

[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

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

Update CoreClr, CoreFx to preview3-26323-05, preview3-26323-06, respectively (#17146)

Fix Path.GetTempFileName on Windows (#17148)

GetTempFileNameW doesn't return the required length of the buffer; it returns the unique number used in the temporary file name.  But the calling code is using that result as a length, resulting in sometimes truncated paths, and sometimes paths that aren't at all related to the original due to state left over in a pooled buffer.

Disable armlb windows jobs

Fixes StringBuilder unbounded size growth in Clear() when we use a mix of Append and Insert (#16926)

* Fixing Clear infinitely growing when we combine usage of Insert and Append. Fixes #27625

* Move debug code to StringBuilder.Debug.cs and Applied code review feedback

* Adding missing debug condition check

* Adding comments and moving Condition on projitems after filename

* Moving the infinite capacity growth fix to Length setter

* Removing originalCapacity and Debug.Assert

* Applying PR feedbacks

* Minor cleanup

* simplifying to single loop

* keeping just one method for ShowChunks function

Merge pull request #17135 from CarolEidt/DevDiv544985

ARM: correctly unassign constant double

Fix RestoreSources for optdata packages

Pass unprocessed args to restore optdata step so it
can override the sources when doing restore

Handle prority as an unprocessed arg for the sync alias

Rename new Stream.Read/Write{Async} Span/Memory source/Destination arguments to buffer (#17141)

* changed to buffer

* More Common changes

* fixed

* another fix

Fix Linux/x86 FuncEvalHijack stack alignment (#17143)

Update BuildTools, CoreClr, CoreFx to preview3-02622-03, preview3-26322-08, preview3-26323-01, respectively (#17123)

Merge pull request #17131 from BredPet/fix_relocation

Fix for #17089

[Arm64] Fix #17095

ARM: correctly unassign constant double

When a `RefTypeFixedReg` is encountered for a floating point register, if it is currently holding a double constant, we need to free both halves - but the current register be either half.

Fix bug for #17089

Signed-off-by: Petr Bred <bredpetr@gmail.com>

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

Merge changes from TFS

Merge pull request #16849 from MattGal/use-dotnetcore-build

Move windows agents to DotNetCore-Build

Merge pull request #17112 from briansull/rollback-14481

Rollback Stack probing change - PR #14481

fix order of Serialize for linux build (#17119)

Fix for #17116. The build would work fine for dynamic events on Windows, but on Linux clang always takes the overloads with <Head, Tail...> until there are no arguments, then complains about no overload takes 0 arguments.

Move windows agents to DotNetCore-Build to see if we can build in VS2017-only environment

Add emitIns_AR_R_I for vextracti/f128

fix Set tests on 32-bit platforms

Implement SetAllVector256

Implement AVX SetVector256

Optimize AVX Insert/Extract intrinsics

Add more tests for AVX Insert/Extract intrinsics

[ARM64] Fix UMEntryThunkCode::Poison method (#17122)

Fix hex value of the instruction ldp x16, x0, [x12]

Update BuildTools, CoreClr to preview3-02622-01, preview3-26322-01, respectively (#17105)

Fix incorrect array dereference. (#17113)

Update CODE_OWNERS.TXT (#17114)

Updating owner on the profiler

Rename {Try}Read/WriteMachineEndian to just {Try}Read/Write (#17106)

Rollback Stack probing change - PR #14481

Fix release build

Remove dependency on libunwind (#17094)

* Add original source of libunwind 1.3-stable

Taken from:
https://github.com/libunwind/libunwind/tree/v1.3-stable

* Enable building of libunwind

This change enables building libunwind and removes
dependency on the libunwind package for all Unixes
except OSX. In OSX the libunwind is part of the OS
and also has support for compact unwind info that
is OSX specific, so we keep using that one.

* Disable warnings in code we don't execute

Disable legacy_backend jits

[tfs-changeset: 1692827]

Update debugging doc on lldb versions and disabling managed debugging (#17107)

Remove REG_OPT_RSVD from LSRA's candidate masks if it's reserved.

Improve DateTime{Offset} "r" and "o" formatting performance (#17092)

Two main changes:
1. Rewrote the formatting to use span, only to then discover that we already had almost exactly the same implementation in Utf8Formatter.  As that one had some extra optimizations around JIT behaviors, I ported that over instead.
2. Avoided [ThreadStatic] lookups unless necessary.

ToString/TryFormat for "o"/"O" improve by ~2.5x.

ToString/TryFormat for "r"/"R" improve by ~3x.

Add More Process Information to EventPipe Traces (#17080)

Fix strings from being concatenated (#17040)