Tiny cleanup - make host_startup_info return void

The original return value was always 0 and never actually checked.

Annotate System.Security.Cryptography.Encoding for nullable (#1535)

* Annotate System.Security.Cryptography.Encoding for nullable

Throw NotSupportedException for WaitHandle.WaitAny with 64 handle on STA threads (#1647)

This change is technically a breaking change, but it is better to throw than silently do the wrong thing.

Fixes #1646

Make PhysicalAddress.GetHashCode thread-safe. (#1649)

Delete RHEL6 leftovers (#1648)

Fixes #423
Closes dotnet/corefx#24525
Closes dotnet/corefx#37739
Closes dotnet/corefx#31792

Simplify STA apartment condition in BlockingCollection.TryAddToAny (#1635)

Add tests for this codepath

[armel/Linux] Fix armel build (#1609)

Option '-DARM_SOFTFP' not passed after #27077 which required for armel build.
This commit fix it.

Annotate System.Security.Claims for nullable (#1431)

* Annotate System.Security.Claims for nullable

Remove dead code in BlockingCollection (#636)

* Remove dead code in BlockingCollection

* Revert "Remove dead code in BlockingCollection"

This reverts commit b0f1f86b909de1a4be37f55c05697bb12343e93b.

* Implement BlockingCollection::IsSTAThread

Change NetworkStream.Socket property from protected to public (#1598)

* Change NetworkStream.Socket property from protected to public. (https://github.com/dotnet/corefx/issues/35410)

* Removes ExposedSocketNetworkStream

* Remove superfluous property

* Update src/libraries/System.Net.Sockets/ref/System.Net.Sockets.cs

Co-Authored-By: Stephen Toub <stoub@microsoft.com>
Co-authored-by: Stephen Toub <stoub@microsoft.com>

Annotate Microsoft.Win32.SystemEvents for nullable reference types (#1621)

* Annotate Microsoft.Win32.SystemEvents for nullable reference types

* Address PR feedback

Unify cmake_minimum_required to 3.14.2 (#1315)

- Set cmake_minimum_required in top level projects to 3.14.2 (the lowest version used by the CI currently)
- Delete cmake_minimum_required from child projects
- Leave the cmake_minimum_required alone in vendored projects (zlib)
- Note CMake version in the docs

Fixed #1311

Add more known Regex patterns to increase test coverage. (#1617)

Annotate System.Diagnostics.FileVersionInfo for nullable reference types (#1614)

* Annotate System.Diagnostics.FileVersionInfo for nullable reference types

* Address PR feedback

Normalize Regex char class containing a single negated character (#1597)

* Normalize Regex char class containing a single negated character

When RegexFC scans the regex node tree looking for what characters can be the first character, if it encounters a negated character, it handles it by adding everything else to the set.  That then ends up defeating optimizations based on IsSingletonInverse, which expects a negated character class containing only the character, as opposed to a non-negated character class containing everything but the character.  There is already a RegexCharClass.Canonicalize method; this PR just adds the check and fix-up for this case.

* Address PR feedback

Partial Revert "Add asserts to catch unintentional use of generic overloads (#1200)" (#1628)

This reverts commit f6d14eeaf290a78e547f4cf15476e61b53e06b5c.

Enable TRACE_GC for the whole gc.cpp (#1604)

Change how we detect SSL3 on Windows (#1270)

Add Library builds for release-testing job (#1596)

* Add Library builds for release-testing job

Add library-build steps to the release-testing job
to adapt live-live builds.

Use Common template for CoreCLR and Library builds

Follow coding conventions (#297)

* Follow coding conventions

* Update SafeProcessHandle.Unix.cs

* Update shmemory.h

* Update shmemory.cpp

* Update MetadataTests.cs

* Update SqlDependency.cs

* Update comcache.cpp

* Update stubmgr.h

* Update SqlDependencyListener.cs

* Update assemblybinder.cpp

* Update Socket.cs

* Address stephen feedback

* Address  feedback

gc, pal: Avoid linking libnuma.so on non-NUMA systems (#1468)

* gc, pal: Avoid linking libnuma.so on non-NUMA systems

In an effort to reduce the system calls performed during the
initialization, this seemed like another low-hanging fruit: only link
in libnuma.so if the system has more than one NUMA node.

* gc, pal: Remove unneeded dlsym() call

The result of dlsym() is never attributed to any variable, and this
function is never called, so it's safe to remove these dlsym() calls.

Fixup orphaned references to GetRawSzArrayData (#1458)

Add LibrariesAllConfigurationsOverridePath (#1620)

Update workflow readme (#1512)

* Update workflow readme

* CR feedback and more detail

* Add daily workflow for libraries dev

* rebuild->build

* More updates

* typo:

* more

* More edits

* testing

* space:

Fix all GC stress failures in Pri1 tests on Unix with crossgen2 (#1619)

* Fix all GC stress failures in Pri1 tests on Unix with crossgen2

Without this PR, 90% of the CoreCLR Pri1 tests are failing due to GC
holes. All of the errors are caused by various bugs in the GCRefMap
generation that this PR fixes. With this PR, all the tests that pass
without GC stress also pass with it.

1. GetCorElementType() was incorrectly casting from TypeFlags to
CoreElementType. It was checking for a range of TypeFlags, but that
range didn't map to CorElementType 1:1. That resulted e.g. in getting
ELEMENT_TYPE_R4 for TypeFlags.IntPtr.
2. Besides that, the GetCorElementType() was also incorrectly mapping
Enum to ELEMENT_TYPE_CLASS.
3. The ReportPointersFromStructInRegisters function was incorrectly
assuming that the false stack entries correspond to stack slots while
they in fact correspond to bytes.
4. Arguments that are passed on stack were incorrectly reported as being
passed in registers (if there were still any registers available)
5. Members of structs embedded in other structs were reporting wrong
offsets (we were missing updating the offset by the delta parameter in
the GcScanValueType when calling GCScanRoots)
6. The code that determines if return values are passed in registers or
on stack was completely missing handling of structs returned in
registers.

* Fix typo in comment

Merge pull request #497 from dotnet/31GA-snap

Snap updates for 3.1 GA

Fixed stage-packages as per comments

Updating genCodeForBinary to be VEX aware (#1344)

* Updating genCodeForBinary to be VEX aware

* Resolving some PR feedback

Generate full platform manifest in official build (#1594)

* Generate full platform manifest in official build

Generates a full platform manifest in the official build by downloading all platforms' artifacts from the CoreCLR and Libraries jobs for each Installer job.

JIT: block local constant propagation of handles when compiling with relocs (#1593)

This gives local constant prop the same behavior as value-number-based constant
prop; both now block handle propagation when the jit must generate relocs (R2R),
to ensure that handle references appear only in simple instruction forms like movs.

Prerequisite to #1309, which can enable such propagation.

R2RDump improvements w.r.t. GC info reporting (#1539)

While I haven't yet managed to nail down the cause of hangs
in the r426654 regression test, I have at least used the opportunity
to improve R2RDump GC info reporting. I have made the reporting
much more compact and more descriptive at the same time by translating
register names and showing stack-relative addresses in a much more
straightforward manner. I'm also proposing to ass a VS solution
for R2RDump - as it now depends on the ILCompiler.Reflection.ReadyToRun
project, it just sucks not to be able to work with both in the VS.

Thanks

Tomas

Fix and Test case for #27924 (#1059)

Fix and Test case for #27924

Always build the OpenSSL shim in portable mode on macOS.

Merge pull request #1226 from AaronRobinsonMSFT/sdk_4099

Fix parsing of metadata when looking for COM related CustomAttributes.

Annotate System.Linq.Expressions for nullable (#1064)

* Annotate System.Linq.Expressions for nullable

[Arm64] Implement AbsoluteDifference hardware intrinsics (#1334)

And use VecBinOpTest.template in AbsoluteCompare* tests to remove duplicated logic in validation

Add binder tracing test for ResolveSatelliteAssembly stage (#1455)

[runtime] Arm64: Add TransposeEven and TransposeOdd intrinsics (#1220)

remove unneeded code (#1521)

* check

* cleanup

Add gcstress option to SuperILC (#1446)

* Add gcstress option to SuperILC

This option allows executing tests with gcstress mode set to a passed
in value.
I have also added additional fixes for race issues with _logWriter.

Add MsQuic implementation of System.Net.Quic (#427)

Fix typo (#1528)

Nullable annotate System.Reflection.Metadata (#1056)

* Nullable annotate System.Reflection.Metadata

* remove unused declaration

* update System.Diagnostics.Stacktrace for System.Reflection.Metadata annotations

* address feedback

* reinstate annotations erroneously removed by generator

* Apply feedback

* address feedback

* address feedback

fix stress script issue (#1452)

Rename file extension of RunnerTemplate.Windows.txt to .cmd (#1470)

So it automatically gets the EOL treatment from https://github.com/dotnet/runtime/pull/1336.

Don't call AdvanceTo(default) in PipeReader.CopyToAsync (#1437)

* Don't call AdvanceTo(default) in PipeReader.CopyToAsync

* Address PR feedback

Remove BaseDomain::IsSharedDomain (always false now) (#1456)

Expose CreateLinkedTokenSource(CancellationToken) (#1379)

CoreFX renames (#1440)

Merge pull request #1348 from stephentoub/moreregexperf

More Regex perf improvements

improve ALPN handling (#1347)

Fix fat call transform for x86 (#1451)

`AddArgumentToTail` with a null first argument will AV.

I *think* this also works with methods that take a return buffer (methods that return large structs), but might require a pair of eyes more experienced in this codebase.

Add a few more tests / improve code coverage

Address PR feedback

Additional code gen improvements

- In Go, Improve code generation from several constructs to reduce code size and increase bounds reduction in the non-backtracking compiler
- In FindFirstChars, use IndexOf instead of Boyer-Moore if the found prefix is only one character in length.
- Hide some debug-only functions from code coverage

Rename "Greedy" subexpressions to "Atomic"

The greedy naming is really confusing (we should fix it in the framework documentation as well).

In general regular expression nomenclature, "greedy" is used as the opposite of "lazy", indicating how much a loop initially consumes, e.g. does `a*` first try consuming as many 'a's as possible or does it first try consuming as few 'a's as possible... it's "greedy" (denoted as `a*`) if it consumes as many as possible and "lazy" (denoted as `a*?` if it consumes as few.

How aggressively it consumes, however, is orthogonal to whether it backtracks.  Whereas `a*` is greedy and backtracking and `a*?` is lazy and backtracking, `(?>a*)` is greedy and non-backtracking and `(?>a*?) is lazy and non-backtracking.

Unfortunately, the nomenclature in the implementation and the documentation describes the `(?> ... )` as being a "greedy subexpression", which then conflates the meaning of "greedy".

The rest of the industry refers to these instead as "atomic", so I've changed it to that in the implementation.

Fix CharInClass handling of invariant in lookup table

The Boolean logic was previously wrong, leading to characters being included in the set even if they weren't meant to be, e.g. given the set [^bc], this should include everything other than b and c, but if invariant culture was being used, it incorrectly included B and C in the set.

Add more tests, and fix some bugs found

Restore netfx test configuration to serve as oracle

With all the changes being made in the regex implementation in this release, we want to make sure we don't regress, and a good way to help doing that is add a bunch of tests and ensure they pass on netfx as well, where appropriate.

Add non-backtracing support for greedy loops and lazyloops

Auto-greedy for last Alternate/Loop/Lazyloop in regex

Avoid unnecessary operations when textSpanPos == 0

Add non-backtracking go support for anchors, alternation, lookahead, repeaters

Use vectorized IndeOf for some notoneloopgreedy loops

Add Notoneloopgreedy to go with One/Setloopgreedy

I initially mistakenly thought this wouldn't be useful, but it is.  For example, the expression ".*\n" can be made non-backtracing.

Auto-greedy last set/one loop in regex

Upgrade some loops followed by boundaries to be greedy

Add optimized non-backtracking code gen for limited subset of regexes

When we can determine from the regex that backtracking isn't needed, we
can generate much smaller / better code in RegexCompiler.GenerateGo.

Improve StringCode lookup in RegexWriter to avoid unnecessary list

Optimize allocation for child lists of RegexNodes

Optimize {Lazy}loops that directly wrap Set/One/Notone

When written as a* these are already transformed appropriately by the parser, but other reductions can expose them in a way that the parser won't see.  For example, (?:a)* ends up being a much more expensive regex even though it's identical functionally.  This reduces the latter to the former, for cases where a {Lazy}loop is found to directly wrap a Set/One/Notone.

Automatically upgrade some loops to be greedy

In an expression like A*B, backtracing may be involved even if it's impossible that it'll do any good, namely if there's no overlap between the A and B sets.  This backtracking can be avoided by using a greedy subexpression, e.g. (?>A*)B, but a) few developers think to do that, and b) even when they do the implementation has more overhead than is desirable.

This change does three things:
1. It introduces primitives for "oneloopgreedy" and "setloopgreedy" that correspond to the existing "oneloop" (a single character in a loop) and "setloop" (a char class in a loop) primitives.
2. It reduces a Greedy node that contains a one/setloop to instead just be a one/setloopgreedy node.
3. It finds cases of one/setloop concatenated with other things, and automatically updates those to be one/setloopgreedy when it can provie that the backtracking would have no benefit.

(2) removes a small amount of overhead, in particular in compiled, because it avoids needing to spit additional tracking state for backtracking.

(3) can likely be expanded in the future to catch more cases, but it handles the common cases now.  This has a very measurable impact on non-matches that would otherwise trigger backtracking into the subexpression.

Code spit EnsureStorage replacement to avoid spilling locals

Only spill if we have to grow the relevant arrays.

Clean up minor style issues in RegexCompiler

Minor RegexCompiler codegen tweak for text[pos++]

The JIT generates slightly better code for:
```C#
text[pos] = ....;
pos++;
```
instead of:
```C#
text[pos++] = ...;
```
so use the IL equivalent of the former.

Remove stale generated local in RegexCompiler

Fix inadvertent closure in EmitCallCharInClass

Add EmitCallCharInClass optimization for singleton

Use IndexOf{Any} for case-sensitive singletons in FindFirstChar

Takes advantage of vectorization in string.IndexOf.

Clean up more code in RegexCompiler

- Only use ilg from "macro" helpers
- Move some blocks of code around

Put back Regex singleton optimization

I had removed it because it seemed illogical that anyone would write such character classes.  But it turns out the implementation/parser itself generates them for certain forms, e.g. "hello|hithere" will create a character class containing just 'h' in order to find the start of the alternation.

Delay JIT'ing until first execution

I'd previously removed multiple delegate allocations that were being incurred every time we needed to run a compiled regex concurrently.  In doing so, however, I inadvertently forced JIT'ing to occur when the Regex was created rather than on first use; that can in turn lead to more start-up overheads for regexes that are created on startup (e.g. to initialize a static field) but that aren't used at start-up, as well as reduce potential parallelism that could otherwise occur for distinct regexes being run for the first time concurrently.  This addresses the issue by still caching delegates, but in addition to rather than instead of the DynamicMethod objects.

Remove dead code in build.ps1 after -help option was added (#1430)

Forgot remove a ? from ref assembly (#1467)

Update superpmi.py helper script (#1474)

Mostly, updated documentation. Added some additional logging
output for improved clarity. Tried to simplify the code so
it is easier to understand, in places.

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

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

- Microsoft.DotNet.XUnitExtensions - 5.0.0-beta.20057.1
- Microsoft.DotNet.VersionTools.Tasks - 5.0.0-beta.20057.1
- Microsoft.DotNet.ApiCompat - 5.0.0-beta.20057.1
- Microsoft.DotNet.Arcade.Sdk - 5.0.0-beta.20057.1
- Microsoft.DotNet.Build.Tasks.Configuration - 5.0.0-beta.20057.1
- Microsoft.DotNet.Build.Tasks.Feed - 5.0.0-beta.20057.1
- Microsoft.DotNet.Build.Tasks.Packaging - 5.0.0-beta.20057.1
- Microsoft.DotNet.Build.Tasks.SharedFramework.Sdk - 5.0.0-beta.20057.1
- Microsoft.DotNet.CodeAnalysis - 5.0.0-beta.20057.1
- Microsoft.DotNet.GenAPI - 5.0.0-beta.20057.1
- Microsoft.DotNet.XUnitConsoleRunner - 2.5.1-beta.20057.1
- Microsoft.DotNet.GenFacades - 5.0.0-beta.20057.1
- Microsoft.DotNet.Helix.Sdk - 5.0.0-beta.20057.1
- Microsoft.DotNet.RemoteExecutor - 5.0.0-beta.20057.1

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

- Microsoft.DotNet.XUnitExtensions - 5.0.0-beta.20057.5
- Microsoft.DotNet.VersionTools.Tasks - 5.0.0-beta.20057.5
- Microsoft.DotNet.ApiCompat - 5.0.0-beta.20057.5
- Microsoft.DotNet.Arcade.Sdk - 5.0.0-beta.20057.5
- Microsoft.DotNet.Build.Tasks.Configuration - 5.0.0-beta.20057.5
- Microsoft.DotNet.Build.Tasks.Feed - 5.0.0-beta.20057.5
- Microsoft.DotNet.Build.Tasks.Packaging - 5.0.0-beta.20057.5
- Microsoft.DotNet.Build.Tasks.SharedFramework.Sdk - 5.0.0-beta.20057.5
- Microsoft.DotNet.CodeAnalysis - 5.0.0-beta.20057.5
- Microsoft.DotNet.GenAPI - 5.0.0-beta.20057.5
- Microsoft.DotNet.XUnitConsoleRunner - 2.5.1-beta.20057.5
- Microsoft.DotNet.GenFacades - 5.0.0-beta.20057.5
- Microsoft.DotNet.Helix.Sdk - 5.0.0-beta.20057.5
- Microsoft.DotNet.RemoteExecutor - 5.0.0-beta.20057.5

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

- Microsoft.NET.Test.Sdk - 16.5.0-preview-20200103-01

* Fix space in resource key to react to arcade resource generation fix

Co-authored-by: Santiago Fernandez Madero <safern@microsoft.com>

Merge pull request #1073 from DrewScoggins/dev/anscoggi/activateLiveBuilds

Fix passing of liveBuilds name string in perf job.

Expose MemoryMarshal.GetArrayDataReference (#1036)

Given an SzArray, returns a reference to where the element at index 0 would be located, even if the array is empty. No variance checks or bounds checks are performed.

Improve RawArrayData comments (#1450)

Update URLs in BOTR (#1441)

- update links to sources from dotnet/coreclr -> dotnet/runtime/src/coreclr
- fix dead urls (github and non-github) under docs/design/coreclr.
- use https://software.intel.com/sites/default/files/article/402129/mpx-linux64-abi.pdf as a new home for x64 System V ABI, as http://x86-64.org is now abandoned. Update source files as well.
- normalize “, ” and ’ to ascii " and ' and remove trailing whitespaces in modified files.

Disable failing BasicTest_AccessInstanceProperties_NoExceptions_Bsd test (#1447)

Fix ReflectionTypeLoadException nullable annotations (#545)

We were previously using `Type[]?`, but the array can actually contain null, so it needs to be `Type?[]?`.  Further, upon reviewing a bunch of call sites, it's clear that a) code expects to be able to pass null for the arrays into the ctor but also b) expects to be able to read Types/LoaderExceptions without doing null checks.  As such, I modified the annotations accordingly, and fix the implementation to match.

Fix nullable annotations on CollectionExtensions.GetValueOrDefault (#1364)

Suppress ExecutionContext flow in SocketHttpHandler's ConnectHelper (#1381)

Ensure LocalEndPoint being updated in SendToAsync(SocketAsyncEventArgs) (#808)

Fixes #915 by assigning _rightEndPoint like in other overloads, introducing a breaking change in the behavior of SendToAsync(SocketAsyncEventArgs).

Augment several "bool IsXx" APIs with [NotNullWhen(true)] (#1372)

If these methods return true, we know the argument passed in wasn't null.  Annotating it as such can help reduce warnings in callers.

Fix GC heap corruption on ARM. (#1389)

The allocate_in_free code path in allocate_in_expanded_heap incorrectly calculated the large (double) alignment padding size when limiting the plug size (SHORT_PLUGS) if set_padding_on_saved_p was true:

    set_padding_in_expand (old_loc, set_padding_on_saved_p, pinned_plug_entry); // Sets the padding flag on the saved plug
    ...
    pad += switch_alignment_size (is_plug_padded (old_loc)); // Reads the padding flag from the old (different!) plug

That caused access violation during a later heap walk since the g_gc_pFreeObjectMethodTable pointer marking the gap was not placed at the right address.

Merge pull request #1346 from vargaz/string-ctor-preserve-dep

Add PreserveDependency attributes to String ctors to mark their dependency on the corresponding String:Ctor methods.

Fix PreserveDependencyAttribute.cs location.

Fix reported result in binder tracing for mismatched MVID when loading assembly by path / stream (#1339)