Clean up JIT build rules

Add SIMD files to x86 build (they aren't used, yet).

Separate out the LEGACY_BACKEND files, to be explicit about them.

Make protojit and standalone build rules more similar.

Merge branch 'botaberg-port_interop_tests'

Merge branch 'port_interop_tests' of https://github.com/botaberg/coreclr into botaberg-port_interop_tests

Merge pull request #4164 from yizhang82/lpstr_test_fix

Fix LPSTR test failure due to incorrect implementation of strncpy_s.

Merge pull request #4160 from tijoytom/master

Windows specific Marshal APIs test.

Merge pull request #3642 from dagood/remove-hardcoded-versions

Remove hardcoded versions from runtests.proj

Merge pull request #4165 from myungjoo/trivialfix/doc1

Typo: incorrect pathname in documentation

Typo: incorrect pathname in documentation

build-rootfs.sh is at ./cross/, not ./ .

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

Fix LPSTR test failure due to incorrect implementation of strncpy_s. It was implemented using snprintf and didn't account for the null terminator behavioral difference. Test is now passing after the fix. We should see if we can always use xplat helpers in all platforms for easier debugging.

Merge pull request #4155 from MichalStrehovsky/debuggableAttribute

Remove mscorlib 1.x strong references from tests

Merge pull request #4161 from yizhang82/remove-marshal

Remove Marshal.Read/Write (AsAny object) overload from CoreCLR.

Merge pull request #4134 from MichalStrehovsky/junkAttributes

Delete junk attributes in tests

Remove Marshal.Read/Write (AsAny object) overload from CoreCLR. I don't expect anyone to use these overloads and it doesn't make sense to keep them. Their semantics are rather convoluted and most likely not what you might expect.

Windows specific Marshal APIs test.

Merge pull request #4159 from kyulee1/update3

ARM64: Update email notification3 -- testing

ARM64: Update email notification3 -- testing

Merge pull request #4125 from wtgodbe/errLog

Add error logging to 4 PAL tests that fail periodically on OSX

Merge pull request #4149 from wtgodbe/triggerPhrases

Add documentation for CI Trigger Phrases

Add documentation for CI Trigger Phrases

Merge pull request #4131 from dotnet/Petermarcu-patch-2

Updating Readme to point to .NET Core SDK

Merge pull request #4153 from kyulee1/update2

ARM64: Update email notification2

ARM64: Update email notification2

Only for arm64 group (both PR/Push) -- testing.

Merge pull request #4148 from AndyAyersMS/InlineOpcodeBins

Inliner: update DiscretionaryPolicy to look at opcodes and arg info

Remove mscorlib 1.x strong references from tests

Referencing mscorlib 1.x forces ILASM to emit a reference to the legacy
DebuggableAttribute constructor (the one that takes bools). That
constructor is not in the .NET Core profile and not all core base class
libraries will have it.

See src/ilasm/asmman.cpp#L389.

Fixing failures of CscBench, Serialization and Deserialization perf tests (#4151)

Merge pull request #4152 from kyulee1/updateemail

ARM64: Update email notification

Merge pull request #4037 from pgavlin/SimplerHashTable

Remove non-portable dependencies from SimplerHashTable.

Inliner: update DiscretionaryPolicy to look at opcodes and arg info

Have the `DiscretionaryPolicy` do some simple opcode histogramming
to try and provide better observations for code size estimation.

Likewise, look at the types and sizes of arguments and report those.

ARM64: Update email notification

Only sending email for Windows run when push occurs.
Skip sending email for private run (PR).

Merge pull request #4106 from janvorli/add-hosting-api-header

Add header file for the hosting API

Merge pull request #4144 from kyulee1/email

ARM64: Add email notification for failing job

Merge pull request #4146 from mmitche/fix-trigger-phrases

Trigger phrase currently has invalid regex

Trigger phrase currently has invalid regex

Should be double quoted for string interpolation

ARM64: Add email notification for failing job

Similar to roslyn job, added an email publisher to the job.
Fixed job title to reflect "Test" for Checked/Release case.
Refined triggered message to enable individual configuration.
So, instead of "test Windows_NT arm64" that runs all configurations,
just "test Windows_NT arm64 Checked" for instance.

Merge pull request #4139 from kyulee1/launcher

ARM64: Launch test build and run in parallel

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

ARM64: Launch test build and run in parallel

This change allows to buildtest.cmd for arm64 in separate.
Currently we build coreclr and tests for arm64 and then run tests in
sequence even though we run the pre-built test suite.
This change pushes down the test built part into the lab post script that
launches buildtests.cmd and arm64postbuild.cmd concurrently. The latter
part is mostly waiting for the results from arm64 box. So we can almost
hide the test build time which is around 30 min.

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

Delete junk attributes in tests

There is no such thing as
`Microsoft.VisualBasic.Globals$StandardModuleAttribute` and
`System.Security.SecurityRulesAttribute` in mscorlib. Presence of these
unresolvable references just trips up static compilers.

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

Updating Readme to point to .NET Core SDK

Updating the readme to point to the getting started page as well as the CLI repo for getting full stack builds of .NET Core and the SDK

Add error logging to 4 PAL tests that fail periodically on OSX

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

Disable MarshalStructAsLayoutSeq.sh outside of Windows

Avoid assert on buffer overflow

Fix a typo in a comment in SimplerHashTable.

Replace SimplerHash with SHash in the interpreter.

This removes the only usage of SimplerHashTable outside of the JIT
and the GC info encoder.

Add JIT and GC info SimplerHashTable behaviors.

These behaviors override the default out-of-memory handling
s.t. it is appropriate for the JIT.

Remove non-portable dependencies from SimplerHash.

- Remove contracts. Pre- and post-conditions have been replaced
  with asserts.
- Hang the "no memory" callback off of Behavior instead of
  calling ThrowOutOfMemory directly
- Remove operator new overloads that defer to the global
  operator new.

Merge pull request #4089 from tijoytom/master

Fixing tests for non-windows platform.

Add header file for the hosting API

Add the header file and also modify coreruncommon.cpp to use it.

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

Remove hardcoded versions from runtests.proj.

Put project.json for Xunit test wrappers in file system instead of generating. This lets dependency validation check the versions and reduces the number of project.json files to restore because all csproj's point to a single project.json and lockfile.

Adds XunitPackageVersion property that is used to always generate a valid project.json for xunit.runner.msbuild.

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