Fixing contracts violations, and adding missing contracts to some APIs.
Changing native format reader failure mode to fail fast instead of throwing and exception.

[tfs-changeset: 1591477]

Merge pull request #3994 from dagood/upgrade-cli

Apply dotnet cli upgrade also to tests\dir.props

Apply dotnet cli upgrade also to tests\dir.props.

Merge pull request #3993 from dagood/upgrade-cli

Upgrade dotnet cli version

Upgrade dotnet cli version.

Merge pull request #3943 from leemgs/upstream-gawk-char-class

Fix bracket expression matches a character at gawk

Merge pull request #3984 from svick/patch-1

Removed obsolete comment from Environment.GetResourceString

Merge pull request #3985 from myungjoo/trivialfix/macro1

Trivial fix: dangerous macro usage

Trivial fix: dangerous macro usage

Add parenthesis to make the macro safe.

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>

Removed obsolete comment from Environment.GetResourceString

Fixes #3979

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

Disabled 8 failing tests

Disabled 8 failing tests

Issues filed: 3968, 3969, 3970, 3971 and 3973

Merge pull request #3966 from kyulee1/fixframe2

ARM64: Fix Frame with compLocallocUsed

ARM64: Fix Frame with compLocallocUsed

When compLocallocUsed is true, stack pointer is changed during the run.
So, JIT should restore stack pointer from frame pointer in the epilog.
Previously, we simpliy copied fp to sp, which is only valid for frameType
= 1.
This implements a right restoring process for other frameTypes (2 and 3).
For frameType = 3, I also simplified creating prolog sequence to support
this feature consistently.

Merge pull request #3963 from adityamandaleeka/statFileParse

Fix stat file parsing to handle executable names with spaces

Merge pull request #3959 from briansull/safe-refactor

Safe Refactor - Remove conditional definition and references to lvIsMultiRegArgOrRet

Merge pull request #2182 from justinvp/keyedcollection_enumallocs

Avoid enumerator allocations inside KeyedCollection<TKey, TItem>

Merge pull request #2178 from justinvp/keyedcollection_nullchecks

Remove unnecessary null checks from KeyedCollection<TKey,TItem>

Fix stat file parsing to handle executable names with spaces.

Safe Refactor - Remove conditional definition and references to lvIsMultiRegArgOrRet

Merge pull request #3954 from leemgs/toolchain-name-unifyin-arm64

[Linux/ARM32/64] Unifying the redundant usages of cross-toolchain

[Linux/ARM32/64] Unifying the redundant usages of cross-toolchain

The name of toolchain name has been increasing gradually.
Let's unify the toolchain name of Linux/ARM{32|64} to prevent the
increase of the fragmentation, and to the manatainability.

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

Merge pull request #3946 from leemgs/toolchain-name-unifying

[Linux/ARM32] Unifying the redundant usages of cross-toolchain

Merge pull request #3902 from JosephTremoulet/ZeroExtend

Unsuppress same-reg zero-extending mov (x64)

[Linux/ARM32] Unifying the redundant usages of cross-toolchain

The name of toolchain name has been increasing gradually.
Let's unify the toolchain name of Linux/ARM32 to prevent the
increase of the fragmentation, and to the manatainability.

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

Fix: bracket expression matches a character at gawk

This is a trivial patch. However, It's must be modified by gawk.
Check it exactly as a plain character (e.g., space, alnum)
within the bracket expression at ./coreclr/generateversionscript.awk

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

Merge pull request #3896 from adityamandaleeka/configvalues4

Add new configuration mechanism for CoreCLR.

Merge pull request #3931 from Godin/fix-typos

Fix copy-paste typos

Merge pull request #3920 from CarolEidt/ChangeLdObjToObj

1stClassStructs: Replace GT_LDOBJ with GT_OBJ

Merge pull request #3928 from manu-silicon/patch-3

Fix invalid path to genXplatLttng.py

Add new configuration mechanism for CoreCLR.

Merge pull request #3934 from briansull/remove-lvaAdjustRefCnts

Remove empty phase lvaAdjustRefCnts

1stClassStructs: Replace GT_LDOBJ with GT_OBJ

In preparation for using block nodes in assignments, change GT_LDOBJ to GT_OBJ.
Also, eliminate gtFldTreeList, which was only
being used in a transitory fashion for x87
codegen - instead create the nodes on the fly as needed for stack fp codegen.
Additional minor cleanup.

Merge pull request #3933 from AndyAyersMS/RefactorPolicy

Inliner: refactor policies to extract common legality portion

Merge pull request #3924 from AndyAyersMS/Fix-VSO-205323

Inliner: fix issue introduced by refactoring

Remove empty phase lvaAdjustRefCnts

Merge pull request #3932 from kyulee1/arm64tool

Fix ARM64 lib toolchain

Unsuppress same-reg zero-extending mov (x64)

Update CodeGen::genIntToIntCast to stop suppressing 32-bit same-register
`mov`s, and to stop assuming that 32-bit enregistered sources already have
the top half of their register clear.  This latter assumption is usually
true, but is not guaranteed across function boundaries by the ABI.  As it
happens, the runtime code that invokes custom attribute constructors can
pass garbage in the top half of such parameters; this change adds a
testcase that fails on that path without this fix.

Inliner: refactor policies to extract common legality portion

Now that we have multiple policies, refactor their implementation
to extract common elements. In particular all the existing
policies have the same legality and ability constraints, so this
logic is extracted to a partial `LegalPolicy` superclass.

Adresses relevant feedback on #3697, where the number of policies
increased from one to two.

No changes in behavior.

Fix ARM64 lib toolchain

This fixes path for the right libs that are consistent with tools.
With this, crossgen.exe can run natively on arm64.

fix

Fix copy-paste typos

Merge pull request #3907 from davmason/master

Fix spurious warning about parameter counts

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

Merge changes from TFS

Merge pull request #3908 from AndyAyersMS/JitTimeData

Inliner: collect jit time data

Fix build break on full framework

[tfs-changeset: 1589984]

Fix invalid path to genXplatLttng.py

Merge pull request #3900 from jkotas/legacynetcf

Delete legacy netcf compat support (final part)

Merge pull request #3921 from myungjoo/fix/3859

Fix #3859 ARM/Unwind: explicitly mark the end of stack

Merge pull request #3923 from myungjoo/doc/arm_0001

Update Linux Build Instruction

Merge pull request #3917 from jkotas/structinregs

Add explicit calling convention modifiers for x86

Update Linux Build Instruction

For Linux/ARM, you need to update libunwind-arm
in order to avoid segmentation faults in libunwind-arm,
which is not included in major Linux distros, yet,
while it is already being addressed in upstreams.

This is realted with #3898 Issue Comment 201098037:
https://github.com/dotnet/coreclr/issues/3898#issuecomment-201098037

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>

Merge pull request #3919 from stephentoub/specialcase_emptystring

Special-case hashcode of empty length string on Unix

Inliner: fix issue introduced by refactoring

Refactoring done in #3799 inadvertently modified logic in
`fgFindJumpTargets` so that stored-to arguments might not be marked as
such in the `lvaTable' when crossgenning a root method. This lead to
asserts in internal test cases.

This change restores the logic to its prior state, and adds a test
case that will trigger an assert if the assembly is crossgenned and
this fix is not applied.

Inliner: collect jit time data

This is an initial cut at collecting the time spent jitting a method
as part of observing the incremental impact of inlines.

It currently only measures the time after inlining has run, because there
are substantial interactions with the EE before this point and the time
spent in those calls is not always going to properly attribute to the
current method. We've had some good internal discussion on other
approaches to this and will likely revisit as time goes on. There was
already some timing support in place for SQM so I've generalized that
and shared it for this case too.

This also fixes an off-by-one issue with the method version which came
up when trying to back-correlate the inline data into the inline trees.
The data dumps now include root method data, and so we capture the IL
size and force inline state of the root method in the data dumps. This
allows us to investiage if observed inliner data is corrlated with the
current size of a method.

Data output is now also supported in RET builds. A custom build that
defines `INLINE_DATA` is required. Output is sent to stderr to work
around issues with crossgen.exe reopening stdout in a non-ascii mode.

Fix ARM/Unwind: explicitly mark the end of stack

Libunwind-arm unw_step returns 0 with unchanged PC
if it hits the bottom of the stack.

Thus, seh-unwind.cpp::PAL_VirtualUnwind should
explicitly mark that it has reached the bottom
as it does with ARM64, FreeBSD, or APPLE for
ARM/Linux.

Fix #3859

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>

Special-case hashcode of empty length string on Unix

This is special-cased in the Windows build, but not currently in the Unix build.

Merge pull request #3885 from sejongoh/test_env_script

Add --test-env option to tests/runtest.sh to set stress mode environm…

Add explicit calling convention modifiers for x86

Merge pull request #3904 from kyulee1/NoMT

Remove static lib dependency on CrossGen

Merge pull request #3905 from kyulee1/arm64cross

Enable CrossGen Compilation for ARM64

Merge pull request #3814 from tijoytom/master

Marshal tests for various scenarios.

Add --test-env option to tests/runtest.sh to set stress mode environment varibles just before tests run.

Fix spurious warning about parameter counts

Enable CrossGen Compilation for ARM64

This simply removes the blocking code that prevents ARM64 crossgen
compilation. Otherwise, we got InvalidProgramExceptions for all methods.
I've validated this allows crossgen to compiles all mscorlib methods.
Toward the end, there is NYI assertion in Zap which is expected.

Remove static lib dependency on CrossGen

The motivation is for enabling ARM64 which does not have correct static
lib with the current toolset.
But looking at other console apps (ilasm/ildasm/coreconsole/corerun) or
coreclr.dll, they also have dependency on msvcrt for Windows.
So, I've decided to make crossgen.exe with the same flavor.
This changes reduces the binary size -- 10M -> 9M (Debug). Release binary
is slightly smaller.
I've validated this by comparing .ni.dll for all FX assemblies that we use for
tests (under CORE_ROOT), which are identical before and after.

Merge pull request #3891 from JohnChen0/r2r

Fix generic struct delegate in ReadyToRun (#3690)

Merge pull request #3682 from mikedn/emplace

Avoid Interval and RefPosition unnecessary initialization and copying

Merge pull request #3846 from brianrob/perftests

Add Performance Tests from the CLR Perf Lab

Delete legacy netcf compat support (final part)

Merge pull request #3899 from manu-silicon/patch-2

Fix typo causing an "unary operator expected"

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

Merge changes from TFS

Merge pull request #3901 from myungjoo/trivial/fix1

Trivial Fix: macro usage of __linux__.

Fix: macro usage of __linux__.

Use #if define(__linux__) instead of if #if __linux__.

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>

Fix typo causing an "unary operator expected"

The default initialization value for doCrossgen was mispelled causing a warning at
./tests/runtest.sh: line 351: [: ==: unary operator expected

Fix build break in full framework

[tfs-changeset: 1589425]

Fix build break in nativeformatreader.h

[tfs-changeset: 1589424]

Merge pull request #3890 from kyulee1/removeryujitpack

Remove Old NuGet RyuJit Package from Build

Merge pull request #3888 from jkotas/platform-assemblies

Remove special casing of platform assemblies

Fix generic struct delegate in ReadyToRun (#3690)

The following code pattern triggers a crash when compiled into Ready To Run:

    struct S<T>
    {
        public void M() { ... }
    }

    class C { }

    Then create a delegate using S<C>.M

The cause of the crash is calling S<C>.M normally reaches a shared generic
method S<__Canon>.M, but for delegate creation we need to use the actual
S<C>.M. Ready To Run generated code that attempts to create delegate using
S<__Canon>.M, which is not allowed.

Merge pull request #3880 from gkhanna79/R2RTestFX

Precompile FX assemblies for test run

Merge pull request #3892 from fadimounir/FixBB

Fix build break

Merge pull request #3893 from wtgodbe/r2r_GCStress

Add per-commit runs & PR trigger phrases for Pri1 R2R GCStress 15 test runs for Windows, OSX, CentOS, and Ubuntu

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

Changed 22 UNSTABLE tests to EXPECTED_PASS

Add per-commit runs & PR trigger phrases for Pri1 R2R GCStress 15 test runs for Windows, OSX, CentOS, and Ubuntu

Delete security transparency tests that are not valid on .NET Core

The security transparency rules are not enforced on .NET Core.

Fix build break

Merge branch 'master' of https://github.com/dotnet/coreclr

Merge pull request #3887 from rahku/master

Fix XunitWrapper log file generation

Remove Old NuGet RyuJit Package from Build

This eliminates the old way of nuget package creation as a part of prep:
https://github.com/dotnet/coreclr/issues/3442.

Changed 22 UNSTABLE tests to EXPECTED_PASS

This reduces the number in the UNSTABLE catagory to 24 tests

Fix XunitWrapper log file generation

Merge pull request #3881 from rahku/issue3730

offset in epilog to match as prolog

Merge pull request #3868 from wtgodbe/xunit

Print raw output for failed tests in CI/Jenkins failure report

Enable support to precompile the FX assemblies at test invocation time.

Add performance tests from the CLR perf lab.

Merge pull request #3735 from fadimounir/HashtableOfTypes

Hashtable of types for R2R modules

Remove special casing of platform assemblies

Title: Hashtable of types for R2R modules
Description: This change replaces the hashtable of types that we dynamically build at module load time with a statically built hashtable at crossgen time, for R2R modules. Typedef tokens and exported type tokens are stored in that hashtable, hashed by their full type name.

Changes include:
- Adding support for NativeFormat hashtables (NativeFormat hashtable reader and writer + supporting functionalities)
- Changes to prevent the allocation and building of m_pAvailableClasses and m_pAvailableClassesCaseIns for a R2R module
- Logic in crossgen to build the hashtable of types, using the NativeFormat (table saved to a new section)
- Refactoring the type lookup functions to return results in a HashedTypeEntry data structure, which supports token based results, as well as EEClassHashEntry_t* based results
- Changes/Cleanup to the ClassLoader::GetClassValue to support the new lookup type
- Fixed small bugs in the std implementation
- Fallback to old hashtable lookup algorithm supported for:
1) Case insensitive type loading
2) R2R image built with older version of crossgen (not having static hashtable)

Tested following scenarios:
- Token based lookups
- Type name based lookups (Type.GetType API), including case insensitive scenarios
- Nested types
- Forwarded types (and forwarded nested types), up to 2 levels of forwarding
- Old R2R image created with previous version of crossgen

offset in epilog to match as prolog

Print raw output for failed tests in CI/Jenkins failure report