Merge pull request #15133 from BruceForstall/FixMissingLabel

Fix genPrepForEHCodegen for arm64

Merge pull request #15131 from BruceForstall/RemoveX86LegacyBackendTesting

Remove x86 legacy_backend build and CI testing

Version: Use int.TryFormat & StringBuilder.Append(int) (#15132)

Now that `int.TryFormat` is available, along with the more efficient
`StringBuilder.Append(int)`, use them in `Version`.

Merge pull request #15034 from CarolEidt/FixDivRegs

Set reg conflicts on contained IND

Reduce allocations in StringBuilder.AppendFormat for primitive types (#15110)

Use the new `TryFormat` APIs to avoid string allocations for primitive types
inside `StringBuilder.AppendFormat`, used by `string.Format`/interpolated
strings.

Change quotes.

Remove x86 legacy_backend build and CI testing

fix the adding (#15121)

Add a core dump example.

Fix typo (#15129)

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

Set reg conflicts on contained IND

For shift, rotate, mod and div, we were only excluding conflicting registers on the non-fixed operand if it is not contained. However, we still want to do that for the base and index (if any) on a contained IND.
Also, fix the LocationInfoListNodePool to attribute its memory usage to CMK_LSRA.

Fix a few of createdump bugs. (#15079)

1) Fix createdump p_vaddr bias/base problem.
2) createdump was loading the DAC from the directory where createdump was executed from and not from the libcoreclr.so path. Fixed this.
3) Don't use the dac if the libcoreclr.so module is not found.
4) Remove the QI of ICoreDebugDataTarget4 in datatarget.cpp. Forgot to do this when the VirtualUnwind function was removed. Caused crash.

Merge pull request #15120 from sdmaclea/PR-ARM64-SIMD-SIMD12-genCodeForStoreLcl

[Arm64] SIMD12 genCodeForStoreLcl*

Merge pull request #15087 from BruceForstall/FixJmpGsCheck

Fix RyuJIT/arm32 GS cookie check before JMP call

Merge pull request #15084 from BruceForstall/FixMulLongBashing3

Fix RyuJIT/arm32 GT_MUL_LONG node bashing

Merge pull request #15086 from BruceForstall/FixBitCastGenTreeDefn2

Fix GenTree definition for GT_BITCAST

Fix debuggable GenTree (#15081)

* Fix debuggable GenTree

Since the `DEBUGGABLE_GENTREE` hack was introduced, many new
GenTree types have been introduced, and gtstructs.h has shifted
around. This change fixes it so we should see accurate GenTree
vtable pointers for all cases where SetOper gets called.

* Formatting

Update jitbench harness to capture steady state data

Update jitbench SHA to pick up new version that reports steady state
median response time, with fractional digits.

Fix genPrepForEHCodegen for arm64

Enable amd64 code for arm64 as well, to match the expectation of
where labels exist in genReportEH. Fixes #14831, a unique case
during MinOpts (and probably only minopts) where dead code follows
a call/finally block sequence.

Merge pull request #15115 from CarolEidt/Enable14665Tests

Re-enable tests disabled for #14665

Merge pull request #14933 from sdmaclea/PR-ARM64-SIMD-Normalize-Struct-Return

[Arm64] SIMD Normalize struct return type

Merge pull request #12552 from mikedn/switch-bt

Use BT in switch lowering

Merge pull request #15117 from sdmaclea/PR-ARM64-SIMD-SIMDIntrinsicSet-life

[Arm64] SIMD fix SIMDIntrinsicSet* op2 life

[Arm64] SIMD12 genCodeForStoreLcl*

Move MemoryStream to shared CoreLib partition (#15116)

- Apply formating and other minor changes from CoreRT
    - Prefer async implementations from CoreCLR since they are more recent and better optimized
- Apply similar changes to UnmanagedMemoryStream as well to keep the two close

[Arm64] SIMD fix SIMDIntrinsicSet* op2 life

Re-enable tests disabled for #14665

Move TaskToApm to shared CoreLib partition (#15113)

- Get TaskToApm in sync with CoreFX copy and move it to shared CoreLib partition
- Delete redundant __Error file
- Delete remaining uses of InternalBlockCopy and replace it with BlockCopy

Merge pull request #15088 from BruceForstall/FixArmStressLclFlds

Fix STRESS_LCL_FLDS for ARM

Update BuildTools, CoreClr, CoreFx to prerelease-02219-01, preview1-25920-01, preview1-25920-01, respectively (#15099)

Allow ByRef return types in Dynamic Method (#15106)

* Allow ByRef return types in Dynamic Method

* Additional cleanup

Fix two issues in managed number formatting (#15105)

* Fix ValueStringBuilder to return correct array to pool

When growing the builder, it's erroneously returning the new array rather than the old one.

* Fix number formatting to not share static buffer

Was missing [ThreadStatic] on a static, such that parallel formatting of numbers might end up sharing the same buffer.  Oops.  But instead of just adding [ThreadStatic], I've switched to starting with a relatively small buffer on the stack.  It should suffice for the majority of performed formatting, and the ValueStringBuilder will grow for longer needs.

* Work around C# compiler warning about stack space being exposed

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

Use BT in switch lowering

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

Fix STRESS_LCL_FLDS for ARM

The code in lvaStressLclFldCB to force alignment for ARM was
using the ADDR node, not the LCL_VAR node. For LONG vars, this meant
computing an incorrect offset into the morphed BLK type local
(4 instead of 8).

Partially fixes #14862

Fix RyuJIT/arm32 GS cookie check before JMP call

The GS cookie check was using r2/r3 registers, after they had been
reloaded as outgoing argument registers for the JMP call, thus
trashing them. Change the temp regs used to r12/lr, the only
non-argument, non-callee-saved registers available on arm32.

Partially fixes #14862

Remove deprecated Copy Task (#15050)

Fix GenTree definition for GT_BITCAST

Nothing should be defined as GenTreeUnOp; use GenTreeOp instead.

Fix RyuJIT/arm32 GT_MUL_LONG node bashing

For RyuJIT/arm32, during decomposition we can bash GT_MUL to
GT_MUL_LONG, which changes its type from GenTreeOp to
GenTreeMultiRegOp. When we do this, we need to initialize
the additional fields in GenTreeMultiRegOp. If we don't,
we get asserts in JitDumps, and presumably other bad things
happen.

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)