Delete FriendAccessAllowedAttribute and associated dead code (#15101)

- Preserved commented out FriendAccessAllowed on types and methods that are still used as friends in System.Runtime.WindowsRuntime

Move debugger attributes to shared partition (#15100)

- Break down DebuggerAttributes.cs into more files, apply cleanup done in CoreRT
- Move two outliers (DebuggerStepperBoundaryAttribute, DebuggerVisualizerAttribute) from CoreFX to CoreLib so that all debugger attributes are together

Delete SuppressUnmanagedCodeSecurity on QCalls

Delete unnecesary SuppressUnmanagedCodeSecurity annotations (#15102)

Merge pull request #15069 from stephentoub/coreclrnumbershared

Add TryFormat span-based methods to most primitive types

Use TryFormat in StringBuilder

Avoid string allocations when appending primitives to StringBuilder.  We try to format into the existing array space, and only fall back to using ToString when there isn't enough room and we're going to grow the builder anyway.

Add TryFormat methods to {S}Byte, {U}Int16/32/64, and Boolean

Change FormatFixed to avoid bounds checks

Match the original native code's use of pointers.

Remove now dead code from managed parsing/formatting.

- Remove dead "bigNumber" code.
- Remove custom wcslen function. Use String's.
- Delete dead fcalls from runtime. FormatInt32, FormatUInt32, FormatInt64, and FormatUInt64 are no longer needed.  Delete them and many of the helpers used only by them.

Improve performance of managed formatting code

- Optimize NumberBuffer passing by reference instead of value. It's a large struct of ~50 bytes; copying it around has non-trivial cost.
- Replace formatting StringBuilder with ref struct and stack allocation. Avoids lots of allocation and associated throughput costs.
- Improve perf of 'D' formatting of 32-bit and 64-bit integers.
- Remove array allocations accessing NumberFormatInfo props.
- Accessing array properties like PercentGroupSizes clones the corresponding field.  That's unnecessary here, as we don't mutate the array.
- Remove int[] allocation from NumberToStringFormat. Span makes it easy to start with stack space and grow to an allocated array as needed.
- Improve perf of hex formatting of integers. Including removing some sizable allocations.
- Manually inline several hot functions called in only one place.
- Tweak some range comparisons in ParseFormatSpecifier.
- Avoid large stackallocs in NumberToString{Fixed}. It's incurring non-trivial overheads.
- Tweak perf of ValueStringBuilder. In particular, make Append(string) faster for the single-char case, which is extremely common in integer formatting due to its prevalence in strings in NumberFormatInfo.

Move integer parsing/formatting logic to shared

Moves existing managed parsing code to shared, to be shared with corert.
Takes the managed formatting port from corert and moves that to shared
as well.

Delete unused Thread::YieldTask (#15091)

Update BuildTools, CoreClr, CoreFx, PgoData to prerelease-02217-03, preview1-25918-04, preview1-25917-04, master-20171116-0044, respectively (#15094)

Fix #15057 MemoryStream.CopyToAsync calls Write / WriteAsync with count=0 (#15070)

Rename mscorlib to System.Private.CoreLib in documentation (#15090)

Fixes #15078

Fix up ConfiguredValueTaskAwaitable signature for new compiler checks (#15092)

Fix MacOSX build break (#15093)

Fixes #15005

Remove deprecated Copy Task (#15050)

Enable FEATURE_DATATARGET4 for Unix Arm builds (#15060)

fix for il to native mapping etw events (#15038)

After investigation the failures look unrelated to my changes. Thanks Jan and Noah.

Generic hardware intrinsics throw exception for non-numeric types (#15068)

Do not check stmts after the tail call candidate if the optimization has been aborted already. (#15065)

Fix #15056

Merge pull request #15066 from BruceForstall/CleanUpGroovy2

Fix two typos in trigger phrases

Fix two typos in trigger phrases

Clean up netci.groovy (#15030)

* Clean up netci.groovy

1. Don't repeat entire list of JIT stress and R2R scenarios over and over;
use functions to check for scenario groups.
2. Merge handling of R2R baseline and stress scenarios.
3. Add function to determine if a scenario is a JIT stress scenario.
4. Fix JIT stress mode display name generator to handle CoreFx tests
as well, preserving the way they are currently displayed.
5. Fix R2R display name generator to handle baseline R2R scenario.
6. Fix bug where x86 R2R baseline job wasn't created.
7. Fix bugs where triggering some jobs, such as R2R jobs, would trigger lots
of other jobs with insufficient trigger strings. Thus, more trigger strings
now have architecture + scenario in them. E.g., "Innerloop" jobs now much
be triggered using a phrase containing "Innerloop".
8. Simplify code that creates builds, and test run commands.
9. Fix mis-casing of COMPlus_JITMinOpts in R2R stress list.
10. Expand list of valid ARM Windows scenarios to match total list of
scenarios, but with some scenarios commented out. Add the
jitstress2 + jitstressregs to the acceptable list. Add note about the
many scenarios in this list that are not in the full scenario list. Add
function to determine if a scenario is in this list.
11. Support crossgen of frameworks using altjit
12. Don't support 'r2r' scenario on arm/armlb/arm64 Windows.

fix format (#15064)

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

Merge changes from TFS

Fix desktop build breaks.

[tfs-changeset: 1681307]

Fix 14455: _opt_relrotarg_valref test failure on amd64. (#15032)

* fix the issue

Do not zero-initialize temps, that were created by jit and do not have GC refs.
The confusion was that `varDsc->lvIsTemp` means `short live` variable, when we wanted to ask is it IL temp or not.

* add a non-stress repro.

don't use Compiler:: if we are currently in this namescape. (#15045)

I was confused that we were outside of Compiler::

HashCode based on xxHash32 (#14863)

* HashCode based on xxHash32

Works by maintaining the xxHash32 state variables (v1 -> v4, length) as well as a queue of values that fall outside of the block size (16 bytes/4 ints). The seed is initialized to random bytes.

Further details, unit tests and history: https://github.com/dotnet/corefx/pull/25013

Added EmitCalli overload (#15040)

Improve SSA dominator computation memory usage (#14965)

* Improve SSA DF computation memory usage

DF(b) can be stored in a vector instead of a hashtable (set). Nothing needs a O(1) membership test, the duplicates that may be generated during DF construction can be easily avoided by observing that:
* each block in the graph is processed exactly once
* during processing the block is added to the relevant DF vectors, at the end of each vector. If the same DF vector is encountered multiple times then checking if the block is already a member is trivial.

* Use the same block BitVec for topo sort and IDom

* Add BitVecOps::TryAddElemD

Makes the code more readable and avoids the duplicated IsShort test that a separate IsMember/AddElemD may generate.

* Improve SSA IDF computation memory usage

Like DF(b), IDF(b) can be stored in a vector if duplicates are avoided. This can be done by using a BitVector like TopologicalSort and ComputeImmediateDom already do. It's cheaper than using a hashtable (even though it requires a "clear" for every block that has a frontier).

Also, storing IDF(b) into a vector makes it easier to track newly added blocks - they're at the end of the vector. Using a separate "delta" set is no longer needed.

change labels to use new Ubuntu machine pool (#15043)

Merge pull request #15031 from dotnet-maestro-bot/master-UpdateDependencies

Update BuildTools, CoreClr, CoreFx, PgoData to prerelease-02215-01, preview1-25915-02, preview1-25915-02, master-20171115-1118, respectively (master)

Merge pull request #15039 from fiigii/fix

Fix #15035

Fire tracepoints for methods with no template (#14987)

Adding null check for implicit cast from array/arraysegment to Span (#15044)

Re-enable Standalone GC job, running daily (#15015)

* Re-enable Standalone GC job, running daily

* osGroup is already defined

* Fix failed assertion

Update BuildTools, CoreClr, CoreFx, PgoData to prerelease-02215-01, preview1-25915-02, preview1-25915-02, master-20171115-1118, respectively

Delete move to return spill in the implicit tail call in the inlinee with several returns. (#14945)

* add fgNeedReturnSpillTemp
* fix the issue
* add the repro without stress mode

Fix #15035

Remove out-of-use method and properties from appdomain (#15029)

Not supported on CoreCLR appdomain's method and properties were removed

#15001

Enable Globalization Invariant mode using Environment Variable (#15024)

* Enable Globalization Invariant mode using Environment Variable

* Prefix the environment variable name with DOTNET_

* Address the feedback

Update BuildTools, CoreClr, CoreFx, PgoData to prerelease-02214-03, preview1-25914-01, preview1-25913-03, master-20171113-1530, respectively (#15007)

move fgFixupStructReturn (#14966)

It was done twice during the tail call optimization and in the args morphing. The problem is if tail call optimization was aborted it was not clear was `fgFixupStructReturn` called or not.

JIT: look through GT_RET_EXPR when forming inline candidates

If an inline candidate has args that come from calls that are inline
candidates, the arg trees for the candidate may be GT_RET_EXPRs. Since
the jit inlines in preorder, any arg inlines will have been resolved
by the time we get to the candidate. So, the jit can look back through
the GT_RET_EXPR to get a better handle on the actual arg tree.

Doing this has several small benefits:
* It short-circuits parts of the subsequent GT_RET_EXPR update and so
saves a little bit of throughput.
* It potentially allows for more streamlined arg passing since the actual
node side effects may be less constraining than the placeholder effects on
the GT_RET_EXPR (which must be "worst case").
* It may unblock inlines as actual arg values can be seen when evaluating
the candidate for profitability.

During testing I found cases where looking at the actual arg seemed to
degrade information. This turned out to be a misuse of `argHasLdargaOp`
to indicate that a caller-supplied argument was potentially locally
aliased.

This misuse was blocking type propagation when the actual arg tree was seen to
be an aliasable local reference instead of the formerly opaque GT_RET_EXPR.
To fix this, I split out the aliased arg case with a different flag. All
that the jit needs to ensure in the aliased case is that it does not directly
substitute the arg; the arg must be evaluated at the point of the call. The
type of the actual can safely propagate to the arg as long as the arg itself
is not modifiable.

Added a utility for walking back through GT_RET_EXPRs and updated the
other case where the jit was doing this to use the utility as well.

Closes #14174.

Merge pull request #15014 from fiigii/vexencoding

Change VEX-encoding selection to avoid AVX-SSE transition penalties

Merge pull request #15026 from jashook/re_add_arm64_stress

Add jitstress arm64 jobs again

Merge pull request #14914 from CarolEidt/Fix14665

ARMARCH: no cascaded adds in addr mode

small changes to get coreclr building on FreeBSD again (#14964)

Add jitstress arm64 jobs again

Merge pull request #15023 from BruceForstall/FixJitOpts

Fix RunTest.cmd argument parsing

Change VEX-encoding selection to avoid AVX-SSE transition penalties

Fix RunTest.cmd argument parsing

The "jitminopts" and "jitforcerelocs" options were eating whatever
option followed them.

Merge pull request #15017 from BruceForstall/DisableArm64Tests

Disable arm64 tests that fail in Debug CI runs

Add object null checks in Memory<T> APIs to support default (#14816)

* Add object null checks in Memory<T> APIs to support default

* Changing Empty property to return default value.

* Update use of object null checks

* Addressing PR feedback.

* Removing typeof char check.

* Fix typo

Fix CppCodeGen break with latest XCode (dotnet/corert#4926)

Latest XCode errors on ordered comparison of pointer with integer

Also, fixed some unnecessary always-true comparisons that the C++ compiler emitted warnings for.

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

Merge pull request #14929 from sdmaclea/PR-ARM64-SIMD-genStructReturn

[Arm64] SIMD genStructReturn

Merge pull request #14934 from sdmaclea/PR-ARM64-SIMD-genMultiRegCallStoreToLocal

[Arm64] SIMD genMultiRegCallStoreToLocal

Merge pull request #14975 from sdmaclea/PR-SIMD-genSIMDIntrinsicInit-0

[ARM64] genSIMDIntrinsicInit contain 0

Merge pull request #14976 from sdmaclea/PR-SIMD-genCodeForStoreLclVar-0

[ARM64] SIMD genCodeForStoreLclVar zero init

Adding Memory CopyTo APIs (#15010)

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

Merge changes from TFS

Merge pull request #14944 from sdmaclea/PR-ARM64-SIMD-Partial-Callee-Save-SIMD12

[ARM64] SIMD Partial Callee Save SIMD12

Disable arm64 tests that fail in Debug CI runs

Addresses #15016, until these can be investigated.

Merge pull request #14953 from mikedn/ssa-mem-track

Track actual SSA memory usage

Merge pull request #15013 from BruceForstall/DisableTests

Disable ARM tests

Merge pull request #14970 from jashook/better_smarty_archiving

Better smarty archiving

Disable VectorConvert_r/VectorConvert_ro for ARM

The are now failing due to #14665.

Enable long-running or resource-intensive GC tests

Specifically,
```
foregroundgc
lohfragmentation
536168
```

These are already disabled for arm64 and in issues.targets and the
various non-Windows *.txt files.

Merge pull request #14988 from briansull/fix-523532

Fix VSO 523532 - Possible mismatch in size on GT_EQ created for switch expansion

Merge pull request #14921 from BruceForstall/EnableRyuJitArm32Crossgen

Enable RyuJIT/arm32 crossgen compilation of S.P.C.dll

Merge pull request dotnet/corert#4918 from dotnet/nmirror

Merge nmirror to master

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

Merge pull request #14984 from jashook/add_new_publish_vars

Use new publish vars

Use new publish vars

Better smarty archiving

fix DevDiv521437

[tfs-changeset: 1680966]

check that VM set pAddr == nullptr (#15003)

When VM returns addr != nullptr from getHelperFtn, it must set pAddr to nullptr.

Enable RyuJIT/arm32 crossgen compilation of S.P.C.dll

Currently, an ARM build and CI test run uses the legacy backend
arm32 JIT. This change enables a build of RyuJIT/arm32 as a
cross-targeting protojit.dll (altjit) in the cross tools (placed
next to the x86 hosted crossgen.exe in the build), and uses that
RyuJIT/arm32 cross-compiler to build System.Private.CoreLib.dll.
The CI system when testing RyuJIT/arm32 will use this to test
the crossgen'ed System.Private.CoreLib.dll built by RyuJIT/arm32
for all "arm" architecture jobs.

Merge pull request #14920 from dotnet-maestro-bot/master-UpdateDependencies

Update BuildTools, CoreClr, CoreFx, PgoData to prerelease-02210-05, preview1-25913-02, preview1-25913-02, master-20171110-0018, respectively (master)

Update BuildTools, CoreClr, CoreFx, PgoData to prerelease-02210-05, preview1-25913-02, preview1-25913-02, master-20171110-0018, respectively

Fix sempahore spinning fix (#15002)

A recent fix around SemaphoreSlim's spinning has led to it spinning even when spinning isn't necessary, resulting in measurable overheads when trying to acquire a semaphore that has count available.  This fixes that by putting back the check to avoid spinning if count is known to be available.

Port https://github.com/dotnet/coreclr/pull/13148 to master (#14999)

Updating the xUnit Performance Api with bug fixes to the benchmark scenarios (#14986)

This newer version fixes collecting data on x86 where:
1. The Jit generate methods at runtime and that are not associated with any loaded module
2. Modules are loaded/unloaded multiple times
3. This newer version of the Api added some breaking changes to the Api interface

Allow non-abstract non-public interface methods in ILAsm (#14998)

This is to support the default interface method feature in C#.
See https://github.com/dotnet/csharplang/blob/5971e753a20a32e5f471a5439c88e667e32413f0/proposals/default-interface-methods.md

ARMARCH: no cascaded adds in addr mode

On ARM and ARM64, cascaded `ADD`s in the op2 position of the root ADD are not folded into a single addressing mode. However, in `gtSetEvalOrder` it assumes that they will, so it walks both operands of the addr looking for more `ADD`s. This leads to an assert because the leaf nodes it winds up with are not the same as those returned by `genCreateAddrMode()`.

Fix #14665

Remove unnecessary Exception._exceptionMethodString field (#14992)

Fix implementation of GCToOSInterface::VirtualCommit to ditch the pages (#14979)

The GC depends on the pages to be zeroed out when they are mapped in again

Move '-all' from filename to args in Mac build (#14990)

Fix VSO 523532 - Possible mismatch in size on GT_EQ created for switch expansion

Disable GC test requires much bigger amount of memory than arm32 machine may ever have (#14981)

Merge pull request #14973 from sdmaclea/PR-ARM64-VECTOR-LD-REG-SIZE

[Arm64] Do not change vector load register size

Run clean -all instead of just -b (#14978)

[Arm64] Do not change vector load register size

[ARM64] SIMD genCodeForStoreLclVar zero init

[ARM64] genSIMDIntrinsicInit contain 0

Merge pull request #14974 from jcagme/master

Removing retry parameters since they are not used anymore

Removing retry parameters since they are not used anymore

Added UnixEpoch to DateTime and DateTimeOffset. (#14648)

* Added UnixEpoch to DateTime and DateTimeOffset.

* Use ticks instead.

* Use DateTime.TicksTo1970

* Consolidated UnixEpochTicks