Cast function pointer to void*

Fix issue building mscorlib.dll

/home/kamil/tmp/coreclr/src/vm/win32threadpool.cpp:2672:13: error: no matching function for call to 'FireEtwThreadPoolIOEnqueue'
            FireEtwThreadPoolIOEnqueue((LPOVERLAPPED)waitInfo, Callback, (dwFlag & WAIT_SINGLE_EXECUTION) == 0, reinterpret_cast<void*>(GetClrInstanceId()));
            ^~~~~~~~~~~~~~~~~~~~~~~~~~
/home/kamil/tmp/coreclr/bin/obj/Linux.x64.Debug/src/inc/clretwallmain.h:972:14: note: candidate function not viable: no known conversion from
      'WAITORTIMERCALLBACK' (aka 'void (*)(void *, unsigned char)') to 'const void *' for 2nd argument; take the address of the argument with &
inline ULONG FireEtwThreadPoolIOEnqueue(
             ^
/home/kamil/tmp/coreclr/src/vm/win32threadpool.cpp:3236:13: error: no matching function for call to 'FireEtwThreadPoolIODequeue'
            FireEtwThreadPoolIODequeue(waitInfo, waitInfo->Callback, reinterpret_cast<void*>(GetClrInstanceId()));
            ^~~~~~~~~~~~~~~~~~~~~~~~~~
/home/kamil/tmp/coreclr/bin/obj/Linux.x64.Debug/src/inc/clretwallmain.h:985:14: note: candidate function not viable: no known conversion from
      'WAITORTIMERCALLBACK' (aka 'void (*)(void *, unsigned char)') to 'const void *' for 2nd argument; take the address of the argument with &
inline ULONG FireEtwThreadPoolIODequeue(
             ^

Reported on Ubuntu-14.04.2 LTS with clang 3.8.0

Merge pull request #4342 from ZhichengZhu/clrstackr

add sos clrstack -r option

Merge pull request #4368 from swgillespie/long-running-gc-fixes

Fix a difference in passing the sequential parameter between Bash and…

Merge pull request #4372 from krytarowski/netbsd-support-66

NetBSD has RAND_MAX = 0x7fffffff

The RAND_MAX value can vary by platform

Noted on NetBSD where it is 0x7FFFFFFF.

Merge pull request #4347 from ramarag/zapnyi

Removing NYI for ZapUnwindData

Merge pull request #4140 from myungjoo/implement/4040

Implement StartUnwindingNativeFrames Linux/ARM

Merge pull request #3240 from krytarowski/netbsd-support-50

Fix issue unveiled on NetBSD: Add PAL__vsnprintf shadow in PAL

Fix a difference in passing the sequential parameter between Bash and Batch

add clrstack -r option

Merge pull request #4359 from gkhanna79/ROLibFix2

DelayLoadAndFixWinRTInterop

Merge pull request #4361 from joperezr/AddServiceableAttribute

Adding Serviceable attribute to mscorlib

Merge pull request #4352 from kouvel/ArmAssertFix

Fix assertion failure on ARM

Merge pull request #4338 from pgavlin/AddJit32Build

Make it possible to build JIT32 in the OSS tree.

Merge pull request #4363 from mmitche/update-corefx-paths

Update the corefx copy steps to point to the new location

Update the corefx copy steps to point to the new location

Adding Serviceable attribute to mscorlib

DelayLoadAndFixWinRTInterop

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

Merge changes from TFS

Implement StartUnwindingNativeFrames Linux/ARM

StartUnwindingNativeFrames for Unix/Linux ARM.
The code is ported from AMD64 code of the same function and OS.

CONTEXT_* are moved from context2.S to asmconstants.h so that
the same macros may be sharaed between multiple files.

Fix #4040

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

Merge pull request #4260 from myungjoo/fix/4259

ARM: fix build warning for unw usage

Fix assertion failure on ARM

StompWriteBarrierEphemeral may be called on the init path, where the runtime is technically not suspended, but has not started yet. Added a check for g_fEEInit in the call.

Merge pull request #4340 from jkotas/equality-comparer

Use beforefieldinit field for Comparer/EqualityComparer.Default

Merge pull request #4353 from myungjoo/fix/4107

ARM/Linux: Mark r0-r3 Saved

ARM/Linux: Mark r0-r3 Saved

Without this, exception handlers (try-catch) that
try to get SP of ThePreStub's caller (managed frame) mislocate
the caller's SP by 4 words (0x10) and get lost while
traversing managed frames.

Fix #4107

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

Merge pull request #4343 from pgavlin/SeparateBitPosition

Move `BitPosition` into its own file.

Fix razzle build breaks in bcl.win

[tfs-changeset: 1596379]

Style fix: remove obsolete contents

We no longer have StartUnwindingNativeFrames.

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

Use beforefieldinit field for Comparer/EqualityComparer.Default

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

Merge changes from TFS

Merge pull request #4335 from AndyAyersMS/InlineStrategy

Inliner: introducing InlineStrategy

Removing NYI for ZapUnwindData

Make it possible to build JIT32 in the OSS tree.

This change adds a new argument to build.cmd, buildjit32, that configures
the build to build and link JIT32 instead of RyuJIT if the sources are
available in `src/jit32`.

Merge pull request #4339 from fadimounir/deltests

Delete fragile tests (will fix tests in a later PR)

Delete fragile tests (will fix tests in a later PR)

Move `BitPosition` into its own file.

This will allow the GC info encoder and the JIT to use this
function without pulling in the entire utilcode header.

Inliner: introducing InlineStrategy

Create the `InlineStrategy` class to hold inline-relevant data that
spans multiple inlines. It tracks the number of inline candidates,
attempts, successes, last successful policy, and holds onto the root
context, and so on. The strategy is responsible for creating contexts
and reporting overall results for a method (either tree-based or
data-based).

The strategy also includes a simple jit time estimate. From various
observations, post-inline jit time can be modelled acceptably by simple
linear relationships on the input IL size. The strategy uses these models
to estimate the initial and current jit time. Estimate units are roughly
microseconds.

The strategy creates a time budget to flag cases where the estimated
jit time increase due to inlining is unreasonbly large. The budget
is initially based on the root method size, and may increase if there
are non-discretionary force inlines.

Once we have a bit more vetting of the budgeting mechanism, policies
will use it to limit inlining in a small number of runaway inlining cases
(see for example #2472).

Remove code under MEASURE_INLINING since the strategy plus the context
tree (optionally extended via policies) contains all that data and more.
Likewise, consolidate a number of the compiler's inlining-related member
variables into the strategy.

Merge pull request #4330 from janvorli/fix-sigsegv-deadlock

Fix possibility of deadlock in the SIGSEGV handler

Fix full framework build break

[tfs-changeset: 1596259]

Merge pull request #4322 from pgavlin/TlsLogEnv

Collect the JIT's TLS variables in a single type.

Merge pull request #4314 from pgavlin/GcInfoSList

Replace `SList` in the GC info encoders.

Fix potential deadlock when calling ExecutionManager::IsManagedCode

The function can deadlock if the calling thread was holding the
ExecutionManager writer lock.

Merge pull request #4308 from joperezr/RemoveWarnings

Remove native build warnings

Updating BuildTools

Merge pull request #4327 from RussKeldorph/structinregs

Fix structinregs test on x86

Fix possibility of deadlock in the SIGSEGV handler

The SIGSEGV handler was transitively calling pthread_attr_get_np to
get current stack limit no matter where the exception happened. The
problem was that if the segmentation violation happened in a C
runtime function, this could lead to deadlock since pthread_attr_get_np
is not signal safe.

The fix is to add callback that will figure out if the failing instruction
was in managed code or in one of the jit helpers and if not, it would
not attempt to handle the signal.

I have also removed non signal safe calls from the signal handlers code
path where we don't know yet whether the interrupted code was in a function
that could collide with a function that we call from the signal handler
or not. This includes TRACE and ASSERT calls that were calling string
formatting functions.

Fix structinregs test on x86

Fix a typo in _MSC_VER resulting in a mismatched calling convention on
x86.  While I'm in here, I'm also fixing some benign mismatched function
pointer type definitions.

Add back Path APIs needed by UWP

[tfs-changeset: 1596013]

Merge pull request #4297 from tijoytom/master

Disabling the failing struct by value scenario.

Collect the JIT's TLS variables in a single type.

Instead of storing pointers to the thread-local compiler instance
and the current logging environment in separate TLS slots, wrap
these values in a struct and store a pointer to the struct in a
single TLS slot. This simplifies and standardizes the JIT's interface
to its TLS storage, especially when it is being statically linked
into another program or library that cannot use standard TLS.

Note that for release builds, the logging environment is not used
and a pointer to the current compiler instance is stored directly
into the JIT's TLS slot as an implementation detail of the TLS
wrapper type.

Merge pull request #4320 from mmitche/add-reporting

Add reporting functionality to the coreclr netci file

Add reporting functionality to the coreclr netci file

Also eliminate a few jobs that aren't being referenced.

Merge pull request #4319 from mmitche/archive-needed

Only archive dylib and .so files from the native coreclr tests

Only archive dylib and .so files from the native coreclr tests

Avoids a few hundred extra files

Disabling the failing struct by value scenario.

Merge pull request #4068 from Lucrecious/master

Generate map files for symbol resolution for Linux native images on PerfView

Refactor `MemoryBlockDesc`.

- Encapsulate link management into `MemoryBlockDescList`.
- Replace `MemoryBlockDesc` and its separately-allocated buffer
  with a new type, `MemoryBlock`, that stores the link to the
  next memory block with the buffer itself.

Merge pull request #4303 from CarolEidt/NoBarrierLclVarAddr

No GC barrier needed for indirection of expressions involving lclVars

Enable SOS gcInfo command. All it does basically is just enable the gcdecoder

Enable SOS gcInfo command. Basically all it does is just enable the code related to sos gcInfo and make it compiled for linux.

Replace `SList` in the GC info encoders.

Instead of using `SList`, manage the links manually. There are only
a few list manipulations in the encoders, so this is not too onerous.

Merge pull request #4295 from fadimounir/typesystemtests

Bulk port of TypeSystem test cases

Merge pull request #4310 from RussKeldorph/arglist

Port arglist tests for x86 and ARM

No GC barrier needed for LclVar address expressions

The check for the case where a barrier is not required when writing to a
local variable needs to include the LocalVarAddr and lclVar address
expression cases.

Update per comments

Merge pull request #4302 from kyulee1/badtest

ARM64: Tests update

ARM64: Tests update

- Delete bad test (sql_stress4) which uses Thread.Abort which is not in
  .NET Core.
- Delete non-existing test (MarshalBoolArray)
- Update time-out for StringConcat4/StringConcat8 which usualy failed due to time-out.
- Made _opt_dbgexplicit5/_opt_relexplicit5 UNSTABLE which pass mostly when
  running twice in the lab.
- Delete bad test (b519927) which uses AppDomain.DefineDynamicAssembly
  which is not in .NET Core.
- Delete bad test (b16241) that has overflow conversion (from double to int).

Fix issue unveiled on NetBSD: Add PAL__vsnprintf shadow in PAL

NetBSD's libc internally uses a function named _vsnprintf. We need to mask
PAL's version of this function to remove clash with the system. With it
snprintf(3) calls were jumping out of libc and landing back to PAL.

It was unveiled with many PAL tests.

Detailed documentation about _symbols in the NetBSD libc is documented
in the NetBSD sources in a file: src/lib/libc/README

Retire unneeded _vsnprintf().

Thanks Jan Vorli (Microsoft) for help with debugging.
Thanks Jan Kotas (Microsoft) for suggesting the right solution.

Fix #3199

Merge pull request #4074 from kouvel/SoftwareWriteWatch

Implement software write watch and make concurrent GC functional outs…

Port arglist tests for x86 and ARM

Arglist tests, which are only valid on Windows, have non-x64 versions that
needed to be ported.

Merge pull request #4273 from sivarv/multireg2

Force IR to be always var=call in case of multi-reg returning call nodes plus refactoring changes

Merge pull request #4255 from pgavlin/GcInfoArrayList

Refactor `StructArrayList` for the GC info encoder.

Implement software write watch and make concurrent GC functional outside Windows

- Implemented software write watch using write barriers
- A new set of write barriers is introduced, each corresponding to an existing one, but which also updates the write watch table. The GC switches to a write watch barrier during concurrent GC, and switches back to a non write watch barrier after the final query for dirty pages.
- The write watch table is alloacted along with the card table
- Since the card table is used differently, different synchonization is used for the write watch table. The runtime is suspended during resize since that is the most infrequently occuring operation, of that, ResetWriteWatch, and GetWriteWatch.
- ResetWriteWatch() doesn't need a suspend, but since the software WW version is much faster than the Windows version, moved it into the suspended region to avoid some synchronization that would otherwise be required
- The background calls to GetWriteWatch() don't need or do a suspend. They only need to synchronize with the resize path, not for the purpose of correct functionality, but to not miss dirty pages such that concurrent GC is effective.

Miscellaneous:
- Fixed runtests.sh to copy mscorlib.dll and delete the Windows version of mscorlib.ni.dll

Merge pull request #4278 from RussKeldorph/t1

Fix two "missing DLL" test failures

Merge pull request #4296 from mmitche/fix-prefix-casing

Fix output casing of prefix dir

Refactoring changes to use ReturnTypeDesc in codegenxarch.

Merge pull request #4279 from swaroop-sridhar/GC

GCInfo: Fix ARM64 GCInfo decoding

Bulk port of TypeSystem test cases

Clean up StructArrayList for GC info.

- Move the code into the GC info encoder, as that is its
  only consumer.
- Remove contracts
- Delete dead code
- Reformat and refactor to current coding standards

Merge pull request #4292 from janvorli/hw-exceptions-change

Fix hardware exception handling in native code on BSD

Merge pull request #4294 from kyulee1/fixsp

ARM64: Fix for recording SP for handler.

Fix output casing of prefix dir

The PREFIX dir is upper case, should be refered to as such.  Blocker for building on Linux

Fix two "missing DLL" test failures

Fix #2234 Missing native DLL
Fix #2237 Missing DLL is a MC++ test hiding alongside a C# wrapper--delete
    test which was mistakenly ported.

Merge pull request #4085 from LLITCHEV/Issue3618-1

Fix a bug in the code for inserting a tree in the linear order.

fix GCInfo build problem, tested a few simple "HelloWorld" program works good!

ARM64: Fix for recording SP for handler.

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

When GC occurs in a catch handler, we compare the current frame with the one
saved before the funclet was invoked.
The recorded SP address was wrong (offset by 16 byte), which caused FindParentStackFrameForStackWalk to null frame.
So, we ended up with assert on `ExceptionTracker::HasFrameBeenUnwoundByAnyActiveException(&m_crawl)`.
The issue was assembly helper, CallEHFunclet records FP instead of SP while SP is adjusted in prolog.

Merge pull request #4290 from leemgs/upstream-clang38

Add clang-3.8 into build.sh

Merge pull request #4187 from stephentoub/lcts_dispose_cancel_race

Fix race condition between CTS.Cancel and Dispose

Fix hardware exception handling in native code on BSD

There is a problem with hardware exception handling when the exception
happens in native code and it is under the HardwareExceptionHolder on
FreeBSD and NetBSD. The problem was that the C++ unwinder was unable
to cross the signal trampoline.
This change fixes the problem by throwing the PAL_SEHException from
the context of the hardware exception itself rather than from the
signal handler.
Since it uses the same code as we were using for StartUnwindingNativeFrames
in CoreCLR before, I have moved that stuff to PAL and let both CoreCLR and
the signal handlers use it too.
At the same time, I have reenabled the paltest_pal_sxs_test1 for FreeBSD and
NetBSD, since it now works.

ARM: fix build warning for unw usage

The warning is coming from libunwind-arm, not coreclr.
Libunwind-arm uses SP/PC registers along with other registers,
which clang complains.

fix #4259

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

Merge pull request #4280 from krytarowski/netbsd-support-59

Use _lwp_self() in THREADSilentGetCurrentThreadId() for NetBSD

Add clang-3.8 into build.sh

According to the official release note, clang 3.8 is also stable version.
http://llvm.org/releases/download.html
Let's add version 3.8 of clang into build.sh script.

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

Merge pull request #4274 from davmason/master

Bug Fixes for EventSource

Fix a bug in the code for inserting a tree in the linear order.

A long standing issue in the Compiler::fgInsertTreeInListBefore method was
found where the gtStmtList of a statement was not updated when a tree is
inserted before the first statement in the list (stmt->gtStmtList ==
insertionPoint). In such case the gtStmtList is not updated to include the
new tree(s) inserted.

After talking to few people offline to understand where this fix should be made
and realizing that the comment on Compiler::fgMakeEmbeddedStmt is
incorrect, I made the necessary fix in Compiler::fgInsertTreeInListBefore
so the newly inserted tree(s) in includedin the gtStmtList of the updated
stmt.

Fixes 3618.

Use _lwp_self() in THREADSilentGetCurrentThreadId() for NetBSD

The _lwp_self() call returns the LWP ID of the calling LWP.
It requires <lwp.h> on NetBSD.

Merge pull request #4101 from sejongoh/detect_ldsfld_access_non_static_field

Check if LDSFLD accesses non-static field

Merge pull request #4277 from mikem8361/rpath

Unable to debug coreclr apps on Debian 8.2 (Issue #4263)

Unable to debug coreclr apps on Debian 8.2 (Issue #4263)

Add set(CMAKE_BUILD_WITH_INSTALL_RPATH TRUE) to dbi and sos cmake files.

Merge pull request #4218 from mikem8361/symbols

Add symbol packages for coreclr runtime.

Merge pull request #4272 from AndyAyersMS/FixSpectralNorm

Fix validation of spectralnorm under xunit-performance

GCInfo: Fix ARM64 GCInfo decoding

GCStress traps were not inserted in ARM64 code correctly because safepoint
offsets were not correctly computed by EnumerateSafepoints().

This change fixes the issue by adding the (-1) adjustment to the ARM64
Safepoint offsets, similar to ARM and AMD64.

Merge pull request #4113 from swgillespie/long_running_gc

Create Long-running GC test job for the CI