Merge pull request #10958 from pgavlin/VSO397793

Preserve VNs in fgMorphIntoHelperCall.

Merge pull request #10936 from BruceForstall/TempRegApi

Introduce API for codegen getting temp registers from gtRsvdRegs

Add a brief comment to the new test.

Small clean up of stackEntry (#10820)

Make all stackDepth equal unsigned int.
Delete possible wrong function (the comment said : "// used in the inliner, where we can assume typesafe code. please don't use in the importer!!", but it was used).
Delete methods, that used arguments as output buffers.

* delete code without effects

Introduce API for codegen getting temp registers from gtRsvdRegs

LSRA puts a set of temporary registers needed by a node in the
gtRsvdRegs register mask. Currently, evey codegen function that
needs to use a temporary register manually manipulates this mask.

Introduce a few functions to make this simpler and more regular:

1. GetSingleTempReg() // Gets a temp; asserts there is exactly one temp reg.
2. ExtractTempReg() // Gets the lowest temp, removes it from gtRsvdRegs so subsequent calls don't see it.
3. AvailableTempRegCount() // Returns a count of available temp registers.

All take an optional register mask, so you can extract a register from
just the set specified by the mask (typically RBM_ALLINT or RBM_ALLFLOAT).

Merge pull request #10955 from adityamandaleeka/makeHandleStoreClass

Make a GCHandleStore class and interface for use by the VM

Merge pull request #10956 from AndyAyersMS/FixMorphUnnecessaryNullCheck

Jit: fix unnecessary null checks in some field accesses

Make HandleStores contain their underlying bucket directly.

[Local GC] Move operations on CLREventStatic to the EE interface (#10813)

* [Local GC] Move operations on CLREventStatic to the EE and add their functionality to the interface

* Fix a missed case

* Split GetWaitForGCEvent into two smaller interface methods to avoid exposing the event itself on the interface

* Initial implementation for Unix

* Complete unix implementation

* Make it work on Windows

* Remove redudant methods from GCToEEInterface

* Fix the Linux build

* First part of code review feedback: make GCEvent dispatch statically (Windows)

* Second part of code review feedback: make GCEvent dispatch statically (Unix)

* Standardize implementation across Windows/Unix (apparently MSVC is more lenient about constructor names than clang)

* Address code review feedback: Add Create*Event methods back onto GCEvent and remove them from GCToOSInterface

* Address code review feedback: remove a dead define

* Remove a bad comment, remove an unnecessary friend class, fix some formatting issues

* Fix an issue when initializing a GCEvent on Linux (should not be allocating an Impl in the constructor)

* Fix the same issue on Windows (less bad, just leaks memory instead of asserting)

[x86/Linux] Use appropriate FCALL macro for DoToDecimal (#10963)

Implement MEM_RESET handling in VirtualAlloc on Unix (#10939)

Rename IGCHandleTable to IGCHandleManager.

Merge pull request #10945 from OmarTawfik/update-runtimefeature-docs

Update comment to reflect how RuntimeFeature members should be defined

Reenable incremental linking (#10943)

Preserve VNs in fgMorphIntoHelperCall.

This method is used to transform a node into a semantically-equivalent
helper call, and should preserve any value numbers that are present on
the the node.

[Local GC] Handle table low-hanging interface violations (#10929)

Jit: fix unnecessary null checks in some field accesses

Morph was sometimes passing the existing MorphAddressContext down
to fgMorphField even when the field access was for a field value.
If that context contained indefinite offsets, morph would then
insert an explicit null check on the object pointer for the field
access.

Typically the field offset is small enough that this explicit check
is not needed. The implicit check done when fetching the field's value
is sufficient protection.

The fix is to have `fgMorphSmpOp` clear out the context for child
`GT_FIELD` nodes, unless the field parent is a `GT_ADDR`. Note
if there is an `op2` node the parent cannot be `GT_ADDR` so these
field children always get an empty context.

No tests added since this kicks in reasonably frequently in corelib and
elsewhere in frameworks.

Closes #10942.

Make a GCHandleStore class and interface for use by the VM.

Merge pull request #10953 from wtgodbe/crossImage

Use updated Docker images for Crossbuild

Use updated Docker images for Crossbuild

Merge pull request #10927 from wtgodbe/PortableOSXWin

Enable building Core_Root for any RID against packages

Merge pull request #10949 from wtgodbe/ImageVersion

Update Ubuntu14.04 Cross Docker image

Update Ubuntu14.04 Cross Docker image

Update comment to reflect how RuntimeFeatures should be defined

Merge pull request #10922 from adiaaida/fixXUnitPerf

Fix run-xunit-perf.cmd for local use

[x86/Linux] Use appropriate FCALL macro for SafeBuffer methods (#10935)

Remove distro specific RIDs from SOS.NETCore's project.json. (#10925)

Ensure Environment.ExitCode works correctly after app domain unloaded. (#10842)

* Ensure Environment.ExitCode works correctly after app domain unloaded.

This PR addresses 2 problems of Environment.ExitCode:

1. Can't get correct exit code of main function.
2. Can't set %errorlevel%.

Details can be found on #6206

Fix #6206

Update CoreClr to preview1-25213-02 (#10937)

Merge pull request #10875 from rahku/fix_x86_gcstress

modify lastAvAddress when adjusting context for Stub

[ARM32/Linux] Build libcoreclrtraceptprovider.so (#10406)

* Build libcoreclrtraceptprovider.so for ARM32 and modify cmake

- Build libcoreclrtraceptprovider.so for ARM32
- Include libcoreclrtraceptprovider.so in nuget package
- Don't build libcoreclrtraceptprovider.so for
Tizen: lttng is not supported in Tizen
- Exclude packaging libcoreclrtraceptprovider.so based on os-name
excluded for tizen

* Remove libcoreclrtraceptprovider.so before test for ARM32/Linux

Remove libcoreclrtraceptprovider.so before test for ARM32/Linux in CI
CI test environment using docker with qemu has some problem to use lttng library

Create ELF core dump on coreclr exceptions and signals. (#10902)

* Create ELF core dump on coreclr exceptions and signals.

When coreclr aborts because of a unhandled managed exception or hardware signal, the
"createdump" utility is launched if the "COMPlus_DbgEnableMiniDump" environment
variable is set to "1".

This utility uses the DAC enumerate memory interface to determine what memory regions
should be written for the managed state plus all the normal Linux memory regions for
the native state like threads, stacks, DSO, etc.

There still needs some to configure some of the "policies" like the name/location of
the core dump and the type of minidump (MiniDumpNormal, MiniDumpWithFullAuxiliaryState,
MiniDumpWithPrivateReadWriteMemory, etc.).

Currently only supported on Ubuntu Linux.

Make createdump smaller by using PAL in DAC module.

Add platform (x64 & arm64) specific optimized jit helpers for statics access (#10912)

Merge pull request #10930 from rahku/marshalfix

HelperMethodFrame missing at certain places in method MarshalNative::PtrToStructureHelper

HelperMethodFrame missing at certain places in method MarshalNative::PtrToStructureHelper

Merge pull request #10914 from hqueue/jit/remove_unused_fuction

Remove unused function in BasicBlock

Merge changes from TFS (#10926)

* Fix desktop build break

[tfs-changeset: 1654125]

* JIT formatting

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)

Enable building Core_Root for any RID against packages

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

Fix run-xunit-perf.cmd for local use

run-xunit-perf.cmd  specifies the full path to the benchmark using
%WORKSPACE%, which a user may not set locally. This change modifies it to
use %CORECLR_ROOT%.

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

Remove unused function in BasicBlock

lastTopLevelStmt() is declared in BasicBlock,
but not defined and unused throughout JIT.

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>

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

Fix CI tizen build error (#10909)

Fix CI tizen build error caused by chmod

modify lastAvAddress when adjusting context for Stub

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