Fix CrossGen documentation error

Merge pull request #4016 from dotnet/revert-3960-check_runtime_to_pull_coredistools

Revert "Add architeture information to a script downloading coredistools"

Merge pull request #4017 from mikem8361/issue3987

Fixed issue #3987 "Debugger fails to stop on entry due to stack walk failure"

Fixed issue #3987 "Debugger fails to stop on entry due to stack walk failure"

By initializing m_rgHijackFunction to NULL.

Merge pull request #3982 from mikem8361/readmemoryav2

Fix ReadMemory AV.

Revert "Add architeture information to a script downloading coredistools"

Merge pull request #4002 from kyulee1/fixcmd

ARM64: Running cmd instead of exe

ARM64: Running cmd instead of exe

Our test framework (smarty) directly runs test executable on corerun.exe.
But some of tests expect arguments via environment variable specified in .cmd.
The fix is to change the test list file to run .cmd. So the expected pass
value is also updated to 0 instead of 100. Now we don't need to pass /host
that specifies corerun.exe.
3 additional tests are now passed:
 InterlockedAddLongWithSubtract/CompareExchangeTString/Polyrec

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

Update with 10 additional passing tests, and two moved to unstable

Merge pull request #4005 from dotnet/revert-3943-upstream-gawk-char-class

Revert "Fix bracket expression matches a character at gawk"

Revert "Fix bracket expression matches a character at gawk"

Update with 10 additional pasing tests, and two moved to unstable

Merge pull request #3955 from davmason/master

Fixing tracelogging on linux

Merge pull request #3996 from kyulee1/fixconv

ARM64: Fix conversion double to unsigned int

ARM64: Fix conversion double to unsigned int

For ARM/ARM64, converting negative double to unsigned int is 0, which aligns with all native compilers.
I also removed the part of tests that depends on architecture specific values.
Since ARM64 tests were prepopulated, I just updated the tag so that we can update the tests later.

Merge pull request #3960 from sejongoh/check_runtime_to_pull_coredistools

Add architeture information to a script downloading coredistools

Merge pull request #3983 from svick/getresourcestring-culture

Remove unused CultureInfo from GetResourceString

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

remove unused CultureInfo from GetResourceString

Fix ReadMemory AV.

Use simple probing to validate read/write memory with a try/catch and explicit h/w exception holder.

Put probing in separate noinline and optnone function for optimized builds.

Fix assert in exception code mapping in context.cpp by handling SIGSEGV subcode SI_KERNEL.

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.

Add architeture information to a script downloading coredistools

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.

Fixing tracelogging on linux

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