Update buildtools version for test builds

Merge pull request #2951 from Dmitry-Me/dontAccessDeletedObject

Don't access deleted object, return proper error code

Don't access deleted object, return proper error code

Merge pull request #2949 from Dmitry-Me/castToMostDerivedInterface

Fix cast to interface other than the most derived one

Merge pull request #2950 from jkotas/WIN32

Stop defining WIN32 in Unix builds

Stop defining WIN32 in Unix builds

WIN32 was defined for 32-bit Unix builds in number of places which is incorrect. It caused build break in lldb header since they use WIN32 define for Windows-specific code.

Fix cast to interface other than the most derived one

Merge pull request #2948 from krytarowski/netbsd-support-22

Fix build on NetBSD, casting pointer to function to pointer to data

Merge pull request #2933 from stephentoub/enum_parse

Improve performance of Enum.Parse/TryParse

Fix build on NetBSD, casting pointer to function to pointer to data

/tmp/pkgsrc-tmp/wip/coreclr-git/work/coreclr/src/vm/method.cpp:5196:16: error: cannot initialize return object of type 'LPVOID' (aka 'void *') with an rvalue of type 'FARPROC' (aka 'long (*)()')
        return GetProcAddress(hMod, (LPCSTR)(size_t)((UINT16)ordinal));
               ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 error generated.

Improve performance of Enum.Parse/TryParse

The allocation profile of Enum.Parse today is fairly poor.  While the design of the method returning an Enum requires at least one allocation for the boxed result (the generic TryParse could be overhauled to avoid this, but still has it), additional allocations shouldn't be necessary in the common cases.  However, for some reason the current code is boxing a 0 on every call.  It's also using string.Trim() to remove whitespace (once for the overall string and then once for each substring), and using String.Split to parse multiple values, which ends up allocating a string[] and an int[] even if there's only one value.

This commit removes all allocations from Enum.Parse other than the boxing of the Enum result and some allocations on the code path where Enum.Parse is handed a string containing a number, in which case additional allocations are involved in using the Convert.ChangeType call.

With an enum like:
```C#
[Flags]
enum Colors
{
    Red = 0x1,
    Orange = 0x2,
    Yellow = 0x4,
    Green = 0x8,
    Blue = 0x10
}
```
using ```Enum.Parse(typeof(Color), "Red")``` repeatedly now results in 5x fewer gen0 GCs and is ~15% faster.  Using ```Enum.Parse(typeof(Color), "Red, Orange, Yellow, Green, Blue")``` repeatedly now results in 23x fewer gen0 GCs is is similarly ~15% faster.

Merge pull request #2940 from krytarowski/netbsd-support-18

NetBSD requires <stdarg.h> for variable argument lists in output.inl

Merge pull request #2943 from krytarowski/netbsd-support-20

NetBSD doesn't ship with SI_KERNEL

Merge pull request #2946 from krytarowski/netbsd-support-21

NetBSD: Disable check for minimal stack size for undefined PTHREAD_ST…

NetBSD: Disable check for minimal stack size for undefined PTHREAD_STACK_MIN

An alternative would be 'sysconf(_SC_THREAD_STACK_MIN)', but it might be no
good reason to prohibit single page stack size (PAGE_SIZE).

Merge pull request #2520 from sperling/profiler

Enabling profiling on Unix/OS X

NetBSD requires <stdarg.h> for variable argument lists in output.inl

NetBSD doesn't ship with SI_KERNEL

SI_KERNEL is available in Linux and FreeBSD. It's unspecified what does it mean.

Linux:
  #define SI_KERNEL       0x80            /* sent by the kernel from somewhere */

  -- include/uapi/asm-generic/siginfo.h

FreeBSD:
                    SI_KERNEL            signal generated by miscellaneous
                                         parts of the kernel
  -- siginfo(3)

Enabling profiling on Unix/OS X.

Merge pull request #2939 from krytarowski/netbsd-support-17

NetBSD doesn't ship with <machine/npx.h>.

Merge pull request #2938 from krytarowski/netbsd-support-16

NetBSD defines _lwp_self(2) in <lwp.h>

Merge pull request #2935 from jkotas/arm-fixes

Fix build breaks in Linux ARM port

NetBSD doesn't ship with <machine/npx.h>.

This is first step towards stopping to assume that HAVE_BSD_REGS_T implies
FreeBSD-style code. This breaks on NetBSD in several places.

Perhaps HAVE_BSD_REGS_T should be renamed to HAVE_FREEBSD_REGS_T.

Check headers with this fashion:
- FreeBSD requires <sys/types.h> for <machine/npx.h>
- FreeBSD requires <sys/cdefs.h> for <machine/reg.h>
- NetBSD doesn't need extra includes for <machine/reg.h>
- NetBSD requires <sys/param.h> before inclusion of <sys/cdefs.h>

Introduce central location for turning FEATURE_EVENT_TRACE on/off

Turn FEATURE_EVENT_TRACE on for x64 Linux only. aarch64 and arm do not
have working lttng in their ubuntu/debian repos. We can expand to other
platforms in the future as they become supported.

NetBSD defines _lwp_self(2) in <lwp.h>

The _lwp_self(2) function prototype is missing in <sys/lwp.h>.

It's not sufficient to include <sys/lwp.h> or <lwp.h> -- as they are not
including one the other. We need to check for both headers and include them
both.

NAME
     _lwp_self - get light-weight process identification

LIBRARY
     Standard C Library (libc, -lc)

SYNOPSIS
     #include <lwp.h>

     lwpid_t
     _lwp_self(void);

DESCRIPTION
     _lwp_self() returns the LWP ID of the calling LWP.

Cleanup top-level defines

- Delete left-over cruft from internal build
- Move Windows-specific defines under WIN32

Merge pull request #2936 from dotnet/revert-2931-glob-revert

Revert "Revert "Add un-prefixed signatures as temporary workaround""

Revert "Revert "Add un-prefixed signatures as temporary workaround""

Add missing files for ARM legacy JIT

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

Merge changes from TFS

Fix build break in internal build

[tfs-changeset: 1570568]

Merge pull request #2931 from jkotas/glob-revert

Revert "Add un-prefixed signatures as temporary workaround"

Merge pull request #2922 from rahku/updateBuildTools

update version of buildtools

Revert "Add un-prefixed signatures as temporary workaround"

This reverts commit 6b1d2938ec4a5a2c64fd849797ec7800ed3ab575.

Merge pull request #2902 from wtgodbe/disableTests

Disable failing pri 1 tests that are unsupported outside of Windows

Merge pull request #2928 from BruceForstall/FixArm

Fix ARM JIT

Merge pull request #2923 from briansull/issue_2849

Fix for Issue 2849 - Assertion failed 'parent->OperGet() != GT_LDOBJ'

Fix ARM JIT

During a previous code cleanup, an assert condition was incorrectly inverted.

Merge pull request #2924 from wtgodbe/pri1

Switch ubuntu PR tests from Release to Checked, and run Pri 1 tests on each commit

Switch ubuntu PR tests from Release to Checked, and run Pri 1 tests on each commit

Merge pull request #2917 from jkotas/globalization

Unique names for GlobalizationNative exports

fix for path to buildtools.coreclr package

Fix for Issue 2849 - Assertion failed 'parent->OperGet() != GT_LDOBJ'

Change the ifdef to use FEATURE_MULTIREG_STRUCT_ARGS instead of
FEATURE_UNIX_AMD64_STRUCT_PASSING because the ARM64 target
also uses GT_LDOBJ when passing 16-byte sized structs

Merge pull request #2916 from erozenfeld/Checkedx86BuildFix

Fix for x86 checked build error.

update version of buildtools

Add un-prefixed signatures as temporary workaround

Unique names for GlobalizationNative exports

For consistency and to enable eventual sharing of the same code with CoreRT, I have changed the naming convention for System.Globalization.Native exports to match dotnet/corefx#4818.

Merge pull request #2904 from ellismg/add-ubuntu-15.10-to-ci

Add Ubuntu 15.10 runs

Merge pull request #2919 from kyulee1/ver

RyuJit version update

Merge pull request #2918 from wtgodbe/serverGC

This should enable Server GC for ubuntu PR CI builds

This should enable Server GC for ubuntu PR CI builds

Disable failing pri 1 tests that are unsupported outside of Windows

Fix for x86 checked build error.

Remove FORCEINLINE from GetUnwindDataBlob so that the compiler
always emits it in codeman.obj. The code in nidump.cpp
(NativeImageDumper::DumpReadyToRunMethod) calls this method so
we were getting link errors if GetUnwindDataBlob wasn't emitted in codeman.obj.
GetUnwindDataBlob is small enough for a compiler to inline it even without FORCEINLINE.

Merge pull request #2914 from Dmitry-Me/unifyRefcountAccess

Unify reference count access

RyuJit version update

Merge pull request #2888 from JohnChen0/coremsbuild

Build mscorlib with CoreCLR MSBuild on Linux and Mac

Unify reference count access

Merge pull request #2900 from BruceForstall/ClrStlUpdate

Fixes and additions to CLR STL functionality

Merge pull request #2901 from JohnChen0/r2r-av

Modify crossgen to always set DLL bit in PE header

Merge pull request #2913 from krytarowski/netbsd-support-15

NetBSD: Fix portability of isspace(3) usage

Merge pull request #2911 from krytarowski/netbsd-support-13

NetBSD: Stop assuming that __hppa__ is HP-UX

Merge pull request #2906 from krytarowski/netbsd-support-12

Stop setting for NetBSD -D_DEFAULT_SOURCE -D_POSIX_C_SOURCE=200809L

Merge pull request #2912 from krytarowski/netbsd-support-14

NetBSD defines VM_MAXUSER_ADDRESS in <machine/vmparam.h>

Merge pull request #2703 from AndyAyersMS/InlineRefactor

Initial round of jit inliner refactoring

NetBSD: Fix portability of isspace(3) usage

NAME
     isspace - white-space character test

LIBRARY
     Standard C Library (libc, -lc)

SYNOPSIS
     #include <ctype.h>

     int
     isspace(int c);

STANDARDS
     The isspace() function conforms to ANSI X3.159-1989 (``ANSI C89'').

CAVEATS
     The argument to isspace() must be EOF or representable as an unsigned
     char; otherwise, the behavior is undefined.

-- NetBSD man-page

This behavior is POSIX and Standard C:

The c argument is an int, the value of which the application shall ensure is a
character representable as an unsigned char or equal to the value of the macro
EOF. If the argument has any other value, the behavior is undefined.

-- The Open Group Base Specifications Issue 6
   IEEE Std 1003.1, 2004 Edition

C11 standard:
The header declares several functions useful for classifying and mapping
characters In all cases the argument is an int, the value of which shall be
representable as an unsigned char or shall equal the value of the macro EOF. If
the argument has any other value, the behavior is undefined.

-- 7.4 Character handling <ctype.h> paragraph 1

NetBSD defines VM_MAXUSER_ADDRESS in <machine/vmparam.h>

/*
 * USRSTACK is the top (end) of the user stack. Immediately above the
 * user stack resides the user structure, which is UPAGES long and contains
 * the kernel stack.
 *
 * Immediately after the user structure is the page table map, and then
 * kernel address space.
 */

-- /usr/include/machine/vmparam.h

NetBSD: Stop assuming that __hppa__ is HP-UX

HPPA is one of the NetBSD ports.

Initial round of jit inliner refactoring

In the jit, a particular inline decision is the aggregate result of checks
in many different parts of the code. The primary goal of this refactoring
is to establity a single object instance that will accumulate the necessary
information, and to ensure there are a small number of places in the code
where the final outcome is known.

That object is the `JitInlineResult`. I've updated it so it is no longer
copyable or assignable. Instead each root instance is passed down stack
by reference to helper methods that update the instance as the inline decision
making process evolves. Methods on the JitInlineResult can be used to advance
the state of what's known. The allowable state transitions are limited and the
code verifies that (when appropriate) a decision was actually made.

There are 4 such root instances:
* the main check done in the inlining pass.
* the candidate identification pass done in the importer.
* a pre-jit viability scan also done in the importer.
* another pre-jit viability scan done in the importer.

(These latter two identify un-inlinable methods at pre-jit time, so
subsequent jit invocations don't expend any cycles considering them).

I've updated various APIs to try and avoid introducing aliased copies or
projections of the inline result, for instance passing it via argument to
a helper that already has access to it via a member variable, or returning
a bool result from one of the various 'canInline' methods. More work is possible
here -- for example the compiler instance probably doesn't need an inline
result member of its own, since whenever it is inlining it has an inline
instance that has an inline result. I've added a few asserts in places to ensure
that if there is more than one plausible path to the result that they all refer
to the same object.

I've also started streamlining a bit of the goto-heavy case analysis in favor
of early returns, which I find more readable. The reporting obligation
is now held by the JitInlineResult destructor. Future work may go further in this
direction, and make the JitInlineResult responsible for never attribute flagging,
logging and dump messages, and perhaps even responsibility for undoing some of
the speculative state modification done when preparing for inlining (eg cleaning up
the locals in `fgMorphCallInlineHelper`).

At this stage the various candidate and failure reasons are still strings.
I added some new strings for in-progress checks so the dumps now show the
"success reason" for an inline. This is an area that will undergo
further refinement. Success reasons and failures and failure reasons
are still not captured in the inline tree; that too is forthcoming.

This change has minor codegen diffs. In cases where the callee has pinned locals
and returns a bool, we'll now identify the callee as a "never" candidate and so
some calls to it will never get the anticipatory wrapping that inline candidates
get. While this wrapping is undone later when inlining fails, it leaves a type
conversion behind and that conversion impacts the generated code slightly: I see
`test eax, eax` to query the return value rather than `test al, al`.

While I was striving for a no-diff change I think taking this one change is
reasonable: it's preferable to identify never candidates rather than repeatedly try
inlining them and fail each time.

Stop setting for NetBSD -D_DEFAULT_SOURCE -D_POSIX_C_SOURCE=200809L

This makes ucontext.h related discovery functions work.

Add Ubuntu 15.10 runs

Part of dotnet/corefx#5349

Merge pull request #2893 from swaroop-sridhar/redirect

Fix an assert failure in GCStress testing

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

Merge changes from TFS

Merge pull request #2868 from kyulee1/cfi

Support for CFI unwind info

Fixes and additions to CLR STL functionality

vector:
Added some const qualifiers.
Added cbegin(), cend(), and data() members.

algorithm:
Added transform()

string:
Implemented a simple subset of std::string. This was implemented for use by a component that will be checked in later. It is currently unused.

Removed ownership tags.

Added "magic" comments to help VIM editor properly handle these files without filename extensions.

Modify crossgen to always set DLL bit in PE header

The jit may try to inline in cases where there are serious mismatches between the type the caller is passing and the type the callee expects. This can cause the jit to issue malformed queries through the jit interface.

The fix is to take an existing assertion check which detects mismatches the jit is not prepared to handle and make the check active in retail builds. If a bad type mismatch is detected, inlining at that call site is aborted.

[tfs-changeset: 1569969]

Support for CFI unwind info

For Unix targeting CoreRT, this will provide platform specific unwind
info which is a dwarf format.
Unlike window’s UNWIND_INFO, we won’t encode the format within RyuJit
since it is not only complex/error-prone but also needs to be adjusted
depending on platforms.
Instead, CFI (call frame information) pseudo instructions are encoded,
which will be passed to CoreRT/ObjectWriter which will translate
them to directly emit CFI directives to LLVM MC stream to establish
frames and layout eh_frame section.
A jit flag is used to dynamically dispatch either Windows’s unwind
blob or this CFI instruction table. No JIT/EE interface change is needed
since the API already expects an opaque blob.
Initially when I looked at what Clang does, the prologue and the
sequence of CFI emissions are a bit different than Windows.
Since we will emit the same sequence of code with the same runtime that
we define, I assume this is unnecessary – I’ve verified the CFI sequence
here can work correctly with libunwind.
Basically we need only 3 operations – push reg is a combination of 1 and
2 below.
 1. Allocation – increase stack frame. Normally subtract esp in x64.
 2. Store – Copy a (callee save) register to stack (memory)
 3. SaveFP – Set frame pointer register

Since Windows operation is based on the relative value (all offsets,
etc are computed from the current point), I also use the similar form of
CFI instructions.
So, mostly one window’s code corresponds to two CFIs – we might optimize
this by aggregating allocation, but I’d like to keep the current
syntax/semantic same as Windows.
This results in a very simple transformation on par with Windows.

Fix an assert failure in GCStress testing

When GCStress uses redirection, it pushes a
RedirectedThreadFrame (Windows) or
ResumableFrame(Unix) on the stack.

The stack walker, when checking for consistency of frame chain
makes a special case for this redirection when running under
GCStress -- but compares only against `RedirectedThreadFrame'.
This caused the StackWalker to think that certain invoke-s were
not redirected, resulting in the failures in some GCStress tests
on Linux.

This change fixes the problem.
Fixes #2848

Merge pull request #2894 from stephentoub/fix_dtparsing

Fix DateTimeFormat.FormatCustomized optimization bug

Merge pull request #2882 from gkhanna79/FixRace

Fix for CoreCLR issue 2881 - Fix race in Default ALC initialization.

Fix for CoreCLR issue 2881 - Fix race in Default ALC initialization.

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

Incorporate PR feedback

Merge pull request #2880 from janvorli/fix-python-script

Fix missing format specifier in Utilities.py

Merge pull request #2890 from sivarv/master

Fix gc-hole while emitting CORINFO_HELP_ASSIGN_BYREF helper call on Unix

Fix DateTimeFormat.FormatCustomized optimization bug

After reverting https://github.com/dotnet/coreclr/pull/2617, which resulted in backslash escapes being included incorrectly, replaces it with a simpler fix that avoids the same allocations and copies by simply passing in the overarching StringBuilder.  The call sites throw away the StringBuilder if an exception occurs, so we can simply write to it directly rather than first writing to a temporary and then dumping that temporary into the overarching result.

Revert "Remove StringBuilder allocation/usage from DateTimeFormat"

This reverts commit 3a8f9cb3c9412f896fd4be808564fe26afc54fc4.

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

Merge changes from TFS

Merge pull request #2884 from wtgodbe/pathFix

This fixes the change that shortens path names of output Pri 1 tests in CI

Fix gc-hole while emitting CORINFO_HELP_ASSIGN_BYREF helper call.
This is a no-gc helper takes its parameters RSI and RDI and doesn't trash
them.

Emit.cpp!emitNoGCHelper() is used by emitter to compute the registers
that are trashed by a helper call to update gc-info state maintained by
emitter (see emitxarch.cpp!emitins_call()).  emitNoGCHelper() on Windows
is modeled to return RBM_CALLEE_TRASH_NOGC (which is same as RBM_CALLEE_TRAHS)
and it happens to work fine because RSI and RDI are not part of
RBM_CALLEE_TRASH set.  But on Unix Amd64, RSI and RDI are part of
RBM_CALLEE_TRASH and hence gets reported as killed by ASSIGN_BYREF
helper and hence the gc-hole.

Fix: exclude RSI and RDI in set of registers killed by ASSIGN_BYREF
helper.

Merge pull request #2887 from stephentoub/license_headers

Update licensing headers

Merge pull request #2883 from stephentoub/cts_size

Reduce size of CancellationCallbackInfo

Update license headers

Don't run finalizers on shutdown, provide Unloading event

API review: https://github.com/dotnet/corefx/issues/5205

This change implements the proposal in the API review above:
- Don't block threads for shutdown
- Don't run finalizers on shutdown (for both reachable and unreachable objects)
- Provide a public AssemblyLoadContext.Unloading event that can be handled to clean up global resources in an assembly
  - As unloading an AssemblyLoadContext is not yet implemented, the event will for the time being be raised shortly prior to shutdown

[tfs-changeset: 1569451]

This fixes the change that shortens path names of output Pri 1 tests in CI

Reduce size of CancellationCallbackInfo

Each registration with a cancelable CancellationToken results in allocating a CancellationCallbackInfo instance to store the relevant state, e.g. the action to invoke, the associated CTS, etc.  CancellationToken.Register has a little-known and very-little-used feature that enables the callback to be automatically sent to a captured SynchronizationContext, and as such the CancellationCallbackInfo has a field to store the SynchronizationContext, too.  But this field is almost always null.

This change reduces the size of CancellationCallbackInfo from 56 to 48 bytes on 64-bit by moving the TargetSynchronizationContext field to a derived type.  The derived instance with the extra field is only allocated when there is a SynchronizationContext captured.

Fix missing format specifier in Utilities.py

The Utilities.py script was missing the `%s` format specifier in
one of the prints and it was causing a build failure for me.

Merge pull request #2879 from janvorli/fix-exception-in-prestub-using-holder-pass1

Fix exception in PreStubWorker call chain - update

Fix exception in PreStubWorker call chain - update

This change fixes one case that the previous fix was missing. This is a case when
the exception in the PreStubWorker call chain is thrown by native code, for example
during jitting the target managed method. In this case, the DispatchManagedException
is called in the first pass, so we need to do the same change as in the previous
fix for the first pass as well.
Since we now have the first managed frame context available in the
DispatchManagedException for the first pass too, I have modified the
UnwindManagedExceptionPass1 to get it as a parameter so that it doesn't
need to do the unwinding to that frame again.

Merge pull request #2866 from LLITCHEV/x86-assert-fix

Disable initialization of _lvArgReg and _lvOtherArgReg on x86.

Merge pull request #2864 from richlander/rich-header

Update copyright and file header guidance for .NET Core

Update copyright for .NET Core and update file header guidance