Merge pull request #4836 from sivarv/simdPromotion

Mark simd type vars as lvRegStruct accurately.

Fixes multidim array Get Accessor for enum datatype (#4808)

Use Buffer.BlockCopy in System.Reflection.Emit (#4812)

Fix a typo in NetBSD documentation. (#4841)

Buffer.BlockCopy: Avoid double-typechecks for arrays of same type (#4807)

Remove FEATURE_HOSTED_BINDER definition (#4838)

The FEATURE_HOSTED_BINDER is always on so remove it from all the sources.

Mark simd type vars as lvRegStruct accurately.

Merge pull request #4803 from pgavlin/RootBinDir

Add a `bindir` parameter to build.sh.

Merge pull request #4831 from pgavlin/gh2300

Move b12011 into testsUnsupportedOutsideWindows.

Merge pull request #4826 from RussKeldorph/vararg

Move Windows-only vararg/arglist tests to the appropriate exclusion file

Merge pull request #4815 from AndyAyersMS/LegacyPolicyInlineLimit

Inliner: add JitInlineLimit check to LegacyPolicy under debug

Merge pull request #4770 from Maoni0/affi

enable affinity for GC threads and some bug fixes

Fix Windows x86 exception handling issue (#4830)

This change fixes an exception handling issue that happens on x86 on Windows
when exception is raised in System.RuntimeType.MakeGenericType.
The problem was caused by GCPROTECT_HOLDER macro in RuntimeTypeHandle::GetTypeByName
that causes popping of GCFrame (and zeroing its m_next field) that's in the middle of the thread's frames list during
the stack unwinding.
That breaks the list and when UnwindFrames happen later and tries to walk the stack, the StackFrameIterator::NextRaw
asserts when checking the validity of the list.
The fix is to move the keepAlive to the managed caller of the RuntimeTypeHandle::GetTypeByName QCall, which removes
the need for the GCPROTECT_HOLDER.
Since it was the only usage of that holder and of the underlying FrameWithCookieHolder class, I've removed those.
In addition to that, I've modified COMModule::GetType and AssemblyNative::GetType to use the same pattern, since
they could also suffer from the problem the GCPROTECT_HOLDER was attempting to fix.

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.

Move b12011 into testsUnsupportedOutsideWindows.

This test requires COM interop.

Add a `bindir` parameter to build.sh.

This parameter can be used to redirect the usual output path from
`./bin` to a different directory.

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

enable affinity and numa for GC and some bug fixes

double the threshold for balancing onto a remote NUMA node - this was proven to be best with the asp.net benchmark.

enable cpu group

Merge pull request #4806 from AndyAyersMS/FixInlineDump

Fix inline dump format issue

Move Windows-only vararg/arglist tests to the appropriate exclusion file

Our position on vararg/arglist is that it is supported on Windows only, so
I'm repurposing #1440 to simply express that position in the exclusion
files rather than removing the tests.

Fixes #1440

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]

Inliner: add JitInlineLimit check to LegacyPolicy under debug

Adds the ability to limit the number of inlines done by the LegacyPolicy.
Useful in trying to binary search for inlines that cause correctness
issues.

This limit impacts inlining in all methods, so effective isolation may
also require use of JitNoInlineRange to hone in on the root method that
is the source of problems.

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: