Merge pull request #9416 from wtgodbe/Suse13

Remove OpenSuse13 from CI

Clean up GetAddrOfSecurityObject (#9423)

Clean up EECodeManager::GetInstance (#9424)

* Clean up EECodeManager::GetInstance

* Use !CROSSGEN_COMPILE instead of CROSSGEN_COMPILE

Clean up EECodeManager::GetGSCookieAddr (#9425)

Clean up EECodeManager::GetReturnKind (#9426)

Clean up EECodeManager::GetFunctionSize (#9427)

Clean up EECodeManager::GetParamContextType (#9428)

Clean up EECodeManager::GetExactGenericsToken (#9429)

Clean up EECodeManager::GetParamTypeArg (#9430)

[x86/Linux] Revise TAStackCrawlCallBackWorker (#9432)

Merge pull request #9431 from BruceForstall/Fix9374

Generate better code for x86 SIMD get[i] intrinsic

Merge pull request #9191 from sandreenko/CORERT-fat-call-transformation

CoreRT fat call transformation

[x86/Linux] Fix Calling Convertion Mismatch inside ilasm/ildasm (#9410)

Delete unnecessary domain sanity check (#9420)

Generate better code for x86 SIMD get[i] intrinsic

For get[i] intrinsic, with LONG base type and constant index, we can
compute the decomposed indices statically.

Fixes #9374

Merge pull request #9392 from BruceForstall/FixGtGetOp2

Add gtGetPossibleOp2()

[ARM32/Linux] Fix definition and CMake for cross-architecture build (#9262)

- Fix ABI flag based on target
- Exclude corefx for cross-architecture component

Span Slice method AggressiveInlining (#9404)

CoreRT calli transformation

Transform managed calli when compiling for CoreRT.

Add gtGetOp2IfPresent()

It was noticed that many uses of gtGetOp2() fully expect op2 to
exist and be non-nullptr, and the use of gtGetOp2() was simply
for style/readability purposes. However, it has a cost, as it
checks whether the tree is binary first, and returns nullptr if
not. For most cases, this is unecessary and expensive.

Introduce a new gtGetOp2IfPresent() function that captures the
previous behavior: checking if the tree node is binary before
returning op2, or returning nullptr otherwise. gtGetOp2() is
changed to simply return gtOp2 directly, without any checking
in non-DEBUG builds. It can be used if you know the op2 exists.

Most uses of gtGetOp2() were left alone (and hence get the new
behavior). The ones that need the old behavior were renamed
gtGetOp2IfPresent().

Mostly newer code is affected, as older code would generally
access gtOp2 directly, whereas newer code started using gtGetOp2()
for style and symmetry, especially after gtGetOp1() was introduced.

Merge pull request #9382 from BruceForstall/CallFinallyCleanup

Remove unnecessary parameter to genCallFinally()

Merge pull request #9386 from BruceForstall/FixSwitchTable

Fix compare/branch switch mode expansion

Filter RefEmit modules in SendEventsForNgenMethods (#9418)

* Filter RefEmit modules in SendEventsForNgenMethods

Fixes #6427

Merge pull request #9417 from wtgodbe/publishNativeBins

Publish test native bins from correct directory

Publish test native bins from correct directory

Remove OpenSuse13 from CI

Merge pull request #9169 from JosephTremoulet/UntrackedMem

Value-number byref loads

Value-number `ByrefExposed` memory and loads

Teach value numbering to keep track of the current value number for
`ByrefExposed` memory, updating it appropriately at relevant stores.
Introduce a new VN operator `ByrefExposedLoad`, used for loads of
address-exposed locals and loads by indirs that don't have more specific
value numbers, which is a function of the current `ByrefExposed` memory VN
and the pointer VN.  This allows loop hoisting and CSE to recognize
redundant loads via byrefs when there are no intervening stores.

Fixes #7903.

Define `MemoryKind::ByrefExposed`

Add a new `MemoryKind` to represent "any memory that byrefs may
reference", called `ByrefExposed`.  All definition points of `GcHeap`
become also definition points of `ByrefExposed`, and additionally, so do
definitions of address-exposed locals.

Because it is a common case (currently happening in 90% of methods in
System.Private.CoreLib) that a method has no definitions of
address-exposed locals, have liveness detect this condition and set the
new `byrefStatesMatchGcHeapStates` flag accordingly; when the states
match, the same `MemoryPhi`s, defnums, and `PerSsaData` are then shared
between the two memory kinds, to avoid excess allocations/processing.

This change defines `ByrefExposed` memory and its def/use points, and
builds SSA for it, but no optimizations make use of it, so this is a
no-diff change.

Introduce `MemoryKind` abstraction

Re-cast the notion of "heap" (in liveness, SSA, and value-numbering) as
one of potentially many `MemoryKind`s, called `GcHeap`.  Update names,
comments, data structures, and signatures as appropriate to parameterize
relevant data/methods over `MemoryKind`.  This change is a no-diff
refactoring, and currently `GcHeap` is the only `MemoryKind`.  Generally,
codepaths which will generically need to process all `MemoryKinds`s
(initializing, dumping, dataflow propagation) now iterate over all
`MemoryKinds`, and codepaths which are sensitive to the semantics of the
specific `MemoryKind` (def/use identification in liveness and value
numbering) are changed to specifically operate on `MemoryKind::GcHeap`.

One notable exception is that `lvMemoryPerSsaData` and `CountForMemoryDef`
are *not* parameterized over `MemoryKind`; there's a single "space" of SSA
defnums for memory defs (though the same tree can incur different defs for
different memory kinds [in which case their defnums will differ]), to
facilitate subsequently sharing SSA nodes across memory kinds when
appropriate.

Refactor fgCurHeapVN and HeapSsaMap updates

Consolodate the code for updating the current heap VN and HeapSsaMap
entries into new helper method `recordHeapStore`, to make it easier to
evolve that code.  Change `fgValueNumberArrIndexAssign` to return the new
heap VN to allow callers to pass it down into `recordHeapStore`
accordingly.

This is just a refactoring; no change to compiler behavior.

Merge pull request #9329 from JosephTremoulet/LateArgVN

Get call arg value numbers from late args

Clean up EnumGcRefs (#9412)

Merge pull request #9408 from jashook/disable_checked_arm64

Disable checked arm64 jobs until jobs are stable

[x86/Linux] Use SP as StackMarker (#9389)

Disable checked arm64 jobs until jobs are stable

Jobs currently are expected to run red until a lstFile update goes in. Therefore,
disable the job as a PR trigger until it is clean.

Add ComVisible(true) back to several special types (#9403)

It is required by WinRT/COM interop. Fixes #9401.

Remove more CAS (#9390)

* Remove PermissionSet

* Remove HostProtectionAttribute

* Remove PermissionState

* Remove S.Security.Permissions

* Remove IPrincipal

* Fix native side

* Remove model.xml again

Fix field type for ByReference<T> and TypedReference (#9284)

Fix field type for `ByReference<T>` and `TypedReference`

Fixes #7894

Add scripts & docs which support creating an Android toolchain on Linux (#9175)

* Add scripts & docs which support creating an Android toolchain on Linux

Merge pull request #9385 from jashook/lower_block_store_arm64

Call LowerBlockStore TreeNodeInfoInitBlockStore

Design portable tail calls with helpers (#8596)

* Design portable tail calls with helpers

Currently we don't handle non-trivial cases of tail calls on Unix due to the non-portable and
complex way it is implemented. This pull request describes a design of a portable way of
handling these cases so that it can be used uniformly over all our jitted and AOT (ahead of time)
platforms.

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

Update CoreClr, CoreFx to beta-25007-01, beta-25007-01, respectively (master)

[x86/Linux] Clean up EHContext-related code (#9395)

Remove bclrewriter (#9351)

* Remove bclrewriter

* Dump crossgen error to Jenkins log

Clean up methods for USE_GC_INFO_DECODER (#9394)

Move GC/Coverage/delete_next_card_table to the Long GC playlist (#9376)

Fix a typo in TailCallFrame::UpdateRegDisplay log (#9381)

Merge pull request #8849 from seanshpark/fixstackalign

[x86/Linux] Stack align 16 bytes for JIT code

[ARM32/Linux] Fix calling convention - stdcall in crossgen (#9364)

* [ARM32/Linux] Fix calling convention - sdcall in crossgen

- Fix calling convention:
not use stdcall for ENCODEMODULE_CALLBACK & DEFINETOKEN_CALLBACK in linux/x86
x86-host/arm32-target crossgen & x86 crossgen
build error in clang 3.9: not allowed calling convention casting

Update CoreClr, CoreFx to beta-25007-01, beta-25007-01, respectively

Fix compare/branch switch mode expansion

It appears we never hit this case due to a very low
minSwitchTabJumpCnt number that almost always forces
us to use jump tables. This was found in the RyuJIT/ARM32
bring-up, because ARM uses a higher number.

Also, add a stress mode to force using the compare/branch expansion
mode more frequently, during stress.

Call LowerBlockStore TreeNodeInfoInitBlockStore

For arm64 call LowerBlockStore before TreeNodeInfoInitBlockStore
as done on other architectures.

Remove unnecessary parameter to genCallFinally()

Merge pull request #9377 from RussKeldorph/defaultprivate

Default-trigger ARM64 testing on certain users' PRs

[x86/Linux] Stack align 16 bytes for JIT code

Change JIT code to align stack in 16 byte used in modern compiler

Default-trigger ARM64 testing on certain users PRs

Re-using the logic that allows certain users to request ARM64 testing to
default-trigger ARM64 testing on their PRs.  Ideally, we'd be able to
separate the list of people who have permission to trigger testing from
those that get testing by default, but that is harder than it seems.

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

Update CoreClr, CoreFx to beta-25006-03, beta-25006-01, respectively (master)

Merge pull request #9325 from wtgodbe/bashScripts

Add #!/bin/bash to beginning of shell test scripts

Merge pull request #9319 from DrewScoggins/X86PerfRuns

Add x86 perf runs to CoreCLR

Merge pull request #8644 from BruceForstall/SimdLongDecomp

Enable SIMD for RyuJIT/x86

Merge pull request #9371 from jkotas/attributes

Delete COM and security transparency attributes

Change /bin/bash to usr/bin/env bash

Merge pull request #8827 from hseok-oh/nuget_armel

Generate nuget package for armel (Tizen, Debian)

Update CoreClr, CoreFx to beta-25006-03, beta-25006-01, respectively

Clean up FillRegDisplay (#9360)

Delete remaining security transparency annotations

Delete ClassInterface attributes

Delete ComVisible attributes

Android - Fix duplicate definitions of INT_MIN and PAGE_SIZE in unit tests (#9172)

* Don't refdefine PAGE_SIZE on Android

* Don't re-declare INT_MIN on Android

* Fixup

Merge pull request #9368 from stephentoub/cq_fixes

Two fixes to ConcurrentQueue

CAS Security cleanup (#9355)

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

Update CoreClr to beta-25006-02 (master)

Two fixes to ConcurrentQueue

- Changed comparisons to use subtraction to avoid overflow issues on sequence numbers.
- Added an exception message where a TODO was left in the code

Update CoreClr to beta-25006-02

Enable SIMD for RyuJIT/x86

This change implements support for Vector<long>, handling
SIMDIntrinsicInit, which takes a LONG, and decomposition of
SIMDIntrinsicGetItem, which produces a LONG.

It also enables SIMD, including AVX, by default for RyuJIT/x86.

Delete dead code from ExecutionContext (#9359)

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

Update CoreClr to beta-25006-01 (master)

Merge branch 'master' into nuget_armel

[x86/Linux] Add Assertion on pvControlPc (#9356)

Update CoreClr to beta-25006-01

Update CoreClr, CoreFx to beta-25005-03, beta-25004-01, respectively (#9350)

Port 121d095ed0b0076fb1c7ff59e6446fd19d506b32 from Windows to Linux (#9346)

Port #5821 to Linux (#9347)

Port #5339 to Linux (#9348)

Merge pull request #9342 from stephentoub/tasks_inlining

Remove NoInlining/StackCrawlMarks from Tasks

Address PR feedback and other cleanup

Addressed PR comment about removing ExecutionContext.Capture(ref StackCrawlMark, ...), which led me to remove NoInlining/StackCrawlMarks from some other places in Timer, Thread, ThreadPool, and Overlapped.

In doing so, I found several other unnecessary members on ExecutionContext that could be removed with minor tweaks elsewhere in the source, e.g.
- Dispose is a nop, so we can remove explicit try/finally's to clean it up
- ExecutionContext.Run(..., bool) just ignores the bool arg, so it can be removed and all call sites redirected (but I've left it in EC, as it appears it's used via internals visible from a library in corefx)
- FastCapture() just calls Capture, so the call sites can be changed (but I've left it in EC for a similar reason)
- PreAllocatedDefault can be removed in favor of Default; etc.
- ExecutionContext.Capture itself checks whether flow is suppressed; doing a check before it adds a TLS lookup and in doing so optimizes for an uncommon case in exchange for making the common case more expensive.  I've removed those checks.

And in the process, I also noticed that several lazily initialized delegates that no longer need to be, and cleaned those up.  These were lazy due to needing to be for security checks that aren't relevant in coreclr.

Remove NoInlining/StackCrawlMarks from Tasks

These crawl marks in Tasks aren't relevant in coreclr (they're passed down to ExecutionContext, which in coreclr just ignores them), yet they required NoInlining, which does impact coreclr.  Removing them all from tasks.

Merge pull request #9343 from stephentoub/tasks_selfreplicating

Remove internal self-replicating support from Tasks

Remove internal self-replicating support from Tasks

In the initial TPL prototype, Tasks had a feature that allowed them to "replicate", meaning to schedule a copy of themselves.  Prior to shipping the initial version, this feature was made internal, as it was deemed unnecessary, but by then the Parallel loops constructs had already been built on top of them.  Now that Parallel no longer depends on this feature (and isn't in CoreLib), we can entirely rip it out.

Rename internal CoreLib Console to LowLevelConsole (#9341)

Merge pull request #9338 from jkotas/against-packages

Add mscorlib.dll back to Microsoft.TargetingPack.Private.CoreCLR

Use StringBuilderCache for Enum.InternalFlagsFormat (#9340)

Remove stripped code (#9269)

Delete essentially all the code that the BCL rewriter is removing.
Added back some code it shouldn't have been removing, eg., serialization code, debugger visualizers, and some changing of 'protected' to 'protected internal'

Merge pull request #9240 from BruceForstall/FixSimdFieldMorphingDeadCode

Remove SIMD dead code

Add mscorlib.dll back to Microsoft.TargetingPack.Private.CoreCLR

Removing it broke BuildFromPackages flavor of test build

Revert "Update CoreClr, CoreFx to beta-25004-01, beta-25004-01, respectively (#9330)"

This reverts commit e8dff5550c022d2854914dc534f9ec3d7fe623ac.

Merge pull request #9324 from gkhanna79/Fix9030

Use fallback load context as binding context for dynamic parent assembly

Remove duplicate attributes from CoreLib (#9327)

Update CoreClr, CoreFx to beta-25004-01, beta-25004-01, respectively (#9330)