Merge pull request #4061 from jamesqo/use-roslyn

Remove UseRoslynCompiler variable from the build system

Merge pull request #4136 from yizhang82/castable_fix

Re-enable ICastable test now that it is passing on Mac

Merge pull request #4099 from ramarag/gcstackwalk

[ARM64] Fixes Updating Registers for managed frames in GC stack walks

Merge pull request #4137 from AndyAyersMS/RelModeILTests

Set DebugType to PdbOnly for rel-mode IL tests

Merge pull request #3874 from swaroop-sridhar/CDT

Add CoreDisTools header

Set DebugType to PdbOnly for rel-mode IL tests

Ensures that execution of these tests always invokes the jit's optimizer,
regardless of the build configuration.

See issue #4124.

Fixes Updating Registers for managed frames in stack walks

Re-enable ICastable test now that it is passing on Mac

Merge pull request #4111 from kyulee1/customlauncher

Restore CustomLauncher to Test Population

Merge pull request #4119 from tarekgh/FixAsyncCurrentCulture

 Make current culture travel with async operations on Linux

Merge pull request #4086 from swgillespie/gc-env-vars

Fix an issue where COREHOST_SERVER_GC and COREHOST_CONCURRENT_GC were…

Merge pull request #4120 from joperezr/FixInternalBuild

Fix Internal builds where ToolsDir doesn't exist

Fix Internal builds where ToolsDir doesn't exist

Merge pull request #4110 from kyulee1/testupdate

ARM64: Test.lst update with the latest lab run

Merge pull request #4117 from LLITCHEV/Issue4115

Fix for bad *nix optimization in CopyObj morphing.

ARM64: Test.lst update with the latest lab run

Fixes https://github.com/dotnet/coreclr/issues/4028
Enabled a few more PASS tests as well based on the lab results.

Restore CustomLauncher to Test Population

Fixes https://github.com/dotnet/coreclr/issues/4076.
Custom launcher is needed for CoreRT tests.

Merge pull request #4078 from joperezr/AddVersioningInOpenBuilds

Adding Common versioning to managed and Native components of coreclr

Merge pull request #3929 from papaslavik/arm_softfp

Arm softfp For Issue #3785

Adding Common versioning to managed and Native components of coreclr

Fix Current Culture to make it travel in async operations

In Windows code we'll already having the current culture travel in
the async operation. this feature was missing in Linux code.

Merge pull request #4112 from steveharter/GlobalizationBufferO

Avoid debug assert on buffer overflow when obtaining invalid locale

Enable execution of Perflab perf tests

Added additional dependencies to the project.json for perflab projects that should enable generation of an assemblylist with all the packages required to execute the tests on the end machine

Merge pull request #3913 from wtgodbe/groovy

Various changes to the CI

Various changes to the CI

Fix for bad *nix optimization in CopyObj morphing.

A bad optimization for*nix was introduced some time ago which is
incorrectly setting the type of a temp var created for spilling.
This cahne removes this optimization and uses the same code used for all
other OSs.

Merge pull request #4095 from BruceForstall/IgnoreBrowseVCdb2

Ignore browse.VC.db files from C/C++ extension for Visual Studio Code

Avoid assert on buffer overflow

Merge pull request #4089 from tijoytom/master

Fixing tests for non-windows platform.

Merge pull request #4057 from yizhang82/master

Fix NativeCallable test issue and make it work x-plat

Merge pull request #4092 from brianrob/function_symbols

Leave Function Name Symbols in Release Binaries For Linux

Merge pull request #4096 from kyulee1/xunit

Test Results Report to ARM64

Test Results Report to ARM64

Tell git to ignore browse.VC.db files from C/C++ extension for Visual Studio Code

Add CoreDisTools header

CoreCLR uses the CoreDisTools library for utilities like GCStress
and SuperPMI. To aid these utilities, this change adds the
CoreDisTools header to CoreCLR repository.

This cleans up duplication of type/enum definitions in CoreDisTools
and CoreCLR sources, and provides for a cleaner way to use the
interface.

The understanding here is that CoreCLR defines the required
CoreDisTools interface, and will publish the header into the
cmake-install directory.

Any repository can pick up this header and supply implementations
for the imported functions. [Currently the only implementation
is the one that uses LLVM disassembler in the LLILC tree].

Two other changes:
1) Define TypeDefs for exported function types in cordistools.h
   to avoid duplicating the signature in importing code.
2) Move to the libcoredistools version 1.0.1, and use the new
   interface in disassembler.cpp

Leave function name symbols in binaries when stripping out debug information.

Merge pull request #4019 from BruceForstall/Fix1977

Fix #1977: always create RBP chains on Unix

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

Fixing test for non-windows platforms.
Disable the failing scneario and file bug.

Fix an issue where COREHOST_SERVER_GC and COREHOST_CONCURRENT_GC were not respected by corerun

Fix #1977: always create RBP chains on Unix

The JIT will now always create RBP chains on Unix platforms. This includes in functions
the use localloc.

To do this, the VM is extended with a new Unix-only AMD64 unwind code: UWOP_SET_FPREG_LARGE.
The existing unwind code which is used to establish a frame pointer, UWOP_SET_FPREG, requires
that the frame pointer, when established, be no more than 240 bytes offset from the stack pointer.
This doesn't work well for frames that use localloc. (Large frames without localloc are ok,
because we don't report the frame pointer in the unwind info except for in functions with
localloc or EnC.)

The new unwind code has a 32-bit range. If used, UNWIND_INFO.FrameRegister
must be set to the frame pointer register, and UNWIND_INFO.FrameOffset must be set to 15
(its maximum value). This code is followed by two UNWIND_CODEs that are combined to form
its 32-bit offset. The high 4 bits must be zero. This offset is then scaled by 16. This
result is used as the FP register offset from SP at the time the frame pointer is established.

Merge pull request #4000 from rahku/issue2963

Fix issue 2963

Fix issue 2963

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

Merge changes from TFS

Merge pull request #4084 from leemgs/upstream-awk-ver-handling-simple-way

Handle 'mscorwks_unixexports.src' with better regexp operator for compatibility

Merge pull request #4077 from tijoytom/master

Disabling the object as variant in struct marshalling scenario

For Issue #3785 Arm Soft FP support: arguments and return values are passed in integer registers for arm-softfp architecture.

Handle mscorwks_unixexports.src with better regexp operator for compatibility

ver3:
According to the gawk manual, The '\S' operator (as shorthand for '[^[:space:]]')
matches any character that is not whitespace. However, '[ \t]' is better than
'\S' for Mac OSX.
(source - https://www.gnu.org/software/gawk/manual/html_node/GNU-Regexp-Operators.html )

ver2:
With ver1, the build-break is caused by different awk versions (by @jkotas).
(Revert PR - https://github.com/dotnet/coreclr/pull/4005)
To resolve the limitation of the ver1, @janvorli proposed that the
\S expression is better than [:space:] for the most compatible solution.

There are a lot of different AWK software (e.g. awk, gawk, mawk, nawk, etc)
and versions among the popular Linux distributions. This patch is a simply
way to handle 'mscorwks_unixexports.src' file with \S operator for more
compatibility amon the different awk versions.

ver1:
It's must be modified by gawk implementation that check it exactly
as a plain character (e.g., space, alnum) within the bracket expression
at ./coreclr/generateversionscript.awk. The [[:space:]] is more appropriate than
the [:space:]]. (PR - https://github.com/dotnet/coreclr/pull/3943)

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>

Add reference to CMakeLists.txt from NativeCallableTests.csproj correctly
Also remove NativeCallable from testsUnsupportedOutsideWindows.txt

Merge pull request #4081 from BruceForstall/FixLinuxJitHalt

Fix COMPlus_JitHalt on Unix by not calling _DbgBreakCheck()

Merge pull request #4080 from adityamandaleeka/fix_cmake_mismatch

Fix if/endif argument mismatch in CMake.

Fix NativeCallableTest to use custom DLL instead of EnumWindows Win32 API

Merge pull request #4069 from swgillespie/gc-test-fixups

Fix a number of tests that make invalid assumptions about object life…

Merge pull request #4072 from briansull/safe-refactor

Safe refactoring changes and code cleanup items

Let COMPlus_JitHalt work on Unix by not calling _DbgBreakCheck()

Safe refactoring changes and code cleanup items

Fixed the dumper so that GT_LCL_VAR and GT_LCL_FLD use a common code path
and we now  display (last use) correctly
Moved the checking for invalid GT_LISTs with an op1 arg of a GT_LIST from gtNewArgList to the constructor
and called IsListForMultiRegArg in the new assert
Renamed IsListofLclFlds to IsListForMultiRegArgs and added ARM64 support
Modifed the dump to display the GTF_MORPHED flag using a '+'
Fix dumper's gtGetLateArgMsg to display each register for a multireg struct passed using a GT_LIST
Modified the NYI_<target> macros to include the target in their message
Fixed the gtCosts for GT_LIST, GT_OBJ, GT_MKREFANY and GT_BOX nodes
When we construct a new GT_LCL_FLD, initialize the gtFieldSeq to NotAField rather than nullptr
Fix comment header for getStructGcPtrsFromOp
Removed the define on ARM64 that changed the asserts(p) macro to notYetImplemented
In NewPutArg renamed the "fp" argument to be called "info" instead

Fix if/endif argument mismatch in CMake.

Package ibcmerge.exe and optimization data for FxCore inside a Nuget package

[tfs-changeset: 1592767]

Disabling the object as variant in struct marshalling
since it fails outside of windows.

Fix a number of tests that make invalid assumptions about object lifetimes. Fixes #3391

Merge pull request #4071 from fadimounir/gentests

Add generics constraint test

Add generics test

Merge pull request #4042 from sivarv/refactor2

Refactor Unix structDesc of GenTreeCall node into MultiRegReturnTypeDesc

Merge pull request #4065 from jashook/smarty_parser

Add a smarty parser.

Merge pull request #4058 from sejongoh/fix_call_setup_runtime_dep

Update setup-runtime-dependencies.cmd to specify runtime explicitly

Refactor Unix structDesc of GenTreeCall node into MultiRegReturnTypeDesc.

Merge pull request #4053 from RussKeldorph/test32

Fix some of the tests failing on x86

Merge pull request #3958 from gkhanna79/FixR2RScripts

Fix to add error checks for crossgen compilation

Add a smarty parser.

This can be used in the future by the arm64 ci to report output.

Fix to add error checks for crossgen compilation

Merge pull request #4045 from LLITCHEV/Issue3601

Don't add nop padding for rejitting a funclet prolog.

Remove UseRoslynCompiler variable from the build system

Merge pull request #4060 from jkotas/remove-r2r-workaround

Remove ReadyToRun workaround

Update setup-runtime-dependencies.cmd to specify runtime explicitly

Remove ReadyToRun workaround

Merge pull request #4051 from sejongoh/fix_singlinkgen_test

Fix singlinkgen test.

Fix some of the tests failing on x86

Some tests were ported exclusively for x64 but originally had non-x64
variants, so I'm just completing the porting of those for x86.  For
dblarray4, it is supposed to run with an env var set.  The way this is
done is pretty ugly ("precommands"), and if we require more of this, we
should add a better way to do it to the test harness.

Fixes 3972
Fixes part of 2877

Enabling Helix perf test runs for CoreCLR

Enabling Helix perf test runs for CoreCLR

Merge pull request #4052 from yizhang82/master

Add mac ulimit -n 2048 workaround and more debug spew in CLI installation into init-tools.sh

Add mac ulimit -n 2048 workaround and more debug spew in CLI installation into init-tools.sh

Fix singlinkgen test.

If the address of SingLink is stored at stack, GC.Collect() cannot collect
the object since the address variable is still alive at stack.

Merge pull request #4004 from sejongoh/use_corefx_outerloop_test

Use corefx outerloop tests

Merge pull request #4047 from kyulee1/fixclean

Fix clean build that deletes Tools

Don't add nop padding for rejitting a funclet prolog.

This is along standing issue in RyuJit. When ngenning the crossgen tool
passes CORJIT_FLG_PROF_REJIT_NOPS flag. The padding for rejitting should
be added only for the main function and not the funclets when compiling a
method. Trying to generate a rejit padding for a funclet in debug and
checked build results in a failing assertion.

Fixes issue 3601.

Fix clean build that deletes Tools

Merge pull request #3961 from natgla/adams

Changing adams benchmark

Merge pull request #4038 from adityamandaleeka/extest

Disable ForeignThreadExceptions test on ARM64

Merge pull request #4031 from dagood/upgrade-cli

Upgrade dotnet cli to 1.0.0-beta-002173

Merge pull request #4041 from erozenfeld/gtTreeHasSideEffectsFix

Fix exponential explosion in gtTreeHasSideEffects.

Merge pull request #4039 from manu-silicon/open_delegates

Disable FEATURE_STUBS_AS_IL on ARM 32-bit

Merge pull request #4034 from gkhanna79/PortFix

Port workaround for AOT packaging

Disable FEATURE_STUBS_AS_IL on ARM 32-bit

This enables open delegates on ARM (an additional 43 tests are now passing)

Disable ForeignThreadExceptions test on ARM64.

Fix exponential explosion in gtTreeHasSideEffects.

gtTreeHasSideEffects has special code to deal with helper calls.
If the helper satisfies certain properties and its arguments don't have relevant
side effects, than the tree corresponding to the helper call is considered not to have
side effects. The bug was that both gtTreeHasSideEffects and gtNodeHasSideEffects
(called by gtTreeHasSideEffects and calling gtTreeHasSideEffects) had code to check
the helper arguments for side effects. That made the algorithm exponential (2^n)
for a tree like the one below.
Because of that ngen would take days to compile the method with such trees.
The fix is to remove the argument checking code from gtTreeHasSideEffects since it always
calls gtNodeHasSideEffects. Compilation time goes back to milliseconds or seconds.

I ran full desktop testing and verified no SuperPMI diffs.

No measurable impact on cqNGenTP.

Fixes #3768.

N139 (255,169)              [000064] --CXG-------             *  call help ref    HELPER.CORINFO_HELP_READYTORUN_CHKCAST $179
N137 (255,163)              [000062] --CXG------- arg0 in rcx \--*  call help ref    HELPER.CORINFO_HELP_READYTORUN_CHKCAST $177
N135 (255,157)              [000060] --CXG------- arg0 in rcx    \--*  call help ref    HELPER.CORINFO_HELP_READYTORUN_CHKCAST $175
N133 (255,151)              [000058] --CXG------- arg0 in rcx       \--*  call help ref    HELPER.CORINFO_HELP_READYTORUN_CHKCAST $173
N131 (255,145)              [000056] --CXG------- arg0 in rcx          \--*  call help ref    HELPER.CORINFO_HELP_READYTORUN_CHKCAST $171
N129 (255,139)              [000054] --CXG------- arg0 in rcx             \--*  call help ref    HELPER.CORINFO_HELP_READYTORUN_CHKCAST $16f
N127 (255,133)              [000052] --CXG------- arg0 in rcx                \--*  call help ref    HELPER.CORINFO_HELP_READYTORUN_CHKCAST $16d
N125 (255,127)              [000050] --CXG------- arg0 in rcx                   \--*  call help ref    HELPER.CORINFO_HELP_READYTORUN_CHKCAST $16b
N123 (255,121)              [000048] --CXG------- arg0 in rcx                      \--*  call help ref    HELPER.CORINFO_HELP_READYTORUN_CHKCAST $169
N121 (255,115)              [000046] --CXG------- arg0 in rcx                         \--*  call help ref    HELPER.CORINFO_HELP_READYTORUN_CHKCAST $167
N119 (253,109)              [000044] --CXG------- arg0 in rcx                            \--*  call help ref    HELPER.CORINFO_HELP_READYTORUN_CHKCAST $165
N117 (239,103)              [000042] --CXG------- arg0 in rcx                               \--*  call help ref    HELPER.CORINFO_HELP_READYTORUN_CHKCAST $163
N115 (225, 97)              [000040] --CXG------- arg0 in rcx                                  \--*  call help ref    HELPER.CORINFO_HELP_READYTORUN_CHKCAST $161
N113 (211, 91)              [000038] --CXG------- arg0 in rcx                                     \--*  call help ref    HELPER.CORINFO_HELP_READYTORUN_CHKCAST $15f
N111 (197, 85)              [000036] --CXG------- arg0 in rcx                                        \--*  call help ref    HELPER.CORINFO_HELP_READYTORUN_CHKCAST $15d
N109 (183, 79)              [000034] --CXG------- arg0 in rcx                                           \--*  call help ref    HELPER.CORINFO_HELP_READYTORUN_CHKCAST $15b
N107 (169, 73)              [000032] --CXG------- arg0 in rcx                                              \--*  call help ref    HELPER.CORINFO_HELP_READYTORUN_CHKCAST $159
N105 (155, 67)              [000030] --CXG------- arg0 in rcx                                                 \--*  call help ref    HELPER.CORINFO_HELP_READYTORUN_CHKCAST $157
N103 (141, 61)              [000028] --CXG------- arg0 in rcx                                                    \--*  call help ref    HELPER.CORINFO_HELP_READYTORUN_CHKCAST $155
N101 (127, 55)              [000026] --CXG------- arg0 in rcx                                                       \--*  call help ref    HELPER.CORINFO_HELP_READYTORUN_CHKCAST $153
N099 (113, 49)              [000024] --CXG------- arg0 in rcx                                                          \--*  call help ref    HELPER.CORINFO_HELP_READYTORUN_CHKCAST $151
N097 ( 99, 43)              [000022] --CXG------- arg0 in rcx                                                             \--*  call help ref    HELPER.CORINFO_HELP_READYTORUN_CHKCAST $14f
N095 ( 85, 37)              [000020] --CXG------- arg0 in rcx                                                                \--*  call help ref    HELPER.CORINFO_HELP_READYTORUN_CHKCAST $14d
N093 ( 71, 31)              [000018] --CXG------- arg0 in rcx                                                                   \--*  call help ref    HELPER.CORINFO_HELP_READYTORUN_CHKCAST $14b
N091 ( 57, 25)              [000016] --CXG------- arg0 in rcx                                                                      \--*  call help ref    HELPER.CORINFO_HELP_READYTORUN_CHKCAST $149
N089 ( 43, 19)              [000014] --CXG------- arg0 in rcx                                                                         \--*  call help ref    HELPER.CORINFO_HELP_READYTORUN_CHKCAST $147
N087 ( 29, 13)              [000012] --CXG------- arg0 in rcx                                                                            \--*  call help ref    HELPER.CORINFO_HELP_READYTORUN_CHKCAST $145
N085 ( 15,  7)              [000010] --CXG------- arg0 in rcx                                                                               \--*  call help ref    HELPER.CORINFO_HELP_READYTORUN_CHKCAST $143
N083 (  1,  1)              [000008] ------------ arg0 in rcx                                                                                  \--*  lclVar    ref    V00 tmp0         u:2 (last use) $140

Merge pull request #4035 from kyulee1/common

Update build script for arm64

Update build script for arm64

Port workaround for AOT packaging

Merge pull request #4007 from adityamandaleeka/extest

Add test for exception handling on foreign threads.

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