Merge pull request #4765 from adityamandaleeka/array_alloc_bounds_check

Add fwrapv compile option

Merge pull request #4780 from mkborg/fix_windows_linuxmscorlib_arm_build

Fix Windows 'linuxmscorlib arm' build

Remove O_RDWR from sem_open and better error codes on failure (#4772)

Remove O_RDWR from sem_open and better error codes on failure

Merge pull request #4763 from AndyAyersMS/InlineIntoControlExamples

Add some tests for optimization of booleans

Merge pull request #4504 from krytarowski/netbsd-support-72

Add initial NetBSD documentation

Merge pull request #4755 from sivarv/mcall

Multi-reg call node support for x64 Unix

Merge pull request #4723 from kyulee1/tailcall

ARM64: Enable Tail Call with Vararg

Merge pull request #4727 from adiaaida/overflowLongArithmetic

Set overflow flag for add/sub hi on x86 TYP_LONG

Multi-reg call node support.

Merge pull request #4757 from swgillespie/gcstress-disable-tests

Disable two tests under GCStress for #4748 and #4708

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

Merge changes from TFS

ARM64: Enable Tail Call with Vararg

Fixes https://github.com/dotnet/coreclr/issues/4475
I've run into `IMPL_LIMITATION("varags + CEE_JMP doesn't work yet")` in
importer.cpp. This change enables ARM64 tail call path same as other
targets.
 1. Similar to amd64 `genFnEpilog`, I made the similar code under
 `!FEATURE_FASTTAILCALL`. Since `EC_FUNC_TOKEN_INDIR` is not defined for
 arm64, I've made NYI for such case.
 2. Added two pseudo branch instructions 'b_tail' and 'br_tail' which form
 jmp instruction encodings but follow call instruction semantics since
 they are used for tail-call.
 3. `GenJmpMethod` is enabled. Code is slightly changed to reflect correct
 float argument handlings and multi-reg support.

Fix Windows 'linuxmscorlib arm' build

Merge pull request #4759 from rahku/crossgen

Fix checking of correct errorlevel for crossgen

Merge pull request #4771 from myungjoo/fix/4746

ildasm: incorrect signess usage with char

Unix/Build: use -fsigned-char

Some architectures including ARM assumes that plain char is
unsigned unlike x64.

Because CoreCLR has been assuming char is signed, we need to
explictly tell the toolchain the signedness of char type.

Fix #4746

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

Merge pull request #4478 from pgavlin/JitPackaging

Create JIT packages during the build.

Merge pull request #4768 from adityamandaleeka/fix_ci_corefxbins

Add quotes around coreFxBinDir example in runtest.sh

Update per review feedback.

Merge pull request #4740 from yizhang82/interop_x86_test

Fix interop x86 tests by fixing mismatching calling convention, and enable [NativeCallable] for every architecture

Add quotes around coreFxBinDir example.

Merge pull request #4753 from AndyAyersMS/InlineForSize

Inliner: initial version of SizePolicy

Add 'fwrapv' compile option.

Add some tests for optimization of booleans

Test case to stress inlining, expression opts, and control flow
simplification for booleans.

Test case has 100 methods named Idxx. All 100 should generate identical
code. On x64 windows we expect to get the 4 byte sequence
```
       0FB6C1               movzx    rax, cl
       C3                   ret
```

Only 22 of the variants get this codegen; there are at least 12 other
sequences ranging in size from 9 to 32 bytes of code.

Likely touches on the same issues raised in #914.

Add initial NetBSD documentation

Thanks for @coypoop for improving the English wording.

Merge pull request #4705 from RussKeldorph/vector3interop

Disable Vector3Interop until failures are addressed

Revert "Fix array allocation bounds checking."

This reverts commit 16c9dfbae5e5ce18ad0519b31df515820fed2579.

ARM: fix stack frame management (#4641)

ARM: fix stack frame management

Fix #4638
This commit revises the implementation of CallEHFunclet to establish
stack frame appropriately.
This commit allows libunwind to work correctly for the frame created by
CallEHFunclet.

Merge pull request #4693 from adityamandaleeka/delegateinvoke_platnotsup

Make Delegate BeginInvoke/EndInvoke throw PlatformNotSupportedException

Add test for Begin/EndInvoke throwing PlatformNotSupportedException.

Fix checking of correct errorlevel for crossgen

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

Merge changes from TFS

Merge pull request #4721 from swgillespie/gcsimulator

Add a "GCSimulator" CI run

Disable two tests under GCStress for #4748 and #4708

Create JIT packages during the build.

Set overflow flag for add/sub hi on x86 TYP_LONG

When we create the hi operation for add and sub on TYP_LONG, we don't
carry the overflow flag to the hi operation. This change sets the overflow
flag on hiResult in lower if it was set on loResult, and adds GT_ADD_HI
and GT_SUB_HI to the operations that can have overflow. We also need to
pass the unsigned flag to the high part in the instance that we are
dealing with an add or subtract with overflow. Fixes #4596.

Remove SEH interactions between the JIT and the EE.

This change replaces the final uses of SEH between the JIT and the EE with a new method
on ICorJitInfo, `ICorJitInfo::runWithErrorTrap`. This method runs a given function under an
error trap that prevents recoverable errors from propagating out of the call. This allows
the JIT to make calls to JIT/EE interface functions that may throw exceptions without
needing to be aware of the EH ABI, exception types, etc. The method returns true if the
given function completed successfully and false otherwise. This change deprecates
all other SEH-related functionality on the JIT/EE interface (e.g. `ICJI::FilterException` and
`ICJI::HandleException`).

This change does not completely eliminate SEH over the JIT/EE interface: there are still a
handful of places where the JIT expects to be able to catch exceptions thrown by
SuperPMI. These uses of SEH will be removed later on.

Fixes #3058 and #4130.

[tfs-changeset: 1601288]

GCSimulator test run

Inliner: initial version of SizePolicy

Add a new inline policy that tries to inline as much as possible
without increasing method size.

Fix one test that had an expected inline that doesn't happen under
the SizePolicy, by marking a method with AggressiveInline.

Merge pull request #4728 from pgavlin/FixTRTContracts

Fix contracts in `tryResolveToken`.

Fixing issue with perf targets restoring test runtime (#4686)

Fix calling convention mismatch in 7 interop tests and enable [NativeCallable] in all architectures.

Merge pull request #4732 from BruceForstall/SignedIntToLongCastNYI

Convert signed int=>long cast SBCG to NYI

Convert signed int=>long cast SBCG to NYI

This causes many tests that fail with silent bad codegen to pass
with NYI fallback. Many tests are removed from the issues.targets
failing tests exclusion list, but added to the
ryujit_x86_no_fallback_issues.targets exclusion list.

Fix contracts in `tryResolveToken` et. al.

`isValidToken`'s contract was too conservative: it was needlessly marked
`THROWS`, `GC_TRIGGERS`, and `MODE_PREEMPTIVE`. Its contract and those
of its callers have been tightened to `NOTHROW`, `GC_NOTRIGGER`, and
`MODE_ANY`.

Merge pull request #4726 from adityamandaleeka/array_alloc_bounds_check

Fix array allocation bounds checking on Unix

Generalize Buffer.BlockCopy optimization for either src or dst (#4736)

* Generalize Buffer.BlockCopy optimization for either src or dst

Fix for buildstatus/Issues/Issues.aspx?iid=797159

[tfs-changeset: 1601059]

Fix handling of FP args on stack in the ArgIterator (#4729)

This change fixes a bug in the ArgIterator that resulted in assigning
floating point arguments that didn't fit to the xmm registers to general
purpose registers in the ArgIterator if there were some general purpose
registers available.

Merge pull request #4694 from ramarag/thisptrret

Arm64:Implemeting ThisPtrRetBufPrecode:

Merge pull request #4616 from LLITCHEV/MultiCoreJit-1

Add ability to AutoStartProfiling for CoreCLR.

Merge pull request #4731 from kyulee1/onlypush

ARM64: Trigger Failure Email for Push Only

ARM64: Trigger Failure Email for Push Only

Arm64:Implemeting ThisPtrRetBufPrecode:
Precode to shuffle this and retbuf for closed delegates over static methods with return buffer

Fix array allocation bounds checking.

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

Merge changes from TFS

Merge pull request #4684 from gkhanna79/CrossgenJIT

Load JIT from custom location for Crossgen

Merge pull request #4718 from kyulee1/nativevararg

ARM64: NativeVarArg

Merge pull request #4637 from adityamandaleeka/synch_cleanup

Remove SYNCHMGR_PIPE_BASED_THREAD_BLOCKING and clean up synchmanager

Add ability to AutoStartProfiling for CoreCLR.

Add ability to do MultiCoreJit profiling in the CoreCLR. This is used for
testing of the MultiCoreJit functionality.

Merge pull request #4709 from myungjoo/fix/4682

Match Inconsistent CMake/MSBuild Compiler Definitions

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

Merge changes from TFS

Merge pull request #4692 from BruceForstall/FuncInfo

Add support for easily collecting JitFuncInfoLogFile for the tests

Merge pull request #4543 from krytarowski/netbsd-support-73

NetBSD limits name of a semaphore to 15 characters (inc. null)

ARM64: NativeVarArg

In PInovke stub call with vararg that has float value,
we did not properly set up argumnent by passing float registers whith do
not conform ARM64 ABI.
The fix is to set the right calling convention in VM to use
integer registers to pass such float values.

Merge pull request #4621 from kyulee1/floatarg

ARM64: Enable Varargs with float argument.

ARM64: Enable Varargs with float argument.

Fixes https://github.com/dotnet/coreclr/issues/2989
ARM64 ABI does not allow float register arguments for vararg calls.
All such float arguments should be copied to integer registers.
In Lower, I explicitly add a GT_COPY for such case and support the code-gen
to expand it properly using `fmov`.
I also skip genFnPrologCalleeRegArgs() for vararg since we've already
saved all integer registers in the prolog.

Merge pull request #4699 from kyulee1/testlst

ARM64: Tests.lst Update

Merge pull request #4689 from AndyAyersMS/NoInlineRange

Implement JitNoInlineRange

Enable crossgen to load JIT from a custom location
Continue to link the JIT in for Windows Arm64 builds until 4717 is fixed.

Add support for easily collecting JitFuncInfoLogFile for the tests

Simply uncomment one of the three methods for doing the collection in
the ryujit_x86_testenv.cmd file.

Merge pull request #4710 from yizhang82/array_warn_fix

Fix warnings in ByValArray

Merge pull request #4707 from RussKeldorph/gcincomp

Mark some GC tests incompatible with GCStress

Merge pull request #4704 from RussKeldorph/lotsofinlines

Mark LotsOfInlines tests as optimization-sensitive

Fix warnings

Fix #4496 (#4503)

* Fix #4496

This commit revises legecy codegen to inserts a bkpt instruction if the current BB
and next BB is not in the same exception handling region.

This code is backported from ARM64 codegen.

* Update ControlPcIsUnwound in UnwindManagedExceptionPassX

* Enable only for ARM32

Enable the patch only for ARM32 in order to fix build break in other
architecture.

Disable WIN32_REGISTRY for UNIX platforms

Unlike managed build scripts, FEATURE_WIN32_REGISTRY has been
enabled for UNIX-like systems in CMake native build scripts.

Fix #4682 part 3.

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

Merge pull request #4556 from JohnChen0/r2r

Ready-to-Run support for generic methods without generic dictionary

Mark some GC tests incompatible with GCStress

These tests rely on counting numbers of collections, which is unreliable
(or at best different than what the test expects) under GCStress.

Support generic methods without generic dictionary

Disable Vector3Interop until failures are addressed

This test is turning several lab runs red.  Disabling for now.

Mark LotsOfInlines tests as optimization-sensitive

The LotsOfInlines test times out under COMPlus_JitStress=1.
(http://dotnet-ci.cloudapp.net/job/dotnet_coreclr/job/master/job/x64_checked_windows_nt_jitstress1/48/testReport/).
This change disables it from running under JitStress modes that might
affect optimizations required for it to run in reasonable time.

Merge pull request #4653 from mikem8361/shimrace

Fix issue #4298 "SIGSEGV_libcoreclr.so!Debugger::GetArgCount"

ARM64: Test.lst Update

Fixes https://github.com/dotnet/coreclr/issues/3970
Fixes https://github.com/dotnet/coreclr/issues/4411
Since https://github.com/dotnet/coreclr/pull/4655 is merged, these tests
seem to be passed in the lab.

Merge pull request #4687 from kyulee1/zap

ARM64: Enabling Crossgen End-to-End Mscorlib

Merge pull request #4698 from adityamandaleeka/fix_ci_corefxbins

Fix CI: Update directory check for case with multiple CoreFX directories

Use Unix version of WORKSPACE.

Update directory check for case with multiple CoreFX directories.

Merge pull request #4691 from adiaaida/longOperators

Separate NYIs for arithmetic operators on longs

Implement JitNoInlineRange

Rework and cleanup ConfigMethodRange. Give it a configurable size. Do
some error detection when parsing the range string. Update comments
and add notes at existing usage sites about the behavior when the
range string is not specified.

Use ConfigMethodRange to implement a JitNoInlineRange option that
suppresses inlining in a specified set of methods. Set this up so
inlning still happens if the range string is empty. Choose capacity so
it can hold the entire range string's set of ranges. Add a new
observation for the cases where enabling this JIT option disables
inlines.

Make compMethodHash available for INLINE_DATA builds. Report this hash
in the Xml inline dumps, so it can provide hash values to feed into
JitNoInlineRange strings.

Merge pull request #4695 from adityamandaleeka/fix_ci_corefxbins

Modify CI to pass in multiple CoreFX bin directories

Add cascading CoreFX directories to Unix CI run

Allow multiple CoreFX bin directories to be specified in runtest.sh.

Fix desktop build break introduced by CS#1600134

[tfs-changeset: 1600494]

Make Delegate BeginInvoke/EndInvoke throw PlatformNotSupportedException.

Separate NYIs for arithmetic operators on longs

To make it more obvious which long operators are causing the NYI to hit,
split out the different operators with their own NYI.

Merge pull request #4676 from BruceForstall/x86Testing

Change RyuJIT/x86 testing

Change RyuJIT/x86 testing

Change the RyuJIT/x86 test runs so we don't set COMPLUS_AltJitAssertOnNYI=1
by default. This means when the JIT hits an NYI, it falls back to
the legacy x86 back-end instead of raising an assert. The issues.targets
test exclusion file is changed to only exclude those tests that fail,
due to assert or silent bad codegen, in this mode, rather than the
(currently) much, much larger number of exclusions if COMPLUS_AltJitAssertOnNYI=1
is set.

The version of issues.targets is renamed as ryujit_x86_no_fallback_issues.targets.

This is a issues.targets file that contains exclusions for running
with RyuJIT/x86 and COMPLUS_AltJitAssertOnNYI=1, so NYI turn into
asserts.

ARM64: Enabling Crossgen End-to-End Mscorlib

Fixes https://github.com/dotnet/coreclr/issues/4350
Fixes https://github.com/dotnet/coreclr/issues/4615

This is a bit large change across VM/Zap/JIT to properly support crossgen
scenario.
1. Fix incorrect `ldr` encoding with size.
2. Enforce JIT data following JIT code per method by allocating them together.
   This guarantees correct PC-relative encoding for such constant data access
   without fix-up.
3. For the general fix-up data acceess, use `adrp/add` instruction pairs with fix-ups.
   Two more relocations types are implemented in all sides.
4. Interface dispatch stub is now implemented which is needed for
interface call for crossgen.

I've verified hello world runs with mscorlib.ni.dll.

Treat NetBSD like FreeBSD and OSX in PAL_VirtualUnwind() (#4551)

This fixes hangs in the runtime observed while running CoreFX managed tests.

Thanks @janvorli and @myungjoo

Fix #4380