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

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

Do not use fragile native image content when generating R2R images (#5715)

Updates tryrun.cmake for ARM / ARM (softfp) (#5673)

While fixing #5316 (PR #5317, PR #5323), all the features related with PROCFS
are disabled in ARM and ARM (softfp) cross build.

As a result, the collision avodance logic in GetProcessIdDisambiguationKey is disabled,
which leads to random coreclr_initialize failures
(CLR failed to initialize itself when the previous execution of CLR with the same PID
abnormally terminates)

This commit tries to re-enable the relevant features for ARM and ARM (softfp) cross-build.

Since the ARM/Linux emulator (which is the current reference in use) supports
/proc/self/stat, /proc/self/status, /proc/self/maps, this commit re-enables
three relevant features (HAVE_PROCFS_STAT, HAVE_PROCFS_STATUS, HAVE_PROCFS_MAPS).

This commit tries to fix #5583.

JapaneseCalendar: Use Registry.LocalMachine instead of RegistryKey.GetBaseKey(RegistryKey.HKEY_LOCAL_MACHINE) (#5725)

JapaneseCalendar is the only code outside of the Registry code that
calls RegistryKey.GetBaseKey(RegistryKey.HKEY_LOCAL_MACHINE) to get a
RegistryKey instance for the local machine hive. Everywhere else uses
Registry.LocalMachine. Update JapaneseCalendar to match, which also
avoids the unnecessary RegistryKey/SafeRegistryHandle allocations.

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

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

Remove obsolete Registry.DynData field from System.Private.CoreLib (#5724)

Registry is internal in System.Private.CoreLib due to being pulled-in
by the dependency closure. The obsolete DynData field is only valid
on Win9X, so there's no need to allocate it and keep it around in
CoreCLR. Remove it from System.Private.CoreLib with an ifdef, along
with other references to the HKEY_DYN_DATA hive.

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

* Previously used emulated build took a lot of time leading to timeout errors
* Changing this CI check to build using cross build

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

Remove non-generic Stack from System.Private.CoreLib (#5703)

The non-generic Stack class only exists internally in
System.Private.CoreLib because it is being used in the Environment
class in one place for resource lookup and therefore is being
pulled-in by the dependency closure. It can be removed by using
List<string> instead.

Merge pull request #5707 from kouvel/BgMarkFreeObjectAdvanceFix

Fix for GC hole when using software write watch

Merge pull request #5692 from AndyAyersMS/DerministicCrypto

Modify Crypto benchmark so it uses a fixed random seed

Fix rethrow of managed exception from native code (#5697)

This change fixes a problem that happens when managed exception is caught
in native code, rethrown and then caught and passed to DispatchManagedException
to be propagated further through managed code frames.
The problem is that while the exception was rethrown, it still has the
TargetFrameSp member set to the frame that has caught it and so the
DispatchManagedException thinks it is just continuing to propagate it in the
second pass instead of starting a new first pass.
The UnwindManagedExceptionPass2 then asserts, because the TargetFrameSp is
below the frame it is about to process.

The fix is to reset the TargetFrameSp to NoTargetFrameSp when the exception
is caught in native code - in PAL_EXCEPT and EX_CATCH_IMPL_EX (which catches
all exceptions).

I have also added few stress log messages for processing exception in native
frames that helped debug the issue.

Fix for GC hole when using software write watch

Issue:
- When using software write watch, getting dirty pages is synchronized with growing the write watch table using gc_lock, as those cannot happen concurrently
- It turns out that a foreground GC can happen at that point, pausing the background GC while it tries to acquire gc_lock
- In this issue, the last object in the previously revisited page is a free object whose range spans one or more pages
- With the intent of skipping the free range, revisit_written_page() records the next object to scan immediately following the free range
- Since only 100 dirty pages are retrieved at a time, the next set of up to 100 dirty pages needs to be requested. Before that, a foreground GC begins, and the background GC pauses trying to acquire gc_lock.
- The foreground GC promotes some objects, and in the process allocates from the free range above, and completes
- One of the moved objects in that previously free range gets a reference to a new object that is not referenced from anywhere else
- The background GC resumes, determines that the moved object's page is dirty, but skips revisiting the object because the previously recorded next object from which to start scanning, already took the free range size into account before it was allocated
- The background GC does not mark through the moved object, so the newly referenced object appears to be free, and is sweeped
- Later, either verify_heap fails or the app crashes, when trying to access that referenced object

Fix:
- Similarly to what is done for the large object heap, when revisiting a page and using software write watch, don't advance last_object when the current object is a free object that spans beyond the current page or high_address
- On the next revisit, it would start scanning from the previously scanned free object (which now may be allocated or will have a smaller size) until it reaches the requested page to scan

Fixes (but not yet closing) #5194

Default to latest behavior when a TFM is not specified

Instead of requiring that a compatibility switch is set in order to get the
latest behavior for all AppContext switches that are based on TFM, have that
behavior on by default.

Merge pull request #5691 from kyulee1/testupdate2

ARM64: Another Tests.lst Update for R2R

Merge pull request #5694 from rahku/sosFix

Fixes AV in sos for arm64

Fixes AV in sos for arm64

Modify Crypto benchmark so it uses a fixed random seed

Fix seed so benchmark behavior doesn't vary randomly from run to run.

ARM64: Another Tests.lst Update for R2R

Fixes https://github.com/dotnet/coreclr/issues/5638
Fixes https://github.com/dotnet/coreclr/issues/5645

doublinkstay.cmd does not repro anymore
increase time-out for DeltaBlue

Merge pull request #5685 from kyulee1/testlst

ARM64: Update Tests.lst

Merge pull request #5621 from swgillespie/buildtest-refactor

Remove need for longgc and gcsimulator tests to have their own build

Remove need for longgc and gcsimulator tests to have their own build

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

Merge changes from TFS

ARM64: Update Tests.lst

Fixes https://github.com/dotnet/coreclr/issues/5641

mscorlib/System.Private.CoreLib cleanup

Deleting files cloned for mscorlib rename - we should not need the clones anymore.

[tfs-changeset: 1611972]

Merge pull request #5679 from dotnet-bot/UpdateDependencies20160610051941

Updating External dependencies to rc4-24210-00

Updating External dependencies to rc4-24210-00

Merge pull request #5667 from kyulee1/testupdate

ARM64: Test Update

Merge pull request #5670 from AndyAyersMS/ObserveCallSiteBlockWeight

Inliner: observe block weight, update schema and data

Merge pull request #5656 from dotnet/kspawa-profiler-doc

Create profiling-api-status.md

Merge pull request #5666 from swaroop-sridhar/LR

ARM64: Don't track LR as part of NV-Context

Merge pull request #5658 from pgavlin/PhaseInterface

Make Phase::{Pre,Do,Post}Phase protected.

Inliner: observe block weight, update schema and data

Add block weight as observation. At first blush I suspect this will be
useful only when the jit has IBC data, and even then only in the root
method blocks.

We'll likely need to either build up better profile estimates for the
inliner or else just try and cope without it somehow.

Dump out block weight and the per-call profitability in the observation
schema and data. Among other things this lets us cross-validate how well
the ModelPolicy predictions match the abstract GLMNET models...

Merge pull request #5649 from pgavlin/removeSplitMethodTrees

Remove `fgSplitMethodTrees`.

Merge pull request #5661 from AndyAyersMS/FirstDraftProfitablityModel

Inliner: implement profitabily side of the ModelPolicy

Merge pull request #5663 from briansull/legacy-cleanup

Code review cleanup items and moved some items into LEGACY_BACKEND

Merge pull request #5652 from RussKeldorph/5555

Fix WeakReference tests depending on JIT not extending lifetimes

Update profiling-api-status.md