Merge pull request #10272 from dotnet/mirror-merge-9051754

Mirror changes from dotnet/corert

Merge pull request #10216 from JosephTremoulet/SpanBench

Update SpanBench test

Update SpanBench test

 - Re-enable the tests that were disabled in e859c309.
 - Re-work how tests are invoked from the command line to require less
   boilerplate
     - Now each test has a single [Benchmark] entrypoint
     - That entrypoint invokes its test's single inner-loop by wrapping it
       in a lambda that it passes to a new helper Invoke method (shared
       across all tests) which handles the xunit vs. command-line
       differences..
     - Main finds the entrypoints by using reflection to search for the
       [Benchmark] attributes, so the explicit list of stringified test
       names is no longer needed and the command line will run the same
       set of tests that xunit-perf does.
 - The new SpanAPI tests now get invoked when this tests is run from the
   command line as well.
 - Add [NoInlining] to the SpanAPI tests' kernels.
 - Add some heap writes and conditional writes to prevent deadcode
   optimization from eliminating tests' kernels (assuming we don't do
   interprocedural constant propagation or deadcode, or store sinking and
   final value calculation, or PRE...).
 - Split the Index SpanAPI tests into one version that uses a
   loop-invariant index and thus should get hoisted out of the loop, and
   another version that uses variant indices.

Merge pull request #10054 from pgavlin/GenTreeUseEdgeIterator

Improve GenTreeUseEdgeIterator.

Delete moved files

Remove JIT-EE version ifdefs (#10273)

* Remove COR_JIT_EE_VERSION define

Currently, we maintain two versions of the JIT-EE interface in the
source tree: the current version, and .NET 4.6 version. This is
managed by the `COR_JIT_EE_VERSION` define. Any new changes to the JIT-EE
interface must be put under `#if COR_JIT_EE_VERSION > 460`. As of now,
there are quite a few of these `#if` statements in the tree.

We started doing this so we could build and ship a CTP (Community
Technology Preview) version of the JIT from the current source tree that
runs against earlier versions of the runtime that are widely available
to customers, so those customers can try out the new JIT and give us
early feedback. This was done during the RyuJIT/x64 bring-up before
shipping .NET 4.6, to get as much feedback as possible before replacing
JIT64. After 4.6 shipped, we re-based the old version of the JIT-EE
interface to the 4.6 version, and started adding the above `#if`
conditions, with the thought that if we ever shipped another CTP, it
would be expected to run on top of 4.6. We have not shipped such a
thing, and we currently have no plans to build another CTP in the same
way we did before. (Early adopters can try out the "live" JIT in the
.NET Core open source project, although that JIT can't be used on the
desktop.)

So, to simplify the code, and reduce the cost of adding to or changing
the JIT-EE interface, we are removing the `COR_JIT_EE_VERSION` define.

Merge pull request #10278 from dotnet-bot/master-UpdateDependencies

Update CoreClr to beta-25118-02 (master)

Move ParameterInfo.cs and Assembly.cs to shared partition. (#10270)

* Clone the files.

* Split the members between shared and nonshared files.

* Replaced ParameterInfo with CoreRt text member by member.

* Replaced Module.cs with CoreRt text member by member.

* Remove dead filter code.

* Replaced Assembly.cs with CoreRt text member by member.

* Transplanted CoreRt files.

This is a reordering without any other changes.
Verified by sorting the old and new files and
diffing.

* Migrated ParameterInfo.cs and Assembly.cs to shared directory.

Leaving Module.cs in place for now as one line is going to
have to change over in CoreRt land.

 ModuleHandle ModuleHandle => new ModuleHandle(this);

* Cleanup.

* Move the final internal methods to RuntimeParameterInfo.

Update CoreClr to beta-25118-02

Merge pull request #10269 from BruceForstall/FixBuildAll

Fix build.cmd "all" to work with pass-through args again

Update CoreClr, CoreFx to beta-25117-08, beta-25118-01, respectively (#10267)

Improve GenTreeUseEdgeIterator.

Instead of executing a rather long set of conditionals on each call
to `GenTreeUseEdge::operator++`, decide on the function that will be
used to advance the iterator's state in the constructor.

Merge pull request #10220 from dotnet/revert-10206-MultiQueue

Revert "Add support for Multiqueing jobs in Helix"

Merge pull request #10274 from AndyAyersMS/FixMoreCasing

Fix yet another casing issue.

Fix overflow when computing dwWeight in CPUGroupInfo::InitCPUGroupInfoArray (#10104)

Fix overflow in dwWeight when there are large number of CPU groups

Merge pull request #10241 from briansull/refactor1

Refactor the flag test of BBF_PROF_WEIGHT

Fix triggers for ILLINK job (#10244)

This change has the following fixes:
1) Fix the trigger phrase for ILLink jobs so that PRs can request ILLink runs per architecture.

2) Pass the linker location explicitly on the runtest.cmd command line
instead of using the %ILLink% variable, since the build/run steps
are different command shells.

3) Don't run the x86compatijit configuration daily in the lab

Fix yet another casing issue.

This might be what is causing the TFS mirror to have hiccups.

FileStream.Unix: open with CLOEXEC unless FileShare.Inheritable set (#10251)

* FileStream.Unix: open with CLOEXEC unless FileShare.Inheritable set

* PR feedback

Move EventInfo.cs to shared partition.

Fix build.cmd "all" to work with pass-through args again

Merge pull request #10238 from pgavlin/SeekUnrollLimit

Reduce case iteration counts in SeekUnroll.

Refactor the flag test of BBF_PROF_WEIGHT
to always call the new method hasProfileWeight()

Merge pull request #10259 from AndyAyersMS/FixNoway

JIT: Fix noway_assert when inlining

Merge pull request #10263 from dotnet-bot/from-tfs

Merge changes from TFS

Merge pull request #10260 from sdmaclea/PR-ARM64-REMOVE-X19-WORKAROUND

[Arm64/Unix] Remove x19 corruption workaround

Clean up GenTreePutArgStk (#10239)

Consolidate constructors; go from 4 to 2.

Reduce `#ifdef` in callers.

Merge pull request #10245 from rahku/fix10048

LoadFromStream should throw badimageformat early on

LoadFromStream should throw badimageformat early on

Merge pull request #10253 from dotnet/mirror-merge-9051394

Mirror changes from dotnet/corert

fix syntax error in build.sh (#10256)

Delete unused ReferenceLocalMscorlib property from test proj files (#10250)

Deleted a few other irrelevant properties as well while I was on it.

Fixes #7711

Add reflection roots for tests calling CreateInstance() (#10242)

Add the XML files identifying the reflection roots to keep the
type (and constructors threof) which are used by tests as
Activator.CreateInstance(typeof(T))

fix formatting

Merge pull request #10192 from AndyAyersMS/InterfaceDevirt

Interface devirt

Address feedback.

[Arm64/Unix] Remove x19 corruption workaround

JIT: Fix noway_assert when inlining

The jit keeps track of which inline arguments can be modified via starg
during the inlinee IL prescan to ensure that a temp is used to represent
such arguments in the inlined body. There is a noway_assert the a starg
is procecssed during importation that double-checks that a temp is really
being used, and this assert was firing.

The prescan guraded the starg tracking with a check to avoid an out-of-bounds
write into the argument table when scanning bad IL with bogus argument numbers.
The predicate was checking the argument number against lvaTableCount.
Unfortunately for inlinees this value is not related to the number of arguments.
In particular it some rare cases it may be an underestimate so the prescan might
end up failing to note a starg. If the the jit then tries to inline and the
caller passes a constant, the assert will then fire.

Fix is to update the guard to use the number of callee arguments. This is
known to be less than the table size by earlier checks.

Added a test case.

Closes #9891.

Delete moved files

Attribute the other types used by FileStream. Adds the internal methods the old CoreFX uses.

https://devdiv.visualstudio.com/DefaultCollection/DevDiv/_workitems?id=396302&_a=edit

[tfs-changeset: 1651151]

CompilerServices APIs

[tfs-changeset: 1651141]

Merge pull request #10248 from mikedn/fix-vnarrbound

Fix ValueNumStore::IsVNArrLenUnsignedBound checks

Merge pull request #10201 from cod7alex/add_csharp_editorconfig_codestyle_settings

Added to .editorconfig csharp code style settings supported by VS 2017

[x86/Linux] Pass return buffer on reverse P/Invoke (#10226)

* Pass return buffer on reverse P/Invoke

* Use PLATFORM_UNIX and track m_cbRetPop locally

* Declare cbRetPop UINT16 instead of int

Support devirtulization of Encoders (#10229)

* Seal more final inherited internal types

need to account for the allocated bytes correctly for the following: (#10162)

need to account for the allocated bytes accurately for the following cases

Merge pull request #10235 from dotnet/mirror-merge-9050192

Mirror changes from dotnet/corert

Move the MemberInfo-derived Info classes to the shared partition. (#10236)

* Copy over CoreRt prose member by member.

* Transplant the CoreRt version of the files.

This is now just a reordering. Verified
by sorting the old and new versions of the
file and diffin.

* Move to shared partition.

* Looks like the #if FEATURE_COMINTEROP wasn't just for show...

Take EventInfo.cs back to unshared status for now...

[Local GC] Break EE dependency on GC's generation table and alloc lock in single-proc scenarios (#10065)

* Remove usage of the generation table from the EE by introducing an
EE-owned GC alloc context used for allocations on single-proc machines.

* Move the GC alloc lock to the EE side of the interface

* Repair the Windows ARM build

* Move the decision to use per-thread alloc contexts to the EE

* Rename the lock used by StartNoGCRegion and EndNoGCRegion to be more indicative of what it is protecting

* Address code review feedback 2 (enumerate the global alloc context as a part of GCToEEInterface)

* Code review feedback (3)

* Address code review feedback (move some GC-internal globals to gcimpl.h and gc.cpp)

* g_global_alloc_lock is a dword, not a qword - fixes a deadlock

* Move GlobalAllocLock to gchelpers.cpp and switch to preemptive mode when spinning

* Repair the Windows x86 build

Merge pull request #10240 from pgavlin/fixerrorlevel

Fix test failure detection when dumps are enabled.

Fix test failure detection when dumps are enabled.

Add a lab job for Coreclr testing via ILLINK (#10200)

This commit adds a job that:
1) Clones https://github.com/mono/linker
2) Builds the linker for netcoreapp2.0
3) Builds CoreCLR and tests using the IlLinker built above

The job is run daily, and can be reqested from PRs as necessary.

Reduce case iteration counts in SeekUnroll.

This test was taking too long to run on x86. After investigation, it
does not appear that there is a fundamental problem with the code
generated for the test; rather, it seems that we are running an
excessive number of iterations when using this test as a correctness
test. This change reduces the number of times we run each case from 10
to 1 unless otherwise specified on the command line.

Fix ValueNumStore::IsVNArrLenUnsignedBound checks

VNFuncApp::m_func should be checked first to be sure that m_args[0] and m_args[1] contain valid VNs. It's not clear if m_func can ever be something other than a binary func in the case of a relop VN but the code doesn't look right as it is.

Add IBC support to managed build (#10180)

Add a new build switch, `ibcinstrument`, that adds `/Tuning` on the `crossgen`
command line when building System.Private.CoreLib and its peers. Automatically
consume IBC optdata during builds when these conditions are met:

1. `ibcinstrument` is *not* passed to the build,
2. optdata is available
3. ibcmerge is available

Note that `optdata` will not yet be restored with this change; once packages for
master are made available, a new package reference will still need to be added.
This PR attempts to unblock manually testing IBC, as well as surrounding
CI/infrastructure work.

To help produce an IBC-optimized build using manually generated profile data,
run the newly added `tests/scripts/optdata/bootstrap.py` script. It will
configure the build to consume IBC data from a path automatically, and print out
that path where profile data can be copied.

Add some test cases

Interface call devirtualization: jit side of changes

Unblock VM queries where the base class is an interface (queries
where the implementing class is also an interface are still blocked
as no resolution is possible). Pass along the context handle that
the jit got from getCallInfo.

Interface call devirtualization: VM side of changes

Extend resolveVirtualMethod to take in the owner type needed to resolve
shared generic interface calls, and implement VM support for interface
call devirtualization. Add a check that the class presented by the jit
implements the interface, to catch cases where the IL is missing type
casts.

Update jit GUID, SPMI and zap to reflect the changed signature.

Fix CoreCLR build break in shared corelib partition (#2997)

Move RelocatedTypeAttribute under PROJECTN ifdef

Enable FileStream in ProjectN build, use CreateFile2 instead of CreateFile.

[tfs-changeset: 1651001]

Removed check for the maximum quantity of arguments (#10232)

Update CoreClr to beta-25116-01 (#10163)

Fixing more file-class correspondence violations. (#10231)

* Cloned files.

* Reduce each file to its namesake class.

* Cleanup

* Fix build break.

Merge pull request #10214 from pgavlin/TimeoutCrashDumps

Take crash dumps for timed-out tests.

Remove a default argument in gentree.cpp to fix internal builds.

[tfs-changeset: 1651089]

Take crash dumps for timed-out tests.

This change updates the test infrastructure to collect crash dumps for
tests that time out. We've been seeing a number of tests that only time
out in CI (e.g. #10076); hopefully this will help us root-cause the
timeouts.

Merge pull request #10164 from stephentoub/fs_pipe_crash

Ignore FileStream.FlushWriteBuffer failures during finalization

Delete mscorlib facade (#10157)

Fixes #7607

Merge pull request #10177 from eerhardt/TZPosixNames

Support bracket enclosed names in TZ POSIX rules

Hide types which are now public in corefx (#10142)

Minor fix to address ?: VC conformance improvement. (#10208)

Visual C++ has made some conformance changes to conditional operator
that will be available under /permissive- and which make the inference of
result type of the conditional operator in these 2 places ambiguous. This
happens because the class type in one of the arguments provides both: the
constructor from and the conversion operator to T - LCWSTR here.

Temporarily disable flaky sleep test (#10221)

Fix Convert.FromBase64CharArray with empty array (#10224)

[x86/Linux] Fix NativeCallableTest (#10060)

* [x86/Linux] Fix NativeCallableTest

* Move m_cbActualArgSize adjustment outside if statement

* [x86/Linux] Adjust m_cbActualArgSize in case pStubMD == NULL

* [x86/Linux] Move m_cbActualArgSize calculation to #else part of '#ifdef _TARGET_X86_'

* [x86/Linux] Remove redundant computations, add comments for m_cbActualArgSize meaning

Merge pull request #10168 from eerhardt/PortablePDBs

Make portable PDBs on Unix instead of no symbols at all.

Enable use of Volatile<T> on arm64 to get correct memory consistency (#10212)

Clean up fcall.h (#10158)

Reorder stack arguments on reverse P/Invoke (#10217)

[x86/Linux] Fix IL_STUB_PInvoke with RetBuf (#10144)

* [x86/Linux] Fix IL_STUB_PInvoke with RetBuf

Fix calling convention and IL_STUB_PInvoke for native functions
which was problematic for native functions that has
a RetBuf argument(struct size <= 8) on x86/Linux.

Fix #10027

Revert "Add support for Multiqueing jobs in Helix"

Merge pull request #10218 from dotnet/revert-10213-buildType

Revert "Add BuildType to helixpublish.proj"

Merge pull request #10219 from dotnet/revert-10210-distroRid

Revert "Make user specify RID for publishing test native bins into"

Revert "Make user specify RID for publishing test native bins into"

Revert "Add BuildType to helixpublish.proj"

Merge pull request #9876 from sdmaclea/PR-ARM64-ForeignThreadExceptionsNative

[Arm64] Enable building ForeignThreadExceptionsNative

Fix ToTitleCase Functionality for Dutch Cultures (#10195)

* Add Methods to Handle Dutch Titlecasing

Added the `IsDutchCulture` and `IsIjAtCurrentPosition` methods to handle
resolving a special-case for Dutch titlecasing, which should properly
capitalize any instances of "IJ" at the beginning of a titlecased word.

* Moved `IsDutchCase()` Call Into Local Variable

Moved the check for Dutch culture outside of the title-loop to avoid unnecessary evaluations.

* Performance Changes and Improvements to Dutch Titlecasing

Removed `IsDutchCulture()` method in favor of an inline approach relying
on the `StartsWith()` method; Remove the `IsIjAtCurrentPosition()`
method in favor of another inline approach to improve performance.

* Minor Formatting Change

Added space for `isDutchCulture` line, as it managed to disappear somewhere along the way.

Merge pull request #10213 from wtgodbe/buildType

Add BuildType to helixpublish.proj

Add BuildType to helixpublish.proj

Change CoreCLR to use Sierra (#10173)

* Switch to full 10.12
* Disable query performance counter test

Merge pull request #10210 from wtgodbe/distroRid

Make user specify RID for publishing test native bins into

Merge pull request #10206 from wtgodbe/MultiQueue

Add support for Multiqueing jobs in Helix

Migrate MethodBase.cs over to the shared partition. (#10202)

* Replace each member with CoreRt prose.

* Lightup IsConstructedGenericMethod

* Copy over CoreRt MethodBase.cs (this is now just a reordering)

* Move MethodBase.cs verbative over to shared partition.

* AAAAND.... it looks like this exercise exposed a bug in CoreRt.

* Resolved merge conflict

Make user specify RID for publishing test native bins into

Move System.Convert code to shared partition (#10178)

* Move System.Convert code to shared partition
* Use ReferenceEquals for type equality

Add support for Multiqueing jobs in Helix

Merge pull request #10196 from wtgodbe/fixHelix

Build Targeting Pack after restoring packages

Added to .editorconfig csharp code style settings supported by VS 2017

Prepare MethodBase.cs for migration to shared partition. (#10194)

* Clone the files w/out changes.

* Distill each file to its intended subset.

* Minimize usings.

* Renamed to *.CoreClr.cs - extended free car wash to fix the nits.

Build Targeting Pack after restoring packages

Merge pull request #10166 from alpencolt/ryu-arm-arr-tests

Added Array tests to CodeGenBringUpTests