Replace sizeof(void*)-like expressions with TARGET_POINTER_SIZE macro

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

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.

Add ReadOnlySpan string-like StartsWith and EndsWith API with globalization support (#16223)

* Add ReadOnlySpan string-like StartsWith API with globalization support

* Add ReadOnlySpan string-like EndsWith API with globalization support

* Fix nits from PR feedback.

* Address PR feedback

* Add unsafe keyword in CompareInfo.Unix methods.

* Address PR feedback and fix signature of Interop.Globalization.EndsWith

* Code formatting cleanup as suggested by VS.

* Revert "Code formatting cleanup as suggested by VS."

This reverts commit 20931fbf1ae113c6acaa3a4b2c0789b46d7ab06b.

* Creating Span specific methods and working around use of IsFastSort

* Code cleanup

* Improve performance by avoiding slicing and fix impl bug

* Remove commented out code and revert some calls to FindString

* Updating checks for Unix to mimic what IsFastSort does.

* Add length checks and include IsFastSort check within the loop

* Fix bounds checks to not include 0x80

* Call correct method when StringComparison is IgnoreCase.

* Update exception type in string compare to remain consistent

Fix clang format warnings

Check for the very uncommon case where when we are replacing a CSE use
and it contains a CSE def which itself has persistent side effects.
For such a case we simply abandon the replacement of the CSE use.

Add Sse2 scalar hardware intrinsics tests

Implement scalar Sse2 hardware intrinsics

Fix registry path for ETW argument lookup. (#16347)

Merge pull request #16344 from brunotag/patch-1

Update UsingYourBuild.md

Merge pull request #16351 from fiigii/fixcont

Remove an incorrect assertion in containment analysis

Merge pull request #15950 from dotnetrt/isafix

On all platforms first member of InstructionSet enum is InstructionSet_ILLEGAL

Fix an incorrect assertion in containment analysis

Fix ambiguous interface method error message (#16341)

Fixes #15864.

Define type initialization semantics for interface instance methods (#16340)

* Define type initialization semantics for interface instance methods

Accessing an instance method on an interface should trigger non-beforefieldinit class constructors (same as it does for valuetypes).

Fixes #15650.

Improve perf of DelegateConstruct (#16345)

Remove a range lookup that's no longer needed.

See related issue #12438.

Fix SSE4.1 encoding

Implement SSE4.2 CompareGreaterThan

Implement SSE4.1 CompareEqual

reenable AVX2 compareEqual.Int64

Merge pull request #16181 from Buyduck/arm-memory-barrier

[RyuJIT/arm32] Support GTF_IND_VOLATILE and GTF_BLK_VOLATILE

Merge pull request #16008 from sdmaclea/PR-ARM64-TEST-SIMD-HWINTRINSIC

[Arm64] Add Simd HWIntrinsic test

Insert contain extract

Merge pull request #16309 from MattGal/eol-some-linuxes

Remove EOL OSes and add new / missing ones

Update UsingYourBuild.md

Apparently it needs a PlatformTarget to be specified as well when targeting x64 architecture

JIT: Allow struct promotion in GS frames (#16220)

Enable struct promotion in methods where the jit is also emtting a stack guard.
Not allowing promotion seems like an unnecessary limitation and leads to poor
performance in methods with both Span and stackalloc.

Closes #16197.

Use DateTime.ToFileTimeUtc to generate serializable DateTime for TraceLogging events. (#16317)

Add TestInsert

Improve diagnostics

[Arm64] Add Simd HWIntrinsic test

JIT: look for escaping byrefs (#16305)

If a byref is passed to a call or used in an assign, assume it escapes and
that subsequent dereferences may be made at arbitrary offsets.

This handles cases where a user takes the address of one field of a struct,
then uses that byref that to access other fields or non-field content within
the struct, and the byref formation and use are split across statements or
across caller-callee contexts.

Closes #16210.

Update CoreClr, CoreFx, PgoData to preview2-26212-01, preview2-26212-01, master-20180212-0035, respectively (#16335)

Improve CMake detection on Windows when not in PATH (#16328)

Port dotnet/corert#5372

curl https://github.com/dotnet/corert/commit/b723f90c611a2c79a0921f95c7299ebb7325eb59.patch | git am -p 3 --directory='src/pal/tools/' --reject

In CMake v10.2, the key `hklm:\SOFTWARE\Kitware` returns:

```powershell
    Hive: HKEY_LOCAL_MACHINE\SOFTWARE\Kitware

Name                           Property
----                           --------
CMake                          InstallDir : C:\Program Files\CMake\
```

with no space after `CMake` and property name `InstallDir`,
instead of `'(default)'`.

Fix stack trace population to get proper source/line info for tier 1 methods (#16302)

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

Update CoreClr, CoreFx to preview2-26211-03, preview2-26211-04, respectively (#16327)

Delete PEFingerPrint.* (#16325)

Unnecessary dummy type on CoreCLR

Update BuildTools, CoreClr to preview2-02511-02, preview2-26210-06, respectively (#16324)

Update CoreClr to preview2-26210-04 (#16321)

Allow jit internal errors to propagate to user visible exception (#16320)

May help users find workarounds for cases where the jits hit some
kind of implementation limit, as in #12480 or #14762.

Update PgoData to master-20180210-0042 (#16319)

Implement certain simple SIMD intrinsics for AVX/AVX2

Add AVX/AVX2 templete-drien tests and enable containment tests

Add new test template

Add test cases for some AVX/AVX2 intrinsics