Increase timeouts for non-windows configs

Merge pull request #2961 from rahku/updatebuildtools

Update buildtools version for test builds

Merge pull request #2979 from Dmitry-Me/writeNullPointerOnNoInterface

Write null pointer for unsupported interfaces

Merge pull request #2719 from janvorli/enable-unix-readytorun-tests

Enable unix readytorun tests

Enable ready to run tests on Unix

This change enables running ready to run tests on Unix.

Merge pull request #2978 from jkotas/prof-breaks

Fix fallout from enabling profiling

Write null pointer for unsupported interfaces

Merge pull request #2976 from krytarowski/netbsd-support-27

Fix build on NetBSD: Introduce check for pthread_getcpuclockid(3)

Merge pull request #2977 from krytarowski/netbsd-support-28

Prefer PAL_PTRACE() over hardcoded FreeBSD-style ptrace(2) calls

Merge pull request #2972 from krytarowski/netbsd-support-24

Fix build on NetBSD: Don't increment iterator twice in for(;;)

Merge pull request #2971 from krytarowski/netbsd-support-23

Add NetBSD target in src/pal/src/CMakeLists.txt

Fix build on NetBSD: Introduce check for pthread_getcpuclockid(3)

This function is part of the optional part of POSIX.

It's available in:
- FreeBSD,
- AIX,
- QNX,
- Linux.

It's missing in NetBSD.

According to Jan Kotas (Microsoft) this function is used for undocumented
resource monitoring feature (FEATURE_APPDOMAIN_RESOURCE_MONITORING) and
unlikely anybody is using this today.

Solution suggested in #2947 "NetBSD doesn't implement pthread_getcpuclockid(3)"

Fix fallout from enabling profiling

- Compiler optimization assumption invalid: EE expects method to exist: System.StubHelpers.StubHelpers:ProfilerBeginTransitionCallback
- Add stubs for ARM

Prefer PAL_PTRACE() over hardcoded FreeBSD-style ptrace(2) calls

Merge pull request #2953 from janvorli/fix-unix-readytorun

Fix ReadyToRun on Unix

Fix build on NetBSD: Don't increment iterator twice in for(;;)

If I'm reading it correctly, we will never catch the ".cormeta" value.

Add NetBSD target in src/pal/src/CMakeLists.txt

Sort platforms alphabetically in src/pal/src/CMakeLists.txt

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

Merge changes from TFS

Merge pull request #2958 from jkotas/arm-fixes

Fix build breaks in Linux ARM build

Merge pull request #2930 from sivarv/retfix

Fix to issue #2700

Merge pull request #2960 from briansull/issue_2925

Remove assert NYI: "Varargs method"

Update buildtools version for test builds

Fix build breaks in Linux ARM build

- Delete redundant WAITORTIMERCALLBACK declarations. These redundant declarations were needed for very old versions of Windows SDK that are no longer supported.
- Remove unnecessary __cdecl modifiers
- Add default case to switch statement

Fix Unix build breaks

[tfs-changeset: 1571015]

Remove assert NYI: "Varargs method"

Of the 27 test cases that hit this assert 22 of them are passing after this change
The remaining test failures are:
  arglist.exe_2994
  arglist.exe_2971
  arglist.exe_2983
  arglist.exe_3006
  b16423.exe_4877

Fix ReadyToRun on Unix

While making the ready to run tests run on Unix, I have discovered that the feature
doesn't really work on Unix. First, I have found that the test was generating the
native images into the NI subfolder, but the corerun on Unix doesn't use this
notation and expects these images to live side by side with the IL executables.
So, the test to verify that an image cannot be loaded multiple times was failing,
since the main exe loaded the IL test.dll instead of the native one and so explicit
load of the native test.dll wasn't considered a secondary load of the same image.
After fixing the test, I have found that attempt to load a ready to run image is
failing with an assert.
There were these issues:
 - the loaded native image was not marked as relocated
 - the EPILOG_WITH_TRANSITION_BLOCK_RETURN macro was missing popping argument registers
 - One of the asserts and one getter didn't take into account the fact that the ready to
   run native image is not considered to have native header.
 - The assembler helpers DelayLoad_Helper and DelayLoad_MethodCall were passing a hex
   number as a parameter to the PROLOG_WITH_TRANSITION_BLOCK, but it was using the "h"
   suffix notation. There is a known issue in the clang assembler which causes such
   suffix to be swallowed so the macro was getting just "10" instead of "10h". The fix
   was to use 0x prefix instead.

This change fixes the problem. The ready to run tests pass with it correctly.

Fix OACR errors for ProjectK

[tfs-changeset: 1570939]

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.

Fixes #2700

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