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>

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)

null terminate the converted LPStr at the right index (#17079)

Add missing license header to test py and cs files (#17102)

Issue #13471

Moving Span APIs that allow skipping visibility checks to MemoryMarshal (#17087)

Small tweaks to Dict asm size (#17096)

Fixed #16564: the test objmonhelper hangs forever with GCStress=0x3

Change directory in AgentTools script

Update BuildTools, CoreClr, CoreFx to preview3-02620-10, preview3-26321-06, preview3-26319-04, respectively (#17012)

Avoid allocating when SuppressFlow'ing in default ExecutionContext (#17081)

* Avoid allocating when SuppressFlow'ing in default ExecutionContext

* Address PR feedback

API consistency for System.Memory APIs (argument names) (#28160)

* AsBytes and Cast: Change arg name source -> span

* CopyTo: Change arg name array/sequence -> source

* Several APIs: Change arg name arraySegment -> segment

* Several APIs: Change arg name readOnlyMemory -> memory

* BinaryPrimitives and Utf8Formatter: Change arg name buffer -> source or destination

* Base64: Change arg name consumed/written -> bytesConsumed/bytesWritten

* Utf8Parser: Change arg name text -> source

* TryGetOwnedMemory: Change arg name ownedMemory -> owner, index -> start

* BuffersExtensions.Write: Change arg name bufferWriter -> writer

* Span Compare APIs: Change arg name first -> span, second/value -> other

* ROSequence.TryGet & BuffersExtensions: Change arg name data -> memory, sequence -> source

* Fix mistake in BinaryPrimitives Writer APIs, source -> destination.

* BufferExtensions.Write: Change arg name source -> span

* SequencePosition.Equals: Change arg name position -> other

* BufferExtensions ToArray: Change arg name source -> sequence

* Rename leftover arraySegment to segment in MemoryMarshal.TryGetArray

* BufferExtensions.Write: Change arg name span -> value

* Contains, StartsWith, EndsWith: Change arg name other -> value

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

Remove arm32/arm64 dead code

Code in `IsContainableImmed()` appears to try to mark as contained
a floating-point 0.0 in comparison instructions. It doesn't work,
as comparison nodes have TYP_INT, and the zero comparison check
doesn't work for 0.0.

Remove the code, and add a tracking issue to implement it correctly.

Updated steps in 'Viewing JIT Dumps' to see optimized code (#17077)

If one follows the current described steps, one won't see the JIT dump for optimized code in the core lib.
This PR adds the necessary step to see optimized code.

Cf. https://github.com/dotnet/coreclr/issues/17065#issuecomment-374772011

Merge pull request #16955 from fiigii/moreavx

Implement more AVX/AVX2 intrinsics

Fix type for lvaCachedGenericContextArgOffs (#14260)

lvaCachedGenericContextArgOffs should be signed since it's initially
assigned with a negative virtual offset. Finally depending on
frame pointer use with different arch, it could become positive or negative.
Luckily enough, so far such issue was by-passed since amd64 encoder didn't
care such signness when encoding move offset even though the value is
negative. For arm64, it is typically positive since FP is saved on the
bottom of stack.

ARM: call compRsvdRegCheck later

For RyuJIT backend, the number of locals can change due to decomposition and lowering.
So, determine whether to reserve a register for accessing large state displacements just prior to register allocation.
This requires ensuring that the stack offset for promoted longs is fixed up, as for promoted reg structs on ARM64.
This is all a bit more difficult on LEGACY_BACKEND because the register allocator can change its mind about whether or not it's going to have a frame, so that is left as before.

Port System.Memory performance improvements to Convert.TryFromBase64Chars (#17033)

* Copy over Base64.DecodeFromUtf8 verbatim

* Don't need the InPlace api

* Change input from Utf8 to Utf16

* isFinalBlock will always be true, so constant fold it through

* I only need a true/false result, so get rid of OperationStatus

* Rename variables to reflect Utf8->Utf16 change

* Update comments and some minor cleanup

* Make Convert use Base64 as a fast-path

* Incorporate PR feedback.

* Delete FromBase64_Decode

* Use a single safe stackalloc

* Rename Base64.cs to Convert.Base64.cs

and make its contents a part of the Convert class.

We should eventually move all the Base64-related
stuff into this new file but not as part of this PR.
We want to be able to diff the implementation change
separately from the code movement.

* Thanks VS for the unasked for indent

Fix header inclusion order; avoids use of RCWCache before definition (#17067)

Fix build break

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

Merge pull request #17056 from BruceForstall/Arm64LclFldContainedZero

Allow arm64 to contain zero in GT_STORE_LCL_FLD

JIT: remove boxing for interface call to shared generic struct (#17006)

Improve the jit's ability to optimize a box-interface call sequence
to handle cases where the unboxed entry point requires a method table
argument.

Added two test cases, one from the inspiring issue, and another where
the jit is then able to inline the method.

Closes #16982.

Fix major System.Memory performance regression on .NET Framework x86 (#28179)

* Use Unsafe.AddByteOffset instead of Unsafe.Add where possible

* Use pointer for NUint backing field

* Delete unnecessary unchecked scopes, fixed comments

* Avoid unnecessary overflow checks when converting from IntPtr

* Switch SequenceEqual to IntPtr pointer arithmetic

This is fixing 10x regression introduced by use of NUint

* Delete NUint operators that are no longer necessary

* Fix CoreCLR build break

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

Move CompareInfo.* to shared CoreLib partition (#17062)

Move EncodingTable and CodePageDataItem to System.Text namespace (#17061)

This is the more logical namespace for it and it matches the namespace that these live under in CoreRT.

Implement more AVX/AVX2 intrinsics

Add tests for AVX/AVX2 intrinsics