Merge pull request #3812 from janvorli/fix-stub

Fix instantiating stub for methods of value types

Merge pull request #3809 from wtgodbe/utf8

Ported managed Utf8/Unicode encoder/decoder to C++ for usage in PAL

Fix instantiating stub for methods of value types

This change fixes a problem with instantiating stubs for methods of value types.
The problem was that the CreateInstantiatingILStub didn't take into account the
fact that methods of value types need to have "this" passed "byref".
The issue manifested itself as a rare corruption of references in array of structs
that were thin wrappers for string reference during GC stack scan. GC thought that
the reference to an array entry is an object reference that starts with method table.
GC marks method table pointers by setting their bit zero to 1. But in this case,
it has accidentally modified an object reference instead and a test was crashing
with wrong object address.

The root cause of the problem is that the instantiating stubs were placed on a global
singleton reference class no matter whether the target method was on a reference class
or a value type.

I have fixed it by putting the stubs on the instantiated target generic type instead.

Ported managed Utf8/Unicode encoder/decoder to C++ for usage in PAL

Merge pull request #3808 from mmitche/arm64-cross-label

Update label for arm and arm64 cross builds on Ubuntu

Update label for arm and arm64 cross builds on Ubuntu

Update the label to the newer image which contains the required rootfs.

Merge pull request #3757 from leemgs/upstream-crossgen-skip-for-arm

The architecture of host pc must be same architecture with target.

Merge pull request #3799 from AndyAyersMS/AdjustInlineObservations

Inliner: make more observations unconditional

Architecture of host must be compitable architecture with target.

Let's make exception handling for different architecture between
host pc and target board during build-time.

V3:
- Simplify the existing 'if' statement

V2:
- Added $HostArch variable to handle the required architectures.

V1:
- Handle "time ./build.sh arm debug clean verbose" command
in case that generate only mscorlib.dll for Linux/ARM as well as the
existing case of if [ $__CrossBuild != 1 ]

In this case, we need to add additional if statements because the
architecture of host pc must be same architecture with target.

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

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

Merge changes from TFS

Merge pull request #3796 from kyulee1/fixalignchk

Fix Stack Alignment Check

Merge pull request #3783 from mikem8361/fixsosbp

Fixed problem with bpmd not working sometimes.

Merge pull request #3745 from manu-silicon/docs

Ensure proper spelling of COMPlus_xxx environment variables

Merge pull request #3797 from svick/patch-2

Update out of date comment in List<T>

Add back some APIs needed by legacy test runner, for now

[tfs-changeset: 1586974]

Fix Stack Alignment Check

This fixes https://github.com/dotnet/coreclr/issues/3747.
If JIT saves the first single store (not pair store) with stack adjustment
required (```spDelta != 0```), we expect the offset to be 8 to account for alignment.
This API ```genPrologSaveReg``` is invoked twice from INT and FP saving respectively
while JIT stores/allocates stacks once for these saving registers back to back.
So, the assertion is not quite right. For instance when we have 1 INT and
1 FP registers to be saved, JIT failed to assert the offset to be 8 in the
first invocation for INT. In fact, for this case, the offset should be 0
(no alignment is required) since we store 2 registers which are already
aligned on 16 byte.

We should consider whether or not the total number of saved registers is odd.
Not only that, even for the pair store, we should assert the offset either
0 or 8 depending on the total number of saved registers. For instance, 2
INT and 3 FP, we want the offset to be 8 when we store the first pair with
stack adjustment required (```spDelta != 0```).
I refactored the code to reflect this issue in ```genSaveCalleeSavedRegistersHelp``` while taking out the
existing assertion in ```genPrologSaveReg``` which is too local.
Similar change is made for the restore case.

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

Enable running of the full Pri0 and Pri1 set of tests

Enable running of the full Pri0 and Pri1 set of tests

Updated with EXPECTED_PASS and EXPECT_FAIL
Marked 50 test with NEW_PASS that will pass after the next test update
Fixed the location of 5 the threadstatic tests

Inliner: make more observations unconditional

There are still some observations within `fgFindJumpTargets` that are
only made when inlining. This change enables them to also be made for
the prejit root case. This partially addresses the inconsistencies
noted in #3482.

Most notably, the simple stack model is now fully enabled for the
prejit root case, and the policy is able to more broadly observe
things like `CALLEE_LOOKS_LIKE_WRAPPER` and
`CALLEE_ARG_FEEDS_CONSTANT_TEST`.  As before, the `LegacyPolicy`
ignores these observations for the prejit root. Forthcoming policies
will likely behave differently.

Two observations that were fatal -- `CALLEE_UNSUPPORTED_OPCODE` and
`CALLEE_STORES_TO_ARGUMENT` -- are now conditionally fatal, since they
do not cause failure for the prejit root case (though arguably they
should).

The `CALLEE_IS_MOSTLY_LOAD_STORE` observation has been removed, and
the conditions under which it fired have been encapsulated within the
`LegacyPolicy`.

No change in behavior is expected. I verified crossgen of mscorlib
still makes all the same inline decisions for the same reasons.

Merge pull request #3781 from rahku/helixFix

Update buildtools version. Shorten filename of tests

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

Update out of date comment in List<T>

Comment for parameterless constructor of List<T> claimed that
the default non-zero capacity was 16, even though _defaultCapacity
is actually 4.

Updated the comment to refer to _defaultCapacity,
to make sure it doesn't need to be updated again
if _defaultCapacity changes in the future.

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]

Update buildtools version. Shorten filename of tests

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.

Fixed problem with bpmd not working sometimes.

Fix some too earlier SOS initialization problem. Now returns an error.

The DAC interface (IXCLRDataProcess) was being created everytime an command was run and the JIT and GC notification tables where being reinitialized each time losing any JIT notifications needed to resolve a breakpoint. Only create the DAC interface instance once. It does need to be flushed each time sos is entered though.

Enable the module load and unload and exception callbacks when the DAC instance is created. This is what windbg does on Windows (along with flushing the DAC each time the target is restarted). It simplifies the breakpoint code; it no longer needs to enable/disable these notification flags.

Cleaned up places where the DAC instance (direct calls to LoadClrDebugDll) and not released properly.

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

Update version based on new content of clrconfigvalues.h

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.

Update code to use COMPlus_ instead of COMPLUS_ when referencing knobs

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

Ensure proper spelling of COMPlus_xxx environment variables

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