Fix for run-only pr1 tests.

Merge pull request #1876 from stephentoub/local_realloc

Implement LocalReAlloc in PAL

Merge pull request #1878 from janvorli/fix-ex-try

Add missing exception holder to EX_TRY

Add missing exception holder to EX_TRY

When the exception handling was changed recently, the exception holder
was added only to PAL_TRY / PAL_EXCEPT macros. But it needs to be
in the EX_TRY as well, otherwise first pass of exception handling
would incorrectly consider some exceptions unhandled.

Implement CoTaskMemRealloc in PAL

Also implement CoTaskMemRealloc in PAL, as it's exposed via the public Marshal.CoTaskMemRealloc.

Implement LocalReAlloc in PAL

Marshal.ReAllocHGlobal is failing because it's expecting to find a LocalReAlloc implementation in libcoreclr's PAL.  This commit adds one.

Merge pull request #1868 from JohnChen0/crossgen-linux

Fix a CrossGen assert on Linux

Merge pull request #1862 from LLITCHEV/Issue1831

Fixed the bug described in GH issue 1831.

Fix a CrossGen assert on Linux

Resource string loading during CrossGen doesn't work on Linux.
Disable it for now. In many cases the loaded string isn't even used.

Merge pull request #1229 from benpye/fix-complus-case-docs

Fix casing of COMPlus_ in documentation

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

Merge changes from TFS

Fix for a build break. The gtNewOperNode is not a member of Lowering, but of Compiler. An extra comp-> dereferencing is needed. Talked to Matt Ellis and committing directly to avoid nightly broken build.

[tfs-changeset: 1541493]

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

Merge changes from TFS

Merge pull request #1851 from ellismg/icu-remove-c-plus-plus

Remove OSX Homebrew ICU dependency

Fix build break.

Wrong local name used in lowerarm64.cpp

[tfs-changeset: 1541475]

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

Merge changes from TFS

Fixed the bug described in GH issue 1831.

When generating unboxing function, if the return value is nullables
returned in 2 registers, creating the temp var to pass to the
helper function is not enough. Also need to make sure the operand type
stack is properly maintained.

Merge pull request #1857 from ramarag/testhygiene

Ensure each test has its own unique directory

Test Cleanup:

1. Ensures each test has its own unique directory
2. Use Project Reference to refer native CMakeLists.txt
3. Avoids copying the facades to the test  build

Merge pull request #1818 from gkhanna79/Fix1774

Fix Unix CoreConsole to correctly determine the path to the entrypoint executable

Cleanup CMakeLists.txt

There were a few problems that needed to be addressed:

 - Our detection logic around testing if ICU supported a feature was
   still checking for C++ stuff instead of the coresponding C
   code (which we ended up using).

 - There was some cleanup we could do now that the OSX and other Unix
   builds were split apart

Fix spelling issues

Address a messaging nit.

Fix issue 1774

Merge pull request #1858 from caslan/addcriticalsectionexports

Add EnterCriticalSection / LeaveCriticalSection exports to libcoreclr

Use correct close function for UNumberFormat

Add EnterCriticalSection / LeaveCriticalSection exports to libcoreclr

Merge pull request #1834 from Priya91/fixnull

Add null checks to StackString usages.

Merge pull request #1854 from janvorli/fix-comment

Fix obsolete comment

Add null checks to StackString usages.

Merge pull request #1832 from bryanar/enable_crossgen_tests

- Changes required to enable crossgen on the tests.
- Added priority level parameter to buildtest.cmd
- Added crossgen parameter to buildtest.cmd
- Updated testing documentation to reflect prioritylevel testing, and crossgen testing.

Fix obsolete comment

Use std::vector instead of calloc

This matches what we do in other places in calendarData.cpp, the RAII
pattern will make it easier to not leak memory.

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

[adityam] Fix break in DllImport code for Desktop CLR.

[tfs-changeset: 1540922]

Link against libicucore on OSX

OSX ships with a copy of ICU (their globalization APIs are built on top
of it).  Since we only use stable c based APIs, we can link against it
using the methods described in using a System ICU in the ICU User's
Guide (basically we disable function renaming, don't use C++ and only
use stable APIs).

The ICU headers are not part of the SDK, so we continue to need ICU
installed via Homebrew as a build time dependency.

Fixes dotnet/corefx#3849

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

Changes required to enable crossgen on the tests.

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.

Hygine cleanups in holders.h

Use "= delete" syntax to make it clear the IcuHolder copy constructor
and assignment opperators are removed.

Remove superfluous "public" modifier on the struct closers used by the
IcuHolders.

Cleanup include directives

Remove use of icu::Locale C++ type

Remove all the uses of the icu::Locale type in favor of just using a
char* which is the raw locale id (which is exactly what all the ICU C
apis use for a locale).

The meat of this change si in locale.cpp to actually handle doing the
conversion from UChar* to char*.  The rest of the places are dealing
with the fallout (GetLocale now has a different signiture and the
.getName() dance is no longer needed as we have a raw locale name all
the time now).

Move off Locale instance methods

To prepare for removing icu::Locale in favor of just using the id
directly, remove all the uses of Locale methods except for .getName().
We now use GetLocale to create a Locale but then turn it into a char*
for all the helper methods.

After this change, we can update GetLocale to do locale parsing into a
char buffer and remove all the locale.getName() calls with just `locale'.

Remove C++ Locale use in EnumSymbols

Get Eras using ICU C API instead of C++

Getting the regular eras is straight forward, we can do the thing we do
for other locale data and just ask ICU using a specific
UDateFormatSymbolType.  For abbreviated eras, there's no C API, but we
can try to just read the data from ICU resources and fall back to the
standard width eras if that doesn't work.

Remove use of ICU C++ DateFormatSymbols

Remove useage of ICU C++ DecimalFormatSymbols

Remove ICU C++ LocaleDisplayNames

Remove use of ICU C++ NumberFormat class

This change removes NumberFormat in favor of UNumberFormat. There is a
bit of work that needs to happen in order to keep the normalization code
we use to convert an ICU pattern so to something we can match against
working.

Instead of UnicodeStrings, the input to the normalization function is
now a UChar* and we build up a std::string during normalization.  This
allows us to also skip a conversion from UChar* back to char* so we can
find the correct pattern in our collection of patterns to examine.

Remove use of ICU C++ DateFormatSymbols

Convert DateFormat to UDateFormat

Remove uses of the C++ DateFormat and SimpleDateFormat classes in favor
of UDateFormat.

As part of this change, it was easier to move some of the code that
converts an ICU format string to a .NET Style format string from native
code up to managed code. This code used UnicodeString and we'll need to
move away from that as well as we remove all the C++ usage.

Convert DateTimePatternGenerator usage to C

Part of the effort to remove our usage of C++ ICU APIs. The major issue
here was that the C++ API used char*'s for some things whereas the C API
used UChar*'s so we needed to define our own copies of some constants.

We also need to manage a buffer ourselves, instead of being able to use
the underlying buffer of a retured UnicodeString.

Remove use of ICU C++ Calendar class

We would like to be able to link against versions of ICU installed as a
"operating system level library" which means we can't take a dependency
on any C++ APIs.

This change moves away from icu::Calendar in favor of UCalendar. I also
introduce a small helper template to manage the lifetime of ICU
resources.

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.