Merge pull request #4274 from davmason/master

Bug Fixes for EventSource

Merge pull request #4101 from sejongoh/detect_ldsfld_access_non_static_field

Check if LDSFLD accesses non-static field

Merge pull request #4277 from mikem8361/rpath

Unable to debug coreclr apps on Debian 8.2 (Issue #4263)

Unable to debug coreclr apps on Debian 8.2 (Issue #4263)

Add set(CMAKE_BUILD_WITH_INSTALL_RPATH TRUE) to dbi and sos cmake files.

Merge pull request #4218 from mikem8361/symbols

Add symbol packages for coreclr runtime.

Merge pull request #4272 from AndyAyersMS/FixSpectralNorm

Fix validation of spectralnorm under xunit-performance

Merge pull request #4113 from swgillespie/long_running_gc

Create Long-running GC test job for the CI

Buffer sending of large manifests

Merge pull request #4157 from chuckries/master

Fix error return in getPath

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

Add symbol packages for coreclr runtime.

Depends on the new buildtools symbol support.

Update build tools version containing the new symbol file support.

Merge remote-tracking branch 'upstream/master'

Fix validation of spectralnorm under xunit-performance

The valiation code wasn't correctly for the iteration strategy
used by xunit-performance.

Closes #4270.

Fix bug where Tags weren't respected in tracelogging.

Increase max etw arguments to 64

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

Add longgc to a new assert

Add new trigger phrases to documentation

Create Long-running GC test job for the CI

Fix JIT crash with static function accessing non-static field

The IL below is invalid since the static function acceses the non-static
field. On Desktop CRL, JIT throws a verification exception. However, CoreCLR
doesn't support verification and JIT crashes. This fix makes JIT throw a
invalid program exception if verification is disabled.

.field int32 i
.method static int32 f()
{
ldsfld int32 Test::i
ret
}

The test also expects InvalidProgramException instead of
VerificationException.

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.

Merge remote-tracking branch 'upstream/master'

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 remote-tracking branch 'upstream/master'

Merge pull request #4205 from mikem8361/sosfx2

Better sos message for runtime not initialized

Merge remote-tracking branch 'upstream/master'

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

Fix error return in getPath

The call to PathCharString.Set returns true on succes, so this code path
was returning false if it succeeded in setting lpPathFileName to
lpFileNameString. It looks like this is just a holdover from the way
this code used to be written.

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.