Merge branch 'dotnet-standards' of https://github.com/jack-pappas/coreclr into jack-pappas-dotnet-standards

Merge pull request #476 from jkotas/viva

Fix some of the issues found by Viva64 static analyser

Merge pull request #469 from mmitche/mac-ci

Remove travis CI and add Mac badges

Fix some of the issues found by Viva64 static analyser

Merge pull request #473 from kangaroo/osx-debuggerpresent

Implement IsDebuggerPresent for OSX

Implement IsDebuggerPresent for OSX

Merge pull request #466 from mikem8361/printexception

Enable sos's PrintException command

Merge remote-tracking branch 'origin/printexception' into printexception

Build the PrintException command and related support functions.

Undef some temporary code that uses the global pointer table and fixed a pre-existing off by one error getting a managed string.

Fixed unnecessary string length addition.

Implement PAL_IsDebuggerPresent for Linux so the managed DebugBreak stops in native debugger.

Finish the off by one string length change.

@janvorli review feedback addressed: fixed buffer off by one error and use '\0' instead of 0.

@jkotas review feedback addressed: increate the file buffer PAL_IsDebuggerPresent uses to 2K.

Merge pull request #472 from jkotas/IsPrecodeByAsm

Delete PrecodeStubManager::PrecodeIsPrecodeByAsm method

Merge pull request #465 from janvorli/nonexistent-context-pointers

Make GC work without context pointers on OSX

Delete PrecodeStubManager::PrecodeIsPrecodeByAsm method

This method was expensive to port. There was just one place remaining that used. I have replaced it with different implementation.

Merge pull request #470 from qbit/master

Initial support for OpenBSD

Increate the file buffer PAL_IsDebuggerPresent uses to 2K.

Make GC work without context pointers on OSX

On OSX, the libunwind doesn't have support for getting context pointers.
This change modifies the way GC handles object pointers in registers
so that when the context pointer cannot be obtained, the object is
pinned so that GC doesn't move it and thus doesn't need to update
the object reference in the register value stored on the stack.

fix build on OpenBSD

fix indent

more indent

fix indenting

Merge pull request #433 from mmitche/fix-indepdent-test-build

Fixes for building tests alone

Remove travis CI and add Mac badges

Fixes for building tests alone

This change fixes an issue where tests cannot be built alone as well as cleans up the paths.  The general path cleanup was done so that the path format and variable names generally follow the main build of the coreclr repo.  Some highlights are:

 * Don't use Cmake as a directory name for intermediates, use intermediates instead.
 * Disable 4820 warning (this is about padding) when building tests.  Reduces noise in output significantly
 * Some fixes in typos and casing in the project files and build scripts

@janvorli review feedback addressed.

Merge remote-tracking branch 'upstream/master' into printexception

Merge pull request #464 from mmitche/remove-ci-script

Remove the CI cmd script, it's unused

Remove the CI cmd script.  Unused.

Merge pull request #453 from janhenke/freebsd

Inital build support for FreeBSD.

Inital build support for FreeBSD.

Add FreeBSD to various build scripts, so this platform is detected and properly
configured.

Merge pull request #459 from japj/japj-457

Remove duplicate object code from cordbdi

Merge pull request #449 from jkotas/mounted-binaries

Fix handling of links and non-Linux file systems in Unix corerun

Merge pull request #460 from japj/japj-458

Fix missing bool type for IsAppXNGen

Fix missing bool type for IsAppXNGen

Remove duplicate object code from cordbdi

eventredirectionpipeline.cpp is #included by platformspecific.cpp,
so remove it from the CMakeLists.txt since that causes linker warnings

Merge pull request #443 from japj/japj-113

Add check and instruction to build.cmd to run from normal prompt

Merge pull request #451 from japj/japj-106

Put Marshal.Read/Write with Object under FEATURE_LEGACYSURFACE

Finish the off by one string length change.

Grouped related Read/Write methods + consistent xml tag ending

rephrase into VS Developer Command Prompt

Implement PAL_IsDebuggerPresent for Linux so the managed DebugBreak stops in native debugger.

Put Marshal.Read/Write with Object under FEATURE_LEGACYSURFACE

The Marshal.Read/Write methods with Object arguments are not part of the
refactored .NET Core System.Runtime.InteropServices contract. Mark them
with FEATURE_LEGACYSURFACE so these get physically excluded from
mscorlib.dll on Linux/Mac.

Fix handling of links and non-Linux file systems in Unix corerun

Merge pull request #441 from stephentoub/openssl_init

Add OpenSSL initialization routine to PAL

Fixed unnecessary string length addition.

Undef some temporary code that uses the global pointer table and fixed a pre-existing off by one error getting a managed string.

Add check and instruction to build.cmd to run from normal prompt

Merge remote-tracking branch 'upstream/master' into printexception

Merge pull request #437 from mikem8361/clrstack

The sos "ClrStack" command now works

Merge remote-tracking branch 'origin/clrstack' into printexception

Code review feedback.

Build the PrintException command and related support functions.

Merge pull request #4 from kangaroo/clrstack

Fixes OSx problems.

Change THREADSilentGetCurrentThreadId to return the pthread_threadid, which matches the tid from lldb

Ensure we find_library on LLDB as well to find LLDB.framework on OSX

Merge pull request #439 from janvorli/add-unhandled-exception-trace-print

Add unhandled exception stack trace print

Add unhandled exception stack trace print

This change adds printing of unhandled managed exception stack trace to
console before exiting.

Fix CI build failures. Undefined functions.

Merge pull request #438 from janvorli/fix-frames-popping

Fix managed exception unwinding through CallDescrWorkerInternal

Exclude the Frame destructor from DAC build

The destructor is not needed on DAC builds and the definition was not being
built for DAC already. Conditionally disabling the declaration too.

Fix managed exception unwinding through CallDescrWorkerInternal

This change fixes issue with exception unwinding in the case when the unwinding
passed through a frame of the CallDescrWorkerInternal function. This function
had personality routine on it, but a windows style personality routine was specified.
The windows one has a completely different signature, so the code was crashing.
When looking into that, I've found that even if I have implemented a proper
Unix style personality routine, it cannot work the same way on Linux as it
used to work on Windows.
This personality routine's goal is to pop Frames from the Frame list in the current
thread so that all frames upto the frame handling the exception are popped.
There are two problems on Linux. First, unlike on Windows, the personality
routine is not passed the RSP of the frame handling the exception in an official
way. Although it can be extracted from the private_2 member of the exception
object during the 2nd pass, it is an implementation detail that we cannot rely on.
Moreover, even if we used that, it would still not be the right frame in all cases
due to the fact that we implement exception filters by catching and rethrowing and
so the frame we would get would be the frame of a filtering catch in case
there was one.
My solution to this problem is to add destructor to the Frame type and let it
pop the frame being destroyed if it is still in the list in the current thread.
That way the native code unwinding automatically takes care of popping the frames.
As an additional changes, I've added handling of the case when the
Thread::VirtualUnwindToFirstManagedCallFrame walks out of stack, fixed a stack
alignment issue in the recently added StartUnwindingNativeFrames function and
a cosmetic change in the UnwindManagedExceptionPass1.

Add PAL test for EnsureOpenSslInitialized

This includes removing an unnecessary assert from the function
that was causing the tests to fail, as well as addressing one
piece of PR feedback.

Add OpenSSL initialization routine to PAL

Several of the CoreFX libraries will be using OpenSSL on Unix for cryptographic needs.  In order for this usage to be safe in a multi-threaded context, OpenSSL needs to be initialized with a locking callback that provides the actual locking.  For now at least, the plan is to do this initialization in a function in libcoreclr.  This commit provides that functionality, which can subsequently be consumed by CoreFX libraries.

Merge remote-tracking branch 'upstream/master' into clrstack

Fix PAL printf functions.  Didn't handle "%*.*p" correctly which is supported under windbg/windows runtime.  Didn't actually support "*" for the width parameter at all for any format char.

Merge pull request #435 from janvorli/nested-exception-handling

Add support for exceptions crossing native frames

Merge pull request #436 from janvorli/resource-loading-workaround

Workaround native resource string loading temporarily

Add support for exceptions crossing native frames

This change adds support for unwinding exceptions that cross native frames.
These are for example exceptions thrown / rethrown from catch blocks.
The exceptions are unwound in an interleaved manner in this case. First,
all managed frames upto the first native frame are unwound (both the first
and second pass), then the native frames are unwound by standard c++ exception
handling, then the next block of managed frames is unwound etc.
The change also implements RtlCaptureContext and changes the managed exception
handling to use it instead of the GetThreadContext. The difference is that the
RtlCaptureContext gets context of the caller while the GetThreadContext gets
a context somewhere deep in the PAL and so unwinding from such a context to the
first managed frame would be walking old stack frames that can already be corrupted.
As an additional change, I have fixed a problem that prevented unwinding of
assembler functions that use the PROLOG_WITH_TRANSITION_BLOCK macro.
The macro was using CFI annotation for the xmm registers and the libunwind
doesn't support that.
Moreover, the PAL_VirtualUnwind return status was not being checked in the
`Thread::VirtualUnwindToFirstManagedCallFrame`, so the failure resulted in an
infinite loop in there.
Finally, the `debug/di/amd64/floatconversion.S` was including the unixasmmacros.inc
using a relative path, now after I've moved it to a different place, I've removed
the relative path since the new path is in the include paths.

Merge pull request #432 from kangaroo/pal-glccfos

Implement GetLogicalCpuCountFromOS in the PAL for OSX

Reflect feedback

Changed #ifndef FEATURE_PAL to #if 0 since the file is in PAL.

Merge pull request #427 from LLITCHEV/work

Fix some struct passing and tail call Jit problems on System V type of O...

Fix some struct passing and tail call Jit problems on System V type of OSs.

This change set contains the following:
1. Removes the PUSH/POP for RDI/RSI functionality from the Jit for Linux. Replaced with
standard register tracking and spilling by the RA.
2. Initialize the non stack homed, register passed param stack offset to 0
and allow the fixup routines to operate on the offset. Tail calls
implementation depends on the offset of the first param be set properly.
3. Initial work to allow for frame chaining using a FP register.

Implement GetLogicalCpuCountFromOS in the PAL for OSX

Workaround native resource string loading temporarily

This change is a temporary workaround for native resource string loading until we have
the resource string support implemented.
Instead of asserting in the resource loading methods, return a string representing
the id for the requested resource.
The change implements a similar workaround for the FMTMSG_GetMessageString which now
returns a string containing "Error errorCode" where the errorCode is the error code
passed to the method.
This way, exceptions that use those two functions can be raised without asserting.

The ClrStack command mostly works except for some address formating (related to %p).  Needed to change how the PAL got the thread id (pthread_self to gettid) and implemented a simple GetThreadContext function.

Merge pull request #434 from AlexGhiondea/concurrentDict

Remove the CDS_COMPILE_JUST_THIS define from the ConcurrentDictionary.cs file

Merge pull request #418 from AlexGhiondea/AppContextFix

Make sure that AppContext respects CompatibilitySwitches.IsAppSilverlight81 flag

Remove the CDS_COMPILE_JUST_THIS define from the ConcurrentDictionary.cs file.

We are not going to build that file stand-alone anymore.

Merge pull request #429 from jkotas/testfixes

Fix error handling in REGUTIL::EnvGetString

Fix error handling in REGUTIL::EnvGetString

Merge pull request #428 from ellismg/custom-entry-point

Add custom entry point support.

Add custom entry point support.

Hosts like ASP.net don't want to call ExecuteAssembly, because their
entry point is library, not a managed exe.  However, forcing cross
platform hosts to actually call CreateAppDomainWithManager and
CreateDelegate themselves requires a bunch of tedious code, since these
hosts don't have access to mscoree.h and the PAL.

This change adds the ability to specificy an assembly, type and method
which can be used as a custom entrypoint.  The signiture of the managed
entry point needs to be `static int E(int, char**)`.

Fix some more build problems/missing globals. The ClrStack command builds and almost runs except for a thread id problem.

Fix some missing variables and other build problems.  The ClrStack command now builds and almost runs. There is a problem with thread ids between the debugger/DAC and the target/VM/PAL.

The "ClrStack" command and related functions now build.

Merge pull request #426 from ramarag/merge-from-tfs

Merge from tfs

Merge remote-tracking branch 'upstream/master' into merge-from-tfs

Resolved Merge Conflict:
tests/src/AllTestProjects.sln

Make sure that AppContext respects CompatibilitySwitches.IsAppSilverlight81 flag.

This flag is used in cases where we don't have a Main() which means we won't be
able to read the TFM. Setting this compatibility flag will force the TFM to be
'WindowsPhone, Version=8.1".

Merge pull request #425 from jkotas/jitpinvokefix

Fix trashing of P/Invoke return value by GC probe

Merge pull request #416 from janhenke/master

Do not hardcode path to bash into the shell scripts.

Merge pull request #423 from jkotas/timerfix

Avoid call to QueryUnbiasedInterruptTime for PAL

Do not hardcode path to bash into the shell scripts.

Use /usr/bin/env bash to discover bash, independet of actual path. Closes #204.

Avoid call to QueryUnbiasedInterruptTime for PAL

Merge pull request #400 from mikem8361/dacglobals1

Implement the DAC global pointer table for Linux

Merge pull request #419 from ellismg/unblock-datetime-now

Return null for GetLocalTzFile

Merge remote-tracking branch 'upstream/master' into dacglobals1

Fix sos on window's build.

Return null for GetLocalTzFile

Throwing an exception from GetLocalTzFile causes DateTime.Now to throw
an exception and the runtime to tear down.  While we are bringing up
the globalization stack on Unix, we can just return null instead of
throwing.  This will cause us to use a zero UTC offset, so the local
times will not actually match the local clock, but we will be able to
get further along.

Merge pull request #415 from jkotas/zapwriter

Fix size computation in ZapWriter::WritePad

Merge pull request #411 from ramarag/fix_test_msbuild_warnings

Adding app.config to get rid of the warning MSB3276 in the Test Build

Fix size computation in ZapWriter::WritePad

Merge pull request #414 from jkotas/assertfix

Make assertion failures to break into debugger

Make assertion failures to break into debugger

Added additional standards (ECMA-372) and CLI technical reports (TR-084, TR-089) to documentation.

Adding app.config to get rid of the warning MSB3276