Fix for a rotation transformation bug.

rotatedValueBitSize should be calculated from genActualType(rotatedValue->gtType) instead of rotatedValue->gtType.
I added a test that was failing with an assert because of this bug: rol32ushort.

Merge pull request #2052 from stephentoub/startswith_null

Add USEARCH_DONE check to StartsWith in System.Globalization.Native

Merge pull request #2057 from bryanar/loader_tests

Porting loader tests from internal legacy tree.

Merge pull request #2060 from mmitche/fix-arm-rootfs

Fix typo in rootfs name

Fix typo in rootfs name

Porting loader tests from internal legacy tree.

Add USEARCH_DONE check to StartsWith in System.Globalization.Native

The StartsWith ICU wrapper was not checking the result of usearch_first
to see if it was USEARCH_DONE, indicating no match found.  This has two
ramifications:
1. When there isn't a match, USEARCH_DONE (-1) gets passed in as the
textLength argument to ucol_openElements, which treats -1 as meaning
the string isn't null-terminated, and thus ends up walking the string
looking for non-ignorable collation elements.  Our tests have been passing
because they've been using strings containing only non-ignorable
elements, and thus the first character checked causes us to bail and correctly
return false.  If nothing else, this is an unnecessary perf overhead.
2. But on top of that if there are only ignorable collation elements
before the first null character in the string (e.g. if the string begins
with a null character), then because we told ICU that the string ended
at the first null character, it'll stop walking the string and return
a match. e.g. "\0bug".StartsWith("test") returns true incorrectly.

This commit simply adds a check for USEARCH_DONE to StartsWith.
EndsWith already has such a check.

Merge pull request #2043 from stephentoub/startsendswith_null

Pass target string lengths to ICU on Unix

Merge pull request #2046 from JohnChen0/heap

Remove an unsafe memory write in HeapFree

Remove an unsafe memory write in HeapFree

The PAL implementation of HeapFree assumes that the memory region always
have at least 4 bytes. Depending on the heap implementation, this is not
necessarily true, and can potentially cause memory corruption.

Merge pull request #2045 from JohnChen0/master

Fix buffer overrun in SetMscorlibPath

Fix buffer overrun in SetMscorlibPath

Pass target string lengths to ICU on Unix

Our current ICU shims for StartsWith, EndsWith, IndexOf, and LastIndexOf
take the length of the source string but not the length of the target
string.  This forces ICU to compute the length of the string by searching
for a null terminator.  We can save those costs and be more accurate
around nulls in the target string by passing the known length in.

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

Merge changes from TFS

Merge pull request #2033 from brianrob/crossgen_perfmap2

Enable Offline Generation of Text-Based Symbol Tables for NGEN Images

Merge pull request #2030 from jkotas/vs2015-update1-buildbreak

Fix build break on VS2015 Update 1

Merge pull request #2013 from mikem8361/createthread

Fix Thread.Start while debugging bug on OSX.

Merge pull request #2031 from janvorli/fix-exceptions-in-filter

Fix processing of unhandled exception in exception filter

Fix Thread.Start while debugging bug on OSX.

The OSX exception logic is running on a separate thread from the one that the exception happened. The CatchHardwareExceptionHolder::IsEnabled used to check for h/w exception holders assumed it was running on the thread the exception/holders happened not the exception notification thread. Moved the h/w exception holder count to the CPalThread object instead of a TLS thread variable so the OSX exception code can check it given a CPalThread instance.

The main problem was that the stubmgr when creating a thread (for the start notification) put a breakpoint in ThePreStubPatch which is in the coreclr text section and because the h/w exception holder check was broken, it thought the breakpoint wasn't the debugger's and aborted the coreclr process.  The other part of this fix is to put a h/w exception holder around the called to ThePreStubPatch in prestub.cpp.

The stubmgr's delegate invoke subclass used the wrong registers for Linux/OSX to read the "this" and parameter registers. Added the proper ifdefs and registers (ecx -> rdi, rdx -> rsi) for the Unix platforms.

On both Linux and OSX, the h/w exception holder check in the exception routing logic needed to check if the int3/trap is in DBG_DebugBreak so asserts always abort/core dump. Move DBG_DebugBreak to an assembly worker so the start/end address can be used for this check.

Fix inconsistent security annotations

[tfs-changeset: 1547876]

Merge pull request #2032 from mmitche/add-cross-packages

Add missing packages for arm/arm64 cross build

Add missing packages for arm64 cross build

Enable offline generation of text-based symbol tables for native images.  This is needed in order to profile applications on Linux using perf_event.

This change also modifies the runtime to emit a perfmap entry when a native image is loaded so that offline trace processing tools can identify when a perfmap file for a native image is needed and generate it.

Merge pull request #2024 from jkotas/crossgen-ifdefs

Improve FEATURE_NATIVE_IMAGE_GENERATION ifdefs

Fix build break on VS2015 Update 1

Fix processing of unhandled exception in exception filter

This change fixes processing of unhandled exceptions in exception filter on Unix.
Before this fix, such exception leads to assert in the exception handling code
due to the fact that the exception tracker of the unhandled exception is not
popped from the exception tracker stack of the thread.
Although the PopTrackers method was being called, it bailed out due to the fact
that the exception tracker belonging to the unhandled exception has empty scanned
stack range.
The fix is to explicitly pop the tracker in the EX_CATCH that catches the
exception from the filter.
I have also re-enabled the JIT test b68872 that has uncovered the issue.

Merge pull request #2020 from pgavlin/StructABITests

Add some additional cases to the struct ABI test.

Merge pull request #2009 from LLITCHEV/Issue1995-1

Fix for issue 1995.

Merge pull request #2026 from AlfredoMS/async_fix

Fix Task so that RunContinuationsAsynchronously flag works with all continuations

Merge pull request #2028 from dotnet/revert-2023-disable-x86-release

Revert "Disable x86 windows release build until it passes"

Fix for issue 1995.

This changes containg a fix for Issue 1995.
When remorphing an argument that has been already completed (in case of
CSE rewriting for example, as in this case) the classification code was
not run properly and it is triggering an assertion. As the Issue 1995
states the assert failure is benign since the generated code is correct -
the change of registers used during completed argument classification is
not allowed and the code is not changing them.
This changes add the necessary code to properly classify and assign
registes for completed arguments remorphing, which complies with the expectations of the failing assert.

Merge pull request #2027 from erozenfeld/RotationImprovements

Implement improvements for rotation matching.

Revert "Disable x86 windows release build until it passes"

Merge pull request #2025 from jkotas/fatal-error

Use volatile store for g_hrFatalError

Implement improvements for rotation matching.

1. Recognize patterns with XOR instead of OR, e.g., (x << 5) ^ (x >> 27).
2. Recognize patterns with instance or static fields and ref params.
Only GTF_PERSISTENT_SIDE_EFFECTS (i.e., calls and assignments) and
GTF_ORDER_SIDEEFF (i.e., volatile accesses) prevent tree matching.
Before this change we used GTF_ALL_EFFECT.

Fix Task so that RunContinuationsAsynchronously flag works with all types of continuation

Use volatile store for g_hrFatalError

Merge pull request #1985 from schellap/master

Emit custom test launcher hook for custom test runners

Improve FEATURE_NATIVE_IMAGE_GENERATION ifdefs

Merge pull request #2023 from mmitche/disable-x86-release

Disable x86 windows release build until it passes

Disable x86 windows release build until it passes

To trigger in the meantime, say: 'test x86 windows release'

Merge pull request #2015 from kouvel/TpaBinder

Fix a couple of issues involving assembly loading

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

Merge changes from TFS

Fix a couple of issues involving assembly loading

Fixes #1740
- After an assembly is loaded through a custom load context, when its dependencies are being loaded and the dependencies are in an app folder but not on the TPA list, don't search app paths when using the TPA binder, and instead use the default binder for the dependent assembly to load dependencies through the same custom AssemblyLoadContext.

Fixes #1187
- In the AssemblyName created by AssemblyLoadContext.GetAssemblyName(), don't initialize the CodeBase property since it's not in the exposed surface area for .NET Core. If the AssemblyName is passed to Assembly.Load(), note that regardless of the path sent to GetAssemblyName(), default search orders are still used to load the assembly by name. A custom AssemblyLoadContext would need to be used to control where an assembly is loaded from.

- Fixed a couple of error messages that mentioned phone

Add some additional cases to the struct ABI test.

These tests cover boundary cases for the SysV ABI.

Merge pull request #2019 from mmitche/no-pal-tests-for-x86

No pal tests for x86 windows nt

Emit custom test launcher hook for custom test runners

No pal tests for x86 windows nt

Merge pull request #2012 from adiaaida/x86failures

Add x86 build to PR test and remove xunit

Merge pull request #2011 from kyulee1/fixbuild

Fix for build break

Merge pull request #1928 from adityamandaleeka/jenkins_enable_unix

Exit runtest.sh with a failure code if any tests fail

Add x86 build to PR test and remove xunit

This change adds an x86 build to the list of jobs started with each PR so
that we make sure we don't break the x86 build. Also remove xunit parsing
from x86 jobs since they do not run tests.

Fix for build break

When building llilc against coreclr in Linux, m_IsSlotTableFrozen is defined under _DEBUG
but it is used without guard "#ifdef _DEBUG"
This is fix for it.

Temporarily disable StructABI test to unblock CI while issues are resolved

Merge pull request #2006 from richardlford/gettypetypeofmatrix

Respond to review of gettypetypeofmatrix test.

Respond to review of gettypetypeofmatrix test.

The automatic formatter missed putting "public" and
"static" in a canonical order, so do that manually.

Also remove a case of Hungarian notation.

Merge pull request #2000 from richardlford/gettypetypeofmatrix

Add gettypetypeofmatrix test.

Merge pull request #1970 from mmitche/enbable-by-default

Enable coreclr tests by default

Enable coreclr tests by default

Merge pull request #1999 from adityamandaleeka/remove_suspension

More PAL thread cleanup (last one)

System.BitConverter.ToString(Byte[] value, Int32 startIndex, Int32 length) value is null message

[tfs-changeset: 1546665]

Add gettypetypeofmatrix test.

Ported the JIT/Directed/gettypetypeof/gettypetypeofmatrix.cs
test from the desktop clr suite to coreclr.

Remove InternalRand.

Remove thread parameter from several more PAL functions.

Merge pull request #1998 from pgavlin/AddNinjaSupport

Add support for ninja to the CoreCLR build.

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

Merge changes from TFS

Add support for ninja to the CoreCLR build.

Just what it says on the tin.

Merge pull request #1990 from stephentoub/fix_eventsource_typo

Fix a typo in EventSource_EventParametersMismatch error string

Merge pull request #1988 from stephentoub/fix_eventsource_emptyarray

Fix exception when writing null/empty arrays to EventSource

Expose Environment.GetCommandLineArgs in mscorlib

[tfs-changeset: 1546251]

Merge pull request #1925 from akoeplinger/remove-env-helper

Remove Env helper class in tests

Merge pull request #1926 from akoeplinger/fix-hardcoded-windows

Fix hardcoded Utilities.IsWindows property

Merge pull request #1987 from bbowyersmyth/StringCodeComments

Improved and corrected some String code comments

Fix a typo in EventSource_EventParametersMismatch error string

Fix exception when writing null/empty array to EventSource

EventSource has two WriteEvent overloads that take a byte[].  If the
byte[] provided is null, it uses an empty array.  It then tries to get the
address of the 0th element of the array, resulting in an
IndexOutOfRangeException.  We could instead just use ```byte* blob =
arg1``` instead of ```byte* blob = &arg1[0]```, but doing so with an empty
array has undefined behavior in C#.  Instead, we just special-case null
and empty arrays.

Improved and corrected some String code comments

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

Merge changes from TFS

Merge pull request #1984 from caslan/removeCSExports

Remove unneeded EnterCriticalSection / LeaveCriticalSection exports

Remove EnterCriticalSection / LeaveCriticalSection exports that are not needed

Fix build break when NO_HOST_CPP_EH_ONLY is defined

[tfs-changeset: 1546051]

Merge pull request #1959 from adityamandaleeka/remove_suspension

Remove thread parameter from a few more places in PAL

Merge pull request #1978 from Djuffin/debug-alloc-fix

Selective VirtualProtect in debugger heap for Realloc

Merge pull request #1980 from pgavlin/StructABITests

Add some missing files.

Add some missing files.

The lack of these files causes the struct ABI tests to fail to build.

Merge pull request #1905 from naamunds/XplatDoc

Add libcurl and libssl packages to instructions

Merge pull request #1841 from caslan/supportforBSTR

Add support for BSTRs on Unix

Merge pull request #1972 from pgavlin/StructABITests

Add some targeted by-value struct passing tests.

Selective VirtualProtect in debugger heap for Realloc

Merge pull request #1963 from Djuffin/debug-alloc-fix

Restrict allocation of executable memory by DebuggerHeap

Merge pull request #1971 from AndyAyersMS/SimplfiyTest

Reduce number of threads used by the LocallocLarge test

Merge pull request #1976 from mmitche/unzip-params

Fix unzip arg order

Fix unzip arg order

Add some targeted by-value struct passing tests.

These tests target the SysV AMD64 ABI in particular, which relies on
field types to decide how to pass by-value structs.

Reduce number of threads used by the LocallocLarge test

This test used to create ~3000 threads. Reduce that to ~300 while we sort out what is behind #1966.

Merge pull request #1968 from mmitche/reenable-coreclr-linux

Reenable coreclr tests by default on Linux

Update trigger phrases to be regex

Quieter zip and ignoring warnings while unzipping

Fix casing of AssemblyName to match the name of the csproj.

Disable a couple more tests temporarily prior to enabling CI runs.

Exit with failure code if any tests fail.

Merge pull request #1967 from mmitche/use-unzip

Switch to using zip/unzip