Merge pull request #10374 from dotnet/mirror-merge-9057970

Mirror changes from dotnet/corert

Move DefaultBinder.cs to the shared partition. (#10355)

FileStream redirection got changed. Add ThreadPoolBoundHandle redirect.

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

[tfs-changeset: 1651640]

Fix leak in Task.WaitAny(..., int) (#10331)

WaitAny is effectively built on top of WhenAny, creating a continuation from the supplied tasks and then blocking on that continuation.  When a timeout is provided, it blocks with that timeout.  But if it doesn't complete within the timeout, it ends up leaking the continuations it created into the constituent tasks.  The fix is simply to force the returned continuation to complete, such that its continuation logic does all of the appropriate cleanup.

Update CoreClr, CoreFx to beta-25121-02, beta-25122-01, respectively (#10310)

Switch coreclr corelib to use resx (#10372)

* Revert "Revert "Switch coreclr corelib to use resx (#10268)""

* Add resources.target override

This fixes the GenerateResource task running on the arm emulator runs.

* Fix run.sh to have a working Tools-Override

[Local GC] Forbid inclusion of gcscan.h from VM and DAC directories (#10332)

* Forbid inclusion of gcscan.h from VM and DAC directories

* Address code review feedback - hoist IsGCHeapInitialized check to an assert, since the heap should definitely be initialized if we are validating objects that ostensibly came from it

Merge pull request #10363 from jkotas/revert

Revert "Switch coreclr corelib to use resx (#10268)"

Revert "Switch coreclr corelib to use resx (#10268)"

This reverts commit 12ef04c22b07f1e4ec5a63009298e4537a2c8e1a.

Fix Linux/x86 call alignment calculation and insertion (#10266)

* Fix Linux/x86 call alignment calculation and insertion

The existing system of calculating per-call alignment adjustments
did not consider the possibility of nested calls. On x86, some
arguments for a call might be pushed on the stack before a
nested call's arguments start being pushed. Thus, a call can't
consider only its own arguments when determining the stack level.

Instead, use the existing genStackLevel variable, updated
dynamically during code generation, to determine the baseline
stack alignment when a call first needs to push something on
the stack, or if it has no stack arguments, whether it needs to
align the stack before a call.

One wrinkle here is that the fgAddCodeRef() function for adding
throw blocks for array bounds checks, and other similar helpers,
need to know the stack level on entry, and this stack level is
computed during argument morphing. There is a tough phase ordering
problem here. So, we bail out and force an EBP frame if there are
any such throw helpers in the function. When using an EBP frame,
the helper block doesn't need to know the stack level -- it is
only needed for ESP frames, needed for unwinding. Note that this
bail out already existed if the same helper needed multiple different
stack levels on entry. We could do slightly better without too much
work by not bailing out for top-level calls with no stack arguments.

* Track max alignment added for nested calls during codegen

This information is needed by Linux/x86 for an assert about
maximum emitter stack depth.

* Make max stack align tracking available in release as well

The assert these are used in is a noway_assert.

* Formatting

[x86/Linux] Invoke gcResetForBB() in genFuncletProlog (#10186)

Merge pull request #10343 from parjong/fix/self_override

Fix bogus assert in resolveVirtualMethodHelper and update related ilproj

Add /silent option to crossgen (#10350)

This sets the NGenOptions.fSilent flag, which prevents displaying
the final output message.

This is useful for JIT asm diff generation.

Switch coreclr corelib to use resx (#10268)

* Switch to resx

* Use roslyn rewriter to switch from GetResourceString to SR

* More GetResourceString changes

* Add missing resource

* Remove Environment.GetResourceString and replace final usages with SR

* Remove comment

* Fix spacing

* Replace final instances of Environment.GetResourceString

* Add another missing resource

* Add back Environment.GetResourceStringLocal because the runtime needs it

System.IO.Path.IsPathRooted does not check if valid drive letter on Windows (#10323)

Merge pull request #10254 from sdmaclea/PR-FIX-SCRIPT-ERROR

Fix script ERRORLEVEL typo

Temporarily disable frequently failing Sleep PAL tests (#10313)

See #8321

Merge pull request #10321 from WinCPP/issue-7060

Issue #7060 - Make StreamReader.ReadBuffer non-virtual

Merge pull request #10327 from mikedn/assertion-info

Refactor GenTree assertion information storage

[x86/Linux] Log UpdateRegDisplay calls (#10335)

Fix bogus assert in resolveVirtualMethodHelper and update related ilproj

[x86/Linux] Use 4-byte align for double in struct (#10340)

[x86/Linux] Correct update on ResumableFrame (#10339)

Merge pull request #10341 from dotnet/mirror-merge-9056283

Mirror changes from dotnet/corert

Delete moved file

NS2.0 - Complete System.Security namespace.

- Add GetObjectData() implementation to System.SecurityExcepton

- Clean up a bit and move to shared partition.

'root' as a special case in UTIL_IsExecuteBitsSet (#10246)

Fix MAC build (#10328)

This is temporary fix  to unblock the CI and coreclr building for MAC. this change should be reverted after having the proper fix for how we load ICU on MAC

Merge pull request #10325 from dotnet/mirror-merge-9055987

Mirror changes from dotnet/corert

Merge pull request #10330 from omajid/fix-markdown-atx-headers

Fix markdown headings

Fix markdown headings

GitHub recently changed how atx headings (beginning with `#`) are
rendered. A space is now required between `#` and the following text
for the heading to be recognized.

Update headings in the docs to match the expected format.

See https://github.github.com/gfm/#atx-headings

Delete moved files

Refactor GenTree assertion information storage

Stealing a bit from gtAssertionNum is ugly  and error prone (e.g. bool HasAssertion() { return gtAssertionNum != 0; } is incorrect if the edge bit is set).

Fix another per code review

Merge pull request #10319 from JosephTremoulet/NullField

Permit null object in `GenTreeUseEdgeIterator`

Move Type.cs and Module.cs to Shared Partition.

Types.cs is split into the main file, Type.Helpers.cs and Type.Enum.cs.
All three will be shared.

- GenericTypeArguments - reconcile with CoreClr
  (the code is identical provided that the underlying
  Type provider implemented IsConstructedGenericType
  correctly. But we can't assume that for third party types.)

- IsInterface and IsSerializable has some CoreClr
  specific logic for RuntimeTypes - moving this out
  to the unshared space for now.

- IsContextFulImpl()/IsMarshalByRefImpl() - corrected
  implemention to match CoreClr.

- IsIntegerType - no CommonRuntimeTypes in CoreClr
  so going back to typeof() checks - the runtime implemented
  types override all this enum code anyway on both runtimes.

- GetRootElementType() - upgraded visibility to "internal"
  as a concession to CoreClr.

- CoreCLR's IsVisible has a fast path - we'll keep that.

- Moved IsRuntimeImplemented() out of Internal.Reflection.Core.NonPortable
  namespace so calls to it can be more easily shared.

- Module.MethodHandle was not implemented correctly for
  non-runtime modules (it's supposed to return default(ModuleHandle)).
  This makes it behave correctly and reconciles Module.cs
  with the CoreCLR version.

Merge pull request #10317 from dotnet/mirror-merge-9055826

Mirror changes from dotnet/corert

Issue #7060 - Make StreamReader.ReadBuffer non-virtual

Permit null object in `GenTreeUseEdgeIterator`

We use `GT_FIELD` for static field loads, which have no instance pointer.
Update the iterator constructor to recognize this as a valid state rather
than fail an assertion.

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

Merge changes from TFS

Fix FileStream for ProjectN

[tfs-changeset: 1651325]

Remove message about falling to default for MethodContext::repGetEEInfo()

[tfs-changeset: 1651465]

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

Merge changes from TFS

Initial document on AssemblyLoadContext Design (#10271)

* Initial document on AssemblyLoadContext Design

Merge pull request #10262 from briansull/refactor2

Updates for the handling of Profile data in the JIT

Make GetModuleHandle() sharable. (#10309)

Merge pull request #9445 from hqueue/arm/ci

[ARM/CI] ARM CI script for arm and armel

Reconciles Type.cs with CoreRT version for move to shared partition. (#10306)

* Clone files and add resource strings.

* Subset each partial class.

* String=>string and Environment.GetResourceString => SR.cs

Doing this upfront with reduce the upcoming diffs - hopefully.

* Add IsRuntimeImplemented() emulator.

* One method was put in the wrong file. Correcting.

* Converted Type.Enum.cs to CoreRT style member by member.

* Converted Type.Helpers.cs to CoreRT style member by member.

* Internalize __Filter.cs into Type.Helpers.cs

* Pretransform to reduce diffs.

Removed contracts and comments,
"abstract public" -> "public abstract"

* Converted Type.cs to CoreRT style member by member.

* Eh.. rather not share IsInterface and IsSerializable than have that #if CORECLR.

* Transplant the CoreRt files (now just a reordering.)

[Arm64/Unix] Set PROCFS_* available in tryrun (#10255)

Desktop side of the change to remove COR_JIT_EE_VERSION

1. Bring over the JIT-EE interface file changes,
2. Stop building ctpjit.dll, and delete its build rules,
3. Stop building skipjit.dll.

[tfs-changeset: 1651372]

Invariant globalization (#10264)

* Invariant Globalization Work

* Convert the testing Exceptions to asserts

* Remove un-needed comment

* Fix typos

* Fix unrelated typo

* Address the PR feedback

* More feedback addressing

* More feedback addressing

* Fix Linux break

* More feedback addressing

* cleanup

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

Update CoreClr to beta-25119-02 (master)

Update CoreClr to beta-25119-02

Fix code to build with older VC++ compiler (#10281)

Remove potential overflow in SafeBSTRHandle (#10182)

Fix a typo in exceptoinhandling.cpp (#10237)

I hoisted the accounting for LOH into allocate_large_objects to avoid missing counting in certain stages during bgc when we go through bgc_loh_alloc_clr instead of adjust_limit_clr (#10284)

Fix for Module.GetMethod(string) (#10283)

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

Update CoreClr to beta-25118-03 (master)

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 CoreClr to beta-25118-03

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

Updates for the handling of Profile data in the JIT

Use the type weight_t instead of unsigned when dealing with BasicBlock weights.
Added new method getCalledCount to return the number of times that the method was called.
Added new method fgProfileRunsCount to return the number of scenario runs for the profile data.
Made the method fgIsUsingProfileWeight public instead of protected
Renamed fgCalledWeight to fgCalledCount
Updated the logic in getBBWeight and added comments explaining how it works.

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