Update with corefx PR changes

Add Unix FileStream Lock/Unlock

Implements FileStream Lock/Unlock by calling the SystemNative functions for fcntl F_SETLK

Use CoreFX Path code (#8132)

* Use CoreFX Path code

This is to address consistency and perf issues by moving the Path
code down to CoreCLR. It is more of an issue with Windows as
normalization requires complicated logic and costly platform calls.

* Delete unused files and avoid Unix crypto lib

* Address feedback, fix bug

Remove Unix crypto files and ifdef out usages completely.
Fix issue with static initialization of PathInternal.

* Use TriState to initialize case sensitivity

Inliner: updates to RandomPolicy (#8128)

Fix issues with RandomPolicy setup in release builds with -DINLINE_DATA.

Reparent RandomPolicy on top of DiscretionaryPolicy to enable inline
data dumps from random inline runs. Remove some now-unneeded overrides.

Add a full dump mode to JitInlineDumpData that reports the inliner-visible
data for all inlines, as opposed to inliner-visible and post-inline
data for just the most recent inline. Small mods to the dumper code to
adjust comma placement for this new mode.

Have the RandomPolicy make the full set of profitability observations
for each accepted inline so they can be dumped.

Merge pull request #8138 from sivarv/rmwFix

Do not generate "inc/dec [mem]" when asked to generate debuggable code.

Tweak params of Console

This isn't the implementation of Console we ship, apparently it's only here for CLR tests, or maybe shouldn't be at all. Just tweaking the param names to match what we ship so it doesn't show up in the diffs.

Merge pull request #8151 from adiaaida/enableTests

Remove test exclusions for tests that now pass

Add RuntimeHelpers.Equals

Fix dotnet/corefx#13692

We added this to the ref in 19fa781 but it was not present in the impl. The tools didn't flag this perhaps because Equals is available on Object, although since this is a new it really ought to know it wasn't referring to the Object one.

Add RuntimeHelpers.Equals

Tweak params

Add the missing implementation of IdnMapping.GetAscii (#8149)

* Add the missing implementation of IdnMapping.GetAsccii

we should have this implementation in place but no idea why got removed in the first place

* Use nameof

Remove test exclusions for tests that now pass

This change removes tests that have been fixed for x86 RyuJIT. It also
updates the Issues for the remaining tests with additional details.

Merge pull request #8146 from jashook/test_update_arm64

Update arm64 tests used by the CI

Merge pull request #8147 from rahku/reflection

Expose MethodBase.IsSecuritySafeCritical

Don't generate inc/dec [mem] while generating debuggable code.

Expose MethodBase.IsSecuritySafeCritical

Update IssuesFeedbackEngagement.md

Expose a member of StrongNameKeyPair, fix bug in SynchronizationContext (#8142)

Related to dotnet/corefx#11636
Related to dotnet/corefx#11808

Fixed string Join method

Fixed string Join(string separator, params object[] values) method.
Calling string.Join(",", null, 1, 2, 3); return empty string but should ",1,2,3".

Update arm64 tests used by the CI.

Merge pull request #8144 from adiaaida/formatArtifacts

Fix the path to build artifacts for formatting

Fix the path to build artifacts for formatting

Enable GCToOSInterface to be defined on the GC side of the GC interface (#8121)

* Re-introduce changes lost in a merge conflict

* Enable GCToOSInterface to be defined behind the GC interface
when building the GC in standalone mode. Provide a skeleton Windows
implementation and the framework for a Unix implementation.

* Address code review feedback

Full support of idn mapping (#8134)

* Enable full implementation of IdnMapping in corelib

* Add missing resources

Merge pull request #8135 from pgavlin/fixExtendPtrVN

Fix value numbering for FieldSeqStore::NotAField.

Format code.

Changing JIT perf test names to fix how they are organized on BenchView (#8083)

* Changing JIT performance tests' names to fix how tests are organized on BenchView.

* Adding changes based on PR suggestions.
- Add double quotes on cmd script
- Keep name casing consistent for the Benchstone benchmarks.

* Updating datacontractxml value to match new class name.

* Removing an extra / on the xmlns string.

* Making sure we are getting latest BenchView tools.
- Changed where BenchView tools are installed to (from C:\tools to %WORKSPACE%)
- Check if the tools already exist, if so, delete them and download them again.

Merge pull request #8110 from BruceForstall/FixLIRCallArgDumping

Fix the arg name/number dumping for call args in LIR

Fix value numbering for FieldSeqStore::NotAField.

Before this change, value numbering produced the same value number for
each occurrence of `FieldSeqStore::NotAField`. This semantics is not
logically consistent with the semantics of `NotAField`, however:
`NotAField` indicates that we cannot reason about the address expression
that it annotates, therefore we cannot reason about the value stored at
the corresponding location. A recent change to admit the VN for
`NotAField` in `ExtendPtrVN` exposed SBCG due to this mismatch in
semantics: distinct array elements were assigned the same value number
because each had the same base coupled with the `NotAField` VN.

This change revises the representation of `NotAField` during value
numbering: instead of reusing the same value number for each
occurrence of `NotAField`, each occurrence is instead assigned a new,
unique value number. These value numbers are represented using a new
chunk attribute to retain the ability to decide whether or not a value
number represents `NotAField` without the use of a more heavyweight map
(e.g. a `VNMap`).

Fixes #8133.

Merge pull request #8125 from stephentoub/divrem_perf

Improve Math.DivRem performance

Expose members of ExecutionContext and AsyncFlowControl (#8107)

Expose members of ExecutionContext and AsyncFlowControl

- Fix ExecutionContext.Capture and AsyncFlowControl.Undo when flow is suppressed
- Expose deserialization constructor for ExecutionContext

Merge pull request #8131 from JohnChen0/xmm

Fix XMM register count in CONTEXT_SetThreadContextOnPort in PAL

Merge pull request #8124 from martijnhoekstra/doc-aggex

fix grammar for AggregateException.Flatten

Fix XMM register count in CONTEXT_SetThreadContextOnPort in PAL

Merge pull request #8127 from jashook/arm64_post_build_script

Change over from the old arm64 scripting.

Change over from the old ar64 scripting.

This change moves away from hosting all of scripts on the Z:\ drive.
The share that used to be shared between the x64 machines and the
arm64 devices. It removes all the old scripting for one python script
arm64_post_build.py, which is responsible for setup of dotnet and
launching
the x64_client.

Merge pull request #8120 from adiaaida/fixDebuggerVarargs

Add special handling of varargs for x86 debugger

Fix crash in FunctionMemberPtrArrayHolder destructor (#8113)

Some elements in FunctionMember array might not be initialized
and contain garbage. This leads to crash when elements are being deleted.

Move array allocation inside FunctionMemberPtrArrayHolder class
and initialize all array elements to nullptr.

clean build-roofs and add lldb option (#8123)

- clean unused __MachineTriple
- add indents to case
- add lld 3.6 or 3.8 as option

Use Math.DivRem in another place in mscorlib

Went through all uses of % looking for places DivRem could be used.  This looks like it's the only one of note.

Improve Math.DivRem performance

Until the JIT is able to eliminate one of the two idiv operations, using a multiplication and subtraction is measurably faster than an extra division.

fix grammar for AggregateException.Flatten

The old phrasing wasn't gramatically correct. It's somewhat
difficult to find a concise summary

Reduce variable scope (#8021)

Merge pull request #8073 from dotnet/enablebinclash

Enable binclash logger

Merge pull request #8086 from pgavlin/VSO278523

Admit more dest addresses in block assign value-numbering.

Admit more dest addresses in block assign value-numbering.

Internal testing revealed an assertion when computing the value number
for block assignments: the VN framework expected any block assignments
to a tracked lclVar to have a destination address of the form
`(addr (lclVar))` or `(addr (lclFld))`. However, the check that it
uses to determine whether or not a block assignment targets a lclVar
also admits addresses formed by some combination of adds of constants
to these patterns (e.g. `(add (const 4) (add (addr lclVar) (const 4)))`.

This fix admits these additional patterns in value numbering by allowing
`ExtendPtrVN(GenTree*, GenTree*)` to supply `NotAField` to
`ExtendPtrVN(GenTree*, FieldSeqNode*)`.

Add special handling of varargs for x86 debugger

On x86, stack arguments must be accessed via the varargs cookie for
varargs. Since the functions would be identical, this change moves the
code from codegenlinear to codegencommon to fix vararg argument
handling in the debugger.

Fixes VSO 284774.

Fix the arg name/number dumping for call args in LIR

Use the arg number in the arg info table. Before this, I saw cases
where the args were named: arg0, arg1, arg1, arg3, instead of:
arg0, arg2, arg1, arg3, for example.

Merge pull request #8111 from BruceForstall/FixTestNewlines

Don't generate explicit newlines in SIMD tests

Merge pull request #8115 from chcosta/updatebuildtools

Update buildtools to version 01008-01

Merge pull request #8031 from pgavlin/VSO283795

Do not fold `(x + null)` in certain cases.

Merge pull request #8101 from sivarv/rmwOpt

Generate "inc/dec [mem]" instead of "add/sub [mem], 1" for Read-Modify-Write (RMW) operations.

Fix test exclusions.

The new regression test should only be disabled for builds that use the
legacy backend.

Merge pull request #8100 from rahku/reflection

Expose Assembly::GetFile(s) apis

Update buildtools to version 01008-01

Fixed string Join method

Fixed string Join(string separator, params object[] values) method.
Calling string.Join(",", null, 1, 2, 3); return empty string but should ",1,2,3".

Merge pull request #8108 from gkhanna79/Arm32Test

Enable WinArm32 P1 Test Build

Don't generate explicit newlines in SIMD tests

Use Console.WriteLine() instead.

Enable WinArm32 P1 Test Build

Inliner: ensure all statements have an inline context (#8064)

* JIT: Retype fgMorphStmt and related variables

There are quite a few variables typed as `GenTreePtr` that must point
at statements -- for instance `fgMorphStmt`. Retype these as `GenTreeStmt*`.
Cleanup dereferences which no longer require a `gtStmt`.

* Inliner: ensure all statements have an inline context

We would previously allow some statements inserted by the inliner
"between" the caller and callee to have a null inline context. This
lack of context could lead to incomplete or misleading inline trees
if the inserted code included calls. In particular some failed inlines
might not show up in the trees.

Set the context for these inserted statements to be the context for the
call being inlined (thus they logically "belong" to the caller).

Add implementation for AssemblyName::EscapedCodeBase (#8072)

Remove conflicting proj files (#8090)

Some test cases have both .ilproj and .csproj files, resulting
in a race condition during build. Removing the .csproj files,
as the MSIL versions appear to be the intended ones.

Merge pull request #8085 from stephentoub/daylight_alloc

Avoid allocations in DateTime.Now

Avoid allocations in DateTime.Now

Every call to DateTime.Now ends up allocating a DaylightTime as it computes the offset from UtcNow.  DaylightTime is just a plain old object that stores two DateTimes and a Timespan.  Ideally we could just change it to be a struct, but it's public.  Instead, this commit adds an internal DaylightTimeStruct, which is the same as DaylightTime except as a value type, and changes some of the internal uses of DaylightTime to instead use DaylightTime.  As a result, DateTime.Now is allocation-free.

Inliner: updates to random inlining (#8088)

Cache the random state on the InlineStrategy instead of on the compiler
instance so that the state is reinitialized and private to each jit request.
That way the random policy evaluations made compiling one method won't
alter the evaluations for subsequent methods. This should make it somewhat
easier to do A/B comparisons under jit stress when changing the jit.

Make it possible to enable the RandomPolicy outside of stress. This may
prove useful in various randomized inline performance studies or as
a simple stress mode on its own.

Random state seed is built from an external seed value (via JitStress or
JitInlinePolicyRandom) and an internal seed value (method hash), so that
random sequences potentially differ for each method but are deterministic
across runs and changes to the jit.

Remove redundant statement (#7900)

Generate inc/dec [mem] instead of add/sub [mem], 1 for read-modify-write operations.

Enable the rest if TimeZoneInfo APIs (#8096)

Expose Assembly::GetFile(s) apis

Rename gitmirrorfile

Merge pull request #8087 from ellismg/selective-mirror-sketch

Update .gitmirror metadata

Merge pull request #8001 from JosephTremoulet/UnrollSimd

Enable unrolling of SIMD_LIMIT loops

Enable unrolling of SIMD_LIMIT loops

Since the Vector<T> abstraction has a `Count` that is not a
C#-compile-time constant, it encourages use of iteration to
search/aggregate individual elements using symbolic indexing, which in
turn leads to codegen that spills the vector to memory for each element
access, and performs bounds checks for each access.  These loops will have
low trip counts that are jit-compile-time constant, and constant indexing
into Vector<T> allows more efficient register-to-register sequences and
bounds-check elision.  This change enables RyuJit's loop unroller when
such a loop is discovered, and increases the size threshold to target
optimizing such loops much more aggressively than the unroller's previous
incarnation.

Add a test with a motivating loop to the Performance/CodeQuality/SIMD
suite.

Closes #7843.

Detect overflow in unroller cost computation

Make sure to avoid trying to unroll cases so large as to overlow the cost.

Allow unrolling loops with multiple branches

Lift both the single-exit restriction and the no-internal-branching
restriction.  Share some utilities with the loop cloner to facilitate
this (particularly `CloneBlockState` and `fgUpdateChangedFlowGraph`).

Unroll loops in inner-to-outer order

There's no need for fixpoint iteration; the loop indices are a pre-order,
so walking them in reverse order will visit inner loops before outer ones.

Stop expecting ASG_ operators in loop unroller

Expect instead to see arithmetic nodes that are arguments of separate
assign nodes.

Merge pull request #7788 from AndyAyersMS/InlinePins

Inliner: support inlining of methods with pinned locals

Update .gitmirror metadata

We no longer have the need to mirror anything into TFS (since we do all
work in the open now) except for the JIT and SuperPMI which want to
continue to use some validation that we can only do out of TFS.

Merge pull request #8042 from kouvel/Api

Expose some things for ns2.0

Expose some things for ns2.0

Mostly:
- Exposed flow-related methods on `ExecutionContext`
- Enabled and exposed `SynchronizationContext` wait overriding

Realted issues:
- dotnet/corefx#11638
- dotnet/corefx#11633
- dotnet/corefx#11635
- dotnet/corefx#11636
- dotnet/corefx#13484

Save registers xmm8..15 on Mac OSX (#8078)

On x64, JIT can generate code that uses all 16 xmm registers.
However, on Mac OSX, we currently only save 8 of these registers.
Thus after a context save/restore, xmm8 through xmm15 are
corrupted. This commit fixes the code to save all 16 xmm
registers. It resolves issue #2266.

Enable time zone for netstandard1.7 (#8074)

* Expose and reimplement TimeZone class

The change is exposing the missing TimeZone APIs from corelib so can be useded in netstandard 1.7.
The change here is re-implementing CurrentTimeZone class to be simpler and also to make it work for Linux platforms. we used TimeZoneInfo class to reimplement it and also cleaned up all used internal calls as not needed anymore.

* Small delta fix

* Fix the code according to the review comments

SOS: Automatically find a managed thread with an exception on Linux/OSX core dumps (#8075)

SOS: Automatically find a managed thread with an exception on Linux/OSX core dumps

To be used in the debugger tests repo for SOS dump testing on Linux/OSX.

Added the "clrthreads -managedexception" option to switch to the first managed thread
that has thrown an exception.

Issue #6518

Format the code and disable the regression test.

The regression test currently fails due to issue #8077.

Merge pull request #8056 from adiaaida/updateFormattingJobPatch

Create patches in formatting job

README.md is the idiomatic approach for github (#8076)

Create patches in formatting job

In the formatting job, if there are formatting errors, we will run the
formatter in --fix mode and generate a patch which will be uploaded to
azure storage.

Merge pull request #8046 from rahku/_resolveAssembly

Expose AppDomain::AssemblyResolve

Enable binclash logger

Expose AppDomain::AssemblyResolve

Update IntrospectionExtensions.cs (#8029)

1. A few tweaks were made to follow the [Coding Style](https://github.com/dotnet/corefx/blob/master/Documentation/coding-guidelines/coding-style.md).
2. A `null` check was done after doing a simple cast (`(IReflectableType)type`), but `rcType` cannot be `null` (it could only be `null` if a [`as`](https://msdn.microsoft.com/en-us/library/cc488006.aspx) cast was done).

BinaryReader: Use cached default encoding instance (#8017)

A new instance of UTF8Encoding used to be created every time
BinaryReader.ctor(Stream) was called, which created an instance of
UTF8Encoding that has no preamble. However, BinaryReader does not use
the preamble at all, so it doesn't matter if the encoding has a preamble
or not. Thus, the cached Encoding.UTF8 instance can be used (which has a
preamble).

Update CoreClr, CoreFx to beta-24709-03, beta-24710-01, respectively (#7922)

Remove unneeded field from BinaryWriter (#8015)

The field only exists in .NET Framework for serialization compat. It
isn't needed on CoreCLR.

StreamWriter/BinaryWriter: Use same cached encoding instance (#8018)

StreamWriter lazily allocates and caches a UTF8NoBOM instance, while
BinaryWriter always allocates new instances. Instead, the same cached
instance can be shared between both writers.

Avoid repeated delegate allocations in FileStream.FlushAsync (#8048)

Avoid the `Action<object>` allocation on each invocation of
`FileStream.FlushAsync` on Windows. This change makes the Win32
implementation match Unix.

ARM-CI : Fix segmentation faults on running tests (#8019)

* ARM-CI : Fix segmentation faults on running tests

To make sure the reason of test failure ramdomly( #6298 ),
We checked segmentation faults occurred from mounted rootfs and the multi thread processing.

So I changed root-fs to the archived root-fs and run tests with --sequential option.

PS. The location of root-fs folder was changed from '/opt' wrote on reverted commit(#7991) to '/mnt' for resolving no space issue.

Remove unnecessary newline in WriteLine (#8047)

Add missing ref emit enum members for ns2.0 (#8062)