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)

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

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 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.

Enable PDB generation for corerun and coreconsole

Corerun and coreconsole are exes, so they don't share the same linker flags as the shared libraries.  Set DEBUG on the EXE flags so that Release always gets debug info.

Don't pass incremental linking, incompatible with debug type

Make DBI build on Linux

- Compile DBI on Linux.
- Link DBI on Linux to the point when only symbols related to INativePipeline are missing.
In order to completely link DBI we'll have to do some development, and it will be addressed by a separate change.

[tfs-changeset: 1411066]

Merge pull request #93 from mmitche/fix-build-error-codes

Fix build.cmd return codes

Fix build.cmd return codes

Build script was accidentally wiping the return code of the first msbuild step, causing us to continue building even in failure.  Fix this and do a little extra cleanup.

Merge pull request #50 from xyziemba/build_fix_1

Make build type and build.sh case insensitive

Fix Typo in messages within CMakeLists.txt

"DEBUG, RELEASE and RELWITHDEBINFO" becomes
"DEBUG, RELEASE, or RELWITHDEBINFO"

Modify else/endif statements to remove optional expressions

This change removes the expressions within the elseif and
endif blocks. This aligns my changes with pull #69 and should
prevent a merge conflict.

Merge pull request #66 from cwmiller/master

Fix build.sh variables

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

- Fix setup-compiler-clang to detect binaries outside of /usr/bin
- Fix ambiguously-sized cmp instructions when comparing g_GCShadow and 0
- Remove references to stropt.h. None of the defiinitions in this header
  are used by the product, and this header is not present on certain
  systems.

Merge pull request #42 from pborreli/typos

Fixed typos

Fix non-null assertion search in assertion table

            if (impAssertion->assertionKind != OAK_NOT_EQUAL ||
                impAssertion->op1.kind != O1K_LCLVAR ||
                impAssertion->op1.kind != O2K_CONST_INT ||
                impAssertion->op1.vn != chkAssertion->op1.vn)

This condition is always true.

Merge pull request #80 from sergiy-k/binplace-on-linux

Enable binplacing of native CoreCLR binaries on Linux

Add default values to CMAKE_BUILD_TYPE

This helps interactive CMake config tools populate the options
to display for the CMAKE_BUILD_TYPE variable.

Enable binplacing of native CoreCLR binaries on Linux

This change adds support for binplacing native CoreCLR binaries on Linux to "$__RootBinDir/Product/$__BuildArch/$__BuildType". It reuses the mechanism that we currently have in place for Windows. Similarly to binplace implementation for Windows, it sets an environment variable (__CMakeBinDir) in build.sh and then uses its value in clang-compiler-override.txt to set CMAKE_INSTALL_PREFIX to the desired target location.

Fixed variable issues in build.sh

The clean argument wasn't properly setting the __CleanBuild variable to trigger the clean build operation.
The current build target/type message was using CMD variable substitution syntax instead of BASH.

Merge pull request #36 from benaadams/patch-1

Avoid unnecessary work for identical locations in Buffer.BlockCopy

Avoid unnecessary work for identical locations in Buffer.BlockCopy

Perform validity checks to ensure parameters are correct but short-circuit
out memmove when exactly the same data would be copied to the same location.

There are a number of occasions; which can be intentional or unintentional,
where the buffer being copied is the same place - e.g an internal buffer is
the same as the return buffer, and there is no need to call memmove's
overwrite safe copy.

Generally the call to BlockCopy will be in a library so it is more practical
to enable the check here rather than alter all the calling functions,
including 3rd party libraries to preform additional checks.

Merge pull request #64 from gablank/linux-build-args

Update usage text to show correct arguments.

Merge pull request #71 from Alexx999/fix_issue_68

Fix build inside badly named directory

Update usage text to show correct arguments.

The usage text printed the arguments with a capital letter,
whilst the script looked for the same arguments but without
the leading capital letter.

Also reordered the argument help list so it is in the same
order as the script accepts them.

Fixed typos

Fix build failure with message "Cannot create a package that has no dependencies nor content." #68

Merge pull request #67 from mogemimi/fix-typo

Fix typo: VT_UNKOWN -> VT_UNKNOWN

Merge pull request #56 from maksqwe/fix_get_jit_manager_list

Fix "pNeeded" increment in GetJitManagerList()

Fix typo: VT_UNKOWN -> VT_UNKNOWN

Merge pull request #62 from GeorgeSapkin/comments

Fixed typos

Merge pull request #53 from ellismg/clean-unicode-characters

Remove non ASCII characters from source files

Merge pull request #58 from ellismg/fix-h2inc-line-wrapping

Fix h2inc.ps1 generating invalid files on Windows 7

Merge pull request #46 from AlexGhiondea/master

Remove extraneous mscorlib.txt file

Fixed type: more_spacee_lock > more_space_lock

Fixed typo: gc_reaon > gc_reason

Fix h2inc.ps1 generating invalid files on Windows 7

On PowerShell 2.0, text written via Write-Output is wrapped based on
the current console settings, even when output is redirected to a
file. This means that the include file we generate will have lines
hard wrapped unless the console is configured to have a width of >
~140 characters. PowerShell 3.0 doesn't seem to have this issue (or
the default width is high enought that we don't run into the
limitation), but PowerShell 2.0 is the default on Windows 7.

This fix explicitly calls Console.WriteLine, which will cause the
output to not be wrapped. Doing things this way means we don't have to
play around with the host's buffer size.

Fixes #57

Fix "pNeeded" increment in GetJitManagerList()

"++" has a higher priority than "*".

Remove non ASCII characters from source files

Our native files were more or less encoded in Windows-1252, which
causes problems when we try to compile them on machines where the
current codepage can't represent everything that Windows-1252
can. With this conversion I just moved characters to their ASCII
counterparts (e.g. no smart quotes, the section marker glyph is now
"Section"). There were two places where I couldn't do the straight
forward thing, in object.h we wanted to insert the Per Mille symbol in
a comment so instead I just spelled out the Unicode codepoint. In
morph.cpp, there was a comment pointing to a paper by Torbjörn
Granlund (note the diaeresis above the second o).  In this case,
unfortuntely I had to just drop the diaeresis.  However, searching for
"Torbjorn Granlund" will lead you to the right person.

Fixes #49

Merge pull request #48 from ellismg/move-shebang-to-top-of-file

Move shebang to top of shell scripts

Make build type and build.sh case insensitive

Before this change, build.sh requires lowercase commands, but
CMAKE_BUILD_TYPE required an uppercase parameter. This fixes that
mismatch by making both case insensitive.

Merge pull request #23 from adityam/patch-2

Correct typo and link to the correct license file

Move shebang to top of shell scripts

This line should be at the top of the file, not after
comments. Otherwise the correct shell is not picked in some cases
(like when you the scripts file via sudo).

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

Merge changes from TFS

Remove extraneous mscorlib.txt file

Move the windows unwinder code out of the debug folder.

It is a preparation for using the DAC unwinder code  as an unwinder
for the jitted code on Linux, because the jitter generates windows
style unwind info.
The unwinder is build as a static library and linked to mscordac.

[tfs-changeset: 1409640]

Merge pull request #35 from SirCmpwn/fix-linux-build

Fix error locating llvm-ar during build on Linux

Merge pull request #38 from ramarag/GC_test

Fixes #37 Missing Files to build GC stress Tests

Fix error locating llvm-ar during build on Linux

This manually locates the llvm toolchain and sets the relevant cmake
variables. The _CMAKE_TOOLCHAIN_PREFIX variable is internal and
undocumented and should probably be avoided.

Missing Files to build GC stress Tests

Merge pull request #32 from jasssonpet/master

Correct typos in paraminstanceapi.h

Update link in PATENTS.txt

Correct typos in paraminstanceapi.h

Merge pull request #20 from ellismg/fix-mscorlib-warnings

Fix mscorlib warnings.

Merge pull request #21 from AlexGhiondea/master

Update copyright header

Merge pull request #24 from DnlHarvey/patch-2

Update README.md

Update README.md

- Fix the coreclr blog post to go to the right link.

Merge pull request #19 from weshaggard/FixCMakeMessage

Update the CMake requirement message in build.cmd.

Correct typo and link to the correct license file

Update copyright header

Update the CMake requirement message in build.cmd.

Fix mscorlib warnings.

PinnableBufferCache uses some of the declaritive CAS attributes which
don't mean anything on CoreCLR. We had disabled this warning
internally, but this copy of the file is specific to open source and
we didn't disable it in this file.

This simply ports the change to disable this warning to mscorlib's
copy of the file.

Merge pull request #18 from mmitche/update-job-names

Update job names

Update job names

Merge pull request #15 from xyziemba/master

Parallelize Linux build by adding "-j `nproc`" to make

Merge pull request #13 from ramarag/GC_test_build

Enabling GC Stress test build

Parallelize Linux build by adding "-j `nproc`" to make

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

Merge changes from TFS

This change fixes a potential problem in unwinding on Linux. PUSH and POP instructions were used to preserve RSI/RDI on the stack when stosd instructions are used to initialize the stack vars. These registers are used as the first two parameters to pass parameters to a callee.
The change makes sure there is a FrameRegisterUsed if these PUSH/POPs need to occur. If there is no FrameRegister, the unwinding, if attempted between the first PUSH and the last POP would cause incorrect unwinding of the stack.

SharedCodebaseChange: Yes
SharedCodebaseChangeRisk: Low

[tfs-changeset: 1409112]