Fix doc comment (#37061)

Looks like a copy/paste error.

Disable additional validation that is run post-build (#37102)

* Disable additional validation that is run post-build
Disable more validation stages that are run post-build. The only
thing left to run during each build at this point is SourceLink validation.
Once that is available, the entire validation stage should be able to be skipped

Rev cmake minimum version from 3.14.2 to 3.14.5 (#34757)

iOS requires 3.14.5 version and we have a special case for it. Since this is a minor version difference, and no distro is particularly providing cmake 3.14.2 package, it is safe to update to 3.14.5 to cover our supported platform matrix.

Discussion: https://github.com/dotnet/runtime/pull/33959#discussion_r396389297
Fixes: https://github.com/dotnet/runtime/issues/33976

Co-authored-by: Alexander Köplinger <alex.koeplinger@outlook.com>

Make URI Unc path recognition more consistent across platforms (#35050)

* Recognize '//' Uris as UNC on Unix

* Add test for Unc equality

* Add test for short Unc paths

* Add test for UNC implicit-explicit roundtrip

* Merge two if statements into one

[browser][wasm][http] Fix System.Net.Http build failure (#37234)

[interp] Remove saving of clause_args from interp state (#37048)

clause_args can be set only from EH. This means that clause_args can be set only in the first executed frame, so there is no need to save it in the interp state. Also make sure we never check for it in the main_loop block, for normal calls and returns. Added exec_frame to the clause_args so we can detect whether a frame is the first frame invoked from EH. We can no longer use clause_args for this purpose because we no longer save it in the interp state.

Makes calls 6% faster

Co-authored-by: BrzVlad <BrzVlad@users.noreply.github.com>

[wasm] Initial addition of Browser WebAssembly support (#35573)

* Add Browser specific files to the project

* Browser specific file modification to not include Quic support

* Add Browser specific support files for Http Handler code

* Add Interop code for JavaScript support

* Remove unused reference

* Add Http handler bindings implementation

- This still needs to have the code implement nullable

* Nullable support

* Add browser files back after upstream merge conflict

* Use attribute Link syntax for Common files to bring in sync with existing format

* Address Missing license header

* Code formatting and removal of comment code.

* Address PR comments.

* Address commit comments

* Add blank line between License and first line of code.  Address comments

* Replace SocketsHttpHandler build for Browser.

- Throws `PlatformNotSupportedException` for properties and methods of the HttpMessageHandler abstract implementation.

* Cleanup SendAsync code when doing the call out to JavaScript Fetch.

- Addresses commit comments.

* Refactor BrowserHttpHandler code.

* Cleanup

* Remove null check as it should be checked in outer classes

* Cleanup var usage to use explicit type.  Address commit comments

- For all of these vars, please replace them with the actual type, except when the type is obvious from a new or explicit cast on the right-hand side.

* Move `.ConfigureAwait(true)` to `.ConfigureAwait(false)`

- Address review comments

* Change accessor of Runtime javascript interop methods.

* Change accessor of Runtime javascript interop methods.

* Address review comments for unused code

* Cleanup leftover debug WriteLines

* Remove the AllowNull attributes as per review comments.

* Update src/libraries/Common/src/Interop/Browser/Interop.Runtime.cs

Co-Authored-By: Marek Safar <marek.safar@gmail.com>
* throw different exception type as per review comment

* use char[] array instead and return new string(array).

- Address review comments

* Initial addition of Browser Interop library

* use char[] array instead and return new string(array).

- Address review comments

* Add project references so the Interop.JavaScript project builds

* Coding Style update

* Split the runtime methods into two modules.

- .Api contains the methods that are only used internally from JavaScript bindings code.

* Move System.Runtime.Interop.JavaScript to \src\libraries

- Address review comments

* Change preprocessor to upper case TARGETS_BROWSER

- Address review comments

* Update src/libraries/Common/src/Interop/Browser/Interop.Runtime.Api.cs

Co-authored-by: Marek Safar <marek.safar@gmail.com>
* Update src/libraries/Common/src/Interop/Browser/Interop.Runtime.Api.cs

Co-authored-by: Marek Safar <marek.safar@gmail.com>
* Update src/libraries/Common/src/Interop/Browser/Interop.Runtime.Api.cs

Co-authored-by: Marek Safar <marek.safar@gmail.com>
* Rename from .Api to .Bridge to address review comments

* Modify .csproj files to reflect .Bridge.cs change.

* Remove unnecessary constant

* Add docs

* Update Bridge link

* Collapse code as per review comment.

* Address review comments on `Task.ConfigureAwait(continueOnCapturedContext: true)`.

- Added comment in source code.

* Address review comment about using PropertyInfo reflection.

- GetMethod("get_Result") is less expensive to use.

* Coding Style - Address review comments

* Part of code style and object documentation to partially address review comments.

* Update src/libraries/Common/src/Interop/Browser/Interop.JavaScript.JSObject.cs

Co-authored-by: Stephen Toub <stoub@microsoft.com>
* Update src/libraries/Common/src/Interop/Browser/Interop.Runtime.Bridge.cs

Co-authored-by: Stephen Toub <stoub@microsoft.com>
* Update src/libraries/System.Net.Http/src/System/Net/Http/BrowserHttpHandler/BrowserHttpHandler.cs

Co-authored-by: Stephen Toub <stoub@microsoft.com>
* Update src/libraries/System.Net.Http/src/System/Net/Http/BrowserHttpHandler/BrowserHttpHandler.cs

Co-authored-by: Stephen Toub <stoub@microsoft.com>
* Update src/libraries/System.Net.Http/src/System/Net/Http/BrowserHttpHandler/BrowserHttpHandler.cs

Co-authored-by: Stephen Toub <stoub@microsoft.com>
* Update src/libraries/System.Net.Http/src/System/Net/Http/BrowserHttpHandler/BrowserHttpHandler.cs

Co-authored-by: Stephen Toub <stoub@microsoft.com>
* Update src/libraries/System.Net.Http/src/System/Net/Http/BrowserHttpHandler/BrowserHttpHandler.cs

Co-authored-by: Stephen Toub <stoub@microsoft.com>
* Add class docs and code style updates

- Address review comments

* Update src/libraries/System.Net.Http/src/System/Net/Http/BrowserHttpHandler/BrowserHttpHandler.cs

Co-authored-by: Stephen Toub <stoub@microsoft.com>
* Fix badly formed XML

* Update src/libraries/System.Net.Http/src/System/Net/Http/BrowserHttpHandler/BrowserHttpHandler.cs

Co-authored-by: Stephen Toub <stoub@microsoft.com>
* Remove unnecessary Property implementations.

* Update src/libraries/System.Net.Http/src/System/Net/Http/BrowserHttpHandler/BrowserHttpHandler.cs

Co-authored-by: Stephen Toub <stoub@microsoft.com>
* Update code style involving the use of `var`

* Modify the underlying sockets field name.

- address review comments.

* Style and address review comments

* Remove reliance on ConnectHelper.cs code as it is not supported on Browser

- This removes the reliance on the QUIC support.

* Update src/libraries/Common/src/Interop/Browser/Interop.Runtime.Bridge.cs

Co-authored-by: Stephen Toub <stoub@microsoft.com>
* Remove null check for httpresponse.Content as it is always assigned to.

- Address review comments

* Address review comments

* Remove methods that will not be implemented right now,

* Add properties back to fix build error

- error : MembersMustExist : Member 'public System.Boolean System.Net.Http.SocketsHttpHandler.XXXXXXXXXXXX.get()' does not exist in the implementation but it does exist in the contract.

* Remove tcs TaskCompletionSource to address review comments

* code style change

* Remove disposing of _abortCts here as it causes runtime errors as being disposed of too early

* Address review comment for GetType()

* Rename library as per discussions

* Add lock around the _boundObjects and _rawToJS access.

- Address concurrency review comments

* Address review comments for disposing of CancellationTokenSource

* Address new HttpContent overloads in .NET 5 that take CancellationToken

* Add project and test structure for JavaScript InteropServices

* Remove previous modification as it is no longer needed.

* Update src/libraries/System.Net.Http/src/System/Net/Http/BrowserHttpHandler/BrowserHttpHandler.cs

Co-authored-by: Stephen Toub <stoub@microsoft.com>
* Remove custom message passed to the PNSE .ctor

* Update `mono` to dotnet or .NET

* Address review comment for internal sealed class

* Address extra message text

* Move these source modules to System.Runtime.InteropServices.JavaScript

* Add interop source modules to System.Runtime.InteropServices.JavaScript project

* Remove placeholder source

* Reference System.Runtime.InteropServices.JavaScript project

* Address review comments Unsafe.SizeOf

* Finish move of sources to InteropServices.JavaScript

* Remove ActiveIssue to address review comments

* Add core implementations

* Add core object implementation for Map

* Remove unused targetframework

* Fix PNSE in HttpClientHandler .ctor

- When creating an instance of the browser webassembly `System.Net.Http.BrowserHttpHandler` the following error is thrown.

```
System.PlatformNotSupportedException: Operation is not supported on this platform.
    at System.Net.Http.BrowserHttpHandler.get_SslOptions()
    at System.Net.Http.HttpClientHandler.ThrowForModifiedManagedSslOptionsIfStarted()
    at System.Net.Http.HttpClientHandler.set_ClientCertificateOptions(ClientCertificateOption value)
    at System.Net.Http.HttpClientHandler..ctor()
    at System.Net.Http.HttpClient..ctor()

```

* Remove extra parameters to `PlatformNotSupportedException`

* Fix tests due to underlying field name changing and reflection being used to obtain the socket field

Co-authored-by: Marek Safar <marek.safar@gmail.com>
Co-authored-by: Stephen Toub <stoub@microsoft.com>

Revert "Enable test for MemoryBarrierProcessWide on arm64. Update comments." (#35974)

* Revert "Enable test for MemoryBarrierProcessWide on arm64. Update comments. (#35817)"

This reverts commit b6341c6e1ada32117639cc409d445fd6b9b9e9f0.

* Update InterlockedTests.cs

rerun CI

Rewrite HashSet<T>'s implementation based on Dictionary<T>'s (#37180)

* Move HashSet to Corelib

* Small style cleanups in Dictionary

And factor out InsertionBehavior into its own file

* Rewrite HashSet based on Dictionary's implementation

This effectively deletes HashSet's data structure and replaces it with the one used by Dictionary, then updated for the differences (e.g. just a value rather than a key and a value).  HashSet used to have the same implementation, but Dictionary has evolved significantly and HashSet hasn't; this brings them to basic parity on implementation.

Based on perf tests, I veered away from Dictionary's implementation in a few places (e.g. a goto-based implementation in the core find method led to a significant regression for Int32-based Contains operations), and we should follow-up to understand whether Dictionary should be changed as well, or why there's a difference between the two.

Functionally, bringing over Dictionary's implementation yields a few notable changes, namely that Remove and Clear no longer invalidate enumerations.  The tests have been updated accordingly.

* Make a corresponding cleanup change to Dictionary

* Use HashSet in Corelib

* Address PR feedback

* Clean up HashSetEqualityComparer

* Port Dictionary's comparer serialization test to HashSet

* Address PR feedback

Fix ActivitySource to log the right amount of stopped (#37211)

* Fix ActivitySource to log the right amount of stopped

* Fix test and typo

Co-authored-by: Chandramouleswaran Ravichandran <chandramouleswaranravichandran@Chandramouleswarans-MacBook-Pro.local>

Add Or int? (#37225)

Update Directory.Build.props (#37218)

Remove workaround for https://github.com/microsoft/msbuild/issues/4474.

[wasm] Include the pinvoke table into a separate file, so the generated declarations don't conflict with the ones in the header files included by driver.c. (#37202)

* [wasm] Include the pinvoke table into a separate file, so the generated declarations don't conflict with the ones in the header files included by driver.c.

Same as https://github.com/mono/mono/pull/19853.

* Update src/mono/wasm/runtime/pinvoke.c

Co-authored-by: Alexander Köplinger <alex.koeplinger@outlook.com>
Co-authored-by: Alexander Köplinger <alex.koeplinger@outlook.com>

Use the wasm version of System.Console on wasm. (#37221)

Reduce Uri ctor overhead (#36915)

* Optimize UriHelper.EscapedAscii

* Contains > IndexOf

* Join CheckForUnicode and CheckForEscapedUnreserved into 1 function

* Rename EscapedAscii to DecodeHexChars

Fix bad documentation links (#37222)

Also, don't make documentation specific to CoreCLR.

Update coverlet version to fix coverage exceptions (#37217)

* Update coverlet version to fix coverage FNFEs

Changelog: https://github.com/coverlet-coverage/coverlet/blob/master/Documentation/Changelog.md#release-date-2020-05-30

-Fix 'The process cannot access the file...because it is being used by another process' due to double flush for collectors driver #https://github.com/coverlet-coverage/coverlet/pull/835

* Update dotnet-tools.json

Reduce duplication in Linux build instructions (#37175)

All the Docker information was duplicated in two places; remove
that duplication.

Add SslStream test for zero-byte reads (#37128)

* Add SslStream test for zero-byte reads

We have higher-level tests for libraries like WebSockets that validate the behavior of zero-byte reads, but we don't currently have one in System.Net.Security's tests.  Adding one.

* Address PR feedback

Define TargetsUnix for wasm. (#37207)

[mono] Enable parallel mode for xunit on Android by default (#37058)

[master] Update dependencies from dotnet/arcade Microsoft/vstest (#37200)

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

Microsoft.DotNet.XUnitExtensions , Microsoft.DotNet.VersionTools.Tasks , Microsoft.DotNet.ApiCompat , Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.Build.Tasks.Feed , Microsoft.DotNet.Build.Tasks.Packaging , Microsoft.DotNet.Build.Tasks.SharedFramework.Sdk , Microsoft.DotNet.Build.Tasks.TargetFramework.Sdk , Microsoft.DotNet.CodeAnalysis , Microsoft.DotNet.GenAPI , Microsoft.DotNet.GenFacades , Microsoft.DotNet.Helix.Sdk , Microsoft.DotNet.RemoteExecutor , Microsoft.DotNet.XUnitConsoleRunner
 From Version 5.0.0-beta.20261.9 -> To Version 5.0.0-beta.20278.4

* Update dependencies from https://github.com/microsoft/vstest build 20200529-01

Microsoft.NET.Test.Sdk
 From Version 16.7.0-preview-20200521-01 -> To Version 16.7.0-preview-20200529-01

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>

Fix fast tail calls on Windows ARM64. (#37192)

LowerFastTailCall has code that looks for incoming stack arguments
that may be overwritten by outgoing stack arguments and copies
them to a temp. The logic was incorrect for Windows Arm64 because it was
assuming shadow space, which only exists in Windows x64 abi.

Fixes #31729.
Fixes #36911.

Delete unused file (#37201)

LowLevelLifoSemaphore: Fix incorrect field offset (#37076)

Fixes #37075

Init dictionary with clear count (#37041)

Use nuint for bytes counter inside SafeBuffer (#37173)

* Use nuint for bytes counter inside SafeBuffer

* One more place to update

Reformat WasmAppBuilder to conform to the coding conventions. NFC. (#37199)

Bump to updated WebAssembly runner from xharness (#37178)

Allow GetReJITIDs calls with NULL to get number of IDs (#37187)

Replace CoreClr issue numbers with Runtime issue links. (#37188)

Also replace tabs with spaces and remove a duplicate entry.

Remove ObsoleteAttribute and EditorBrowsableAttribute (#37184)

Fix Mono C++ build. (#37153)

Remove built-in WinRT support from the runtime (#36715)

fixed a bug in updating the gen0 budget for AllocateUninitializedArray (#37148)

For AllocateUninitializedArray, we don't consume all the memory we get
from the gen0 budget but we are not putting back the part we don't consume.
This artificial consumption in budget can mean we trigger a GC way too early
when we haven't actually allocated that much.

Make H3StaticTable class static (#36222)

* Make class as static

* Update H3StaticTable.cs

* Update H3StaticTable.cs

* Update H3StaticTable.cs

Add activity Ids and Context to log scopes (#37092)

* Add activity Ids and Context to log scopes

* Fix name casing

* Address the feedback

* Address the feedback

* Validate ActivityTrackingOptions input and add test

[interp] Remove assertions from release and add more to debug builds (#37160)

* [interp] Remove some assertions from hot path

We can keep them only for debug builds, where in the future we can add more verbose assertions, like ones for stack overflows.

* [interp] Add more assertions on debug build

Most of the weird interp bugs are due to overflows in the interp stacks. Since we now test interp on CI with debug build, try to catch them early.

Fix fixed dictionary sizes in apple crypto key generation

Both dictionaries had a fixed size of two, however three items were being added to it.
This is undefined behavior in the documentation.

Allow DynamicallyAccessedMembersAttribute on methods (#36340)

Allow DynamicallyAccessedMembersAttribute on methods

Remove AnonymousType ILLinkTrim entry for EventSource (#37083)

Instead of using an anonymous type to do the logging, just pass the string directly.

Contributes to #35199

Update dependencies from https://github.com/mono/linker build 20200529.1 (#37157)

Microsoft.NET.ILLink.Tasks
 From Version 5.0.0-preview.3.20278.2 -> To Version 5.0.0-preview.3.20279.1

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>

[browser][wasm] Add initial webassembly instructions document. (#37123)

[mono] Remove some dead code from Delegate.DynamicInvoke (#37085)

* [mono] Remove some dead code from System.Delegate.DynamicInvokeImpl

Remove some dead code from `System.Delegate.DynamicInvoke`.  It should be the
case that `Method` is non-`null` when we call `DynamicInvoke` - the delegate
method should be known at the time of the invoke, and it should not be picked
based on the arguments.

Fixes https://github.com/dotnet/runtime/issues/37008

Update area-owners.md (#37171)

Merge pull request #37147 from swaroop-sridhar/publishhost

Publish singlefilehost on Linux, Osx, FreeBSD

Fix Linux requirements document (#37145)

Update linux-requirements.md

Fix copy/paste error, update wording related to LLVM APT feed.

Add more verbose logging (#37089)

* log out stdout/stderr of subprocess
* instead of using Process.Kill, gracefully exit

Define _FILE_OFFSET_BITS=64 (#33266)

* Define _FILE_OFFSET_BITS=64

* Set unconditionally for all hosts

* Move forward

Publish singlefilehost on Linux,Osx,FreeBSD

singlefilehost is currently only published in the microsoft.netcore.app.host.win-x64
Add it to host packages for other platforms.

[mono] Implement AssemblyBuilder.GetSatelliteAssembly (#36589)

[mono] Implement AssemblyBuilder.GetSatelliteAssembly

Contributes to https://github.com/mono/mono/issues/14788

Merge pull request #37098 from terrajobst/api-review-process

Add information about API review schedule

Reword section

Update docs/project/api-review-process.md

Co-authored-by: Eric StJohn <ericstj@microsoft.com>

Disable BackgroundWorkerTests.TestFinalization on mono interpreter mode (#37137)

Adding arm64 as a platform for the interpreter tests (#37133)

Co-authored-by: Steve Pfister <steve.pfister@microsoft.com>

[mono] Implement AssemblyBuilder.GetReferencedAssemblies (#36346)

[mono] Implement AssemblyBuilder.GetReferencedAssemblies

GCCoverageInfo support 32-bit Tiered compilation (#37116)

* GCCoverageInfo support 32-bit Tiered compilation

Update the GC coverage info code path to respect Tiered compilation
  on a 32-bit platform.

Reduce ConcurrentDictionary.TryGetValue overhead (#37081)

* Clean up style in ConcurrentDictionary

No functional changes.

* Optimize ConcurrentDictionary for no comparer as well as value type keys

* Special-case StringComparer.Ordinal in ConcurrentDictionary

* Store repeatedly used fields into locals

* Use FastMod in ConcurrentDictionary

* Allow atomic writes of {U}IntPtr

From ECMA 335:
"The native size types (native int, native unsigned int, and &) are always naturally aligned (4 bytes or 8 bytes, depending on the architecture)"
and
"A conforming CLI shall guarantee that read and write access to properly aligned memory locations no larger than the native word size (the size of type native int) is atomic (see §I.12.6.2) when all the write accesses to a location are the same size. "

* Address PR feedback

Add mono interpreter test leg to CI (#35568)

This change enables the interpreter on CI as well as providing an option for the local test run.

[runtime] Fix Mono BStr marshaling, cleanup IL generation (#35804)

* [COM] Set and free BSTR size correctly

Apparently, while the previous 4 bytes contain the size, the actual allocation is either 4 or 8 bytes depending on the platform! See https://github.com/dotnet/runtime/blob/fcd862e06413a000f9cafa9d2f359226c60b9b42/src/coreclr/tests/src/Common/Platform/platformdefines.cpp#L418

* [meta] Cleanup string marshalling and support additional conversions

* [COM] Align BSTR creation to 16 bytes

* Enable test

* Feedback

* Fix offset

* Add support for AnsiBStr and TBStr marshalling

Also makes BStr changes netcore-only and enables AnsiBStr test. No TBStr test appears to exist.

* String marshaling IL gen refactoring

* Fix Windows build

* Feedback

Use dictionary to track PackageRelationships instead of searching a list (#35978)

This change adds an OrderedDictionary<TKey,TValue> type to System.IO.Packaging to keep a list of relationships that have been added, while ensuring that the ids are unique with decent performance characteristics. The current implementation searches a list which gets really slow with large numbers of relationships. A Dictionary by itself does not work for this as the order of addition is important. Thus, this combines a LinkedList to track the order, while a Dictionary is used to for quick look up of existing items.

remove some old Sierra craft in ping test (#36987)

Co-authored-by: Tomas Weinfurt <furt@Shining.local>

Support async KeyValuePair (de)serialization (#36607)

* Support async KeyValuePair (de)serialization

* Address review feedback

* Fix up test

[CBOR] Add Bugfixes, XML Documentation and other refinements (#36770)

* Remove buffer pooling and IDisposable implementation from CborWriter

* rename CborWriter.GetEncoding()/TryWriteEncoding() to Encode()/TryEncode()

* move exception messages to resource file

* rebase on top of latest cbor changes

* move writer exception messages to resource strings

* fix failing tests

* Implement a CborWriter.Reset() method

* remove clearArray argument from Reset() function

* add XML docs for shared CBOR types

* tolerate invalid utf8 encodings in lax conformance levels

* flip default validateConformance value in SkipValue() method

* tidy up CborReader internal state; add more comments

* more CborReader internal state tidying up

* fix strict conformance level rollback bug

* Deprecate CborReader.BytesRemaining property

* Change return type of CborReader.ReadEncodedValye() to ReadOnlySpan<byte>

* factor CborReaderState to a separate file, add xml docs

* Add xml docs for CborReader.cs and CborReader.Integer.cs

* string reader should not throw OverflowException on large indefinite-length

* simplify integer decoding implementation

* add xml docs for CborReader.String.cs

* add missing docs to CborReader.cs

* clean up CborReader.String.cs and add more code comments

* add xml docs for array and map readers

* refine CborReader.PeekTag() error semantics & add xml docs

* add CborReader.Simple.cs xml docs

* add xml docs for CborReader.SkipValue.cs

* remove unnecessary usings

* rename encodeIndefiniteLengths parameter to convertIndefiniteLengthEncodings and change default value.

* add xml docs to CborWriter.cs

* Render major types using their numeric values in exception messages

* fix handling of root-level break bytes when multiple root-level items are permitted

* add xml docs to CborWriter.Integer.cs

* Cleanup CborReader.Simple.cs

* add xml docs to CborWriter.String.cs

* add xml docs for CborWriter array and map methods

* add xmldocs for CborWriter.Tag.cs

* add xmldocs for CborWriter.Simple.cs; refine behaviour and add tests

* add conformance level tests for CborWriter.WriteEncodedValue()

* Update src/libraries/System.Security.Cryptography.Encoding/tests/Cbor/CborReader.Simple.cs

Co-authored-by: Jeremy Barton <jbarton@microsoft.com>
* Update src/libraries/System.Security.Cryptography.Encoding/tests/Cbor/CborReader.Map.cs

Co-authored-by: Jeremy Barton <jbarton@microsoft.com>
* address PR feedback

* Update src/libraries/System.Security.Cryptography.Encoding/tests/Cbor/CborWriter.cs

Co-authored-by: Jeremy Barton <jbarton@microsoft.com>
* Update src/libraries/System.Security.Cryptography.Encoding/tests/Cbor/CborWriter.cs

Co-authored-by: Jeremy Barton <jbarton@microsoft.com>
* Update src/libraries/System.Security.Cryptography.Encoding/tests/Cbor/CborWriter.cs

Co-authored-by: Jeremy Barton <jbarton@microsoft.com>
* address feedback

* have lax conformance level tolerate reserved simple value encodings

* fix xmldoc issues

* minor docs fixes

* add more docs fixes

* remove redundant <see> elements in xml docs

* Remove more <see> elements from <summary> docs

Co-authored-by: Jeremy Barton <jbarton@microsoft.com>

Use friendly name in exception if OID value is null. (#37110)

Update dependencies from https://github.com/mono/linker build 20200528.2 (#37126)

Microsoft.NET.ILLink.Tasks
 From Version 5.0.0-preview.3.20276.2 -> To Version 5.0.0-preview.3.20278.2

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>

JIT: some small improvements to tail duplication (#37038)

1. Allow duplicating when predecessor is BBJ_NONE
2. Require that predecessor and successor reference the same local
3. Make sure that local is not address exposed
4. Check up to two statements in predecessor for local reference
5. Require successor to compare local to constant, or local to local

Changes inspired by #36649 but don't actually improve CQ for that case (yet).

Also, add morph to post-phase whitelist because it seems odd not
to dump the IR after morph.

[runtime] Implement the many argument version of the Vector128.Create () intrinsic. (#37036)

Fix namespace for Nullable converters (#37019)

[wasm] Fix the handling of i8/u8 in get_wrapper_shared_type_full (). (#37056)

Fixes https://github.com/mono/mono/issues/19837.

<!--
Thank you for your Pull Request!

If you are new to contributing to Mono, please try to do your best at conforming to our coding guidelines http://www.mono-project.com/community/contributing/coding-guidelines/ but don't worry if you get something wrong. One of the project members will help you to get things landed.

Does your pull request fix any of the existing issues? Please use the following format: Fixes #issue-number
-->

Co-authored-by: vargaz <vargaz@users.noreply.github.com>

Move thread pool initialization to native side (#36789)

Move thread pool initialization to native side

- Following up from https://github.com/dotnet/runtime/pull/36697. Currently the initialization occurs when a static variable is accessed before calling into the VM, and after thinking about it some more the intent and need to initialize the thread pool can be a bit unclear
- The initialization is specific to this implementation and the managed side doesn't necessarily need to know about it
- Moved the initialization to the native side similarly to other thread pool APIs
- Added some more assertions to relevant managed-to-native entry point paths to ensure that they are not called before the thread pool is initialized
- This adds a non-volatile check when requesting a worker thread, which is already a very slow path. I ran into some issues with testing on the arm64 machine with updated locally built libcoreclr.so, so wasn't able to test that. I looked at the baseline profile and looking at time spent exclusively in ThreadPoolNative::RequestWorkerThread() and what it already does, I don't think the change would result in any significant perf difference.

[wasm] Test runner improvements. (#37097)

* Fix the test runner script.

* Add a ExtraFiles property to the wasm app builder.

* Add helper make targets.

* Bump sdk version to pick up wasm support.

* Fix make targets.

* Compile wasm runtime with --enable-minimal=threads.

Update Microsoft.DotNet.Build.Tasks.SharedFramework.Sdk. (#36889)

* Update Microsoft.DotNet.Build.Tasks.SharedFramework.Sdk.

Unblocks #36715

* Enable creating the LightCommand package drop to fix MSI creation.

* Update shared framework sdk to 20277.7

Add linux arm/arm64 libraries checked coreclr test runs (#36910)

* Add linux arm/arm64 libraries checked coreclr test runs

* Add linux_musl_arm64

* Disable BitmapTests on linux checked coreclr

ARM/ARM64: Optimize virtual call stub for R2R and JIT (#36817)

Optimize virtual call stub for R2R ARM/ARM64 scenarios

Loosen property name collision detection involving hidden properties (#36936)

* Loosen property name collision detection involving hidden properties

* Delay ignored prop cache creation; add more tests

* Clarify comments

Add ICU App-Local support on Mono (#37037)

* Add ICU App-Local support on Mono

* PR Feedback

Fix multi-reg copy (#37062)

Also, fix dump for multi-reg copy.

Fix #37059

Add information about API review schedule

[master] Update dependencies from mono/linker dotnet/xharness (#37051)

* Update dependencies from https://github.com/mono/linker build 20200526.2

Microsoft.NET.ILLink.Tasks
 From Version 5.0.0-preview.3.20271.1 -> To Version 5.0.0-preview.3.20276.2

* Update dependencies from https://github.com/dotnet/xharness build 20200527.1

Microsoft.DotNet.XHarness.Tests.Runners
 From Version 1.0.0-prerelease.20275.1 -> To Version 1.0.0-prerelease.20277.1

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>

uddate Ubuntu and Alpine RID (#37031)

Co-authored-by: Tomas Weinfurt <furt@DESKTOP-0J9757E.corp.microsoft.com>

Improve test ALPN detection (#36928)

* Improve test ALPN detection

* feedback from review

* add tvos

* update casing

Fix build with FEATURE_GDBJIT enabled (#37054)

This commit fixes broken build in case when following arguments added to
build.sh script: --cmakeargs -DFEATURE_GDBJIT=true

Enable jumps4 tests, since the issue has been fixed via mono/mono PR #19809 (#37015)

Add a BundleTestWasmApp target to build wasm test apps for libraries. (#36739)

Beginning of a BundleTestWasmApp target to build wasm test apps for libraries.

Update the -vs switch and add docs for it (#37026)

* Remove testhost DOTNET_ROOT from vs switch

We don't need to point to the testhost folder anymore for libraries for
supporting VS Test Explorer.

* Update docs

* Fix verbosity alias

Reduce app size for iOS (#36902)

We did the same mistake for Android - `libmonosgen-2.0.dylib` shouldn't be copied to bundle (we already have a mono runtime there)

Make ApkBuilder less verbose (#37050)

The output is still pretty useful and x4 times less verbose.

[browser][wasm] InteropServices.JavaScript backing code. (#37044)

[interp] Remove return value indirection (#37004)

* [interp] Remove return value indirection

When returning from a method we used to save the return value into frame->retval. We would then have separate opcodes for normal and void call and, depending on the type of call, we would access the retval and push it on the stack. In case of valuetype returns, the value is not returned in the proper space so we have an additional MINT_VTRESULT that copies the return around in the caller frame. In addition to this we had retval, is_void locals, saved as part of the current frame state which just make things confusing and are not needed.

This commit simplifies the call procedure by completely moving the responsability of the return to the called method. Simply put, once a method is ready for the call, it will leave its stack and vtstack to the state after pop-ing the arguments. The called method will then directly push the result to the parent frame's stack/vtstack. This means the calling code doesn't care about the type of the return anymore, the IL instructions after the call will just expect the return on the stack, if any, and use it naturally.

We still make use of frame->retval for interp entry frames (since we don't have a parent frame to push to), for pinvoke and jit_call frames out of convenience, but we should consider removing this variable to further trim down InterpFrame.

* [interp] Remove call opcodes that are no longer necessary

* [interp] Split up CALL and CALLVIRT opcodes

MINT_CALLVIRT is soon to be removed while MINT_CALL is the most common call opcode.

* [interp] Fix jit calls done through MINT_CALLVIRT

The callee should push the vt stack if there is a vt return, except when we are calling out of the interpreter, in which case the calling opcodes handle the push explicitly. We weren't handling the case of exiting the interpreter when doing a jit call through MINT_CALLVIRT. In order to keep the same structure for MINT_CALLVIRT, which doesn't need to know about the vt_res_size, we handle the vtstack change when doing a jit call by caching the return size in the InterpMethod, together with other jit_call information. Maybe we can refactor this later.

* [interp] Cleanup call opcode emitting

Add the opcode and set its data in the same place, making the code
easier to follow.

* [interp] Split up MINT_CALL_NAT in 2 opcodes

This opcode had two different cases embedded in it. Clean up the code,
split the opcode in two, removing also a dirty recursive call.

* [tests] Disable a few tests that fail with this PR due to false pinning

There are over a hundred tests calling GC.WaitForPendingFinalizers and potentially flaky with mono. We might have to disable all of them or tweak them to be mono friendly.

Avoid HttpClientHandler.ServerCertificateCustomValidationCallback keeping first request message alive (#37021)

RuyJIT: `Array.Length / cns` can be converted to unsigned div (mod) (#32368)

* Optimize `array.Length / cns` and `array.Length % cns` if cns is positive

* Address feedback

Fix a couple of bugs in multi-reg stores (#36994)

* Fix a couple of bugs in multi-reg stores

Fix #36912

Ensure Vector3 inequality doesn't use the hidden 4th element (#36953)

* Renable Vector3InequalityTest

* Fixing ARM64 LowerHWIntrinsicCmpOp to always treat the extra element of a TYP_SIMD12 as equivalent

* Adding a regression test for Vector3InequalityTest

* Update src/coreclr/src/jit/lowerarmarch.cpp

Co-authored-by: Carol Eidt <carol.eidt@microsoft.com>
* Applying formatting patch

Co-authored-by: Carol Eidt <carol.eidt@microsoft.com>

Align assembler mnemonics and operands in R2RDump (#37034)

* Align assembler mnemonics and operands in R2RDump.
* For ARM64 images dump instruction bytes as 4-byte hexadecimal integers.
* Fix incorrect usage of _methods.Count.
* Fix the corrupted R2RFormat.png file.

Disable AddThresholdTest in all archs (#37032)

Use nint instead of IntPtr in _RegisterFrozenSegment. (#37029)

rearrange tfms for intellisense (#37027)

Simple sideloaded profile guided optimization support (#36887)

1. Add code so the runtime can read profile data from a text file and then
make this data available to the jit, so it will be easier to test the jit's
behavior when profile data is available.
2. Add code so the runtime can create profile text files by telling the jit
to insert probes into jitted code that write into a persistent buffer, which
is saved to text at shutdown. This is mainly intended to make creation of the
sideloaded profile data files easier, since their internal structure must
exactly match artifacts from jitted methods.
3. Add a mode where jit-instrumented Tier0 code can feed counts into Tier1
jitting. This is an experiment to see what benefit, if any, might accrue from
having  profile data available for Tier1 jitting.

External format is text so it can be edited by hand or checked into the repo
alongside tests. Binary format would be more compact but opaque.

In all cases the jit internally sees a profile buffer identical to the one it
sees with IBC; the only minor changes in the jit are to initialize the count
field of each probe entry to zero and to only emit the entry helper call for
prejitted IBC.

Only root methods are instrumented, and only root method blocks are given
profile weights. Broadening to handle inlinees is future work, facilitated by
the new capabilites added here.