Add sos to the development package

Merge pull request #179 from maksqwe/hresult_return_fix

AcquireNoThrow(): return type must be HRESULT

Merge pull request #180 from maksqwe/check_result_fix

Properly check return value of OpenProcessToken()

Merge pull request #205 from richlander/master

Update docs and links to docs

Make links repo-relative

Update documentation and links

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

Merge changes from TFS

Fix assignment of float register when this present for System V systems

This change addresses few issues:
1. It shuts off an assert that is relevant for Windows AMD64 calling conventions.
2. It doesn't initialize stack offset for any args - on System V there is no outgoing args space for the callee reserved. And the spilled param offsets are calculated later than the init of arg vars.
3. It stops assigning the first float register to use  to the last used int register. The float registers are managed independently. In presence of this pointer the first float register on the callee side was calculated wrongly.

No asmdiffs for Windows. DDR tests passed.

[tfs-changeset: 1413622]

Merge pull request #172 from KindDragon/master

Add an .editorconfig file

Merge pull request #190 from akoeplinger/patch-1

Fix typos in botr-faq.md

Merge pull request #134 from poizan42/add-glossary

Add glossary to documentation

Fix typos in botr-faq.md

Merge pull request #187 from nico/warnfix

Fix a few Wextra-tokens warnings.

Fix a few Wextra-tokens warnings.

Merge pull request #184 from richlander/master

Add BotR FAQ

Merge pull request #185 from mmitche/gitter-badger

Gitter badger

Make column bold

Add gitter badge

Add BOTR FAQ

Properly check return value of OpenProcessToken()

Return type of OpenProcessToken() is BOOL.

AcquireNoThrow(): return type must be HRESULT

But all users of this function take result like "HRESULT".

Merge pull request #173 from AlexGhiondea/fixbuild

Provide defaults for the values that come from the build.cmd script.

Merge pull request #97 from mmitche/dev-package

Add development package to build

Merge pull request #175 from nicodeslandes/fix_buildtest_error_message

Fixed error message when running tests\buildtest.cmd

Add new nuget package for development usage

Add new nuget package that can be developed against.  Also modify the cmake lists of a few directories to add in missing PDBs.

Provide defaults for the values that come from the build.cmd script.

Add an .editorconfig file

What is .editorconfig:
http://editorconfig.org/#overview

Plugins for IDEs to support:
http://editorconfig.org/#download

Merge pull request #160 from kangaroo/master

Set the mach exception default to MachException_SuppressDebugging

Removed an extra space in buildtest.cmd, causing it to display an error.

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

Merge changes from TFS

Fix build break with minimal Windows OS CRT

va_copy is not available in the minimal Windows OS CRT

[tfs-changeset: 1412529]

Merge pull request #166 from richlander/master

Update mscorlib doc and add entry to readme

Update mscorlib doc and add entry to readme

Merge pull request #163 from richlander/master

Add Mscorlib and Calling Into the Runtime document

Merge pull request #138 from kangaroo/va_list_safety

va_list safety

Add Mscorlib and Calling Into the Runtime document

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

Merge changes from TFS

precompile common headers to reduce build time for win builds. With this build time on my machine has reduced from 19mins to 10mins.

[tfs-changeset: 1412352]

Set the mach exception default to MachException_SuppressDebugging for now, so that debugging with lldb isnt painful

Merge pull request #155 from kangaroo/master

config.h.osx was debug generated, causing issues with release builds

Merge pull request #146 from poizan42/fix-writeable-strings

Fix writeable string warnings. Closes #139

Minor code structure refactoring as requested by @jkotas in PR #138

Merge remote-tracking branch 'origin/master' into va_list_safety

Merge remote-tracking branch 'origin/master'

Do not define _DEBUG on all builds.

Merge pull request #69 from mathstuf/cmake-touchups

CMake touchups

Merge pull request #149 from richlander/master

Update author format

Update author format

Fix writeable string warnings. Closes #139

Use list(APPEND) rather than set(var ${var} ...) in CMake

This avoids a variable expansion and avoids the chances of a typo being
introduced in variable names.

Simplify configuration-specific definitions in CMake

Factor out the configurations supported rather than duplicating the
logic three times.

Use foreach(IN LISTS) syntax in CMake code

It avoids extra separation on semicolons if there are any and skips a
variable expansion.

Remove excess arguments to endfunction() and else()

Arguments to else() are always confusing and endfunction only needs to
match the first argument.

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

Merge changes from TFS

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

Merge changes from TFS

Fix undefined symbols in release build of libcoreclr on Linux

When building a release version of libcoreclr.so on Linux the LLVM toolchain makes optimizations that result in undefined symbols in libcoreclr.so which causes dlopen(“libcoreclr.so”) to fail at runtime. There were three problematic functions:

1. CQuickMemoryBase<SIZE, INCREMENT>::ReSizeNoThrow(SIZE_T iItems)
2. FixupDispatcherContext(...)
3. CodeGen::inst_set_SV_var(GenTreePtr tree); - this function is marked as inline and has the entire body under “#ifdef  DEBUG”.

To fix the problem with ReSizeNoThrow I added the “__attribute__((used))” attribute to the declaration of the function. This attribute is enabled for the llvm toolchain only. For the other two functions I have simply removed ‘inline’ since these functions are not performance critical.

[tfs-changeset: 1412107]

Add missing .gitmirrorall file.

We mirror the newly created Documentation fully between TFS and Git.

[tfs-changeset: 1412105]

va_copy exists on windows now as well as a define, so we can unify this code

Ensure that every usage of va_start is complemented by a va_end

va_arg can modify the va_list according to the specification

SString will attempt to use va_list multiple times in the scenario where the
target buffer is not large enough.  On platforms like OSX, va_arg modifies
the va_list, making these re-attempts overread and fault.  The solution is to
va_copy the va_list prior to use, to ensure the original is maintained for
future use.

Merge pull request #108 from jkotas/master

Fix Windows x64 corerun.exe to run anycpu 32-bit preferred exe

Merge pull request #130 from njy/patch-1

possible unresolved tension

Merge pull request #119 from cwmiller/fix-linux-short-filenames

Fix std::out_of_range exception with short filenames (Linux)

Add glossary to documentation

possible unresolved tension

http://xkcd.com/859/

Merge pull request #75 from PashaPash/resourceset_deadlock

Fixed deadlock in System.Resources.ResourceSet

Merge pull request #125 from kangaroo/issue-120

-nostdinc++ is not recognized by apple clang, but its also not needed

Merge pull request #126 from vkairys/documentation-links

Update intro-to-clr.md

Update intro-to-clr.md

-nostdinc++ is not recognized by apple clang, but its also not needed

Merge pull request #117 from kangaroo/osx

Initial Mac OSX Support

Remove static casting in favor of a helper method so we retain type safety with the compiler

Fix std::out_of_range exception with short filenames (Linux)

An unsigned integer was being used while scanning the CLR path to locate assemblies. If a file existed in the directory with a name shorter than the longest extension being checked for (.ni.dll), then the variable would underflow and cause an exception when used as an index on the filename.

Refactor to be explicit about the fact that there is no handler yet

Remove PowerPC code, as its unused anymore

Remove the old version of DBG_CheckStackAlignment and make sure the assembly version is guarded on _DEBUG like the original

Remove dead NPROCS code now that we use getconf

Remove cordbi from the OSX build, since its still missing some symbols

Refactor the DBG_CheckStackAlignment support to be more consistent with the rest of the PAL

Fixup a few changes missed when rebasing OSX support

Initially __LINUX__ was defined when building for OSX as well, bringing
in LONGDOUBLE_IS_DOUBLE 1, and some of the local SEH funcations.
Add a __APPLE__ guard around these as well, so they're properly included.

Initial Mac OSX Support

Supports building up the VM, PAL and various components on Mac OSX 10.10
There are some oddities with the Apple assembler not generating short jumps
and not supporting 1 byte relocs.

Merge pull request #100 from xyziemba/parallel_make

Use # of processors + 1 available to OS scheduler on Linux when building

Fix PAL_Initialize error reporting

Ensure that when PAL_Initialize fails, the last error is always set. There were failure cases where it didn't set the last error.
Change assert for sizeof(CRITICAL_SECTION) < sizeof(PAL_CRITICAL_SECTION) in CriticalSectionSubSysInitialize to static assert and change the return type to void.

[tfs-changeset: 1411882]

Merge pull request #115 from richlander/master

Initial commit of 'Introduction to the CLR'

Merge pull request #86 from maksqwe/assertion_search_fix

Fix non-null assertion search in assertion table

Initial commit of 'Introduction to the CLR'

Fixed deadlock in System.Resources.ResourceSet

ResourceSet internal enumerator uses the following lock order:
1. lock _resCache (as _reader._resCache)
       in System.Resources.ResourceSet+ResourceEnumerator.Entry
2. lock reader (this)
       in the body on of _reader.AllocateStringForNameIndex

RuntimeResourceSet.GetObject uses the following lock order:
1. Lock on reader
2. Lock on _resCache
    (on the same instance as passed to the reader ctor)

That causes deadlock when one thread tries to enumerate the ResourceSet
while another thread reads a specific object from the same set.

Added lock in System.Resources.ResourceSet+ResourceEnumerator.Entry,
so both code paths locks on reader first.

Fix Connect Issue ID #580259

Merge pull request #112 from kangaroo/drop-librt

Remote -lrt from the link as its not needed, and not cross platform

lrt is not cross platform, lets put this back in when we verify this branch on linux

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

Merge changes from TFS

Fix missing CHAIN_SUCCESS_COUNTER symbol in libcoreclr.so

Certain versions of the clang toolchain build libcoreclr.so with this symbol undefined. This symbol is just an alias of g_dispatch_cache_chain_success_counter and the issue was probably caused by the fact that we pass the CHAIN_SUCCESS_COUNTER as a parameter to macro and that macro then appends the @GOTPCREL to the symbol. It seems that in the problematic case, the assembler appends the @GOTPCREL to the CHAIN_SUCCESS_COUNTER instead of the original g_dispatch_cache_chain_success_counter symbol.
Changing the
CHAIN_SUCCESS_COUNTER = g_dispatch_cache_chain_success_counter
to
#define CHAIN_SUCCESS_COUNTER g_dispatch_cache_chain_success_counter
fixes the issue.

[tfs-changeset: 1411600]

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

Merge changes from TFS

Fix Windows x64 corerun.exe to run anycpu 32-bit preferred exe

The bug was caused by call to LoadLibrary in Windows corerun and coreconsole hosts that refused to load 32-bit preferred exe.

I have removed the problematic call to LoadLibrary. The purpose of this call was to normalize the exe path. The exe path is already normalized and so this call was unnecessary.

Fixes #92

Fix several warnings in Linux build

1) Usage of partially uninitialized variable when the compiler didn't know the condition is always true. This condition was calling a VolatileRead and only the VolatileRead actually matters so that right PDB annotation is generated and not optimized out in debug optimized builds. So I've removed the if.
2) Usage of nostdinc++ option as a general option - it complained when building C files
3) Usage of an unknown warning disabling compiler option with clang 3.5. The option was needed for clang 3.5.1 and later.
4) Usage of partially uninitialized variable in daccess.cpp - there is a code path that the compiler can see that doesn't initialize the status variable.
5) Empty body of a for loop warning in one or two PAL tests.

[tfs-changeset: 1411579]

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

Merge changes from TFS

Use N+1 processors instead of N procs on Linux build

N+1 processors appears to perform 2-3% faster than using
just N processors. I've done an A/B test both locally and on
a cloud VM.

Fixes for this and the previous commit were reported by Sedar
Dilek <sedar.dilek@gmail.com>. Thanks!

Fix build on Linux by resolving case sensitivity issues in md/datasource

[tfs-changeset: 1411310]

Use # of processors availabe to OS scheduler on Linux

`getconf _NPROCESSORS_ONLN` provides the number of cores available to
the OS scheduler, while `nproc` provides the number of cores available
to `nproc` itself. We want to use the former value since make spawns
new processes instead of running compilation tasks in the same process.
(FWIW, the two values are usually the same, but `getconf` gets us the
value that we actually want.)

Additionally, `getconf` is available on Mac OSX while `nproc` is not.

I also fixed the printout of the `make` command that's actually
executed when we build.

Merge pull request #96 from mmitche/exe-pdbs

Enable PDB generation for corerun and coreconsole

Merge pull request #82 from pgavlin/master

Build fixes for clang3.7 and systems without stropt.h.