Add regression tests for two previously fixed bugs.

One of the repros is in il rather than C# because the
repro was sensitive to what different versions of C#
compiler generated.

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

Merge changes from TFS

Merge pull request #3793 from jkotas/hostprotectionattribute

Add back HostProtectionAttribute to mscorlib

Use "debian.8" instead of "debian.8.2"

CoreFX uses "debian.8" as the rid for debain packages, whereas CoreCLR
was still on "debian.8.2". Move to "debian.8" because it matches what's
in /etc/os-release and what the CLI will generate by default.

[tfs-changeset: 1586899]

Add back HostProtectionAttribute to mscorlib

Merge pull request #3780 from AndyAyersMS/DiscretionaryInliner

Inliner: create DiscretionaryPolicy

Merge pull request #3790 from wtgodbe/timeout

Increase timeouts for R2R jobs to 4 hours. Also enable mistakenly disabled CentOS R2R jobs

Merge pull request #3770 from sejongoh/fix_setup_runtime_dep_cmd

Fix setup-runtime-dependencies.cmd

Merge pull request #3782 from sergiy-k/runtestsfix2

Patch runtest.sh to skip netstandard15aot assemblies

Increase timeouts for R2R jobs to 4 hours. Also enable mistakenly disabled CentOS R2R jobs

Merge pull request #3786 from jashook/arm64_ci_timeouts

Arm64 Windows timeout increase

Merge pull request #3773 from wtgodbe/WindowsPri1

Switch default Windows x64 Release PR test to pri1 in CI

Arm64 Windows timeout increase

Currently the arm64 ci is timing out at 2 hours. When this happens the arm64 machines are left in a undefined state. Increasing the timeout to avoid this.

Merge pull request #3771 from jashook/smarty_scripts

Add lst_creator tool and smarty output parser

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

Merge changes from TFS

Merge pull request #3774 from rahku/Helix

Add xunit console runner dependency

Patch runtest.sh to skip netstandard15aot assemblies.

Merge pull request #3740 from ramarag/createlayouts

Creates RunTime Dependency Layout as a seperate step

Merge pull request #3779 from briansull/issue-3778

Fix  for Issue 3778

Inliner: create DiscretionaryPolicy

The `DiscretionaryPolicy` is similar to the `LegacyPolicy`, but does not
use size limits. So there is no "always" inline class and no "too big to
inline" class. Also, discretionary failures do not trigger noinline
stamping.

It is installed if `JitInlinePolicyDiscretionary` is set nonzero.

See #3775 for some background on where this new policy will be used.
This is a first cut and further refinement is likely.

Also removed the unused `NoteDouble` methods since the double-valued
observations now are kept internally within `LegacyPolicy` and it's
not very likely we'll introduce new cases where doubles are needed.

Add xunit console runner dependency

Fix setup-runtime-dependencies.cmd

Merge branch 'master' into issue-3778

Fix issue 3778

Move the initialization of compGenTreeID to earlier in Compiler::compCompileHelper

Remove unused files from CoreCLR

[tfs-changeset: 1586532]

Merge pull request #3762 from AndyAyersMS/FixInconsistentPolicy

Use root compiler instance for inline result and policy

Switch default Windows x64 Release PR test to pri1 in CI

Add lst_creator tool and smarty output parser

Merge pull request #3751 from jashook/arm64_enable_dbg_chk

Remove release only check.

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

Merge changes from TFS

Use root compiler instance for inline result and policy

When constructing an InlineResult (and subsequent InlinePolicy)
always use the root compiler instance, so that all policy evaluations
done in a method are consistent.

Add an assert that the initial fact replay during `fgFindJumpTargets`
(force inline state and code size) doesn't change the candidacy when
inlining -- it should just be re-establishing what we knew in the
initial candidate scan.

Closes #3760.

Merge pull request #3761 from kyulee1/fixstack

ARM64: Fix Large Stack Allocation

ARM64: Fix Large Stack Allocation

This fixes https://github.com/dotnet/coreclr/issues/3666.
When frame size is greater than 512 (frametype=3), JIT saves
callee-save registers followed by var arg registers, and then handles the
remaining frames. When the first callee-save register is saved, the stack
size is expected to be grown including delta amount which is for var arg registers.
For this failing case, actually we don't have any callee-save registers to be
saved while we still need to home var arg registers.
This fix is to handle such case in
```genSaveCalleeSavedRegistersHelp/genRestoreCalleeSavedRegistersHelp``` to
ensure allocating/deallocating delta stack size.

Merge pull request #3758 from gkhanna79/TestGCStress

Enable tests to run under a specified GCStress mode

Merge pull request #3755 from jkotas/failing-tests

Disable tests that are failing in CI on Unix

Disable tests that are failing in CI on Unix

Merge pull request #3697 from AndyAyersMS/RandomInliner

Random inliner

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

Remove nugetbuild myget feed

Enable tests to run under a specified GCStress mode

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>

Prune Windows mscorlib

In model.xml, I went over:
- FEATURE_ conditions that are disabled on Unix
- FEATURE_  conditions related to NetCF
- All ApiFxInternal methods
And flipped most them either to public, ImplRoot or deleted them.

Testing: Run all open CoreCLR and CoreFX tests, built legacy build razzle

[tfs-changeset: 1586209]

Remove release only check.

Allows debug and checked builds to run tests on arm64 machines.

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

Creates RunTime Dependency Layout as a seperate step

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