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.

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

Fix copy&paste bug (#17059)

Fix unloading of images mapped by PAL (#17053)

* Fix unloading of images mapped by PAL

The m_FileView member of MappedImageLayout is a CLRMapViewHolder,
which runs CLRUnmapViewOfFile on release. This is fine on Windows,
where the MappedImageLayout constructor calls
CLRMapViewOfFile. However, with FEATURE_PAL, the constructor calls
PAL_LOADLoadPEFile, which simulates LoadLibrary and records multiple
mapping entries. Each entry increases the refcount of the underlying
file handle PAL object. PAL_LOADUnloadPEFile should be called in this
case (instead of CLRUnmapViewOfFile), to decrease the refcount for
each mapping entry.

Fixes https://github.com/dotnet/coreclr/issues/15189.

* Fix build failure

Use flat layout for GetAssemblyName (#17052)

* Use flat layout for GetAssemblyName

This change will result in the use flat layouts for the temporary
layouts created by GetAssemblyName. This allows crossgen'd PE images
created for a different OS to be loaded in the code path that
retrieves the assembly name.

* Respond to PR feedback

Add comment, and remove dead code.

Allow arm64 to contain zero in GT_STORE_LCL_FLD

The codegen to handle this in codegenarm64.cpp, CodeGen::genCodeForStoreLclFld()
already exists.

Add new ICLRDebuggingLibraryProvider2 interface for OpenVirtualProces that works on Linux (#16978)

Add new ICLRDebuggingLibraryProvider2 interface for OpenVirtualProcess that works on Linux.

The old ICLRDebuggingLibraryProvider::ProviderLibrary returns module handles that can't
be properly supported on Linux.

ICLRDebuggerLibraryProvider2 returns the module path string instead of a handle. Works on
both Windows and Linux. If ICLRDebuggerLibraryProvider2 is QI'ed, OpenVirtualProcess will
NOT fall back to ICLRDebuggerLibraryProvider if ProviderLibrary2 fails.

HRESULT ProvideLibrary2(
    [in] const WCHAR* pwszFileName,
    [in] DWORD dwTimestamp,
    [in] DWORD dwSizeOfImage,
    [out] LPWSTR* ppResolvedModulePath);

ppResolvedModulePath - Where *ppResolvedModulePath is a null terminated path to the module dll. On Windows it should be allocated
with CoTaskMemAlloc. On Unix it should be allocated with malloc. Failure leave it untouched.

Merge pull request #17049 from briansull/fix-fgInstrumentMethod

Fix for fgInstrumentMethod

JIT: tolerate nonzero constant byrefs in impCheckForNullPointer (#17042)

With the advent of #16966 we may now see constant nonzero byrefs from
things like RVA statics. Tolerate these in `impCheckForNullPointer`.

Note previously we'd type these as ints/longs and so bail out of
`impCheckForNullPointer` after the first check.

Closes #17008.

delete unused debug arg from genChangeLife and compChangeLife. (#17041)

* delete unused debug arg from genChangeLife(VARSET_VALARG_TP

* remove unused debug arg from compChangeLife

Fix jit-format

Remove obselete buildtests (#17050)

BuildTests definition now lives under buildpipeline\tests\

Fix for fgInstrumentMethod

There was a bug creating profiling images using /Tuning
We computed the wrong value for addrOfBlockCount at the end of this method
This would result in method having proper block counts,
but still not getting marked as being executed.

This regressed in PR #14640

Fix ProjectN build break and work around MCG bug (dotnet/corert#5576)

Unblocks integration to TFS.

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

Switch over to managed Marvin implementation for string hashing (#17029)

Adding running tests on Alpine for coreclr build (#17038)

Publish x86_arm/crossgen during Linux/arm build (#16953)

* Publish CrossCrossGen during Linux/arm build
* Update DockerTag to the image that includes two rootfs-s

Fixing the x86 ScalarUnOpTest template to no longer write out of bounds