Merge pull request #5871 from mikedn/modopt

Improve div/mod by const power of 2

Merge pull request #5925 from RussKeldorph/j32_issues

Fix syntax error in x86_jit32_issues.targets

Merge pull request #5904 from pgavlin/WalkTreePostExecutionOrder

Obey GTF_REVERSE_OPS in fgWalkTreePost.

Merge pull request #5905 from sejongoh/group_jitstress_jobs

Move all JIT stress jobs into a sub directory on CI

Fix syntax error in x86_jit32_issues.targets

Merge pull request #5924 from wtgodbe/groovalicious

Run all tests under server GC during code coverage job

Run all tests under server GC

Merge pull request #5921 from JohnChen0/master

Disable a test case till issue #5501 is fixed

[SOS] [Linux] Fix incorrect symbol loading from portable pdb (#5896)

* Fix incorrect symbol loading from portable pdb

* Change return value to S_OK

Generate symbols packages for Windows CoreCLR binaries. (#5902)

* Generate symbols packages for Windows CoreCLR binaries.

Issue #5832.

* Add dll/exe back to symbol package.

Merge pull request #5919 from gkhanna79/FixCOC

Add Code of Conduct details to README.md

Disable a test case till issue #5501 is fixed

Optimize integer div/mod by const power of 2 in lowering

Optimizing GT_DIV/GT_UDIV/GT_MOD/GT_UMOD by power of 2 in codegen is problematic because the xarch DIV instruction has special register requirements. By the time codegen decides to perform the optimization the rax and rdx registers have been already allocated by LSRA even though they're not always needed (as it happens in the case of unsigned division where CDQ isn't used).

Since the JIT can't represent a CDQ instruction in its IR an arithmetic shift (GT_RSH) has been instead to extract the dividend sign. xarch's SAR is larger than CDQ but it has the advantage that it doesn't require specific registers. Also, arithmetic shifts are available on architectures other than xarch.

Example: method "static int foo(int x) => x / 8;" is now compiled to
mov      eax, ecx
mov      edx, eax
sar      edx, 31
and      edx, 7
add      edx, eax
mov      eax, edx
sar      eax, 3

instead of
mov      eax, ecx
cdq
and      edx, 7
add      eax, edx
sar      eax, 3

As a side-effect of this change the optimization now also works when the divisor is too large to be contained. Previously this wasn't possible because the divisor constant needed to be modified during codegen but the constant was already loaded into a register.

Example: method "static ulong foo(ulong x) => x / 4294967296;" is now compiled to
mov      rax, rcx
shr      rax, 32

whereas before a DIV instruction was used.

This change also fixes an issue in fgShouldUseMagicNumberDivide. The optimization that is done in lower can handle negative power of 2 divisors but fgShouldUseMagicNumberDivide handled those cases because it didn't check the absolute value of the divisor.

Example: method "static int foo(int x) => return x / -2;" is now compiled to

mov      eax, ecx
mov      edx, eax
shr      edx, 31
add      edx, eax
sar      edx, 1
mov      eax, edx
neg      eax

instead of
mov      eax, 0x7FFFFFFF
imul     edx:eax, ecx
mov      eax, edx
sub      eax, ecx
mov      edx, eax
shr      edx, 31
add      eax, edx

Add tests for integer div/mod by const

Add Code of Conduct details to README.md

fix build break on ARM caused by f1889 (#5914)

Merge pull request #5913 from dotnet-bot/UpdateDependencies20160622051611

Updating External dependencies to beta-24222-00

Cleanup hardware exception handling (#5916)

Currently, we were checking for the exception address being in a marked jit
helper and unwinding to the caller context if it was there at two places.
Once in the HandleHardwareException to get context for the faulting exception
frame and once in DispatchManagedException to get the first managed frame.
This change modifies the code in HandleHardwareException to update the context
record in the exception itself so that we can leave out the same change in
the DispatchManagedException.
I have also added updating of the exception address in the exception record
so that the exception address is set to be in the managed caller of the helper.

Updating External dependencies to beta-24222-00

Merge pull request #5908 from briansull/Fix-Remorph-Call

Fix for Remorph of a call with nonstandard args

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

Merge changes from TFS

Fix for Remorph of a call with nonstandard args

Fixing build break on x86 and arm.

[tfs-changeset: 1614031]

Merge pull request #3097 from vors/docx

Convert SOSAndICorDebug.docx to SOSAndICorDebug.md

Merge pull request #5900 from jashook/gc_stress_incompat

Set b14364 as gcstress incompat

Merge pull request #5898 from jashook/gcstress_timeout_update

Up the default timeout for GCStress

Merge pull request #5907 from Maoni0/stress_txt

updated stress run description

added a note about the reason why we ignore failures from tests in the stress run infrastructure

Move all JIT stress jobs into a sub directory on CI

Merge pull request #5903 from wtgodbe/groovester

Archive entire obj directory in coverage jobs

Archive entire obj directory

Merge pull request #5894 from Maoni0/seg_table

We need to have seg_mapping_table aligned on its natural alignement

Merge pull request #5899 from dotnet-bot/UpdateDependencies20160621051655

Updating External dependencies to beta-24221-00

Merge pull request #5892 from sejongoh/group_jitstress_jobs

Initial changes to move JIT stress jobs into jitstress directory on CI

Generic dictionary minor performance improvement and simplification for R2R (#5690)

A set of refactoring changes to slighly improve the performance of generic dictionary access on R2R images, and simplifying the codebase:
    1) Removing dependency on CEEInfo::ComputeRuntimeLookupForSharedGenericTokenStatic (and deleting the API).
    2) Stop parsing the generic type/method signatures when generating the R2R dictionary lookup assembly stub.
    3) Avoid re-encoding the generic type/method signatures in a new in-memory blob using a SigBuilder  (signatures used directly from the R2R image)
    4) Moved the parsing/loading of type/method signatures to Dictionary::PopulateEntry()
    5) Dictionary index and slot number are now encoded in the generated assembly stub instead of the signature (stub takes a pointer to a data blob, which contains the signature, the dictionary index and slot, and the module pointer)

Merge pull request #5895 from briansull/UnboxingStub-Fix

Fix for generic unboxing stub with x8 return buffer argument

Merge pull request #5813 from mikedn/no-setflags

Remove useless gtSetFlags calls

Obey GTF_REVERSE_OPS in fgWalkTreePost.

This particular sort of tree walk should correspond to the
tree execution order.

Merge pull request #5885 from brianrob/docker_perf

Update Linux Tracing Instructions for Docker Containers

Set b14364 as gcstress incompat

This test is set to be excluded to in interal testing under gc stress.
Add that information here as well.

Updating External dependencies to beta-24221-00

Up the default timeout for GCStress

Merge pull request #5887 from gkhanna79/Fix5883

Update Prerelease version to Beta and increment patch version.

Fix for generic unboxing stub with x8 return buffer argument

When we have nonstandard args we have to handle them before
we setup to expect the return buffer arg.

Added additional JitDump information about the call and its return type in impImportCall.

All of the ARM64 EXPECTED_PASS tests will now pass when hasFixedRerBufReg is enabled.
Additional changes are needed for the IL_STUB_PInvoke to get the Interop tests to pass
But these tests are currentlt alreay in the EXPECTED_FAIL catagory.

Update Prerelease version to Beta and increment patch version.

Merge pull request #5893 from rahku/test

Enable building only native test components

Merge pull request #4983 from benaadams/patch-2

Add slack channel

Merge pull request #3074 from Dmitry-Me/useElseIfForExclusive

Use else-if for mutually exclusive conditions

We need to have seg_mapping_table aligned on its natural alignement (ptr size). When it's not aligned,
aside from the perf problem, we also have a functional problem when an address that's not on a heap
segment is passed - we could read an intermediate value which would cause an AV. If the address is
on a heap segment it means we are guaranteed to read a ptr size atomically. This only affects functions
like IsHeapPointer which can be run when EE is not suspended.

Enable building only native test components

Merge pull request #5889 from wtgodbe/Loader

Fix race condition in Loader regression 523654 test

Merge pull request #5879 from mattwarren/patch-1

Added links to CoreCLR issues, small formatting fix

Merge pull request #5888 from adityamandaleeka/generate_core_dumps_runtest

Set coredump_filter to include shared library code in CI core dumps

Initial changes to group JIT stress jobs into a sub directory on CI

Add jitstress2_jitstressregs1 scenario first and then will add others
after testing.

Fix race condition in Loader regression 523654 test

Merge pull request #5886 from swgillespie/longgc-netci

Switch to double-quotes for interpolated string in netci.groovy

Set coredump_filter to include shared library code.

Update Linux tracing instructions for Docker containers.

Switch to double-quotes for interpolated string

Merge pull request #5882 from stephentoub/fix_ev_bytearray

Fix EventSource deserializing of byte[]s

Merge pull request #5875 from pgavlin/JitStdoutVbuf

Disable buffering on jitstdout.

Fix EventSource deserializing of byte[]s

It's incorrectly repeating the first element for all elements.

Merge pull request #5876 from dotnet-bot/UpdateDependencies20160620061718

Updating External dependencies to rc4-24220-00

Added links to CoreCLR issues, small formatting fix

Fix GC background thread start in OOM (#5840)

This change fixes a problem that happened when the `gc_heap::create_bgc_thread`
calls SetupUnstartedThread and it fails to allocate a Thread object and throws.
The GC doesn't expect that and so when the stack is unwound, the `gc_heap::gc_started`
stays set. Now we try to allocate memory for the Throwable for the exception, so we go
to GC heap and since there is not enough space, we end up calling
`gc_heap::try_allocate_more_space`, which calls `gc_heap::wait_for_gc_done`.
And that’s the end of it, since the `gc_started` is still set and we wait forever.

The fix is to catch exception from the SetupUnstartedThread. I have also fixed couple of
places where this method was being called and the exception was not expected.

As an additional cleanup, I have moved the thread creation in GC to the GCToEEInterface.

Updating External dependencies to rc4-24220-00

Disable buffering on jitstdout.

This avoid the need to call fflush() throughout the code.

Merge pull request #5380 from sejongoh/x86_decompose_storeind

Decomposing 64-bit storeind instruction to 32-bit storeind instructions

Decomposing 64-bit storeind instruction to 32-bit storeind instructions

Fix source/line info in unhandled exceptions (#5874)

* Fix no source/line info for unhandled exceptions.

Issue #5828.

* Remove comment.

Remove the corner cases from sscanf test (#5873)

Update GC from CoreRT (#5859)

https://github.com/dotnet/corert/tree/master/src/Native/gc 53276850bc2e7e531908434b2d062fcff3e1d58d

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

Merge changes from TFS

Merge pull request #5865 from kyulee1/fixgcstruct

ARM64: Fix GC hole for Multi-Reg Struct

Merge pull request #5863 from adityamandaleeka/generate_core_dumps_runtest

Add option to runtest.sh to enable limited dump generation.

Merge pull request #5868 from AndyAyersMS/BooleansRule

JIT: standardize on bool for return type of GenTree OperIs methods

Merge pull request #5862 from adityamandaleeka/remove_dead_code

Remove some dead code in notifyexternals

Merge pull request #5831 from briansull/fix-5759

Fix for issue 5795

JIT: standardize on bool for return type of GenTree OperIs methods

We had a mixture of bool and int return types. Standardize on bool.
Otherwise callers must match the mixed typing to avoid the fatal
C4800 warning on windows.

Remove dead code in notifyexternals

Enable core dump generation in CI test jobs

ARM64: Fix GC hole for Multi-Reg Struct

Fixes https://github.com/dotnet/coreclr/issues/5817
When we pass struct in multi-reg, we dropped the GC info on creating
instruction.
The fix is that for GT_LCL_FLD, we enforce non-float type size while
holding the gc type.

Validate with small cutdown. And also confirmed that the gc range is
reasonable in System.Reflection.RuntimeParameterInfo.GetParameters.

Merge pull request #5836 from RussKeldorph/nogcsosx

Disable GCStress=C periodic triggers for OSX

Merge pull request #5856 from AndyAyersMS/ConstantObservations

JIT: Cleanup code in and around fgFindJumpTargets

Add option to runtest.sh to enable limited dump generation.

Fix for issue 5795

We incorrectly pass the ret buf arg in x9 instead of x8

It is required that we call SortArgs and EvalArgsToTemps whenever we have any register arguments
We were not doing that when we only had a retBufArg argument in x8.

Fix for UNIX_AMD64 structs

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

Merge changes from TFS

JIT: Cleanup code in and around fgFindJumpTargets

Reorder code to remove gotos. Extract two large chunks of logic as
subroutines. Add braces and update comments. Bring `fgStack` class up
to current Jit coding conventions.  Reunite some orphaned comments for
fgFindJumpTargets.

No behavior change expected. No diffs seen via jit-diff tool. No diffs
seen in desktop testing.

Merge pull request #5839 from swgillespie/gc-batch-fix

Fix GC test batch script skip conditions that were behaving unexpectedly

Fix full framework build break

[tfs-changeset: 1613359]

Merge pull request #5852 from gkhanna79/Fix5837

Correctly resolve binding from Resolving event for static load

Changes to update TFS packages post 1.0

[tfs-changeset: 1613358]

Add missing check to allow assembly bound using the Resolving event to be used during static binding scenario

Merge pull request #5843 from dotnet-bot/UpdateDependencies20160617052045

Updating External dependencies to rc4-24217-00

Use LEAF_END_MARKED instead of LEAF_END for ARM64 debugbreak.S (#5850)

This commit applies the changes in #5847 (which fixes #5845) into ARM64
stub.

Use Use LEAF_END_MARKED instead of LEAF_END for various functions (#5849)

This commit fixes #5848.

Use LEAF_END_MARKED instead of LEAF_END for DBG_DebugBreak (#5847)

This commit tries to fix #5845

Merge pull request #5842 from prajwal-aithal/devel/arm-ci-emul-fix

ARM-CI: Create mount path if not already present

Updating External dependencies to rc4-24217-00

ARM-CI: Create mount path if not already present

* Previous PRs assumed that the mount path is already created on the CI machine
* This results in errors if the same is not true

Signed-off-by: Prajwal A N <an.prajwal@samsung.com>