Merge pull request #5382 from davmason/filtering

Add filtering logic to XplatEventLogger

Merge pull request #5432 from RussKeldorph/fix5428

Remove unnecessary dependencies from JIT tests causing R2R failures

Fix failing tests on Ubuntu R2R configuration (#5435)

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

Merge changes from TFS

Merge pull request #5436 from joperezr/DisableNativeResourceCompileForManagedCode

Disabling NativeResourceCompile Target for Managed libraries

Fix readytorun tests asking for user input (#5459)

'readytorun/mainv1' and 'readytorun/mainv2' tests on Linux wait for human
response asking permission to remove write-protected file:

    rm: remove write-protected regular file 'test.dll'?

'rm' command shall be used with '-f' flag to make it non-interactive.

Fix CFI annotation of AMD64 assembler helpers on Unix (#5446)

This change fixes annotations of AMD64 assembler helpers on Unix. The helpers that
don't use RBP as a stack frame were missing setting the RSP as a CFA register and
unwinding through them was failing.
This was hit in R2R scenarios, with the DelayLoad_Helper_Obj not being unwindable.

Avoid calling prestub through wrong MethodDesc (#5439)

While fixing up a call from a Ready-to-Run method, we don't always
have the right implementation MethodDesc for the target of the call.
Thus it is potentially unsafe to DoPrestub using the MethodDesc we have,
causing a test failue (Loader.classloader_regressions_429802_CMain)
in issue #5366. This is fixed by not calling DoPrestub from
ExternalMethodFixupWorker.

Some types were only supposed to be present when FEATURE_COMINTEROP was set.

This change fixes that.

[tfs-changeset: 1610157]

Linux/ARM: Support -O3 optimization level in release build (#5396)

Currently, we can build coreclr with -O1 flag of clang in release
build mode for Linux/ARM. This instruction is to enable -O3 optimization
level of clang on ARM/Linux by fixing the bug of llvm.

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

Merge pull request #5443 from swaroop-sridhar/CDTver

CoreDisTools Package: Make minor version generic

Merge pull request #5402 from AndyAyersMS/InlineAllowCpBlk

Inliner: enable inlining of methods with cpblk

CoreDisTools Package: Make minor version generic

Change the version requirement of CoreDisTools package
from 1.0.1-prerelease-00001 to 1.0.1-prerelease-*
so that the test infrastructure picks up the latest
available version.

If only the 1.0.1-prerelease-xxxxx number changes,
there is no change in the library API. There may be
a change in the implementation.

The current need for this change is that we need to pick up
a new version of the CoreDisTools package (with an
implementation fix) for X86 testing.

Introduce additional types to the reference assembly for mscorlib and typeforwards for those type to the right assembly.

[tfs-changeset: 1610086]

Merge pull request #5433 from jashook/abs_single

Disable perf_codequality_math_functions

Disabling NativeResourceCompile Target for Managed libraries

Disable perf_codequality_math_functions

abssingle relies on a certain level of floating point precision and fails if the diff is outside of that precision.

Remove unnecessary dependencies from JIT tests causing R2R failures

b05623 did basically nothing so I deleted it.

I removed a presumably useless VisualBasic attribute and other unnecessary
assembly references from the other two tests.

Fixes #5428

Merge pull request #5411 from pgavlin/RVAStaticsUnix

Disable RVA statics tests outside Windows.

Merge pull request #5417 from kyulee1/fixdetail

Fix Details for ILAsm/ILDasm/ClrJit Binaries

Fixing GUID_NULL. (#5415)

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.

Fix Details for ILAsm/ILDasm/ClrJit Binaries

Fixes https://github.com/dotnet/coreclr/issues/5408

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

Disable RVA statics tests outside Windows.

These tests are not supported in general and are already disabled
for Windows. Disable them for *nix as well.

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

Add filtering logic to XplatEventLogger

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

Inliner: enable inlining of methods with cpblk

Allow jit to inline methods with the cpblk IL opcode. Add a test case
where such an inline happens.

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