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)

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

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 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.

Added JIT_Stelem_Ref helper function for Linux (#5390)

Use explicit calling convention for coredistools

coreclr is built with /Gz (stdcall), explicitly use __cdecl calling
convention for calling functions in coredistools

Fix for issue  #5467: [R2R] InvalidCastException in timer tests. (#5478)

We were not using the correct MethodTable pointer when constructing delegate objects to interface methods

Limit exports on debugger modules. (#5442)

Remove obsolete ExecuteAssembly hosting API from coreclr.

Needed to add palrt to dbi instead of trying to export from palrt.

Didn't end up adding the --no-allow-shlib-undefined linker option because Centos
and FreeBSD link with lots of undefined symbols from the C++ runtime libraries.

Merge pull request #5462 from pefoley2/uname_p

Properly handle uname -p output that contains spaces

Merge pull request #5247 from davmason/master

Enable managed heap dumps on Linux

Merge pull request #5382 from davmason/filtering

Add filtering logic to XplatEventLogger

Merge pull request #5432 from RussKeldorph/fix5428

Remove unnecessary dependencies from JIT tests causing R2R failures

Fix failing tests on Ubuntu R2R configuration (#5435)

Fix desktop NGen break caused by CoreCLR change, and add a missing contract

[tfs-changeset: 1610323]

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

Merge changes from TFS

Merge pull request #5436 from joperezr/DisableNativeResourceCompileForManagedCode

Disabling NativeResourceCompile Target for Managed libraries

Properly handle uname -p output that contains spaces

e.g. Intel(R) Xeon(R) CPU E5-2603 0 @ 1.80GHz

Fix readytorun tests asking for user input (#5459)

'readytorun/mainv1' and 'readytorun/mainv2' tests on Linux wait for human
response asking permission to remove write-protected file:

    rm: remove write-protected regular file 'test.dll'?

'rm' command shall be used with '-f' flag to make it non-interactive.

Fix CFI annotation of AMD64 assembler helpers on Unix (#5446)

This change fixes annotations of AMD64 assembler helpers on Unix. The helpers that
don't use RBP as a stack frame were missing setting the RSP as a CFA register and
unwinding through them was failing.
This was hit in R2R scenarios, with the DelayLoad_Helper_Obj not being unwindable.

Avoid calling prestub through wrong MethodDesc (#5439)

While fixing up a call from a Ready-to-Run method, we don't always
have the right implementation MethodDesc for the target of the call.
Thus it is potentially unsafe to DoPrestub using the MethodDesc we have,
causing a test failue (Loader.classloader_regressions_429802_CMain)
in issue #5366. This is fixed by not calling DoPrestub from
ExternalMethodFixupWorker.

Some types were only supposed to be present when FEATURE_COMINTEROP was set.

This change fixes that.

[tfs-changeset: 1610157]

Linux/ARM: Support -O3 optimization level in release build (#5396)

Currently, we can build coreclr with -O1 flag of clang in release
build mode for Linux/ARM. This instruction is to enable -O3 optimization
level of clang on ARM/Linux by fixing the bug of llvm.

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

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

This fixes ARM64 Linux compilation error
```
src/vm/gccover.cpp:1219:14: error: case value evaluates to 3134983650, which cannot be narrowed to type 'int' [-Wc++11-narrowing]
        case INTERRUPT_INSTR_PROTECT_RET:
             ^
src/vm/gccover.h:105:41: note: expanded from macro 'INTERRUPT_INSTR_PROTECT_RET'
```

Merge pull request #5443 from swaroop-sridhar/CDTver

CoreDisTools Package: Make minor version generic

Merge pull request #5402 from AndyAyersMS/InlineAllowCpBlk

Inliner: enable inlining of methods with cpblk

Fix wrong filename for case sensitive filesystem.

CoreDisTools Package: Make minor version generic

Change the version requirement of CoreDisTools package
from 1.0.1-prerelease-00001 to 1.0.1-prerelease-*
so that the test infrastructure picks up the latest
available version.

If only the 1.0.1-prerelease-xxxxx number changes,
there is no change in the library API. There may be
a change in the implementation.

The current need for this change is that we need to pick up
a new version of the CoreDisTools package (with an
implementation fix) for X86 testing.

Introduce additional types to the reference assembly for mscorlib and typeforwards for those type to the right assembly.

[tfs-changeset: 1610086]

Merge pull request #5433 from jashook/abs_single

Disable perf_codequality_math_functions

Disabling NativeResourceCompile Target for Managed libraries

Disable perf_codequality_math_functions

abssingle relies on a certain level of floating point precision and fails if the diff is outside of that precision.

Remove unnecessary dependencies from JIT tests causing R2R failures

b05623 did basically nothing so I deleted it.

I removed a presumably useless VisualBasic attribute and other unnecessary
assembly references from the other two tests.

Fixes #5428

Merge pull request #5411 from pgavlin/RVAStaticsUnix

Disable RVA statics tests outside Windows.

Merge pull request #5417 from kyulee1/fixdetail

Fix Details for ILAsm/ILDasm/ClrJit Binaries

Fixing GUID_NULL. (#5415)

Merge pull request #5356 from swaroop-sridhar/Trap

GCStress: Fix a race-condition

Merge pull request #5394 from erozenfeld/TailCallBug5164

Fix for methods that tail call via helpers