Merge pull request #3641 from erozenfeld/GetCORSystemDirectoryInternaLFix

Fix for a bug in GetCORSystemDirectoryInternaL

Merge pull request #3640 from rahku/EHTestPort

Fix arm64 build break

Merge pull request #3603 from bendono/doc

Fix spelling and grammar.

Fix for a bug in GetCORSystemDirectoryInternaL

Change 7045ca7a6de381b382a709dc0a61f49ca713d160 introduced a bug in
GetCORSystemDirectoryInternaL. The second parameter of GetRuntimeDirectory
is in/out so passing a pointer to an uninitialized cchBuffer was wrong.
The fix is to initialize cchBuffer to MAX_PATH - 1.

This was causing failures in some non CoreClr ngen scenarios.

Merge pull request #3639 from svick/patch-1

Remove incorrect escaping in example in BOTR

Fix arm64 build break

Merge pull request #3638 from sejongoh/quick_path_fix_for_init_tool_sh

Fix shell command to execute init-tool.sh on CI

Remove incorrect escaping in example in BOTR

The example was casting to `ArrayMethodDesc\*` (note the backslash, possibly there to avoid italics formatting in some previous version?), which doesn't make sense. Now it casts to `ArrayMethodDesc*`.

Also, since I'm already changing this snippet, enable syntax highlighting as C++ for it.

Merge pull request #3589 from dagood/test-dependency-validation

Add dependency validation/tooled-upgrade and make dependencies valid under new rules

Fix shell command to execute init-tool.sh on CI

Fix the centralized GC test project.jsons and other new project.jsons from merge.

Upgrade new test project.json files from merge.

Add documentation for dependency validation.

Add System.Threading.Thread to tests to restore missing types.

Upgrade System.Runtime dependency for JIT tests to get GCHandle.

Change system.runtime 4.0.20 to 4.1.0 rc2 dependency to try to copy the runtimes to CORE_ROOT.

Fix new project.json files.

Fix hardcoded Microsoft.DotNet.xunit.performance version.

Upgrade System.Runtime.InteropServices to prerelease level to avoid type conflicts with System.Runtime.

Update dependencies on packages that don't exist.

First checks to see if a stable version of that dependency was restored, and if not, found the stable.minor.patch version with the prerelease spec of the dependency.

Add working validation rules for xunit packages and apply them.

Remove potentially unused System.Numerics.Vectors.WindowsRuntime reference and spare WindowsRuntime.

Update all versions to validate.

Add validation to test build.

Don't validate generated files, and make the xunit.runner.msbuild workaround only apply to that specific package.

Merge pull request #3629 from leemgs/upstream-crossbuild2

Add comment about disabling crossgen command for cross build

Merge pull request #3394 from swgillespie/gctests

Initial bringup of new set of ported GC tests.

Merge pull request #3489 from rahku/EHTestPort

Port EH tests.

Port EH tests.

Remove unnecessary expressions.

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

Initial bringup of new set of ported GC tests.

Merge pull request #3598 from joperezr/DisableOpenSourceSign

Disable OpenSourceSign and update BuildTools

Disable OpenSourceSign and update Build Tools

Add comment about disabling crossgen command for cross build

As we all know, the current cross build generates exectuable
'crossgen' with arm architecture (without x86 architecture).
So, it can not be run to make mscorlib.ni.dll from mscorlib.dll
on X86-based build machine.
Let's add a comment why we have to skip crossgen command in
build_mscorlib() of build.sh in case of cross build for Linux/ARM.

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

Merge pull request #3599 from ramarag/arm64nativetests

Enable native test binaries to be built for Arm64

Merge pull request #3626 from AndyAyersMS/InlineRefactorFixNames

Fix various naming convention issues in inliner code

Merge pull request #3624 from adityamandaleeka/remove_threadstatic_duplicates

Remove duplicate ThreadStatic tests

Merge pull request #3625 from pgavlin/SupportPackageVersion

Accept a build number arg in BuildRyuJitPackage.

Enable native test binaries to be built for Arm64

Fix various naming convention issues in inliner code

Per the jit coding conventions, rename all class member functions to be
CamelCase. Rename all class data members to begin with `m_`.

Merge pull request #3562 from tijoytom/master

Improve interop test coverage.

Accept a build number arg in BuildRyuJitPackage.

Merge pull request #3602 from AndyAyersMS/InlineRefactorNativeSize

Inline refactoring: move native callsite estimator

Remove duplicate ThreadStatic tests.

Merge branch 'master' of https://github.com/dotnet/coreclr

Inline refactoring: move native callsite estimator

Move the native call site estimation logic into `LegacyPolicy`. Rework
the remaining per-callsite bonus multipliers so that they fit into the
observation framework. Remove `hasNativeSizeEstimate` and instead
check that the inline is a candidate and its observation is a
discretionary inline. Fix header comment in `fgFindJumpTargets`.

skip ci please
Disable test for legacy backend

Merge pull request #3567 from mikem8361/dbgshimcp

Add process launch APIs to dbgshim

Merge pull request #3621 from kyulee1/arm64

Add folks to private run

Add folks to private run

Merge pull request #3595 from janvorli/fix-stack-walk-with-finally

Fix stack walking on Unix in case of finally

Merge pull request #3558 from sejongoh/fix_gc_stress_setup_fail

Fix GC stress setup failure on Unix-like OSes

Fix spelling and grammar.

Added CreateProcessForLaunch, ResumeProcess and CloseResumeHandle that can
be better supported cross-plat.

Merge pull request #3574 from AndyAyersMS/InlineRefactorMoveStateMachineMerge

Inline refactoring: move state machine into LegacyPolicy

Merge pull request #3588 from janvorli/fix-exception-in-finalizer

Fix Unix exception handling in finalizers

Merge pull request #3478 from kyulee1/var

Fix for asserting var addr on stack

Merge pull request #3575 from stephentoub/argexception_names

Add missing parameter name to GetEra ArgumentOutOfRangeException

Merge pull request #3594 from pgavlin/GcInfoEncoderCleanup

Move some definitions out of gcinfoencoder.h.

Move some definitions out of gcinfoencoder.h.

There are a number of macros, method bodies, and type definitions that
do not need to be in gcinfoencoder.h. Instead, these definitions have
been moved into gcinfoencoder.cpp.

Merge pull request #3587 from pgavlin/UseListInLSRA

Use jitstd::list in the LSRA.

Fix stack walking on Unix in case of finally

The issue is that the code in the StackFrameIterator::Filter that handles cases
when a funclet frame that was already removed from the stack due to native frames
unwinding works for catch funclets only and not for finally ones.
The ExceptionTracker::GetCallerOfActualHandlingFrame is set for catch funclets
only. To make it work for the finally funclets as well, we need to use information
from the ExceptionTracker::m_EnclosingClauseInfoForGCReporting instead.

There was also another problem in the Filter method that caused the function to
spin in an infinite loop when a parent of a funclet was also a funclet. In
such case, the code in the method rechecks the current frame but the special
functionality to check the exception trackers data needs to be skipped for
the recheck.

And finally, when we find that the current frame was a parent of an unwound
funclet from the evidence in the exception trackers, we also need to set
the fSkippingFunclet.

Merge pull request #3592 from AndyAyersMS/FixAssert

Inliner: fix assert about repeated failing observations

Disable tests failing on Linux

Use jitstd::list in the LSRA.

This removes the only usage of ArrayList in the JIT.

Merge pull request #3469 from wtgodbe/raceCond

Fix race condition in Pri 2 tests, and revert those tests to Pri 1

Inliner: fix assert about repeated failing observations

When an inlining attempt is going to fail, the jit aims to bail out of
the attempt as quickly as possible, since any further work on this
inline will just waste time and memory. The jit monitors this in part
by asserting that it only makes one failing observation.

However, there are as few places where failing fast during inlining is
not currently possible, and repeated failing observations may occur.

This change loosens the assertion check to allow repeated
`CALLSITE_TOO_MANY_LOCALS` observations. This observation is made
at a fairly deep stack in `lvaGrabTemp` with no easy route to bail
out in case of imminent failure.

Closes #3586.

Merge pull request #3555 from LLITCHEV/Issue3540

Remove the the RBM_LOWINT define. Change the LsraLimitSmallIntSet to have a better representation of callee saved
Unrelated failure in Ubuntu x64 Checked Build and Test.

Fix GC stress setup failure on Unix-like OSes

Merge pull request #3573 from vcsjones/fix-3566

Fix failing test in IntPtrGetHashCode.

Merge pull request #3577 from gkhanna79/FixUbuntu15

Fix Ubuntu 15.10 Build break

Fix Unix exception handling in finalizers

When unhandled exception happens in a finalizer thread and there are
no managed frames till the bottom of the stack,
the Thread::VirtualUnwindToFirstManagedCallFrame then fails fast.
The fix is to make the Thread::VirtualUnwindToFirstManagedCallFrame to
return even in case no managed frame is called, which is indicated by
its returning 0 as the resulting IP address. The DispatchManagedException
then checks that and rethrows the exception instead of trying to call
UnwindManagedExceptionPass1.
I have also added INSTALL_UNHANDLED_MANAGED_EXCEPTION_TRAP to the
FinalizerThread::FinalizerThreadStart so that the unhandled exception
doesn't escape the stack.
And I also needed to modify the UNINSTALL_UNHANDLED_MANAGED_EXCEPTION_TRAP
to detect case when the unhandled exception filter was already executed so
that it doesn't double-report the unhandled exception.

Fix for casting byref to double in morph

This fixes https://github.com/dotnet/coreclr/issues/3517.
JIT didn't cast byref to double in morph, which caused
this assertion later.
The fix is to handle such case as well.

Merge pull request #3548 from kyulee1/fixgc

ARM64: Fix for handling GC register for call site

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

Merge changes from TFS

Merge pull request #3576 from leemgs/upstream-crossbuild

ARM: skip generating native image for mscorlib in case of cross build

Fix Ubuntu 15.10 Build break

ARM: skip generating native image for mscorlib in case of cross build

We need to skip generating native image for mscorlib in build_mscorlib()
of ./coreclr/build.sh when we do cross build for Linux/ARM on Ubuntu
14.04 x86 64bit PC.

Signed-off-by: Geunsik Lim geunsik.lim@samsung.com
Signed-off-by: Prajwal A N an.prajwal@samsung.com
Signed-off-by: MyungJoo Ham myungjoo.ham@samsung.com

Add missing parameter name to GetEra ArgumentOutOFRangeException

The exception's parameter name is incorrectly getting set to the intended message ("Time value was out of era range.").

Inline refactoring: move state machine into LegacyPolicy

The state machine used to estimate callee native code size is now an
internal part of the LegacyPolicy.

During `fgFindJumpTargets` the policy determines if the state machine
is need, and if so, sets one up. `fgFindJumpTargets` then notifies the
policy of the various opcodes in the method, and the policy uses this
to drive the state machine forward.

When the IL scan is completed, the `fgFindJumpTargets` notifies the
policy, then, for the discretionary inline case, grabs the native size
estimate, and uses that to evaluate inlinability.

The knowledge of when inlines are "always", "forced", or
"discretionary" is now encapsulated by the policy instead of being
distributed in various places in the code.

Remove the RBM_LOWINT define. Change LsraLimitSmallIntSet to have a better representation of callee saved
register for stress mode testing on System V OSs.

Remove the RBM_LOWINT define and change the LsraLimitSmallIntSet register
mask for System V OSs to provide a good
ratio between caller and calles saved register for the purposes of stress
testing.

Fixes issue #3540.

Merge pull request #3564 from jashook/x86_lb_excludes

x86 lb excludes

Disabling failing tests.

Fix failing test in IntPtrGetHashCode.

Test was not updated to reflect changes in GetHashCode.

Fix #3566

Merge pull request #3556 from AndyAyersMS/InlineRefactorEarlyForce

Inline refactoring: revise candidacy determination

Merge pull request #3441 from ramarag/pal_longfile

Enabling LongFile Support in PAL

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

Merge changes from TFS

Merge pull request #3477 from DrewScoggins/master

Update Linux build instructions

x86 lb excludes

Improve interop test coverage
Updating license headers
Delete duplicate test

Fix build break on Desktop arising out of MoveFileWrapper cleanup.

[tfs-changeset: 1582170]

Inline refactoring: revise candidacy determination

Rework the code to determine the key aspects of candidacy up front
rather than figuring it out eventually. In particular this ensures
that the two stages of evaulation for inlines both start at the same
point for candidates.

This will help streamline subsequent work to move the state machine
into the policy, since the state machine is only needed for the
case where the candidate is a discretionary inline, and the policy
now tracks this directly.

For the `LegacyPolicy`, if an inline is both smaller than the always
inline size and marked force inline, attribute its inlining to its
small size.

Remove bulk of the of the "evaluation in progress" candidate
observations since they don't add much value. As a result `noteCandidate`
is not needed as an external API, and can be repurposed internally as
`setCandidate`.

Change how the prejit root case is tracked to make it explicit
from the context rather than a callback to the compiler.

Enabling LongFile Support in PAL

ARM64: Fix for handling GC register for call site

This fixes https://github.com/dotnet/coreclr/issues/2731.
Looking at GC heap corruption, JIT incorrectly reports the live range of
GC references so GC swept the live objects.
The issue was when JIT emits call instruction, GC live register is not
processed at all, which JIT treated them as dead.
The fix is to implement this missing part for ARM64.

Reapply change 1578859: Add a new set of APIs for JIT configuration.

SuperPMI has been udpated to accommodate this change, and should no
longer crash when run using a JIT built with these changes.

[tfs-changeset: 1582144]

Fix race condition in Pri 2 tests, and revert those tests to Pri 1

Merge pull request #2878 from jashook/x86_exclude_update

Update x86 lb excludes and turn on ci testing for x86 lb.

Merge pull request #3521 from LLITCHEV/Issue3225

Remove a duplicate address tree consumption.

New x86 excludes and turn on ci testing for x86.

Merge pull request #3543 from leemgs/upstream-cross-arch2

ARM: Remove Distro/OS that have not been tested

ARM: Remove Distro/OS that have not been tested

Let's remove rest Distro/OSes to keep only the Distro/OS that have been tested
until someone actually start using/testing CoreCLR on RHEL, Debian, OSX, and so on.

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

Merge pull request #3470 from erozenfeld/ValueNumbersForR2RHelpers

Improvements for ReadyToRun helpers in JIT value numbering.

Merge pull request #3508 from hughbe/patch-1

Fix incorrect exception parameter name in GC

Merge pull request #3538 from vcsjones/fix-3531

Fix compilation errors on tests.

Fix compilation errors on tests.

Fix #3531

Merge pull request #3536 from JohnChen0/master

Modify zaprequire switch to allow Ready-to-Run images