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

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

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

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

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.

Fix rough edges after CLI update (dotnet/corert#3983)

* Move build-restore.cmd back to build-managed

buildpipeline needs to be updated to make this split

* Cleanup dir.props/dir.targets

* Add back VS clues

* Reformat whitespaces to match VS convention

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

Fix bad range check in TextInfo.IndexOfStringOrdinalIgnoreCase (#12478)

Breaks:

   int idx = "\uabcd".IndexOf("", 1, StringComparison.OrdinalIgnoreCase);

should return 1, returns -1 instead.

(propagated fix from CoreRT)

Implement JIT_NewArr1_R2R as R2R wrapper for JIT_NewArr1 to support both MethodTable-based and TypeDesc-based helpers. (#12475)

Related issue: #12463

Fix legacyjit CI testing

Merge pull request #12473 from hqueue/arm/ryujit/lsra_use_helper

[RyuJIT/ARM32][WIP] Update LSRA with helpers

[Linux][GDB-JIT] Fix memory leak (#12452)

Use NewArrayHolder for pVars in LocalsInfo class

[RyuJIT/ARM32] Update LSRA with new helper

- Introduce updatePreviousInterval()
- Update LSRA with helper

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

support spmi collection for r2r calls. (#12343)

* CreateAgnostic_CORINFO_SIG_INFO

Also you pSig_Index always when it is index in the buffer map,
as we do for sigInst_classInst_Index for example.

* spmi Restore_CORINFO_RESOLVED_TOKENout

delete another duplicate.

* SpmiDumpHelper

* spmi cleaning: GetFunctionEntryPoint and GetAddressOfPInvokeTarget

* support GetReadyToRunHelper and GetReadyToRunDelegateCtorHelper in spmi.

* Fix typo that annoys desktop build.

* spmi GetReadyToRunDelegateCtorHelper

* spmi: use agnostic type for GetReadyToRunDelegateCtorHelper value.

* spmi dump Agnostic_CORINFO_LOOKUP

add stubLookup to dmpGetCallInfo

* clean dmpGetMethodSig and dmpGetTailCallCopyArgsThunk

[RyuJIT/ARM32] Use helper for LSRA

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

Merge pull request #12460 from pgavlin/SinglePassRationalize

Rephrase rationalize as a single IR walk.

Format code.

Merge pull request #12423 from CarolEidt/ContainDoNotEnreg

Mark lvDoNotEnregister lclVars as contained

Merge pull request #12457 from pgavlin/RewriteIntrinsicsRefactor

Refactor the rationalizer's instrinsic rewriter.

Rephrase rationalize as a single IR walk.

This walk performs both pre- and post-order visits. The pre-order visit
is responsible for rewriting intrinsics as user calls where necessary;
the post-order visit is resonsible for converting HIR nodes into LIR
nodes.

Use LIR utilities in the intrinsic rewriter.

This simplifies the code a bit and prepares it for some changes to
perform single-pass rationalization.

Merge pull request #12440 from pgavlin/MaintainIsUnusedValue

Maintain `LIR::IsUnusedValue` in the backend.

Update a header comment.

Format code and address PR feedback.

Replace array type handle with method table in arguments of array allocation helpers (#12369)

* Remove direct usage of type handle in JIT_NewArr1, with except of retrieving template method table.

* Assert that array type descriptor is loaded when array object's method table is set.

* Pass template method tables instead of array type descriptors to array allocation helpers.

Refactor the rationalizer's instrinsic rewriter.

Change its two constituent static methods into instance methods.

Merge pull request #12441 from pgavlin/TryGetUseInline

Avoid the use edge iterator in `TryGetUse`.

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

Merge nmirror to master

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

Format code.

Return early in `TryGetUse` for unused nodes.

Now that `IsUnusedValue` is maintained during the first half of the
backend, `TryGetUse` can terminate early if it is set.

Note that this may return false negatives if the `IsUnusedValue` flag is
incorrectly set, but remains correct if said flag is incorrectly
cleared.

Remove an IR walk during lowering.

Now that the `IsUnusedValue` flag is maintained by decomposition and
lowering, it does not need to be recalculated before LSRA.

Maintain `LIR::IsUnusedValue` in the backend.

This flag indicates whether or not the SDSU temp produced by a node is
ever read. This information is needed by the register allocator, but is
also useful in other parts of the compiler (for example, `TryGetUse` can
return immediately if this flag is set). Setting this flag properly in
rationalize and maintaining it through decomposition and lowering allows
us to remove an IR walk immediately prior to LSRA.

Merge pull request #12439 from pgavlin/SkipDecomp

Update compLongUsed in rationalize.