Merge pull request #3602 from AndyAyersMS/InlineRefactorNativeSize

Inline refactoring: move native callsite estimator

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

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

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

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

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

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

Modify zaprequire switch to allow Ready-to-Run images

Address issue #3527.

Merge pull request #3446 from kyulee1/FixIlasmTool

Fix for not using ilasm from the build binary

Merge pull request #3528 from leemgs/upstream-pal-linux-macro

Replace all uses of __LINUX__ with the lower case

Merge pull request #3525 from leemgs/upstream-cross-arch

Add arm[64] to isMSBuildOnNETCoreSupported()

Merge pull request #3530 from ramarag/fixtest

Fix syntax error in test code

Fix syntax error in test code

Replace all uses of __LINUX__ with the lower case

According to https://sourceforge.net/p/predef/wiki/OperatingSystems/
(Pre-defined Compiler Macros) and http://www.faqs.org/docs/Linux-HOWTO/GCC-HOWTO.html
(Porting and compiling:Automatically defined symbols), the __linux__ marcro is the
right way to detect systems based on the Linux kernel and is the POSIX compliant.

Signed-off-by: Geunsik Lim <geunsik.lim@samsung.com>
Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>

Improvements for ReadyToRun helpers in JIT value numbering.

The following ReadyToRun helpers are now treated similarly to their normal
counterparts in value numbering:
CORINFO_HELP_READYTORUN_NEW
CORINFO_HELP_READYTORUN_NEWARR_1
CORINFO_HELP_READYTORUN_ISINSTANCEOF
CORINFO_HELP_READYTORUN_CHKCAST
CORINFO_HELP_READYTORUN_STATIC_BASE

In particular, this allows CSE-ing calls to the last 3 of the above helpers when
possible. #3281 is an issue for CORINFO_HELP_READYTORUN_STATIC_BASE.

Compiler::fgValueNumberHelperCallFunc is refactored to reduce code
duplication.

Closes #3281.

Merge pull request #3505 from LLITCHEV/Issue3360

Reenable an assert.

Merge pull request #3523 from briansull/struct16-fix

Fixes for 16-byte struct argument passing for ARM64

Fixes for 16-byte struct argument passing for ARM64

Workarounds to fix Issue 3151 and Issue 2987
14 additional tests are now passing
When passing two register TYP_STRUCT type suisng PUTARG_REG we add a physical register kill for the second register
When generating code for a 16-byte GT_LDOJ that conbatisn GC references we reverse the instructions when necessary
Added new utility method lvIsMultiregStruct() that returns true for the ARM64 16-byte struct type
Updated Tests.lst file

Add arm[64] to isMSBuildOnNETCoreSupported()

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

Merge pull request #3513 from mikem8361/soschanges

Fix two sos issues.

Merge pull request #3518 from gkhanna79/OSXRepro

Define Platform and OutputPaths

Merge pull request #3519 from JohnChen0/longpath

Restore linking debug corerun.exe with retail CRT

Merge pull request #3511 from vcsjones/fix-3506

Improve GetHashCode for IntPtr/UIntPtr on 64-bit.

Merge pull request #3507 from gkhanna79/OneCore2

Fix CRT linkages for uCRT

Merge pull request #3452 from fujiy/patch-1

Anchor Tag fix

Reenable an assert.

Back in December an assert was commented out in the code (when a big PR
was done).
The issue was that in one case of a tail call test there was a LcLVar with
type of TYP_STRUCT with a single eightbyte.
This should not have happened since in such cses the type of the LclVar is
normalized to the type of the first structs' eightbyte.
Enabling the assert has no effect on the test in question - it passes.
I suspect the issue was resolved sometime back (in January) when we
discovered there were some bad merges in the big PR wiping . As I recall, at least
one was related to a taken out tail call fix.

Fixes issue 3360.

Define Platform and OutputPaths before Microsoft.Common.CurrentVersion.targets is included.

Remove a duplicate address tree consumption.

In void CodeGen::genEmitCallvoid CodeGen::genEmitCall override that takes
a GenTreeIndir AST we consume the indir addr node. Same thing is done in
codegenxarch.cpp just before this method is called. This PR removes the
later and thus leaving one consumption of the indirTree->Addr() tree.

Improve GetHashCode for IntPtr/UIntPtr on 64-bit.

The GetHashCode behavior on IntPtr and UIntPtr truncated the upper 32 bits.
This change uses the behavior of long and ulong, respectively, to generate
the hash code.

UIntPtr no longer strips off the top-most bit for GetHashCode purposes.

Fix #3506

Restore linking debug corerun.exe with retail CRT

The long path support feature modified debug corerun.exe with debug
CRT (msvcrtd.dll). This made it more difficult to use corerun.exe,
since the debug CRT is not always available. It turns out this change
is no longer necessary, so reverting part of the change to link debug
corerun.exe with retail CRT (msvcrt.dll) again.

Update Linux install instructions

Removed all of the instructions that related to running the hello world
sample on Linux.  Also removed the dependency on mono from the
intsructions as this is no longer needed.  Finally, I removed the steps
that related to using Windows as that is no longer needed for the Linux
build.

Merge pull request #3495 from pgavlin/HistogramCleanup

Clean up the JIT's histogram type.

Merge pull request #3512 from benpye/arm-monotonic-coarse-tryrun

Add value to arm/arm64 tryrun files for HAVE_CLOCK_MONOTONIC_COARSE.

Fix for not using ilasm from the build binary

Using ilasm from the build is a desire to test ilasm tool -- in fact we've now had ildasm-ilasm round-trip tests,
so this is not critical either.
But when building cross-target like ARM64, running such native ilasm in the different host (x64) is not an option.
Given the current test architect where tests are all populated in Windows,
I just fall back to use one from VS tool same as other tools like CSC.
Ideally, we should download such host tools based on host target especially
when we start populating tests on non-Window platforms.
Currently two of tests are disabled due to the issue #3517.

Merge pull request #3509 from kyulee1/arm64build

Fix for disabling incremental build for ARM64

Merge pull request #3497 from adityamandaleeka/threading_test_port

[WIP] Open source some more threading tests

Fix C++11 issue when converting string to char *

Merge pull request #3500 from AndyAyersMS/InlineRefactorInlineHints

Inline refactoring: convert hints into observations

Clean up the JIT's histogram type.

Fix two sos issues.

ARM Disassemble problem and reenterency bug on Windows.

Add value to arm/arm64 tryrun files for HAVE_CLOCK_MONOTONIC_COARSE.

Merge pull request #3491 from rahku/Helix

add TargetsWindows property required by cloudtest.targets in buildtools package

add TargetsWindows property required by cloudtest.targets in buildtools package

Change priority of tests in baseservices/threading to 1.

Merge pull request #3502 from myungjoo/fix/3462_arm_linux_stack

ARM-Linux Fixes (Issue #3462)

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

Merge changes from TFS

Merge pull request #3404 from gregg-miskelly/DbgShimFix

DbgShim!RegisterForRuntimeStartup: Avoid extra call to callback

Fixup CRT linkages for uCRT

Merge pull request #3156 from sejongoh/add_gc_stress_modes

Add GC stress modes

Merge pull request #3337 from sejongoh/reuse_existing_function_to_set_stress_modes

reuse getStressModeScriptStep