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

Merge changes from TFS

Merge pull request #1849 from janvorli/fix-fp-registers-offset

Fix floatingpoint register offsets in TransitionBlock for Unix

Merge pull request #1830 from erozenfeld/RotateBits

     Generate efficient code for rotation patterns.

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

Merge changes from TFS

adding more context

[tfs-changeset: 1540889]

Addressing Desktop Build break due to addition of a new non windows event in Clretwall.man

[tfs-changeset: 1540865]

Merge pull request #1816 from stephentoub/string_indexof_perf

Improve string.{Last}IndexOf perf on Unix for Ordinal/OrdinalIgnoreCase

Merge pull request #1840 from caslan/customnotificationfix

Fixed an issue with getting event size not working for custom notific…

Update IndexOfOrdinalIgnoreCase to use full code units

Improve string.{Last}IndexOf perf on Unix for Ordinal/OrdinalIgnoreCase

Our current implementation of IndexOfOrdinal for strings on Unix uses Substring to get the piece of the source string we care about; this results in an unnecessary allocation / string copy.  When using OrdinalIgnoreCase, we also convert both the source and search strings to upper-case using ToUpperInvariant, resulting in more allocations.  And our LastIndexOfOrdinal implementation delegates to IndexOfOrdinal repeatedly, incurring such allocations potentially multiple times.

This change reimplements Ordinal searching in managed code to not use Substring, and it implements OrdinalIgnoreCase searching via new functions exposed in the native globalization shim, so as to use ICU without having to make managed/native transitions for each character.

With the changes, {Last}IndexOf with Ordinal/OrdinalIgnoreCase are now allocateion-free (as you'd expect), and throughput when startIndex/count and/or OrdinalIgnoreCase are used is increased significantly, on my machine anywhere from 20% to 3x, depending on the inputs.

Merge pull request #1833 from kouvel/Inline2

Move some functions to allow them to be inlined with clang

Fix floatingpoint register offsets in TransitionBlock for Unix

The conversion between floating point register index and offset was incorrectly
multiplying the the index by 8 instead of 16. The xmm registers in the
TransitionBlock on Unix take 16 bytes each, not 8.

Merge pull request #1839 from mmitche/fix-linux-coreclr-tests

A couple tweaks to coreclr linux tests

A couple tweaks to coreclr linux tests

Fixed an issue with getting event size not working for custom notification event

Merge pull request #1820 from richlander/rich-cleanup

Update license attribution

Move some functions to allow them to be inlined with clang

For the moment, GetThread, _rotl/_rotr, Interlocked..., and similar functions were moved to allow inlining them with clang.

The plan is to eventually enable link-time optimization (LTO) to cover inlining across translation units.

Renamed some code files in PAL tests to compile as C++ to use the C++ inline semantics.

Merge pull request #1827 from brianrob/tracinginit

Initialize LTTng tracepoint providers automatically

Merge pull request #1751 from ramarag/eventsource

Enable EventSource Logging in Linux via Lttng

 Generate efficient code for rotation patterns.

     This change adds code to recognize rotation idioms and generate efficient instructions for them.

     Two new operators are added: GT_ROL and GT_ROR.

     The patterns recognized:
     (x << c1) | (x >>> c2) => x rol c1
     (x >>> c1) | (x << c2) => x ror c2

     where c1 and c2  are constant and c1 + c2 == bitsize(x)

     (x << y) | (x >>> (N - y)) => x rol y
     (x >>> y) | (x << (N - y)) => x ror y

     where N == bitsize(x)

     (x << y & M1) | (x >>> (N - y) & M2) => x rol y
     (x >>> y & M1) | (x << (N - y) & M2) => x ror y

     where N == bitsize(x)
     M1 & (N - 1) == N - 1
     M2 & (N - 1) == N - 1

     For a simple benchmark with 4 rotation patterns in a tight loop
     time goes from 7.324 to 2.600 (2.8 speedup).

     Rotations found and optimized in mscorlib:
     System.Security.Cryptography.SHA256Managed::RotateRight
     System.Security.Cryptography.SHA384Managed::RotateRight
     System.Security.Cryptography.SHA512Managed::RotateRight
     System.Security.Cryptography.RIPEMD160Managed:MDTransform (320 instances!)
     System.Diagnostics.Tracing.EventSource.Sha1ForNonSecretPurposes::Rol1
     System.Diagnostics.Tracing.EventSource.Sha1ForNonSecretPurposes::Rol5
     System.Diagnostics.Tracing.EventSource.Sha1ForNonSecretPurposes::Rol30
     System.Diagnostics.Tracing.EventSource.Sha1ForNonSecretPurposes::Drain
     (9 instances of Sha1ForNonSecretPurposes::Rol* inlined)

     Closes #1619.

Merge pull request #1828 from ellismg/fix-directory-casing

Fix directory casing of `regressions' in test/src

Enable EventSource Logging in Linux via Lttng

Fix directory casing of `regressions' in test/src

We had both tests/src/Regressions and tests/src/regressions which causes
problems because the on disc casing of files now differs based on if you
have a case insensitive file system or not.

This was confusing the the mirror a great deal because the casing of the
files is unified on the TFS side but the mirror was seeing these files
as renames.

Initialize LTTng tracepoint provider as part of loading libcoreclr.so.

Merge pull request #1809 from mmitche/add-x86-builds

Add x86 builds to CI (most off by default)

Add x86 builds to CI (most off by default)

* Builds are available through trigger phrases
* x86 windows_nt on for pushes by default
Additionally:
 * fixes workspace parameter quoting for Linux coreclr tests

Merge pull request #1788 from naamunds/UpdateXplatDoc

Update Linux and OS X build instructions

Update Linux and OS X build instructions

CoreClr (including mscorlib) and CoreFx can now be built using Mono on
both Linux and OS X; this change updates the instructions to reflect
that.  It also moves the instructions for the alternative option of
building the x-plat bits on Windows to cross-building.md.

This change also adds System.Globalization.Native, which is now
necessary.

At some point the FreeBSD documentation should also be updated to be
consistent with the Linux and OS X instructions; ideally, they should be
combined into one file, as discussed in https://github.com/dotnet/coreclr/issues/1590.

Merge pull request #1780 from adityamandaleeka/librarySearchPaths

Fix DllImport search logic with respect to prefixes/suffixes

Add STANDARD_VM_CONTRACT and remove loop that's no longer needed.

Merge pull request #1824 from Priya91/fixbug

Fix lpFileName leak in InternalCreateProcess.

Always attempt to load specified DLL name first before trying variations

Remove ReversePInvoke test from failing tests list.

Move prefix/suffix logic to dllimport

Fix lpFileName leak in InternalCreateProcess.

Merge pull request #1822 from janvorli/fix-pal-new-usage

Remove new operator usage in PAL

Remove new operator usage in PAL

This change fixes an issue caused by recent change in filename buffers
allocations. It was using the operator new, but it should not be used
in PAL since it invokes overriden operator new in the coreclr.
The fix is to use InternalMalloc instead.

Merge pull request #1808 from om3n07/master

Correct comments on Array sorting methods

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

Merge changes from TFS

Merge pull request #1817 from bryanar/pri1testfix

Fixing test infrastructure that affects pri1 tests.

Remove bad wording
One team borrowed code from another.

Add attribution for lookup3.c

Remove duplicate license

Add attribution for RFC 3492

Correct attribution for UUIDs and GUIDs

Remove NR reference
After speaking to the authors, the attribution comment is not required for this instance. However, Numerical Recipes in C is an awesome book written by some very smart and classy folks. It's a great book.

Merge pull request #1744 from Priya91/palstring-rest

Use StackString in PAL file apis.

Merge pull request #1813 from adityamandaleeka/pal_cleanup

Clean up cleanup code in PAL

Fix boundary bugs on buffer lengths. Include null char in MAX_LONGPATH.

Fixing some pri1 tests.

Merge pull request #1806 from LLITCHEV/SystemVStructs

Implementation of System V ABI struct passing.

Implementation of System V ABI struct passing.

This PR adds support for System V x86_64 ABI classification and calling
convention to the VM and the Jit, including, but not limited to Ubuntu
Linux and Mac OS X.
The general rules outlined in the System V x86_64 ABI (described at
http://www.x86-64.org/documentation/abi.pdf) are followed with a few
little exceptions, described below:
1. The hidden argument for by-value passed structs is always after
the ÎéÎíthisÎéÎí parameter (if there is one.). This is a difference with
the Sysetem V ABI and affects only the internal jit calling conventions.
For PInvoke calls the hidden argument is always the first parameter since
there is no ÎéÎíthisÎéÎí parameter in this case.
2. Managed structs that have no fields are always passed by-value on
the stack.
3. The jit proactively generates frame register frames (with RBP as a
frame register) in order to aid the native OS tooling for stack unwinding
and the like.

Remove unused full_cleanup parameter

Merge pull request #1811 from mikem8361/exceptfix

Managed unhandled exceptions for debugging and other fixes

Merge pull request #1805 from sergiy-k/nodebugmsgs

Disable DEBUG_MESSAGES in release build of CoreCLR PAL

Fix issue #1782 unhanded exceptions don't generate core dumps by changing HandleFatalError to end up calling TerminateProcess (which calls the unix abort()) instead of ExitProcess.

Fix assert when unhandled exception is unwound and restored some previous code in the first pass dispatcher so the managed debugger notification is sent at the right time (during the first pass instead after everything is unwound).

Also fixed issue #1799 by not masking SIGTERM anymore.

Merge pull request #1775 from mmitche/add-coreclr-linux-tests

Add coreclr Linux tests to CI definition

Fixup

Correct comments on Array sorting methods

Changed "recrusively" to "recursively" and "subtrack" to "subtract".

https://github.com/dotnet/coreclr/issues/1801

Merge pull request #1803 from adityamandaleeka/fix_4cf34fe

Revert some changed files from an incorrectly merged commit

Disable DEBUG_MESSAGES in release build of CoreCLR PAL

This reduces the size of libcoreclr by 1.3MB and improves performace results by a few percent on some benchmarks.

Merge pull request #1802 from jkotas/networking

Delete remaining vestiges of PAL networking support

Merge pull request #1800 from stephentoub/invariant_casing_fix

Fix Turkish i casing with invariant culture on Unix

Revert some changed files from an incorrectly merged commit.

Commit 4cf34fe seems to have included some unintentional changes which are
affecting the product. This reverts those files to what they should be.

addressing desktop build failure

Code which does not have FEATURE_EVENT_TRACE defined is using artifacts from it

[tfs-changeset: 1539531]

Delete remaining vestiges of PAL networking support

Fix Turkish i casing with invariant culture

It turns out there are some differences in the casing data used by Windows and ICU for our invariant culture.  In particular, Turkish 'i' is handled differently.

This change splits the shim ChangeCase function into three: ChangeCaseTurkish (used for Turkish culture), ChangeCaseInvariant (used for Invariant culture), and ChangeCase (used for everything else.  ChangeCaseInvariant includes the new special cases.  Call sites in the  managed code are localized to a single function that determines which native function to invoke.

Merge pull request #1785 from bryanar/managed-test-port

Managed Test Port

Removed #ifndef CORECLR blocks from shmemory.

Remove MAX_LONGPATH constraint checks from file name string length.

Use StackString in PAL file apis.

Managed Test Port

This is a collection of managed runtime tests from an internal legacy test tree.

Fix build break with WindowsOS CRT

_get_wpgmptr is not available in WindowsOS CRT. Replace it with GetModuleFileName.

[tfs-changeset: 1538931]

Merge pull request #1789 from jkotas/crsttypetool

Add CrstTypeTool source code

Add CrstTypeTool source code

This tool is required for updating autogenerated crsttypes.h file

Merge pull request #1779 from ramarag/control_xplat_event_logging

Controlling Xplat Event logging

Fix Windows x86 build - use macro for friend declaration

The friend declaration conflicted with the extern declaration on x86 because the macro uses fastcall.

[tfs-changeset: 1538868]

Merge pull request #1786 from kouvel/BuildFix

Fix Windows x86 build - use macro for friend declaration

Controlling Xplat Event logging:
Adding an environment variable COMPlus_EnableEventLog to turn on/off
Xplat Event Logging, by default Xplat Event logging is turned off

Merge pull request #1783 from nguerrera/explicit-location-model

Be explicit about exposing Assembly.Location in model.xml

Fix Windows x86 build - use macro for friend declaration

Merge pull request #1781 from nguerrera/expose-mvid

Expose Module.ModuleVersionId

Pass __IntermediatesDir to msbuild in build.sh

Currently, building coreclr x-plat fails if not running as root due to
the build trying to copy files to "/ToolRuntime".  The issue is that
$(IntermediatesOutputRootPath) is not set when running from build.sh,
and testruntime.targets in buildtools defines ToolRuntimePath as
"$(IntermediateOutputRootPath)ToolRuntime\".  The fix is to pass
__IntermediatesDir to msbuild in build_mscorlib, which results in
$(IntermediatesOutputRootPath) being properly set.

This change also adds a skipcoreclr flag to allow the user to skip the
CoreCLR part of the build, similar to the existing skipmscorlib flag.

Be explicit about exposing Assembly.Location in model.xml

Remove unnecessary condemnation and suspension in PALCommonCleanup.

Remove CPalSynchronizationManager::Shutdown

Remove unused step logic from PALCommonCleanup.

Fixup

Fixup

Fixup

Fixup

Expose Module.ModuleVersionId

Merge pull request #1778 from swaroop-sridhar/comment

Fix a comment in GcInfoEncoder.h

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

Merge changes from TFS

Fix a comment in GcInfoEncoder.h

Fix a stale comment about stack-slot-base
in the GcInfo Encoder interface.

Merge pull request #1690 from bryanar/changes-to-test-tree

Changes to Test Tree Infrastructure

Add missing .gitmirror file.

This new folder was introduced in 465f56ef0c9c4b125d890e7ac0757e77d54e193c and needs a .gitmirror file.

[tfs-changeset: 1538359]

Fixup

Merge pull request #1770 from adiaaida/addSkipMscorlib

Add skipmscorlib option to build.cmd