Merge pull request #12575 from wateret/armel/floatret

[RyuJIT/armel] Move returned float to float reg

Support interprter build when INTERP_TRACING is not set (#12625)

Remove unused global variable (#12620)

Hide methods in IdDispenser (instead of using DacNotImpl) (#12624)

Update to latest System.Memory

Update CoreClr, CoreFx to preview2-25504-01, preview2-25504-01, respectively

Delete unnecessary code (#12607)

[RyuJIT/armel] Move returned float to float reg

Returned float values are in r0(float) or r0:r1(double).
These values must be moved to proper float register after call.

Fix #12574

Merge pull request #12608 from pgavlin/FixDSE

Fix a bug in LIR dead store removal.

create GetAssertionIndex to convert from unsigned to AssertionIndex (#12554)

Fix a bug in LIR dead store removal.

This came in with the recent operand visitor changes: instead of marking
the operands of a dead indirect store unused, DSE would mark the operands
of its address unsued.

[x86/Linux] Restore PTR_MethodTable in FastPrimitiveArrayAllocator (#12569)

Move the bitset iterator for bitsetasuint64inclass inside the bitset. (#12553)

It is how it is done for others bitsets.

Improve fgAddRefPred (#12585)

* Improve fgAddRefPred

Only do one search through the predecessor list looking for an
existing predecessor edge, or looking for the proper location to
insert a new predecessor edge. Previously, we traversed the list
twice (or part of twice) for new edges. For new edges, stop the
search when we reach the correct location in sorted order.

This reduces x86 release minopts instruction count by 0.42% and
full opts instruction count by 0.33% (of a superpmi replay of
the tests).

Fixes #12582

* Update for feedback

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

Merge nmirror to master

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

Reducing diffs with CoreRT (#12579)

* Rename m_firstChar to _firstChar to reduce CoreRT diffs
* Reducing diffs with CoreRT
* Delete dead code

Merge pull request #12568 from hseok-oh/ryujit/fix_11781

[RyuJIT/ARM32] Fix using fgMorphMultiregStructArg for mkrefany

Remove unnecessary update of RegRecord in LSRA (#12474)

* Remove unnecessary update of RegRecord

We don't need to clear assignedInterval of regRec,
because it was already cleared by checkAndClearInterval() above.

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

- Add a assertion and comment for the assertion

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

[RyuJIT/ARM32] Return HFA struct (#12453)

* [RyuJIT/ARM32] Return HFA struct

- Enable return HFA struct
- Merge ARM32/ARM64 CodeGen::isStructReturn(GenTreePtr treeNode)
- Merge ARM32/ARM64 CodeGen::genStructReturn(GenTreePtr treeNode)

* Add comment

Add TODO comment to optimize two-float loading

Merge pull request #12539 from ellismg/update-prerelease-tag

Update to master prerelease2 tag

Merge pull request #12527 from gsfreema/relative-paths

Fix contributing workflow commit message link

Fix an issue where g_ephemeral_low would never get bashed into the write (#12563)

barrier when starting up with Server GC (x86 only).

Merge pull request #12577 from dpodder/optdata-170630

Update optdata to version 20170630-0138

Merge pull request #12236 from rartemev/fix_legacyjit_tests

Fix legacyjit CI testing

Merge pull request #12570 from mskvortsov/FixVisitOperands

Add a missing case to GenTree::VisitOperands()

Merge pull request #11815 from hseok-oh/ryujit/struct_split_small

[ARM32/RyuJIT] Lowering: split struct argument less than 16 bytes

Merge pull request #12576 from gsfreema/readme-fixes

README.md link and grammar fixes

Merge pull request #12559 from rartemev/fix_build_break

Fix build failure in internal builds

Update optdata to version 20170630-0138

README.md link and grammar fixes

Fixing various typos, grammar mistakes, and broken links in the root README.md file.

Add a missing case to GenTree::VisitOperands()

Merge pull request #12566 from adityamandaleeka/fix_license_header_winrtdq

Fix license header on new file

Merge pull request #12540 from pgavlin/OperandVisitor

Add a generic operand visitor to `GenTree`.

[ARM32/RyuJIT] Lowering: enable passing split struct less than 16 bytes

- Enable passing split struct (less than 16 bytes) in registers and on stack
morphing phase
Lowering phase
- Block at LSRA phase - add NYI_ARM
- Print split struct

[RyuJIT/ARM32] Fix using fgMorphMultiregStructArg for mkrefany

Block using fgMorphMultiregStructArg function for mkrefany.
Morphing for mkrefany struct is already done in fgMorphArgs function.

Fix license header on new file.

Merge pull request #12535 from adityamandaleeka/dispatcher_queues_coreclr

Support DispatcherQueues when getting WinRT dispatcher

Update README.md

Add link to archive of officially released .NET Core downloads (#12551)

Make libnuma soft dependency (#12556)

Load libnuma dynamically, so it is not linked at compile time and
hence avoid making it a hard dependency at runtime.

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

Merge nmirror to master

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

Use public ConditionalWeakTable iterator (#12564)

Port https://github.com/dotnet/corert/pull/4030/commits/a3837f7fc60a1b7db872cfef6463f74fb1ffeafb to CoreCLR

Move JIT_TrialAlloc declaration into jitinterfacex86.cpp (#12562)

Remove m_ prefixes in BinaryReader.cs (#12561)

Merge pull request #12557 from briansull/add-verbose

Implement a /verbose flag to show the verbose output from crossgen

Address PR feedback.

Format code.

Implement a /verbose flag to show the verbose output from crossgen

Added info on /verbose option to PrintUsageHelper()
Convert printf(ascii) to wide strings in methodtable.cpp
In build.cmd echo the crossgen /CreatePdb command line

Fix build failure in internal builds

Change LSRA to use the operand visitor.

Refactor LSRA's operand use builder.

Replace some uses of the operand iterator with the operand visitor.

Add a generic operand visitor to `GenTree`.

This visitor accepts any invokable argument (e.g. lambdas, functors,
etc.) with the following signature:

    `GenTree::VisitResult Visitor(GenTree* operand)`

This visitor is typically more efficient than the operand iterator
due to better opportunities for inlining and shorter dynamic path
lengths when deciding which operands need to be visited.

Make ResourceManager netstandard 2.0 API behave correctly (#12536)

* Make ResourceManager netstandard 2.0 API behave correctly

In netstandard 2.0 we have enabled the APIs like GetObject, GetStream,..etc. these are not functioning correctly in UWP apps because was assuming we always uses PRI files. The changes here is to initialize the managed resources too and let these APIs behave as if we are running inside desktop app.

* added #ifdef

* Remove the throwing as PRI can handle case insenitive lookup

* remove un-needed resources

Merge pull request #12532 from jashook/fix_arm64_and_arm32_package_restore

Fix the package restore for arm64 and arm32

Use the number of GC threads to calculate spin count (#12525)

* use the number of gc threads to calculate spin count

* use n_heaps

Fix the package restore for arm64 and arm32

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

Merge changes from TFS

Merge pull request #12360 from wateret/armel/dblarg

[RyuJIT/armel] Support `double` argument passing

Delete file moved to shared partition

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

Merge nmirror to master

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

Reduce the verbosity of optdata sync on stdout (#12543)

Set the "StandardOutputImportance" attribute on the RestoreOptData
target to low, matching the "RestoreNETCorePlatforms" target. Logging
to sync.log remains verbose, but stdout on the console is made much less
verbose.

Fixes #12127

Fix GC contract violation (#12542)

Delete test that tests pinning of System.Object (#12537)

See dotnet/corefx#21673 for motivation. This test fails on CoreRT.

[RyuJIT/armel] Support `double` argument passing

- Fix for putting `double` arguments between Lowering and Codegen phase
- Rename GenTreeMulLong to GenTreeMultiRegOp
  GT_PUTARG_REG could be GenTreeMultiRegOp on RyuJIT/arm

Fix #12293

[x86/Linux] Adjust SP in throwing helper block (#12504)

* [x86/Linux] Adjust SP in throwing helper block

* Fix format error

* Use funKind intead of siInFuncletRegion

* Use genSPtoFPdelta (and add some comments)

* Fix a typo

* Revert irrelevant changes

Update to master prerelease2 tag

release/uwp6.0 is going to use preview1 and we don't want to publish
different versions of the package with the same package identity

Merge pull request #12528 from pgavlin/NoPgoOption

Add a flag to build.{sh,cmd} to disable PGO.

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

Merge changes from TFS

Merge pull request #12487 from pgavlin/GenTreeVisitor

Add a new, faster tree walker.

Remove unreached() macro

[tfs-changeset: 1663789]

Support DispatcherQueues when getting WinRT sync context.

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

Merge changes from TFS

Fix a release build break.

[tfs-changeset: 1663777]

Move ReaderWriterLockSlim to shared partition

Enable SourceBranch to be passed from the pipeline parameters

Enable SourceBranch to be passed from the pipeline parameters

Format code.

Address PR feedback.

Fix build error

[tfs-changeset: 1663755]

Fix usage style.

Reduce excessive loop count in interlocked tests (#12222)

Use proper ifdef for EmitProfilerComCallXXX methods (#12502)

Fix debug assert (#12496)

Fix build error when FEATURE_READYTORUN_COMPILER is disabled (#12505)

Add a flag to build.{sh,cmd} to disable PGO.

This is helpful when performing JIT throughput measurements.

[RyuJIT/ARM32] Enable passing large split struct argument (#12050)

* [RyuJIT/ARM32] Enable passing large split struct

This enables passing split struct larger than 16 bytes.
To support splitted struct, it defines new GenTree type - GenTreePutArgSplit.
GenTreePutArgSplit is similar with GenTreePutArgStk,
but it is used for splitted struct only
and it has additional field to save register information.
GenTreePutArgSplit node is generated in lower phase.

* Apply reviews: split struct argument passing

- Fix some comments:
genPutArgSplit, GenTreePutArgStk, GenTreePutArgSplit, NuwPutArg, ArgComplete
- Add assertion check in genPutArgSplit, genCallInstruction
- Rename variable: baseReg
- Change flag for GenTreePutArgSplit: _TARGET_ARM && !LEGACY_BACKEND
- Change type of gtOtherRegs in GenTreePutArgSplit
- Remove duplicated code: NewPutArg
- Implement spill & restore flag for GenTreePutArgSplit

* Apply reviews

- Rebase
- Update managing spillFlag for split struct
- Implement spill & restore code generation
- Fix typos and rename variables
- Fix bug related to print gentree for split struct

* Fix bug and comments

- Fix bug in regset.cpp
- Add comments in morph.cpp's NYI_ARM
- Fix comments' typo in lsraarmarcp.cpp

Fix contributing workflow commit message link

A recent change moved the Commit Messages section to a different
markdown file.  This change fixes that link, and provides relative
paths for links where appropiate.  This keeps people in their own
fork of the repo when browsing on github.

Enable SourceBranch to be passed from the pipeline parameters

sets SkipBranchAndVersionOverrides=false so SourceBranch and SourceVersion
are correctly passed through the pipeline file.

set SkipBranchAndVersionOverrides=true for the publish definitions as
they don't build from the SourceBranch and instead use a different
tools branch to do their work.

Convert lvaMarkLclRefs to a visitor.

Convert lva{Inc,Dec}RefCntsCB to a visitor.

Convert rationalize to the generic visitor.

Convert fgWalkTree{Pre,Post,} to use the new visitor.

Add a generic tree visitor.

This visitor replaces the function-pointer-based walkers with a more
modern design centered around the curiously-recurring-template pattern.

Cleanup some instances of project.json from the repo (#12484)

* Replace ScriptGenerator project.json with csproj

* Update project.json occurrences in docs
Fix few typos.

* Remove project.json comments

Fix remove in list (#12497)

`remove` uses `erase` to delete the entry in list.
Since `erase` returns the next iterator, the iterator shouldn't be increased
when the entry is found and deleted.

Remove disabled tests for Marshal.IsComObject(null) (#12359)

Partially remove relocations from Class section of NGEN-ed images (#11962)

* Remove relocations for ParamTypeDesc::m_TemplateMT.

* Remove relocations for LayoutEEClass::m_LayoutInfo.m_pFieldMarshalers.

* Prepare RelativeFixupPointer.

* Remove relocations for FieldMarshaler::m_pFD and FieldMarshaler_*::m_*.

Remove unused declaration (#12501)

spmi add R2R field to getFieldInfo (#12498)

Fix JIT_NewArr1 8-byte alignment for ELEMENT_TYPE_R8 on x86.