Merge pull request #7945 from ramarag/MemoryFailPoint

Disabling checks in MemoryFailPoint for Xplat

Merge pull request #7908 from sandreenko/fix_tail_call_in_R2R

Correct pattern check for tail calls

Fix a issue where ETW was losing RCW and CCW in heap dumps. (#7971)

Basically our builk events for RCWs and CCWs were too big, causing ETW to reject them, and thus losing information.

However you have to have at least several hundred befor you have enough to force the problem.

Merge pull request #7958 from pgavlin/VSO280123

Make a few VN methods tolerant of NoVN.

Make CrossGen documentation up to date (#7973)

Correct pattern check

sequense
localVariable = cast.int(call.small_type());
return localVaribale
is correct tail call pattern.

Merge pull request #7956 from pgavlin/VSO280110

Properly update block flags during branch opt.

Format code.

Merge pull request #7959 from pgavlin/VSO280116

Fix an assert condition in lowering.

Format code.

Merge pull request #7968 from AndyAyersMS/CaptureNewObservations

Inliner: capture new observations

Mark NonRandomizedStringEqualityComparer as serializable (#7966)

Inliner: capture new observations

Rebase the DiscretionaryPolicy on the EnhancedLegacyPolicy, and
capture and report some of the new observations that have been added
recently.

This change adds a new HAS_GC_STRUCT observation in addition to the
more specific RARE_GC_STRUCT, so that the DiscretionaryPolicy can
be notified of GC struct locals and temps for all candidates, not
just ones invoked from rare call sites.

No changes in codegen.

Revises 'JitTimer::PrintCsvHeader' to use 'append' mode (#7073)

* Revises 'JitTimer::PrintCsvHeader' to use 'append' mode

The current implementation of 'JitTimer::PrintCsvHeader' opens a CSV
file with 'read' mode and re-opens it with 'write' mode if it is absent.

The current implementation has two problems:
First, "read" file pointer will be closed even if it is null,
which is reported in #7072.
Second, "write" file pointer will be leaked as there is no corresponding
fclose.

This commit rewrites 'JitTimer::PrintCsvHeader' to fix #7072.

ParseNumber & related more efficient (#3163)

Merge pull request #7936 from vancem/EventSourceDisposeRace.11-01-16

Fix Deadlock associated with ETW commands happening during dispose.

Fix race condition (#7928)

Suppress SOS.NETCore.dll error msg in release mode (#6975)

Suppress error msg when running coreclr without
SOS.NETCore.dll in release mode

Fix an assert condition in lowering.

Tail calls via the tail call helper must live in GC-safe blocks. A block
is GC-safe if either it or all of its dominators are also GC-safe. If we
are not optimizing, we will not compute dominators, and will therfore
not mark blocks whose dominators are all GC-safe. When deciding whether
or not we need to make a block GC-safe during tail call morphing,
however, we allow a tail call in an unmarked block iff the entry block
is GC-safe, as the entry block trivially dominates every reachable
block. This change fixes an assert that was not allowing for a tail call
in a block that is not marked GC-safe where the entry block is marked
GC-safe.

Set pRetVal as NULL on error inside 'VIRTUALCommitMemory' (#7920)

Merge pull request #7917 from pgavlin/VSO280120

Fix VSO280120.

Make a few VN methods tolerant of NoVN.

There were a few methods on the VN store that were not resilient to
being provided with the "no value number" sentinel value. This lack of
resilience caused assertion failures during assertion propagation when
generating assertions for trees marked with "no value number".

Properly update block flags during branch opt.

`fgOptimizeBranch` was updating the flags for the predecessor block
using the flags from the wrong destination block. This was causing an
assert in `AllocateObjects` when verifying that all blocks that contain
`allocObj` nodes are properly marked with `BBF_HAS_NEWOBJ`.

Fix VSO280120.

st.lclFld decomposition was splitting a st.lclFld into two of the same
without bumping the destination var's ref count.

Check pointer before dereferencing it (#7930)

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

Merge changes from TFS

Merge pull request #7943 from sivarv/rotateFix

Exclude GT_ROL/ROR from ==/!= 0 compare peep optimization

Specify .NETCoreApp,Version=v1.1 for test projects (#7937)

Fix build break

[tfs-changeset: 1636334]

Merge pull request #7951 from jkotas/revert-correct-parameter-names

Revert "Correct parameter names."

Revert "Correct parameter names."

This reverts commit 0cae769f3c60bf2a696d02a88527a46d5b211cd5.

Merge pull request #7950 from jkotas/revert-parameter-name-changes

Revert "Correct parameter names."

Revert "Correct parameter names."

This reverts commit a212082aa4cea8529b4c78844e2a7408ff6f7969.

[ARM] Implement BitPosition with arm intrinsics (#7819)

Merge pull request #7904 from shrah/master

Update S.R.InteropServices to ns2.0

Fix build break 814276: FEATURE_SPAN_OF_T disabled in runtime but enabled in DAC

[tfs-changeset: 1636322]

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

Merge changes from TFS

Expose missing API in models.xml (#7940)

progress towards https://github.com/dotnet/corefx/issues/12812 and https://github.com/dotnet/corefx/issues/12398

Update GCInfoDecoder with CoreRT changes (#7938)

Porting most of the changes from https://github.com/dotnet/corert/pull/2116 to make GCInfoDecoder compile in CoreRT to keep the different copies in sync.

Disabling checks in MemoryFailPoint for Xplat till primitives like VirtualQuery
are implmemented properly

Rotate Left/Right xarch instructions don't set ZF and ZF flag.

Fix GitHub build break caused by change #1636179

[tfs-changeset: 1636227]

Enable the rest of netstandard1.7 globalization APIs (#7935)

* Enable more netstandard 1.7 Globalization APIs

* Enable Windows support for the new APIs

* Fix spaces and line alignments

Merge pull request #7923 from AndyAyersMS/FixNoCandidateBug

Jit: spill stack for non-candidate calls

Fix deadlock when commands race with EventSource Shutdown

CrossGen: Copy all entries of Debug Directory Table (#7849)

Fixes issue #6482.

Fix build break 814217  by removing inconsistencies between BCL and runtime

[tfs-changeset: 1636179]

Handle simd checks in OptimizeRangeCheck

When the index is a constant, this code applies to `GT_SIMD_CHK` equally
well as it applies to `GT_ARR_BOUNDS_CHECK`.

Use nameof throughout mscorlib (#7929)

Merge pull request #7677 from CarolEidt/StructOpts

Enable optimization of structs

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

Merge changes from TFS

Merge pull request #7883 from adiaaida/fixDecomposeRotate

Fix DecomposeRotate for > 32 case

Use resolver to get IL header and size (#7924)

Fix build break on x86

[tfs-changeset: 1636135]

Use normal verbosity logging when building tests (#7921)

Test build log file is ~4GB with diag verbosity. With normal it's about 20MB.
This seems to speed up the test build for me considerably.

See #7916.

Move SR adapter into root (#7925)

CoreFX creates SR in the System namespace. While putting the CoreCLR SR
adapter in the root may be unconventional, it allows us to keep the
shared code identical.

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

Merge changes from TFS

Merge pull request #7912 from JosephTremoulet/FlagVectorCounts

Flag vector counts

Merge pull request #7910 from JosephTremoulet/FixInitTest

Fix check for initial induction variable value

Fix check for initial induction variable value

Checking that `head` dominates `entry` is insufficient, becuase there may
be a codepath from `head` through other nodes that modify the induction
variable to `entry`.  Check instead that all preds of `entry` are either
`head` or in the loop.

Resolves #7906.

Merge pull request #7909 from JosephTremoulet/FixLoopEntryCheck

Skip removed loops in `optBlockIsLoopEntry`

Fix build break

[tfs-changeset: 1636112]

Jit: spill stack for non-candidate calls

Closes #7907.

Spill the evaluation stack at non-candidate calls since the call may
modify locals or args that are currently live on the stack.

In the attached test case I can't get the problematic IL sequence to
happen for the `F0` variant, but it is there for the `F1` variant.

No diffs seen in corlib or any of the framework assemblies that are
run via jit-dasm.

Merge pull request #7854 from dotnet-bot/master-UpdateDependencies

Update CoreClr, CoreFx to beta-24631-02, beta-24631-01, respectively (master)

Merge pull request #7918 from RussKeldorph/repoint-corefx-1.1.0

Make dotnet/coreclr reference corresponding dotnet/corefx branch

Merge pull request #7867 from Clockwork-Muse/ArgumentException

Replace hardcoded parameter names to ArgumentException with nameof

Make dotnet/coreclr reference corresponding dotnet/corefx branch

We run CoreFx tests under JitStress modes in CoreCLR.  Previously, we were
always pulling down CoreFx master, which is bad for CoreCLR release
branches when things are out of synch.

Merge pull request #7885 from pgavlin/VSO278376.2

Fix VSO 278376.

Merge pull request #7905 from pgavlin/VSO279380

Fix VSO 279380.

Add LPFLG_SIMD_LIMIT flag

This flag identifies loops whose limit was a Vector<T>.Count expression.
This can be used in heuristics to target more aggressive optimizations at
SIMD code.

Merge pull request #7852 from AndyAyersMS/InlinerSmallFixes

Inliner: Fix LegacyPolicyAssert; Dump prejit noinline reasons

Fix DecomposeRotate for > 32 case

In DecomposeRotate, we had a bug where we assumed loOp1 was
always gtLong's op1 and hiOp1 is always gtLong's op2 when we
call RepresentOpAsLocalVar, which is not true in the > 32 case, where we
swap hi and lo before generating the shift instructions for the rotate.
This change refactors the code so that we can correctly call
RepresentOpAsLocalVar with which of gtLong's ops hi and lo are.

A couple of changes included:
      - Make sure we have a way to disable building with the Roslyn compiler. That should only happen in the internal build.
      - Update the FeatureSpanOfT to also condition on using the latest compiler
      - Introduce an if/def in string.cs for code that uses the ref locals / ref return.
      - Make sure we only include the CryptographicException when building CoreCLR
      - Update the PInvokeMap.txt file to include the right set of APIs.
      - Remove duplicated APIs from PInvokeMap.txt

[tfs-changeset: 1636041]

Merge pull request #7884 from AlexGhiondea/Port_SecureString

Bring to CoreCLR the SecureString implementation from CoreFx.

Add GTF_ICON_SIMD_COUNT flag

This flag is used to indicate that an integer constant in the IR is the
result of replacing the Vector<T>.Count intrinsic.  This can be used to
drive heuristics focusing more aggressive optimizations on SIMD code.

Skip removed loops in `optBlockIsLoopEntry`

This allows running this utility downstream of loop unrolling.

Merge pull request #7886 from jkotas/SpanOfT

Merge SpanOfT branch into master

Fix VSO 279380.

The liveness pass attempts to remove dead stores that it encounters
during its liveness computation. Successfully removing a dead store,
however, may invalidate the results of the computation. Therefore, the
pass operates as a fixed-point algorithm: it recomputes liveness until
either the contents of the function do not change or the removed code
does not change the results.

When running liveness on LIR, the correct flag--`fgStmtRemoved`--was not
being set if a dead store was succesfully removed. This caused the
liveness pass to terminate early and produce invalid results in certain
cases. This fix sets `fgStmtRemoved` when a dead store is successfully
removed in an LIR block.

Update S.R.InteropServices to ns2.0

This is the final diff with net standard 2.0 except the SecureString functions. In particular,it adds the following:

M Marshal.BindToMoniker(System.String)
M Marshal.ChangeWrapperHandleStrength(System.Object,System.Boolean)
M ExternalException.ToString
T HandleRef

Merge pull request #7758 from pgavlin/VSO267565

Fix P/Invoke cookie passing on x86.

Fix VSO 278376.

genFloatToFloatCast was not checking whether or not its source operand
was an in-memory, register-optional node before comparing its source
register against its target register. This caused an assert in checked
builds (and SBCG in release builds) because the source temp was never
released.

SecureStringMarshal was not correctly being build for Unix.

Merge pull request #7901 from AlexGhiondea/FixBinCompat

Get latest behavior when missing TFM for BinaryCompatibility class

Add workaround for JIT crash

Change Span fields to private readonly + other review feedback

Update CoreClr, CoreFx to beta-24631-02, beta-24631-01, respectively

When reading the TFM for the BinaryCompatibility class on CoreCLR, set the TFM to be the latest version we have in our code.

This ensures that we are going to get the latest behavior (4.5.2+) for all the quirks that were using that class.

Add the SecureStringMarshal and CryptographicException to model.xml

Merge pull request #7897 from stephentoub/stream_copyto

Use ArrayPool in Stream.CopyTo{Async}

Remove StreamHelpers.ArrayPoolCopy.cs

It's no longer necessary now that Stream.CopyToAsync itself uses ArrayPool.

Fixing Typo in variable name in DateTime (#7898)

Avoid boxing in calls to enum's GetHashCode (#7895)

DateTime.ToString() Fast Track for RFC1123 (#7891)

* DateTime.ToString() Fast Track for RFC1123

Fix P/Invoke cookie passing on x86.

On x86, the P/Invoke cookie (when required) is passed on the stack
after all other stack arguments (if any).

Use ArrayPool in Stream.CopyTo{Async}

Add boilerplate doc comments to trivial methods

Streamline implementation of ToArray to call CopyTo<T> directly
Fix missing pinning in CopyTo<T>
Move and rename some JitHelpers to S.R.CS.Unsafe to make different implementations more similar

Fix build break on 32-bit platforms

Disable Span<T> GC reporting under TODO to workaround JIT issue

added implicit cast operator from ArraySegment to Spans (#7297)