Merge pull request #6597 from yizhang82/test-warnings

Fix warnings in interop tests

Merge pull request #6572 from JosephTremoulet/starg

Allow inlining when inlinee has `starg`

Allow inlining when inlinee has `starg`

The inliner's restriction that methods that make use of the `starg` opcode
cannot be inlined is somewhat artificial.  The comments indicate that the
reason for this excluion is because the inliner forward-substitutes
argument expressions in place of parameter references, but the codepaths
required to force evaluating an argument expression at the callsite and
reference that result in place of parameter references is already in place
to handle `ldarga` and complex argument expressions.

This change
 - adds an `argHasStargOp` flag to the `inlArgInfo` type, similar
   to the existing `argHasLdargaOp` flag
 - removes the `CALLEE_STORES_TO_ARGUMENT` inline observation and instead
   sets the new flag for `starg` cases
 - updates `impInlineFetchArg` to force evaluation to a temp for arguments
   that are marked `argHasStargOp`
 - updates `starg` processing in the importer to store to the
   corresponding temp when importing for an inlinee

Resolves #6014.

Merge pull request #6579 from CarolEidt/LessConservativeGtObj

Less Conservative GtObj

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

Merge changes from TFS

Fix newly introduced typo in Documentation/README

Fix warnings in interop tests and disable IsComObject(null) scenario as it is failing in non-Windows

Fix link to all BotR chapters in table of contents

Merge pull request #6612 from vancem/Doc_Add_BOTR_TOC

Add a table of contents to the book of the runtime

Pull Request feedback

Add a table of contents to the book of the runtime

 (so you have one link to go to that represents the entire book)

Adding support to the following generic dictionary entry slots for R2R: (#6291)

DeclaringTypeHandleSlot
ConstrainedMethodEntrySlot

Fix contract violation (violation caused by a copy/paste bug in PR 6200

[tfs-changeset: 1620960]

Merge pull request #6558 from bryanar/dumpling_ci2

Collect core dump URLs from dumpling service to raise up to CI.

on fail: create local_dumplings.txt with the url of the uploaded core dump file if dumpling is enabled. After, concatenate all of the local_dumplings.txt in to $(PWD)/dumplings.txt

Updating External dependencies to beta-24404-00 (#6601)

Added additional comments to AppendFormatHelper to explain the parsing. (#1261)

* Added additional comments to AppendFormatHelper to explain the parsing.
* Create private static readonly properties for Index Limit and Width Limit.

Merge pull request #6596 from yizhang82/interop-api

Add more interop API into System.Runtime

Merge pull request #6589 from briansull/issue-5954

Fix for Issue 5954

Adding Support for FieldDescSlot generic dictionary entries for R2R generic code. (#6200)

This enables generic code with ldtoken instructions for fields on generic types to be compiled into R2R.

This change makes newer versions of the runtime compatible with older version R2R images,
but not vice-versa. Therefore, the major version is increased to 2 with this change.

This change adds more encodings to a R2R image without changing the format of any previously encoded
entity, and is backwards compatible.

Add more interop API into System.Runtime and update mscorlib.cs

Fix for issue 5954 - Assert 'fieldSeq != FieldSeqStore::NotAField()'

Added new method GenTree:OperIsLocalField()
Fixes DefinesLocalAddress to properly update pIsEntire when we have a LocalField node type
Fixed fgValueNumberBlockAssignment to handle a GT_COPYBLK or GT_COPYOBJ operation
  where we have an incomplete field sequence for the Dst/LHS node.
  We already were handling an incomplete field sequence for the Src/RHS node.
  Update the dump to properly print "new uniq" whenever we issue a conservative value number.

Merge pull request #6592 from Priya91/runpath

Modify rebuild scenario.

Update documentation on how to build corefx (#6594)

Merge pull request #6586 from kyulee1/fixbuildtest

ARM64: Fix Build Test Break

Modify rebuild scenario.

Merge pull request #6534 from RussKeldorph/fix6236

Fix build warnings in Vector3Interop test

Merge pull request #6578 from rahku/version

RC2 release used version 4.6.*. However RTM release started to use th…

ARM64: Fix Build Test Break

Merge pull request #6584 from kyulee1/distest

ARM64: Disable Two Tests

Merge pull request #6529 from JosephTremoulet/AssertionOrder

Optimize bounds checks with correct assertion set

ARM64: Disable Two Tests

https://github.com/dotnet/coreclr/issues/6583

Merge pull request #6576 from Priya91/runpath

Use absolute paths for specifying projects to build.

RC2 release used version 4.6.*. However RTM release started to use the default values of 1.0.* by mistake. Revert back to using 4.6 as major & minor version number

Less Conservative GtObj

Don't mark GT_OBJ with GTF_EXCEPT or GTF_GLOB_REF if it is a local struct.
Also, fix changes needed in arm register allocation and code generation to handle the case where a struct argument is promoted, which appears to be code that was never exercised.
These changes are in preparation for further changes for First Class Structs. I am working to replace block ops with assignments, and was attempting to get to zero diffs, but it is quite difficult to do so given some inconsistencies in the treatment of GT_OBJs of locals.

Optimize bounds checks with correct assertion set

Assertion prop defers removing redundant bounds checks until processing
the parent comma, because the rewrite involves the comma as well.  The
code currently has a bug in that its check for whether the bounds check
is redundant also occurs when processing reaches the comma, which is wrong
because the assertion set may then include assertions generated by the RHS
of the comma (there is code to compensate for the assertion generated by
the bounds check itself, which is the LHS of the comma, but no compensation
for the RHS).

This change moves the analysis of whether bounds checks are redundant to
the proper spot in the processing order, and delays only the rewrite to
the processing of the comma; the redundancy of the bounds check is
communicated via a new opcode-specific flag: GTF_ARR_BOUND_INBND.

Fixes #6318.

Update cross-building.md

Update windows-test-instructions.md

Update unix-test-instructions.md

Fix build warnings in Vector3Interop test

Fixes #6236

Swallow exception in gc_heap::fire_etw_allocation_event (#6571)

The gc_heap::fire_etw_allocation_event calls TypeHandle::GetName that can throw
due to OOM. But we don't want failures to fire event cause a failure during the GC,
so such exception should be swallowed and no event fired in such case.
This was discovered as a contract failure during test runs with ETW logging enabled
when this function was transitively called from Thread::PerformPreemptiveGC which
was recently modified to have NOTHROW contract.
Interestingly, this was not detected by the static contract checker.

Update windows-instructions.md

Use absolute paths for specifying projects to build.

Merge pull request #6564 from kyulee1/fixbuild

ARM64: Fix Build Break

Slightly increase throughput of string.Concat(object[]) (#6547)

Merge pull request #6566 from Priya91/run2

Change processing of args in build scripts to keep =.

ARM64: Fix Build Break

Change int return values to BOOL for coreclr delegates (#6551)

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

Merge changes from TFS

Rename ZeroFreeMemoryUTF8 to ZeroFreeCoTaskMemUTF8 in ref (#6561)

Documentation: Additional optimization levels for ARM/Linux (#6548)

This instruction is to enable -Ofast or -Oz optimization level of clang on ARM/Linux.
Below is file-size comparison result of major components among -O3, -Ofast,
and -Oz on ARM/Linux-based Raspberry Pi2 board (Ubuntu/ARM 14.04 32bit).

* Linux/ARM: Release Build + O3(default): stripped , commit: 561b64d2c210b4d999de7f1ac55756704eaba784 (Jul-26-2016)
------------------------------------------------------------------------------------------------------------------
-rwxr-xr-x 1 leemgs leemgs     18,472 Jul 27 16:14 corerun
-rwxr-xr-x 1 leemgs leemgs  1,512,828 Jul 27 18:33 libclrjit.so
-rwxr-xr-x 1 leemgs leemgs  4,525,328 Jul 27 18:33 libcoreclr.so
-rwxr--r-- 1 leemgs leemgs     29,696 Jul 27 00:55 ./mscorlib.dll
-rwxr--r-- 1 leemgs leemgs  2,211,840 Jul 27 00:54 ./System.Private.CoreLib.dll
Total:   8,298,164 bytes

* Linux/ARM: Release Build + Ofast: stripped , commit: 561b64d2c210b4d999de7f1ac55756704eaba784 (Jul-26-2016)
-------------------------------------------------------------------------------------------------------------
-rwxr-xr-x 1 leemgs leemgs    18,496 Jul 26 05:00 ./corerun
-rwxr-xr-x 1 leemgs leemgs 1,512,844 Jul 26 04:59 ./libclrjit.so
-rwxr-xr-x 1 leemgs leemgs 4,525,360 Jul 26 04:59 ./libcoreclr.so
-rwxr--r-- 1 leemgs leemgs    29,696 Jul 26 00:49 ./mscorlib.dll
-rwxr--r-- 1 leemgs leemgs 2,211,840 Jul 26 00:48 ./System.Private.CoreLib.dll
Total:   8,298,236 bytes

* Linux/ARM: Release Build + Oz: stripped , commit: 561b64d2c210b4d999de7f1ac55756704eaba784 (Jul-26-2016)
---------------------------------------------------------------------------------------------------------
-rwxr-xr-x 1 leemgs leemgs    14,400 Jul 29 05:02 ./corerun
-rwxr-xr-x 1 leemgs leemgs 1,181,160 Jul 29 04:40 ./libclrjit.so
-rwxr-xr-x 1 leemgs leemgs 3,407,200 Jul 29 04:40 ./libcoreclr.so
-rwxr--r-- 1 leemgs leemgs    29,696 Jul 29 00:49 ./mscorlib.dll
-rwxr--r-- 1 leemgs leemgs 2,211,840 Jul 29 00:48 ./System.Private.CoreLib.dll
Total:  6,844,296 bytes

Signed-off-by: Geunsik Lim <geunsik.lim@samsung.com>

Merge pull request #6565 from Priya91/checkrun

Add /m option for builds.

Change processing of args in build scripts to keep =.

Merge pull request #6533 from pgavlin/LSRA

Replace the LSRA stack with a hash table.

Add /m option for builds.

Merge pull request #6546 from pgavlin/LivenessCleanup

Refactor fgComputeLife.

Add function headers for new liveness methods.

In particular, `Compiler::fgComputeLife{Call,Local}` now have function
headers.

Merge pull request #6557 from Priya91/runfix

Fix errors in build scripts from run tool update.

Merge pull request #6366 from pgavlin/RemoveEmbeddedStatementsPlan

Add the proposal for embedded statement removal.

Replace the LSRA stack with a hash table.

LSRA currently uses a stack to find the `LocationInfo` for each register consumed
by a node. The changes in this stack's contents given a particular node are
governed by three factors:
- The number of registers the node consumes (`gtLsraInfo.srcCount`)
- The number of registers the node produces (`gtLstaInfo.dstCount`)
- Whether or not the node produces an unused value (`gtLsraInfo.isLocalDefUse`)

In all cases, `gtLsraInfo.srcCount` values are popped off of the stack in the
order in which they were pushed (i.e. in FIFO rather than LIFO order). If the
node produces a value that will be used, `gtLsraInfo.dstCount` values are
then pushed onto the stack. If the node produces an unused value, nothing is
pushed onto the stack.

Naively, it would appear that the number of registers a node consumes would be
at least the count of the node's non-leaf operands (to put it differently, one
might assume that any non-leaf operator that produces a value would define at
least one register). However, contained nodes complicate the situation: because
a contained node's execution is subsumed by its user, the contained node's
sources become sources for its user and the contained node does not define any
registers. As a result, both the number of registers consumed and the number of
registers produced by a contained node are 0. Thus, contained nodes do not
update the stack, and the node's parent (if it is not also contained) will
pop the values produced by the contained node's operands. Logically speaking,
it is as if a contained node defines the transitive closure of the registers
defined by its own non-contained operands.

The use of the stack relies on the property that even in linear order the
JIT's IR is still tree ordered. That is to say, given an operator and its
operands, any nodes that execute between any two operands do not produce
SDSU temps that are consumed after the second operand. IR with such a
shape would unbalance the stack.

The planned move to the LIR design given in #6366 removes the tree order
constraint in order to simplify understanding and manipulating the IR in the
backend. Because LIR may not be tree ordered, LSRA may no longer use a stack
to find the `LocationInfo` for a node's operands. This change replaces the
stack with a map from nodes to lists of `LocationInfo` values, each of
which describes a register that is logically defined (if not physically
defined) by that node. Only contained nodes logically define registers that
they do not physically define: contained nodes map to the list of
`LocationInfo` values logically defined by their operands. All non-contained
nodes map to the list of `LocationInfo` values that they physically define.
Non-contained nodes that do not define any registers are not inserted into
the map.

Add ExtraParameters to config.json to enable --.

Fix errors in build scripts from run tool update.

Initialize `doAgain`.

Refactor fgComputeLife.

- Extract the liveness computations for call and local var nodes out into
  fgComputeLifeCall and fgComputeLifeLocal, respectively.
- Split fgComputeLife into two versions: one for the RyuJIT backend and
  one for the legacy backend. The former leaves out the QMARK/COLON
  processing that is necessary in the latter.

Merge pull request #6491 from adiaaida/formatting2

Massage code for clang-format

Merge pull request #6400 from Priya91/useruntool

Use run tool in coreclr dev workflow.

Update linux-performance-tracing.md

Apply run on recent changes in master.

Respond to PR feedback.

Remove clean option from arm script, which deletes the copied test bits in CI.

Grant 777 for run.sh

Use run tool in coreclr dev workflow.

Merge pull request #6493 from rahku/crossgen

adds testhost package

adds testhost package

Merge pull request #6521 from adityamandaleeka/remove_internal_free

Remove CorUnix::InternalFree from PAL

Merge pull request #6281 from papaslavik/cse_sort_sz

The sort for CSE size optimization should be different from exec

Merge pull request #6515 from kyulee1/fixcontract

Remove useless contract in EventListener

Merge pull request #6479 from dagood/update-dependencies

Using full versions, update CoreFX dependencies to beta-24326-03

Fix a build break.

[tfs-changeset: 1619855]

Using full versions, update CoreFX dependencies to beta-24328-05.

To fix tests, change dnxcore50 -> netcoreapp1.0 and add test_runtime.

Merge pull request #6506 from gkhanna79/FixReflectionLoadContext

Enable RefEmitted assemblies to have Fallback LoadContext notion

Massage code for clang-format

This change starts the process of updating the jit code to make it ready
for being formatted by clang-format. Changes mostly include reflowing
comments that go past our column limit and moving comments around ifdefs
so clang-format does not modify the indentation. Additionally, some
header files are manually reformatted for pointer alignment and marked
as clang-format off so that we do not lose the current formatting.

Merge pull request #6523 from dotnet-bot/UpdateDependencies20160729052446

Updating External dependencies to beta-24329-00

Merge pull request #6499 from ramarag/unc_cwd

Fxing the paths that are resolved relative to a network share

Updating External dependencies to beta-24329-00

Merge pull request #6394 from adityamandaleeka/ymm_save_restore

Save and restore ymm registers in signal handlers.

Merge pull request #6512 from dotnet-bot/UpdateDependencies20160728083626

Updating External dependencies to beta-24328-00

Expose missing members of Array in contract to prep for dotnet/corefx#9998. (#6519)

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

Merge changes from TFS

Remove CorUnix::InternalFree.

Merge pull request #6496 from adityamandaleeka/virtual_cleanup

Add in-memory circular buffer for logging virtual memory operations.

Update how we pass parameters when building CoreFx (#6511)

Change how we pass parameters when building CoreFx

Also clear CONTEXT_XSTATE if fpregs aren't available.

Merge pull request #6480 from davmason/master

Merge changes with .net native.

The fix for DisposeOnShutdown is already in here, these are the changes that exist in ProjectN that are not in CoreCLR yet.

Add extended state to RtlRestoreContext

Remove useless contract in EventListener

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

Fxing the paths that are resolved relative to a network share

Updating External dependencies to beta-24328-00

Skip dummy null argument for string constructors under CoreRT ABI (#6498)

Fix for build issue #807024, assemble debug helpers for ARM64.

[tfs-changeset: 1619712]