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)

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.

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.

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.

Merge pull request #5959 from pgavlin/AOTPlaceholderFile

Add AOT placeholder files to the JIT package.

Add braces to avoid operator priority confusion (#5964)

Support for updating a nonStandard arg in-place the ArrayStack when it is modified by fgMorphTree

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

Merge changes from TFS

Refactor TreeNodeInfoInit(): extract lots of functions

Add AOT placeholder files to the JIT package.

This is intended to prevent accidental inclusion in UWP projects.

Fix GC profiling on arm-softfp (#5952)

This commit updates one preprocessor directive inside gc.h to initialize
dwEtwRootKind field of ScanContext when GC_PROFILING is defined while
FEATURE_EVENT_TRACE is not. This happens to platforms like arm-softfp.

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

Merge pull request #5939 from BruceForstall/FixPinvoke

Fix RyuJIT/x86 P/Invoke inlining

Merge branch 'master' into retbuf-nonstandard

Fix build break in internal build

[tfs-changeset: 1614475]

Merge pull request #5935 from rahku/test

Fix test runner batch file to provide correct TPA path

Merge pull request #5936 from briansull/fullIntArgRegMask

Introduce the method fullIntArgRegMask() to replace many uses of RBM_ARG_REGS

Merge pull request #5937 from ellismg/pass-build-id-flag-to-linker

Pass --build-id=sha1 to linker explicitly

Merge pull request #5962 from dotnet/revert-5945-devel/arm-ci-checked-fix

Revert "ARM-CI: Prevent creation of job for Checked configuration"

Revert "ARM-CI: Prevent creation of job for Checked configuration"

Make check-definitions compatible with Python 3.

Import and use the Python 3-compatible print_function from __future__.

Enable RyuJIT/x86 PInvoke lowering

Fixes #4181 "NYI_X86: Implement PInvoke frame init inlining for x86"

The main work here is to handle the custom calling convention for the
x86 CORINFO_HELP_INIT_PINVOKE_FRAME helper call: it takes EDI as an argument,
trashes only EAX, and returns the TCB in ESI.

The code changes are as follows:
1. Lowering::InsertPInvokeMethodProlog(): don't pass the "secret stub param" for x86.
Also, don't store the InlinedCallFrame.m_pCallSiteSP in the prolog: for x86 this is done
at the call site, due to the floating stack pointer.
2. LinearScan::getKillSetForNode(): for helper calls, call compHelperCallKillSet() to get the killMask,
to account for non-standard kill sets.
3. Morph.cpp::fgMorphArgs(): set non-standard arguments for CORINFO_HELP_INIT_PINVOKE_FRAME.
4. compHelperCallKillSet(): set the correct kill set for CORINFO_HELP_INIT_PINVOKE_FRAME.
5. codegenxarch.cpp::genCallInstruction(): set the ABI return register for CORINFO_HELP_INIT_PINVOKE_FRAME.
6. lowerxarch.cpp::TreeNodeInfoInit(): set the GT_CALL dstCandidates for CORINFO_HELP_INIT_PINVOKE_FRAME.

5 & 6 are both needed to avoid a copy.

With this change, the #1 NYI with 18415 hits over the tests is gone.
The total number of NYI is now 29516.

Enable out-of-proc unwinding on ARM (#5946)

Fix #5855

Merge pull request #5890 from adiaaida/x86LongCallFinal

Enable GT_CALL with long ret types for x86 RyuJIT

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

ARM-CI: Prevent creation of job for Checked configuration

Merge pull request #5944 from dotnet-bot/UpdateDependencies20160623052003

Updating External dependencies to beta-24223-00

Scripts: verify compiler definitions of native and managed (#4675)

* Scripts: find out compiler definitions of CMake

In order to find mismatch between native and managed,
we need to know the list of definitions of native.

The copmiler definitions are stored at cmake.definitions

This addresses the complaints of #4674

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>
* Scripts: add check-definitions.py

scripts/check-definitions.py checks the consistency between
the native-build (CMake) compiler definitions and
the managed-build (MSBuild/mscorlib) compiler definitions
at build-time and prints out potentially dangerous
inconsistencies.

In order to get the proper results, managed build should
be executed after the native build (build.sh will do so
if no options such as skipnative or skipmanaged are given.)

Fix #4674

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>
* Scripts: allow check-definitions py to ignore the harmless

The third argument of check-definitions.py specifies harmless keywords
to be suppressed from emitting warning messages.

Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>
* Scripts: add ignored cdefine keywords for warning

As an example of how to declare compiler definition keywords
that are harmless to be inconsistent between the native and the
managed, we have added FEATURE_IMPLICIT_TLS and FEATURE_HIJACK.

Developers may add more keywords in System.Private.CoreLib.csproj
if the keywords are verified to be harmless; i.e., although
the keywords exist in both cmake and clr.coreclr.props,
the keywords are NEVER used in either side of the sources
or the keywords only happen to have the same name while they
denote the completely different semantics and may be disjoint.

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