Merge pull request #1199 from lmolkova/w3cTestSuiteCrashes

W3C Trace-Context: Allow hex chars in version and don't crash on non-hex chars in traceFlags

Merge pull request #1873 from jaredpar/standard

Disable Apline.38 ARM64 runs

Tweak documentation on NotNullAttribute (#1891)

Tagging @BillWagner @stephentoub @RikkiGibson

The use of `[NotNull]` on an input parameter is allowed. The typical example is an `AssertNotNull` method. That behavior was [specified](https://github.com/dotnet/csharplang/blob/master/meetings/2019/LDM-2019-05-15.md#simple-postconditions), but does not appear in the docs. Adding a mention.

I assume that changes here propagate to docs automatically.

Improvements for null check folding. (#1735)

optFoldNullChecks attempts to remove GT_NULLCHECK nodes that are
post-dominated by indirections on the same variable. These changes
implement a number of improvements.

1. Recognize more patterns.
Before these changes only the following pattern was recognized:
x = comma(nullcheck(y), add(y, const1))

followed by

indir(add(x, const2))

where const1 + const2 is sufficiently small.

With these changes the following patterns are recognized:

nullcheck(x)
or
x = comma(nullcheck(y), add(y, const1))

followed by

indir(x)
or
indir(add(x, const2))

where const1 + const2 is sufficiently small.

2. Indirections now include GT_ARR_LENGTH nodes.

3. Morph has an optimization
((x+icon1)+icon2) => (x+(icon1+icon2))
These changes generalize it to handle commas:
((comma(y, x+icon1)+icon2) => comma(y, x+(icon1+icon2))

That exposes more trees to null check folding.

4. Fix a bug in flow transformations that could lose BBF_HAS_NULLCHECK flag
on some basic blocks, which led to missing opportunities for null check folding.

5. Make safety checks in optCanMoveNullCheckPastTree
(for trees between the nullcheck and the indirection) both more correct
and less conservative. For example, we were not allowing any assignments
if we were inside try; however, assignments to compiler temps are safe since
they won't be visible in handlers.

5. Increase the maximum number of trees we check between GT_NULLCHECK and
the indirection from 25 to 50.

7. Refactor the code and move pattern recognition and safety checks to
helper methods.

8. Add missing BBF_HAS_NULLCHECK and OMF_HAS_NULLCHECK when we create GT_NULLCHECK nodes.

Merge pull request #1525 from vitek-karas/OpenInstallerTestSln

Setup environment to open the installer test sln

Remove unused/unnecessary defines (#1878)

Port stubbed out Regex.CompileToAssembly (#1850)

* Port stubbed out Regex.CompileToAssembly

This ports Regex.CompileToAssembly.  The "only" thing missing is the ability to actually save the Assembly out, which is obviously critical.  I'm putting this commit up because I've been using the generated assembly to help debug/improve the code generation, using temporary hacks locally to get the Assembly saved to disk.  Hopefully in the near future we can have a permanent solution in place.

* Code cleanup

Fix MSVC warn as errors... (#1876)

* Fix warning as error

* Fix mismatched declaration/definition

Remove stray sos reference (#1875)

Use bitmask operation to check if a slot is untracked (#1870)

Enable overriding CoreCLR and Libraries configurations in root… (#1826)

* Override the configuration of the CoreCLR and Libraries builds during the root build when CoreCLRConfiguration and LibrariesConfiguration are specified respectively.

* Add coreclrConfiguration and librariesConfiguration parameters to build scripts.

* Update docs.

* Make arguments lowercase on bash.

* Fix typos.

Add Crossgen pack for crossgen2 tool (#1859)

* Add Crossgen pack for crossgen2 tool

* Inline sharedfx tooling support for Crossgen pack

* Support publishing crossgen pack

* "Crossgen pack" => "Crossgen2 pack"

Fix bug with PInvokes having byref arguments. (#1843)

* Fix bug with PInvokes having byref arguments.

Make IsMarshalingRequired return true for a byref argument

Disable Apline.38 ARM runs

These are consistently failing on all runs. Disabling to get the build
green again while we dig into this

https://github.com/dotnet/runtime/issues/1871

Fixes runtime issue #1739 (#1840)

* Fixes runtime issue 1739
Added additional check for Jit Stress mode STRESS_LCL_FLDS
This stress mode requires that the types exactly match

* Code review feedback

Merge pull request #1759 from vitek-karas/NativeSearchDirsFix

hostfxr_get_native_search_directories must fail when given CLI command

There's no check that when running with a "command" like hostfxr_get_native_search_directories that the code will not try to locate SDK and run CLI commands.
If it happens not only it does wrong things, but the return values are bogus (it doesn't fill the output buffer).

This change adds the necessary check to avoid this completely. It also makes sure that the output buffer size is always correctly set, even on errors.

Moved all the tests for hostfxr_get_native_search_directories to use nativehost and add a test to cover this case.

Fix Linux_x64_glibc condition to build RPM/Deb (#1861)

Adding area owners for new mono labels (#1865)

Implement required HttpClient apis in System.Net.Quic (#1703)

Update status badge link (#1855)

add benchmarking and profiling workflow docs links (#1856)

Spanify and cleanup System.Security.Principal.Windows

* spanify internal constructor, caller and callees
* rename locals and fields to match current style
* save the SecurityIdentifier objects for WindowsIdentity's Is(Something) properties as statics

revert CPU clipping to 3.0, 2.x behavior (#1851)

ZipFile.CreateFromDirectory: ensure entryEncoding parameter is being passed (#1825)

Remove the option to specify VS path

Coalesce adjacent loops in concatenation RegexNodes (#1838)

* Coalesce adjacent loops in concatenation RegexNodes

This augments the reduction phase of concatenation nodes to combine adjacent one/notone/setloops, e.g. `a*a+a{1,2}b` becomes `a{2,}b` (previously added optimizations will then see that the a loop can be made atomic and replace it with the equivalent of `(?>a{2,})b`).  This has several benefits.  First, it simplifies the node tree, creating less work for IR writer and less work for the interpreter/compiler.  Second, it gives the compiler more opportunity to choose how the loop should be represented, when and how to unroll, etc.  Third, it enables the auto-atomicity step to apply to more loops (as in the previous example).  And most importantly, it can drastically reduce backtracking (especially with the atomicity optimization, but even without that).  An expression like `a*a*a*a*a*a*b` run against an input like `aaaaaaaaaaaaaa` could previously take a very long time; now, it'll be very fast.

* Follow-up from previous PRs this is now rebased on

Fix tests on linux

Different path normalization behavior

Implement SuppressGCTransition attribute in crossgen2 (#1772)

Fixes dotnet/coreclr#27188.

Fix tracing rundown tests with optimizations and large version bubble (#507)

* Fix tracing rundown tests with optimizations

The test is failing when compiled with crossgen2 with optimizations due
to the fact that it didn't find System.Diagnostics.Tracing loaded. The
reason is that due to cross module inlining, the
System.Diagnostics.Tracing was not needed anymore.
I have fixed it by checking for the
Microsoft.Diagnostics.Tracing.TraceEvent assembly.

Do not look at Size when doing Auto layout (#1805)

Seems like the deleted line was copypasted from the Sequential/Explicit case, but it's incorrect.

Fixes #1770.

Flush GC log on shutdown (#1721)

Recording the reason why we change the generation to condemn during joined_generation_to_condemn() to the event stream (#1042)

Port CreateDirectory method as a DirectorySecurity extension (#1834)

Address PR feedback from old PR (#1837)

more tests

fix IsW3C check

Allow hex chars in W3C Trace-Context version and fix crash with invalid traceFlags

Expand RegexNode atomicity test to loops at the end of alternation branches (#1769)

Given an expression like `(abcd*|efgh*)ijk`, we will now see that if the `i` after the alternation doesn't match and we backtrack into the alternation, we won't need to backtrack into the `d*`, as it can't give back anything that would allow the `i` to match.

Automatically anchor regexes beginning with .* (#1706)

Review large alignment code in GC (#1351)

Improve comments and fix a couple of places where large alignment was not taken into account. Set pad_in_front to zero if SHORT_PLUGS is not defined.

Prompt dialog when running GUI app and hostfxr is < 3.* (#1222)

* Prompt dialog when trying to run a GUI app and hostfxr < 3.*

* Check for the ability to set custom error writer in hostfxr instead of checking its version.

* Put error writer inside main_fn_v2 scope.
Don't add unnecessary custom prefixed errors.

* Remove unnecessary if.

* Update error message and url.

* Trigger only when running an apphost gui.

* Check if we redirect apphost traces instead of using ifdefs.

* Add tests.

* Address feedback

* Cleanup.

* Remove unnecessary class HostFxrVersion

* Cast StatusError for comparison with rc

Fix dumping of GCSlotFlags in R2RDump (#1833)

There were two issues. If a slot was marked as untracked, we were
replacing whatever flags there were set by GC_SLOT_UNTRACKED. After
fixing that, the other problem was that the GCSlotFlags were not marked
as Flags and so we were not printing it as combination of flags.

Remove Windows_NT_arm64 from checked libraries rolling tests (#1839)

JIT: ensure we get the right oper when constructing lcl var and lcl fld (#1824)

Remove the constructors that default in the oper and force callers to
be explicit about which one they want.

Fixes #1668

Fix Crossgen2 compilation of System.Private.CoreLib (#1762)

With the switch-over from GetRawSzArrayData to GetArrayDataReference
we must now instantiate the method over the array element type.
Also, we shouldn't be harvesting pseudo-tokens generated during
stub emission.

Thanks

Tomas

Update macOS build requirements to include openssl

Remove Windows_NT_arm64 run tests (#1827)

missing check for `readonly` case in x86 array ::Address stub (#1798)

The check was previously done off the flags after masking the lower 2 bits in the typedesc. The whole thing is now gone.

Documentation refinements (#1738)

* Documentation refinements

* Fix a formatting issue

* Code review updates

Update the CoreCLR testing workflow doc (#1806)

Nullable annotate System.IO.Compression.ZipFile (#1811)

* Nullable annotate System.IO.Compression.ZipFile

* address feedback

enable nullable annotations for System.Dynamic.Runtime (#1810)

copy CoreRun to the testhost directory to make benchmarking and profiling of local builds easier (#1756)

Fix crossgen2 handling of sequential layout with explicit size (#1763)

Crossgen2 is incorrectly not marking types with sequential layout and
explicit size as custom layout, which results in JIT generating incorrect
code for copying such structs. One example is the
Loader/classloader/SequentialLayout/Regress/ExplicitSize/ExplicitSize
test that has a struct with single int element, sequential layout and
explicitly set size to 16. The getClassAttribsInternal was not setting
CorInfoFlag.CORINFO_FLG_CUSTOMLAYOUT for this type and so JIT was
generating code that copied just the int element.

This change fixes it. Runtime uses similar check.

JIT: refine handle blocking logic (#1788)

Assertion prop uses `O2K_CONST_INT` for handles, even on 64 bit targets. Make
suitable adjustments to the handle propagation blocking logic.

Problem was introduced in #1593 and exposed by #1309.

Fixes #1777

Fix Linux tests mainv1, mainv2 (#1790)

Don't use CR in Unix bash scripts.

This appears to be fallout from #583, which added a
second variable that then created a need to add newlines.

Delete duplicated R2R version definition and sync to reality (#1764)

We're not generating R2R version 3.0: we started generating 4.1 when nop padding/jump stamps were removed. (Seems like R2R 4.0 was actually skipped entirely but we ended up with the 4.0 version in one of the copies of the struct because only major version was bumped for jump stamps; minor was untouched - minor went up with cuckoo metadata before that and that didn't involve crossgen2 at that point)

Didn't dig into history to find out why it's duplicated in the first place, but it doesn't look like a good place to be in.

Change all ActiveIssues to use Uris instead of numbers (#1768)

Update repo link to dotnet/runtime (#1781)

Fix impSIMDMinMax (#1774)

When taking the no-AVX, no-SSE42 path, ensure that it calls `lvaSetStruct` before `gtNewLclvNode`.

Fix #1667

Use a single pipeline with runtime conditioned jobs in PRs (#1473)

* Use a single pipeline with runtime conditioned jobs in PRs

* PR Feedback

* Disable long running tests checked runtime

Delete checked-in Unicode data files in favor of runtime-assets package (#1743)

IDNA files (in System.Globalization.Extensions.Tests) not yet deleted; will tackle this later

Fix Issue 1104 (#1734)

* Fix the incorrect change where 'alreadyPresent' was incorrectly initialized.
The PR that introduced this issue: https://github.com/dotnet/coreclr/pull/23109

* Add test case

Update ReportGenerator to 4.4.2 (#1778)

Jitstress fixes for tailcall tests (#1771)

Make STRESS_GENERIC_VARN more compatible with methods that make explicit tail
calls. Don't add gc checks for explicit tail calls, and remove the code in
morph that blocks tail calls if gc checks are active.

Update the tailcall test to to disable STRESS_UNSAFE_BUFFER_CHECKS. This can
be reverted when #341 is merged.

Fixes #1752.

Update dotnet-tools.json (#1773)

[master] Update dependencies from 4 repositories (#1758)

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

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

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

- ILLink.Tasks - 0.1.6-prerelease.20062.1

* Update dependencies from https://github.com/microsoft/vstest build 20200113-03

- Microsoft.NET.Test.Sdk - 16.5.0-preview-20200113-03

* Update dependencies from https://github.com/dotnet/runtime-assets build 20191210.1

- System.Windows.Extensions.TestData - 5.0.0-beta.19610.1
- System.Security.Cryptography.X509Certificates.TestData - 5.0.0-beta.19610.1
- System.Net.TestData - 5.0.0-beta.19610.1
- System.IO.Packaging.TestData - 5.0.0-beta.19610.1
- System.IO.Compression.TestData - 5.0.0-beta.19610.1
- System.Drawing.Common.TestData - 5.0.0-beta.19610.1
- System.ComponentModel.TypeConverter.TestData - 5.0.0-beta.19610.1

* Revert "Update dependencies from https://github.com/mono/linker build 20200112.1"

This reverts commit 0855288b45de9c39aacc55864fa42a32a4d80052.

Co-authored-by: Viktor Hofer <viktor.hofer@microsoft.com>

Fix build on Linux

JIT: handle some side effects in box pattern match (#1741)

Handle simple side effects in the `box; br` pattern based optimization.

Fixes #1713.

Cse tuning (#1463)

* cse-tuning branch

1. Changed csdLiveAcrossCall to a bool (zero-diff)

* 2.  Added the remaining zero-diff changes from my old coreclr branch (zero-diff)

* 3. Incoming stack arguments don't use any local stack frame slots

x64  5 improvements 0 regressions,  Total PerfScore diff: -10.72
x86 16 improvements 5 regressions,  Total PerfScore diff: -72.95

* 4.  Locals with no references aren't enregistered  (zero-diffs)

* 5. Fix handling of long integer types, they only use one register not two.

    x64 250 improvements 51 regressions,  Total PerfScore diff:   -459.09
  arm64 162 improvements 16 regressions,  Total PerfScore diff:  -1712.52

* 6. Adjust computation of moderateRefCnt and aggressiveRefCnt values

     x64 280 improvements 81 regressions,  Total PerfScore diff:   -274.78
   arm64 264 improvements 61 regressions,  Total PerfScore diff:   -911.00
     x86  87 improvements 42 regressions,  Total PerfScore diff:   -123.46
   arm32 195 improvements 81 regressions,  Total PerfScore diff:   -239.10

* 7.  slotCount refactor (zero-diffs)

* 8.  Enable the use of the live across call information

      x64 125 improvements 136 regressions, Total PerfScore diff:   +427.43
    arm64  83 improvements 153 regressions, Total PerfScore diff:   +260.68
      x86 218 improvements 193 regressions, Total PerfScore diff:   +199.81
    arm32 145 improvements 181 regressions, Total PerfScore diff: -33283.10

arm32 method with improvement:
    -33864.40 (-2.87% of base) : System.Private.CoreLib.dasm - TypeBuilder:CreateTypeNoLock():TypeInfo:this (2 methods)

* 9.  Adjust the cse_use_costs for the LiveAcrossCall case

      x64  61 improvements  61 regressions, Total PerfScore diff:   -189.03
    arm64  90 improvements  49 regressions, Total PerfScore diff:   -463.42
      x86  88 improvements  80 regressions, Total PerfScore diff:   -238.61
    arm32 101 improvements  63 regressions, Total PerfScore diff:   -259.50

* 10.  If this CSE is live across a call then we may need to spill an additional caller save register

          x64  73 improvements  45 regressions, Total PerfScore diff:   -279.88
        arm64  45 improvements  76 regressions, Total PerfScore diff:    -90.94
          x86  13 improvements  14 regressions, Total PerfScore diff:    -21.55
        arm32  45 improvements  33 regressions, Total PerfScore diff:    -78.60

* 11.  (x64 only)  floating point loads/stores encode larger, so adjust the cse def/use cost for SMALL_CODE

   No diffs in System.Private.Corelib

* 12. Remove extra cse de/use costs for methods that have a largeFrame or a hugeFrame

       x64 199 improvements  50 regressions, Total PerfScore diff:   -2061.36
     arm64  11 improvements   3 regressions, Total PerfScore diff:     -46.84
       x86 136 improvements  80 regressions, Total PerfScore diff:   -1795.00
     arm32  50 improvements  35 regressions, Total PerfScore diff:    -132.30

* clang-format

* Code review feedback

Removed increment of enregCount on _TARGET_X86_ when we have compLongUsed:
    Framework diffs
    Total PerfScoreUnits of diff: -654.75 (-0.00% of base)  diff is an improvement.
    79 total methods with Perf Score differences (55 improved, 24 regressed), 146432 unchanged.

Fixed setting of largeFrame/hugeFrame for ARM64
    Zero framework diffs.

:

* run jit-format

* correct some wording in comments

* reword a comment

Add link to shared portion of CoreLib to .gitignore (#1607)

In the coreclr repo, one could simply findstr in the System.Private.CoreLib directory to find stuff.

Since in the runtime repo the shared portion of the CoreLib moved to src/libraries/, but private portion lives in src/coreclr, the files are split. We can work around it locally by making a symbolic link.

That way runtime devs that just grep/findstr stuff don't have to suffer entering the src/libraries directory where tab completion doesn't work because there's 180 directories prefixed by `System.`.

Fix PEReader creation and caching (#1742)

Can't use a fixed stack-based pointer to initialize something on the heap and expect it to continue working :)

Fix description of -skipnuget in src/coreclr/build.sh (#534)

Revert Async changes to SNIPacket to fix deadlock issues (#778)

hostfxr_get_native_search_directories must fail when given command line for a CLI command

There's no check that when running with a "command" like hostfxr_get_native_search_directories that the code will not try to locate SDK and run CLI commands.
If it happens not only it does wrong things, but the return values are bogus (it doesn't fill the output buffer).

This change adds the necessary check to avoid this completely. It also makes sure that the output buffer size is always correctly set, even on errors.

Moved all the tests for hostfxr_get_native_search_directories to use nativehost and add a test to cover this case.

Remove two always false properties in UriSyntax (#696)

pal_random: Open /dev/urandom with O_CLOEXEC (#1249)

The third argument to open(2) is ignored by the system call unless
O_CREAT is passed in the second argument.  O_CLOEXEC should be OR-ed
with the second argument.

Mark some Nullable<T> members as readonly (#1727)

Value, HasValue, GetValueOrDefault

Disable TestOnExecuteCustomCommand (#1745)

This is failing on the libraries outerloop builds.

Tracked by #1724

Add minimum length optimization to regex (#1624)

We now do a cursory examination of the RegexNode tree in order to determine the smallest length string that could possibly match.  For example, the regex "(?:123|4567)8" has a minimum length of 4 (if the input were "1238"), and the regex "\d{1-2}-\d{1-2}-\d{2-4}" has a minimum length of 6 (e.g. "1-10-20"). If the input string is shorter than that minimum length, FindFirstChar will now immediately return false, as it can't possibly match, avoiding spending time evaluating a pattern that couldn't possibly match.

Nullable annotate System.Linq.Queryable (#979)

* Nullable annotate System.Linq.Queryable

* update annotations based on System.Linq.Expressions annotation

* address feedback

* revert System.Linq.Expressions reference source

* make IQueryable LINQ methods compatible with annotated IEnumerable signatures

* remove unnecessary nullable annotation

* address feedback

Fix token encoding crash when compiling without version bubble (#816)

* Fix token encoding crash when compiling without version bubble

Implement the ability to get token values for typeRefs that are not part of the version bubble. This fixes the ability to compile the CoreFX set without the --large-bubble switch

Update Version.Details.xml with new UnicodeData test asset (#1731)

Fix linuxnonjit that got accidentally renamed to linuxonjit (#1694)

The PR #1292 has accidentally renamed the linuxnonjit to linuxonjit.
This change fixes it.

Update System.Drawing.Common index to get latest package for harvesting (#1729)

Update CharUnicodeInfo to Unicode 12.1; update StringInfo to UAX29 (#328)

- Updates CharUnicodeInfo.GetCategory from Unicode 11.0 to Unicode 12.1
- Optimizes some methods in Char and CharUnicodeInfo to have better codegen
- Updates StringInfo to use UAX29 extended grapheme cluster segmentation logic

Fix comments in zapimage.cpp (#1629)

I noticed that base addresses in the comments were not correct (constants in the switches.h file had been updated without updating the comments in this file), and while at it, fixed spelling and grammar in other comments.

improve cert validation diagnostic on  OSX (#1513)

* propagate error if cert validation fails on OSX

* feedback from review

* feedback from review

* update style

* using System.Net

Cleanup remaining rhel/centos 6 conditions (#1659)

* Cleanup remaining rhel/centos 6 conditions
Also fixes some errors reported by shellcheck in `init-distro-rid.sh`.

* Update function comments

JIT: build basic block pred lists before morph (#1309)

Build basic block pred lists before morph, instead of after, and add an early
flow optimization pass. Fix up a few places where ref counts or pred lists
were not properly maintained in morph.

The early flow opt pass enhances the local assertion prop run in morph (by
fusing blocks), allows the jit to avoid morphing some unreachable blocks
(thus saving a bit of TP), and lays the groundwork for more aggressive early
branch folding that would be useful (eg dotnet/runtime#27935).

Fix test property evaluation on netfx vertical (#1715)

When running on netfx vertical there's a chance that the `XunitConsoleNetCore21AppPath` property which is defined in https://github.com/dotnet/arcade/blob/master/src/Microsoft.DotNet.XUnitConsoleRunner/src/build/Microsoft.DotNet.XUnitConsoleRunner.props#L4 isn't available because the packages isn't restored / imported. Guarding against that by checking if the property is set.

Enable upgrading loops before alternations to be atomic (#1690)

Changing loops to be atomic is beneficial because it avoids unnecessary backtracking (and if everything in the expression can be proven to avoid backtracking, we also can generate better code overall).  This change improves upon our detection to now consider the case where the second node is an alternation, e.g. with the expression `\s+(something|anything)`, we can change that to the equivalent of `(?>\s+)(something|anything)` because we can see that for the alternation to succeed, it would never begin with a space, and therefore there's nothing the preceding loop could "give back" that would let the alternation succeed.

Use platform-matrix for Installer jobs (#1623)

* Use platform-matrix for Installer jobs

Minimal changes to root pipelines to avoid conflicts

* Fix Linux vs. linux mismatch in musl jobs

Fix runtest.py test analysis and repro creation (#1702)

Comment out the code that munges characters in the test output.
This is corrupting the output. Don't delete it until it is
better understood if this is actually needed in some case.

Fixes #1682

Fix arguments to DebugEnv() constructor, which allows repro
scripts to be written for failing tests.

Remove unused function in deps parsing (#1683)

[reflection] Add additional TypeInfo.FullName tests (#1613)

Move Assembly.GetEntryAssembly method to shared part (#1670)

Add basic Capture support to non-backtracking RegexCompiler (#1639)

Lots of regexes end up using captures, either by design or accidentally because parens are a natural grouping mechanism.  Support for them means many more regexes can now use the non-backtracking compiler that produces much better code gen.

We now also auto-atomicify more loops by ensuring we traverse concatenations as part of the checks.  This means that an expression like "(a|b)*c" now becomes the equivalent of "(?>([ab])*)c.

And we remove unnecessary Atomic nodes at the root, e.g. "(?>foo)" becomes just "foo".

Port changes to crossgen2 shared files (#1709)

Changes from dotnet/corert#7944 to enable x86 support.

Remove codebase argument from AssemblyNative::Load (#1697)