Merge pull request #9454 from mikedn/cgtun-to-cne

Morph (x GT_GT.unsigned 0) into (x GT_NE 0)

Merge pull request #9743 from weshaggard/UpdateUsingYourBuildDocs

Update UsingYourBuildDocs to use .NET Core 2.0 SDK

Update UsingYourBuild docs to use .NET Core 2.0 SDK

Merge pull request #9741 from brianrob/issue-9733

Disable Test Broken By CoreFX PR # 15767

Merge pull request #9655 from sdmaclea/PR-ARM64-JITGC-WIP3

[Arm64] Correct CORINFO_HELP_ASSIGN_BYREF killed mask

Disable broken tests.

Merge pull request #9727 from BruceForstall/DontBuildJitLib

Speed up JIT builds

Simplified MinOpts GC ref tracking. (#9231)

Simplified MinOpts GC ref tracking - when a method is compiled with MinOpts for targets with the 64-bit GC info encoder we mark all GC slots as untracked and we omit encoding call sites with no live tracked GC refs explicitly in the GC tables; this can be controlled using the new COMPlus_JitMinOptsTrackGCrefs environment flag.

Android: workaround absence of getcontext/setcontext (#9725)

* Android: workaround absence of getcontext/setcontext
use RtlCaptureContext and RtlRestoreContext

Reduce code in SafeHandleAddRef (#9724)

If DangerousAddRef returns rather than throws, its argument will be true, so there's no need to branch on it.

Faster List Add (#9539)

[Arm64] Fix genCodeForCpObj()

CORINFO_HELP_ASSIGN_BYREF is the functional equivalent to

   *dst = *src;
   ++dst;
   ++src;

Note: Description ignores barriers.

Since src and dst are incremented the registers are marked dead.

In genCodeForCpObj(), the incremented values are will be reused
and therefore must be treated as live.  Add code to explicitly
force them live for this use case.

Merge pull request #9363 from tannergooding/math-intrinsics

Updating jit/valuenum to properly handle the single-precision versions of the math intrinsics.

Speed up JIT builds

We don't need to build the static lib and crossgen static lib
versions of the JIT unless FEATURE_MERGE_JIT_AND_ENGINE is set.
And, since we currently don't set it, this means doing two
fewer JIT builds.

Merge pull request #9654 from sdmaclea/PR-ARM64-JITGC-WIP2

[WIP] [Arm64] Fix GC liveness for CodeGen::genPutArgStk()

Merge pull request #9653 from sdmaclea/PR-ARM64-JITGC-WIP1

[WIP] [Arm64] Fix GC hole in CodeGen::genCodeForCpObj()

Improve stack overflow reporting (#9650)

* Improve stack overflow reporting

This change modifies the SIGSEGV handling to use an alternate stack so that
we can safely detect and report stack overflow even in case when we are
really out of stack. Before, we were able to detect stack overflow and report
it only when JIT inserted stack probes (for functions with frames larger than
4kB) and so there was still space on the stack to run the sigsegv handler.
It brings in some additional complexity, since we need to switch to the original
stack of the thread once we figure out the sigsegv is not a stack overflow and
if we return from the hardware exception handler, we need to switch back to the
alternate stack before returning from the sigsegv handler.
Also, the alternate stack is created per thread and so we need to correctly destroy
it when a thread terminates and also install it on foreign threads that enter PAL.
This also requires creating fake stack frames to enable the libunwind to walk
the stack from the exception handler to the sigsegv location.

* Fix stack unwinding in CallDescrWorkerInternal

While testing the change to enable stack overflow handling, I've noticed that
the PROLOG_SAVE_REG_PAIR and PROLOG_SAVE_REG_PAIR_INDEXED macros are missing
.cfi_def_cfa_register fp. That resulted in inability to unwind through the
CallDescrWorkerInternal, since this function dynamically allocates stack slots
and so the default sp based frame doesn't work.

CMake configuration for cross building for Android (#9174)

* Configuration files for cross building for Android arm64

* Simplify toolchain script

* Fix trymake values for Android

Merge pull request #9712 from AndyAyersMS/FixSomeTestProjFiles

Fix jit test projects to specify debug type and optimization flags

Add test coverage for ldtoken with members (#9713)

The test coverage for `ldtoken field X` and `ldtoken method X` in the
repo is very small - this test tries to improve things a little.

In particular, it tests that we get the right reflection objects, and
that we can do this from shared and unshared code.

Merge pull request #9714 from BruceForstall/FixRegMasks

Fix non-Windows amd64 register mask initialization

Merge pull request #9702 from adiaaida/fixX86PerfTesting

Fix x86 perf testing

[Arm64] Fix GC hole in CodeGen::genCodeForCpObj()

Update tempreg liveness

[Arm64] Fix GC liveness for CodeGen::genPutArgStk()

Updating ValueNumStore to no longer upcast float to double when performing evaluations.

Separate Comparer and EqualityComparer creation logic into a non-generic class (#9640)

Fix projects that specified DebugType incorrectly

Make conditional DebugType properties uncondtional. Fix debug type
and optimize settings for _d and _do suffixed tests. Remove conditional
DebugType propery if there is also an unconditonal property.

Merge pull request #9704 from pgavlin/corefx15913

Correctly pass 12-byte GT_SIMD nodes on x86.

Merge pull request #9660 from dotnet-bot/master-UpdateDependencies

Update CoreClr, CoreFx to beta-25021-03, beta-25022-02, respectively (master)

Update CoreClr, CoreFx to beta-25021-03, beta-25022-02, respectively

Clean up ClrUnwindEx (#9482)

* Clean up ClrUnwindEx

Dropping some unnecessary conditional statements in jit\valuenum.cpp

Updating jit/valuenum to properly handle the single-precision versions of the math intrinsics.

Fix non-Windows amd64 register mask initialization

The arrays were being initialized with register numbers, not
register masks. The arrays are only used on non-Windows amd64
for homing circular incoming argument conflicts. Perhaps we
never saw this happen?

Fix jit test projects to specify debug type and optimization flags

This partially addresses the issues raised in #4124 and #4724 by explicitly
and unconditionally setting DebugType and Optimize flags for various test
project files.

The projects that were fixed are those that did not specify any debug
flag settings at all.  The fix is to set DebugType to PdbOnly and to set
Optimization to true, unless the test has an _r or _rel suffix, in which
case Optimization is set false.

This impacts 1497 projects, skipping over 311 projects flagged as needing
fixes. The projects that remain are the CodeGenBringUp set (for which we plan
to clone projects to test the various flag combinations), projects that
specify flags but do so in ways where the effective flag settings may vary
from DBG to RET, projects that specify the flags redundantly, or projects
where the flags are not consistent with the "test case name suffix" rules.

The edits were done by a tool using linq-to-xml and so some incidental
reformatting has happened.

Fix x86 perf testing

We did not use a different architecture in run-xunit-perf.cmd, so all the
x86 runs were uploading to the same architecture. This change fixes it so
x86jit32 runs upload to their own architecture. Also fixes the delayed
expansion issue for testenv.

Use CORINFO_CALLCONV to check for unmanaged calls (#9697)

`GTF_CALL_UNMANAGED` is a member of the wrong enum and it's something
that would be set on `call->gtFlags` except that at this point it isn't.
Check against the actual unmanaged calling conventions.

[x86/Linux] Fix framepointer while unwinding (#9678)

Use pCallerContext when getting frame pointer

Add a regression test.

Merge pull request #9703 from wtgodbe/CrossgenCode

Have Crossgen Failures exit in a dedicated exit routine

Correctly pass 12-byte GT_SIMD nodes on x86.

During lowering, all TYP_SIMD12 GT_SIMD nodes are retyped to TYP_SIMD16.
This is correct except in the case that the GT_SIMD node is going to be
passed by-value on x86, in which case this retyping causes us to push 16
bytes rather than 12. This change recognizes a GT_SIMD node that has
been retyped by checking the value of gtSIMDSize during LowerArg and
uses TYP_SIMD12 for the putarg instead of TYP_SIMD16 (aside from
differences in detecting retyped nodes, this is the same transformation
we already do when passing TYP_SIMD12 lclVars that are being used as
TYP_SIMD16 on x86).

Fixes https://github.com/dotnet/corefx/issues/15913.

Have Crossgen Failures exit in a dedicated exit routine

Merge pull request #9695 from jashook/up_timeout_for_arm64_gcstress

Up the timeout for gcstress arm64 jobs.

Up the timeout for gcstress arm64 jobs.

Fix compilation on macOS with case sensitive FS (#9693)

Improve string.Equals OrdinalIgnoreCase performance for matching chars (#9213)

* Improve string.Equals OrdinalIgnoreCase performance for matching chars

* Added code comment

Add --xunitOutputPath option to runtest.sh (#9646)

Clean up FixupDispatcherContext (#9685)

Use ADJUST_PC_UNWOUND_TO_CALL instead of _TARGET_XXX_ (#9686)

Merge pull request #9676 from parjong/fix/issue_9576

Fix bogus assert in flowgraph and update related ilproj

Rewrite assert using bbInTryRegions

Update related .ilproj

Fix formatting error

Comment out bogus assert in flowgraph

Merge pull request #9672 from jashook/arm64_regression

Add new regression.

Merge pull request #9671 from benaadams/revert-encoding

Revert "Fast-path for ASCII & UTF8 Encoding ASCII data (#9187)"

Merge pull request #9670 from stephentoub/cs_random

Delay Random allocation in ConcurrentStack

Add new regression.

Revert "Fast-path for ASCII & UTF8 Encoding ASCII data (#9187)"

This reverts commit 77e676be9fab34471c4ae9f16a7a51a06bd470ee.

Merge pull request #9661 from AndyAyersMS/FixTailCallStress1

JIT: tolerate GT_COMMA in tail call position

Remove a few explicit static cctors (#9669)

Merge pull request #9665 from sdmaclea/PR-ARM64-PT_KNONVOLATILE

[ARM64] Use PT_KNONVOLATILE... for cross

Delay Random allocation in ConcurrentStack

Fast-path for ASCII & UTF8 Encoding ASCII data (#9187)

[Arm64/Unix] Fix cmake race condition (#9666)

On a clean build SET(REQUIRE_LLDBPLUGIN false) was being ignored because the cmake
cache variable had not been created

Change order of cmake file to be more natural and to resolve

Merge pull request #9667 from sivarv/backout

Back-out promotion of structs containing fields of struct of single field

Backout promotion of structs containing fields of struct of single field.

[ARM64] Use PT_KNONVOLATILE... for cross

In order to enable cross compilation support, ARM64
needs to use PT_KNONVOLATILE_CONTEXT_POINTERS

Merge pull request #9657 from AndyAyersMS/FixFinallyChainMergeAssert

JIT: relax assert in fgMergeFinallyChains

Merge pull request #9636 from sivarv/shouldPromote

Refactor some of fgPromoteStruct() checks into lvaCanPromoteStructVar() and lvaShouldPromoteStructVar()

JIT: tolerate GT_COMMA in tail call position

Now that we can tail call from inlined call sites, we may see the call
embedded in a GT_COMMA, if the call's return value is ignored. This triggered
an assert the tail call transformation.

Closes #9641.

Merge pull request #9658 from chcosta/fix_coreclr_packaging

Exclude ni files from linux arm32 builds

Synchronous toolchain.cmake files with CoreFX (#9409)

Signed-off-by: Jiyoung Yun <jy910.yun@samsung.com>

Merge pull request #8843 from mikedn/decompose-cast-mul

Improve int->long cast decomposition

Merge pull request #9615 from JosephTremoulet/UpdateCompareVN

Update array length compare value numbers on CSE

Merge pull request #9649 from dotnet-bot/master-UpdateDependencies

Update CoreFx to beta-25017-02 (master)

Merge pull request #9228 from fujiy/fixcorefx4805

Add/Expose String.EndsWith(char) and String.StartsWith(char)

Merge pull request #9656 from RussKeldorph/corefx_x86

Fix corefx x86 testing and small netci.groovy fixes

Remove ubuntu 16.10 arm build from packaging

[ARM64/Unix]  (#9500)

* [Arm64/Unix] Update arm64 *.S files to match *.asm

* [Arm64/Unix] Fix CONTEXTToNativeContext()

* [Arm64/Unix] ThrowExceptionFromContextInternal

* [Arm64/Unix] Preserve x8 argument register

* [ARM64/Unix] Add CFI directives

Add native unwind info

* [Arm64/Unix] Fix RtlRestoreContext

* [Arm64/Unix] Restore FP from CurrentContextPointers

* [Arm64/Unix] fix pointer math

* [Arm64/Unix] Fix CallDescrWorkerInternal personality

* [Arm64/Unix] More Fp fixups

* [Arm64/Unix] CallEHFunclet machine state

Restore non-volatile machine state in CallEHFunclet

* [Arm64/Unix] CallDescrWorkerInternal

Use empty stack slot to save argument

* [Arm64/Unix] RtlVirtualUnwind update pointers

* [Arm64] LazyMachState fixes

* [Arm64/Unix] disable USE_REDIRECT_FOR_GCSTRESS

When FEATURE_PAL is enableds USE_REDIRECT_FOR_GCSTRESS
is not supported

* [Arm64] ClearRegDisplayArgumentAndScratchRegisters()

* [Arm64] Remove unnecesary copy in TransitionFrame

* [Arm64/Unix] Fix comment per review

* [Arm64/Unix] move constants per review

* [Arm64/Unix] Use ldp per review

Also fix indentation

* [Arm64/Unix] Fix indentation per review

* [Arm64/Unix] Remove m_Unwound per review comments

* [Arm64/Unix] Use PREPARE_EXTERNAL_VAR to access globals

* [Arm64/Unix] Fix more whitespace per earlier review comments

Remove contraction from property name

Exclude ni files from linux arm32 builds

Fix corefx x86 testing and small netci.groovy fixes

run-corefx-tests.py fixes
- Switch to using the same corefx build switches as the corefx CI.

netci.groovy fixes
- Add COMPlus_Ready2Run=0 to existing ZapDisable jobs to get the coverage
  we intended to have in these jobs.  Previously, R2R images were slipping
  through.

- Add architecture to the trigger strings fore corefx jobs so it's
  possible to request x86 corefx runs.

- Add the -u option to python command line in netci.groovy so Python
  output is synchronized with subcommand output in the CI logs.

- Fix whitespace in name formatting of corefx_baseline jobs

- Remove trailing (sorry, my editor did this and now it is too hard to
  undo.  Add ?w=1 to Github diff URL to filter out ws diffs.)

Merge pull request #9580 from sdmaclea/PR-ARM64-CROSSGENFAILS

[Arm64/Unix] Add crossgen fails

JIT: relax assert in fgMergeFinallyChains

In some rare and likely IL-only cases the jit may create unreachable
callfinallys. This trips up finally chain merging since it spots a
merge opportunity but no merge actually happens.

Relax the assertion checks so that `canMerge && !didMerge` is now ok.

Closes #9651.

Update CoreFx to beta-25017-02

[x86/Linux] Remove unnecessary x86-specific part (#9627)

This inlined assembly is no longer necessary for x86.

Clean up nested WIN64EXCEPTIONS (#9644)

Clean up dummy implementation in regdisp.h (#9643)

[x86/Linux] Fix GetCallerSp (#9384)

* [x86/Linux] Fix GetCallerSp

* Do NOT pop stack argument for TransitionFrame

* Add ResumeEsp to CONTEXT

Merge pull request #9645 from DrewScoggins/EmptyVariableFix

Fix x64 perf run break

Fix x64 perf run break

After the checkin to add legacy JIT testing support for x86 we broke x64
runs.  THe issue is that we do not pass anything in to the TEST_ENV
variable on x64 and when a variable is empty and you attempt to use the
if exists syntax you fail.  I have added a nonsense file name that we
set this to before we go through the args loop. I was not able to find a
better solution and want to get the lab unblocked, but any batch masters
feel free to solve this the right way.

Merge pull request #9642 from dotnet-bot/master-UpdateDependencies

Update CoreFx to beta-25017-01 (master)

Merge pull request #9639 from parjong/fix/wrong_comment

Fix wrong comment

Update CoreFx to beta-25017-01

Merge pull request #8778 from dnickless/master

Performance improvement: Make ConcurrentDictionary with Enum values also profit from atomic writes.

Merge pull request #9637 from wtgodbe/noMoreSuse

Remove all references to OpenSuse13.2

Merge pull request #9628 from dotnet-bot/master-UpdateDependencies

Update CoreFx to beta-25016-03 (master)

Fix wrong comment

Merge pull request #9610 from sdmaclea/PR-ARM64-CALLSITE

[Arm64/Unix] Correct ControlPC when unwinding