Merge pull request #4690 from sejongoh/use_rax_for_ret_buf

Return RetBuf address via RAX for Windows 64

Merge pull request #4795 from sivarv/simdret

[x64 Unix] Avoid SIMD types returned in two return registers from requiring to go through memory.

Merge pull request #4791 from kyulee1/fasttailcall

ARM64: Enable Fast Tail Call

ARM64: Enable Fast Tail Call

Fixes https://github.com/dotnet/coreclr/issues/4420
Like x64, enable fast tail call but tail call opt is still off.
This means explicit tail call is converted to epilog + jmp IP0.
Tail call recursive/tail call via helper is still not enabled yet.

Merge pull request #4814 from BruceForstall/FixGcRefAssert

Fix GC ref frame offset range assert

Merge pull request #4811 from BruceForstall/FixFunctionTrace

Fix COMPlus_JitFunctionTrace=1 to work better with NYI

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

Merge changes from TFS

Merge pull request #4806 from AndyAyersMS/FixInlineDump

Fix inline dump format issue

Merge pull request #4797 from JohnChen0/master

Remove an assert in AssemblySpec::EmitToken

Merge pull request #4810 from BruceForstall/Fix4666

Mark EDX as trashed by the no-gc helper calls

Merge pull request #4819 from BruceForstall/FixRyujitX86CheckedTestRun

Disable tests which fail in a RyuJIT/x86 Checked build test run

Merge pull request #4821 from ellismg/fix-unix-builds

Fix conditional in Unix build definition

Fix conditional in Unix build definition

Disable tests which fail in a RyuJIT/x86 Checked build test run

Fix COMPlus_JitFunctionTrace=1 to work better with NYI

Fix GC ref frame offset range assert

In emitSetFrameRangeGCRs(), on x86, there is the following assert:
  assert(offsHi <  0);
This is saying that the highest offset of a stack frame gc reference
is negative w.r.t. the frame pointer. This is not quite true: the
highest offset is exclusive of the range (the lowest offset is inclusive).
I found a test case where the 'this' pointer, a gc ref, was located
immediately below the frame pointer, in range [ebp-4,ebp), so offsHi==0.

The reason we don't hit this more is because this assert is only in
the JitDump code (under #ifdef DEBUG and 'if (verbose)'). (This
also seems wrong.)

I changed the assert to (offsHi <= 0).

This change ports the support files from NetFxDev1. It doesn't change behavior yet. Adding a feature for the app context switch for path compat as we don't intend to use it in core.

[tfs-changeset: 1602284]

Mark EDX as trashed by the no-gc helper calls

Fixes #4666

Fix inline dump format issue

Lost some implict casts to float when I updated SIZE_SCALE to an enum
constant, so now make the casts explicit.

Just impacts dump output.

Also, add a newline to LegacyPolicy dump to keep messages distinct.

Return RetBuf address via RAX for Windows 64

The AMD64 native calling convention requires return buffer address to be
returned by callee in RAX. This changeset makes JIT follow this rule.
Especially, in case that a managed method returns a struct value to its
native caller, the caller reads a pointer to the return value from RAX.

Asm diffs show expected differences such as 'mov rax, reg' at the end of
functions and some different register assignments.

Merge pull request #4800 from jashook/x86_ci_fix

Fix x86 netci changes to build correctly.

Avoid SIMD type return values from requiring to go through
memory for returning.

Fixup netci changes to build correctly.

Also more verbosely show which jit and arch the job is.

Merge pull request #4782 from BruceForstall/NyiLongExTree

RyuJIT/x86: convert SBCG to NYI for multi-level expression trees

Merge pull request #4792 from dagood/remove-core_root-workaround

Remove CORE_ROOT workaround

Remove an assert in AssemblySpec::EmitToken

AssemblySpec::EmitToken has a PRECONDITION that prohibits emitting AssemblyRef
tokens to WinMD files, except inside CrossGen. However, there are scenarios,
such as XML serialization, that need to emit such tokens. Thus this PRECONDITION
is removed.

Merge pull request #4790 from jashook/x86_netci_two

Add ryujit x86 pr trigger.

Merge pull request #4767 from ramarag/VSD_ResolveWorker_indirection

[Arm64] Initialize indirection cell for delegates

[Arm64] Initialize indirection cell for delegates

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

RyuJIT/x86: convert SBCG to NYI for multi-level expression trees

In general, we decompose longs by walking the tree bottom-up. So, e.g.,
we'll decompose 'add' as follows:

  + (gt_long(lo1, hi1), gt_long(lo2, hi2))
=>
  gt_long (+ (lo1, lo2), +Hi (hi1, hi2))

But what if lo1 is '+' and hi1 is '+Hi'? So,

  + (gt_long( + (lo3, lo4), +Hi (hi3, hi4) ), gt_long(lo2, hi2))
=>
  gt_long (+ ( + (lo3, lo4), lo2), +Hi ( +Hi (hi3, hi4), hi2))

But we can't put any instructions between '+ (lo3, lo4)' and '+Hi (hi3, hi4)'
or we'll lose the carry bit.

We need to model this differently, or introduce temps. Until this is
done, convert the silent bad codegen into NYI fallback.

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

Remove CORE_ROOT workaround. .NET CLI no longer uses the value.

Add ryujit x86 pr trigger.

Adds ryujit x86 testing on pr, and collapses x86 lb to just run checked jobs.

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