Update VS dependency details (#5578)

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

Fixing an error in PAL_pow that caused it to return an incorrect value for a couple scenarios. (#5399)

* Fixing an error in PAL_pow that caused it to return an incorrect value for a couple scenarios.

* Adding additional PAL_pow tests.

Fix Details for ILAsm/ILDasm/ClrJit Binaries

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

Merge pull request #5409 from gkhanna79/FixDetails

Fix details for CoreCLR binary

Merge pull request #5400 from weshaggard/FixPackageLicenseFiles

Include license files into the packages

Disable RVA statics tests outside Windows.

These tests are not supported in general and are already disabled
for Windows. Disable them for *nix as well.

Fix details for CoreCLR binary

Enable FEATURE_HIJACK for ARM (#5404)

Currently, FEATURE_HIJACK is disabled for ARM, and thus CoreCLR got
stuck if GC is invoked from JITed code.

This commit attempts to port FEATURE_HIJACK from ARM (windows) in order
to fix #5403.

Validate new object construction in Ready to Run (#5398)

Existing Ready to Run implementation doesn't fully validate requests
to create object instances, and allows some invalidate requests (e.g.,
to create an instance of an abstract class). This causes failure of
test case Loader.classloader_generics_Instantiation_Negative_abstract01
in issue #5366.

Include license files into the packages

Add filtering logic to XplatEventLogger

Merge pull request #5386 from briansull/arm32-hfa-fix

Fix for HFA args on ARM32

Add limit to number of generics methods to compile by CrossGen (#5383)

Recursive generic definitions can easily make CrossGen take very
long time to complete. This is the cause of a failure in issue #5366.
This is fixed by limiting the number of methods to compile by CrossGen.

Added additional fix for ARM32 in lvHfaSlots

Improvements to System.Math InternalCall code. (#4847)

* Adding some basic System.Math performance tests.

* Renaming 'floatnative' to 'floatdouble'.

* Removing outdated workarounds in the floatdouble interop code.

* Renaming 'finite.cpp' to 'math.cpp'

* Updating the double-precision math tests.

* Updating PAL_EPSILON to be more precise.

JIT_TailCall helper has an implicit assumption that all tail call arguments live

on the caller's frame. If an argument lives on the caller caller's frame, it may get
overwritten if that frame is reused for the tail call. Therefore, we should always copy
struct parameters if they are passed as arguments to a tail call.

The simple il regression test has a scenario similar to that of the F# repro in #5164.

Closes #5164.

Import `new` multi-dimmensional array using the non-vararg helper (#5313)

* Factor out import of `new` multi-dimmensional array
* Implement import of `new` multi-dimmensional array using the non-vararg helper
* Delete RETURN_TYPE_IS_COMPOSITE inlining observation because of it is never hit.
* Add R2R id for non-vararg helper
* Reformat affected code to follow coding conventions

Merge pull request #5375 from pgavlin/CSECleanup

Remove some lexical-CSE-related code.

GCStress: Fix a race-condition

This change ensures that calls to CORINFO_HELP_STOP_FOR_GC() themselves
are not converted to GC-Stress traps -- thus preventing the race between
the GC thread, and a managed thread under GCStress.

Identification of calls to CORINFO_HELP_STOP_FOR_GC():
Since this is a GCStress only requirement, its not worth special identification in the GcInfo
Since CORINFO_HELP_STOP_FOR_GC() calls are realized as indirect calls by the JIT, we cannot identify
them by address at the time of SprinkleBreakpoints().
So, we actually let the SprinkleBreakpoints() replace the call to CORINFO_HELP_STOP_FOR_GC()
with a trap, and revert it back to the original instruction the first time we hit the trap in
OnGcCoverageInterrupt().

Upside: No change to GCInfo or JIT is necessary
Downside: Need to decode a few bytes on every GCStress interrupt.

Fixes #4794

Merge pull request #5365 from kouvel/NamedMutexTests

Named mutexes: fix handle leak, add stress tests