Merge pull request #14859 from justinvp/rom

ReadOnlyMemory: Fix copy/paste mistake in doc comment

Fix perf-prep for throughput (#14845)

There is a missing ! before checking to see if the throughput benchmarks
have already been downloaded. Currently, we download them if they
already exist, where we want to download them if they don't already
exist. This change fixes that.

Print duration to xunit output from `runtest.sh` (#14489)

Merge pull request #14858 from jashook/turn_off_publish_coreclr

[BuildPipeline] Turn off publishing to myget

[BuildPipeline] Turn off publishing to myget

Do not publish to myget, unless the 'myget' PB_PublishType
is passed.

Merge pull request #14849 from sdmaclea/PR-ARMARCH-genRangeCheck-gtThrowKind

[Arm64/Arm] genRangeCheck gtThrowKind

ReadOnlyMemory: Fix copy/paste mistake in doc comment

Update BuildTools, CoreFx, PgoData to prerelease-02203-01, preview1-25902-08, master-20171103-0222, respectively (#14834)

[Arm64/Arm] genRangeCheck gtThrowKind

Merge pull request #14836 from stephentoub/valuetaskinterfaces

Move I{Configured}ValueTaskAwaiter interfaces to correct location

Merge pull request #14824 from CarolEidt/Fix14798

Fix copyOrMoveRegInUse

Mark applicable structs as readonly (#14789)

In a few cases (e.g. nullable), I added readonly to fields in order to allow readonly on the type.

Merge pull request #14838 from BruceForstall/Fix14551

Handle TryGetUse case with GT_PUTARG_SPLIT of GT_FIELD_LIST

Merge pull request #14841 from BruceForstall/FixArm64Build

Fix arm64 build

Fix arm64 build

to have broken the arm64 build. Fix one file by removing non-ASCII
characters.

fix (#14837)

GTF_RET means that this flag is valid only for GTF_RET node, but this check was missed.
The issue was in collision between GTF_RET_MERGED and GTF_CALL_UNMANAGED.

Merge pull request #14801 from briansull/newfix

Fixes Assertion failed 'firstILBlock->hasProfileWeight()

Handle TryGetUse case with GT_PUTARG_SPLIT of GT_FIELD_LIST

Move I{Configured}ValueTaskAwaiter interfaces to correct location

They need to be in the shared partition.

JIT: Merge legacy inlining policies (#14815)

Merge the LegacyPolicy and EnhancedLegacyPolicy into a unified policy that
behaves like the EnhancedLegacyPolicy. Rename this policy to the DefaultPolicy
since it is in fact the default inline policy.

We had been keeping the LegacyPolicy around in case we ever needed to revert
back to the initial RyuJit inline behavior, but that safeguard no longer seems
necessary.

Remove some of the checks in flowgraph.cpp that alter behavior based on policy
as they are no longer needed.

Remove the jit config setting that allowed selection of the LegacyPolicy.

This is the first stage in fixing #14441.

Allow coredumps inside docker (#14814)

Merge pull request #14811 from jashook/signing_var_changes

Orchestrated build Config signing.

Update BuildTools, CoreClr, PgoData to prerelease-02202-02, preview1-25901-06, master-20171102-0019, respectively (#14766)

Orchestrated build Config signing.

Sign if Real or Test is passed.

Merge pull request #14796 from sdmaclea/PR-SIMD-lower-bounds-test

Add SIMD get_Item lower bounds test

Merge pull request #14809 from sdmaclea/PR-ARM64-genSIMDIntrinsicGetItem

[Arm64] SIMDIntrinsicGetItem

Merge pull request #14820 from sdmaclea/PR-ARM64-SIMDIntrinsicRelOp

[Arm64] genSIMDIntrinsicRelOp

Merge pull request #14827 from jashook/wrong_eq

Invert eq

Invert eq

Remove invalid assert in CLRLifoSemaphore (#14805)

After https://github.com/dotnet/coreclr/pull/14535, the assert is no longer valid. A thread that times out does not decrement the count of waiters signaled to wake because a timed-out thread does not observe a signal to the object that was waited upon. If there were no other waiters to observe the signal, the count of waiters signaled to wake can become greater than the waiter count. This is valid, and tracks how many signals are pending. I don't think there is a good way to update the assert to reflect this, removed the assert.

[Build Pipeline] Add ability to skip testing (#14813)

Merge pull request #14804 from dotnet/tfUrlUpdate

TransportFeed URL update

Switch to /utf-8 for MSVC (#14822)

* Add /source-charset:utf-8 to force MSVC to compile source as UTF-8.

* Fix the illegal character in siginfo.cpp.

Fix copyOrMoveRegInUse

When checking whether a `copyReg` or `moveReg` is still in use, only consider it in use if it is assigned to the register we care about.
We can have strings of `copyReg`s to different registers, especially in stress modes.

Fix #14798

Merge pull request #14797 from BruceForstall/FixBuildCmdPriorityHandling

Fix handling of `-priority=N` argument in build.cmd

Fixes Assertion failed 'firstILBlock->hasProfileWeight()

When creating a new top block we need to use inheritWeight to properly initialize the the block weight and the profile data flag
The fix now handles multiple predecessor blocks that each contribute profile weights to the newT block.

[Arm64] genSIMDIntrinsicRelOp

Handle SIMD12 special case

[Arm64] SIMDIntrinsicGetItem

Fix srcCount
Fix item containment
Implement [Arm64] genSIMDIntrinsicGetItem w/ non const index
Handle out of range const index

Move Dictionary to shared CoreLib partition (#14795)

Add perf pipeline job full groovy (#14397)

This change converts our perf testing to use pipeline jobs. Pipeline
jobs allow us to do the following:

1) Test on the same commit for each of the test legs
2) Parallelize the build and test steps.
3) Separate the build and test steps from one another. This gives us the
ability to use the same build assets for all of the test legs of the
same configuration. It also allows us to build on virtual machines and
test on perf machines, so we only use the perf resources for testing.
4) Have different test scenarios for PRs and rolling. This isn't
strictly a benefit of pipeline jobs, but certainly is made easier by
them.
5) Allows us to have one trigger for PR jobs which will get us all the
perf testing scenarios.

This change also cleans up the groovy scripting for perf testing.

Merge pull request #14791 from sdmaclea/PR-ARM64-impSIMDSelect

[Arm64] Fix impSIMDSelect

Fix sync's location. (#14817)

Merge pull request #14808 from sdmaclea/PR-ARM64-genSIMDIntrinsicDotProduct

[Arm64] Fix genSIMDIntrinsicDotProduct

Merge pull request #14812 from BruceForstall/AddPeriodicAltjitTriggers

Enable periodic triggers for `x86_arm_altjit` and `x64_arm64_altjit` jobs

[Local GC] Fix (another) ScanContext layout issue when building without FEATURE_REDHAWK (#14777)

Enable periodic triggers for x86_arm_altjit and x64_arm64_altjit jobs

These jobs are getting clean enough where having them regularly run
will be valuable.

JIT: fix overly aggressive box-unbox.any assert (#14799)

A recently added assert was trying to ensure that `compareTypesForEqualty`
caught all the box-unbox.any cases we got before by direct handle comparison.
But there are cases where identical handles cannot resolve equality
comparisons, eg when the handles represent shared types like __Canon, and
the assert was not screening for those.

Seems simplest just to remove the assert, it was really only of value when
coding up the new jit interface method.

Closes #14780.

JIT: convert fixed-sized locallocs to locals, enable inlining (#14623)

Optimize fixed sized locallocs of 32 bytes or less to use local buffers,
if the localloc is not in a loop.

Also "optimize" the degenerate 0 byte case.

Allow inline candidates containing localloc, but fail inlining if any
of a candidate's locallocs do not convert to local buffers.

The 32 byte size threshold was arrived at empirically; larger values did
not enable many more cases and started seeinge size bloat because of
larger stack offsets.

We can revise this threshold if we are willing to reorder locals and see
fixed sized cases larger than 32 bytes.

Closes #8542.

Also add missing handler for the callsite is in try region, this was
an oversight.

[Arm64] Fix genSIMDIntrinsicDotProduct

Revert sync test sync binaries (#14807)

Updating xUnit Benchmark Scenarios to capture Etw with Pmc support. (#14734)

- Adding 'tiered' jitting
- Enable JitBench to capture Pmc data (Profile=On)
- Fixed some xUnit performance warnings
- Enforce DOTNET_MULTILEVEL_LOOKUP environment to be turned off
- Specify `dotnet publish` output directory.

Publishing to the transport feed main feed

Supporting publishing to the main feed where the relative path is empty

Merge pull request #14792 from sdmaclea/PR-SIMD-negative-get_Item

SIMD Handle negative indicies in get_Item

Merge pull request #14788 from sdmaclea/PR-ARM64-9b-INSTR

[Arm64] Instruction requires 9 bits now

Merge pull request #14781 from fiigii/x86reg

[RyuJIT/X86] Limit byte register candidates for CRC32

TransportFeed URL update

Updating URL to point to the new main feed

[RyuJit/ARMARCH] lower arg with list of float fields. (#14753)

apply bitcast on reg fields
rename ReplaceArgWithPutArgOrBitcast

Merge pull request #14761 from hseok-oh/ryujit/fix_13746

[RyuJIT/ARM32] Fix double type PutArgStk check bug on LSRA init

Fix handling of `-priority=N` argument in build.cmd

This argument is intended to be passed through directly to msbuild,
but CMD doesn't handle `=` well, so using `-priority=N` with
`build all` didn't work, for instance.

Add special handling for the `-priority` argument to get around
this problem.

Fix (#14778)

skip commas and nops.

Add SIMD get_Item lower bounds test

Merge pull request #14779 from fiigii/fixclone

Fix #14773

Merge pull request #14629 from sdmaclea/PR-ARM64-SIMD-CODEGEN

[Arm64] Add SIMD codegen

Merge pull request #14616 from sdmaclea/PR-ARM64-SIMDINTRINSICLIST

[Arm64] Implement simdintrinsiclist.h

[Arm64] Fix impSIMDSelect

Merge pull request #14790 from sdmaclea/PR-ARM64-insEncodeDatasizeLS

[Arm64] Revise insEncodeDatasizeLS

SIMD Handle negative indicies in get_Item

[Arm64] Revise insEncodeDatasizeLS

[Arm64] Instruction requires 9 bits now

Update URLs to use for formatter job (#14785)

Need 2.0 SDK now. Not sure of the best URLs but these seem to work.

Also shorten path so we don't run into windows pathlength issues.

Limit byte register candidates for CRC32

Merge pull request #14767 from CarolEidt/Fix14373

Fix checking for reg in use

Remove intermediate sync for test build (#14775)

Fix slot size for transformed long

Fix slot size as 2 for transformed long from double
Use expected slot size for assertion check

fix #14773

PR feedback & fix an error in `isRegInUse`

Merge pull request #14705 from hseok-oh/ryujit/fix_14666

[RyuJIT/ARMARCH] Use ActualType for PutArgStk size

[Arm64] Add SIMD codegen

Clean up YieldProcessorNormalized (#14739)

Move YieldProcessorNormalized into separate files

Clean up YieldProcessorNormalized

Make NonRandomizedStringEqualityComparer serialize into GenericEqualityComparer<string> (#14756)

Fix checking for reg in use

Combine the check for an inactive but not-yet-freed interval, with the check for other cases of a register that is in use at the current location. This simplifies the code and fixes the assert condition for double registers.

Fix #14373

Merge pull request #14614 from hqueue/arm/ryujit/issue_14374_pr2

[RyuJIT/ARM32] Fix resolving TYP_DOUBLE and TYP_FLOAT intervals

Merge pull request #14748 from dseefeld/addRetryInInitTools

Add retry logic for setup of dotnet cli

Change leading bracked to char as well

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

Merge pull request #14759 from stephentoub/gettaskforresult_inlining

Don't inline GetTaskForResult when await'ing ValueTask

Move System.Tuple to shared CoreLib partition (#14762)

Update BuildTools, PgoData to prerelease-02131-01, master-20171031-0018, respectively (#14764)

Update CoreFx to preview1-25831-01 (#14760)

[RyuJIT/ARM32] Fix double type PutArgStk check bug on LSRA init

Fix to handle hardfp/softfp all

[RyuJIT/ARM32] Fix resolving TYP_DOUBLE and TYP_FLOAT intervals

- When generating fromReg to toReg move to resolve edges,
  we have to consider either of fromReg and toReg can contain double type.

- Add a new helper, i.e. addResolutionForDouble()

- Fix bug when reserving temp reigster for TYP_DOUBLE by introducing tempRegDbl.
  There was a bug when it can't find a double reg, but find a even-numbered float reg,
  because we check validness of tempFltReg with register number only.
  We found that it is not easy to check validness of tempRegFlt,
  if we use tempRegFlt for both TYP_DOUBLE for TYP_FLOAT.
  Therefore we decide to introduce tempRegDbl to reserve a temp double reigster separately.

- Apply review feedback

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

Add retry logic for setup of dotnet cli

Don't inline GetTaskForResult when await'ing ValueTask

When await'ing a ValueTask, we should only ever end up calling AwaitUnsafeOnCompleted when the ValueTask wraps a non-null _task; if its _task were null, IsCompleted would have returned true due to treating the _result as the successful result.  However, the value task's AsTask() is currently used via an inlined interface invocation in AwaitUnsafeOnCompleted to get the _task from the ValueTask, and AsTask ends up inlining the _task==null branch that inlines AsyncTaskMethodBuilder'1.GetTaskForResult, which ends up bloating the asm unnecessarily.  We can simply change the interface implementation used by AwaitUnsafeOnCompleted to not inline GetTaskForResult.  In a simple example, this cuts the asm for AwaitUnsafeOnCompleted by ~50%.

Add wrapper support for getUnboxedEntry (#14752)

Set toolset dir only for ARM64 (#14755)

Merge pull request #14730 from fiigii/SSE4

[RyuJIT] Rename and Simplify SSE3_4 flag

[Local GC] Fix an issue where the size of ScanContext differs between EE and GC (#14747)

Fix build warning (#14751)

Update BuildTools to prerelease-02130-03 (#14750)

Merge pull request #14683 from weshaggard/MergeDevRelease200

Merge changes from dev/release/2.0.0

Update code in AwaitUnsafeOnCompleted to use interface matching

Given that the jit can now avoid boxing on some interface calls to value types,
generalize the patterns introduced in AwaitUnsafeOnCompleted in #14718 by
using interfaces instead of checking for specific types.

Also move the catch-all processing back in line as the jit can now fold the
interface tests early and not pull in the EH portion of the method unless it is
needed.