Add a comment at the use of RewriteTopLevelComma.

Refactor top-level comma rewriting.

Top-level commas were being handled in three separate parts of
rationalize: TreeTransformRationalization, RewriteSimpleTransforms,
and DoPhase. Furthermore, the transform performed by the latter
was distinct from the transforms performed by the former two parts,
as those parts called out to RewriteTopLevelComma in order to do
the necessary rewrite. This change collects the two transforms
in RewriteTopLevelComma and centralizes the comma handling in
RewriteSimpleTransforms.

With the comma handling centralized in RewriteSimpleTransforms,
TreeTransformRationalization was not doing much of anything, so
it has been removed and RewriteSimpleTransforms renamed to
TreeTransformRationalization.

Merge pull request #6137 from papaslavik/int_softfp_fix

Fix for #5877: pre-spill just the floating arguments

Merge pull request #5966 from mikedn/x86-neg-long

Implement GT_NEG decomposition

Merge pull request #6198 from briansull/multireg-refactor

Refactor and cleanup work for passing and returning struct types

Merge pull request #5948 from jyoungyun/fix/issue_5947

Add testsRunningInsideARM.txt

Fix exceptions in GC wave 2 (#6220)

I've removed blocks with PAL_TRY that were under CATCH_GC. I also had to
fix contract problem in GCToEEInterface::GcStartWork that was marked as
THROWS due to StubHelpers::ProcessByrefValidationList being marked as
THROWS, but the StubHelpers::ProcessByrefValidationList in fact doesn't
throw since it has body wrapped in EX_TRY / EX_CATCH.

This also fixes a problem that started to appear after my previous exceptions
fix change.

Implement GT_NEG decomposition

Refactor and cleanup work for passing and returning struct types

Replaces argOrReturnTypeForStruct with two new methods:
  getArgTypeForStruct - Provides information on how to pass a struct type
  getReturnTypeForStruct - Provides information on how to return a struct type

A struct can be passed or return in the following different ways:
1. A struct type may be passed/returned as a primitive type when its size is small
2. A struct type may be passed/returned by reference to a copy
3. A struct type may be passed/returned by value using multiple registers
4. A struct type may be passed by value using a copy on the stack

Incorporated code review feedback with expanded comments.

Merge pull request #6215 from dotnet-bot/UpdateDependencies20160711061707

Updating External dependencies to beta-24311-00

Merge pull request #6221 from pgavlin/RemoveDeadCodeInRationalize

Remove dead code in Rationalize.

Remove dead code in Rationalize.

Most of the removed code deals with QMARK rewriting. These nodes
are now rewritten as part of fgMorph, so the rationalizer does not
need to deal with them (other than asserting that they do not
exist).

Other dead code:
- The `Rationalizations` type
- A few split-related declarations and helpers

Merge pull request #6204 from AndyAyersMS/ModelPolicyDepthCheck

Inliner: add depth check to ModelPolicy

Merge pull request #6216 from hqueue/fix/typo_20160710

Fix a misleading typo in comment

Merge pull request #6202 from BruceForstall/RefactorDecomp

Refactor RyuJIT/x86 long decomposition code

Fix a misleading typo in comment

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>

Avoid repeated virtual method calls in List.Contains (#6212)

Updating External dependencies to beta-24311-00

Refactor RyuJIT/x86 long decomposition code

All decomposition code is now in decompose.cpp, in a Decompose class.
Each node type that is decomposed has its own member function to do
the decomposition. Various helpers have been added to reduce code
duplication. In general, though, the code is as it was before.

Inliner: add depth check to ModelPolicy

Give the ModelPolicy the standard depth bailout logic. This mainly
serves to prevent runaway inlining for recursive methods marked with
AggressiveInlining attributes.

Merge pull request #6201 from AndyAyersMS/LowerCallsiteWeight

Inliner: updates to ModelPolicy

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

Merge changes from TFS

Inliner: updates to ModelPolicy

Updates to bring CS and TP impact of the ModelPolicy into more acceptable
ranges.

For CS, reduce the call site weights to values that are more in keeping
with the legacy policy weights. Local test runs show this does not
drastically alter CQ and brings CS down below LegacyPolicy levels, on
average.

Implement an early out rejection based solely on ILSize. The threshold
value is set by conservatively determining when ILSize alone indicates
the method in question will never be inlined (note the policy itself
does not have an explicit ILSize cutoff). See comments for
`ModelPolicy::NoteInt` for details. Note if we adjust the model's size
and profitability estimates, this threshould will also need updating.

CQ (as measured by the CoreCLR perf tests) continues to show about a
2.5% geomean improvement over LegacyPolicy.

Fix exceptions in GC (#6192)

The GC code in general doesn't expect exceptions to be thrown from functions
it calls. However, the event creation functions were in fact throwing, so
in case of OOM, exception would be thrown from various places of the code.
This change creates non-throwing versions of the event creation functions
and replaces all usages of the throwing ones in the GC.
In addition, I've removed blocks with PAL_TRY that were under NO_CATCH_HANDLERS
define and also added Object::SetAppDomainNoThrow, which allowed me to removed
EX_TRY from GCHeap::StressHeap where the Object::SetAppDomain was called before.

Merge pull request #6193 from BruceForstall/FixTypos

Fix some typos in comments

Merge pull request #6195 from rahku/crossgen

Fix SOS bpmd for windows

Merge pull request #6197 from wtgodbe/heapVerify

Add new scenario to code coverage job

Add new scenario to code coverage job

Merge pull request #6176 from sivarv/lsraTodo

Optimize use of a reg-optional lclVar that is marked for both reload and spillAfter

Fix SOS bpmd for windows

Fix some typos in comments

Fixing the build break/unsigned to signed warning.
This checkin onbehalf of Tijoytk

[tfs-changeset: 1616784]

Optimize use of a reg-optional lclVar that is marked for both reload
and spillAfter.

Merge pull request #6163 from josteink/resolve-python

Resolve Python2.7 when executable is not called 'python'.

Update contributor guidelines (#6183)

Merge pull request #6177 from BruceForstall/FixMultiregStructArgNyi

Fix RyuJIT/x86 fgMorphMultiregStructArg NYI

Resolve Python2.7 when executable is not called 'python'.

Fix for #5877: pre-spill just the floating arguments

Fix RyuJIT/x86 fgMorphMultiregStructArg NYI

The code leading to the fgMorphMultiregStructArg NYI shouldn't have been taken for x86.
In fgMorphArgs, I put the setting of `hasMultiregStructArgs = true` in one case under `#ifdef`.

Disable tests failing with a newly exposed assert, `'isPhiDefn || type != TYP_STRUCT'`,
tracked by #6180.

I also fixed a number of asserts on strings that would never fail the way they were
written -- `assert("string")` instead of `assert(!"string")` -- and fixed some typos/grammar.

Merge pull request #6081 from adityamandaleeka/virtual_temp

Clean up unused code related to memory mapping

Merge pull request #6021 from CarolEidt/LessConservativeGtObj

Less conservative gt obj

Merge pull request #6175 from tijoytom/master

MarshalAs(UnManaged.ByValArray) overflow.[Resubmitting the PR]

Merge pull request #6150 from sivarv/lsraRefactor

Outstanding code review feedback from reg-optional PR #6092

Merge pull request #6170 from wtgodbe/signarm

Revert sign.builds so that arm64 bins can be signed

Merge pull request #6173 from kyulee1/gc03

ARM64: Enable Daily Job for GcStress0x3

MarshalAs(UnManaged.ByValArray) overflow.
For non-blittable embedded array in structs we ignored the SizeConst and wrote past the
buffer when number of elementsin the arrayis greater than SizeConst.Fix is to truncate
the array at SizeConst
MarshalAs(UnManaged.ByValTStr)
Very subtle case when the SizeConst == Number of bytes required to marshal , we
write the null one past the buffer.This happens only on  machine with non-english
(multi-byte) locale as default. Fix is to check the number of bytes required and
truncate the correctly leaving space for the terminating null.

ARM64: Enable Daily Job for GcStress0x3

Merge pull request #6090 from kyulee1/gctest3

ARM64: Test Update for GcStress0x3

Avoid box allocation in Encoding.EncodingName (#6142)

`Encoding.EncodingName` concatenates a `string` with an `int` via
`string.Concat(object, object)`, which results in the `int` being
boxed.

This change avoids the box by calling `int.ToString()`,
allowing `string.Concat(string, string)` to be used.

Revert sign.builds so that arm64 bins can be signed

Code review feedback.

Merge pull request #6114 from jamesqo/encodingforwarder

Consolidate much of the duplicated code across Encoding implementations

Merge pull request #6152 from mikedn/x86-addsub-lo

[WIP] Introduce GT_ADD_LO and GT_SUB_LO

ARM64: Test Update for GcStress0x3

Merge pull request #6161 from dotnet-bot/UpdateDependencies20160707062011

Updating External dependencies to beta-24307-00

Avoid heap allocating in char.ConvertFromUtf32 (#6141)

Updating External dependencies to beta-24307-00

Convert SymbolType.m_format to be a string (#6133)

Avoid box allocation in DateTimeFormat.FormatCustomized (#6147)

The current implementation calls `string.Concat(object, object)`, which
results in an `int` box allocation.

Avoid the box allocation by calling `int.ToString()`, allowing
`string.Concat(string, string)` to be used.

Avoid box allocation in TypeBuilder.DefineDataHelper (#6148)

The current implementation calls `string.Concat(object, object)`, which
results in a box allocation.

Avoid the box allocation by calling `int.ToString()`, allowing
`string.Concat(string, string)` to be used.

Avoid box allocation in ActivityTracker (#6149)

The current implementation calls
`string.Concat(object, object, object)`, which results in a box
allocation.

Avoid the box allocation by calling `long.ToString()`, allowing
`string.Concat(string, string, string)` to be used.

make all public properties of DebuggableAttribute visible outside mscorlib (#6153)

Add testsRunningInsideARM.txt

ARM emulator cross build checks are automatic on a per PR basis.
But it does not check unit tests because it takes a lot of time in emulator.
So this file contains the minimum test cases for ARM CI.

Those test cases are picked according to the following rules.
Reference target: Raspberry Pi2

1. Failed at least 3 times on June.
2. Exclude long time tc (Huge*)
3. Exclude daedlock tc (Burgers)
4. Exclude failed tc currently.

Signed-off-by: Jiyoung Yun <jy910.yun@samsung.com>

Merge pull request #6157 from rahku/crossgen

Do not clean tests folder when skiptests option is used along with clean

Merge pull request #6151 from mikedn/runbench-coreclr

Fix incorrect RunBenchmarks help text

Do not clean tests folder when skiptests option is used along with clean

Merge pull request #6130 from rahku/crossgen

ARM64: Fix incremental build problem for cross-components

Fix incremental build problem for cross-components

Merge pull request #6139 from mkborg/fix_lldb_search_order

Fix LLDB search order

Merge pull request #6146 from ellismg/fix-sign-targets-syntax-error

Fix syntax error in sign.builds

Fix incorrect RunBenchmarks help text

RunBenchmark's help text claims that there's a -coreclr option when in fact the option is -runner.

Introduce GT_ADD_LO and GT_SUB_LO

Fix syntax error in sign.builds

Merge pull request #6015 from kouvel/NamedMutexMoreFlockCases

Named mutex: Use flock instead of pthread process-shared mutex in som…

Merge pull request #6135 from dotnet-bot/UpdateDependencies20160706051943

Updating External dependencies to beta-24306-00

Fix LLDB search order

By default 'find_library' without 'NO_DEFAULT_PATH' keyword searches
first in default locations specified by '${CMAKE_SYSTEM_PREFIX_PATH}'
and only then in locations specified via 'PATHS' keyword
('${WITH_LLDB_LIBS}' in our case). As result it always picks-up default
LLDB ignoring the one explicitly specified via '${WITH_LLDB_LIBS}'.
This change modifies search order giving priority to '${WITH_LLDB_LIBS}'
and '${WITH_LLDB_INCLUDES}' over default locations.

Updating External dependencies to beta-24306-00

Named mutex: Use flock instead of pthread process-shared mutex in some more cases

Workaround for #5456:
- Sometimes, a timed wait operation is not getting released, causing a hang
- Due to the hang, it is not possible to detect this issue with code
- Temporarily disabled the use of pthread process-shared mutexes on ARM/ARM64. File locks will be used instead.

Workaround for #5872:
- On Alpine Linux, a pthread process-shared robust mutex is detecting the case where a process abandons the mutex when it exits while holding the lock, but is putting the mutex into an unrecoverable state (ENOTRECOVERABLE) instead of assigning lock ownership to the next thread that is released from a wait for a lock and notifying of abandonment (EOWNERDEAD).
- Added a test case to detect this issue, to have it use file locks instead

Close #5456

Merge pull request #6098 from CarolEidt/BlkNodeRefactors

More Blk Op Refactoring

Merge pull request #6102 from briansull/inliner-dump

Improve the dump for inline return expressions

Merge pull request #6101 from BruceForstall/RyuJITx86EH

RyuJIT/x86: add support for EH

Merge pull request #6094 from mikedn/x86-uncontained-long

Allow uncontained GT_LONG nodes

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

Merge changes from TFS

Merge pull request #6100 from BruceForstall/BotrX86EHUpdate

Add information on x86 EH to CLR ABI

Merge pull request #6093 from rahku/arm64retbuf

Revert Pinvoke ILStub calli signature for desktop clr

only use config on coreclr

[tfs-changeset: 1616092]

Remove TODO on throwing exception

Merge pull request #6106 from mikedn/modopt-feedback

Address div/mod CR feedback

More Blk Op Refactoring

Code refactoring in preparation for 1st Class Structs:
Factor out the legacy code generation for CpObj.
Abstract some functionality to member methods.
Move some block-related method definitions.
Make fgOrderBlockOps legacy-only (it is unused in RyuJIT).
Factor out the code to create a block init value.
Add a new IsLocalExpr() method.
Change some variable names to distinguish between objects and their addresses.

Move the SecuritySafeCritical attributes to EncodingForwarder

Merge pull request #6120 from dotnet-bot/UpdateDependencies20160705051942

Updating External dependencies to beta-24305-00

Better performance for MulticastDelegate.Equals (#6113)

Better performance for MulticastDelegate.Equals

One of the flags MAP_SHARED or MAP_PRIVATE has to be specified. MAP_PRIVATE seems to fit here. (#6089)

Updating External dependencies to beta-24305-00

Add note on UTF8Encoding.GetByteCount to EncodingForwarder

Fix CoreFX test failures

[UTF8Encoding] Remove unncessary comments

[UTF7Encoding] Remove unncessary comments

[UTF32Encoding] Remove unncessary comments

[UnicodeEncoding] Remove unncessary comments