Create a GCHandleTable interface with Init/Shutdown.

Separate finalizer thread creation from GC initialization in EE Startup.

Merge pull request #10503 from sandreenko/fgOptimizeDelegateConstructor-cleanup

small fgOptimizeDelegateConstructor code refactoring

Get VS to quit bugging me about replacing SR.cs already. (#10530)

Fix https://github.com/dotnet/coreclr/issues/10358

Add support for CoreCLR testing via ILLINK on Linux

Made changes to the test infrastructure to
1) Generate <test>.sh with commands to invoke ILLINK and run the output
2) Add runtest.sh --link=<path-to-illink> to invoke the ILLINK testing

Fix comment on mark::recover_plug_info (#10516)

Merge pull request #10507 from eerhardt/TZBinarySearch

Binary Search in TimeZoneInfo.GetAdjustmentRuleForTime

[Local GC] Use standard C++ types (bool) and consistent types (void*) on the interface (#10463)

* [Local GC] BOOL -> bool on IGCHeap

* [Local GC] size_t -> void* on IGCHeap

* [Local GC] Silence warnings by being explicit about BOOl -> bool conversions

* Address code review feedback: FinalizeAppDomain BOOL -> bool

* Fix warnings

* Address code review feedback:
  1) Fix a missed default parameter (FALSE) on a parameter of type bool,
  2) Fix invocations of the diagnostic callbacks to use boolean literals
     instead of TRUE and FALSE,
  3) Fix various invocations of GC interface methods in the VM to use
     boolean literals instead of TRUE and FALSE

* Address code review feedback: fix inconsistency

Fix usage text of runtest.sh (#10518)

Update CoreClr to beta-25128-03 (#10517)

Binary Search in TimeZoneInfo.GetAdjustmentRuleForTime

Since Unix uses IANA tzdata files, there can be hundreds of adjustment rules for each time zone.  To make the search for a given rule faster, use a binary search since the rules are guaranteed to be in chronological order.  See ValidateTimeZoneInfo.

Fix #5716

Don't ignore exceptions thrown from handlers of some events (#10502)

Fixes dotnet/corefx#14747:
- Events include: AssemblyLoadContext.Unloading, AppDomain.ProcessExit
- Made the same change for AppDomain.DomainUnload for consistency, but it's not raised

Fix link for up-for-grabs issues (#10488)

* Fix link for up-for-grabs issues

Move deleted files and remove m_ prefix in shared file

Add DateTimeFormat and Parse to shared and remove moved file

Update CoreClr, CoreFx to beta-25127-03, beta-25124-07, respectively (#10468)

Some missing stragglers (#10509)

Implement TLS field handles for RyuJIT/x86 (#10510)

Implement TLS field handles for RyuJIT/x86

This isn't used by .NET Core, but helps us run desktop testing.

small changes in fgOptimizeDelegateConstructor

Delete unnecessary condition.
Use default methods to work with lists.
ArgList construction reordering.

Add heuristic to trigger GC to finalize dead threads and clean up han… (#10413)

Add heuristic to trigger GC to finalize dead threads and clean up handles and memory

A thread that is started allocates some handles in Thread::AllocHandles(). After it terminates, the managed thread object needs to be collected by a GC for the handles to be released. Some applications that are mostly idle but have a timer ticking periodically will cause a thread pool thread to be created on each tick, since the thread pool preserves threads for a maximum of 20 seconds currently. Over time the number of handles accumulates to a high value. Thread creation adds some memory pressure, but that does not force a GC until a sufficiently large handle count, and for some mostly idle apps, that can be very long. The same issue arises with directly starting threads as well.

Fixes #6602:
- Track a dead thread count separately from the current dead thread count. This is the count that will contribute to triggering a GC, once it reaches a threshold. The count is tracked separately so that it may be reset to zero when a max-generation GC occurs, preventing dead threads that survive a GC due to references from contributing to triggering a GC again in this fashion.
- If the count exceeds a threshold, enumerate dead threads to see which GC generation the corresponding managed thread objects are in. If the duration of time since the last GC of the desired generation also exceeds a threshold, trigger a preferably non-blocking GC on the finalizer thread.
- Removed a couple of handles and some code paths specific to user-requested thread suspension, which is not supported on CoreCLR

Move Pointer.cs to the shared partition. (#10499)

* Move Pointer.cs to the shared partition.

(The two internal helpers are not yet used
on CoreRT but that's only because CoreRT
doesn't yet implement Pointer support
for Reflection invoke. Exposing them
unconditionally now to save us the trouble
of removing the #ifdef's later.)

Path may contain spaces. Wrap in quotes. (#10493)

Merge pull request #10484 from mikedn/map-vn-assert

Correctly update bitvector in optAddVnAssertionMapping

Merge pull request #10489 from stephentoub/capturethrow

Add static ExceptionDispatchInfo.Throw method

Add static ExceptionDispatchInfo.Throw method

Change typedef for BOOL in gcenv.base.h to int. (#10486)

Update for Android cross build script (#10483)

Updated URLs for libandroid-support and libandroid-support-dev downloads.
Added URLs and folders for libunwind and libunwind-dev
Removed libunwind build

CoreFx 14486 Update API summary. (#10485)

Correctly update bitvector in optAddVnAssertionMapping

When short bitvectors are used only the first assertion was stored in the map stored bitvector, all subsequent assertions were stored into copies of that bitvector.

Enable global invariant on OSX (#10470)

* Enable Globalization Invariant on OSX

* Fix typo

* Fix small typo

* Remove static link to ICU lib

* Addressing the feedback

Typo correction (#10482)

Delete pointless <internalonly/> comments (#10479)

Issue #7061 Remove private _Prop from StreamReader, StreamWriter (#10480)

Some BitVecOps improvements (#10437)

* Avoid virtual calls when allocating bitvectors

* Use specialized BitVecOps when possible

* Avoid unnecessary bitvector copying

* Cleanup CSE_DataFlow

Add Interlocked.MemoryBarrierProcessWide (#10476)

Contributes to #16799

Remove return statement and throwing directly for obselete methods. (#10477)

Merge pull request #10469 from briansull/ibc-dump

Change the Jit Disasm message when we have profile data

Merge pull request #10466 from briansull/stress-fix

Fix for JitStress test failures

Move DateTime to shared partition (#10460)

* Move DateTime to shared partition

Depends on dotnet/corefx#17484

* Use CoreCLR specific impl of UtcNow on windows

* Use the internal call on unix as well

* Port good corert changes

Cleanup unnecessary import (#10472)

Rename resource strings to match resx (#10475)

Fixed m_ prefixes in a few files from shared partition (#10473)

Change the Jit Disasm message when we have profile date to include
information about the edge weights and the value of fgCalledCount.
Example:

; with IBC profile data, edge weights are valid, and fgCalledCount is 145

CoreFx 14486 Remove argument validation in DangerousCreate (#10462)

CR feedback
Removed assert about BB_NONE

Remove dead strings (#10445)

Removed 436 dead strings from .resx, kept 1034

Sort entries in shared project (#10465)

Fix for JitStress test failures

Merge pull request #10459 from dotnet/mirror-merge-9061692

Mirror changes from dotnet/corert

Delete moved file

Use Array.Copy with index and remove coreclr specific function

NS2.0 Complete DBNull's surface area. (#3085)

* Copy over CoreCLRs' DBNull.cs

* Make it presentable, then move it into the shared partition.

Update CoreClr to beta-25123-08 (#10435)

Merge pull request #10149 from JosephTremoulet/DestSingleField

Scalarize single-field-to-lclVar block copy

Merge pull request #10451 from ionuttamas/patch-1

Update ryujit-overview.md

[Linux][GDB-JIT] Add lexical scopes info for local variables (#10307)

* Fix memory leak when getting local variable names

* Pass locals from SymbolReader using new LocalVarInfo struct

* Map IL to native addresses for variable scopes

* Dump variables with scopes

* Add scope info to VarDebugInfo

* Remove unnecessary usage of lines info in BuildDebugInfo

* Use holders for scopes and local var names

* Move array of FunctionMember from global scope to local scope

* Move SymbolNames from global scope to local scope

Copy-paste errors found by PVS-Studio. (#10450)

I used PVS-Studio static analyzer to check this project. I would like to suggest a variant of the way to fix the copy-paste errors, detected with the help of V778 diagnostic. Description of the diagnostic: https://www.viva64.com/en/w/V778/

Update ryujit-overview.md

Update ci-trigger-phrases.md (#10444)

- Add Linux arm/x86 CI trigger phrases
- Add Ubuntu16.10 x64 pri0 trigger phrase
- Remove Ubuntu15.10 x64 pri0 trigger phrase

Add PNSE messages (#10440)

* PNSE messages

* threading

* marshal

* interop svc

* More

* more

* Remove env var special casing for UWP

* Pending question

* remove pinvoke

Merge pull request #10426 from BruceForstall/FixIlasmRoundtrip

Enable FEATURE_METADATA_EMIT_ALL for non-crossgen compiles

[Local GC] Redirect fatal errors to the EE (#10436)

* [Local GC] Redirect fatal errors to the EE

* Address code review feedback: don't fetch the IP when reporting a fatal error

GetContainingObject is inconsistent with the lowest address it chooses. (#10438)

For profiling purpose it only cares about condemned ranges; for byref
validation it cares about any object on the heap.

Merge pull request #10400 from briansull/ibc-counts

Change setBBProfileWeight so that we no longer multiply a profile counts by 100

Merge pull request #10434 from dotnet/mirror-merge-9060302

Mirror changes from dotnet/corert

Move StringBuilder to shared partition (#10357)

* Move StringBuilder to shared partition

* Fix PR comments

* s/CoreClr/CoreCLR

* Remove unused ThreadSafeCopy overload

* Actually rename the file

Delete NewMerger

This was only used as part of C++ link.exe for IJW

Enable FEATURE_METADATA_EMIT_ALL for non-crossgen compiles

This allows ilasm roundtrip test to work with NetStandard 2.0
changes where C# compiler emits a ".permissionset" attribute
into the assembly that ildasm emits. This define enables the
APIs that ilasm uses to process this attribute.

Re-enable ilasm roundtrip test.

Fixes #8418

Merge pull request #10417 from viva64/master

Missing comma

Add missing

Remove moved files

porting desktop changes to enable up to 1024 CPUs (#10404)

porting desktop changes to enable up to 1024 CPUs

Move IReflect.cs to shared partition. (#3074)

 Split AssemblyAttributes.cs into properly named files.

Move the easy files in System\Reflection to the shared partition. (#3060)

* Prepare the "easy" System\Reflection .cs files for move to shared partition

Files to be moved the shared partition (after https://github.com/dotnet/coreclr/pull/10387 side is in)

AmbiguousMatchException.cs
AssemblyAttributes.cs
AssemblyContentType.cs
AssemblyNameFlags.cs
Binder.cs
BindingFlags.cs
CallingConventions.cs
CustomAttributeFormatException.cs
DefaultMemberAttribute.cs
EventAttributes.cs
ExceptionHandlingClauseOptions.cs
FieldAttributes.cs
GenericParameterAttributes.cs
ICustomAttributeProvider.cs
ImageFileMachine.cs
InterfaceMapping.cs
IntrospectionExtensions.cs
InvalidFilterCriteriaException.cs
IReflectableType.cs
ManifestResourceInfo.cs
MemberFilter.cs
MemberTypes.cs
MethodAttributes.cs
MethodImplAttributes.cs
Missing.cs
ModuleResolveEventHandler.cs
ParameterAttributes.cs
ParameterModifier.cs
PortableExecutableKinds.cs
ProcessorArchitecture.cs
PropertyAttributes.cs
ReflectionContext.cs
ReflectionTypeLoadException.cs
ResourceAttributes.cs
ResourceLocation.cs
StrongNameKeyPair.cs
TargetException.cs
TargetInvocationException.cs
TargetParameterCountException.cs
TypeAttributes.cs
TypeFilter.cs

* Prepare the "easy" System\Reflection .cs files for move to shared partition

Files to be moved the shared partition (after https://github.com/dotnet/coreclr/pull/10387 side is in)

AmbiguousMatchException.cs
AssemblyAttributes.cs
AssemblyContentType.cs
AssemblyNameFlags.cs
Binder.cs
BindingFlags.cs
CallingConventions.cs
CustomAttributeFormatException.cs
DefaultMemberAttribute.cs
EventAttributes.cs
ExceptionHandlingClauseOptions.cs
FieldAttributes.cs
GenericParameterAttributes.cs
ICustomAttributeProvider.cs
ImageFileMachine.cs
InterfaceMapping.cs
IntrospectionExtensions.cs
InvalidFilterCriteriaException.cs
IReflectableType.cs
ManifestResourceInfo.cs
MemberFilter.cs
MemberTypes.cs
MethodAttributes.cs
MethodImplAttributes.cs
Missing.cs
ModuleResolveEventHandler.cs
ParameterAttributes.cs
ParameterModifier.cs
PortableExecutableKinds.cs
ProcessorArchitecture.cs
PropertyAttributes.cs
ReflectionContext.cs
ReflectionTypeLoadException.cs
ResourceAttributes.cs
ResourceLocation.cs
StrongNameKeyPair.cs
TargetException.cs
TargetInvocationException.cs
TargetParameterCountException.cs
TypeAttributes.cs
TypeFilter.cs

* Move to shared.

Merge pull request #10336 from noahfalk/wer_resource

Support Windows Minidump

Mark b425314 Incompatible (#10431)

b425314 has internal logic to determine if it making forward progress

If test takes too long, test self check fails and test aborts with a
failure.  This makes this test difficult in GCStress/HeapVerify situations
where run time is substantially longer.

This test was already marked incompatible for x86.  Remove x86 condition
to mark incompatible on al platforms.

Merge pull request #10432 from AndyAyersMS/ImproveTypes3

Jit: track ref types for inlinee locals and args

Update CoreClr to beta-25123-07 (#10428)

Use roslyn common props (#10430)

[PAL] Do NOT check execute bit in INIT_FindEXEPath (#10342)

* Do NOT check execute bit in INIT_FindEXEPath

* Do NOT lookup path during PAL_Initialize

This commmit removes path lookup inside INIT_FindEXEPath, and renames
INIT_FindEXEPath as INIT_EXEPath.

Migrate the Reflection serialization holders to the shared partition. (#10429)

* String => string, SR.GetResourceString() => SR.

* Copy over CoreRT files verbatim.

* Get things building and running again.

* Don't bring CoreRT's visibility sins into CoreCLR

* String => string, Object => object

* De-"m_" the field names.

* Move the files to the shared directory.

Merge pull request #10411 from rahku/fix10197

Re-enable tests

Merge pull request #10397 from briansull/ibc-dump

Improve the dumping of BasicBlock weights and edge counts in the JIT

Merge pull request #10402 from BruceForstall/FixMulCseBug

Fix CSE bug with GT_MUL and GTF_MUL_64RSLT

Jit: track ref types for inlinee locals and args

The jit may use new temporaries for inline locals and args. For temps
representing ref type locals and args, propagate the class handle from
the appropriate signature. This makes more types to be available for
devirtualization.

Also in addition to capturing arg ref types, cleanup `impInlineFetchArg`
by commoning out the table lookups and adding a header comment.

[Unix] runtest.sh crossgen should exit on failure (#9907)

* Fix crossgen segfault

* Check crossgen in runtests.sh

* Do not use test -v

CI showed bash 4.2 feature test -v is not present on
OSX CI systems.  Fall back to more standard bash test

We have found and fixed a security weakness (CWE-188) using PVS-Studio tool: Analyzer warning: V557 Array overrun is possible. The value of 'dwCode - 1' index could reach 8.
PVS-Studio is a static code analyzer for C, C++ and C#: https://www.viva64.com/en/pvs-studio/

Scalarize single-field-to-lclVar block copy

Morph has some logic to scalarize struct copies where the RHS is a scalar
and the LHS is a single-field struct.  This change adds the symmetric
logic where the LHS is a scalar and the RHS is a single-field struct.
This pattern shows up in Span<T> methods that initialize the data pointer
field, due to the way scalar replacement operates on spans.

Move IReflect.cs to shared partition. (#10415)

I used PVS-Studio static analyzer to check this project. I would like to suggest a variant of the way to fix the error, detected with the help of V534 diagnostic. Description of the diagnostic: https://viva64.com/en/examples/v534/ (#10414)

Incorrect variable 'i' from the outer loop is used in condition of the inner loop.

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

Update CoreClr, CoreFx to beta-25123-03, beta-25122-02, respectively (master)

[x86/Linux] fix broken stack in DelayLoad_Helper when converting to AT&T syntax assembly. (#10383)

* fix broken stack in DelayLoad_Helper when converting to AT&T syntax assembly.
"push \frameFlags" was changed to "pushw $0x0" by converting to AT&T syntax assembly.

Signed-off-by: ragmani <ragmani0216@gmail.com>
* [x86/Linux] modify pushing frameFlags stack from Intel syntax to AT&T syntax.

Signed-off-by: ragmani <ragmani0216@gmail.com>

Update CoreClr, CoreFx to beta-25123-03, beta-25122-02, respectively

Merge pull request #10064 from rahku/cgroup

memory limit in containers on linux

Re-enable tests

memory limit in containers on linux

Various fixes for generating portable RID packages (#10409)

Add handling of multiple CMake installations in probe-win (#9980)

* Add handling of multiple CMake installations in probe-win
Check if multiple CMake versions are installed, and use by default the first path available

Fixes #8221

* Fix typo in value accessor

* Restore previous null check in probe_win

* Remove redundant assignment

Removing the special handling in classlibnative for atan2(±∞, ±∞) and pow(-1.0, ±∞). (#10295)

* Removing the special handling in classlibnative for atan2(±∞, ±∞) and pow(-1.0, ±∞).

* Fixing up the logic for HAVE_COMPATIBLE_POW in the PAL layer.

Fix fallout from switching corelib to use resx (#10407)

Fixes #https://github.com/dotnet/corefx/issues/17413