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]

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)

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.

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