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

Merge pull request #287 from mmitche/output-paths

Place binaries and intermediates in folders that include OS

Place binaries and intermediates in folders that include OS

Rework the output paths so that the OS may appear in them.  This then follows the same form as corefx.  This solves a number of problems:
    1) When building unix mscorlib after the regular build, we would use the wrong intermediates for incremental.
    2) When testing windows after building unix mscorlib we'd use the wrong outputs and fail.

This change removes our cmake directory and moves cmake generated files into intermediates (under OS/arch paths)
The reason is that the intermediates go under here when actually doing the build, and on unix based systems, we need to generate the cmake files for different builds into different directories for the purpose of preserving incremental builds.

Change clean behavior

Clean should simply delete the root binary.  We also should not make assumptions about
whether necessary directories are available when we begin to build and should create them every time if
necessary.

Create directories as needed.

Don't build the lldb plugin until we can install the lldb-3.5-dev package on the CI machines.

Merge pull request #320 from richlander/rich-build

Clarify Visual Studio pre-req

Merge pull request #326 from stephentoub/mres_datetime

Change MRES to use DateTime.UtcNow instead of Now

Change MRES to use DateTime.UtcNow instead of Now

In debug builds, ManualResetEventSlim is storing the last time that Set and Reset are called, using DateTime.Now.  This in turn requires time zone information, which is currently not implemented on Unix, causing any usage of ManualResetEventSlim.Set/Reset in a debug build to throw a NotImplementedException.

This change just switches the usage to be DateTime.UtcNow instead of DateTime.Now to avoid the need for time zone information.  I was tempted to remove these debug-only fields entirely, but in case someone's actually using them during debugging, doesn't seem particularly harmful to leave them.

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

Merge pull request #322 from libengu/AltJitNameChange

Change alternative JIT name to LLILC

Allow alternative JIT configurable.

Change alternative JIT name to LLILC

Clarify Visual Studio pre-req

Merge pull request #317 from janvorli/fix-cfi-annotation

Fix cfi annotation for NDirectImportThunk and JIT_RareDisableHelper

Merge pull request #319 from pgavlin/FixClangWarnings

Fix warnings generated when building with Clang on Linux.

Fix warnings generated when building with Clang on Linux.

There were three classes of warnings that needed to be fixed:
1) Warnings for unused typedefs
2) Warnings for pointer to boolena conversions that were always true
3) Warnings for taking the absolute value of unsigned values

(1) accounted for the majority of the warnings. All of these warnings
originated from macro definitions debugreturn.h, staticcontract.h,
or pal_assert.h. In the first two cases, typedefs were used as
compile-time checks to ensure certain invariants inside of method or
function bodies. These cases were addressed by using an empty member
of the typedef under an "if (0)" guard. In the third case, typedefs
were used for compile-time assertions that may appear in any location
in a source file; these have been updated to use C++11's static_assert.

(2) and (3) were much more straightforward: the former was addressed by
explicitly comparing the source pointer against NULL and the latter by
removing the offending calls to abs().

This change also contains a small update to .gitignore to ignore the
binaries directory and CMake artifacts created by build.sh.

Merge pull request #3 from kangaroo/soscommand1-osx

OSX Support

Merge pull request #316 from richlander/master

Add dotnet standards doc

Merge pull request #312 from janvorli/pal-exception-tests

Enable several PAL exception tests

Fix SOS command to not break the OSX build, and be linkable against apples LLDB.framework

Fix cfi annotation for NDirectImportThunk and JIT_RareDisableHelper

The title says all

Merge pull request #313 from pgavlin/LibUnwindCheck

Move the error that is emitted when libunwind is missing into CMake.

Add dotnet standards doc

Move the error that is emitted when libunwind is missing into CMake.

This provides a friendlier user experience when libunwind is not
installed, as configuration fails earlier than the build does.

Remove duplicate check for number of exception arguments

I have added a check for the exception arguments in my last commit, but it turns
out there was already one, only in a wrong place. So the exceptionRecord.NumberParameters
was set incorrectly to the unclipped value.

More review feedback changes. Fixed message when the sos module isn't found. Changed the plugin make file to build on OSx only if the environment vars are set. Fixed IP2MD command on windows.

Merge pull request #308 from janvorli/exception-handling

Reimplement native exception handling for PAL

Enable several PAL exception tests

This change enables several PAL exception tests to test the
new exception handling stuff.

Merge pull request #310 from kangaroo/issue-307

Check for libunwind in configure

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

Review feedback.  Fixed the sos plugin make file to use the LLDB_INCLUDE_DIR and LLDB_LIB_DIR environment variables if set. Otherwise, it assumes the lldb-3.5-dev package is installed.

Cleanup more

I am removing few other things from the seh-unwind.cpp. First, the code that verified the
stack unwindibility actually couldn't work since in most cases, it would reach a managed
frame and then get screwed. It happened to few people in the recent past and they
were mislead into thinking that it is a problem of the coreclr unwinder.
There is no reasonable way to find out whether a frame is managed or native inside PAL,
that's why I am removign that stuff.
Also, there was now unused vectored exception handling stuff that I am also removing.
Another change is in the pal.h / pal.cpp where the type of the argv parameter was
const char **argv, which prevented passing in char** (the compiler was ok with it in C,
but failed in C++). I have changed it to const char * const *argv, which allows
passing in the char** as well.

Add comments to PAL_TRY, PAL_EXCEPT etc.

This change just adds comments to those methods.

Improve error messaging to be a bit more helpful on linux

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

Merge changes from TFS

Unwind also needs _DEFAULT_SOURCE, in fact most everything does.  Lets move it to toplevel

Check for libunwind in configure, and setup seh-unwind to have possible other future alternatives

Add missing dummy PAL_CheckVirtualUnwind in Release

With the previous commit, I haven't noticed that the dummy PAL_CheckVirtualUnwind() is now
missing in the release build. Adding it back.

Merge pull request #309 from jkotas/deadcode

Delete dead code

Reflect PR feedback

Remove few classes / macros from pal.h that are not used anymore.
Fix some line continuation backslash identations.

Delete dead code

Reimplement native exception handling for PAL

This change removes the preexisting exception handling in PAL that was simulating Windows
SEH using a lot of low level machinery. The only remaining part is the code that raises
the exception and that extracts the exception context information.
The PAL_TRY, PAL_EXCEPT, PAL_FINALLY, ... macros are reimplemented using plain C++ exception
handling.

Change lldb plugin to use lldb dev package for its include and lib files.

NOTE: Now there is another pre-requisite package to install: sudo apt-get install lldb-3.5-dev

Not sure what to do for OS-X.

Update README.md

Fixed broken contributing guide link.

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

Merge pull request #294 from AlexGhiondea/localPublish

Setup the CoreCLR repo to use the LocalPackagePublish mechanism.

Fixed Windows build for the DAC by restore the lib order back. Add the MAKEDLLNAME macros to properly format the "sos" and "mscoredaccore" library names when loaded.  Cleaned up the sos command to only load the sos module once the first time the command is executed. Fixed a g_clrData ref count bug in LoadClrDebugDll.

OS bug 1704585: WoW64 breakpoint in crossgen!debugError when compiling ARMCHK

Remove call to DebugBreak when crossgen encounters a case not supported by MDIL. Such scenarios are expected in production.

Also fixed an uninitialized field in ZapperOptions, which caused some warnings from crossgen to be lost.

[tfs-changeset: 1418191]

Merge branch 'master' into soscommand1

Use the MAKEDLLNAME macro to properly name the dynamically loaded libraries .

Setup the CoreCLR repo to use the LocalPackagePublish mechanism.

This will enable the 'local publish' of the CoreCLR package to a location where
it can be picked up by the CoreFx build. What this means is that you will be able
to build the CoreCLR package and use that to run the CoreFx tests on top of it.

Changes required to make this happen:
 - Refactor the managed build to make sure the buildtools gets restored before
   we attempt to use the targets it provides.
 - Take a dependency to the Microsoft.DotNet.BuildTools to bring in the packages.targets
 - Update the $(PackagesDir) to point to the <repo root>\packages to make it consistent with CoreFx
 - Hook-up the LocalPackagePublish targets to the build of the CoreCLR NuGet packages

Use environment vars to point to the lldb source and libraries.  If not set, the lldb plugin doesn't compile.

LLDB_SRC_DIR - the root of the lldb source i.e. ~/linux/llvm/tools/lldb
LLVM_BUILD_DIR - the build directory where the lldb library is found.

Merge pull request #271 from kangaroo/more-va_list-safety

Remove PAL_printf_arg_remover

Merge branch 'unix_issue177'

Ifdef out some PE specific DAC code. Still need to replace some of it with a linux specific to build/read the dac globals table.

Reflect CR feedback

Rename VirtualUnwind in PAL to PAL_VirtualUnwind
Move .cfi annotation inside macros and update PROLOG_WITH_TRANSITION_BLOCK
to use these macros. I've verified that the annotation is correct by
stepping through the asm code and verifying that the stack trace is
correct at every instruction.

Merge pull request #292 from ellismg/remove-unix-specific-encoding-workaround

Remove Unix specific hack in Encoding.GetEncoding

Merge pull request #289 from kangaroo/master

Fix redefine of HAVE_FTRUNCATE_LARGE_LENGTH_SUPPORT

Remove Unix specific hack in Encoding.GetEncoding

During Hello World bring-up for Unix, we ran into a crash which at the
time I thought was incorrectly due to some missing support in
globalization leading to a first chance exception being thrown.

Instead the problem was that due to a JIT bug, a hashtable that the
Encodings code tried to construct was getting bad data passed to the
constructor which caused an exception to try to be thrown.  The change
I provided caused the hashtable codepath to not be taken and hence
masked the issue.

Now that the JIT bug is fixed, we can remove this workaround. I
verified that HelloWorld still runs on Unix with these changes.

Merge pull request #284 from kangaroo/unix_issue177

Refactor libunwind to work on osx

Fix redefint of HAVE_FTRUNCATE_LARGE_LENGTH_SUPPORT

Move the UNWIND_CONTEXT_IS_UCONTEXT_T check into cmake as well

Merge branch 'master' into unix_issue177

Conflicts:
src/pal/src/config.h.linux
src/pal/src/config.h.osx
src/pal/src/exception/seh-unwind.cpp

Refactor linux libunwind to work on osx

libunwind on linux exposes the unw_context_t directly as a
ucontext_t, but on OSX its an opaque data structure.  Additionally
UNW_REG_SP is read/write on OSX, but read-only on Linux.  As such
we need to diverge the libunwind code a bit depending on what flavor
we are running on.  There is one OSXTODO around the context pointers,
since we do not have unw_get_save_loc on OSX.

Merge pull request #283 from kangaroo/issue-281

The ftruncate tests eat up a lot of disk space, and build time -- disabl...

Merge branch 'master' into soscommand1

The ftruncate tests eat up a lot of disk space, and build time -- disable them

One more fix to DAC build. The wrong "delete" code was being used causing a fault.

Merge pull request #278 from richlander/richlander-ecma

Add type loader doc

Merge pull request #272 from kangaroo/pal-cleanup

PAL cleanup

Merge pull request #279 from richlander/richlander-ecma6

Update ECMA links to 6 ed

Implement native stack unwinding for Linux
This change implements native stack unwinding using the libunwind on
Linux. I have also fixed bunch of issues / details in the related code:
1) 0x in front of %p inside format string
2) Subtraction of -1 from dl_info.dli_sname
3) Added .cfi_xxxx annotation to the CallDescrWorkerInternal and the
LEAF_ENTRY / LEAF_END macros.
4) Changed local labels in the CallDescrWorkerInternal to be prefixed by
.L to see the CallDescrWorkerInternal in the stack trace
5) Changed moveOWord to use movdqu - it was being called with one of the
parameters unaligned

Update ECMA links to 6 ed

Add type loader doc

Merge pull request #277 from jonorossi/patch-1

Fixed syntax error in example code in mscorlib.md

Fixed syntax error in example code in mscorlib.md

More DAC build fixes. Always enable SELF_NO_HOST defined.

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

Merge pull request #274 from richlander/master

Add authors to code guide

Add authors to code guide

Remove PowerPC

Remove solaris support

Clean up pal configure stage

This commit removes a bunch of dead checks in the configure stage, as well
as fixing inconsistencies in HAVE_FTRUNCATE_LENGTH_ISSUE/HAVE_FTRUNCATE_LENGTH_BUG
off_t in file.cpp was declared after the goto, making it a protected scope, so it
needed to be moved up.

Remove PAL_printf_arg_remover - what its doing is against the standards, and breaks on some platforms

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

Merge changes from TFS

Merge pull request #268 from kangaroo/master

di can be built, but still cannot be linked.

Fix x86 build break

[tfs-changeset: 1415626]

Minor pal fixes when initialized for a dll.

di can be built, but still cannot be linked.

Merge pull request #266 from jkotas/simplify-ifdefs

Simplify platform ifdefs

Merge pull request #255 from kangaroo/pal-config-with-cmake

Refactor the PAL build configuration to detect at configure time

Fixed comments, replaced references to Evanesco with ARM

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

Merge changes from TFS

Correct the return values for the ungetc check

Improve workaround description and limit it to just MS compiler

[tfs-changeset: 1415470]

Merge pull request #265 from mmitche/debug-dev-package

When building debug, author the development package with an appropriate name

Missing file in last commit.

Fix building under linux on a linux file system (instead of mounted windows ntfs)

Simplify platform ifdefs

Simplify platform ifdefs like #if defined(_WIN64) || defined(_TARGET_ARM_) to #if !defined(_TARGET_X86_) based on @BruceForstall suggestion

Merge pull request #261 from stephentoub/fix_semaphore

Implement CreateSemaphoreExA/W via CreateSemaphoreA/W

Refactor the PAL build configuration to detect at configure time

The current PAL has binary check-ins of the config.h for Linux and MAC, long
term this is not sustainable.  This is a first pass at cleaning this up.
Please note, there are some static platform independent defines as I tried to
keep consistency with the current outputs of the not-checked-in configure.in.
As we clean up the PAL the CMakeLists should be revisited and redundant pieces
removed.