Merge pull request #5863 from adityamandaleeka/generate_core_dumps_runtest

Add option to runtest.sh to enable limited dump generation.

Merge pull request #5868 from AndyAyersMS/BooleansRule

JIT: standardize on bool for return type of GenTree OperIs methods

Merge pull request #5862 from adityamandaleeka/remove_dead_code

Remove some dead code in notifyexternals

Merge pull request #5831 from briansull/fix-5759

Fix for issue 5795

JIT: standardize on bool for return type of GenTree OperIs methods

We had a mixture of bool and int return types. Standardize on bool.
Otherwise callers must match the mixed typing to avoid the fatal
C4800 warning on windows.

Remove dead code in notifyexternals

Enable core dump generation in CI test jobs

Merge pull request #5836 from RussKeldorph/nogcsosx

Disable GCStress=C periodic triggers for OSX

Merge pull request #5856 from AndyAyersMS/ConstantObservations

JIT: Cleanup code in and around fgFindJumpTargets

Add option to runtest.sh to enable limited dump generation.

Fix for issue 5795

We incorrectly pass the ret buf arg in x9 instead of x8

It is required that we call SortArgs and EvalArgsToTemps whenever we have any register arguments
We were not doing that when we only had a retBufArg argument in x8.

Fix for UNIX_AMD64 structs

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

Merge changes from TFS

JIT: Cleanup code in and around fgFindJumpTargets

Reorder code to remove gotos. Extract two large chunks of logic as
subroutines. Add braces and update comments. Bring `fgStack` class up
to current Jit coding conventions.  Reunite some orphaned comments for
fgFindJumpTargets.

No behavior change expected. No diffs seen via jit-diff tool. No diffs
seen in desktop testing.

Merge pull request #5839 from swgillespie/gc-batch-fix

Fix GC test batch script skip conditions that were behaving unexpectedly

Merge pull request #5852 from gkhanna79/Fix5837

Correctly resolve binding from Resolving event for static load

Changes to update TFS packages post 1.0

[tfs-changeset: 1613358]

Add missing check to allow assembly bound using the Resolving event to be used during static binding scenario

Merge pull request #5843 from dotnet-bot/UpdateDependencies20160617052045

Updating External dependencies to rc4-24217-00

Use LEAF_END_MARKED instead of LEAF_END for ARM64 debugbreak.S (#5850)

This commit applies the changes in #5847 (which fixes #5845) into ARM64
stub.

Use Use LEAF_END_MARKED instead of LEAF_END for various functions (#5849)

This commit fixes #5848.

Use LEAF_END_MARKED instead of LEAF_END for DBG_DebugBreak (#5847)

This commit tries to fix #5845

Merge pull request #5842 from prajwal-aithal/devel/arm-ci-emul-fix

ARM-CI: Create mount path if not already present

Updating External dependencies to rc4-24217-00

ARM-CI: Create mount path if not already present

* Previous PRs assumed that the mount path is already created on the CI machine
* This results in errors if the same is not true

Signed-off-by: Prajwal A N <an.prajwal@samsung.com>

Merge pull request #5841 from rahku/master

Fix arm64 build break

Fix arm64 build break

Merge pull request #5822 from adityamandaleeka/setup-runtime-dependencies_changes

Remove set -x from tests/setup-runtime-dependencies.sh.

Fix GC test batch script skip conditions that were behaving unexpectedly

Disable GCStress=C periodic triggers for OSX

Fixes #5763

Merge pull request #5835 from wtgodbe/buildFix

Fix error when passing msbuild args to build.cmd/buildtest.cmd

Fix error when passing msbuild args to build.cmd/buildtest.cmd

Mark Volatile methods with NonVersionable attribute (#5830)

Fix some typos in setup-runtime-dependencies.cmd

Merge pull request #5821 from kyulee1/fixwb

ARM64: Fix WriteBarrier

ARM64: Fix WriteBarrier

I think this finally fixes most of long outstanding GC issues that appear in different
forms so far that include #4877, #4879, #4890 and more.

The issues was WriteBarrier implementation.
When we update card table, the address has been already incremented by 8.

With this fix, all corefx tests seem to run on xunit framework without crashing.
I verified this with an out-of-date build locally, so there are not found
assembly errors like System.Runtime.Extensions with different versions, though.
These are what I tested so far that safely finished all runs with very high percentage of pass.
System.Collections.Tests
System.Dynamic.Runtime.Tests
System.Linq.Expressions.Tests
System.Linq.Parallel.Tests

Potentially more coreclr tests will pass. I will update them as tests go.

Merge pull request #5824 from dotnet-bot/UpdateDependencies20160616051753

Updating External dependencies to rc4-24216-00

Remove end labels from regular asm procs on Unix (#5777)

All ASM proc now unnecessarily have xxx_End label marking
the end of the proc. This causes confusing stack traces in
some cases, especially on OSX, when instead of a name of
an actual function, a name of the previous one with the
_End attached to it is shown.
This change fixes that by modifying the LEAF_END macro to
not to be defined as an equivalent of the LEAF_END_MARKED.

Merge pull request #5819 from AndyAyersMS/InlineTestsV1

Inliner: simple performance tests

Updating External dependencies to rc4-24216-00

Merge pull request #5794 from briansull/vm-retbuf

Fixes for two JIT helpers that should not use the x8 RetBuf argument on ARM64

Remove set -x from tests/setup-runtime-dependencies.sh.

Fixes for two JIT helpers that should not use the x8 RetBuf argument on ARM64

Fixes these two JIT helpers" CORINFO_HELP_GETFIELDSTRUCT and CORINFO_HELP_UNBOX_NULLABLE
We no longer set GTF_CALL_M_RETBUFFARG for the two special Jit helpers
We insert the extra byref argument in the x0 position for these two JIT helpers

Merge pull request #5820 from wtgodbe/groovyR2R

Update crossgen strings for coreclr coverage tests

Update crossgen strings for coreclr coverage tests

Merge pull request #5815 from BruceForstall/RemoveAmd64Legacy

Remove AMD64 code from legacy code path

Merge pull request #5814 from JohnChen0/r2rtest

Modify CrossGen test script to work around cmd.exe bug

Inliner: simple performance tests

Tests that demonstrate that constant arguments passed to callees can
lead to inlining performance wins. All numeric types are tested, as
well as strings.

Tests are generally written as positive/negative pairs with calls to
the same inlinee. This exposes the call site sensitivity. Performance
of the positive test should be better than the negative test.

The overall benchmarking approach used here is something that we might
consider adopting for other small-scale codegen performance tests.

Fix exception stacktrace when ulimit is hit OSX. (#5802)

* Fix exception stacktrace when ulimit is hit OSX.

Issue #5782.

Wrap the everything in try/catch replacing the existing try/finally
and removed the recursion protection thread var.

* Restore recursion protection.

Merge pull request #5786 from prajwal-aithal/devel/arm-ci-cross-build-fix

ARM-CI: Convert Linux ARM emulator CI check to cross build (fixes PR #5680)

Remove AMD64 code from legacy code path

There were some AMD64 #ifdefs and code in the legacy code path from an
ancient AMD64 porting effort. They are unused, and were confusing to me
when searching the code base, so I removed them.

Update SpecializeComparer for enums (#5805)

Fixes #5751

Fix sos.DumpIL on LP64 systems with s/long/LONG/g (#5810)

sos.DumpIL command used readData<long> to disassemble CIL,
this failed on LP64 systems like Linux where long is 8 bytes.

This commit tries to fix #5457.

Merge pull request #5807 from Maoni0/obj_addr

Need to filter addresses not in the heap range in NextObj

Modify CrossGen test script to work around cmd.exe bug

Merge pull request #5682 from lemmaa/fix-error-index-must-be-an-integer-in-range

[ARM64/Linux] Fix `error: index must be an integer in range.`

Initial support of breakpoint setting by source file + line on Linux using portable pdb reader (#5688)

* Initial support of breakpoint setting by source + line on Linux

* Fix macro redefinition error in OSX build

* Fix after review from Mike McLaughlin

[ARM64/Linux] Fix `error: index must be an integer in range.`

- Fix invalid varible name `*X19_X28' to '*X19_X29'
- Remove code for capturing `fp` status.

Need to filter addresses not in the heap range in the NextObj verification function.

Merge pull request #5773 from Maoni0/grow_card_2

Making the fix of runtime suspension check complete

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

Merge changes from TFS

My last fix of checking whether the runtime was suspended was incomplete.
I needed to check at the 3 other places. Before the software write watch
implementation, this check was in the write barrier code itself and was
doing the right thing. So I am just using the same thing (ie, IsGCThread)
to check whether the runtime is suspended.

Fix full framework build break

[tfs-changeset: 1612754]

Merge pull request #5798 from wtgodbe/remCovr

Remove unecessary coverage arg passed to coreclr test run

Remove unecessary coverage arg passed to coreclr test run

Add missing inttypes header for musl-libc (#5729)

Also replaced the usages of `__unit64_t` and `__int64_t` with
`uint64_t` and `int64_t` respectively.

Merge pull request #5791 from swgillespie/bash-bracket

Fix bracket spacing in runtest script

Fix incorrect buffer limits in Module.FullyQualifiedName QCall (#5753)

Merge pull request #5695 from AlexGhiondea/DefaultToLatest

Default to latest behavior when a TFM is not specified

Merge pull request #5792 from wtgodbe/groovyQuote

Remove extra quote from runtest arg in coverage job

Remove extra quote from runtest arg in coverage job

Fix bracket spacing in runtest script

Merge pull request #5789 from kyulee1/fixgchole

ARM64: Fix a GC hole for Indirect Branch

Merge pull request #5787 from dotnet-bot/UpdateDependencies20160614051831

Updating External dependencies to rc4-24214-00

ARM64: Fix a GC hole for Indirect Branch

`br reg` was handled like a regular call under `IF_BR_1B` resulting in
`emitOutputCall` which updates GC vars/regs.
This is only true when this branch is used for a tail-call.
We used the same logic for indirect jump for switch expansion, which is
incorrect.
The fix is to correct instruction category so that such indirect jump is
handled same as an normal instruction under `IF_BR_1A`.

This fixes a part of https://github.com/dotnet/coreclr/issues/4877.
The failing point from the description is now passed, but still another
consistency failure far later on after tests are discovered/started.

Link libintl in case of Alpine (#5731)

This delta detects Alpine Linux by using `uname -v` (kernel version)
and add the `libintl` in the linker list. This is because `gettext` is
not part of (per-installed in) Alpine Linux but available via the
package manager. To build CoreCLR, `gettext-dev` is one unlisted
per-requisite.

Updating External dependencies to rc4-24214-00

ARM-CI: Convert Linux ARM emulator CI check to cross build (fixes PR #5680)

* Previously used emulated build took a lot of time leading to timeout errors
* Changing this CI check to build using cross build
* Resolves CI issues caused by previous identical PR #5680

Signed-off-by: Prajwal A N <an.prajwal@samsung.com>

Refactor MethodTable::ContainsStackPtr (#5754)

- Rename ContainsStackPtr to IsByRefLike. It is the term used for this kind of types in ECMA spec.
- Change the check to be based on flag instead of hard coded list of types
- Remove redundant unused method of the same name on EEClass

Fix PAL executable allocator locking (#5770)

We call the ExecutableAllocator from two places in PAL. One is the
VirtualAlloc when the allocation type contains MEM_RESERVE_EXECUTABLE.
The other is MAPMapPEFile.
While the former is called inside of the virtual_critsec critical section,
the latter doesn't take that critical section and so in some race cases,
we were returning the same address twice - once for VirtualAlloc and
once for MAPMapPEFile. That resulted in strange memory corruption cases.
The fix is to use the same critical section for the MAPMapPEFile too.

I am also reverting the change in the virtual commit that was made when
we have thought that the culprit might be in the mprotect.

Merge pull request #5702 from mjsabby/master

Add Profiling APIs that fail on non-Windows OS

Merge pull request #5766 from dotnet/revert-5680-devel/arm-ci-cross-build

Revert "ARM-CI: Convert Linux ARM emulator CI check to cross build"

Revert "ARM-CI: Convert Linux ARM emulator CI check to cross build"

Merge pull request #5764 from wtgodbe/coverage

Update coverage test to have correct configuration for coreFX tests

Merge pull request #5762 from swgillespie/ci-longgc

Pass --long-gc and --gcsimulator to long GC and GCSimulator CI jobs, respectively

Update coverage test to have correct configuration for coreFX tests

Delete bad assert (#5757)

Merge pull request #5713 from wtgodbe/mutex

Keep mutex alive to avoid race condition in openmutexpos4 test

Delete unnecessary cruft from internal mscorlib reference assembly (#5756)

Pass --long-gc and --gcsimulator to long GC and GCSimulator CI jobs, respectively

Keep mutex alive to avoid race condition in openmutexpos4 test

Specialize Comparer<T> for enums to avoid boxing (#5503)

Partial revert "Fix problem with virtual memory commit in OOM scenario on Linux (#5609)" (#5744)

#5609 is suspect to cause intermittent crashes on OS X. Keep the fix for Linux, but revert to using the original code on OS X.

Undo StompWriteBarrierResize reordering (#5740)

The recent commit introduced race by changing where StompWriteBarrierResize is called relative to other global state updates. Revert this change to fix the intermittent crashes that we are seeing because of that. We will look at fixing the theoretical problem that this reordering was trying to address in better way separately.

Add check for gnu/lib-names.h (#5727)

The header `<gnu/lib-names.h>` is not present on musl-libc based OSes.
Those names are not listed in any system header and this is deemed
irrelevant to have such a header in musl according to:
http://www.openwall.com/lists/musl/2013/11/09/1

Check for unavailable cache-size macro (#5728)

In musl-libc world _SC_LEVEL1_DCACHE_SIZE is undefined.

Correct comment (#5738)

Merge pull request #5544 from vancem/EtwSessionFix

Fix a problem where stale old, closed ETW sessions might be turned on.

Typedef __ptrace_request for musl-libc (#5730)

Ref: http://www.openwall.com/lists/musl/2015/10/01/3

Add check for sys/sysctl.h (#5726)

On some operating systems `<sys/sysctl.h>`is not present yet `sysct`
function in available. One example is Alpine Linux which uses musl-libc
as opposed to GNU libc.

Ref: dotnet/corefx#6253

Add Profiling APIs that fail on non-Windows OS