Merge pull request #12814 from pgavlin/BuildUses

Clean up the code that builds uses a bit in LSRA.

Merge pull request #12811 from JeffCyr/concurrentqueue-maxsegmentlength

Applied MaxSegmentLength restriction in ConcurrentQueue

Format code.

Applied MaxSegmentLength restriction in ConcurrentQueue

Merge pull request #12804 from pgavlin/VSO462274

Mark shift helpers that produce an unused value as such.

Merge pull request #12550 from sergign60/arm_build

[CoreRT armel] refix for PINVOKE calls in CoreRT #12373

[RyuJIT/ARM32] Kill the R4 register on exit if the call is the stub (#12662)

* [RyuJIT/ARM32] Kill the R4 register on exit if the call is the virtual stub call.

Related #11838

In this issue, farthestRefPhysRegRecord was nullptr in the end of allocateBusyReg(). That's because R4 register had been busy since before allocation of R4 Register.
It was caused that the R4 register was not killed on exit of the stub call.

On ARM architecture, the argument of stub call transfer by R4 register.
And when the stub is called, R4 register would be the status to busy.
R4 register has to be killed for leaving the busy state after stub call.

* Fix the formatting

* Get the vs param from virtualStubParamInfo

* Fix formatting

* Remove REG_ARG_4 define

* Change to better codes

* Add checking other platform cases

Merge pull request #12777 from dotnet-maestro-bot/master-UpdateDependencies

Update CoreClr, CoreFx to preview2-25514-01, preview2-25514-03, respectively (master)

Update CoreClr, CoreFx to preview2-25514-01, preview2-25514-03, respectively

[CoreRT armel] refix for PINVOKE calls in CoreRT #12373

Merge pull request #12787 from briansull/zap-interface

Improvements in the Zapper to VM interface, no longer throw exceptions across the interface.

Merge pull request #12803 from wtgodbe/disableJITTestArm

Disable TalcallVerifyWithPrefix test for ARM

Clean up the code that builds uses a bit in LSRA.

These cleanups are enabled by the recent changes to SIMD8 handling and
the removal of `fixedCandidateMask`.

Merge pull request #12785 from wtgodbe/DeleteTest

Disable StringJoin test14

Mark shift helpers that produce an unused value as such.

Fixes VSO 462274.

Disable TalcallVerifyWithPrefix test for ARM

Disable StringJoin test14

[CoreCLR] Display the contents of init-tools.log (#12795)

* Update attribute-value in security build definition.

* Display init-tools log.

* Ensure execute permission is available.

Merge pull request #12781 from wtgodbe/ArmToolset

Add VC Runtime binaries to Core_Root for ARM tests

Delete test

Merge pull request #12793 from wateret/fix-legacy-12685

[LegacyJIT/ARM] Fix const fold assert condition

Change missed by the CoreLib mirror (#12792)

[LegacyJIT/ARM] Fix const fold assert condition

Fix #12685

[Arm64] Enable Span clear optimization (#12788)

Merge pull request dotnet/corert#4141 from MichalStrehovsky/fixBuildBreak

Merge nmirror to master

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

Merge pull request #12769 from pgavlin/SimplifyContainCheck

Simplify the `isContainedNode` check in LSRA.

Mark a `GT_FIELD_LIST` used by a `PUTARG_STK` contained.

Uses occur too early otherwise.

Merge pull request #12779 from pgavlin/ByrefAsserts

Loosen an assertion in the emitter re: byrefs.

Merge pull request #12783 from CarolEidt/RemovePhysRegDst

Remove unused GT_PHYSREGDST node

Improvements in the Zapper to VM interface, no longer throw exceptions across the interface.

Catch the exception on the VM side of the interface call and report the error to the zapper using an Error() function call rather than letting the exception go across tyhe interface call.

The interface methods LoadIBCTypeHelper and LoadIBCMethodHelper no longer need a ThrowsViolation in their contracts.

Instead oif using BAD_FORMAT exceptions for a failures to load IBC data type and methods, we now
throw TypeLoad exception.

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

Merge changes from TFS

Remove unused GT_PHYSREGDST node

Update optdata in master to version 20170712-0121 (#12770)

Add VC Runtime binaries to Core_Root for ARM tests

Loosen an assertion in the emitter re: byrefs.

This assertion was attempting to ensure that one of the sources of an
instruction with a particular format that produced a byref was itself a
gcref or a byref. Its attempt to do so was overly strong, however, as it
rejected IR of the following form (specifically, the `add r, m`
generated by `t8 = ADD byref REG rdx`).

```
Generating: N005 (  3, 10) [000005] ------------        t5 =    CNS_INT(h) long   0x1c348f05a78 static Fseq[??_7type_info@@6B@] REG rax
IN0001:        mov      rax, 0x1C348F05A78
                                                             /--*  t5     long
Generating: N007 (  5, 12) [000006] -c----------        t6 = *  IND       ref    REG NA
Generating: N009 (  1,  1) [000009] ------------        t9 =    CNS_INT   int    8 REG rdx
IN0002:        mov      edx, 8
                                                             /--*  t9     int
Generating: N011 (  2,  3) [000010] ------------       t10 = *  CAST      long <- int REG rdx
IN0003:        movsxd   rdx, edx
                                                             /--*  t6     ref
                                                             +--*  t10    long
Generating: N013 (  8, 16) [000008] ------------        t8 = *  ADD       byref  REG rdx
IN0004:        add      rdx, bword ptr [rax]
Byref regs: 00000000 {} => 00000004 {rdx}
```

This changes loosens the assertion s.t. it merely ensures that the
instrution is either an add or a sub that produces a byref.

Merge pull request #12768 from pgavlin/RemoveFixedCandidateMask

Remove `fixedCandidateMask` and its uses.

Fix a build break.

[tfs-changeset: 1665563]

Format code.

Remove dead LazyInitializer file from #12697 (#12767)

* Delete LazyInitializer.cs

* Update LazyInitializer.cs

Simplify the `isContainedNode` check in LSRA.

Now that containedness is explicit this can simply check `isContained`.

Remove `fixedCandidateMask` and its uses.

This function dosen't really serve any purpose: all of its uses boiled
down to no-ops.

Implement a new approach for SIMD8/LONG interactions. (#12590)

SIMD8 values need to be converted to longs under a small number of
situations on x64/Windows:
- SIMD8 values are passed and returned as LONGs
- SIMD8 values may be stored to a LONG lclVar

Currently, LSRA performs some gymnastics when building use positions in
order to ensure that registers are properly allocated. This change is a
stab at a different approach: rather than pushing this work onto the RA,
lowering inserts `GT_BITCAST` nodes where necessary to indicate
that the source long- or SIMD8-typed value should be retinterpreted as
a SIMD8- or long-typed value as necessary. The RA performs one specific
optimization wherein it retypes stores of `GT_BITCAST` nodes to
non-register-candidate local vars with the type of the cast's operand
and preferences the cast to its source interval.

This approach trades slightly larger IR for some functions that
manipulate SIMD8 values for tighter code in buildRefPositions.

Merge pull request #12654 from wateret/armel/gtcopy_putargreg

[RyuJIT/armel] Do not update reg for PutArgReg copy

Merge pull request #12762 from wateret/fix-legacy-12686

[LegacyJIT/ARM] Fix RegSet assert failure

Merge pull request #12708 from YongseopKim/ryujit/arm32/add_nyi_promoted_struct_in_genPutArgStk

[RyuJIT/ARM32] Add NYI_ARM for promoted struct to genPutArgStk

Merge pull request #12741 from pgavlin/IRDocs

Update the RyuJIT IR documentation.

Address PR feedback.

[LegacyJIT/ARM] Fix RegSet assert failure

Fix RegSet assert failure while codegen for CpBlk and InitBlk

Fix #12686

[RyuJIT/ARM32] Add NYI_ARM for promoted struct to genPutArgStk

- Before solving the issue(#12657), add NYI_ARM to genPutArgStk().
- This commit's code's objective is same to the PR(#12655).
- This is from the issue(#12622).

Merge pull request #12749 from stephentoub/remove_cts_throw_switch

Remove ThrowExceptionIfDisposedCancellationTokenSource switch

Merge pull request #12752 from dotnet-maestro-bot/master-UpdateDependencies

Update CoreClr, CoreFx to preview2-25511-05, preview2-25512-01, respectively (master)

Update CoreClr, CoreFx to preview2-25511-05, preview2-25512-01, respectively

Merge pull request #12680 from CarolEidt/RefactorContain

Extract containment analysis

Merge pull request #12735 from erozenfeld/RecursiveTailCallBug

Fix for a bug in tail recursion elimination.

PR feedback

Merge pull request #12730 from briansull/remove-ibc-error-string

Remove m_pIBCErrorNameString and IBCErrorNameString()

Merge pull request #12746 from crummel/updateBuildToolsForSymbolFix

Update BuildTools to fix output path for signing.

Remove ThrowExceptionIfDisposedCancellationTokenSource switch

This is a legacy switch from desktop; we used to throw exceptions in certain cases, now we don't, and for whatever reason it was deemed an app compat issue on desktop, so a switch was added to continue throwing that exception if the switch was set.  This was all then inherited into coreclr. But there's no reason anyone would want to opt-in to the behavior on core, and the checks for it are actually measurable in scenarios that stress registering/unregistering cancellation tokens.

I'm simply deleting the switch.

Merge pull request #12732 from wateret/fix-runtestdesc

Remove duplicated description in runtest.sh

Update BuildTools to fix output path for signing.
This takes dotnet/buildtools#1603 which fixes the IntermediateOutputPath -
this is required by MicroBuild for signing.

Delete bitset iterators defines (#12642)

Delete BLOCKSET_ITER_INIT

Rename blkNum to bbNum.
We usually use bbNum (basic block number), rather than blkNum(block
number).
This change allows to grep for iterator and etc easier.

Delete BITVEC_ITER_INIT and BITVEC_INIT_NOCOPY.

Delete VARSET_ITER_INIT.

Rename some variables around varSet iterators.

Merge pull request #12729 from CarolEidt/Fix12724

Extract Lowering of SIMD12 locals

Fix typo in method-descriptor.md (#12742)

Merge pull request #12743 from pgavlin/RemoveUseDef

Remove `GTF_VAR_USEDEF`.

Fix crossgen debug directory generation problems. (#12715)

* Fix crossgen debug directory generation problems.

The first problem was that when the existing/incoming PDB debug
directory entry was a portable PDB (MinorVersion == 0x504d), the
ngen/native PDB added had the same MinorVersion indicating that
it was a portable PDB (but it never can be).

This was fixed by setting MinorVersion to 0 when creating the
ngen PDB debug directory entry.

The second problem was that the ngen PDB entry was being created
even when crossgen was run on linux/mac, etc.

The fix was to ifdef NO_NGENPDB the save ngen PDB entry code.

Merge pull request #12695 from dotnet-maestro-bot/master-UpdateDependencies

Update CoreClr, CoreFx to preview2-25510-01, preview2-25511-02, respectively (master)

Remove `GTF_VAR_USEDEF`.

Nothing sets this flag.

Fix build.sh break by poorly timed dotnet first time experience (#12727)

Depending on machine state, calling `dotnet` to dump the PGO and IBC
package version might trigger the first time experience message. This
breaks build.sh where it tries to parse the package version from `dotnet
msbuild`'s output. The fix is to disable the first time experience path
in these two invocations of `dotnet`.

Extract Lowering of SIMD12 locals

Fix #12724

Update the RyuJIT IR documentation.

The most important parts of these changes update the description of the
IR to cover HIR, LIR, and the differences between the two. Other updates
include some minor restructuring, typo fixes, rephrasings, and JIT dump
refreshes.

Update CoreClr, CoreFx to preview2-25510-01, preview2-25511-02, respectively

Fix for a bug in tail recursion elimination.

Tail recursion elimination transforms a tail call into a loop.
If compInitMem is set, we may need to zero-initialize some locals. Normally it's done in the prolog
but the loop we are creating can't include the prolog. The fix is to insert zero-initialization
for all non-parameter non-temp locals in the loop. Liveness phase will remove unnecessary initializations.

We never hit this case with normal C# code since C# definite assignment rules ensure that there are
no uninitialized locals in the generated msil. In the repro the method with tail recursion is a dynamic method
and it has an uninitialized local.

Remove duplicated description in runtest.sh

Remove m_pIBCErrorNameString and IBCErrorNameString()

Merge pull request #12670 from parjong/fix/SOS_StackWalk

[ARM/Linux] Fix DacStackWalk

Extract containment analysis

In preparation for moving containment analysis to the 1st phase of Lowering, refactor most of it into separate methods. For some cases, e.g. block stores, it is more efficient to keep the logic integrated, as it will move to a similarly integrated place in Lowering.

Also, remove some dead/unused code for ARM.

Merge pull request #12722 from CarolEidt/DumpFixes

A couple more fixes to JitDump

Merge pull request #12674 from CarolEidt/LowerStoreLoc

Move StoreLoc type widening to Lowering

Change heading size

Merge pull request #12628 from hseok-oh/ryujit/fix_12602

[RyuJIT/ARM32] Clear assertion for refany struct and struct type temp

Put in a warning that the instructions no longer work properly

Remove  another reference to System.Private.Corelib.ni.dll

Remove reference to System.Private.Corelib.ni.dll

The build now only has the NGENed image (called but called System.Private.Corelib.dll.   Remove references to the *.ni.dll version

Merge pull request #12580 from wtgodbe/disableSync

Disable sync.cmd -n for Windows binaries

A couple more fixes to JitDump

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

Merge changes from TFS

Speed up fgLocalVarLiveness in minopts. (#12665)

In particular:
- Do not re-sort lclVars
- Do not run LVA
- Do not set `mustInit`
- Do not set last uses
- Do not perform DSE

Merge pull request #12717 from wtgodbe/disableJITTestArm

disable hijacking test for Arm32

Merge pull request #12712 from helloguo/errorHandle

Fix error handle of PAL API SetThreadGroupAffinity

disable hijacking test for Arm32

Add symbol signing to published packages. (#12291)

* Add symbol signing to published packages.
- Enable MicroBuild signing steps.
- Update build to call new BuildTools symbol signing targets.

Disable sync.cmd -n for Windows binaries

Fix optdata bootstrap test script & Linux skiprestoreoptdata behavior (#12541)

Fixes #12171.

* bootstrap.py was printing an improperly-cased path to put PGO/IBC
  training data, which breaks on case-sensitive file systems, i.e., on
  Linux.

* On Linux, `skiprestoreoptdata` was improperly skipping PGO even if
  optdata was already present due to some of the conditional checks in
  build.sh. This cleans that up to better match the intent of the
  switch, and to fix Linux behavior to better match Windows.

Merge pull request #12710 from hseok-oh/ryujit/fix_12703

[RyuJIT/ARM32] Fix register allocation for split GT_FIELD_LIST struct

Update the telemetry logging to use MEASURES instead of TELEMENTRY

[tfs-changeset: 1665164]

[RyuJIT/ARM32] Fix register allocation for split GT_FIELD_LIST struct

To fix register allocation for split GT_FIELD_LIST struct,
set source register for each node of GT_FIELD_LIST

fix error handle

Merge pull request #12572 from wateret/armel/dblarg_stack

[RyuJIT/armel] Passing stack double arguments

[RyuJIT/ARM32] Avoid promoting structures which can be handled by HFA (#12604)

* [RyuJIT/ARM32] Avoid promoting structures which can be handled by HFA

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>
* [RyuJIT/ARM32] Add comments for ARM32

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

Note that tests don't have to be built to view JIT dumps (#12699)