Merge pull request #3592 from AndyAyersMS/FixAssert

Inliner: fix assert about repeated failing observations

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.

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

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

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

Inline refactoring: convert hints into observations

This change updates the inlining code to use observations in place of
the InlineHints and hint-like things (eg "HasSimd"). A number of new
observations were added in support of this.

The `compInlineeHints` member of the compiler object was removed as
the same information is now tracked by the inline policy. The policy
also now contains most of the weights and combining logic used to
compute the profitabiliy boost for the candidate.

There is one subtle and tricky aspect to the change. For the most part
the hints were ignored during the prejit-root analysis, but the mostly
load-store hint was propagated and influenced the inlines done when
crossgenning mscorlib. See #3482 for more on this particular quirk.

I've preserved this behavior by "passing" the prejit inline result
into `fgFindJumpTargets`. In actuality the result is passed by
temporarily setting the `compInlineResult` compiler member variable.
Then, in `fgFindJumpTargets`, the load-store observation is the only
one not guarded by `compIsForInlining`. In a subsequent change I'll redo
the guards at other observation sites and put the policy aspects of these
observations into the policy object.

This addresses another piece of the work outlined in #3371.

Fix incorrect exception parameter name in GC

Fix for disabling incremental build for ARM64

The fix (d58885c9a37d857f10d90f6db7246630d4ce8980) broke ARM64 debug
build.
The right fix is to disable such flag for all archs.

197654 [x86] [scanRuntime] Fix contracts in gcenv.os.cpp

[tfs-changeset: 1581565]

Fix Stack Unwind Behavior of Libunwind-ARM

Disable UNW_ARM_METHOD_DWARF, which gets problematic with
assembly-created stacks in ARM systems.

Although previous commits have fixed the compatibility issues
between cpp stacks and libunwind-arm, the assembly part
(asmhelpers.S) had similar problems.

Fix #3462

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>

Fix PAL_VirtualUnwind Behavior with Clutters in PC

Libunwind-ARM recommends to read LR to get value of PC
while unwinding stack, which often causes the caller
(PAL_VirtualUnwind) to have LSB of PC set.

Such behavior incurs errornous behavior of PAL_VirtualUnwind
if the output is fed back to PAL_VirtualUnwind as the input,
which is observed with VirtualUnwindToFirstManagedCallFrame().

This commit clears LSB of PC in WinContext to prevent it.

Note that this is a partial fix for the issue #3462.
(this does not fix assembly stacks)

Fix #3462

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>

Add GC stress modes

Merge pull request #3498 from kyulee1/disinc

Fix for disabling incremental link in Windows

reuse getStressModeScriptStep

Merge pull request #3499 from wtgodbe/nameTooLong

Fix file-name-too-long error for all builds, not just pri1

Fix file-name-too-long error for all builds, not just pri1

Remove needless references to test library.

Merge pull request #3472 from AndyAyersMS/InlineRefactorFixPlumbing

Inline refactoring: rework logic in compCompileHelper

Fix for disabling incremental link in Windows

I'm still getting linker error (mostly debug build) which often caused a
trouble to reboot the machine.
Looking at vcxproj files, we didn't correctly disable incremental link.
There is some ordering issues in linker flag definitions.
The disabling flag should be at the end for each build type
to override the default behavior.

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

Merge changes from TFS

Add license headers to ported threading tests.

Remove Monitor/EnterExit15.

Merge pull request #3488 from benpye/arm-redefinition

Fix ARM build regression

Add two files that should have been part of changeset #1581242.

[tfs-changeset: 1581263]

Exclude DefaultStackCommit test from non-Windows platforms.

Open source tests from baseservices/threading.

Refactor Utilcode's IAllocators.

- `DefaultAllocator` has been replaced with `HostAllocator` in RyuJIT, which
  uses the JIT hosting interface to allocate and free memory.
- The definition of `DefaultAllocator` has been moved into its own file, as it
  remains in use by the interpreter and the binder.
- `ProcessHeapAllocator` has been moved into JIT32, as that was its only
  remaining user.
- `AllowZeroAllocator`'s static field has been changed to an instance field
  to avoid the need to define storage for the static field in Utilcode.

[tfs-changeset: 1581242]