Merge remote-tracking branch 'upstream/master'

Merge pull request #2149 from alexandrnikitin/optimizer-remove-unsed-var

Remove unused variable from Optimizer Compiler::optHoistLoopExprsForTree method

Merge pull request #2132 from AndyAyersMS/FixLinuxTestScript

Fix generated test script to properly check for debugger

Merge pull request #2142 from steveharter/IcuPerf

Improve DateTime.Parse performance by reducing string comparisons

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

Merge changes from TFS

Improve DateTime.Parse performance by reducing string comparisons

Open sourcing ILDASM

[tfs-changeset: 1551732]

Merge pull request #2148 from manu-silicon/build

Build script now use current platform to compile

Merge pull request #2147 from manu-silicon/minor

Consistency fix for shell variables

Remove unused variable from Optimizer Compiler::optHoistLoopExprsForTree method

Build script now use current platform to compile

No need to specify the architecture to compile on Linux, we use
"uname -p" to infer it.
Removed __MSBuildBuildArch which was not really used, we can use
__BuildArch instead.
Added warning for x86.

Consistency fix for shell variables

Use 0 instead of false since later we compare it with 1.

Merge pull request #2136 from gsfreema/docs

Cleaning up Contributing Workflow documentation

Merge pull request #2134 from jkotas/NO_CRT

Delete code under !NO_CRT define

Delete code under !NO_CRT define

It is always defined - even in internal builds

Cleaning up Contributing Workflow documentation

The line 'Please following our commit messages guidance, below.' is
incorrect since the commit messages section is above.  To future proof
this line, I'm updating the line to provide a link to the Commit
Messages section.  Also, correcting 'Please following' to say 'Please
follow'.

Merge pull request #2129 from maksqwe/GenTreeBoundsChk_fix

Fix GenTreeBoundsChk::gtThrowKind check in GenTree::Compare(). It was comparing the same field of op1 instead of op1 and op2.

Merge pull request #2018 from AndyAyersMS/LessLogging

Reduce volume of MSBuild logging when running tests via xunit

Merge pull request #2133 from gsfreema/master

Fixing a grammar mistake in Documentation\README.md

Fixing a grammar mistake in Documentation\README.md

Simply using the correct conjugation of the verb 'support'.

Fix generated test script to properly check for debugger

Also tried fixing the generated help to show that you need `=`. This should be conditional based on `HasParam` but since there is only one switch it works for now.

Fix GenTreeBoundsChk:: gtThrowKind check in GenTree::Compare()

Merge pull request #2112 from ellismg/cache-ucollators

Cache UCollators in a Locale

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

Merge changes from TFS

Merge pull request #2117 from hickford/patch-1

Add link from readme to CoreFX

Fix build breaks in desktop build

[tfs-changeset: 1550852]

Merge pull request #2123 from stephentoub/cts_perf

Separate CancellationTokenSource linking into a derived class

Merge pull request #2122 from adityamandaleeka/testArgsQuoteFix

Replace double quotes around test arguments with single quotes.

Merge pull request #2096 from LLITCHEV/SysVTests

Fix for issue 2058.

Merge pull request #2120 from kyulee1/osx

Package spec for RyuJit on OSX

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

Merge changes from TFS

Cache UCollators in CompareInfo

Creating a UCollator is an expensive operation and we are presently
doing it on ever collation operation. We can improve this by caching
the UCollators we use for collation on the CompareInfo object itself.

This change introduces a new method GetSortHandle which gives back an
opaque wrapper which can be used in collation operations instead of a
culture name.

Internally we represent this is a struct holding the two types of
UCollators we care about (if we add additional collators per locale with
different options to handle other types of CompareOption flags, we can
cache these as well).  Collation methods can get a `const UCollator*`
reference from the sort handle which is safe to share across
threads (per the ICU Design Guidelines[1]).

Unfortunately, tracking the lifetime of the SortHandle itself is not as
straightfoward as I would like. Right now, we use a SafeHandle to wrap
the internal handle and rely on the finalizer of the class to clean up
the native resources. However this means that the following code sample
will create two finalizable objects:

```csharp
    var c1 = new CultureInfo("en-US").CompareInfo;
    var c2 = new CultureInfo("en-US").CompareInfo;
```

If this ends up being an issue, we could explore an approach where we
keep a cahce of SortHandles in managed code and pass out references to
that SortHandle which would let us share a single SortHandle for a given
locale across more than one CompareInfo object.

Wins are seeing in places where we previously did lots of string
comparisions in a tight loop (for example: dotnet/corefx#3811) moving
these operations down to ~6ms per iteration vs ~330ms on my local machine.

[1]: http://userguide.icu-project.org/design

Merge pull request #2107 from pgavlin/SysVCleanup

Small cleanups in SysV classification.

Separate CancellationTokenSource linking into a derived class

The m_linkingRegistrations field is only used on CancellationTokenSource instances created by CreateLinkedTokenSource.  This commit creates a private LinkedCancellationTokenSource to contain that field and related logic, so that the field is pay for play.  This shaves off ~11% of normal CancellationTokenSource's memory consumption.

Replace double quotes around test arguments with single quotes.

Fix build breaks in legacy build

[tfs-changeset: 1550765]

Added missing URL to .NET Foundation forums

Grammar fix-up

Added Full Framework support info

Package spec for RyuJit on OSX

Merge pull request #2116 from jkotas/gc-update

Update GC from CoreRT

Merge pull request #2114 from sergiy-k/byrefrax

Do not preserve volatile registers in JIT_ByRefWriteBarrier

Update README.md

Link to CoreFX repo

Fix build breaks after GC update

Do not preserve volatile registers in JIT_ByRefWriteBarrier

RyuJIT assumes that all volatile registers can be trashed by the CORINFO_HELP_ASSIGN_BYREF helper (i.e. JIT_ByRefWriteBarrier).
So we don't need to save and restore them and can reduce the size of the barrier by a couple of instructions.

Update GC from CoreRT

https://github.com/dotnet/corert/tree/master/src/Native/gc c03d823a11f54e734e340813e6cd1e543ca448ef

Merge pull request #2110 from janvorli/refactor-gc-types

Refactor gc types

Merge pull request #2109 from jkotas/glossary

Add ProjectN and Redhawk to the glossary

Adding a condition to exclude a registry check on certain builds.

Fix few missing casts

Strangely enough, the build was successful locally, but not on the CI
machines.

Merge pull request #2108 from wtgodbe/pathTests

Deleted path tests from coreclr repo

Merge remote-tracking branch 'upstream/master'

Merge pull request #2047 from eerhardt/ShortDate

DateTimeFormat.ShortDatePattern should use CLDR 'short' format on Unix.

Add ProjectN and Redhawk to the glossary

Some formatting cleanup

Code formatting (alignment) needs to be fixed at few places.

Deleted path tests from coreclr repo

Merge pull request #2100 from gsfreema/master

Fixing a typo in developer-guide.md.

Merge pull request #2104 from wtgodbe/deleteTests

Deleted duplicate globalization and localization tests from coreclr

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

Merge changes from TFS

Small cleanups in SysV classification.

- Clean up/clarify the logic in CEEInfo::getSystemVAmd64PassStructInRegisterDescriptor
- Move a static helper function off of MethodTable

Replaced remaining types

LPARAM, SSIZE_T

Merge pull request #1981 from pgavlin/SysV

Fix struct padding issues with SysV struct classification.

Deleted duplicate globalization and localization tests from coreclr

Add casts to APIs that require LONG / DWORD parameters

Fix ARM64 build break

[tfs-changeset: 1550309]

Fixing a typo in developer-guide.md.

The words 'are' and 'and' were mistyped and need to be switched.

Merge pull request #2098 from janvorli/fix-threadpool-shiftwaitarray

Fix issue with threadpool and wait for multiple objects on Linux

Changed LONG and ULONG

Fix issue with threadpool and wait for multiple objects on Linux

There is a method ThreadpoolMgr::ShiftWaitArray which uses memcpy to do
a move a segment of the waitPointer and waitHandle arrays one position
down, so the source and destination ranges overlap. However, it uses
memcpy, which on Linux copies items starting from the last one. So
the arrays get corrupted after the memcpy, containing multiple copies
of the last element and not containig some elements that were expected
to move.
The fix is to use memmove which should be used when the source and
destination memory regions overlap.

Fix for issue 2058.

AndyAyersMS ran the valgrind on the parts of the VM/Jit and discovered the
2 erors described in issue #2058. One of them is that theorder of two assert condition was
swapped. The other was that there were a number of uninitialized variables that
in some cases could cause undefined behavior.

Update osx-instructions.md

Added text to describe increasing the file limit so mscorlib will not fail during build with file limit errors.

Merge pull request #2092 from BruceForstall/CleanupTargetHIfdefs

Simplify JIT target.h #ifdefs for ETW_EBP_FRAMED

Merge pull request #2094 from rahku/master

Fix dac verification errors caused by 48617ae

Merge pull request #2091 from mikem8361/fixfreebsd

Fixed FreeBSD sos plugin build.

Fix dac verification errors caused by 48617ae

Simplify JIT target.h #ifdefs for ETW_EBP_FRAMED

Many macros related to register usage were #ifdef'ed such that under ETW_EBP_FRAMED, EBP
doesn't appear in the macro. The complication was made much worse when UNIX support was
added because it doubled the number of such macros.

I simplified this by defining macros for EBP based on ETW_EBP_FRAMED that can then be used
in subsequent macros defining various register sets and lists. This greatly reduces the
number of #ifdefs.

Fixed FreeBSD sos plugin build.

Merge pull request #2068 from mikem8361/palsxs

Add new pal test that tests exceptions across multiple PALs

Add new pal test that tests exceptions across multiple PALs in different modules.

Currently disabled for FreeBSD.

Fix struct padding issues with SysV struct classification.

This change revises the handling of padding bytes between struct fields
and bytes that trail the last field in a struct. These changes address
problems with structs that contain > 8 bytes between fields as well as
C# fixed arrays (which are emitted as a struct with a single field of
the fixed array element type and an explicit size derived from the
number of elements and the size of the element type).

The classification given to a particular byte that is not part of a
formal field depends on whether that byte is between fields or
trails the final field in a struct. Bytes in the former category--
padding bytes--are classified as NO_CLASS as per the SysV ABI spec.
Bytes in the latter category--"leftover" bytes after the final
field of a struct--are assigned the classification of the final
field in the struct. This allows C# fixed-size arrays to work
properly, as their representative structs are logically (but
unfortunately not physically) structs with element-count fields of
the element type. Were they explicit, each of these fields would
have the same classification.

Replace most windows style types by c++ types

This change replaces DWORD, LONGLONG, ULONGLONG, SIZE_T, BYTE, LONG32 and few more by
standard c++ types. The DWORD still remains present in signatures of Windows APIs used
by the GC. There are few issues with those that need to get fixed.

Merge pull request #2088 from kyulee1/corert

A few fixes for standalone Jit (RyuJit) for CoreRT integration

Merge pull request #2084 from benpye/revert-arm-build-break

Revert change in 4e06e42 that broke ARM cross build

A few fixes for standlone Jit build

1. Fix for assertion in fgAddFieldSeqForZeroOffset
The fix relaxed the assertion under FEATURE_REF_ZERO_OFFSET_ALLOWED.
The feature is enabled when the standalone jit is built.

2. Package spec files for Windows/Linux and redirection.

3. Rename ProtoJit to RyuJit

Merge pull request #2087 from blackdwarf/master

Add additional data to PATENTS.TXT

Merge pull request #2055 from LLITCHEV/SysVTests

Add the tests used to bring the System V struct passing/returning fun…

Add the tests used to bring the System V struct passing/returning functionality.

There are:
1. Tests for passing managed struct.\
2. Tests for returning managed structs.
3. Tests for PInvoke and reverse PInvoke using callbacks and native SO.

Add additional data to PATENTS.TXT

Merge pull request #2079 from mikedn/no-decomp

Ifdef out useless code in JIT lowering

Merge pull request #2082 from BruceForstall/Fix2053

Fix RyuJIT assert 'emitCurIG != emitPrologIG'

Revert change in 4e06e42 that broke ARM cross build

Merge pull request #2061 from akoeplinger/utf-escape

Replace Unicode symbols with escape sequence in test source files

Merge pull request #2010 from ellismg/update-test-documentation

Update testing instructions

Merge pull request #2064 from akoeplinger/failcount-zero

Don't print the "Some tests failed" message when _failCount is 0

Merge pull request #2062 from akoeplinger/dont-hardcode-libc.dylib

Fix two Marshal tests to not hardcode OSX libc

Merge pull request #2042 from kouvel/StartupFlags

Allow setting some startup flags outside Windows

Merge pull request #2049 from stephentoub/uuid_generate_random

Use uuid_generate_random instead of uuid_generate in CoCreateGuid

Merge pull request #2067 from akoeplinger/fix-overflow-gcstress

Fix OverflowException in b115557.cs GC test on systems with fewer than four processors/cores

Merge pull request #2073 from mmitche/fix-exit-code

Exit outerlevel build call if innerlevel exits with non-zero code

Fix RyuJIT assert 'emitCurIG != emitPrologIG'

This occurs when the call-to-finally block exists as the last code in the function
(including any funclets). After we generate all "normal" code, we start over and
generate code for all the prologs and epilogs. In this case, we didn't reset the
emitForceNewIG global, which determines if we should force a new IG to be created
the next time code is generated. It is used for specifying no-GC regions. This fix
initializes it (along with emitNOGCIG) at the beginning of generating and prolog
or epilog.