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

Separate Profile=On and Off for perf pipeline (#14963)

* Separate Profile=On and Off for perf pipeline

In pipeline jobs, we can separate these two runs into separate runs.
Doing so decreases the time spent running windows jobs since we are not
running them in sequence. Only run Profile=On scenarios for PRs. This
should reduce the time spent running the PR job.

This change also adds the xunit max iteration parameters.

[ARM64] SIMD Partial Callee Save SIMD12

Merge pull request #14708 from mikedn/rc-ssa

Change RangeCheck to use the available SSA data

Fix build break when VS2015 is not installed. (#14948)

* Fix build break when VS2015 is not installed.

If you ONLY install VS2017 (now the common scneario)
and build coreclr, it will fail  with COM with a Class_Not_Registered error

This is because a build-tools package is depending on an old COM object
that has gone way in VS2017.

While we sort getting a proper fix, this makes the work-around (which
is registering the DLL by hand) discoverable.

See https://github.com/dotnet/coreclr/issues/11305 for details.

* typo

Fix clean.cmd to work if no parameters are passed (#14969)

Merge pull request #14967 from jashook/small_netci_fixes

Fix arm(64) jobs

Fix arm(64) jobs

[Arm64] SIMD genMultiRegCallStoreToLocal

[Arm64] SIMD genStructReturn

Merge pull request #14961 from jashook/arm64_windows_nt_rework

Rework Arm64 Windows jobs to flow jobs

Change RangeCheck to use the available SSA data

The necessary information (the assignment node and the block where it appears) can be obtained from the SSA data, there is no need to build a map of SSA definitions. Doing that consumes almost half of the time taken by RangeCheck and a sizeable chunk of memory.

[Local GC] Unify background GC thread and server GC thread creation (#14821)

* Initial cut, ignoring thread affinity

* Integrate thread affinity

* Affinity for standalone Windows

* Add 'specialness' and the thread name as arguments to CreateThread

* First crack at unified implementation

* Set priority for server GC threads

* Remove unused parameter

* Address code review feedback and remove some dead code that broke the clang build

* Use char* on the interface instead of wchar_t (doesn't play well cross-platform)

* Rename IsGCSpecialThread -> CurrentThreadWasCreatedByGC

* Code review feedback and fix up the build

* rename CurrentThreadWasCreatedByGC -> WasCurrentThreadCreatedByGC

* Fix a contract violation when converting string encodings

* Thread::CreateUtilityThread returns a thread that is not suspended - restarting a non-suspended thread is incorrect

* CreateUnsuspendableThread -> CreateNonSuspendableThread

Merge pull request #14882 from 4creators/cleanall

[Infrastructure] Improve clean.{cmd|sh} by adding git clean -xdf cleanup step, unifying implementation with corefx and adding VBCScompiler.exe kill logic and

Rework Arm64 Windows jobs to flow jobs

This drops our dependency on the internal Arm64CI

Merge pull request dotnet/corefx#25092 from brianrob/missing_define

Add PLATFORM_WINDOWS define for MDT EventSource package

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

Remove Linux perf from pr pipeline job (#14943)

* Remove Linux perf from pr pipeline job

We don't have a lot of linux perf machines and the sheer number of PR
jobs is causing them to be overloaded, so the perf leg is taking too
long. Disable them for prs for now until we can increase capacity.

* Disable baseline jobs

Merge pull request #14954 from dotnet/revert-14722-arm64_windows_nt_rework

Revert "Rework Arm64 Windows jobs to flow jobs"

Revert "Rework Arm64 Windows jobs to flow jobs"

Track actual SSA memory usage

Currently SsaBuilder use comp->getAllocator() in a lot of places so a lot of the memory it allocates lands in CMK_Generic instead of CMK_SSA.

Same for CopyProp phase.

Merge pull request #14722 from jashook/arm64_windows_nt_rework

Rework Arm64 Windows jobs to flow jobs

Rework Arm64 Windows jobs to flow jobs

This drops our dependency on the internal Arm64CI

[RyuJIT/arm32] Fix GC hole in address mode generation requiring temps (#14949)

* [RyuJIT/arm32] Fix GC hole in address mode generation requiring temps

In cases requiring a temp register to compute a portion of an LEA,
where the partial computation involves a GCREF/BYREF base register,
the temp register must be marked as a BYREF, so it can get
updated during a GC.

This fixes a case like this:
```
add r2, r0, r1 LSL 2
ldr r0, [r2+24]
```

where `r2` needs to be marked BYREF.

Related to #14856.

* Formatting

Merge pull request #14906 from stephentoub/add_memory_string

Add string support to ReadOnlyMemory<char>

Merge pull request #14918 from BruceForstall/NoLegacyUnsafeCpBlk

Don't allow unsafe CpBlk for legacy backend

Merge pull request #14935 from sdmaclea/PR-ARM64-SIMD-genCodeForLclFld

[ARM64] SIMD genCodeForLclFld

Add foreach support to Span<T> and ReadOnlySpan<T> (#14922)

Adds a ref struct Enumerator, and a corresponding GetEnumerator method to each.

Merge pull request #14912 from briansull/add-b124443

Added missing JIT test V1-M15-SP2/b124443/b124443.cs

Adding placeholder Span debugger proxy (#14749)

* Adding placeholder Span debugger proxy.

* Remove unnecessary unsafe keyword.

Small number of bug fixes. (#14900)

- Added stability prefix to the scenario benchmark (JitBench)
- Specify output directory to the `run-xunit-perf.cmd` script and avoid the extra step to xcopy files to the archive folder.
- Added a command line parser class to the illink scenario, and changed its behavior where it used to fail when a new command line option passed to xUnit was not recognized.
- Save the output log of the tests into the sandbox-logs folder.
- Updating the label of the machine pool used by the illink scenario

Merge pull request #14942 from jcagme/master

Adding IsPublic to CreateAzureContainer

Adding IsPublic to CreateAzureContainer

Change Sym publish to use UnzipSymbolPackagesForPublish (#14941)

Add missing license header (#14936)

Increase retry attempts in publish (#14940)

Pass in property SkipCreateWindowsPdbsFromPortablePdbs to fix (#14939)

Remove RelativePath from Publish (#14938)

Added missing test JIT/Regression/CLR-x86-JIT/V1-M15-SP2/b124443/b124443.cs
Also added an IL version of this test for regression repro on CoreCLR

Merge pull request #14928 from sdmaclea/PR-ARM64-SIMD-genPutArgStk

[Arm64] SIMD genPutArgStk

Merge pull request #14937 from jcagme/master

Updating BuildTools version

Updating BuildTools version

[ARM64] SIMD genCodeForLclFld

Merge pull request #14861 from sdmaclea/PR-ARM64-SIMD-genFnPrologCalleeRegArgs

[Arm64] SIMD genFnPrologCalleeRegArgs

Remove ConfigurationGroup from Publish Symbols (#14931)