Fix preventing memory allocation in signal handler (#16485)

There was a subtle bug. When the hardware exception handler returns back
to the signal handler, the exception's CONTEXT record may contain
modified registers and so the changes need to be propagated back to the
signal context. But the recent change #16384 was restoring the signal
context from the originally grabbed context instead of the one that's
pointed to by the exception, which is different.

I have also added a little optimization - the contextRecord that was
added is not needed, since the signalContextRecord can be used as the
initial context record for the exception. So we can save the
contextRecord and also copying to the signalContextRecord from it.

Fix build break with older VS versions (#16522)

Adding a TODO-XArch to `Lowering::ContainCheckHWIntrinsic` to track https://github.com/dotnet/coreclr/issues/16497

Adding the missing function header comments to hwintrinsiccodegenxarch.cpp

AsReadOnlySpan -> AsSpan rename to fix build breaks

Rename string-slicing extension methods (dotnet/corefx#27328)

* Rename string-slicing extension methods

As part of

https://github.com/dotnet/corefx/issues/26894

the api folks have approved renaming AsROSpan and AsROMemory
on string instances to AsSpan and AsMemory (as the "readonly"
is obvious given the read-only nature of the input.)

This puts the renaming in effect. Basically a big search-replace
commit.

* Fix OpenSSL build break

* I see this is going to be a treadmill

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

Merge pull request #16511 from CarolEidt/FixDD541653

Handle a restored double Interval at block boundary

Update BuildTools, CoreClr, CoreFx, PgoData to preview2-02522-02, preview2-26222-07, preview2-26222-04, master-20180222-0036, respectively (#16501)

Handle a restored double Interval at block boundary

During the process of freeing registers that are no longer live at the start of a new block, we may restore a 'previousInterval'. If that is a double (and the freed interval was float), we need to skip the next float.

Add ReadOnlySpan string-like Equals/CompareTo/IndexOf/Contains API with globalization support (#16467)

* Add ReadOnlySpan string-like Equals/CompareTo/IndexOf/Contains API with globalization support

* Address PR feedback.

* Fix unix implementation

Remove most preemptive checks from GetFullPath (#16478)

* Removing the colon block that tosses out paths that are not device path.

Fixes: #26359

* Striping 2nd and 3rd args of PathHelper.Normalize

* Build on Maryam's changes

- Remove all path validity checks outside of
    - Null
    - Embedded null
    - Empty or all spaces (effectively empty)
- Remove PathStartSkip helper
- Use span overloads for StringBuffer usage
- Clean up some comments

* Address feedback

* Tweak to match other AsSpan methods

Merge pull request #16473 from CarolEidt/FixDD541648

LEA on rhs of block copy can't be contained

Add CoreLib helpers for MemoryExtensions.AsSpan/AsMemory(T[], int) (#16505)

* Add CoreLib helpers for MemoryExtensions.AsSpan/AsMemory(T[], int)

(Part of https://github.com/dotnet/corefx/issues/26894)

We intentionally don't have (T[], int] constructor overloads
for Span and Memory. So as not to incur unnecessary argument checks,
we implement this directly in CoreLib and will invoke it from
CoreFx for the fast-Span version.

* Expose an internal array-int constructor instead.

Remove relocations for MethodTable::m_pParentMethodTable for Linux ARM (#15915)

Fix impl of ReadOnlySpan ToLower/ToUpper for Unix. (#16496)

Merge pull request #16475 from briansull/vso-569605

Fix for assert during CSE processing

Remove OwnedFlagBit from TryGetOwnedMemory index

Rename AsReadOnlySpan -> Span

Update BuildTools, CoreClr, CoreFx, PgoData to preview2-02521-06, preview2-26222-01, preview2-26222-01, master-20180221-0048, respectively

Fix Span Debugger Display String to correctly show the string contents (#27338)

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

Adding tests for the SSE3 and SSSE3 hardware intrinsics

Adding support for the SSE3 and SSSE3 hardware intrinsics

Updating the x86 hardware intrinsic test templates to allow more configurability

Fix EventPipe Test Failures Under GCStress (#16494)

* Disable inducedgc test when GCStress is enabled.

* Suppress host contract violation in EventPipe::WalkManagedStackForThread.

Fix StructPacking test for ARM (#16484)

Use RuntimeInformation.ProcessArchitecture instead of
RuntimeInformation.OSArchitecture, as for Windows ARM we normally
run in the WoW layer.

Also, `double` on ARM is 8 byte aligned, but was going down the x86
code path that assumes 4-byte alignment.

Finally, add this test to the Windows ARM and Windows ARM64 test
runs by adding it to the respective tests.lst files.

Fix TryGetOwnedMemory xml docs

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

clang format

Codereview feedback

Show the expected stack trace from a rethrown exception. (#16464)

* Show the expected stack trace from a rethrown exception.

Fix #15780

* Remove now unused methods

- StackTraceArray::AppendSkipLast
- StackTraceElement::PartiallyEqual
- StackTraceElement::PartialAtomicUpdate

Add more benchmarks to JitBench (#16353)

* Add more JitBench Benchmarks

a) JitBench includes several more benchmarks and a refactored runner to make it easier to add more
b) A new JitBench/unofficial_dotnet SDK style build project offers simpler developer workflow at the command line and in VS when working with these Benchmarks
c) Misc build issues with local dotnet build in the test tree have workarounds now, but official CI builds still don't support it
d) run-xunit-perf.py has support to run a specific assembly from the testBinLoc folder intended to handle selecting a binary from the folder of binaries produced in the .Net Core SDK project build process. However until CI build can support building projects that way the xunit-perf support is unused.

whitespace fix

Fix for assert during CSE processing
Assertion failed '!comp->gtTreeHasSideEffects(tree, GTF_PERSISTENT_SIDE_EFFECTS_IN_CSE)'
Fixes VSO 569605

Disable tiered compilation for collectible assemblies (#16437)

Disable tiered compilation for collectible assemblies

- An assembly can be marked collectible with methods not marked as dynamic
- The method desc and code can be deleted and a new method desc reallocated in the same location, code versioning doesn't handle this at the moment

LEA on rhs of block copy can't be contained

We don't usually create an LEA on the rhs of a block copy - we check the type of the indir, and if it's struct we avoid the copy. However, in this case, the rhs was the address of a scalar field within a struct, so the indir was not TYP_STRUCT.

Dial back aggressive checks in Path (#16447)

* Dial back aggressive checks in Path

Aggressive checks are preventing crossplat and extended Windows solutions. This change:

- Doesn't throw on empty paths fro GetDirectoryName, GetPathRoot
- Doesn't consider colon when looking at path segments on Windows
- Moves non-shared code out of PathInternal
- Fix span GetDirectoryName to handle multiple separators

* Comment updates

* Check for valid drive letter with PathRoot
Remove other usages of PathSkip (related to #16311)

Moved extensions.cs to shared folder in coreclr (#16459)

* Moved extensions.cs to shared folder

* added to shared to projItems

* Added new constructor

* Parameter Named Changed

Revert TLS model change of the gCurrentThreadInfo

This change causes crashes due to incorrect TLS variable initialization
on Alpine Linux. The initial-exec model that the variable was modified
to use recently cannot be safely used in shared libraries.

Add TryGetOwnedMemory (#16455)

* Add TryGetOwnedMemory

* Feedback

* spelling

Add CI perf repro script (#16414)

* Add CI perf repro script

There wasn't any good way to reproduce what happens in CI for perf jobs because groovy can't be directly run. As a first step to solving this I created a python script that reproduces the perf leg I needed.

Merge pull request #16408 from fiigii/sse4load

Implement SSE4.1 LoadAlignedVector128NonTemporal

Prevent memory allocation in signal handler (#16384)

* Do not allocate exception for signal from non managed code

If the signal occurs in not managed code we cannot use heap.
We should call signal-safe functions only from signal handler.

Create exception object on stack for checking source of signal.
If signal is from managed code we can use memory allocation to create
persistent exception on heap as copy of volatile exception on stack.

If signal from unmanaged code we do nothing and call base signal handler.

fix https://github.com/dotnet/coreclr/issues/16338

Replace ValueStringBuilder.set_Length with indexer (dotnet/corefx#27274)

Indexer that let's you see and edit the content of the pending string is more efficient and flexible.

Fixes #26643

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

Merge pull request #16436 from brunotag/patch-1

Update UsingYourBuild.md

Update CoreClr to preview2-26220-01 (#16445)

Merge pull request #16444 from mikedn/vec2load

Handle TYP_SIMD8 correctly in genCodeForLclFld

Update CoreClr, PgoData to preview2-26219-05, master-20180219-0044, respectively (#16439)

Handle TYP_SIMD8 correctly in genCodeForLclFld

When loading a TYP_SIMD8 local field movsd should be used, not movups. Unlike ins_Move_Extend, ins_Load does the right thing and it's consistent with indirs.

Inline String->ReadOnlySpan (#16440)

Merge pull request #16422 from CarolEidt/Fix15037

Don't set delayRegFree if no target

Delete workaround for #16197 (#16433)

Update CoreClr, PgoData to preview2-26218-01, master-20180217-0048, respectively (#16430)

Delete some more dead code related to #16304 (#16432)

Update UsingYourBuild.md

amend

Inlinable MulticastDelegate == (#16431)

delete unused mirror files (#16423)

Fix #16402 (#16406)

* Handle matching null binder

* Early out HashMap::Rehash()

Merge pull request #16413 from briansull/more-vso-566984

Fix for assert(sideEffList) in clr\src\jit\optcse.cpp, Line: 2151

Update CoreClr to preview2-26217-01 (#16426)

Update CoreClr to preview2-26216-09 (#16420)

Updating the JIT to take EnableSSE3_4 into account when setting the supported instruction sets

Code Review - Removed unnecessary assert.

Fix whitespace formatting

Added new tree walker helper optHasCSEdefWithSideeffect to determine up front if we have
a nested CSE def with side effects.  The refs counts will be wrong when we start unmarking
some CSE uses and then later abandon the replacement operation.
So now we decide upfront if we have a CSE def with side effects before we call optUnmarkCSEs.

Don't set delayRegFree if no target

The `delayRegFree` flag on a use indicates that it interferes with the target register of the same node. This doesn't make sense if there is no def, and will cause the register to unnecessarily be kept alive until after the next actual reference.
This led to an assert in the verification pass on arm64. Checking and asserting for this case exposed another case for x86/x64.

Fix #15037

Merge pull request #16364 from CarolEidt/FixDD543057a

Correctly handle check for busy double reg

Merge pull request #16393 from BruceForstall/CleanupRunTest

Clean up runtest.cmd

Correctly handle check for busy double reg

Fixes the second case in DevDev bug 543057

Update BuildTools, CoreClr, CoreFx, PgoData to preview2-02516-02, preview2-26214-07, preview2-26216-03, master-20180216-0045, respectively (#16339)

Fix dlStack test (#16410)

delete liveness workarond. (#16416)

Fix regression & enable ClearInitLocals in System.Text.RegularExpressions (#27146)

* Fix bug in RegexWriter ref struct change

* Remove duplicate ValueListBuilder implementation

* Enable ILLinkClearInitLocals

* Rename concrete ValueListBuilder impl to Pop

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

Fix impl of ReadOnlySpan StartsWith/EndsWith for Unix. (#16418)

reenable readytorun\tests\mainv* for arm jitstress (#16417)

Fix coreOverlayDir in Unix test instructions (#16415)

Add ReadOnlySpan string-like ToLower/ToUpper API with globalization support (#16379)

* Add ReadOnlySpan string-like ToLower/ToUpper API with globalization support

* Address PR feedback.

Fix for assert(sideEffList) in clr\src\jit\optcse.cpp, Line: 2151

If we return WALK_ABORT this assert will fire when the side effect list 'keepList' is empty.
With this fix we don't return WALK_ABORT when the keepList is empty, instead we just place
the CSE def into the keepList

Implement SSE4.1 LoadAlignedVector128NonTemporal

address GT_ADDR(GT_HWIntrinsic) nodes

Fix importer and morph with HWIntrinsic SIMD types

Fix typo in GC log (#16398)

Merge pull request #16160 from sdmaclea/PR-ARM64-SIMD-INSERT

[Arm64] Implement Simd.Insert

Implement RuntimeHelpers.PrepareMethod/PrepareDelegate for CoreCLR (#16382)

* Implement RuntimeHelpers.PrepareMethod/PrepareDelegate for CoreCLR

CoreCLR implementation of this method triggers jiting of the given method only.
It does not walk a subset of callgraph to provide CER guarantees because of CERs
are not supported by CoreCLR.

Fixes #15522

Typos (dotnet/corert#5386)

* acces -> access

* accesss -> access

* acknowleges -> acknowledges

* actitivy -> activity

* Additonally -> Additionally

* Activable -> Activatable

* adjustting -> adjusting

* Aggresive -> Aggressive

* agressively -> aggressively

* agressive -> aggressive

* Aleady -> Already

* alernate -> alternate

* aligh -> align

* Alloctor -> Allocator

* allready -> already

* Althought -> Although

* Amonst -> Amongst

* anthing -> anything

* Anthyhing -> Anything

* Aquired -> Acquired

* aquire -> acquire

* Architecure -> Architecture

* Arthimatic -> Arithmetic

* artiface -> artifact

* artifical -> artificial

* Arugment -> Argument

* assigining -> assigning

* atomicly -> atomically

* auxillary -> auxiliary

* avilability -> availability

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

StringComparer Create(culture, CompareOptions) overload (#16334)

* String Create options overlaod

* Minor Change

* Feedback

* validation on options

* obsolete removed

* Implementing Iserializable and removing ignorecase

* HashCode and serialization changes

* made inline

* Space Corrected

sigsegv_handler: handle case when it is called on original stack (#16276)

* CatchHardwareExceptionHolder: use GetCurrentPalThread instead of
InternalGetCurrentThread in IsEnabled method.

InternalGetCurrentThread tries to create pal thread if it doesn't
exist for the current thread. It's unnecessary because in this case
there are no hardware exception handlers for such thread.

Also CatchHardwareExceptionHolder::IsEnable is called from signal
handlers and during pal thread creation non-async-signal-safe
function are called.

* vm/threads: change tls model for gCurrentThreadInfo variable

We should use initial-exec tls model to avoid memory allocations
during first access to this variable because it may ocuur in
signal handlers.

* sigsegv_handler: handle case when it is called on original stack

If sigsegv_handler is called on original stack (for example, if segmentation
fault occurs in native application's thread that hasn't alternate signal stack)
we should call common_signal_handler directly othersize sigsegv_handler's
stackframe will be corrupted.

Adding Argument name to the Exception (#16372)

[RyuJit] Stack level setter (#15597)

* create a new phase: StackLevelSetter

* add repro

* Fix grammar mistakes

* use the default hash

* delete values from the map.

* create gentree::OperIsPutArgStkOrSplit

* fix more comments

* delete an extra condition that is always true

* use GTSTRUCT_2_SPECIAL for PutArgStk

* extract fgUseThrowHelperBlocks

* optimize memory for amd64 and additional checks for x86

* change checks

The previous version was wrong, because morph can call fgAddCodeRef several times for the same instruction during different phases.

* fix comments

* fix genJumpToThrowHlpBlk

* small ref in genJumpToThrowHlpBlk

* fix rebase problems.

* use fgUseThrowHelperBlocks instead of !opts.compDbgCode

* add throwHelperBlocksUsed for throughput.

Clean up runtest.cmd

Don't do NI and LOCK deletion (which is slow) if doing
GenerateLayoutOnly.

Plus, other smaller changes to make the file easier to read,
such as adding more comments.

[armel tizen] Fixed dynamic code allocation and code pitching for ARM (#16352)

* [armel tizen] Fixed dynamic code allocation for ARM

* Added comment

* Used ThumbCodeToDataPointer

* Changed to PCODEToPINSTR

Merge pull request #16367 from briansull/vso-566984

Fix for CSE - Assertion failed '!"Badly linked tree"'

Mark sharedgenerics test as GCStressIncompatible (#16391)

Merge pull request #16380 from kbaladurin/fix-uaf

Histogram: doesn't allocate additional memory to store counts

Fix the fired asset in CompareInfo.IndexOf (#16373)

Implement WaitHandle.SignalAndWait on Unix (#16383)

Part of fix for https://github.com/dotnet/coreclr/issues/10441

Add VSD additional param for DIRECT calls. (#16267)

* add a virtualStubParamInfo as an argument

* try to delete duplicates

expect many failures for arm in lower because we delete: // Change the call type, so we can add the extra indirection here, rather than in codegen

* Revert legacy workaround from lower.

* Fix GetNonStandardAddedArgCount

[Arm64] Additional Insert comments

Historam: doesn't allocate additional memory to store counts

It allows to remove deallocation in destuctor that can lead to uaf
for static Historgam's objects during shutdown.

Fix for issue #13153 - change SetGCConcurrent to TemporaryDisableConcurrentGC for profiler startup path. (#16314)

Handle ELEMENT_TYPE_PTR in ILCodeStream::EmitLDIND/EmitSTIND (#16381)

Fixes #16371.

Fix ARM/ARM64 hijacking in tail calls (#16039)

* Fix ARM/ARM64 hijacking in tail calls

This change fixes an issue that can happen when a function that has tail
calls is hijacked. There are two potential issues:

1. When a function that tail calls another one is hijacked, the LR may be
stored at a different location in the stack frame of the tail call
target.
So just by performing tail call, the hijacked location becomes invalid and
unhijacking would corrupt stack by writing to that location.

2. There is a small window after the caller pops LR from the stack in its
epilog and before the tail called function pushes LR in its prolog when
the hijacked return address would not be not on the stack and so we would
not be able to unhijack.

The fix is to prevent hijacking of functions that contain tail calls.

* Enable the tailcall hijacking test for ARM64

The test JIT/Methodical/tailcall_v4/hijacking should be passing now on
ARM64.