PR feedback and fix test.

Don't eliminate a cpblk when offsets are different

When `fgMorphCopyBlock()` encounters a cpblk between two different offsets of the same local, if both fields are not recognizable to the JIT (i.e. the FieldSeqNode returned by IsLocalAddrExpr() and/or the gtFieldSeq field of a GT_LCL_FLD is FieldSeqStore::NotAField()), they should not be considered equal (and therefore should not be morphed to a `GT_NOP`).

Fixes #17969

Specify ordering between CrstUniqueStack and CrstReadyToRunEntryPointToMethodDescMap (#17938)

Fix alternate stack for Alpine docker on SELinux (#17936)

For some reason, the Alpine docker container running on a SELinux host maps
heap as RWX. When we allocate alternate stack from the heap, we also
change the protection of the first page to PROT_NONE so that it can
serve as a guard page to catch stack overflow. And when we free the
alternate stack, we restore the protection back to PROT_READ |
PROT_WRITE. The restoration fails in Alpine docker container running on
a SELinux host with EPROT failure and the SELinux log reports that an
attempt to change heap to executable was made. So it looks like the
kernel has added the PERM_EXEC to the permissions we have passed to the
mprotect call. There is a code in the mprotect implementation that can
do that, although I don't fully understand the conditions under which it
happens. This is driven by the VM_MAYEXEC flag in the internal VMA block
structure.
To fix that, I've modified the alternate stack allocation to use mmap /
munmap instead of C heap allocation.

Replace easy Substrings with AsSpan/Slices (#17916)

* Replace several Substrings with AsSpan/Slices

* Address PR feedback

fix semicolon and priorities for new excluded tests (#17934)

Merge pull request #17911 from CarolEidt/Fix16377

Arm64: Pass promoted struct using GT_FIELD_LIST

Improve checking of GTF_CALL flag. (#17857)

The checker ensures that GTF_CALL is set only where
it's needed: on GT_CALL nodes, on GT_INTRINSIC nodes where
intrinsics are implemented by user calls, and on nodes that
have a child with GT_CALL flag set.

Update CoreClr, CoreFx to preview1-26509-05, preview1-26509-05, respectively (#17930)

Reduce allocation in StringBuilder marshaling (#17928)

When marshaling back results for a StringBuilder argument in a P/Invoke:
- for UTF8 it's allocating a new char[], pinning it, and then handing that off to StringBuilder.ReplaceBufferInternal, which itself then allocates a new char[], and all of that happens even if the StringBuilder's char[] is already big enough to handle the results, which is commonly the case due to the StringBuilder having been sized appropriately from the get-go.
- for both Unicode and Ansi, it's similarly allocating a new char[] invariably, even if the existing buffer is already sufficient.

This commit cleans that up.

While ideally we wouldn't use StringBuilders at all in coreclr/corefx for marshaling, we still do in some places, e.g. Dns.GetHostName.  While we should separately fix that to avoid using a StringBuilder at all, it's useful to demonstrate the impact of the change here:
```C#
[Benchmark]
public static string GetHostName() => Dns.GetHostName();
```
on my machine results in:
```
      Method |  Gen 0 | Allocated |
------------ |-------:|----------:|
 Before      | 0.2668 |   1.09 KB |
 After       | 0.1392 |     584 B |
```

Restrict what we archive for perf testing (#17918)

* Restrict what we archive for perf testing

* change all perf legs archival

* Change txt save to *_log.txt

* Reduce number of days to keep artifacts

Update arm/arm64 test lists (#17855)

* update arm64list

* exclude JIT\HardwareIntrinsics\X86 for arm64

We do not want to spend arm64 machine resourses on them.

JIT\HardwareIntrinsics\X86\Sse\LoadAlignedVector128_ro was left enabled to test the failing path.

* update arm test list

* exclude JIT\HardwareIntrinsics\X86 for arm

* exclude failing tests for arm32

* exclude failing tests for arm64

Update BuildTools, CoreClr, CoreFx to preview1-02808-01, preview1-26509-01, preview1-26509-01, respectively (#17923)

Rename mscorlib to System.Private.Corelib (#17926)

* diff from just renaming folder mscorlib to System.Private.CoreLib

* Updating build.proj to reflect name change

Fixes: #17905

Handle SIMD field of GT_FIELD_LIST in codegen.
Add header to test case.

Update DotnetCLIVersion.txt (#17925)

The current version is broken for RHEL. This change revs up the version
to 2.1.300-rtm-008820, which is fixed for RHEL.

Delete irrelevant comment (#17921)

Partial implementation of Thread.GetApartmentState (dotnet/corert#5781)

Related to #5776

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

Reflection: Fix DefaultValue exception with enums (#17917)

This is related to dbcfd2f9d1, but about enums instead of `DateTime`.

Given e.g. a generic method with a parameter `T arg = default(T)`
where `T` is the generic type parameter, if that is instantiated with
some enum as the generic type argument, then querying the default
value of `arg` using `ParameterInfo.[Raw]DefaultValue` will throw a
`FormatException`.

(The use of generics means that the C# compiler always records a
`null` constant in metadata, which isn't usually the case for enums.)

Update CoreClr, CoreFx to preview1-26508-05, preview1-26508-04, respectively (#17913)

PR feedback

Fix failing System.Reflection.Emit tests (#17915)

Commit fceac03e82 removed a test from `TypeBuilder.SetConstantValue`
that is too strict when called from `ParameterBuilder`, but that both
`FieldBuilder` and `EnumBuilder` require. Add the same test back, but
in `FieldBuilder.SetConstant`.

Move ConcurrentExclusiveSchedulerPair to shared CoreLib partition (#17914)

 Remove ConcurrentDictionary from System.Private.CoreLib (#17908)

* Delete stale PRENET45 conditional code

* Remove ConcurrentDictionary dependency from ConcurrentExclusiveSchedulerPair

Speed up Array.Reverse by using ref reassignment (#17891)

* Speed up Array.Reverse by using ref reassignment

* Optimize MemoryExtensions.Reverse

Arm64 debugger step into behavior (#17912)

- Fix #ifdef in StubManagerHelpers::StubGetSecondArg that erroneously referred to TARGET_ARM instead of TARGET_ARM64
- Add condition in StubManagerHelpers::GetTailCallTarget to refer to X12 as is used by the various helpers that can pause in the midst of stepping
  - Fix StubDispatchFixupStub to use X12 as tail call register instead of X9 to match the other tail-calling stubs

Arm64: Pass promoted struct using GT_FIELD_LIST

Merge pull request #17910 from jashook/vc_runtime

Add vcruntime to core_root for arm(64) jobs.

Add vcruntime to core_root for arm(64) jobs.

Update CoreClr, CoreFx to preview1-26507-05, preview1-26507-04, respectively (#17894)

Reflection: Fix DefaultValue for optional DateTime (#17877)

Querying the default value of an optional `DateTime` parameter using
`ParameterInfo.DefaultValue` can throw a `FormatException` if that
default value is a null constant in metadata, which is how the C#
compiler encodes a default value of `default(DateTime)`.

This commit fixes that error by adding the proper handling for null
metadata constants with `DateTime` parameters.

Reflection.Emit: Allow `ParameterBuilder.SetConstant(null)` for value-typed parameters (#17887)

* Add test for dotnet/corefx#26184

The Roslyn C# compiler encodes an optional, value-typed parameter's
default value of `default(TValueType)` as a null reference constant in
metadata. Add a test that verifies that reflection can do the same
using `ParameterBuilder.SetConstant(null)`.

* Always allow ParameterBuilder.SetConstant(null)

* Remove test project as requested in review

These tests move to CoreFX. See dotnet/corefx#29532.

Merge pull request #17847 from dotnet-maestro-bot/master-UpdateDependencies

Update BuildTools, CoreClr, CoreFx to preview1-02804-04, preview1-26506-01, preview1-26506-01, respectively (master)

Rename netcoreapp2.1 -> netcoreapp2.2

Delete unnecessary internal layer of Span globalization helpers (#17890)

This change makes the code both smaller and faster. For example, the following is about 1.4x faster with this change:

```
ReadOnlySpan<char> s1 = "Hello world";
ReadOnlySpan<char> s2 = "world";
for (int i = 0; i < 100000000; i++) s1.EndsWith(s2, StringComparison.OrdinalIgnoreCase);
```

Also, I have ported GetCaseCompareOfComparisonCulture code size optimization from #16434 while I was on it because of it fit well with the rest of the changes.

Update BuildTools, CoreClr, CoreFx to preview1-02804-04, preview1-26506-01, preview1-26506-01, respectively

Remove use of ByReference<T> in Memmove ABI (#17889)

Use new ref reassignment feature instead

disable dialog box for forced unhandled exception (#17888)

Split unix test builds in slices (#17785)

* Split unix test builds in slices

Ports #17161 to linux

* Address review feedback

Fix System.String over-allocation (#17876)

BaseSize for System.String was not set correctly. It caused unnecessary extra 8 bytes to be allocated at the end of strings that had `Length % 4 < 2` on 64-bit platforms.

This change makes affected strings proportionally cheaper. For example, `new string('a', 1)` in a long-running loop is 7% faster.

Step into multicast delegate (#17879)

- Fix call of multicast debugger trace helper to be called before the first delegate invocation instead of after it

Merge pull request #17866 from briansull/byref-fix

[ARM-ARCH] Ensure that any byrefs created in genCodeForIndexAddr cannot point outside of the array

Merge pull request #17781 from BruceForstall/Fix17738

Fix for ARM secure delegate non-standard register arg

[Linux/x86] Fix tests in case of 4-byte alignment for 64-bit types (#17829)

JIT/Directed/RVAInit/nested
JIT/Directed/RVAInit/simple
JIT/Regression/CLR-x86-JIT/V1.2-Beta1/b103058/b103058

Port Marshal UTF8 fixes from CoreRT (#17868)

Merge pull request #17873 from stakx/documentation-corrections

Some corrections in build documentation

Some corrections in build documentation

typo (#17867)

Serialable should have been Serializable
publically should have been publicly

Clarifying comment on overflow (#17865)

* Clarifying comment on overflow

* Nits

Fix for Issue 17823
Ensure that any byrefs created in genCodeForIndexAddr cannot point outside of the array
Used the scaled add instructions for more powers of two
Only use the safe byref sequence when generating fully interruptible code.

Fix faulty assert in Utilities.SelectBucketIndex (#17863)

0 is a valid length.

Delete incorrect comment (#17859)

Rename internal Utf8String to MdUtf8String (#17856)

We want to start prototyping Utf8String in CoreFxLab
and for that, we'll need a bare-bones System.Utf8String
class exposed from System.Private.CoreLib.

Unfortunately, CoreLib already has an internal
struct named System.Utf8String. Since it's only
an internal type, we'll exercise eminent domain
on its name now and get these noise changes out of
the way.

Fix for ARM secure delegate non-standard register arg

For ARM, doing a secure delegate call requires adding
a custom calling convention argument R4 as the address of the
secure delegate invoke indirection cell. This is done using the
fgMorphArgs nonStandardArgs mechanism, and the argument is added
at the end. For calls with 4 or more register arguments, this
didn't work: we would initially set the non-standard arg as a
non-register argument, and the nonStandardArgs check didn't
consider converting an argument from a stack argument back to
a register argument. The fix allows nonStandardArgs to be either
stack or register arguments, no matter what their place in the
argument list would imply.

Fixes #17738

Fix arm test list creator for Windows (#17852)

* use "\n" for splitting lines

* update comment

Merge pull request #17854 from weshaggard/FixSourceBuildM

Switch source build property to DotNetBuildFromSource

remove duplicates

Enable long running CoreFx Ubuntu/arm tests (#17836)

Switch source build property to DotNetBuildFromSource

Detect source-build via DotNetBuildFromSource instead of
DotNetBuildOffline which is set for the tarball build.

Merge pull request #17737 from fiigii/vectortests

Port more Vector tests to Vector128/256<T>

Merge pull request #17845 from briansull/fix-17841

ARM64 like ARM32 also needs a much larger instruction group size

Merge pull request #17848 from CarolEidt/Repro17756

Reenable test for 17756

Update DotnetCLIVersion.txt (#17843)

Reenable test for 17756

Adding back test for windows event log (#17821)

* update test dependencies

* add test

* Allow more leeway for time

* modify test per PR comments

ARM64 like ARM32 also needs a much larger instruction group size
because the prolog can be quite large.

Add runtimeconfig.json support for tiered compilation (#17840)

GC env vars (#17837)

* update server gc export for runtest.sh

* remove CORECLR_SERVER_GC

* Change CORECLR_CONCURRENT_GC to COMPlus_gcConcurrent as well

* Remove it from coreruncommon.cpp

Update CoreClr, CoreFx to preview1-26430-05, preview3-26430-04, respectively (#17832)

Delete unnecessary caches (dotnet/corefx#29392)

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

Update CoreClr, CoreFx to preview1-26428-04, preview3-26428-04, respectively (#17813)

[Linux/x86] Fix 64 bit shift inconsistencies (#17826)

Apply #15443 and #15949 for Linux/x86

Merge pull request #17811 from noahfalk/fix_jitbench

Fix jitbench

Update ARM32 Cross Build Docker Image (#17819)

Update Windows arm32 corefx test failure exclusions (#17803)

* Update Windows arm32 corefx test failure exclusions

All Windows arm32 corefx test jobs should pass with these exclusions.

* Add back Microsoft.Win32.Registry.Tests exclusion

Fix Number.ParseNumber to not assume '\0' at the end of a span (#17808)

* Fix Number.ParseNumber to not assume '\0' at the end of a span

This routine was written for parsing strings, which are implicitly null-terminated, and it doesn't factor in string length but instead uses tricks to exit loops when the next character is null.  Now that the routine is also used for spans, this is very problematic, as spans need not be null terminated, and generally aren't when they represent slices, and expecting a null termination like this can result in walking off the end of valid memory.

I would like to see all of this code rewritten to use span.  In the interim, though, as a short-term fix I've changed all dereferences of the current position to compare against the length of the span (or, rather, a pointer to the end), and pretend that a null terminator was found if we've hit the end.

* Address PR feedback

Add better portable PDB caching to System.Diagnostics.StackTrace. (#17804)

Add portable PDB caching to StackTrace.

This is the mscorlib side of the change.

Merge pull request #17806 from adityamandaleeka/fix_unmanaged_target_reg

Fix register being shifted in PInvoke stub

Merge pull request #17793 from briansull/allocOutArgSpace

Allocate the lvaOutgoingArgSpaceVar early

Use HiddenArg instead of hardcoded x12.

Fix jitbench

Addressed 3 issues:
1) coreclr and CoreFx were out of sync -> update dependencies.props
2) Word2Vec fails on x86 sometimes with OutOfMemory -> disabled it there because it appears the behavior is by design
3) CommandLineParser 2.1.1 doesn't restore anymore? -> NuGet was already rolling forward to 2.2.0 but changing it in the source removes the warning when using dotnet.exe to run JitBench

Update CoreClr, CoreFx to preview1-26427-01, preview3-26427-01, respectively (#17791)

Remove some fgMorphArgs dead code (#17788)

For the arm64 case of a special return buffer argument (always x8),
we don't need the code to check for it, because a few lines above all
non-standard args cause us to "continue", and skip the code.

Also, remove a duplicative #ifdef.

Fix register being shifted in pinvoke stub.

Simplify VectorRet test

Port more Vector tests to Vector128/256

Fix GC test failures (#17766)

* Fix finalizearraysleep and handlecopy gc tests

* revert changes to nstructtun

* forgot to add this fix in

* fix handlecopy

* fix nstructtun

* fix nstructtun

clang tidy

Allocate the lvaOutgoingArgSpaceVar very early, right after the user locals.
This ensures that it gets a low LclVar number so that we don't hit
the IMPL_LIMITATION associated with offsets > 255 for LclVar numbers above 32767

Merge pull request #17694 from adityamandaleeka/fix_struct_passing_arm64

Account for copied bytes in struct copy loop when generating GT_PUTARG_STK code [ARM64]

Update BuildTools, CoreClr, CoreFx, PgoData to preview1-02725-03, preview1-26426-01, preview3-26425-01, master-20180424-0039, respectively (#17749)

Don't optimize away Task code needed for debugger (#17776)

This prevents the IL linker from optimizing away some properties/methods
related to tasks that are used by a debugger but are not referenced
anywhere else in coreclr.

This specifically fixes async callstack frames for the xplat C# debugger.

Fix AsyncLocal<class> changed event to not be raised multiple times for one change in value (#17767)

Fix AsyncLocal<class> changed event to not be raised multiple times for one change in value

Functional fix for https://github.com/dotnet/coreclr/issues/17758

Fix TailCallStress mode to do more legality checks (#17763)

Check with EE whether tail call is allowed before adding
tail. prefix in TailCallStress mode.

Merge pull request #17757 from BruceForstall/MakeArmUbuntuCorefxTestRunClean

Disable all failing Ubuntu arm corefx tests

Patch LTTng ust-tracepoint.h on ARM/Trusty when building crossrootfs. (#17762)

Merge pull request #17747 from weshaggard/RemoveAlpineBuild

Remove Alpine 3.6 builds

Delete unused tests\arm64\fails.testlist (#17745)

* Reenable fixed arm64 tests

* delete the unused file

Merge pull request #17697 from mikedn/vec-hnd

Initialize Compiler::Vector128/256ULongHandle in compInit

Merge pull request #17725 from dotnetrt/FixNixTestBuild

Fix unix test build by removing unnecessary 'managed_test_build' semafore file