Delete dead code (#25513)

Get rid of warnings while building tests (#25514)

[master] Update dependencies from dotnet/arcade (#25503)

* Update dependencies from https://github.com/dotnet/arcade build 20190629.2

- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19329.2
- Microsoft.DotNet.Build.Tasks.Feed - 2.2.0-beta.19329.2
- Microsoft.DotNet.Build.Tasks.Packaging - 1.0.0-beta.19329.2
- Microsoft.DotNet.Helix.Sdk - 2.0.0-beta.19329.2

* Update dependencies from https://github.com/dotnet/arcade build 20190630.1

- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19330.1
- Microsoft.DotNet.Build.Tasks.Feed - 2.2.0-beta.19330.1
- Microsoft.DotNet.Build.Tasks.Packaging - 1.0.0-beta.19330.1
- Microsoft.DotNet.Helix.Sdk - 2.0.0-beta.19330.1

[master] Update dependencies from dnceng/internal/dotnet-optimization (#25504)

- optimization.IBC.CoreCLR - 99.99.99-master-20190701.1
- optimization.PGO.CoreCLR - 99.99.99-master-20190701.1

[master] Update dependencies from dotnet/corefx (#25497)

- Microsoft.NETCore.Platforms - 3.0.0-preview8.19330.3
- Microsoft.Private.CoreFx.NETCoreApp - 4.6.0-preview8.19330.3

Disable windows formatting job in PRs (#25507)

Temporarily disable this job to stop redness in PRs until #25499 is addressed.

[master] Update dependencies from dotnet/arcade (#25476)

- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19328.2
- Microsoft.DotNet.Build.Tasks.Feed - 2.2.0-beta.19328.2
- Microsoft.DotNet.Build.Tasks.Packaging - 1.0.0-beta.19328.2
- Microsoft.DotNet.Helix.Sdk - 2.0.0-beta.19328.2

Update dependencies from https://dev.azure.com/dnceng/internal/_git/dotnet-optimization build 20190628.3 (#25498)

- optimization.IBC.CoreCLR - 99.99.99-master-20190628.3
- optimization.PGO.CoreCLR - 99.99.99-master-20190628.3

Remove an old CoreFX exclusion file (#25500)

Plus, add comments to the new exclusion file

Update documentation for corefx testing in coreclr repo (#25493)

* Update documentation for corefx testing in coreclr repo

* Apply suggestions from code review

Co-Authored-By: Jan Kotas <jkotas@microsoft.com>
* Update a command line; improve clarity

Re-enable corefx tests (#25490)

Re-enable corefx tests

Fix GCStress issue on x86 and Arm (#25445)

* Fix GCStress issue on x86 and Arm32
  When we write one of the new gc stress instruction, such as INTERRUPT_INSTR_PROTECT_FIRST_RET
  we might be writing it in the epilog region on x86 or ARM as a direct call can be the last
  instruction before an epilog.  (This isn't allowed on x64)
  This fix expands the set of instructions we are allowed by IsMarkerInstr() toinclude these
  newly added gc stress instructions.
  added comment to GetGcMarkerExceptionCode
  Fix DAC build

* Code review feedback

* Additional code review chnages

Fix a mutex abandon case with WaitHandle.WaitAll in the PAL (#25452)

Fixes https://github.com/dotnet/coreclr/issues/25108
- Upon a `WaitAll` when all waits are already satisfied, the abandoned flag is overwritten with the abandoned state of the last wait object in the array
- So if the first wait object is an abandoned mutex and the second wait object is a signaled event, the `WaitAll` succeeds and does not report that anything was abandoned
- Fixed to accumulate into the flag instead of overwriting it

Stop using reflection on pollingcounter tests (#25489)

Don't return TRUE on tracepoint_enabled if LTTng library does not exist on build (#25459)

Use regular static variable for EEConfig storage (#25479)

Fixes #25475

Fix an assertion failure in the thread pool's QueueUserWorkItemCallbackBase (#25482)

Fixes https://github.com/dotnet/coreclr/issues/25242
- The issue could be that there is no memory barrier (or in the wrong place) on the thread reading the value, and an old value could be cached and read

Avoid the multiplication generated by mc.exe (#25454)

Update dependencies from https://github.com/dotnet/corefx build 20190627.9 (#25477)

- Microsoft.NETCore.Platforms - 3.0.0-preview8.19327.9
- Microsoft.Private.CoreFx.NETCoreApp - 4.6.0-preview8.19327.9

Add windows arm32 native vararg excludes (#25429)

* Add windows arm32 native vararg excludes

* Fix typo

arm32_ci_script.sh: Fix tizen armel build (#25087)

Enable parsing multiple providers for LTTngConfig variable (#25451)

* Enable parsing multiple providers for LTTngConfig variable

* Handle case where LTTngConfig==NULL

* renaming some methods

* Add comment about the format of environment variable being parsed

Add test for IncrementingEventCounter (#25462)

* Add test for IncrementingEventCounter

* Some cleanup

* Remove unused using

Move coreclr-corefx pipelines into their own files (#25465)

removed unnecessary WriteLine form a test (#25460)

Merge pull request #25424 from dotnet/darc-master-e048b5ab-4671-4e74-8671-932e8299b22b

[master] Update dependencies from dotnet/arcade

Fix VersionBubbles test (#25392)

The test was not crossgening with large version bubble enabled since
the part of the .csproj that generates part of the .sh/.bat test
running scripts was commented out. There were also bugs in the
commented out code. Also, to ensure that it works, tiered compilation
and jit minopts need to be disables to ensure that the test method
won't be re-jitted.

Don't reload or spill for keepAliveThis (#25441)

Fix #25433

React to new compiler nullability warnings

[master] Update dependencies from dotnet/core-setup (#25426)

- Microsoft.NETCore.App - 3.0.0-preview7-27826-04

Update dependencies from https://github.com/dotnet/corefx build 20190626.15 (#25453)

- Microsoft.NETCore.Platforms - 3.0.0-preview8.19326.15
- Microsoft.Private.CoreFx.NETCoreApp - 4.6.0-preview8.19326.15

Fix SIGSEGV in GC when dealing with large double[] on ARM32 (#25403)

* Fix for #25252
* revert to special casing 8-alignement for double[] in GC helpers for x86

Update linux-instructions.md

Update dependencies from https://github.com/dotnet/arcade build 20190626.44

- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19326.44
- Microsoft.DotNet.Build.Tasks.Feed - 2.2.0-beta.19326.44
- Microsoft.DotNet.Build.Tasks.Packaging - 1.0.0-beta.19326.44
- Microsoft.DotNet.Helix.Sdk - 2.0.0-beta.19326.44

Update dependencies from https://github.com/dotnet/arcade build 20190626.2

- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19326.2
- Microsoft.DotNet.Build.Tasks.Feed - 2.2.0-beta.19326.2
- Microsoft.DotNet.Build.Tasks.Packaging - 1.0.0-beta.19326.2
- Microsoft.DotNet.Helix.Sdk - 2.0.0-beta.19326.2

Hardcode LKG version of ucrtbased.dll (#25444)

* Hardcode LKG version of ucrtbased.dll

* Only use hardcoded UCRT version when search path fails

* Add missing single-quotes

Fix typo & make the code leaner (#25442)

Enum.HasFlag generates bigger IL and depends on complex JIT optimization for a good code.
The classic bit check is strongly preferred accross CoreCLR and CoreFX.

Fix ActivityTracker ES_BUILD_STANDALONE build

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

Tweak nullable attribute per API review (#25443)

Update docs (#25446)

- Delete references to Jenkins CI
- Add note on Squash&Merge

EventPipe events should only be sent to sessions that are listening to the event (#25401)

Fix EncoderNLS / DecoderNLS regression in reporting error index (#25397)

Also fixes incorrect asserts in the Encoding type

[createdump] Write page offset in NT_FILE note (#25384)

Update branding to preview8 in master (#25438)

[master] Update dependencies from dotnet/corefx (#25425)

* Update dependencies from https://github.com/dotnet/corefx build 20190626.2

- Microsoft.Bcl.AsyncInterfaces - 1.0.0-preview7.19326.2
- Microsoft.NETCore.Platforms - 3.0.0-preview7.19326.2
- Microsoft.Private.CoreFx.NETCoreApp - 4.6.0-preview7.19326.2

* Reenable globalization tests

Tweak some annotations on EventRegistrationTokenTable<T> (#25386)

Delegate inspection API in the DBI (#25362)

* Add ICorDebugDelegateObjectValue interfaces
* Expose appropriate HR (CORDBG_E_UNSUPPORTED_DELEGATE)
* Partially DACize DelegateObject
* Add DacDbi method for delegate inspection

Merge pull request #25326 from BruceForstall/CrossPlatTestHost

Expand corefx testing in coreclr repo

Currently, corefx testing is done on Windows/x64.
Expand testing to include:
1. Linux/x64, Linux/arm, Linux/arm64, Windows/x86 platforms.
   Windows/arm should be added after corefx official builds
   publish tests and test manifest. Windows/arm64 should be
   added after corefx tests are published, and we have
   sufficient hardware.
2. All JIT stress modes that are used in normal coreclr testing.

Testing is split into 4 Azure DevOps Pipelines: coreclr-corefx,
coreclr-corefx-jitstress, coreclr-corefx-jitstressregs, and
coreclr-corefx-jitstress2-jitstressregs.

In addition, we now use the corefx meta-package Microsoft.Private.CoreFx.OOB
instead of a list of individual assemblies when constructing
the testhost.

Fix disabling wrong session on EventPipe::Shutdown (#25409)

- It's safe to call EventPipe::Disable during shutdown (eventpipe being disabled)

Fixes link to install SOS on linux (#25420)

Don't show AggressiveInlining items in exception stacks (#25408)

Reverting the ShouldRoundUp logic to not change for custom numeric format strings. (#25400)

Fix incorrect signature of QCall (#25419)

Fix LTTng filtering for runtime providers (#25374)

* LTTng filtering

* Fix Windows build, make IsInitialized check faster

* Cleanup/change default to no keywords/level set

* Keep the default to be the current policy - enable everything

* more cleanup

* No need to check for initialization

* Fix Windows build

Tell the compiler to emit nullable attributes for public API only (#25398)

Fix ReJIT failures (#25377)

emit rundown event for generic methods in R2R images (#25371)

* emit rundown event for generic methods in R2R images

* fix mac build

* Code Review Feedback

Expand corefx testing in coreclr repo

Currently, corefx testing is done on Windows/x64.
Expand testing to include:
1. Linux/x64, Linux/arm, Linux/arm64, Windows/x86 platforms.
   Windows/arm should be added after corefx official builds
   publish tests and test manifest. Windows/arm64 should be
   added after corefx tests are published, and we have
   sufficient hardware.
2. All JIT stress modes that are used in normal coreclr testing.

Testing is split into 4 Azure DevOps Pipelines: coreclr-corefx,
coreclr-corefx-jitstress, coreclr-corefx-jitstressregs, and
coreclr-corefx-jitstress2-jitstressregs.

In addition, we now use the corefx meta-package Microsoft.Private.CoreFx.OOB
instead of a list of individual assemblies when constructing
the testhost.

Remove the Crossgen-specific VSD flag from R2RDump (#25391)

Based on JanK's advice I have removed the VSD flag from R2RDump.
The flag should get never observed by the dump tool as it's just
an intermediate contract internal to the Crossgen ZAP logic.

Thanks

Tomas

Delete DeserializationBlockedException (#25393)

Contributes to dotnet/corefx#36723

Policheck comment adjustment

Update dependencies from https://github.com/dotnet/corefx build 20190624.8 (#25379)

- Microsoft.Bcl.AsyncInterfaces - 1.0.0-preview7.19324.8
- Microsoft.NETCore.Platforms - 3.0.0-preview7.19324.8
- Microsoft.Private.CoreFx.NETCoreApp - 4.6.0-preview7.19324.8

React to compiler respecting nullable attributes on properties

Update dependencies from https://github.com/dotnet/arcade build 20190624.24

- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19324.24
- Microsoft.DotNet.Build.Tasks.Feed - 2.2.0-beta.19324.24
- Microsoft.DotNet.Build.Tasks.Packaging - 1.0.0-beta.19324.24
- Microsoft.DotNet.Helix.Sdk - 2.0.0-beta.19324.24

Make sure EventPipeProvider::m_keywords is accurate after EventPipeConfiguration::Disable (#25358)

JitBench: Run PowerShell with an execution policy (#25259)

PowerShell is useless without this argument in default Windows configurations.

Map C Locale to Invariant (#25373)

ICU map the C locale to en-US_POSIX which has a non desired collation behavior as it doesn't support case insensitive comparison.
This change is to force mapping C locale to Invariant instead.

Updating the zapper to filter out some named intrinsics which are dependent on the ISAs supported by the target CPU. (#25365)

Fix StackChange value for OpCodes.Stelem (#25382)

Fixes dotnet/corefx#37955

Update dependencies from https://github.com/dotnet/core-setup build 20190624.03 (#25380)

- Microsoft.NETCore.App - 3.0.0-preview7-27824-03

Add pr.yml ci.yml internal.yml pipelines to /eng/pipelines (#25385)

Remove outputPath parameter for EventPipe IPC protocol (#25363)

Fix StackFrameIterator::IsValid check race (#25359)

* Fix StackFrameIterator::IsValid check race

During GC stress >= 4, there could be a race when we would compute
bRedirectedPinvoke as false, but before the condition of the following
_ASSERTE is evaluated, the thread that is being walked pushes a
ResumableFrame to the explicit frames stack of that thread in the GC
marker handler.

The fix to prevent this race is to evaluate all the conditions that
formed the bRedirectedPinvoke after the conditions in the _ASSERTE.

Add a note about ReadyToRun to ManagedNativeHeader

Update dependencies from https://dev.azure.com/dnceng/internal/_git/dotnet-optimization build 20190624.3 (#25381)

- optimization.IBC.CoreCLR - 99.99.99-master-20190624.3
- optimization.PGO.CoreCLR - 99.99.99-master-20190624.3

Stop deallocating provider/events on shutdown (#25369)

Brick table (#25349)

Fix brick table logic to fix perf issue in several ASP.NET tests, remove #ifdef FFIND_OBJECT.

What I observed was that some GCs spent a lot of time in find_first_object called from find_object, which is called during stack scanning to find the containing object for interior pointers. A substantial fraction of generation 0 was being scanned, indicating that the brick table logic didn't work properly in these cases.

The root cause was the fact that the brick table entries were not being set in adjust_limit_clr if the allocation was satisfied from the free list in gen0 instead of newly allocated space. This is the case if there are pinned objects in gen0 as well.

The main fix is in adjust_limit_clr - if the allocation is satisfied from the freelist, seg is nullptr, the change is to set the bricks in this case as well if we are allocating in gen0 and the allocated piece is above a reasonable size threshold.

The bricks are not set always set during allocation - instead, when we detect an interior pointer during GC, we make the allocator set the bricks during the next GC cycles by setting gen0_must_clear_bricks. I changed the way this is handled for server GC (multiple heaps). We used to multiply the decay time by the number of heaps (gc_heap::n_heaps), but only applied it to the single heap where an interior pointer was found. Instead, I think it's better to instead set gen0_must_clear_bricks for all heaps, but leave the decay time unchanged compared to workstation GC.

Maoni suggested to remove the #ifdef FFIND_OBJECT - interior pointers are not going away, so the #ifdefs are unnecessary clutter.

Addressed code review feedback:
 - add parentheses as per GC coding conventions
 - use max instead of if-statement
 - merge body of for-loop over all into existing for-loop

Fix broken ETW events (#25364)

Make s_numberOfSessions Volatile as advertisement to other devs that the data is being used across multiple threads and a portion of that usage is lock-free (#25366)

Events with keyword 0 needs to be fired (#25361)

Address last round of nullable feedback - S.Resources.ResourceManager, System.Reflection.Emit and ILGeneration (#25318)

* Address nullable feedback for System.Resources.ResourceManager

* Address nullable feedback for System.Reflection.Emit.ILGeneration

* Address nullable feedback for System.Reflection.Emit

* PR Feedback and clean-up redundant casts

Add /eng/pipelines/*.yml (#25325)

Throw platform unsupported error for windows arm32 varargs (#25320)

* Throw unsupported platform for native vcarargs on Arm32 Windows

* Re-enable baseservices vararg test

Updating Dragon4 to ensure the number buffer always provides a significant digit if one exists. (#25096)

* Updating Dragon4 to ensure the number buffer always provides a significant digit if one exists.

* Changing System.Number.RoundNumber to not round up floating-point numbers.

* Re-enabling the RealFormatterTestsBase CoreFX tests

* Updating Number.RoundNumber to take a isCorrectlyRounded parameter and to use IEEE compliant rounding for floating-point numbers.

* Change SinglePrecisionCustomFormat to 7, ensuring it matches the value used in netcoreapp2.1

Switch from IndexOf(string) to Contains(string) for better readability and performance (#38757)

* Fix performance IndexOf(string) -> Contains(string)

* Make comparison ordinal

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

Make Vector*.ToString() Culture invariant (#25343)

* Remove IFormattable
* Hardcode characters
* Add tests

Multiple bug fixes (#25308)

- Fixes EventPipe does not properly dispose of itself on an active session error #25228
  - On EventPipeSession::ThreadProc, if an error occurs on the IPC streaming, then EventPipe::Disable will be invoked.
- Fixes Prevent EventPipe Sessions IDs from being reused on disconnect #25229

Remove AggressiveOptimization flag from System.Text (#25356)

Based on conversation with Levi, I don't think this flag is needed.

The flag currently controls two things:
* Disqualifies the method from precompilation
* Disables Tier-0 JIT for the method

Removing the attribute allows us to precompile some of the methods (not all, because of `Vector<T>`). It measurably helps improve startup time because many of the attributed methods are complex and take a while to compile.

Once tier-1 JIT kicks in, the generated code should be identical to what we were getting before.

JIT: see if jmp offset will fit in 32 bit displacement (#25348)

On x86 we can always reach any IAT_PVALUE entry via a 32 bit indirect
jump, and proper handling of jmp epilogs depends on this. So check if the
target address is reachable in 32 bits, and if so, use the jmp [addr] form
on x86, and the jmp [rip + disp] form for x64.

Fixes #25345
Fixes #25346
Undoes the assertion change from #25302
Fixes #25286

Update R2R format doc to 3.0 (#25297)

I have updated the various enumerations and tables. I didn't try
to go overboard w.r.t. understanding tables I'm not yet familiar
with, most notably the diagnostic, inlining and profiling tables.

Update dependencies from https://dev.azure.com/dnceng/internal/_git/dotnet-optimization build 20190624.1 (#25354)

- optimization.IBC.CoreCLR - 99.99.99-master-20190624.1
- optimization.PGO.CoreCLR - 99.99.99-master-20190624.1

Update dependencies from https://github.com/dotnet/arcade build 20190623.4 (#25351)

- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19323.4
- Microsoft.DotNet.Build.Tasks.Feed - 2.2.0-beta.19323.4
- Microsoft.DotNet.Build.Tasks.Packaging - 1.0.0-beta.19323.4
- Microsoft.DotNet.Helix.Sdk - 2.0.0-beta.19323.4

Update dependencies from https://github.com/dotnet/core-setup build 20190623.05 (#25353)

- Microsoft.NETCore.App - 3.0.0-preview7-27823-05

Update dependencies from https://github.com/dotnet/corefx build 20190623.4 (#25352)

- Microsoft.Bcl.AsyncInterfaces - 1.0.0-preview7.19323.4
- Microsoft.NETCore.Platforms - 3.0.0-preview7.19323.4
- Microsoft.Private.CoreFx.NETCoreApp - 4.6.0-preview7.19323.4

Extend the assert for zero-offset fields (#25341)

CoreRT has an [actual field](https://github.com/dotnet/corert/blob/ba3ba929c88bd80ec5d4dd9cf6917471ad8fa180/src/System.Private.CoreLib/src/System/Object.CoreRT.cs#L30) for the method table table pointer in `System.Object`. CoreCLR does shenanigans when laying out `System.Object` to leave a pointer-sized hole instead.

I think the zero-offset field is triggering these asserts.

Force secondary await continuations to run asynchronously (#25280)

* Force secondary await continuations to run asynchronously

For performance reasons, await continuations have been invoked synchronously, meaning they're invoked as part of the antecedent task's completion (as long as that task allows it, as long as there's sufficient stack space, etc.)  This generally works out well in the case where there's a single await continuation, which is far and away the common case.  However, it can cause problems in situations where there are multiple await continuations, as those continuations will end up being serialized, which can lead to slowdowns and deadlocks in niche situations.  To address that, this commit backs off a bit.  The first await continuation is still invoked synchronously, but subsequent await continuations are invoked asynchronously, such that they are not blocked by a previously registered await continuation.

* Fix nits

Update dependencies from https://github.com/dotnet/core-setup build 20190622.05 (#25338)

- Microsoft.NETCore.App - 3.0.0-preview7-27822-05

MemoryExtensions ToUpper / ToLower throws for overlapping buffer (#25327)

Update dependencies from https://dev.azure.com/dnceng/internal/_git/dotnet-optimization build 20190623.1 (#25339)

- optimization.IBC.CoreCLR - 99.99.99-master-20190623.1
- optimization.PGO.CoreCLR - 99.99.99-master-20190623.1

Update dependencies from https://github.com/dotnet/arcade build 20190622.2 (#25336)

- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19322.2
- Microsoft.DotNet.Build.Tasks.Feed - 2.2.0-beta.19322.2
- Microsoft.DotNet.Build.Tasks.Packaging - 1.0.0-beta.19322.2
- Microsoft.DotNet.Helix.Sdk - 2.0.0-beta.19322.2