ARM64: Fix Frame with VarArg

Fixes https://github.com/dotnet/coreclr/issues/4424
JIT homes incoming vararg (8) registers to the top of the stack.
When stack location is computed for local vars, JIT didn't
take it into consideration causing the overlapped stack
offsets for local vars and varargs. The stack offsets of local vars
corresponding to varargs should point to this home area instead of
the newly alloacted slot.

Partly fix the build issue when path to git repository has spaces (for example "d:\z z\coreclr"). (#4388)

The same issue in "d:\z z\coreclr\Tools\versioning.targets"(288): $(IntermediateOutputPath) should be "$(IntermediateOutputPath.TrimEnd('\'))" and all other path should be quoted with " in this line.

Update ParamArrayAttribute.cs (#4442)

A minor doc error (presumably some copy'n'paste issue).

Fixes https://github.com/dotnet/coreclr/issues/4252 (#4384)

Change Join and Concat to access iterator.Current value once. Delegate null checking to StringBuilder.Append

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

Merge changes from TFS

Merge pull request #4398 from sejongoh/vector3_testcase

Vector3 Interop tests

Remove __ConsoleStream from CoreCLR

[tfs-changeset: 1598441]

Fix heap corruption in CPalSynchronizationManager::DoMonitorProcesses (#4454)

The PAL's WaitFor*Object support had a bug where if one waited on a process,
stopped, waiting and closed the process handle before the process exitted, then
later on when the process exitted it would corrupt the heap here:

   LONG CPalSynchronizationManager::DoMonitorProcesses(
       CPalThread * pthrCurrent)
    {
...
        if (lRemovingCount > 0)
        {
...
                // Set process status to PS_DONE
                pNode->pProcLocalData->ps = PS_DONE;

The issue is that the local process data would be delete when the process
handle was closed. But the synchronization manager would still have a reference
to it.

To fix this, I changed the synchronization manager to hold onto the PAL object in
addition to the local process data.

Remove GOLDEN macro. (#4484)

Merge pull request #4481 from swaroop-sridhar/aagc2

GCInfo: Fix code-offset normalization on ARM

Don't allocate in NullStream.CopyToAsync (#4383)

Vector3 Interop tests

The native type of Vector3 is struct {float x,y,z} whose size is 12 bytes. RyuJit uses 16-byte register or stack location to store a Vector3 variable with the assumptions below. New testcases are added to check whether RyuJit properly handle interop cases.

RyuJIt back-end makes two assumptions about Vector3 types.

Assumption1: Vector3 type args passed in registers or on stack is rounded to POINTER_SIZE and hence on 64-bit targets it can be read/written as if it were TYP_SIMD16.

Assumption2: Vector3 args passed in registers (e.g. unix) or on stack have their upper 4-bytes being zero. Similarly Vector3 return type value returned from a method will have its upper 4-bytes zeroed out.

Merge pull request #4450 from mikem8361/shiminit

Fix dbgshim race conditions

Merge pull request #4486 from wtgodbe/coverage

Move deletion of PAL tests to after they need to be run

Move deletion of PAL tests to after they need to be run

Merge pull request #4444 from wtgodbe/covrTest

Add coverage test to CI that will run once a week

Merge pull request #4480 from Maoni0/enable_vh

enable heap verification for coreclr on all builds

enable heap verification for coreclr on all builds

Merge pull request #4455 from adityamandaleeka/configKnobDoc

Update configuration knob documentation to reflect host configuration options

GCInfo: Fix code-offset normalization on ARM

Fix NORMALIZE_CODE_OFFSET() on ARM to be a loss-less encoding
because safepoint code-offsets are encoded with a -1 adjustment.

Merge pull request #4458 from ramarag/CrossDefinitions

Correcting the definition for crosscomponents

Merge pull request #4399 from erozenfeld/HandleConstantProp

Don't perform constant propagation when relocatable handles are involved

Merge pull request #4451 from sivarv/stopforgc

Model kill set of CORINFO_HELP_STOP_FOR_GC correctly on unix.

Fix dbgshim race conditions

There are two race conditions in the register runtime startup logic: 1) in
CreateProcess (or CreateProcessForLaunch) between the fork() and the execv()
where for VS and mdbg we "see" the coreclr module loaded because the debugger
process and newly created child have the same modules loaded. 2) in attach
between when the child/debuggee loads the coreclr module and when coreclr
finishes initializes the global dac table. In both causes DebugActiveProcess
returns an error when one of the race conditions is hit.

The fix is to create the "continue" named semaphore on the coreclr/debuggee
process side and dbgshim uses that to determine if the coreclr process is
ready and initialized. Fixes issue #4244.

Open the old continue semaphore name and if it doesn't exists create a the new
continue semaphore name for backwards compatibility (see issue 4410).

Do PAL initalization on entry to the public functions instead of the DLLMain handler
which will only be called if the PAL's LoadLibrary is used to load dbgshim. It will be
a lot easier on the debuggers like mdbg to just use dlopen/dlsym and not have to
also call DLLMain.

Add coverage test to CI that will run once a week

Merge pull request #4453 from AndyAyersMS/InlineCleanup

Inliner: restore ability to dump jit time, plus some cleanup

Merge pull request #4472 from kyulee1/delarm64

Delete Non-Windows Arm64 Job

Delete Non-Windows Arm64 Job

Adding Versioning to Native components in non-Windows builds. (#4393)

Adding Versioning to Native components in non-Windows builds.

Merge pull request #4470 from kyulee1/ryutoclr

Package Name Update from RyuJit to ClrJit

Merge pull request #4459 from brianrob/nativeimage_perfmap

Fix empty perf map files generated by crossgen

Name Update from RyuJit to ClrJit for or package

Merge pull request #4469 from kyulee1/ryujit8

RyuJit Version Update

RyuJit Version Update

Merge pull request #4464 from Maoni0/docfix

added missing step in vs install

added missing step in vs install

Fix empty perf map files generated by crossgen.

Correcting the definition for crosscomponents

Update configuration docs to reflect host configuration options.

Merge pull request #4434 from briansull/struct16-abi

ARM64 Work Item 3817, 3524 - Struct16 decomposition

Merge pull request #4449 from pgavlin/JitBuildName

Change the names of the JIT products to `clrjit`.

Inliner: restore ability to dump jit time, plus some cleanup

I zeroed out the jit time measurement when refactoring the data dumping
code to live in the InlineStrategy class, because it required adding a
helper to the Compiler class. Didn't mean to leave it this way, but did.
So, fixing it how I'd meant to all along.

Also, move a few of the inlining related constants and setup computation
over to InlineStrategy. I'd move `DEFAULT_MAX_INLINE_SIZE` too, but it
ends up getting used for some non-inlining purposes, so will hold off on
cleaning that part up.

Don't perform constant propagation when relocatable handles are involved.

This change ensures that value-number-based constant propagation won't
attempt to propagate constants whose value calculations involve handles
when those handles are relocatable and have to be recorded with the VM.

DDR is clean.

4 methods are affected in SuperPMI:

33.09:         Code Size improvements (code sizes in bytes):
33.09:         <filename>                                            baseline        diff improvement  %improvement    #funcs
33.09:         CLR_SH_1_Clean_Thin_Unique_all.dasm                         41          49          -8        -19.51         1
33.09:            1 functions regressed (8 total bytes regression)
33.09:               Regression # 1 goes from    41 to    49, diff of     8 (16.33%) :: <Module>:main():int
33.09:         JIT_SH_Clean_Thin_Unique_all.dasm                         3952        3973         -21         -0.53         9
33.09:            1 aggregated functions improved (4 total bytes improvement)
33.09:               Improvement # 1 goes from  3056 to  3052, diff of     4 ( 0.13%) ::  7 x Exploit:ExploitTarget()
33.09:            2 functions regressed (25 total bytes regression)
33.09:               Regression # 1 goes from    54 to    62, diff of     8 (12.90%) :: <Module>:main():int
33.09:               Regression # 2 goes from   842 to   859, diff of    17 ( 1.98%) :: <Module>:BadFunc(long)
33.09:         TOTAL                                                     3993        4022         -29         -0.73        10

Model kill set of CORINFO_HELP_STOP_FOR_GC correctly on unix.

Merge pull request #4427 from AndyAyersMS/InlineSizeAccumulation

Inliner: estimate overall code size impact

Change the names of the JIT products to `clrjit`.

- The static library to be linked into the CLR is now
  `clrjit_static`
- The static library to be linked into crossgen is now
  `clrjit_crossgen`
- The dynamic library is now `clrjit`

Merge pull request #4443 from jkotas/gitignore

Add *.VC.db to .gitignore

ARM64 Work Item 3817, 3524 - Struct16 decomposition

    Changes to support passing of MultiReg structs using GT_LISTs
    Optional support for  struct promotion for multireg structs:
    To enable set  FEATURE_MULTIREG_STRUCT_PROMOTE to 1
    Morphs the 16-byte structs at the end of fgMorphArgs
    Careful refactoring to avoid changes to UNIX_AMD64 code
    Covers all of the 16-byte struct expansion cases in fgMorph
    Added function header comments
    Passing the tests for Arm64
    No AsmDiffs for non-Arm64 targets.
    Codegen uses Contained nodes for PUTARG_STK 16-byte stack args
    Created a genPutArgStk method for Arm64
    Updated Tests.lst with 16 additional passing tests

Merge pull request #4425 from kyulee1/cleantest2

ARM64: Another round of clean-up Tests.lst

Inliner: estimate overall code size impact

Accumulate the per-inline size impact estimates into an overall inline
size impact estimate. No size budgeting (yet); this just provides data
for further study.

Merge pull request #4396 from pgavlin/AMD64UnwindHeader

Define the AMD64 unwind codes in their own header.

Merge pull request #4439 from kyulee1/winonly

ARM64: CI triggered by Push for Windows only

Add *.VC.db to .gitignore

Merge pull request #4397 from RussKeldorph/reenable

Reenable tests per #3664

Merge pull request #4416 from RussKeldorph/cs8002

Suppress CS8002 for JIT test projects

Merge pull request #4377 from swaroop-sridhar/aagc1

ARM64: Fix GC encoding settings

Merge pull request #4433 from krytarowski/netbsd-support-69

Fix issue building mscorlib.dll

ARM64: CI triggered by Push for Windows only

Merge pull request #4418 from rahku/arm64EnableCrossgen

Enable running crossgen as part of arm64 build

Merge pull request #3879 from manu-silicon/xplat

Enable XplatEventLogger on all non-Windows platforms

Merge pull request #4405 from yizhang82/delegate_fix

Fix SysV calling convention bug in interop. We need to treat delegate…

Merge pull request #4387 from brianrob/ngenrundown

Add support for ready to run to NGEN rundown

Cast function pointer to void*

Fix issue building mscorlib.dll

/home/kamil/tmp/coreclr/src/vm/win32threadpool.cpp:2672:13: error: no matching function for call to 'FireEtwThreadPoolIOEnqueue'
            FireEtwThreadPoolIOEnqueue((LPOVERLAPPED)waitInfo, Callback, (dwFlag & WAIT_SINGLE_EXECUTION) == 0, reinterpret_cast<void*>(GetClrInstanceId()));
            ^~~~~~~~~~~~~~~~~~~~~~~~~~
/home/kamil/tmp/coreclr/bin/obj/Linux.x64.Debug/src/inc/clretwallmain.h:972:14: note: candidate function not viable: no known conversion from
      'WAITORTIMERCALLBACK' (aka 'void (*)(void *, unsigned char)') to 'const void *' for 2nd argument; take the address of the argument with &
inline ULONG FireEtwThreadPoolIOEnqueue(
             ^
/home/kamil/tmp/coreclr/src/vm/win32threadpool.cpp:3236:13: error: no matching function for call to 'FireEtwThreadPoolIODequeue'
            FireEtwThreadPoolIODequeue(waitInfo, waitInfo->Callback, reinterpret_cast<void*>(GetClrInstanceId()));
            ^~~~~~~~~~~~~~~~~~~~~~~~~~
/home/kamil/tmp/coreclr/bin/obj/Linux.x64.Debug/src/inc/clretwallmain.h:985:14: note: candidate function not viable: no known conversion from
      'WAITORTIMERCALLBACK' (aka 'void (*)(void *, unsigned char)') to 'const void *' for 2nd argument; take the address of the argument with &
inline ULONG FireEtwThreadPoolIODequeue(
             ^

Reported on Ubuntu-14.04.2 LTS with clang 3.8.0

Merge pull request #4423 from Priya91/osxcorefx

Reflect osx innerloop changes from corefx.

ARM64: Another round of clean-up Tests.lst

- Removed UNREACHED category completely
- Tag more issues to failure cases

Reflect osx innerloop changes from corefx.

Merge pull request #4414 from kyulee1/cleantest

ARM64: Clean Up Tests.lst

Enable running crossgen as part of arm64 build

ARM64: Fix GC encoding settings

This change fixes certain settings in the GCInfo encoder/decoders.
It also fixes the debug info for the stack pointer.

ARM64: Clean Up Tests.lst

- Remove all the issue tags that are resolved, or correct the issue number.
- Try to move UNSTABLE to either EXPECTED_PASS or EXPECTED_FAIL.
- Delete security transparency tests unavailable in .NET Core -- NonVirtualCall/OverwriterArray

Merge pull request #4415 from mmitche/update-ubuntu-deps

Update coreclr dependencies

Suppress CS8002 for JIT test projects

"Referenced assembly 'Foo, ...' does not have a strong name."

Happens when a .csproj has a ProjectReference for a .ilproj.  It's not
particularly interesting to give these test assemblies strong names, so
I'm just suppressing the warning.

Fixes #4232
Fixes #4233
Fixes #4234
Fixes #4235
Fixes #4236
Fixes #4237

Update coreclr dependencies

Fix SysV calling convention bug in interop. We need to treat delegate field (NFT_DELEGATE) as INTEGER (as per SysV spec) since it is marshaled as a function pointer, otherwise we would end up passing it via stack which would cause all sorts of badness. Same for SAFEHANDLE/CRITICALHANDLE. It's not clear to me what the right semantics for DATE/CURRENCY is but they are Windows-only types, so we probably don't care yet.
This fixes MarshalStructAsLayoutSeq test on mac/linux.

Merge pull request #4374 from AndyAyersMS/CheckToolRestore

Detect if powershell version is suitable for restoring build tools

Enable XplatEventLogger on all non-Windows platforms

Because on non-Windows platform, we assume event tracing, we also requires
it to compile native code for non-Windows platform.
Event tracing is only enabled on AMD64 devices for now on Unix platforms.

Add support for ready to run to NGEN rundown.

Merge pull request #4349 from rahku/master

enable build of cross target components.
This enables the build of following cross target components:
 1. crossgen
 2. mscordacore.dll
 3. sos.dll
 4. mscordbi.dll
 This is currently only enabled for arm64 and generates x64 binaries. For cross builds it uses the same hierarchy of cmake files from source root folder. At the end of build, build calls cmake again to generate solution for cross build specifying both host & target arch. This PR adds host & target arch configuration in cmake. I decided to reuse CLR_CMAKE_PLATFORM_ARCH_* to be as the host arch and CLR_CMAKE_TARGET_ARCH_* to be as target arch. For non-cross builds they are one and same and differ in cross builds. I have also refactored root CMakeLists.txt into smaller files. Binaries for cross builds are placed at bin\product<OS.Arch.Type><hostArch> (eg for arm64 they will be at bin\product\OS.arm64.debug\x64)

enable build of cross target components.
Currently only enabled for arm64

Merge pull request #3915 from hughbe/region-info-params

Add parameter names to RegionInfo constructor

Merge pull request #3535 from hughbe/patch-3

Add parameter names to ArgumentExceptions in Guid

Merge pull request #4247 from leemgs/upstream-cross-toolchain-unify2

Linux/ARM: Unifying the redundant usages of objcopy

Merge pull request #4283 from krytarowski/netbsd-support-61

Implement GetEntrypointExecutableAbsolutePath() on NetBSD

Merge pull request #4371 from krytarowski/netbsd-support-65

Add NetBSD support in dir.props

Merge pull request #4348 from AndyAyersMS/FixPerfMapAV

Fix a segfault in perf map generation

Merge pull request #4373 from krytarowski/netbsd-support-67

NetBSD: Assume GNU GCC compatible va_list in Clang/LLVM

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

Merge changes from TFS

Merge pull request #4312 from mjsabby/muks/enable_rejit_xplat

Enable ReJIT on all platforms

Merge pull request #4392 from kyulee1/fixround

ARM64: Fix Round Operation

ARM64: Fix Round Operation

Math.Round is implemented as a target intrinsic for Arm64.
JIT emitted `frinta` which rounds the ties to away.
As described in MSDN
https://msdn.microsoft.com/en-us/library/system.math.round(v=vs.110).aspx#Round2_Example,
we should round the ties to even.
So, I corrected `frintn` instruction instead for such intrinsic expansion.

I've also identified that `EvalMathFuncUnary` incorrectly folds the round
operation when the argument is constant at the compile time.
The logic itself is not the right semantic as described above in MSDN.
I made a separate helper function which is essentially a duplicate of
classlib. This is not Arm64 specific fix but applies to all other targets.
So, I just fixed it while I'm here.

Fix nidump to avoid crashing while working with Ready to Run images

[tfs-changeset: 1597064]

Merge pull request #4394 from gkhanna79/VCRTArm64Fix

Fix Arm64 break

Merge pull request #4391 from RussKeldorph/buildarch

Build break: Replace $(Platform) with $(BuildArch) in arch-specific IL projects

Define the AMD64 unwind codes in their own header.

This matches the CFI unwind codes and allows the JIT to be more
precise about its includes.

Reenable tests

Merge pull request #4390 from pgavlin/JitCleanup

Cleanup some minor JIT issues.

Fix Arm64 break

Merge pull request #4370 from krytarowski/netbsd-support-64

Print message about detected NetBSD

Merge pull request #4378 from krytarowski/netbsd-support-68

Implement GetProcessIdDisambiguationKey() for NetBSD

Merge pull request #4323 from BruceForstall/CLRABI

Add CLR ABI document to Book of the Runtime