Update PgoData to release-20180720-0153 (#19055)

Merge pull request #19028 from dotnet-maestro-bot/release/2.1-UpdateDependencies

Update CoreClr, CoreFx to servicing-26719-02, servicing-26719-03, respectively (release/2.1)

Update CoreClr, CoreFx to servicing-26719-02, servicing-26719-03, respectively

Update CoreClr, CoreFx, PgoData to servicing-26718-03, servicing-26718-03, release-20180719-0105, respectively (#19014)

Update PgoData to release-20180718-0150 (#18980)

Merge pull request #18955 from dotnet-maestro-bot/release/2.1-UpdateDependencies

Update PgoData to release-20180717-0134 (release/2.1)

Update PgoData to release-20180717-0134

Update PgoData to release-20180716-0253 (#18931)

Update PgoData to release-20180715-0216 (#18921)

Update PgoData to release-20180714-0608 (#18909)

Update PgoData to release-20180713-0043 (#18899)

Skip old time zones adjustments with offsets higher than 14h (#18305) (#18874)

* Skip old time zones with offsets higher than 14h

* apply offline feedback

* fix braces

Update PgoData to release-20180712-0122 (#18883)

Merge pull request #18811 from dotnet-maestro-bot/release/2.1-UpdateDependencies

Update BuildTools, CoreClr, CoreFx, PgoData to rc1-03006-01, servicing-26708-02, servicing-26708-02, release-20180711-0041, respectively (release/2.1)

Update BuildTools, CoreClr, CoreFx, PgoData to rc1-03006-01, servicing-26708-02, servicing-26708-02, release-20180711-0041, respectively

Merge commit 'c6e7fbc5516842945b9b375f7724438dece10627' into release/2.1

Merge pull request #18757 from CarolEidt/Fix17969in2.1

Port the fix for Issue 17969 to Rel/2.1

Update BuildTools, CoreClr, CoreFx, PgoData to rc1-02913-01, servicing-26606-02, servicing-26703-08, release-20180706-0055, respectively

Port the fix for Issue 17969 to Rel/2.1

Port to 2.1: Fix alternate stack cleanup on MUSL (#18687)

The MUSL implementation of sigaltstack checks that the ss.ss_size is
larger or equal than the MINSIGSTKSZ even when the ss_flags is set
to SS_DISABLE even though Linux man page for sigaltstack states that
when this flag is set, all other ss fields are ignored.

We were not setting the ss_size in this case and it was causing a memory
leak for each thread that has terminated on MUSL based Linux distros
like Alpine.

Glibc implementation doesn't check the ss_size when the SS_DISABLE is set
so the problem was really MUSL specific.

Merge 'v2.1.1'

.NET Core 2.1.1

Port to 2.1: Fix alternate stack cleanup on MUSL (#18687)

The MUSL implementation of sigaltstack checks that the ss.ss_size is
larger or equal than the MINSIGSTKSZ even when the ss_flags is set
to SS_DISABLE even though Linux man page for sigaltstack states that
when this flag is set, all other ss fields are ignored.

We were not setting the ss_size in this case and it was causing a memory
leak for each thread that has terminated on MUSL based Linux distros
like Alpine.

Glibc implementation doesn't check the ss_size when the SS_DISABLE is set
so the problem was really MUSL specific.

JIT: Fix bug in finally cloning caused by unsound callfinally reordering

Port of #18348 to release/2.1

We need to make sure that if we reorder a callfinally during finally cloning
that the callfinally is actually the one being targeted by the last block in
the try range.

Closes #18332. Linked issue has some more detailed notes.

Fix SpanHelpers.ClearWithoutReferences alignment detection (#18222)

Optimize Array.Clear using SpanHelpers (#18101)

Reimplement most of Array.Clear in managed code using Span Clear helpers.

Fixes dotnet/corefx#29848

Fix number formatting exception with empty NumberGroupSizes array (#18221) (#18667)

Allow ILCodeVersion to fallback to default IL (#18502)

* Allow ILCodeVersion to fallback to default IL

For compat with profilers that used our APIs in unexpected ways we can allow
the ILCodeVersion to fallback to the default IL code when no IL was explicitly
given.

* Fix incorrect usage of ILCodeVersion::AsNode (issue #18602)

When the debugger is querying the active rejit IL for an IL method that has not been rejitted it incorrectly creates a VMPTR_ILCodeVersionNode for a code version that shouldn't have one.

AssemblyName parsing with unicode/emoji crashes (#18309)

Fix handling of generating relative path to parent (#18460)

Fixes #30263

Update branding to 2.1.3 (#18651)

mach_absolute_time as the primary clock source on macOS (corefx#30391) (#18505) (#18526)

macOS 10.12+ supports clock_gettime (HAVE_CLOCK_MONOTONIC is defined)
However, mach_absolute_time has better resolution and should be used
as the primary clock source.

Update SDK used to build jitutils to 2.1 RC (#18290)

In anticipation of taking a dependence on netcoreapp2.1 for the
forthcoming PMI work from jitutils.

Also make sure to shut down build servers before cleaning up the SDK.

Update PgoData to release-20180604-0043 (#18275)

Update PgoData to release-20180603-0040 (#18265)

Update PgoData to release-20180602-0037 (#18250)

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

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.

Update g_highest_address and g_lowest_address in StompWriteBarrier(WriteBarrierOp::StompResize) on ARM (#18107)

Use sysconf(_SC_NPROCESSORS_CONF) instead of sysconf(_SC_NPROCESSORS_ONLN) in PAL and GC on ARM and ARM64

Update PgoData to release-20180531-0037 (#18214)

Update PgoData to release-20180530-0051 (#18194)

Update branding to 2.1.1 (#18161)

Merge pull request #17998 from dotnet-maestro-bot/release/2.1-UpdateDependencies

Update CoreClr, CoreFx, PgoData to rtm-26515-07, rtm, release-20180529-0037, respectively (release/2.1)

Update dependencies to rtm

Update CoreClr, CoreFx, PgoData to rtm-26515-07, rtm, release-20180529-0037, respectively

Update CoreClr, CoreFx, PgoData to rtm-26514-02, rtm-26514-02, release-20180514-0049, respectively (#17987)

Merge pull request #17965 from jashook/port_build_test_sh_changes

Port build-test.sh changes from master to release/2.1

Split unix test builds in slices (#17785)

* Split unix test builds in slices

Ports #17161 to linux

* Address review feedback

Merge pull request #17725 from dotnetrt/FixNixTestBuild

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

Update CoreClr, CoreFx to rtm-26508-04, rtm-26508-03, respectively (#17927)

Update CoreClr, CoreFx to rtm-26505-03, rtm-26505-03, respectively (#17892)

Update BuildTools to rc1-02804-05 (#17885)

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

Update CoreClr, CoreFx to rtm-26504-02, rtm-26504-02, respectively (#17883)

Update BuildTools, CoreClr, CoreFx to rc1-02803-01, rtm-26502-03, rtm-26502-02, respectively (#17874)

Merge pull request #17875 from riarenas/riarenas/revert-private-intermediate-containers

Revert "Make blobfeed containers private. (#17792)"

Revert "Make blobfeed containers private. (#17792)"

This reverts commit 4950b038c84c223ddd9fc198dcf5722d46e21552.

Update BuildTools, PgoData to rc1-02801-02, release-20180502-0048, respectively (#17860)

Make blobfeed containers private. (#17792)

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

Update PgoData to release-20180501-0050 (#17849)

Merge pull request #17846 from weshaggard/FixSourceBuild

Switch source build property to DotNetBuildFromSource

Disable GC Coop mode switching during fatal error handling during GC exception (#17710) (#17844)

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

Add portable PDB caching to StackTrace.

This is the mscorlib side of the change.

Switch source build property to DotNetBuildFromSource

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

Move from rc1 to rtm prerelease label

Initialize Compiler::Vector128/256ULongHandle in compInit

Merge pull request #17779 from weshaggard/RemoveAlpineBuilds

[release/2.1] Remove Alpine 3.6 builds

Update ARM32 Cross Build Docker Image (#17819) (#17834)

Update BuildTools, CoreClr, CoreFx, PgoData to rc1-02726-01, rc1-26423-02, rc1-26423-06, release-20180430-0045, respectively (#17696)

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

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 Number.ParseNumber to not assume '\0' at the end of a span (#17808) (#17820)

* 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

Remove Alpine 3.6 builds

The alpine 3.6 builds have been replaced with the more generic
linux-musl builds so removing them.

Make intra-build containers private (#17682)

Make intra-build containers private

Use volatile load to read brick table entries (#17718)

Fixes #17716

disable DevDiv_255294 in stress modes. (#17618)

Preserve pinned flag in {ReadOnly}Memory<T>.Slice (dotnet/corefx#29246) (#17712) (#17714)

* Preserve pinned flag in {ReadOnly}Memory<T>.Slice

* Address PR feedback.

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

Port two changes from CoreFX missed by mirror: (#17713)

commit b4d701a72c20b695715371a99b48473053b63250
Author: Ahson Khan <ahkha@microsoft.com>
Date:   Wed Apr 11 13:43:36 2018 -0700

    Add CreateFromPinnedArray to System.Memory ref and add tests (#28992)
   * Fixing bug in Memory.Pin and adding API to uapaot baseline

commit 76e01040fcfdb1c652ef1bf4e8e123c7db4e1be8
Author: Ahson Khan <ahkha@microsoft.com>
Date:   Mon Apr 16 01:54:54 2018 -0700
    Update xml comment for {ReadOnly}Memory.Pin method (#29137)

fix DevDiv_601045 assert (#17685)

Merge pull request #17489 from dotnet-maestro-bot/release/2.1-UpdateDependencies

Update BuildTools, CoreClr, CoreFx, PgoData to rc1-02719-01, rc1-26419-03, rc1-26419-03, release-20180419-0038, respectively (release/2.1)

Merge pull request #17675 from weshaggard/addlinuxmusl21

[release/2.1] Add linux-musl build leg

Fix reading Time zone rules using Julian days (#17672)

Remove debug/checked builds (#17663)

The debug/checked builds don't publish and aren't submitted for testing currently.  To reduce resource usage during the build, do not build these pipelines for now.

Update BuildTools, CoreClr, CoreFx, PgoData to rc1-02719-01, rc1-26419-03, rc1-26419-03, release-20180419-0038, respectively

Fix OutputRID for linux-musl official build

Need to include the `-x64` in the output rid.

Add linux-musl build leg

Skip container creation (#17655)

Skip container creation if not in flatcontainer mode
Container creation isn't required and would be incorrect if the ExpectedFeedUrl's account name didn't match AccountName

Merge pull request #17615 from CarolEidt/Port17575To2.1

[Arm64] Disable SIMD in crossgen (added as part of #14633)

Merge pull request #17616 from CarolEidt/Port17560To2.1

Fix extra register-dependency on mem-form cvtsd/s2ss

Fix GC tests. (#17594) (#17613)

* Fix dlbigleak

* cleanup dlstack

* Fix doublinknoleak

* Fix doublinkstay

* Fix dlcollect

* Fix doublinkgen

* Fix dlbigleakthd

Fix extra register-dependency on mem-form cvtsd/s2ss

[Arm64] Disable SIMD in crossgen (added as part of #14633)

Merge pull request #17599 from BruceForstall/CleanupUbunuArmBuildMachine_2.1

Cleanup ubuntu arm build machine 2.1

Merge pull request #17586 from RussKeldorph/21last

Merge master into release/2.1

Clean up Ubuntu arm build machines

Set prerelease label to rc1

Merge commit 'master' into release/2.1

Update CoreClr, CoreFx, PgoData to preview3-26416-01, preview3-26416-01, master-20180415-0051, respectively (#17540)

[Arm64] Add full barrier after locking operations (#17567)

Merge pull request #17569 from dotnet/dev/unix_test_workflow

Don't use runtesttilstable script for Ubuntu arm jobs

Avoid creating illegal byref pointers (#17524)

Byref pointers need to point within their "host" object -- thus
the alternate name "interior pointers". If the JIT creates and
reports a pointer as a "byref", but it points outside the host
object, and a GC occurs that moves the host object, the byref
pointer will not be updated. If a subsequent calculation puts
the byref "back" into the host object, it will actually be pointing
to garbage, since the host object has moved.

This occurred on ARM with array index calculations, in particular
because ARM doesn't have a single-instruction "base + scale*index + offset"
addressing mode. Thus, we were generating, for the jaggedarr_cs_do
test case, `ProcessJagged3DArray()` function:
```
// r0 = array object, r6 = computed index offset. We mark r4 as a byref.
add r4, r0, r6

// r4 - 32 is the offset of the object we care about. Then we load the array element.
// In this case, the loaded element is a gcref, so r4 becomes a gcref.
ldr r4, [r4-32]
```
We get this math because the user code uses `a[i - 10]`, which is
essentially `a + (i - 10) * 4 + 8` for element size 4. This is optimized
to `a + i * 4 - 32`. In the above code, `r6` is `i * 4`. In this case,
after the first instruction, `r4` can point beyond the array.
If a GC happens, `r4` isn't updated, and the second instruction loads garbage.

There are several fixes:
1. Change array morphing in `fgMorphArrayIndex()` to rearrange the array index
IR node creation to only create a byref pointer that is precise; don't create
"intermediate" byref pointers that don't represent the actual array element
address being computed. The tree matching code that annotates the generated tree
with field sequences needs to be updated to match the new form.
2. Change `fgMoveOpsLeft()` to prevent the left-weighted reassociation optimization
`[byref]+ (ref, [int]+ (int, int)) => [byref]+ ([byref]+ (ref, int), int)`. This
optimization creates "incorrect" byrefs that don't necessarily point within
the host object.
3. Add an additional condition to the `Fold "((x+icon1)+icon2) to (x+(icon1+icon2))"`
morph optimization to prevent merging of constant TYP_REF nodes, which now were
being recognized due to different tree shapes. This was probably always a problem,
but the particular tree shape wasn't seen before.

These fixes are all-platform. However, to reduce risk at this point, the are
enabled for ARM only, under the `FEATURE_PREVENT_BAD_BYREFS` `#ifdef`.

Fixes #17517.

There are many, many diffs.

For ARM32 ngen-based desktop asm diffs, it is a 0.30% improvement across all
framework assemblies. A lot of the diffs seem to be because we CSE the entire
array address offset expression, not just the index expression.