Merge pull request #5356 from swaroop-sridhar/Trap

GCStress: Fix a race-condition

Merge pull request #5394 from erozenfeld/TailCallBug5164

Fix for methods that tail call via helpers

Fixing an error in PAL_pow that caused it to return an incorrect value for a couple scenarios. (#5399)

* Fixing an error in PAL_pow that caused it to return an incorrect value for a couple scenarios.

* Adding additional PAL_pow tests.

Merge pull request #5409 from gkhanna79/FixDetails

Fix details for CoreCLR binary

Merge pull request #5400 from weshaggard/FixPackageLicenseFiles

Include license files into the packages

Fix details for CoreCLR binary

Enable FEATURE_HIJACK for ARM (#5404)

Currently, FEATURE_HIJACK is disabled for ARM, and thus CoreCLR got
stuck if GC is invoked from JITed code.

This commit attempts to port FEATURE_HIJACK from ARM (windows) in order
to fix #5403.

Validate new object construction in Ready to Run (#5398)

Existing Ready to Run implementation doesn't fully validate requests
to create object instances, and allows some invalidate requests (e.g.,
to create an instance of an abstract class). This causes failure of
test case Loader.classloader_generics_Instantiation_Negative_abstract01
in issue #5366.

Include license files into the packages

Merge pull request #5386 from briansull/arm32-hfa-fix

Fix for HFA args on ARM32

Add limit to number of generics methods to compile by CrossGen (#5383)

Recursive generic definitions can easily make CrossGen take very
long time to complete. This is the cause of a failure in issue #5366.
This is fixed by limiting the number of methods to compile by CrossGen.

Added additional fix for ARM32 in lvHfaSlots

Improvements to System.Math InternalCall code. (#4847)

* Adding some basic System.Math performance tests.

* Renaming 'floatnative' to 'floatdouble'.

* Removing outdated workarounds in the floatdouble interop code.

* Renaming 'finite.cpp' to 'math.cpp'

* Updating the double-precision math tests.

* Updating PAL_EPSILON to be more precise.

JIT_TailCall helper has an implicit assumption that all tail call arguments live

on the caller's frame. If an argument lives on the caller caller's frame, it may get
overwritten if that frame is reused for the tail call. Therefore, we should always copy
struct parameters if they are passed as arguments to a tail call.

The simple il regression test has a scenario similar to that of the F# repro in #5164.

Closes #5164.

Import `new` multi-dimmensional array using the non-vararg helper (#5313)

* Factor out import of `new` multi-dimmensional array
* Implement import of `new` multi-dimmensional array using the non-vararg helper
* Delete RETURN_TYPE_IS_COMPOSITE inlining observation because of it is never hit.
* Add R2R id for non-vararg helper
* Reformat affected code to follow coding conventions

Merge pull request #5375 from pgavlin/CSECleanup

Remove some lexical-CSE-related code.

GCStress: Fix a race-condition

This change ensures that calls to CORINFO_HELP_STOP_FOR_GC() themselves
are not converted to GC-Stress traps -- thus preventing the race between
the GC thread, and a managed thread under GCStress.

Identification of calls to CORINFO_HELP_STOP_FOR_GC():
Since this is a GCStress only requirement, its not worth special identification in the GcInfo
Since CORINFO_HELP_STOP_FOR_GC() calls are realized as indirect calls by the JIT, we cannot identify
them by address at the time of SprinkleBreakpoints().
So, we actually let the SprinkleBreakpoints() replace the call to CORINFO_HELP_STOP_FOR_GC()
with a trap, and revert it back to the original instruction the first time we hit the trap in
OnGcCoverageInterrupt().

Upside: No change to GCInfo or JIT is necessary
Downside: Need to decode a few bytes on every GCStress interrupt.

Fixes #4794

Merge pull request #5365 from kouvel/NamedMutexTests

Named mutexes: fix handle leak, add stress tests

Merge pull request #5384 from wtgodbe/publishing

Publish pkg/symbolpkg folders within Release/Debug folders

Publish pkg/symbolpkg folders within Release/Debug folders

Proposed fix for HFA args on ARM32

Remove some lexical-CSE-related code.

Lexical CSE is no more; remove some related code:
- Remove the `lexicalCSE` parameter and related checks from `optConfigDisableCSE`.
- Remove the `VALNUM_CSE_ENABLED` symbol: this symbol is unconditionally
  set to 1 and doesn't control much other than the behavior of
  `optConfigDisableCSE`

Speed up CreateComparer in Comparer/EqualityComparer (#5315)

Fix for issue 5343: Assert Failure: !"Cannot take the address of an uninstantiated generic method." (#5347)

The problem was that the delegates code base was not correctly instantiating interface GVMs on target types
(after finding the target method on the target type using slot numbers), and ended up calling uninstantiated
generic method definitions on delegate invokes.

Merge pull request #5372 from kyulee1/eh11

ARM64: eh11_large exclusion

Merge pull request #5376 from wtgodbe/publishFix

Publish packages into folders named for their configuration

Publish packages into folders named for their configuration

Merge pull request #5306 from jamesqo/jitdump-docs

Add new document on how to setup/view JIT dumps

ARM64: eh11_large exclusion

This test often fails in release run.

Merge pull request #4918 from stephentoub/task_parent_size

Shrink Task size by 10%

Add new document on how to setup/view JIT dumps

Remove duplicate avoid in comments from a few files (#5363)

Remove double avoid in comments

Merge pull request #4909 from lemmaa/unit-test-enhancement

Add extra options to 'runtest.sh' for effective test monitoring.

Add more Ready-to-Run generics tests (#5190)

Add named mutex stress tests

- The existing PAL tests are run in parallel, each in a separate thread, for a specified duration

Fix handle leak when creating or opening named mutex fails

- After the pal object is registered, it gets a second reference from the handle. Upon an error after that, the handle needs to be revoked in addition to releasing the initial reference.

ARM/Linux: Fix Exception Handler PAL Test Fail (#5360)

The return address should be kept intact, not recovered
just yet. It is going to be recovered by libgcc's
rescore_core_regs much later, few instructions before
heading back to "catch".

Fixes #5358

Comment typo fix: removed double word (#5359)

The overflow check comment for AllocateStringFastMP_InlineGetThread contained an extra "avoid".

Fix JIT PInvoke Check Failure on Linux/ARM (#5322)

Disable SP check on ARM

Merge pull request #5350 from pgavlin/gh5292and5286

Mark two tests as always optimized and exclude two others from x86/JI…

Add options to 'runtest.sh'

- '--show-time'
   display execution sequence and approximate running time for each
   test cases.
- '--no-lf-conversion'
   skip LF conversion for each test case script.

Note: `date` on Mac OS X doesn't support `%N` format specifiler.
In this case, we don't need precise time, so just ignore the time
less then a second.

Merge pull request #5353 from ramarag/windowsr2r

Refactoring Arguments passed to runtest.cmd

Refactoring Arguments passed to runtest.cmd

Merge pull request #5340 from pgavlin/ComputeFPlvls

Refactor fgComputeFPlvls

Mark two tests as always optimized and exclude two others from x86/JIT32.

The former tests require optimization in order to pass and the latter require further
deliberation re: the appropriate fix (see #5286 for more info).

Fixes #5292.

Merge pull request #5339 from kyulee1/fixmemset

ARM64: Fix JIT_MemSet

Merge pull request #5335 from swgillespie/gcstress-disable

Disable a large number of GC tests from running under GCStress

Merge pull request #5294 from ellismg/enable-other-distros-for-building

Enable other distros for building

Merge pull request #5334 from pgavlin/LongMathRegparam

Remove LONG_MATH_REGPARAM from the JIT.

Fix for issue 5241: [RTM] Interop\ICastable\Castable\Castable.cmd CONTRACT VIOLATION by ReadyToRunInfo::TryLookupTypeTokenFromName (#5296)

Merge pull request #5301 from RussKeldorph/try2

Second attempt to properly modify environment for stress test modes

Support building on all supported distros

With the updated buildtools, we should now be able to build on all the supported distros:

 - Centos 7
 - Fedora 23
 - openSUSE 13.2
 - RHEL 7.2
 - Ubuntu 14.04
 - Ubuntu 16.06

I also cleaned up how we compute the RID for the tools to restore.

Move gtComputeFPlvls next to gtSetEvalOrder.

Rename fg{FPstLvlRedo,ComputeFPlvls} to gt*.

This variable and function are more appropriately grouped with the GenTree-related
funcaionality, as they are peers of gtSetEvalOrder.

ARM64: Fix JIT_MemSet

Init value should be treated as an unsigned char.

Disable a large number of GC tests from running under GCStress

Remove LONG_MATH_REGPARAM from the JIT.

This symbol was never defined as true, and as such the various bits of code predicated
on it were dead. This change removes the symbol and its associated code.

Merge pull request #5189 from AlexGhiondea/buildPackage

Include the targeting pack in the build process.

Port String.IndexOf(char) and LastIndexOf(char) from CoreRT (#5327)

Update Windows instructions (#5312)

* Small improvements to Windows instructions

* copy missing System.Private.CoreLib.dll

* syntax highlighting

* Use ref assemblies from CoreFX instead of NuGet

* Used AnyOS directories for CoreFX assemblies

Also use built facade assemblies.

AnyOS directories are safer, since they don't rely on tests to be built.

(Building main project usually creates just AnyOS, building test project creates the Windows_NT version.)

Update tryrun.cmake for arm-softfp (#5323)

Updates tryrun.cmake for ARM cross build (#5317)

Enable 'System.Private.CoreLib.dll' build on cross build mode (#5320)

Currently, build.sh uses 'BuildOS' and 'BuildArch' to determine whether
it is possible to build 'System.Private.CorLib.dll'.

This commit revises build.sh to use 'HostOS' and 'HostArch' instead of
'BuildOS' and 'BuildArch' when it checkes whether it is possible to
build 'System.Private.CoreLib.dll', which allows to build
'System.Private.CoreLib.dll' in cross build mode.

Fix: if errorlevel 0 is almost always true (#5308)

Explanation: if errorlevel n actually tests if the errorlevel is
greater than or equal to n. See: http://ss64.com/nt/if.html

Merge pull request #5240 from wateret/mscorlib-arm-softfp

Support CoreLib build for arm-softfp

Merge pull request #5087 from myungjoo/fix/4859/janvorli_suggestion

Fix/4859 Unit Test 143837 Fail

Avoid unnecessary ContingentProperties allocations

We don't need to allocate it for child tasks if there isn't actually a parent.

Make m_contingentProperties non-volatile

Most accesses, including those to get the recently moved m_parent field, do not need to be volatile.  This commit makes the field non-volatile and then adds usage of Volatile.Read only where needed.

Move Task's m_parent to ContingentProperties

Shrinks the size of Task by 10% in the common case (when ContingentProperties doesn't need to be expanded).

Fix ucol_setMaxVariable detection for Gentoo Linux (#5309)

The issue was the symbol is exported by the ICU lib. Including headers
was not enough. The linker requires the libraries to succeed.

With this fix, CoreCLR successfully builds on Gentoo Linux 100%.
Tested with LXC gentoo container on Ubuntu machine.

Steps to configure and build:
https://gist.github.com/jasonwilliams200OK/1a2e2c0e904ffa95faf6333fcd88d9b8

Fix #5160

Don't use libstdc++ wrappers for stdlib.h (#5304)

This prevents #undef of min/max macros. Refs #5006.

ARM: Remove workaround no more required

With recent concrete unwinding/exception handling routines
of coreclr-arm-linux, we no longer need this workaround.

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>

ARM/THUMB PC Least Bit Set for Range Check

Fix #4859

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>

Merge pull request #5093 from wateret/fix-returnstruct-arm-softfp

Fix Assert Error "Unexpected size when returning struct by value" on …

Merge pull request #5307 from ellismg/fix-rhel-build

Fix RHEL 7 Build

Fix RHEL 7 Build

There were two issues:

 - $__BuildArch is not defined in init-tools.sh, so we would always
   download the ubuntu version of the CLI, since our check for
   `rhel.7.2-x64` would never fire.

 - There was a small RHEL specific bug in BuildTools itself that
   manifested when we actually were able to download the CLI and try to
   restore the runtime.

Merge pull request #5290 from joperezr/AddingCommitHashToManagedLibraries

Adding commit hash to all built binaries.

Ensure Ready-to-Run module is activated in generics dictionary lookup (#5288)

Under certain conditions, a Ready-to-Run module is not yet fully
activated when its contents are used. This can cause asserts in
debug build. It can also cause module constructor to be run too late.

Merge pull request #5299 from CarolEidt/FixArm64

Fix a build break due to PR #5101

Merge pull request #5295 from adityamandaleeka/noexcept_nothrow_new

Add noexcept specifier to some non-throwing new operators

Remove the dnvm-based installation instructions from the docs (#5271)

* Replace DNX docs with a link to the official .NET website

* Group Get/Learn About .NET Core into one section

Merge pull request #5236 from leemgs/upstream-buildtest.cmd-help-update

Documentation: Adding flag descriptions of unit test scripts

Second attempt to properly modify environment for stress test modes

My amateurish bash skills are showing.  I forgot that you have to
source a script to modify the environment, though, to be fair, this
line was already here when I got here. :)

Fix the named semaphore leak on OSX (and Linux) (#5269)

* Change the dbgshim launch handshake back.

The debugger side now creates the name semaphores like before and the transport pipe
existence determines that coreclr is ready.

Changed when the transport pipes are created: synchronously on the main thread.

Correctly set and check the HAVE_PROCFS_* defines.

* Code review feedback.

Fix Assert Error "Unexpected size when returning struct by value" on arm-softfp

For only arm-softfp there is an assert error when returning struct as value.

Fixes #4924

Signed-off-by: Hanjoung Lee hanjoung.lee@samsung.com

Documentation: Adding flag descriptions of unit test scripts

Let's append the meaning of the 'priority' and 'gcstresslevel' flag.
The additional description of the 'priority' flag  is derived from
./coreclr/tests/dir.props. And, that of the 'gcstresslevel' flag is
derived from ./coreclr/src/gc/sample/gcenv.h.

ver2:
Moved 'gcstresslevel' flag from ./tests/buildtest.cmd to ./tests/runtest.*.

ver1:
Added 'priority' and 'gcstresslevel' flag in ./tests/buildtest.cmd

Signed-off-by: Geunsik Lim <geunsik.lim@samsung.com>

Fix a build break due to PR #5101

The declaration, definition and uses of eeGetMethodFullName
didn't match.

Merge pull request #5263 from briansull/avoid-x8

Avoid using register x8 as it will be used as the RetBuf argument in the future

Merge pull request #5277 from pgavlin/gentree

Remove a redundant recursive call to `gtSetEvalOrder`.

Merge pull request #5257 from Priya91/failingtest

Disable failing test to enable corefx runs.

Merge pull request #5261 from ramarag/gcstress

Refactoring the way GC Stress logic is used while running tests

Avoid using register x8 as it will be used as the RetBuf argument in the future

Placed usage of REG_SPILL_CHOICE under LEGACY_BACKEND and removed them for RyuJit only targets
Moved REG_R8 for ARM64 to later in the REG_VAR_ORDER
Change the definition of REG_PINVOKE_ registers to avoid using REG_R8
Change the definition of REG_TMP_0 and REG_TMP_1 to avoid using REG_R8
 the REG_TMP defines are only used in one place: CodeGen::genEmitGSCookieCheck

Merge pull request #5101 from CarolEidt/FixIssue2554

Use JitFuncInfoLogFile to check if SIMD intrinsics are recognized.

Excluding Weak-GCReferences tests from GC stress runs (#5289)

Adding commit hash to all built binaries.

Introduce a central packages.builds file that controls building the nuget packages.

This removes the need to have individual packages added to the build scripts to generate them.

Merge pull request #5266 from vancem/EventSoruceComplexTypeFix

EventSource Events that use complex types do not get logged (silently…

Merge pull request #5279 from RussKeldorph/unix_stress

Fix environment settings for Unix JIT stress scenarios

Fix environment settings for Unix JIT stress scenarios

The environment settings necessary to run tests in these modes were being
dropped.  They need to be tunneled through properly in a couple of places.

Fixes #5278

Fix GCStress 0xC on Unix (#5276)

This change fixes GCStress 0xC on Unix that was not working due to the way that
we check that a specific IP is in managed code. To make that check safe,
a precondition checking that the thread is in cooperative GC mode was added
some time ago. But that causes a problem with the GCStress 0xC and PInvoke
stubs. The PInvoke stub switches to preemptive mode at some point and when
the code beyond that point is instrumented and the corresponding hardware
exception occurs, the exception address is not evaluated as being in managed
code and so the exception terminates the application.
This fix excludes the safety check of GC mode when we are running with GC stress
mode 8 or 4 (or both -> 0xc).