Merge branch 'master' into pipeline-work

Enable negative type-check results in CoreLib version bubble (#23549)

This negative type-check results are necessary to generate good code for AsyncTaskMethodBuilder.AwaitUnsafeOnCompleted

re-add test exclusion for #23545 (#23598)

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

- Microsoft.NETCore.App - 3.0.0-preview4-27529-11

Update dependencies from https://github.com/dotnet/arcade build 20190329.2 (#23592)

- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19179.2
- Microsoft.DotNet.Helix.Sdk - 2.0.0-beta.19179.2

Fix ARM64 isFullyImplementedIsa (#23590)

Update dependencies from https://github.com/dotnet/core-setup build 20190329.11

- Microsoft.NETCore.App - 3.0.0-preview4-27529-11

Implement IEquatable to Memory and ReadOnlyMemory (#23586)

Clean up AssemblyLoadContext30Extensions test (#23581)

Build test against corefx ref facade
Enable Assembly.Load(byte[], ...) test

Merge pull request #23579 from sandreenko/removeFedoraRuns

Remove Fedora runs.

Increase GCStress test timeouts. (#23576)

New machines are slower that we had before, so increate the timeout for the longest runs.

Make tracelogging GCStressIncompatible. (#23578)

A new way of tracking variables (#23373)

* Defining VariableLiveRange class

* Adding some typedefs to avoid rewriting

* Defining VariableLiveDescriptor class

* Initializing VariableLiveRange structures before BasicBlock code is being generated

* Getting a siVarLoc for variable homes from a given LclVarDsc and stack level

* Defining VariableLiveKeeper class

* Reporting VariableLiveRanges on changes of variable livenesss or variable homes

* Adding USING_VARIABLE_LIVE_RANGE flag to enable disable VariableLiveRange

* Send VariableLiveRanges to debugger

* Reporting variable homes on prolog

* Wrong argument

* Miss to change variable homes count before sending them to debugger

* Adding dumper of VariableLiveRanges for each blocks and end of code generation

* Close all open VaribleLiveRanges on last BasicBlock

* Changing order of properties initialization on VariableLiveRange constructor

* Type error on assignation

* Rephrasing comments, moving dumps and fixing typos

* Changing const VARSET_TP* for VARSET_VALARG_TP on args

* Variable home was variable location in VariableLiveRange context

* Rephrase and rename of VariableLiveKeeper properties

* Missing some renames

* Adding const where BasicBlock should not be modified

* siBeginBlock and siInit have support for debug code for VariableLiveRange and siScope info

* Adding USING_VARIABLE_LIVE_RANGE flags on methods definition.

* Variable home -> variable location

* Renaming and rephrasing names and uses of VariableLiveRange

* Moving LiveRangeDumper ctor to class declation

* Removing destructors

Signed-off-by: Brian Bohe <brianbohe@gmail.com>
* Removing blank spaces and reordering functions inside class definition

Signed-off-by: Brian Bohe <brianbohe@gmail.com>
* Miss to increment the index after refactoring

* Logic for keeping the last BasicBlock end IL offset is shared between siScope and VariableLiverange for debug code

* Missing to print on debug the last block VariableLiveRanges

* Avoid updating VariableLiveRange when unspilling and dying at the same assembly instruction

* Rephrasing #ifs and #ifdefs

* Calling VariableLiveKeeper in one line

* Avoid copying siVarLoc on genSetScopeInfo

* Removing unused args from eeSetLVinfo

* Changing VariableLiveKeeper ctor

* Typo

Signed-off-by: Brian Bohe <brianbohe@gmail.com>
* Updating VariableLiveDescriptor ctor

Signed-off-by: Brian Bohe <brianbohe@gmail.com>
* Error on first argument

Signed-off-by: Brian Bohe <brianbohe@gmail.com>
* Changing reference for pointer

Signed-off-by: Brian Bohe <brianbohe@gmail.com>
* Renaming assembly offset -> native offset

* removing unnecesary comments and asserts

Signed-off-by: Brian Bohe <brianbohe@gmail.com>
* Update VariableLiveRange dump message

Signed-off-by: Brian Bohe <brianbohe@gmail.com>
* Moving VariableLiveRanges classes inside VariableLiveKeeper

* Wrong flag name

* Adding documentation about how we track variables for debug info

Signed-off-by: Brian Bohe <brianbohe@gmail.com>
* Adding opened issues to doc file

Signed-off-by: Brian Bohe <brianbohe@gmail.com>
* Changing dump tittle

Signed-off-by: Brian Bohe <brianbohe@gmail.com>
* Renaming VariableLiveKeeper property

Signed-off-by: Brian Bohe <brianbohe@gmail.com>
* Update documentation

Signed-off-by: Brian Bohe <brianbohe@gmail.com>
* Updating comments on flags

Signed-off-by: Brian Bohe <brianbohe@gmail.com>
* Setting Scope Info as default way of tracking variables for debug info

Signed-off-by: Brian Bohe <brianbohe@gmail.com>

Fix HW intrinsic containment bugs (#23558)

* Fix HW intrinsic containment bugs

For the Fma case (#23430), fix the handling of contained 3-operand HW intrinsic nodes.
For the Bmi case (#23534), fix a bad assert placement, and re-enable the Bmi tests.

Fix #23530
Fix #23534

* Add guard for Fma test

Assembly.Load ALC name (#23574)

Use normalized path for ALC name

Remove Fedora runs.

GCStress fails there because it can't use external disasembler when it is available in CoreRoot.

Merge pull request #23533 from briansull/Issue_23441

Fix for issue 23411

Increase GCStress test timeouts.

New machines are slower that we had before, so increate the timeout for the longest runs.

Rework Debug/Release builds
Revert Check build changes

Use GenTreeStmt* where it is implied. (#22963)

* Extract `impAppendStmt` and `impExtractLastStmt`.

* Delete `BEG_STMTS` fake stmt.

Use new functions to keep the list updated.

* Retype `impTreeList` and `impTreeLast` as statements.
Rename `impTreeList` and `impTreeLast` to show that they are statements.

* Fix fields that have to be stmt.

* Start using GenTreeStmt.

Change `optVNAssertionPropCurStmt` to use GenTreeStmt.
Replace `GenTree* stmt = block->bbTreeList` with `GenTreeStmt* stmt = block->firstStmt()`.
Save results of `FirstNonPhiDef` as `GenTreeStmt`.

* Replace do-while with for loop.

* Change type inside VNAssertionPropVisitorInfo.

* Delete unused args fron `optVNConstantPropOnTree`.

* Update fields to be stmt.

Update optVNConstantPropCurStmt to use Stmt.
Change `lvDefStmt` to stmt.
Update LoopCloning structs.
Update `optDebugLogLoopCloning`.
Make `compCurStmt` a statement.
Update declaration name in `BuildNode`.

* Clean simple cpp files.

Clean valuenum.
Clean ssabuilder.
Clean simd.
Clean optcse.
Clean loopcloning.
Clean copyprop.
Clean optimizer part1.

* Start cleaning importer, morph, flowgraph, gentree.

* Continue clean functons.

Clean assertionprop.
Clean morph.
Clean gentree.
Clean flowgraph.
Clean compiler.
Clean rangecheck.
Clean indirectcalltransofrmer.
Clean others.

* Create some temp stmt.

* Delete unnecessary noway_assert and casts.

* Init `impStmtList` and `impLastStmt` in release.

* Response review 1.

Simplify and unify Vector64/128/256 platform-agnostic intrinsic handling (#23028)

* Simplify and unify Vector64/128/256 platform-agnostic intrinsic handling

* Removed unsupported ISAs

Rework the if statements

clone the whole code block and have two separate blocks for jobs
 - one for coreclr-ci and another for coreclr-outerloop.

Use char16_t for *_W macros in corhdr.h on Linux/MacOs (#23559)

It fixes COR_CTOR_METHOD_NAME_W and many other macros on Linux
to be of type char16_t[] (not wchar_t[]) by using W("") instead of L"".

Bug: #21977

exclude failing test. (#23566)

Update dependencies from https://github.com/dotnet/corefx build 20190328.7 (#23555)

- Microsoft.NETCore.Platforms - 3.0.0-preview4.19178.7
- Microsoft.Private.CoreFx.NETCoreApp - 4.6.0-preview4.19178.7

Remove / move to outer loop:
coreclr-ci (Test Pri0 Linux_musl arm64 checked)
coreclr-ci (Build Linux_musl arm64 debug)
coreclr-ci (Build Linux arm64 debug)
coreclr-ci (Build Linux_musl arm64 release)
coreclr-ci (Build Linux arm debug)
coreclr-ci (Build Linux_rhel6 x64 checked)
coreclr-ci (Build Linux_rhel6 x64 debug)
coreclr-ci (Build Windows_NT arm64 Debug)
coreclr-ci (Test Pri0 Linux_rhel6 x64 checked)
coreclr-ci (Build Linux_musl x64 debug)
coreclr-ci (build OSX x64 Debug)
coreclr-ci (build OSX x64 Release)
coreclr-ci (Build Windows_NT arm Debug)
coreclr-ci (build Linux x64 Debug)
coreclr-ci (build Linux x64 Release)
coreclr-ci (Build Linux arm release)
coreclr-ci (Build Windows_NT x64 Debug)
coreclr-ci (build Windows_NT x86 Release)

Correctly marshal structure return values in member functions on Win-x64 and Win-x86 (#23145)

* In Windows-x64, if we have a native member function signature with a struct return type, we need to do a by-ref return.

* Implement support for marshalling structure return values via a return buffer argument correctly in instance signatures on AMD64-Windows.

* Change field initialization ordering to satisfy warning.

* Try to narrow down the conditions that trigger these changes to just COM methods.

* Don't bash the return type on AMD64 Windows. Only treat it as a byref return buffer.

* PR feedback.

* Enable returning structs from COM methods via a return buffer on x86 for structs <= 8 bytes.

* Add test for struct returns with ThisCall. Extend the "struct return buffer" fix to functions marked as unmanaged thiscall since they all must be instance methods

* Don't include the return-type-bashing switch on AMD64 platforms.

* Don't do the signature swapping/copy on non-instance functions with struct returns.

* Cast the return type of GetStubTargetCallingConv to the right calling convention enum type.

* If we're doing a thiscall, marshal the "this" parameter before the return buffer (if the return buffer exists) on all platforms.

* Remove temporary logging code I added in for debugging.

* Clean up class naming.

* Try using a vtable instead of a pointer-to-member-function.

* Remove delete of class with non-virtual destructor

Merge pull request #23423 from franksinankaya/gcc_cleanup_16

SSE abstraction for GCC and abstract __if_exists

Change Timer implementation on Unixes to use only one scheduling thread (#7071)

* Change Timer implementation on Unixes to use only one scheduling thread

- Separated from https://github.com/dotnet/corert/pull/7066

* Address feedback from https://github.com/dotnet/corert/pull/7066

* Remove reference to s_lock

* Reduce work inside lock

* Move _id

* Fix duplicate timers in scheduled timer list, move info to TimerQueue

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

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

- Microsoft.NETCore.App - 3.0.0-preview4-27528-14

Update dependencies from https://github.com/dotnet/arcade build 20190329.1 (#23554)

- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19179.1
- Microsoft.DotNet.Helix.Sdk - 2.0.0-beta.19179.1

if exists

llvm

Unify clang with GNUC

Enable SSE for GCC

 Updating BuildTools, CoreCLR to preview4-03828-01, preview4-27528-71, respectively (#23536)

* Removing FeedTasksPackageVersion from dependencies.props
* The corresponding metadata was removed in https://github.com/dotnet/coreclr/pull/22884

* Updating BuildTools, CoreCLR to preview4-03828-01, preview4-27528-71, respectively

* Adding a ! in String.Searching.cs

Avoid boxing allocations for async in Tier0 (#22984)

Delete simple unused vars2 (#23491)

* Clean `valuenum.cpp`.

* Clean `emitarm64.cpp`.

* Clean `lsraarm64.cpp`.

* Clean `lsraarmarch.cpp`.

* Clean `lowerarmarch.cpp`.

* Clean `lower.cpp`.

* Clean `ssabuilder.cpp`.

* Clean `simd.cpp`.

* Clear `simdcodegenxarch.cpp`.

* Clean `lowerxarch.cpp`.

* Clean `scopeinfo.cpp`.

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

* Update dependencies from https://github.com/dotnet/core-setup build 20190328.01

- Microsoft.NETCore.App - 3.0.0-preview4-27528-01

Simplify StackFrame class (#23483)

Delete RuntimeThread.cs (#23532)

No longer needed

Improve performance of Marvin32 (#22816)

Exclude failing in PRs tests 2. (#23540)

* exclude failing in PRs tests

* And another one.

* Exclude for all platforms.

Delete `setup_coredis_tools` from runtest.py. (#23495)

* Delete `setup_coredis_tools` from runtest.py.

* delete dead `setup_coredis_tools` .

Fix for issue #23441
Added test case GitHub_23411

Fix increment calculation in IncrementingPollingCounter (#23502)

* Add test for IncrementingPollingCounter

* Fix a bug in Increment calculation in IncrementingPollingCounter

* Remove setting DisplayName property since that's a private property for now

* fix comment

* Remove unused variables

Exclude failing in PRs tests. (#23535)

* exclude failing in PRs tests

Fix codegen for StoreNonTemporal (#23511)

* Fix codegen for StoreNonTemporal

Also, add some asserts and mark some intrinsics as not supporting containment.

Fix #23509

Restrict Ubuntu.1404.Arm32.Open from running against PRs (#23523)

Struct & SIMD improvements (#22255)

* [WIP] Struct & SIMD improvements

- Enable CSE of struct values when handle is available (and add code to get the handle of HW SIMD types)
- Don't require block nodes for SIMD assignments
- Don't set `GTF_GLOB_REF` on `GT_OBJ` if it is local
- Set `lvRegStruct` on promoted SIMD fields
- Add tests for #19910 (fixed with this PR) and #3539 & #19438 (fixed with #21314)
- Additional cleanup

Fix #19910

Fix Helix queue names (#23468)

* Fix Helix queue names

* Fix open Deb9 queue name.

Switch to BYOC BuildPools in internal project (#23501)

* dnceng-linux-internal-temp -> BuildPool.Ubuntu.1604.Amd64
* dotnet-internal-temp -> BuildPool.Windows.10.Amd64.VS2017

Merge pull request #23519 from BruceForstall/RestoreWindowsArmBuildJobs

Restore Windows arm32/arm64 innerloop build jobs

Nullable: System.Object (#23466)

Nullable: all calendars (#23469)

* calendar

* nullable: all calendars

* fix likely corert error

Nullable: System.Char (#23480)

Restore Windows arm32/arm64 innerloop build jobs

These are required by the innerloop flow jobs.

Merge pull request #23325 from VSadov/CmpExchA64

Resolve 22303   (interlocked failures on ARM64)

GetPinnableReference on String.cs (#23428)

* GetPinnableReference on String.cs

* Add attributes for debugger and performance

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

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

- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19177.11
- Microsoft.DotNet.Helix.Sdk - 2.0.0-beta.19177.11

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

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

- Microsoft.NETCore.Platforms - 3.0.0-preview4.19178.1
- Microsoft.Private.CoreFx.NETCoreApp - 4.6.0-preview4.19178.1

Move QueryPerformanceCounter PInvokes to shared (dotnet/corefx#36409)

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

Nullable updates for byte type (#23460)

Nullable changes for boolean type (#23451)

* Nullable changes for boolean type

Nullable: String (#23450)

nullable: number (#23454)

Nullable: Uint32, (U)Int16, (U)Int64 (#23449)

* Nullable: UInt32

* (U)Int16, (U)Int64

* Make NumberFormatInfo non-nullable

Nullable: Int32 (#23443)

* Nullable: Int32

* remove restore and move nullable up front

* remove trailing \n after #nullable to make it easier to remove later

ALC Enhancement nits (#23506)

isCollectible default should be false
String should be string

ALC enhancements (#22273)

* Initial implementatation of ALC enhancements

Add public constructor with name
Add Name property
Add Assemblies property
Add All property
Override ToString
Name Default and Assembly.Load(...) ALCs
Add AssemblyLoadContext30Extensions test

Extend WindowsEventLog test for EntryPointFilter (#23178)

* Extend WindowsEventLog test

Verify EntryPointFilter solves logging problem with native host swallowing exceptions

WindowsEventLog only look at new entries
WindowsEventLog remove time check

Add mechanism to corhost to emulate host swallowing all exceptions

* PR feedback

Update DllMap doc (#22804)

* Update DllMap doc

Update DllMap design doc:
* To note the NativeLibrary APIs that can be used to implement DllMap
* To point to a sample implementation of DllMap.

Add ALC.ContextualReflection.md (#23335)

Revert "Delete unused variables in jit. Part 2. (#23481)" (#23488)

This reverts commit 6cb120cc74ca61fc314e3cc43007778d2aafd041.

Delete unused variables in jit. Part 2. (#23481)

* Clean `valuenum.cpp`.

* Clean `emitarm64.cpp`.

* Clean `lsraarm64.cpp`.

* Clean `lsraarmarch.cpp`.

* Clean `lowerarmarch.cpp`.

* Clean `lower.cpp`.

* Clean `ssabuilder.cpp`.

* Clean `simd.cpp`.

* Clear `simdcodegenxarch.cpp`.

* Clean `lowerxarch.cpp`.

* Clean `scopeinfo.cpp`.

* Clean `hwintrinsiccodegenxarch.cpp`.

Eliminate EventPipeBuffer::Close()

Fixing the codegen for x64 intrinsics to use EA_8BYTE where needed. (#23461)

Add call to setup-stress-dependencies.sh to build-test.sh. (#23435)

Update CoreFX arm64 test exclusions. (#23463)

Replacing the /Trusted_Platform_Assemblies switch with a simpler /r switch that can be used multiple times in the command line to provide a list of reference assemblies. Reference assemblies can now be passed as individual file entries instead of a giant string of semi-column separated entries. (#23462)

Use DotNet-HelixApi-Access only in internal project (#23479)

Handle addressing modes for HW intrinsics (#22944)

* Handle addressing modes for HW intrinsics

Also, eliminate some places where the code size estimates were over-estimating.

Contribute to #19550
Fix #19521

Make tests outerloop again

Cleaning up the P/Invoke signatures for QPF and QPC to avoid marshalling and pinning (#36071)

* Cleaning up the P/Invoke signatures for QPF and QPC to avoid marshalling and pinning

* Responding to PR feedback.

* Dropping support for low resolution stopwatch

* Removing BestFitMapping=false

* Removing unneeded unsafe declarations

* Addressing more PR feedback

* Fixing the pal_time header file

* Fixing the Unix P/Invoke signatures for GetTimestamp and GetTimestampResolution to return ulong.

* Fixing GetTimestampResolution to scale the result for HAVE_CLOCK_MONOTONIC

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

Merge pull request #23430 from janvorli/disable-ijw-load-in-collectible-alc

Disable loading IJW assemblies into collectible ALC

Replace Win32 resource reading logic with cross platform implementation (#23363)

* FindResource direct implementation in PEDecoder

* Fixup bugs identified in resource reading

JIT: don't treat whitespace as separator in assembly name lists (#23410)

Only recognize ";" as a separator, so that we can support exclusions or inclusions
of assemblies whose names include spaces.

Impacts the altjit exclude list, and the disasm include list.

Addresses dotnet/jitutils#200.

Mark test GitHub_18582 as optimization sensitive (#23434)

Jitstress on OSX will turn on cloning stress in `Test` and cause stack overflows
cloning the large `GT_LIST` subtree of the call.

See issue #23346.

Disabled new IJW tests

There were couple of IJW tests added since the master I've based my
changes on. Adding disabling of those for unloadability testing.

Disable loading IJW assemblies into collectible ALC

Merge pull request #23413 from luhenry/fix-gh22302-2

Improve detection of CPU limits when running inside a Container

Miscellaneous R2RDump improvements (#23215)

1) In Naked mode, hide the distinction amongst METHOD_ENTRY vs.
METHOD_ENTRY_REF_TOKEN vs. METHOD_ENTRY_DEF_TOKEN as it's not
important for correctness and causes undesirable churn in diffs.
This required propagating the DumpOptions around in a couple
of places.

2) For historical reasons, the DisassemblingTypeProvider had
its own implementations of metadata formatting that is now provided
by MetadataNameFormatter. I have removed at least a part of this
duplication in this change. [It was causing undesirable diffs as
one version used to output type owner assemblies whereas the other
did not.]

Thanks

Tomas

Re-enable Index/Range Jit test (#23415)

typo: Double word "both" (dotnet/corefx#36272)

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

typo: Double word "never" (dotnet/corefx#36300)

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

typo: Double word "to" (dotnet/corefx#36264)

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

typo: Double word "that" (dotnet/corefx#36269)

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

typo: Double word "a" (dotnet/corefx#36271)

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

typo: Double word "in" (dotnet/corefx#36270)

* typo: Double word "in"

* Update src/System.Console/src/System/Console.cs

Co-Authored-By: nschonni <nschonni@gmail.com>
Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

typo: Double word "the" (dotnet/corefx#36274)

* typo: Double word "the"

* Apply suggestions from code review

Co-Authored-By: nschonni <nschonni@gmail.com>
Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

typo: Double word "this" (dotnet/corefx#36273)

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