Merge pull request #10917 from stephentoub/fix_badges

Fix build badges in OfficialAndDailyBuilds.md

[x86/Linux] Fix ResumeEsp before resume (#10749)

* [x86/Linux] Correctly unwind FCALL frames

* Fix Up ResumeEsp inside ProcessCLRException

* Fix Context Before PC Update

[x86/Linux] 16-byte aligned BackPatchWorkerAsmStub (#10759)

* [x86/Linux] 16-byte aligned BackPatchWorkerAsmStub

* Use fixed alignment padding

* Do NOT push ebx (no longer required)

Merge pull request #10905 from pgavlin/VSO406158

Fix a number of reordering issues in shift decomposition.

[x86/Linux] 16-byte aligned DelayLoad_MethodCall (#10760)

Merge pull request #10706 from sdmaclea/PR-ARM64-GC-EmitInsLdStOp

[Arm64] Fix GC Holes in emitInsLoadStoreOp

Merge pull request #10923 from wtgodbe/ImproveReporting

Improve reporting parameters for pipebuild

Merge pull request #10911 from adityamandaleeka/handle_table_local_gc_hide_internals

[Local GC] Hide handle table internals from VM side

Improve reporting parameters for pipebuild

Fix closed delegate deserialization to account for extension methods (#10901)

Functional fix for #9597:
- The code was incorrectly assuming that the serialized target type info refers to the target object's type; it actually refers to the method's reflected type, which may not be the same type when it's an extension method

Merge pull request #10243 from sdmaclea/PR-CALLI-LOWERING-ALLOWED

Allow target of calli to be lowered to ldc.i4

NS2.0 AssemblyName.ReferenceMatchesDefinition() (#10919)

This api was broken from its debut in 2.0 (and yet somehow
scored a 2.9% on .NET api catalog)

 https://connect.microsoft.com/VisualStudio/feedback/details/752902/assemblyname-referencematchesdefinition-returns-true-even-though-the-parameters-dont-match

In 2012, the MSDN docs were changed to match its broken behavior:

   Return Value
   Type: System.Boolean
   true if the simple assembly names are the same; otherwise, false.

   Remarks

   The comparison depends only on the simple assembly name. It ignores version, culture, and public key token.

A more accurate description would be:

   This api takes the simple names of the ref and def.
   Then it parses the simple names *as if they were*
   full-blown assembly names with optional Version, Culture
   and PKT (!)

   Then it applies a RefToDef test to that name using a test
   that's quite separate from the one used by the actual
   CoreCLR loader (so any similarities between
   it and the actual binding rules is purely coincidental.)

   Oh, and that code assumes the Def name came from trusted
   sources so if it's has a null Culture property, it NULL-AV's.

We could replicate this behavior on CoreRT (relying on
its own AssemblyBinder) but the more I progressed, the
more the venture failed to pass the laugh test (though
at least we got a nice AssemblyBinder cleanup
as a side effect.)

So what we'll do is make the api behavior match the
MSDN description (and likely come close enough to the compat bar)
but in a more straightforward fashion.

(The RefMatchesDef routine is used on one case - to
match assembly refs against a list of friend-access
assembly defs. So it's not quite dead code though
it probably should be.)

Merge pull request #10904 from BruceForstall/ArmMergeCodegenFuncs

Merge similar arm32/arm64 codegen functions

[x86/Linux] Funclet-based synchronization (#10791)

* [x86/Linux] Funclet-based synchronization

* Fix x86/Windows build error

* Revise per feedback

* Fix format error

Merge pull request #10852 from mskvortsov/ryujit-arm32-ldst-offset

[RyuJIT/ARM32] Encodable offsets for ld/st

Format code.

Update CoreClr to preview1-25212-04 (#10879)

Fix resource string name

Delete moved files

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

Change default clang version to clang3.9 in cross build (#10889)

If we use clang3.9 version, we will use O3 option for optimization.
Otherwise, use O1 option as before.

Fix build badges in OfficialAndDailyBuilds.md

- None of the badge links were correct, resulting in every badge always showing
as "Building" regardless of actual status
- The OSX badge links would 404.
- We were still using shields.io badges, which have been unreliable.

I've fixed all of these.

RyuJIT/ARM32: encodable offsets for ld/st

Rename overloaded "table" to "store" for VM side concept.

Fix CI tizen build error (#10909)

Fix CI tizen build error caused by chmod

Hide knowledge of handle table buckets and home heaps from VM side.

Delete Ref_ContainHandle and ContainsOBJECTHANDLE.

Add GetNewHandleTable and GetGlobalHandleTable to interface.

Remove GetHandleTableForHandle and add CreateDuplicateHandle.

Replace GetHandleTableContext with GetHandleContext.

Merge pull request #10882 from adityamandaleeka/double_init_appdomain_creation

Remove double initialization of variables in app domain creation

Merge pull request #10877 from adityamandaleeka/handle_table_local_gc_nullchecks

[Local GC] Move handle null checking code to VM side

Move files to shared CoreLib partition (#10885)

Merge pull request #10816 from adityamandaleeka/handle_table_local_gc_init

[Local GC] Move handle destruction code to IGCHandleTable

Merge pull request #10907 from wtgodbe/BuildTypeReporting2

Pass null instead of empty for PB_BuildType

Pass null instead of empty for PB_BuildType

Merge pull request #10906 from wtgodbe/BuildTypeReporting

Pass PB_BuildType as a Pipeline ReportingParameter

Pass PB_BuildType as a Pipeline ReportingParameter

Fix a number of reordering issues in shift decomposition.

Shift decompostition contained a fair number of IR rewrites that were
unintentionally reordering nodes. This change fixes these instances s.t.
they no longer reorder code.

Merge pull request #10899 from wtgodbe/PipelineReporting

Add last set of reportingParameters to pipelines.json

Merge similar arm32/arm64 codegen functions

Add last set of reportingParameters to pipelines.json

Merge pull request #10867 from AndyAyersMS/FixSingleDefBug

Jit: fix issue with single-def type propagation

CoreFx #15622 Dictionary<TKey, TValue>.Remove(TKey, out TValue) (#10203)

Merge pull request #10880 from BruceForstall/ArmCommon

Create common files for ARM32 and ARM64 common code

Merge pull request #10876 from pgavlin/VSO406156

Fix a few errors in cast decomposition.

Move a few more types from CoreFX to CoreLib (#10893)

Reducing shards between CoreCLR and CoreRT shards

Merge pull request #10894 from dotnet/Add_Build_Type

Add some BuildType and Subtype info to CoreCLR build pipeline definition

Merge pull request dotnet/corert#3288 from dotnet/nmirror

Merge nmirror to master

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

Create common files for ARM32 and ARM64 common code

Create codegenarmarch.cpp, lowerarmarch.cpp, lsraarmarch.cpp.

For codegenarmarch.cpp, I copied over a bunch of functions that
are almost identical between arm and arm64. I left the functions
intact (didn't change anything), so created one for ARM32 and
one for ARM64, unless they were truly identical. A follow-up
will be to actually merge the functions.

For lowerarmarch.cpp and lsraarmarch.cpp, I did the function
merge for nearly (or completely) identical functions, since
there were so many that were so similar.

Add some BuildType and Subtype info to CoreCLR build pipeline definition

Merge pull request #10886 from hseok-oh/ci/fix_ci_error

Fix CI test error: permission error

Merge pull request dotnet/corert#3285 from dotnet/nmirror

Merge nmirror to master

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

Remove sudo in netci.groovy

Overwrite permission in coreoverlay generation

fix runtest.sh to overwrite corerun's permission in coreoverlay generation

Fix netci.groovy

Fix netci.groovy instead of test script to avoid rebase

Fix CI test error: permission error

Fix CoreCLR/CoreFX copy order: permission problem

Do not spill eeStack after ldtoken opcode. (#10215)

Do not spill eeStack if the last opcode was ldtoken.

Remove double initialization of variables in app domain creation.

Fix some static methods on ThreadPool to wait for initialization (#10869)

Fix for part of #10521

Merge pull request #10782 from parjong/fix/x86_funclet_unwinding

[x86/Linux] Use GCInfo for funclet unwinding

Move handle null checks to VM side.

Update CoreClr to preview1-25210-04 (#10862)

Merge pull request #10871 from pgavlin/PipelineCompatJit

Package compatjit.dll in pipeline builds.

Merge pull request #10866 from pgavlin/VSO405852

Fix VSO bug 405852.

Fix a few errors in cast decomposition.

The cursor was not guaranteed to be in the correct location after cast
decomposition due to the location in which the high result was typically
placed. This mispositioning can cause re-decomposition, which can lead
to asserts e.g. when encountering a GT_LONG node that was left behind by
a previously-decomposed node. This change ensures that the high result
immediately follows the cast node so that the cursor will be in the
correct location following decomposition of the cast.

empty array (#10841)

Make EmptyArray private in favor of Array.Empty<T>

Merge pull request #10814 from rartemev/ilasm_reference_resolver_fix

Added 'netstandard' corelib name into ilasm reference resolver

Merge pull request #10808 from sdmaclea/PR-ARM64-enable-initblk-unroll

[Arm64] enable initblk loop unrolling

Extract genInterruptible setting as a separate region

Use hasPartialArgInfo instead of isPartialArgInfo

add test with deep execution tree without stores. (#10835)

Add a test that should fail with stack overflow if the compiler doesn't spill trees from time to time.

Package compatjit.dll in pipeline builds.

Just what it says on the tin.

Format code.

Move handle destruction code to IGCHandleTable.

Conflicts:
src/vm/gchandletableutilities.h

Fix VSO bug 405852.

This bug was due to a mismatch between the semantics of the FP modulus
operator as specified by ECMA-335 and as implemented by value numbering.

ECMA-335 requires the following behavior for FP modulus:
- If the divisor is 0 or the dividend is not finite, the result is NaN
- If the divisor is not finite and is not NaN, the result is the
  dividend.
Value numbering, however, simply used `fmod`, which does not have these
semantics. This change implements the required semantics in VN.

Add back UnmanagedType.LPUTF8Str (#10859)

CoreLib Build System portability updates (#10839)

Updates to corelib project file for target specific includes.  Intended
to ease maintainability of existing targets and clarify necessary work
for new targets.

Fix contract to allow call into GarbageCollect() to throw (#10856)

I did not intend to mark this function as NOTHROW, and there is no reason to expect NOTHROW here. Other functions that call into GarbageCollect() allow THROWS.

Merge pull request #10823 from adityamandaleeka/move_variable_handle_creation

Refactor variable strength handle creation

Add empty System.Runtime.CompilerServices.RuntimeFeature.IsSupported() API (#10779)

Added netstandard corelib into ilasm reference resolver

Fix perf penalty for creating dictionaries the easy way. (#10855)

We went to the trouble of mitigating the perf hit
introduced by using randomized string hashing.

I assume that said perf benefit wasn't meant to be denied
to the 90% case where people use the nullary
constructor to create the Dictionary...

Jit: fix issue with single-def type propagation

To avoid overly aggressive type propagation when there are multiple
reaching definitions, only propagate types to single-IL-def locals
when the definiting value comes from the same basic block as the store.
We check this conservatively by insisting that the block's entry stack
be empty.

Added a test case where the jit will improperly devirtualize without
such a check.

Closes #10858.

[Arm64] Cleanup/fix InitBlkUnroll

Update CoreFx to preview1-25210-01 (#10850)

Implement slicing APIs for ArraySegment<T> (#9926)

Incorrect parameter order found by PVS-Studio (#10497)

* Incorrect parameter order found by PVS-Studio
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 V3066 diagnostic https://www.viva64.com/en/w/V3066/

Get rid of "#if FEATURE_RANDOMIZED_STRING_HASHING" in shared partition. (#10851)

Since CoreCLR defines it (for now) and CoreRT doesn't, let's not have
"#if's" on this in shared code. It just creates confusion.

The only ones are in StringComparer.cs and fortunately, they are quite
expendable as...

1. It's only purpose is serialize Hashtables and Dictionaries in
   a way that CLR 1.1 can deserialize them. We don't care about that in
   .NET Core.

2. In CoreCLR as of today, this infrastructure is not changing the serialization
   blob one iota since all the GetEqualityComparerForSerialization() methods
   return "this."

So this removes the appearance of downgrading the comparers to non-randomized
versions for serialization purposes (and yes, we could go on and remove
all the other "let's do it for Everett and RTM" stuff in Hashtable.cs, but
I want to keep this PR on topic. Hashtable.cs is not going to be imported
into CoreRT so it's low priority for me.)

Updating CODE_OWNERS.TXT (#10834)

Added Nuget package implementation owners

Forbid lea without index and base. (#10771)

Delete FEATURE_READYTORUN define for CoreLib build (#10840)

This define does not have much value for the CoreLib build, and it was missing on arm64.
Also deleted dead props and targets files left over from the legacy build

[x86/Linux] Set ResumeEsp as Caller Sp when unwond to native frame (#10838)

[x86/Linux] 16-byte aligned ResolveWorkerChainLookupAsmStub (#10754)

* Fixed issue, when structures with many fields had wrong size in dwarf (#10797)

Move files to shared CoreLib partition (#10837)

Move variable handle creation.

[Local GC] Move knowledge of the free object method table outside of the GC (#10755)

* [Local GC] Move knowledge of the free object method table outside of the GC

* Revert "[Local GC] Move knowledge of the free object method table outside of the GC"

This reverts commit 02979e5eb12dd44edb2603e16411dd7d7ffc377d.

* Address code review feedback: leave ArrayBase alone, as it should be part of the binary contract between the EE and the GC

* Correct a bad merge

Move files to shared CoreLib partition (#10830)

Update VS2017 'Individual Components' install info (#10695)

A few small changes to make it easier to compare selections in the
installer UI against the list in the doc:

 - Re-order the list to match the order presented in the installer UI
 - Place the individual items under groupings matching the installer
 - Change casing to match installer UI

Delete reference to model.xml since it no longer exists (#10827)