Merge pull request #5589 from adityamandaleeka/sleep_return_type_confusion

Fix confused uses of return value of InternalSleepEx.

Merge pull request #3864 from mikedn/mdarray-init

Add support for multi-dimensional array initialization

Merge pull request #5622 from parjong/fix/issue_5620

Allows the use of IsHFA only when FEATURE_HFA is enabled

Merge pull request #5618 from rahku/CiFix

Renable disabled arm64 R2R tests

Merge pull request #5607 from Maoni0/grow_card

need to account for server GC threads calling grow

Allows the use of IsHFA only when FEATURE_HFA is enabled

Fix problem with virtual memory commit in OOM scenario on Linux (#5609)

A stress tests have uncovered a mysterious issue with virtual memory committing on
Linux. During some stress tests, all of a sudden, segments of GC heap got unmapped
or virtual allocator has reserved a block of memory twice.
It turned out that the issue was caused by a strange behavior of mmap in the OOM case.
When we commit a subrange of previously reserved memory (reservation uses mmap with
PROT_NONE, commit uses mmap with PROT_READ | PROT_WRITE) and the mmap fails due to
OOM, it ends up leaving the whole range that was supposed to be committed completely
unmapped. A later attempt to reserve virtual memory finds this block as a free one
and happily reserves a block in it. But our VM allocator is not aware of this issue.

The fix is to use another way to commit pages in a reserved range - using mprotect
to change their protection from the initial PROT_NONE to PROT_READ | PROT_WRITE.
This way doesn't have the problem. I've verified that it works on OSX too, so
I am switching to that way unconditionally.
Decommit still uses mmap so that we don't have to keep track of dirty pages and
so that the pages are really returned to the system in decommit.

Merge pull request #5601 from Maoni0/av_fix_final

software ww fix

Renable disabled arm64 R2R tests

Merge pull request #5614 from adityamandaleeka/debian_84_badges

Update Debian build status badges/links to 8.4

Update Debian build status badges/links to 8.4.

Fix error in RemoveDuplicateAssemblies task for perf (#5605)

Merge pull request #5586 from adityamandaleeka/osx_strip_debug

Strip debugging symbol table entries on OS X Release builds

Switch to CORINFO_HELP_NEW_MDARR_NONVARARG

need to account for server GC threads calling grow

Merge pull request #5604 from rahku/CiFix

temporarily disable R2R tests for arm64

Address CR feedback

Merge pull request #5495 from lemmaa/disable-nyi-features

[ARM64/Linux] Disable not yet implemented features.

Merge pull request #5361 from wateret/fix-hfa-armsoftfp

[ARM/Linux] Fix arm-softfp calling convention

temporarily disable R2R tests for arm64

Add support for multi-dimensional array initialization

This extends impInitializeArrayIntrinsic to support MD arrays in addition to SD arrays. This includes support for SD arrays which are created via CORINFO_HELP_NEW_MDARR when the lower bound is not specified or known to be 0.

The generated code is similar to the code generated for the SD arrays, for example:
  call CORINFO_HELP_NEW_MDARR
  mov rdx, 0x202AB822050
  lea rcx, bword ptr [rax+32]
  vmovdqu ymm0, qword ptr [rdx]
  vmovdqu qword ptr [rcx], ymm0
  mov r8d, dword ptr [rdx+16]
  mov dword ptr [rcx+16], r8d

Merge pull request #5426 from CarolEidt/IsOne

Add methods to check for integer constants one and minus one

Merge pull request #5597 from dotnet-bot/from-tfs

Merge changes from TFS

Linux/ARM: Revert -O1 to -O3 for Linux/ARM Release Build. (#5598)

We have completed the bug fix of llvm in order to support thread-local
storage(TLS) via  __thread variable with -O3 optimization level of clang.
Let's enable the -O3 optimization level of clange for release build
on Linux/ARM.

Signed-off-by: Geunsik Lim <geunsik.lim@samsung.com>

Fixing perf runs on helix (#5102)

* Fixing issue with perf targets restoring test runtime

* Fixing perf runs on helix

software ww fix

Merge pull request #5587 from dotnet-bot/from-tfs

Merge changes from TFS

Fix build issue buildstatus/Issues/Issues.aspx?iid=802303

Ready2Run opt-out should be under #ifdef since closed Arm64 build does not compile with Ready2Run enabled.

[tfs-changeset: 1611275]

Merge pull request #5581 from AndyAyersMS/InlineReplayNoForceInline

Inliner: allow replay log to override force inlines

Fix backwards compatibility bug: RC2 dbgshim debugging RTM runtime (#5593)

Merge pull request #5543 from briansull/feature-x8-refbuf

ARM64: ABI - Support for using register x8 as the return buffer argument for structs

Merge pull request #5547 from schellap/master

Fix packaging attributes

ASP.NET 5 -> Core (#5591)

Updates tryrun.cmake for ARM64 cross build (#5406)

[ARM64/Linux] Disable not yet implemented features.

This patch temporary disables NYIs of SOS for ARM64 architecture.

Methods to check for integer constants and zero

These are primarily motivated by simplifying the RyuJIT tutorial,
but also make the existing code a bit cleaner.

Fix confused uses of return value of InternalSleepEx.

Merge pull request #5534 from swgillespie/weak-reference-get-generation

Disable two tests that rely on an API not exposed on coreclr

Merge pull request #5573 from dotnet-bot/from-tfs

Merge changes from TFS

Strip debugging symbol table entries on OS X Release builds.

Merge pull request #5577 from kyulee1/r2rci

ARM64: Support R2R Private Run in CI

ARM64: ABI - Support for using register x8 as the return buffer argument for structs

This is feature complete and fixes #4949 but is being checked in disabled as it has dependencies
on some additional work in the VM #4950 (saving and restoring register x8 in thePrestub, etc..)

In target.h
Added defines on ARM64 for the additional argument register used to pass the return buffer
This is register x8 and when it used it is considered the 9th argiment with an arg num of 8 (zero based)
Added new method hasFixedRetBuffReg() that will return true on ARM64 (when this feature is fully enabled)
Added new method theFixedRetBuffReg() that returns register REG_R8 (x8) for ARM64
Added new method theFixedRetBuffArgNum that returns RET_BUFF_ARGNUM (8) for ARM64
Updated isValidIntArgReg to allow for a fixed return buff register

In codegencommon.cpp:
Increase the static size of regArgTab[] by one for the integer registers
This allows us to optionally support having a fixed return buffer registers
We now track the maximum index into regArgTab[] using argMax which is
either the old value of  regState->rsCalleeRegArgCount or thefixedRetBufArgNum()
Corrected the emitAttr size value for 64-bit targets to use EA_PTRSIZE instead of EA_4BYTE

In codegeninterface.cpp: (and other places)
Renamed rsCalleeRegNum to be rsCalleeRegArgCount to better reflect what it represents

In compiler.hpp
Updated genMapIntRegArgNumToRegNum and genMapIntRegNumToRegArgNum to handle
the fixed return buffer register for ARM64

In lclvars.cpp
Updated lvaInitRetBuffArg to handle the fixed return buffer register for ARM64

In morph.cpp
Updated fgMorphArgs to properly handle the fixed return buffer register for ARM64

In regalloc.cpp
Updated the assert check raUpdateRegStateForArg to allow for the fixed return buffer register for ARM64

Updated with changes from code review feedback

ARM64: Support R2R Private Run in CI

Merge pull request #5574 from wtgodbe/debian

Update Debian 8.2 to 8.4 to match coreFX

Fix project and release notes URL to 1.1.0

Merge pull request #5576 from ericstj/serviceable

[master] Update buildtools to add Serviceable to nupkgs

Inliner: allow replay log to override force inlines

ReplayPolicy only impacts discretionary inlines, so could not control
force inline candidates. This has lead to some confusion during inline
studies.

Since most cases of force inline are performance related, allow replay
to override and control these force inline candidates.

This might be useful down the road to study how likely it is that a force
inline directive is either unnecessary or perhaps even detremental to
performance.

Update VS dependency details (#5578)

Disable two tests that rely on an API not exposed on coreclr

Update buildtools to add Serviceable to nupkgs

Update the way the type forwarders are build to make sure they are correctly included in the build

[tfs-changeset: 1611158]

Update Debian 8.2 to 8.4 to match coreFX

Add back internal Array ctor

[tfs-changeset: 1611139]

Update first-class-structs.md

Add a link to Issue #5556, and correct some formatting issues.

Merge pull request #5525 from pgavlin/gh5484

Skip reloads and copies in `gcIsWriteBarrierCandidate`.

Merge pull request #5554 from yizhang82/rtlzeromemory-fix

Use Ntdll!RtlZeroMemory instead of kernel32!RtlZeroMemory to support NanoServer

Merge pull request #5533 from wtgodbe/groovyFXFix

Copy linux builds from the right place

Merge pull request #5480 from rahku/arm64EnableR2R

ARM64: R2R - Implement CreateDictionaryLookupHelper stub

Fix for issue 3146: dotnet build is slower on Linux than Windows (#5524)

Fixing the loader logic on Unix to mimic the behavior on Windows,
in order to be able to successfully load and use R2R images on
Unix where the file extension does not have the ".ni." portion.

Use cached empty arrays for "".ToCharArray (#5504)

Update GC from CoreRT (#5522)

https://github.com/dotnet/corert/tree/master/src/Native/gc 3bd148c2089a781458b99d787395377a25bd23d0

Delegate ABI Test (#5507)

Detects issue #5275

Merge pull request #5539 from JohnChen0/r2r

Add config switch to selectively disable R2R images

Update wrong newline behavior for callipinvoke tests (#5553)

Signed-off-by: Jiyoung Yun <jy910.yun@samsung.com>

Disable HeapDump on Linux if FEATURE_EVENT_TRACE is not set (#5550)

This commit tries to disable heap dump on linux if FEATURE_EVENT_TRACE
is not set in order to fix #5546.

Fix GC references reporting in tailcalls with helpers on Windows (#5538)

This change fixes a problem with reporting a range of exactly 3 continuous
GC references in tailcall with helper. The code was asserting in that case
in debug build and generating incorrect offsets in release build.
The problem was that the offset generation and assert was off by 1. It was
supposed to generate offset of the last reference in the range, but instead
generated offset after the last reference.

Merge pull request #5545 from dotnet-bot/from-tfs

Merge changes from TFS

Merge pull request #5528 from CarolEidt/IsLclVarUpdate

Factor out method to check for a lclVar update tree

Use Ntdll!RtlZeroMemory instead of kernel32!RtlZeroMemory to support Nano Server

Merge pull request #5548 from adityamandaleeka/remove_mmap_ignores_hint

Remove MMAP_IGNORES_HINT

Add config switch to selectively disable R2R images

New config switch ReadyToRunExcludeList can be used to specify a
list of assembly simple names (separated by ';') that cannot use
Ready to Run images.

Factor out method to check for a lclVar update tree

This is functionality currently incorporated in optIsLoopIncrTree(),
but which is also useful for the RyuJIT tutorial.

Remove MMAP_IGNORES_HINT.

Introduce explicity typeforwards to internal types that are needed by WinRT

GenFacades does not correcty create the type forwards for internal types even if there are internalsvisibleto attributes setup.

[tfs-changeset: 1610866]

Fixed issue #5391 "libsosplugin.so getting an RPATH of /usr/lib/llvm on RHEL" (#5482)

Merge pull request #5452 from seanshpark/fixgccover

ARM64/Linux: fix compile error 'cannot be narrowed to type int'

Copy linux builds from the right place

ARM64: R2R - Implement CreateDictionaryLookupHelper stub

Merge pull request #5529 from brianrob/spellcheck

Fix Incorrect URLs in Linux Tracing HOWTO

Add doc on how to building lldb for OS X. (#5520)

Fix incorrect URL.

Skip reloads and copies in `gcIsWriteBarrierCandidate`.

The change is actually in `gcWriteBarrierFormFromTargetAddress`. Because copies/reloads
may be placed inside of address-forming trees and because such operations are
transparent w.r.t. the GC-ness of their operand, these nodes should be skipped during
the check for write barriers.

Fixes #5484.

Merge pull request #5513 from pgavlin/RemoveOldJITPackaging

Remove the deprecated JIT packaging tools.

Merge pull request #5512 from pgavlin/ToplevelPrereleaseLabel

Use top level PreReleaseLabel

Merge pull request #5511 from brianrob/linuxperf

Add Performance Tracing Documentation for Windows and Linux

Add performance tracing documentation for Windows and Linux.

Remove Mono section from  OS X build docs (#5499)

With some minor formatting changes.

Merge pull request #5448 from lemmaa/arm64-test-build-3

[ARM64/Linux] Fix wrong filename for case sensitive filesystem.

Remove the deprecated JIT packaging tools.

Use top level PreReleaseLabel

Some projects were overriding PreReleaseLabel instead of inheriting the
parent one in the top level dir.props. In addition, some projects had
the value set to 'rc4' instead of 'rc3'.

Clean this up so we always use the label in the root dir.props

[ARM64/Linux] Fix 'error: 109 enumeration values not handled in switch' (#5493)

This fixes ARM64 Linux compilation error
```
/home/sjlee/git/coreclr/src/jit/emit.h:988:61: error: 109 enumeration
values not handled in switch: 'IF_NONE', 'IF_LABEL', 'IF_EN9'...
[-Werror,-Wswitch]
```

Linux/ARM: Adding rpath for shared library of exception_handling.pal_sxs.test1 (#5328)

Let's add the 'rpath' in order to fix error happened while
'paltest_pal_sxs_test1' (ELF) of PAL test is loading a shared library.
This 'rpath' statement is derived from the existing files such as
./ToolBox/SOS/Strike/CMakeLists.txt and dlls/mscordbi/CMakeLists.txt.

Ver2:
Apply 'rpath' in case of the Linux platform only

Ver1:
Let's add 'rpath' statement to the Linux platform environment
that we have experienced issue.

Signed-off-by: Geunsik Lim <geunsik.lim@samsung.com>
CC: Mike McLaughlin <mikem@microsoft.com>

Fix allocation failure handling (#5473)

While investigating OOM related issues, I have hit two places where we were
not handling correctly the case when memory allocation failed.
In the StackString::OpenStringBuffer, when the Resize failed to allocate
properly sized buffer, we were returning the small internal buffer instead
of NULL. So the caller expected that the full requested size was prepared
and filled it with data, overwriting memory.
In CorUnix::InternalMapViewOfFile, when we have failed to allocate the
PMAPPED_VIEW_LIST to hold information on the mapping, we have unmapped
the memory again, but we have still returned the address of the now
unmapped mapping. The caller, MapViewOfFileEx, then returned this
address to its caller, since it didn't expect the CorUnix::InternalMapViewOfFile
to set its output parameter in the case of failure.

Fix for #5275 Delegate Invocation On ARM Soft FP (#5362)

Update new struct versioning rule (#5429)

If I understand this document, any change to a struct that invalidates its layout in other versioning bubbles would be considered a breaking change. I believe this would include adding or removing a field or changing the type of a field in an internal struct if that struct is included as a (possible non-public) field in a public struct; just because a struct is internal doesn't mean it's not subject to the new versioning rule.

Merge pull request #5468 from dotnet-bot/from-tfs

Merge changes from TFS

Add more opcodes to x86 stack unwinder (#5491)

Add x86 opcodes 0x01 and 0x03 (ADD instruction) to stack unwinder.
These instructions are generated by C++ compiler as part of the
JIT_StaticFieldAddress_Dynamic method, triggering issue #5464.

Remove all usages of CONTRACTS_FULL from mscorlib (#5454)

* Remove CONTRACTS_FULL from Array
* Remove CONTRACTS_FULL from ArraySortHelper
* Remove CONTRACTS_FULL from ICollection
* Remove CONTRACTS_FULL from IDictionary
* Remove CONTRACTS_FULL from IEnumerable
* Remove CONTRACTS_FULL from IList
* Remove CONTRACTS_FULL from IReadOnlyCollection
* Remove CONTRACTS_FULL from IReadOnlyDictionary
* Remove CONTRACTS_FULL from IReadOnlyList
* Remove CONTRACTS_FULL from the nongeneric collections
* Remove CONTRACTS_FULL from IConvertible and IFormattable
* Remove CONTRACTS_FULL from System.IO
* Remove CONTRACTS_FULL from System.Reflection

Merge pull request #5486 from jashook/x86_cc_fix

Use explicit calling convention for coredistools

Fix encoding of gccover.cpp. (#5458)

This file fails to compile while under codepage 932 (Shift JIS).
This commits removes or replaces the offending chacters with ASCII.