Merge pull request #11484 from pgavlin/DisableLongRunningTests

Disable a few long-running tests.

Update CoreFx to preview2-25310-01 (#11483)

Disable a few long-running tests.

These tests consistently time out under GC stress or heap verification.
Each disabled test is known to complete successfully given adequate
time.

Fix lldbplugin to properly build with lldb 3.9. (#11480)

[x86, R2R] Exclude DevDiv_278523 because it fails in R2R

Look at #11476 for the description.

Merge pull request #11465 from JosephTremoulet/StaticHoist

Stop hoisting statics above cctors

Merge pull request #11351 from sdmaclea/PR-ARM64-RRSS-SSRR

[Arm64] Add emitIns_R_R_S_S & emitIns_S_S_R_R

delete tabs from the exclude list

Stop hoisting statics above cctors

Extend the logic in optLoopHoist around `CLS_VAR` nodes to ensure we don't
hoist any tree that contains a `CLS_VAR` for a field with the flag
`CORINFO_FLG_FIELD_INITCLASS` set, unless we also (identify and) hoist the
corresponding static init helper call.  The previous logic was
insufficient in that it blocked hoisting of singleton `CLS_VAR` nodes, but
not hoisting of trees that contain `CLS_VAR` nodes as sub-trees.

Add flag `GTF_FLD_INITCLASS`/`GTF_CLS_VAR_INITCLASS` so that optLoopHoist
can recall which fields are paired with static init helper calls, and only
block the hoisting of their trees.

Fixes #10780.

Replace tabs with spaces.

Return DevDiv_255294 test to the exclude list for x86

Because it causes out of memory exception in crossgen mode.

Fix static analysis issues (#11466)

Fix static analysis issues

Merge pull request #11406 from sdmaclea/PR-ARM64-ENABLE-FEATURE_TAILCALL_OPT

[Arm64] Enable FEATURE_TAILCALL_OPT

Stop loading crossgen-ed binaries at preferred address on Unix (#11467)

This change removes attempt to load crossgen-ed binaries at preferred
address on Unix and uses whatever address the mmap returns instead.

Fix issue #11446. (#11455)

This issue was a GC hole that was caused by bad IR produced by
recursivelty promoting a struct with the following form:

```
struct S
{
    struct T
    {
        object o;
    };

    T t;
}
```

In this case, the recursive promotion logic created a single lclVar to
represent `S.t` and retyped it using the type of `T.o`, but did not
rewrite IR that treated the single lclVar as a struct (e.g. instances of
`lclFld/st.lclFld Vx` where `Vx` referred to the lclVar that represented
`S.t` were not rewritten an `lclVar/st.lclVar Vx`). This IR in
turn confused the incremental liveness used during code generation,
which caused the stack slot holding `o`'s value to die too early.

This change fixes codegen's incremental liveness to handle this case.

Update CoreClr, CoreFx to preview2-25309-01, preview2-25309-01, respectively (#11427)

Enable build on old Linux (#11414)

This change enables building coreclr on old Linux distros that
don't have PR_SET_PTRACER and don't have _xstate struct in the
standard sigcontext.h header.

Avoid throwing in the globalization managed callbacks (#11454)

* Avoid throwing in the globalization managed callbacks

* Add Assert

FailFast stderr logging with stacktrace (#11397)

Merge pull request #11457 from jashook/arm64_pri1_tests

[Arm64] For arm64 jobs build pri1 tests by default

Merge pull request #11451 from pgavlin/gh11447

Fix issue #11447.

[Arm64] For arm64 jobs build pri1 tests by default

This relates to coreclr #11423, enabling pri1 tests to
be built.

Add assembly level attribute on CoreLib (#11449)

Add the "createdump" to the linux runtime package. (#11441)

Fix issue #11447.

This issue was caused by the code for PInvoke frame initialization reading
from the stub argument register after it had been trashed by prior
instructions. This change fixes this issue by inserting the PInvoke
frame initialization at the beginning of the scratch block into which it
is inserted rather than at the end.

Note that a more correct fix here would be to represent the stub
argument as a lclVar rather than a physical register; that work is
tracked by #11450.

Make CoreCLR work properly under PaX's RANDMMAP (#11382)

Make CoreCLR work properly under PaX's RANDMMAP

Issues:
- The ExecutableMemoryAllocator is used to attempt to map native images into a memory range near libcoreclr so that its helper table can use short relative addresses for jumps to libcoreclr
  - RANDMMAP typically prevents mmap calls with a specific address from reserving memory at the requested address, so the executable memory allocator fails to reserve any memory. When Server GC is enabled, the large GC heap can exacerbate the issue by taking address space near libcoreclr.
- Native images are loaded far from libcoreclr, and now jump stub space needs to be allocated near the native image, but RANDMMAP typically prevents this too
- NGenReserveForJumpStubs is intended to reserve some memory near mapped native images for jump stubs, but that reservation is done with a separate mmap call in the same way as above and RANDMMAP typically prevents this too
- The JIT needs to allocate memory for code that may need to jump/call to a native image or libcoreclr, which may require jump stubs near the code that cannot be allocated. CodeHeapReserveForJumpStubs reserves space in code heap blocks without using a separate call to mmap, so this works, but without this environment variable by default there is still a good chance of failing.
- See https://github.com/dotnet/coreclr/blob/56d550d4f8aec2dd40b72a182205d0a2463a1bc9/Documentation/design-docs/jump-stubs.md for more details

Fixes #8480
- It would be ideal to fix all of the above properly, such that there would never be a need to attempt reserving memory within a certain range. Since we're running out of time for 2.0, I figured the following simpler, temporary solution that should cover most of the practical cases, may be appropriate for 2.0.
- Extended ExecutableMemoryAllocator to reserve address space even when it cannot do so near libcoreclr
- Had ClrVirtualAllocWithinRange use the executable memory allocator to reserve memory for jump stubs when the requested range is satisfied
- This covers a maximum of ~2 GB of executable code and should cover most of the practical cases. Once this space is exhausted, under RANDMMAP, native images loaded later will fail, and for jitted code the environment variable above can be used.

Merge pull request #11445 from jashook/fix_build_break

Add define for FEATURE_SIMD

Merge pull request #11443 from DrewScoggins/CopyFix

Fix copy error in perf script

Merge pull request #11231 from russellhadley/UpdateEHDoc

Update EH WriteThru doc with review feedback.

Update doc with review feedback.

Factor in additional exception cases from Andy and fix some wording from
Joe.

Add define for FEATURE_SIMD

This fixs a build break in the desktop product.

Use profile information to differentiate files

[Arm64] Revise per comments

[Arm64] Add emitIns_R_R_S_S

[Arm64] Add emitIns_S_S_R_R

Minor typo corrected (#11444)

Change xcopy to copy

Fix copy error in perf script

When I merged the PR that enabled the lean runs I added another call to
our run script.  This run script had an assumption that it would only
ever be called once and as such would try and copy two files that had
the same name.  This would cause it to prompt to overwrite or not and
fail the run.  This change makes each copy unique based on the
collection flags that were used for collection.

Make Debugger class static (#11439)

* Make Debugger class static

It's static in the .NET Standard and .NET Core refs, and in CoreRT's
implementation, so make it static in the CoreCLR implementation as well.
Also, minor cleanup while making changes here.

Log Events to EventPipe on Linux (#11433)

* Implement the EventPipe object model for providers and events.

* Plumb Runtime Events into EventPipe (#11145)

Plumb runtime ETW events into the EventPipe.

* Fix bug where all events except for SampleProfiler events were never enabled.

* Plumb EventPipeEventInstance through the EventPipe.

* Implement EventPipeFile and FastSerializer.

* Write event contents to the EventPipeFile.

* Only build EventPipe on Linux.

* Conditionally add a sentinel value marking event end.

* Send SampleProfiler events to the EventPipeFile.

* Fix provider ID printing to JSON file.

* Write the start date/time, timestamp, and clock frequency into the trace file.

* Support unloading of EventPipeProviders.

* Handle failure cases when we can't walk the stack or are shutting down.

* Fix a bug where we pass a null src pointer to memcpy.

Delete dead code (#11431)

Delete moved file

Delete moved files and fix build breaks

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

Move more files to shared CoreLib partition (#11429)

Remove ArrayList (#11426)

Also cleanup FEATURE_NONGENERIC_COLLECTIONS while on it.

Merge pull request #11428 from DrewScoggins/LeanRuns

Fix typo in perf.groovy

Fix typo in perf.groovy

Had a typo where I was passing deafult instead of default as a flag to
the collection harness.  This fixes that.

[Arm64/Unix] Enable FEATURE_USE_SOFTWARE_WRITE_WATCH_FOR_GC_HEAP (#11375)

* [Arm64/Unix] Enable FEATURE_USE_SOFTWARE_WRITE_WATCH_FOR_GC_HEAP
* [Arm64/Unix] Enable FEATURE_MANUALLY_MANAGED_CARD_BUNDLES

Add a GC Reliability Framework job to the CI (#11280)

* Add GC Reliability Framework CI job

* Three changes:
  1) To address offline feedback, update the CI trigger phase document
  and the GC testing instructions
  2) Set the timeout for GC RF CI jobs to 24 hours, so they don't time
  out (they will run for 15 hours by default)
  3) Refactor "scenario == 'gc_reliability_framework' to its own method
  so that new GC RF scenarios can easily and cleanly be added if
  desired in the future

Merge pull request #11425 from jashook/build_arm64_tests

Build tests for arm64 runs.

Add compact entry points for ARM (#11109)

Build tests for arm64 runs.

This has been separated to allow coreclr #11423 to be tested
incrementally in the PR.

Update CoreClr to preview2-25305-03 (#11421)

Return null from CultureData.GetLocaleInfoEx (#11422)

Fix build breaks

Switch over to shared Interop.Globalization.cs on Windows (#3528)

- Match CoreCLR refactoring done for invariant culture
- Use actual unsafe types in the interop signatures instead of opaque IntPtr
- Use Unsafe.As to get a pointer to stack allocated structure instead of allocating GCHandles
- Remove use of NativeCallable for enumeration to make the same code work on both CoreRT and CoreCLR. (I have kelp the NativeCallable annotations commented out so that we can switch back to them once C# gets the intrinsics to make it possible.)

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

Merge pull request #11405 from rartemev/issue_10857_addingseq

Fixed memory instruction selection for integer register which contants  simd8 type in ELT hook prolog

Path.GetDirectoryName() throw exception when path is empty or has only white spaces (#11403)

* Throw exception when path is empty or has only white spaces

Removed Dictionary<TKey, TValue>.GetValueOrDefault Methods (#11388)

* Removed Dictionary.GetValueOrDefault Methods and the method calls in Rtpe.cs,CustomAttribute.cs

* Replaced the Dictionary.GetValueOrDefault Methods

* Used !s_pca.ContainsKey(caType) instead of s_pca.ContainsKey(caType) == false

Add lean run support to CoreCLR perf (#11347)

In the xunit performance harness we added support for a "lean" run which
only runs the stopwatch to collect duration and none of the ETW
providers are turned on.  This adds support in the automation for doing
runs that use this lean feature.

Merge pull request #11411 from justinvp/delegate_opequality

Use ReferenceEquals instead of Delegate.op_Equality for sentinel checks

Update CoreClr, CoreFx to preview2-25305-01, preview2-25305-01, respectively (#11384)

Fixed issue #11383 (#11385)

Merge pull request #11361 from mikedn/imp-strlen-except

Add missing GTF_EXCEPT when importing string length intrinsic

Fix incorrect metadata on SOS.NETCore.dll (#11395) again (#11415)

There was a simple msbuild property.

Issue #11381

Add missing GTF_EXCEPT when importing string length intrinsic

Move UnmanagedMemoryStream and related types to shared CoreLib partition (#11409)

Add new test for COMPlus_TailcallStress=1 bug, currently disabled (#11410)

Add new test for COMPlus_TailcallStress=1 bug, currently disabled

Merge pull request #11401 from wtgodbe/DeleteNis

Rename all xxx.ni.dll's to xxx.dll for Helix Crossgen runs

Fixed memory instruction selector for integer register which contants simd8 type

Update contributing-workflow.md

Refactor arm32/arm64 per-node codegen into functions (#11377)

These new functions can sometimes be shared, as they are identical or
virtually so.

In addition, the driver function genCodeForTreeNode() has been
made common.

Rename all xxx.ni.dll's to xxx.dll for Helix Crossgen runs

Fix incorrect metadata on SOS.NETCore.dll (#11395)

Issue #11381

Update first-class-structs.md

Update first-class-structs.md

Use ReferenceEquals instead of Delegate.op_Equality for sentinel checks

The reference equality check is more efficient than
Delegate.op_Equality.

Remove UnmanagedMemoryStream overflow checks. (#11376)

* Remove UnmanagedMemoryStream overflow checks.
* Catch negative UMS PositionPointers
* Switch UMS PositionPointer exception order

Fix ProjectN build break (dotnet/corert#3517)

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

[Arm64] Enable FEATURE_TAILCALL_OPT

System.IO.Path.GetPathRoot string.Empty or whitespace character string should throw ArgumentException (#11387)

* GetPathRoot string.Empty and string whitespace throws ArgumentException

* throw in next line

* Refined method description

* Unix impl adjusted

* Indentation

* pr feedback

* Delete Path.Unix.cs

* Adding param to ArgumentException

Reenable MapViewOfFile PAL test1 (#11393)

This change reenables the test that was having problems on systems when the /tmp
is created on tmpfs file system which doesn't support direct access (unbuffered).
The fix is to explicitly use /var/tmp instead (and /data/local/tmp/ on Android).

Remove stale section on managed debugging. (#11392)

Merge pull request #11370 from CarolEidt/ILOffsetDump

Dump GT_IL_OFFSET value in hex

Document Linux libnuma dependency (#11367)

Disable symbol package MyGet publish on release/* (#11372)

Delete moved file

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

Merge nmirror to master

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

Update CoreClr, CoreFx to preview2-25303-03, preview2-25303-04, respectively (#11354)

Createdump fixes and cleanup. (#11368)

Fixed "with heap" option to actually include all the heaps. The way /proc/$pid/maps
was parsed missed most of the RW data/heap regions.

Added -u/--full full core dump options.

Removed the useless m_memStatus variable in enummem.cpp.

[Arm64/Unix] Revise JIT_MemSet (#11217)

* [Arm64/Unix] Revise JIT_MemSet

Use DC ZVA
Use stp
Correctly handle short set lengths
Simplify code & pseudo code
Use uint*_t to make pseudo code more readable

Adding System\Runtime\CompilerServices\IsByRefLikeAttribute type (#11369)

described in: https://github.com/dotnet/corefx/issues/19212

Fix some static analysis warnings

Most fixes are just addressing the use of (signed) enum-typed variables
as array indices.  Casting to unsigned allows us to cheaply include the
lower bound checks in the existing upper bound checks.  I would prefer
to force the underlying types of enumerations to be unsigned, but that
is a relatively new C++ feature and could have broader consequences than
I want to risk at this point.

The one other fix to asmparse.cpp just suppresses a warning that, while
technically valid, is not causing a real problem.  Perhaps this will get
better if/when #2305 is addressed.

Dump GT_IL_OFFSET value in hex

Remove COMCoverage::nativeCoverBlock (#11364)

Remove this unused function and the corresponding references in
ecalllist.h

Fix Binder NI to IL fallback (#11341)

* Fix Binder NI to IL fallback

Always try NI first

* Fix Windows warning as error

* [Arm64/Unix] Revise PR per feedback

* More cleanup

Place precompiled SPC.dll in native assets folder. (#11359)

Place precompiled SPC.dll in native assets folder.

Fix load from (#11342)

* Add LoadFrom dependencies to the LoadFrom list

* Add a comment

Implement Type.IsTypeDefinition property on CoreCLR (#11355)

This api was just approved.

https://github.com/dotnet/corefx/issues/17345

Revert "[ARM32/Linux] Copy tests.zip only for CI test and use x86 unittest"

This reverts commit 2ec8b5c2cc6d68ca02eb743e679997a8f11c90aa.