Merge pull request #5958 from jashook/disable_timeouts_under_gcstress

Disable tests that timeout under gcstress

Fix undefined locals in desktop test builds. (#6073)

When for desktop out of the ProjectK branch, the previous change left unassigned member variables.  Moved the pragma to include them.

Merge pull request #6075 from kyulee1/settime

ARM64: Set timeout per scenario

ARM64: Set timeout per scenario

Merge pull request #6072 from wtgodbe/archive2

Archive the right artifacts in coverage build

Archive the right artifacts in coverage build

Merge pull request #6044 from briansull/split-lvIsMultiRegArgOrRet

Split the lvIsMultiiRegArgOrRet into two bits

Merge pull request #6041 from adityamandaleeka/fix_strip_logic

Fix platform logic in symbol stripping code

Merge pull request #6070 from wtgodbe/sign

Only sign non-arm64 binaries

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

Merge changes from TFS

Only sign non-arm64 binaries

Split the lvIsMultiRegArgOrRet into two bits

Currently for all targets that have MultiReg return LclVars we require that they not be struct promoted.
But on ARM64 we do allow for MultiReg #argument LclVars to be struct promoted.
Thus we have to disentagle this value into two distinct bool values to implement MultiReg returns for Arm64.

Disable tests that timeout under gcstress

After increasing the timeout time for gcstress, these tests continue to fail
due to timeouts. To get the gcstress legs green and keep them within a manageable
amount of time they will be disabled.

Have the jit capture the SuperPMI method index for use in correlating outputs from different runs.

Useful for matching SPMI outputs in cases where jit changes cause differential replay failures so lining up results side-by-side doesn't work.

If the jit is not hosted under SPMI, the index will report as zero.

[tfs-changeset: 1615278]

Fix building on desktop bug #234633. (#6045)

ifdef the references to System.Diagnostic.StackFrame.

Merge pull request #6054 from kyulee1/gcjob

ARM64: Enable a gcstress private job

Merge pull request #6040 from rahku/arm64retbuf

ARM64: update tests.lst

Merge pull request #6000 from gkhanna79/Arm64Nuget

Generate nuget packages for Arm64 build

ARM64: Enable a gcstress private job

Merge pull request #6049 from dotnet-bot/UpdateDependencies20160629051946

Updating External dependencies to beta-24229-00

Merge pull request #6053 from kyulee1/ilrtrel

Relase build only for ilasm round trip tests in CI

Relase build only for ilasm round trip tests in CI

This eliminates spurious job display in Jenkins web which is unnecessary.

Merge pull request #5950 from wateret/fix-5949

[ARM/Linux] Disable unittest `_il_reltypeEqualOp`

Remove unittests that contain undefined behaviour

Following tests contain undefined behaviour.
(Casting double to an integer type for a value that cannot be represent as the integer type.)

- JIT.Methodical.casts.ilseq._il_reltypeEqualOp._il_reltypeEqualOp
- JIT.Regression.CLR-x86-JIT.V1-M10.b08107

Updating External dependencies to beta-24229-00

Merge pull request #6043 from BruceForstall/DisplayPlatformInJitDump

Display the build platform and architecture in the JitDump

Display the build platform and architecture in the JitDump

Merge pull request #6038 from BruceForstall/PinvokeFeedbackCleanup

Address P/Invoke inlining code review feedback and cleanup

Merge pull request #6042 from wtgodbe/archive

Archive coverage reports from the right place

Archive coverage reports from the right place

Merge pull request #6035 from briansull/fix-5642

Fix issue #5642

Merge pull request #6036 from briansull/dump-fix-inlining

Do not enable the verbose Jit Dump during the inlining decision phase.

Fix path to BOTR GC section in gc.cpp (#6039)

Fix platform logic in symbol stripping code.

update arm64 tests.lst

Address P/Invoke inlining code review feedback and cleanup

Address some code review feedback from #5939. Also, do a little cleanup.
Specifically:
1. Make many GenTreeCall accessors 'const'.
2. HasNonStandardArgs() and GetNonStandardArgCount() are used for a specific purpose in the fast
tailcall implementation. However, the fgMorphArgs() "non-standard args" mechanism is now used for
other purposes, so the names of these didn't match their intent. Also, the logic had decayed compared
to the fgMorphArgs logic it was mimicing. I renamed them to add the word "Added", and wrote some
comments to be more explicit about their use, as well as indicate in fgMorphArgs() that they need
to be kept in sync.
3. Added several GenTreeCall::IsHelperCall() accessors, and replace some code with them.
4. Added RBM_INIT_PINVOKE_FRAME_TRASH define to remove an `#ifdef`.
5. Removed an assert loop in LowerFastTailCall() since it's no longer accurate given the new users of `isNonStandard`.

Merge pull request #5978 from rahku/arm64retbuf

Fix Pinvoke IL Stubs to emit correct return type in signature of call…

Fix Pinvoke IL Stubs to emit correct return type in signature of calli instruction

Do not enable the verbose Jit Dump during the inlining decision phase.

Merge pull request #6030 from kyulee1/config

Fix default optimization setting for System.Private.CoreLib

Fix issue #5642

Only consider hfa types when multireg return is enabled for the target
Update Test.Lst with new passing test

Fix Windows context to Unix context translation on AMD64 (#6027)

There was a bug in the context translation between the Windows context
and Unix context on AMD64 caused by the fact that the Unix context
gregs array contains CS, GS and FS in a single field (REG_CSGSFS) and
the MCREG_SegCs accessor macro was incorrectly written to use the
whole field as CS. So writing the CS into the Unix context also
cleared the GS, FS and the topmost 16 bits described as padding.
This issue was exposed on the Linux kernel >= 4.6.0 where the padding
for some reason was not zero, probably used by the kernel for some
internal purposes.
I have fixed it by changing the accessor to modify only the 16 bits
corresponding to the CS.
I have also changed the code in the inject_activation_handler to
save cycles and not to copy the Windows context back to the Unix one
in case the activation function was not called and so the context
was not possibly changed.

Merge pull request #6032 from wtgodbe/coverFix

Build coreclr as part of flow job in coverage jobs

Build coreclr as part of flow job in coverage jobs

Fix default optimization setting for System.Private.CoreLib

This disable optimization for System.Private.CoreLib under debug build.

Package native Arm64 crossgen as well.

Merge pull request #6025 from dotnet-bot/UpdateDependencies20160628051939

Updating External dependencies to beta-24228-00

Fix 'unicode/uchar.h' file not found (#6009)

The latest version of Homebrew does not set up the include path for `icu4c` so we have to force the creation of symbolic links like this is done for OpenSSL.

Move intermediate file into intermediate dir (#5981)

The intermediate file, cmake.definitions, is moved into
the intermediate directory: bin/obj/OS.Arch.Conf/

The script won't emit error for not having the cmake
results because some systems (Windows) do not use Cmake
and users might use "skipnative".

Fixes #5976

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

Updating External dependencies to beta-24228-00

Merge pull request #6019 from swgillespie/gcsimulator-timeout

Remove one GCSimulator tests that takes way too long

Merge pull request #6017 from pgavlin/SIMDBugFixes

Fix three SIMD-related bugs.

Merge pull request #5992 from prajwal-aithal/devel/arm-ci-script-improv

ARM-CI: Improve readability of the ARM CI script

Remove one GCSimulator tests that takes way too long

Address PR feedback.

- Add comments regarding the legality of a null type handle in
  `gtNewCpObjNode`
- Fix a CC issue in `gtNewTempAssign`

Fix three SIMD-related bugs.

The first two bugs involve SIMD nodes that are created by the JIT
for intermediate SIMD operations. These interediate operations
are not directly represented in metadata) and may therefore use
SIMD types that are not present in metadata. For example, metadata
may only use Vector<T>, but the IR may represent some intermediate
operations using the natural vector type for the target machine.
In these cases, the JIT will not be able to derive a type handle
for the SIMD type, which was causing a couple of issues:
- `gtNewTempAssign` was generating a `GT_ASG` node instead of a
  `GT_COPYBLK` when given trees involving these operations and
  types
- `gtGetStructHandleIfPresent` was returning an invalid type handle
  when dealing with SIMD-typed `GT_IND` nodes

The fix for the former is to check for SIMD-typed trees in
`gtNewTempAssign` and `gtNewCpObjNode`; the fix for the latter is
to check for SIMD-typed trees before checking for the array index
flag on a `GT_IND` in `gtGetStructHandleIfPresent`.

The third bug is that `gtNewTempAssign` was not setting the
`lvSIMDType` flag on local vars after changing their type to a SIMD
type.

Merge pull request #6002 from pgavlin/GenTreeDebugFlags

Refactor debug-only GenTree flags.

Merge pull request #5971 from BruceForstall/RefactorTreeNodeInit

Refactor TreeNodeInfoInit(): extract lots of functions

Add `GTF_DEBUG_NONE`.

This is a marker that is used to indicate the absence of debug GenTree
flags.

Merge pull request #6007 from dotnet-bot/UpdateDependencies20160627051740

Updating External dependencies to beta-24227-00

Updating External dependencies to beta-24227-00

Rename files under src/mscorlib/corefx/System/Globalization to match CoreRT (#6005)

corefx/System/Globalization was forked from CoreRT originally. Rename the files to match the current CoreRT names.

ARM-CI: Improve readability of the ARM CI script

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

Merge pull request #6003 from Clockwork-Muse/master

Add missing parameter names to BitArray.CopyTo exceptions

Copy gtDebugFlags where necessary.

This also renames `GTF_DEBUG_MORPHED` to `GTF_DEBUG_NODE_MORPHED` for
consitency with the other node-specific debug flags.

Add parameter names to exception.

Switch from literal parameter names to nameof

Refactor debug-only GenTree flags.

Move the four debug-only GenTree flags into their own field. This frees
up two bits across both the RyuJIT and the legacy backend.

Change to Generate nuget packages for Windows Arm64 builds.

Merge pull request #5993 from briansull/fix-5639

Fix for issue 5639

Merge pull request #5998 from briansull/Update-ReturnTypeDesc

Incremental work on ReturnTypeDesc for multiReg return support

Merge pull request #5985 from ericstj/incrementCoreCLR+Jit

Update CoreCLR & JIT package versions

Merge pull request #5979 from kyulee1/boolfold

ARM64: Fix Fold Type For optOptimizeBools

Use libc.so as default libc name (#5990)

Based on https://github.com/dotnet/coreclr/pull/5727#discussion_r68369439,
it turned out `/lib/libc.musl-x86_64.so.1` was just a symlink in
Alpine Linux to actual lib `/lib/ld-musl-x86_64.so.1`.

This delta makes libc.so as default name for libc and removes all the
detection for musl and FreeBSD from cmake etc.

Changes to ReturnTypeDesc support more than two return registers

Changed m_regType into an array
Renamed the Initialize method to InitializeReturnType

Fix for issue 5639

Added check for hfa type
Update Test.lst removing the appropriate R2R_FAIL tags

Update supported profiler APIs

ARM64: Fix Small Operation For optOptimizeBools

Fixes https://github.com/dotnet/coreclr/issues/5955

When merging two boolean operations (cmp/jmp) in optOptimizeBools, JIT
creates type based on operand type.
There is a case the operand type is bool (1 byte) so JIT creates a tree
that does 1 byte operation which arm64 cannot encode.
The fix is to bail out such case for arm in general.

This passes all 1375 tests in System.Net.Http.Unit.Tests.

Merge pull request #5991 from kyulee1/stacklimit

ARM64: Fix StackLimit

Merge pull request #5973 from cmckinsey/DevDiv_216571

Fix SBCG in value-numbering constant assertprop

Merge pull request #5940 from briansull/retbuf-nonstandard

ARM64: Change the fixed return buffer argument to be handled as a non-standard argument

Fix SBCG in value-numbering constant assertprop

Value-numbering can assign the same value number to nodes
of different type as long as they have the same bit-representation.
However when substituting a constant into a tree node of different
type, we want the bit-exact representation and not the same value.
Assertion prop should reinterpret int/float value changes rather
than coercing them.

Add an il test that exposes a bug in assertionprop when doing
value-numbering based constant propagation. A cpblk can be used
to copy a scalar value between differing types but for which there
is no change in bit representation.

ARM64: Fix StackLimit

Fixes https://github.com/dotnet/coreclr/issues/5640
We often got AV for tests that do a large stack allocations.
The fix is to match stack limt setup same as x64.

Merge pull request #5986 from dotnet-bot/UpdateDependencies20160624051958

Updating External dependencies to beta-24224-00

Fix typo to apply mac build workaround only on OSX (#5988)

/init-tools.sh:72 tries to compare strings like this:
if [ "$OS"=="OSX" ]; then

What it actually does is different to what it looks like:
$ echo $OS
$ if [ "$OS"=="OSX" ]; then echo EQUAL; fi
EQUAL
$ if [ "$OS" == "OSX" ]; then echo EQUAL; fi
$

This commit fixes the typo and makes OSX workaround run on macs only.

Fix Issue #5542 (#5827)

On arm32, NullReference exception for interface call is not handled
properly. Especially, when callsite is a virtualstub(dispatch/resolve), AV exception
occurs on indirect cell. Since this stub is not a part of
managed or markedjithelper, exception handler considers this is a native
seg fault.
I added :
(1) FaultingExceptionFrame construction routine for virtualstub
(2) Checking and IP-adjusting routine for VirtualStub

Use BIT64 define for 64-bit specific code. (#5987)

Using WIN32/WIN64 for 32/64-bit architecture dependent code confuses
developers especially those who develop coreclr for non-Windows
systems. Therefore, such definitions are modified to BIT64.

Fixes #4737

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

Updating External dependencies to beta-24224-00

Update CoreCLR & JIT package versions

Increment these versions to facilitate stable repack of 1.0.2.

Merge pull request #5983 from prajwal-aithal/devel/arm-ci-checked-fix

ARM-CI: Prevent creation of job for Checked configuration (rebased over #5961)

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

Merge changes from TFS

ARM-CI: Prevent creation of job for Checked configuration

Previously we checked if the configuration being built is either of Debug or Release
after the job had already been created. This resulted in a job being created for
Checked configuration with no commands to execute (for this job).
We prevent the unnecessary creation of job for the Checked configuration by
returning control at the very beginning of the <LinuxARMEmulator, arm, Checked> iteration

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

Merge pull request #5960 from pgavlin/CheckDefPython3

Make check-definitions compatible with Python 3.

Merge pull request #5923 from mikedn/x86-store-lclvar

Fix statement insertion point in GT_STORE_LCL_VAR decomposition

[SOS][Linux] Support of reading local variables from portable PDB (#5897)

* Initial support of reading local variables from portable pdb using System.Diagnostics.Debug.SymbolReader.dll

* Use SysAllocString and SysFreeString for memory management

* Fix coding style after review

Merge pull request #5961 from jashook/arm32_ci_ubuntu_cross_fix

Seperate the ubuntu-arm build and emulator build

Undo accidental change

[tfs-changeset: 1614521]

Add missing contracts

[tfs-changeset: 1614520]

Seperate the ubuntu-arm build and emulator build

The name for the arm32 ubuntu cross build and the arm32 emulator build are the same. This
also disables the ubuntu cross build per pr.

Merge pull request #5968 from RussKeldorph/jitstress

Move R2R JitStress jobs into the jitstress folder with the others.