Fix setup-runtime-dependencies.cmd

Merge pull request #3697 from AndyAyersMS/RandomInliner

Random inliner

Merge pull request #3743 from dagood/remove-nugetbuild-source

Remove nugetbuild myget feed

Merge pull request #3742 from kyulee1/fixgchole2

ARM64: Revisit for Fix GC hole in indirect call site

Merge pull request #3656 from leemgs/upstream-calling-convention-cdecl

Remove __cdecl__ calling convention for [-Werror,-Wignored-attributes]

Merge pull request #3725 from leemgs/upstream-doc-cross-arm

Documentation: Compiling mscorlib for ARM Linux

Remove __cdecl__ calling convention for [-Werror,-Wignored-attributes]

The cross build for Linux/ARM is broken while doing the compilation
by the [-Werror,-Wignored-attributes] policy of the cross compiler.
Let's remove __cdecl__ calling convention from ./test/src/Interop folder
by handling __cdecl macro handling in xplatform.h for Linux/ARM.

Signed-off-by: Geunsik Lim geunsik.lim@samsung.com
Signed-off-by: Prajwal A N an.prajwal@samsung.com
Signed-off-by: MyungJoo Ham myungjoo.ham@samsung.com

Documentation: Compiling mscorlib for ARM Linux

Let's update the out-of-date contents of mscorlib.

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

Merge pull request #3737 from sivarv/bugFix

Fix to issue #3734 - a block with jump kind BBJ_RETURN could also be empty if the method's return type is TYP_VOID

Merge pull request #3700 from rahku/updateBuildTools

Update buildtools version to 00198

Merge pull request #3739 from gkhanna79/FixRTJson

Fix runtime.json to include all platforms in the redirection package.…

Remove nugetbuild myget feed from package restore.

ARM64: Revisit for Fix GC hole in indirect call site

This fixes https://github.com/dotnet/coreclr/issues/3738.
The fix I made in
https://github.com/dotnet/coreclr/commit/4dfd323dab88b902fc9479efa60cb5d6b7659e94
 used 3rd/4th operand to keep GC info, which actually conflicts with
 address field which is unioned with these operands.
So, I go back to the original fix that I proposed below:

Indirect call (```br``` or ```blr```) target is encoded with a register
which the first operand internally represents.
Unfortunately, call sites use the first two operands to hold GC callee-save registers.
So, this GC register information was overridden by the call target operand
in the indirect(virtual) call sites.
The fix is to split branch instruction categories for these two instructions
while keeping ```ret``` same as before. They internally use the third
operand to encode the target since the first two are used for GC info.
The reason I didn't change ```ret``` is because the return instruction
is created as a small instruction (not using operand 3 and more).

Merge pull request #3741 from wtgodbe/r2r

Fix file-too-long errors caused by adding R2R tests to CI

Fix file-too-long errors caused by adding R2R tests to CI

Fix to issue #3734

Random inliner

This change creates an inline RandomPolicy which assigns a random
benefit to each discretionary inline candidate. Force inlines are
honored as they are with LegacyPolicy. The pool of discretionary
candidates is widened to include the "always inline" cases from the
LegacyPolicy. Discretionary inlines are accepted with a likelihood
that decreases with IL size, in a manner that mirrors the acceptance
probability of the LegacyPolicy.

The random inliner is intended to be used mainly as a stress testing
tool.

The RandomPolicy is enabled in non-retail builds when `JitStress` is
set to a nonzero value. By default the RandomPolicy will be used for
50% of the method being compiled, and the LegacyPolicy (in stress mode)
for the other 50%. Which policy is used for a given method depends on
the stress value. The stress value also forms part of the random seed,
so different random patterns can be invoked by varying the `JitStress`
value.

To enable the RandomPolicy for all methods, set `JitStress` to something
nonzero, then set `JitStressModeName=STRESS_RANDOM_INLINE`.
To do this without also enabling other stress modes, set `JitStressModeNameOnly=1'.

Fix runtime.json to include all platforms in the redirection package. Also update the CLI build version (taken from CoreFX).

Merge pull request #3706 from rahku/fixbug

fix dir of disabled test

Merge pull request #3705 from wtgodbe/r2r

Enable pri0 crossgen build/tests per-commit, and pri1 by trigger phrase

Merge pull request #3729 from gkhanna79/FixNugetPackage

Fix 3724 - Include Runtime.json in redirection package

Merge pull request #3711 from rahku/Helix

Add dependency to apisets correctly

Enable pri0 crossgen build/tests per-commit, and pri1 by trigger phrase

Add dependency to apisets correctly
Update version of apisets

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

Merge changes from TFS

Fix 3724 - Include Runtime.json in redirection package

Merge pull request #3720 from gkhanna79/FixTestTimeOut

Fix timeout determination for tests

Merge pull request #3694 from RussKeldorph/stresstags

Exclude doublearray/* from GCStress runs

Merge pull request #3680 from mikedn/bad-disasm

Fix some JIT disassembly issues

Merge pull request #3698 from kyulee1/Fixovf

ARM64: Fix genJumpToThrowHlpBlk

Merge pull request #3692 from kyulee1/fixuint2float

ARM64: Fix casting unsigned int to double

Fix timeout determination for tests

Merge pull request #3699 from jkotas/tests

Test fixes

Merge pull request #3634 from ramarag/testbuild

Enabling standalone builds for Test Project of any Priority

Merge pull request #3702 from wtgodbe/disableEH

Disable failing Hardware EH test on non-windows

Merge pull request #3654 from ramarag/lttngperf

Using StackString to improve memory consumption in Lttng EventProvider

fix dir of disabled test

Disable failing Hardware EH test on non-windows

Merge pull request #3695 from tarekgh/FixUmAlQuraCalendarStaticDataInitialization

Fix UmAlQuraCalendar static data initilaization

Make sure that the AppContext.GetData method is part of model.xml.

This was missed in the previous check-in.

[tfs-changeset: 1585251]

Update buildtools version to 00198

Add latent Roslyn dependency to project.json

Roslyn loads System.Security.Cryptography.Algorithms via reflection

Delete bad tests

These tests depend on Thread.Abort and reflection emit APIs that are not part of supported .NET Core surface

Merge pull request #3679 from kyulee1/fixgchole

ARM64: Fix GC hole in indirect call site

ARM64: Fix genJumpToThrowHlpBlk

This is additional fix to https://github.com/dotnet/coreclr/issues/3667.
When JIT invokes this API with direct jump (EJ_jmp), JIT also tries to
circumvent the target by reversing the jump kind in debug path.
In this case, actually we should directly invoke the target.

Before (Fail)
```
 b       G_M36945_IG03
 bl      CORINFO_HELP_THROWDIVZERO
G_M36945_IG03:
```

After (Pass)
```
 bl      CORINFO_HELP_THROWDIVZERO
```

Expose AppDomain.GetData via AppContext.

This is useful in certain scenarios where there is a need for information to be passed using the AppDomain.GetData methods.

[tfs-changeset: 1585237]

Merge pull request #3696 from briansull/update-tests-lst

Update tests lst for ARM64

Merge branch 'master' into update-tests-lst

Updated with NEW_PASS tests

ILAsm/ILDAsm packages

This creates packages for ILAsm/ILDAsm respectively.
These tools depend on CoreCLR, so they are tied together with the same version.
Note we use ProjectK build for packaging Windows binaries so these tools are basically desktop one while cross-platform binaries are from Github build as usual.
When CoreCLR packaging and publication are migrated to Github build, so will these tools.

[tfs-changeset: 1585183]

ARM64: Fix GC hole in indirect call site

This fixes dotnet#3663.
Indirect call (```br``` or ```blr```) target is encoded with a register
which the first operand internally represents.
Unfortunately, call sites use the first two operands to hold GC
callee-save registers.
So, this GC register information was overridden by the call target operand
in the indirect(virtual) call sites.
The fix is to use 3rd/4th operands instead of 1st/2nd operands to hold GC info.
Ideally we should use different field name and also ensure constness when
we set up the operand so that it's never written more than once.
https://github.com/dotnet/coreclr/issues/3693 is filed.

Fix UmAlQuraCalendar static data initilaization

The current way of declaring the static data is cauing null reference exception
when the static constructor get called. the reason is the static initialization
method InitDateMapping is using the static array rawData which is not really
get initilaized yet when InitDateMapping execute.

The fix is moving rawData inside InitDateMapping.

https://github.com/dotnet/corefx/issues/6774#issuecomment-196044580

Merge pull request #3687 from kyulee1/fixzeroinit

ARM64: Fix ZeroInit of Locals

Exclude doublearray/* from GCStress runs

Doublearray tests are incompatible with GCStress.

Fixes #2759

ARM64: Fix casting unsigned int to double

When casting unsigned int to double, JIT emitted ```scvtf``` instead of ```uscvtf```.
The issue was JIT used dst type instead of src type for checking the sign.
This fixes a part of https://github.com/dotnet/coreclr/issues/3667.

Merge pull request #3659 from RussKeldorph/stresstags

Add test mode exclusions

Fix some JIT disassembly issues

1. (v)movsd operands were messed up in disassembly:
vmovsd   qword ptr [rsp+28H]ymm0, qword ptr
instead of
vmovsd   qword ptr [rsp+28H], ymm0

2. cvtxx2yy insructions had GPR names prefixed with x/y:
vcvtsi2ss ymm0, yrcx
vcvttss2si yrsi, ymm6

ARM64: Fix ZeroInit of Locals

When zero-initializing locals, JIT emits wrong instruction sequence --
e.g, 28 byte zero-intialization as shown below.
The issue was JIT passed wrong arguments to emitIns_R_R_I.

Before (Fail)
```
 stp     xzr, xzr, [x2],#16
 str     xzr, [x2,#2]  --> just two byte offset (no x2 post-increment)
 str     wzr, [x2]
```

After (Pass)
```
 stp     xzr, xzr, [x2],#16
 str     xzr, [x2],#8
 str     wzr, [x2]
```

Merge pull request #3685 from jkotas/ifdefs

Fix ifdefs

Merge pull request #3662 from tarekgh/FixGetYearForJapaneseCalendar

Fix calculating the calendar year in JapaneseLunisolarCalendar

Merge pull request #3673 from briansull/update-tests-lst

ARM64: Major Update for new tests to run using Smarty.exe

Remove remaining strongname validation on Windows

CodeCLR always skips strongname validation. Some code for strong name validation was still compiled for Windows, but not for Unix. This change removes the differences between Windows and Unix by never compiling the strong name validation code in.

Remove incorrect use of FEATURE_LEGACYSURFACE ifdef

Merge pull request #3683 from sergiy-k/runtestsfix1

Exclude netcore50aot assemblies in runtests.sh

Add test mode exclusions

Add the ability to exclude individual tests that are incompatible with
certain test modes.  If a test is incompatible with or too slow under
GCStress, add

    <GCStressIncompatible>true</GCStressIncompatible>

as a property to the test's project file.  If a test requires optimization
or is sensitive to precise optimization patterns, use

    <JitOptimizationSensitive>true</JitOptimizationSensitive>

Exclude netcore50aot assemblies in runtests.sh

Merge pull request #3671 from brianrob/multicorejitthread

Use Default Stack Size for Multi-Core JIT Playback Thread

Merge pull request #3678 from sergiy-k/runtestsfix

Patch the runtests.sh script to work around an issue with corefx assemblies

Merge pull request #3674 from jkotas/tail-call

Fallback to JIT for explicit tailcalls in R2R

Patch the runtests.sh script to work around an issue with corefx assemblies

Change https://github.com/dotnet/corefx/pull/6779 in CoreFx added corert
assemblies to the mix of assemblies that are built for CoreCLR on Unix
which broke the logic in runtest.sh that creates an overlay for running
tests. This change creates a work around for the problem.

Merge pull request #3590 from jkotas/deadcode

Delete dead code

Merge pull request #3631 from cmckinsey/DevDiv_199169

Fix bad codegen for initblk with large fill value

Merge pull request #3675 from mmitche/fix-names

Fix name on trigger

Fix name on trigger

Fallback to JIT for explicit tailcalls in R2R

Major Update for new tests

Updated Tests.lst and preserved Categories tags
We can run all of the Pro, Pri1 tests
We currently run only the ones that were passing
from the previous Tests.lst file

Using StackString to improve memory consumption in Lttng EventProvider

Merge pull request #3672 from mmitche/update-badges

Update badges to point to new master branch jobs

Update badges to point to new master branch jobs

Merge pull request #3197 from mmitche/branchify

[Do Not Merge] Move coreclr to branch model

Merge pull request #3658 from leemgs/upstream-doc-crossbuild

Documentation: Remove the "patching Urcu" section

Fix calculating the calendar year in JapaneseLunisolarCalendar

The months in this calendar is lunar months. We can have the case when a new era
starts in the middle of a month which means part of the month will belong to one
era and the rest will belong to the new era. When calculating the calendar year
number for dates which in the rest of the month and exist in the new started era,
we should still use the old era info instead of the new era info because
the rest of the month still belong to the year of last era.

Example of such date is Feb 2nd 1989. Using such date while the era starts
on Jan 8th 1989 make this date belong to that new era which it is part of
the month started with previous era. Using the new era info will have the year
we are calculating equal to the era offset (i.e. year = m_EraInfo[i].yearOffset)
and we’ll end returning zero for the year. The fix is to use the previous era
info to calculate the year

issue https://github.com/dotnet/corefx/issues/6775

Merge pull request #3645 from sejongoh/increase_gctest_time_out

Increase timeout and crond schedule for GC stress testing

Move coreclr to branched CI model

This moves coreclr to the branch-based CI model.  In this model, instead of having the CI generate jobs out of dotnet/coreclr's master branch for all branches (using all sorts of odd naming suffixes of course), the master CI lists tell the CI which branches it should look at for CI config info.  It passes the branch name to the netci file, which then can use it to tell the SCM to pull code from specific branches or set branch specific PR triggers.

Merge pull request #3653 from briansull/migrate-tags

Add migrate-tags.py script for migrating Smarty tags

Merge pull request #3252 from mjp41/master

Fixed parent pointers becoming corrupt in GC benchmark for Red/Black trees

Added Header and other Codereview feedback

Documentation: Remove Patching Urcu Section

Atomic function parameter types Urcu is already fixed.
We don't need any manual patching for the cross build.
Let's remove this section from now on.

Signed-off-by: Geunsik Lim geunsik.lim@samsung.com
Signed-off-by: Prajwal A N an.prajwal@samsung.com
Signed-off-by: MyungJoo Ham myungjoo.ham@samsung.com

a This is a combination of 2 commits.

Increase timeout and crond schedule for GC stress testing

Merge pull request #3630 from sejongoh/pull_down_coredistool_windows

Pull down coredistool package on Windows

Update Multi-Core JIT to use the default stack size for the JIT playback thread.

Add migrate-tags.py script for migrating Smarty tags
Also added a test case for the script

Fix bad codegen for initblk with large fill value

When the initialization value of an init block is constant we attempt to
generate an inline expansion sequences to execute the stores. In order to
generate wider stores the constant value is expanded so that each byte
position holds the initial byte value. In some cases the constant on the
initblk may be a value not in the range of 0..255 and this constant
expansion will generate the wrong value. This change fixes the x64, x86,
and arm64 (comment only) expanses to use only the lower byte value of this
constant. The ARM32 backend already does this, and the other JIT32/JIT64
JITs shipping in Desktop also do this correctly.

Enabling standalone builds for Test Project of any Priority

Merge pull request #3647 from dagood/fix-pri1-test-dependencies

Fix priority 1 tests: add missing dependencies

Merge pull request #3632 from AndyAyersMS/InlineRefactorMoveProfitability

Inline refactoring: move profitability assessment to LegacyPolicy

Merge pull request #3652 from rahku/EHTestPort

exclude test baseservices\exceptions\regressions\Dev11\147911

exclude test baseservices\exceptions\regressions\Dev11\147911

Merge pull request #3644 from JohnChen0/master

Fix a GC hole in a Ready-to-Run helper

Add new dependencies to tests where package upgrade moved types.