Merge pull request #3295 from chuck-mitchell/feature

Initial version of CoreCLR JIT Performance Harness

Merge pull request #4015 from sejongoh/update_dotnet_cli_path

Fix tests/setup-runtime-dependencies.sh as per dotnet cli changes

Merge pull request #4006 from jashook/arm32_ci_pr

Add arm32 build per pr.

Merge pull request #4024 from AlexGhiondea/mscorlibBuild

Fix issues with building mscorlib inside VS

Merge pull request #3953 from LLITCHEV/Issue3604

Make sure 4 outgoing arg slots are generated for methods that call

Merge pull request #4029 from pgavlin/Host64Bit

Replace uses of _WIN64 with _HOST_64BIT_.

Replace uses of _WIN64 with _HOST_64BIT_.

This terminology is more accurate. This change also corrects
a few bits of code that were making target-related decisions
based on the host's bitness.

Merge pull request #3914 from cmckinsey/GitHub_3600

Fix for stack overflow in CFG/DOM analysis GH#3600

Merge pull request #4030 from kyulee1/updateryujit

Update RyuJit version

Update RyuJit version

Add arm32 build per pr.

Merge pull request #4018 from mikem8361/sosplugfix

sos fails to load if binaries are copied to different location than built

Merge pull request #4009 from pgavlin/EmitXarchClarify

Clarify register aliasing in the X86 encoder.

Merge pull request #4014 from dagood/upgrade-cli

Upgrade dotnet cli to take win7 fix

Merge pull request #3967 from stephentoub/empty_memorystream

Avoid array allocation in "new MemoryStream()"

Fix tests/setup-runtime-dependencies.sh as per dotnet cli changes

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

Merge changes from TFS

We should no longer need to use the '/test:moduleName=CommonLanguageRuntimeLibrary' flag when building mscorlib.

Do not use the 'Visible' metadata on the items in the ItemGroup 'MscorlibSources'

We are going to import that itemgroup into the 'Compile' ItemGroup which will
make them show up in VS. Having the 'Visible' metadata on the items means we end
up with duplicated items in Solution Explorer.

Merge pull request #4008 from leemgs/upstream-typo-build-mode

Improve description of build type spelling for consistency

Merge pull request #4020 from JohnChen0/master

Fix CrossGen documentation error

Merge pull request #3944 from leemgs/upstream-cross-arm-stripping-symbols

[Linux/ARM32] objcopy: Support cross-build  for stripping symbols.

Merge pull request #3989 from bbowyersmyth/StringEquals

Port String.Equals from CoreRT

Merge pull request #3981 from manu-silicon/context

Fix RtlRestoreContext

Fix CrossGen documentation error

Fix RtlRestoreContext

Due to macro expansion, the previous computed offsets to restore Lr and Pc
were incorrect, causing a memory corruption (see Issue #3856 for C# code
reproducing that problem).
Made assembly more obvious when just restoring Sp, Lr and Pc.

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

Revert "Add architeture information to a script downloading coredistools"

Fix R2R fixups for delegate ctors

- Add getReadyToRunDelegateHelper to JIT-EE interface. This method has an extra argument describing the type of the delegate. This argument is required to emit correct fixups.
- Call the new method in both JIT and JIT32
- Add the new method to superpmi
- Bump minor version of R2R file format
- Add regression test for #3975 to R2R unit test

Fixes https://github.com/dotnet/coreclr/issues/3975

[tfs-changeset: 1591698]

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.

Add back explicitly loading mscordaccore so libsos will properly load when installed at a different location than it was built.

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

Getting SOS to work on ARm64:

     This mainly involved DACizing the VM code.
      A bulk edit for changing RUNTIME_FUNCTION to T_RUNTIME_FUNCTION

[tfs-changeset: 1591667]

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

Upgrade dotnet cli to take win7 fix.

Change spelling of build type for consistenty

This is trivial patch.
Let's display a help manual with lower case for better consistency
when the developers run "./build.sh --help".

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

Clarify register aliasing in the X86 encoder.

Certain registers are aliased with control bits in the ModRM byte
of an X86 instruction. Encapsulate the checks that accommodate this
aliasing into a pair of functions.

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

Make sure 4 outgoing arg slots are generated for methods that call
THROW_HELPERS for Windows x64.

This is a long standing bug in RyuJit.
When compiling a method for debug mode if there is a call to THROW_HELPER
(and it is the only call) the frame for the method that calls the
THROW_HELPER is not properly set - it is missing the 4 ougoing arg slots.

This was discovered by R2R codegen. In normal jitting of such method in
debug mode the jit always adds a call to CORINFO_HELP_DBG_IS_JUST_MY_CODE
early enough, so the frame size will be properly calculated. This call is
not generated for R2R.

If the method generated for R2R has only a call to a throw helper
and it is compiled for debug mode, the call is not generated in the IR.
The CodeGen generates the call directly instead. The call is generated
after the stack frame is calculated. This leads to improperly generated
stack frame that doen't have the 4 outgoing arg slots that the Windows x64
ABI requires.

Compiling such method for release mode is never a problem since in
FlowGraph we always generate IR for a block that contains the
THROW_HELPER. This happens before the stack frame is calculated, so the 4
outgoing stack slots are always calculated.

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

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

Port String.Equals from CoreRT

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

Avoid array allocation in "new MemoryStream()"

The default MemoryStream ctor is unnecessarily allocating a 0-length array.

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

Fixed build breaks

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

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

objcopy: Support cross-build for stripping symbols

Currently, the 'objcopy' does only support native-build since
the commit number 4efef62f0ead16b4f8ca2c6ccf0a77ce112f73b9
Let's add cross-build as well as native-build to strip symbols.

v3:
- support if statement for arm64

v2:
- Replace 'EQUAL' with 'DEFINED' for handling Other Linux distributions
  as well as Ubuntu 14.04 and for handling in case of native-build.

v2:
- support cross-build to strip debug symbols of native arm binaries

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

[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

Fix for stack overflow in CFG/DOM analysis GH#3600

Rewrite the DFS numbering of flow-graph and dominator tree
traversals to use a non-recursive formulation. hugeexpr1 now passes
(previously stack overflow) and the dominator tree traversal is now
around 10 seconds faster than before for this input set of 24,000 basic
blocks.

I have verified there are no asm diffs with this change.

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