Merge pull request #4266 from janvorli/fix-brick-table-issue

Fix brick table issue on Unix

Merge pull request #4261 from yizhang82/byvalarray_fix

Fix ByValArray support.

ByValArray marshalling is implemented actually in OleVariant.cpp, which implements both regular array marshalling and variant marshalling). Regular array marshalling code should be supported in x-plat but it was incorrectly #ifdef-ed out. Fix by only #ifdef the COM interop VARIANT part.

Fix #4262 and #4194

Merge pull request #4269 from adiaaida/longSub

Add support for x86 long subtract operator

Merge pull request #4206 from BruceForstall/RyujitX86

Clean up JIT build rules

Merge pull request #4221 from BruceForstall/Cleanup1

Cleanup: be consistent about compCalleeFPRegsSavedMask #ifdefs

Merge pull request #4239 from BruceForstall/Cleanup2

Cleanup: remove getFramePointerReg()

Merge pull request #4242 from BruceForstall/Cleanup3

Add genEstablishFramePointer() to enapsulate frame pointer creation

Merge pull request #4243 from BruceForstall/Cleanup4

Minor JIT changes

Merge pull request #4268 from RussKeldorph/ignore

Add missing arch-specifc strings in .gitignore

Add support for x86 long subtract operator

With #3296, we have all the support for subtracting longs on x86. Subtract
can use the same logic as addition for longs.

Fixes #4189.

ByValArray marshaling support is implemented in OleVariant.cpp (which actually implements both regular array marshalling and variant marshalling) and is incorrectly #ifdef-ed out under x-plat. Fix by only #ifdef the COM interop VARIANT part.

Re-enable ByValArray test as well.

Add missing arch-specifc strings in .gitignore

I would prefer that the test build not drop cruft outside of designated
folders like bin, but unless/until that happens we should at least make
the architectures' builds consistent.

Merge pull request #4240 from LLITCHEV/Issue4147

Add test for issue 4115.

Merge pull request #4245 from AndyAyersMS/ModelPolicy

Inliner: initial version of a model-based policy

Fix brick table issue on Unix

There is a problem with Clang compiling a loop that sets a range of brick
table entries that are "shorts" as a memset. That causes it to be written
by bytes in some cases on OSX (and it is a mere chance that the memset on
Linux is implemented so that it doesn't use byte access).
This was causing corruption of the brick table on OSX.
The fix is to mark the pointer that iterates over the brick table as volatile
to prevent the compiler from using the memset.

Merge pull request #3990 from myungjoo/testing/fix3856

Fix Linux-ARM Broken Stack w/ ThePreStub

Merge pull request #4257 from jkotas/noexecstack

Pass noexecstack option to the assembler on Linux

Add a test for Issue 4147.

This checkin adds a test for Issue 4147.

Merge pull request #4250 from yizhang82/nativevariant_fix

Fix GetObjectForNativeVariant test issue. It should roundtrip the actual object instead of IntPtr. Also add proper variant clean up code.

Fix #4192.

Pass noexecstack option to the assembler on Linux

Merge pull request #4253 from jamesqo/faulty-trues

Fix a couple of erroneous assert(trues)

Merge pull request #4256 from yizhang82/chararray_fix

Change Marshal.SizeOf to sizeof to calculate buffer size because Marshal.Copy copies managed contents, while Marshal.SizeOf gives native size. Fix #4254

Change Marshal.SizeOf to sizeof because Marshal.Copy copies managed contents and does not do marshalling.

Merge pull request #4249 from yizhang82/lptstr_fix

Fix LPTSTRTest failure - it has a incorrect constant probably due to bad merge. Fix #4197

Fix GetObjectForNativeVariant test issue. It should roundtrip the actual object instead of IntPtr. Also add proper variant clean up code.

NOTE that I had to keep Marshal.IUnknownForObject call to force COM initialization.

Fix LPTSTRTest failure - it has a incorrect constant probably due to bad porting. Also remove StringMarshal tests from issues.targets

Fix a couple of erroneous assert(trues)

Merge pull request #4248 from dotnet/revert-4199-struct16-abi

Revert "ARM64 Work Item 3817, 3524 - Struct16 decomposition"

Revert "ARM64 Work Item 3817, 3524 - Struct16 decomposition"

Merge pull request #4162 from ramarag/ResolveWorkerChainLookupAsmStub

[Arm64]:Fixing the Logic of ResolveWorkerChainLookupAsmStub for VSD

Merge pull request #4202 from MichalStrehovsky/applicationException

Remove usage of ApplicationException in tests

[Arm64]:Fixing the Logic of ResolveWorkerChainLookupAsmStub

Inliner: initial version of a model-based policy

Add a model-based size predictor to the `DiscretionaryPolicy` and
dump its data along with observations. This can be used to assess
the performance of the model. Currently using a linear model that
can explain about 55% of the variability in code size impact for
mscorlib. Whether this holds up for other situations is still to
be determined.

Create a `ModelPolicy` that relies on this code size model to make
inlining decisions. This is very much a work in progress as the code
size models are still not well vetted and there is no real code
quality model yet.

Merge pull request #4207 from swgillespie/437657

Remove a test that doesn't provide much value, fails in parallel test…

Merge pull request #4199 from briansull/struct16-abi

ARM64 Work Item 3817, 3524 - Struct16 decomposition

Minor JIT changes

Add a useful assert about SP to FP offsets.

Add "const" to a few member functions.

Add genEstablishFramePointer() to enapsulate frame pointer creation

Cleanup: replace getFramePointerReg() with preexisting genFramePointerReg()

Cleanup: be consistent about compCalleeFPRegsSavedMask #ifdefs

Merge pull request #4205 from mikem8361/sosfx2

Better sos message for runtime not initialized

Merge pull request #4163 from pgavlin/GCInfoQSort

Replace CQuickSort with qsort in GC info encoders.

ARM64 Work Item 3817, 3524 - Struct16 decomposition

Changes to support passing of MultiReg structs using GT_LISTs
Optional support for  struct promotion for multireg structs:
To enable set  FEATURE_MULTIREG_STRUCT_PROMOTE to 1
Morphs the 16-byte structs at the end of fgMorphArgs
Careful refactoring to avoid changes to UNIX_AMD64 code
Covers all of the 16-byte struct expansion cases in fgMorph
Added function header comments
Passing the tests for Arm64
No AsmDiffs for non-Arm64 targets.
Codegen uses Contained nodes for PUTARG_STK 16-byte stack args
Created a genPutArgStk method for Arm64

Remove a test that doesn't provide much value, fails in parallel test environments, and p/invokes to kernel32.dll

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

Merge changes from TFS

Merge pull request #4203 from mmitche/remove-unused

Remove unecessary jobs

Better sos message for runtime not initialized

Also fixed the corerun sos command checked build problem.

Remove unecessary jobs

The jobs that involve multiple build stages (like most things xplat) always use the native bits from the default scenario.  Remove builds that are unused

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.

Remove usage of ApplicationException in tests

ApplicationException is not in the .NET Core profile. Most of these
tests only use it to crash the test with an unhandled exception and it
doesn't matter what exception type it is.

Add missing .gitmirrorall file

[tfs-changeset: 1594511]

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.

Replace CQuickSort with qsort in GC info encoders.

This removes another utilcode dependency from the GC info encoders.

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…