Merge pull request #14547 from CarolEidt/Fix14539

Arm64: Use op2 type for LOCKADD

format spmi sources. (#14545)

format spmi sources.

Merge pull request #14553 from briansull/fix-warn

Fix for x86 desktop build break

Arm64: Use op2 type for LOCKADD

Fix #14539

Merge pull request #14555 from wtgodbe/AzureBlobSymPkg

Use C#-friendly path for indexing symbol packages

Use C#-friendly path for indexing symbol packages

Fix for x86 desktop build
Fix warning C4389: '==' : signed/unsigned mismatch

Merge pull request #14541 from stephentoub/taskbeginwait

Avoid TaskWaitBegin/End events in Task.Wait on already completed task

GetEnvironmentVariable: Avoid StringBuilder marshaling overhead (#14502)

* GetEnvironmentVariable: Avoid StringBuilder marshaling overhead

Try a stack allocated buffer first, then fallback to using the shared
array pool.

CLRLifoSemaphore cleanup (#14535)

- Removed volatile loads. They don't actually help with anything. Should help a bit on arm, haven't tested.
  - I had added them initially out of pattern and could have left them out. There was some concern before that without a volatile load a compiler could replace local uses with a memory load and that would change the meaning of what is intended, but that doesn't apply to these kind of loops because it would be incorrect to do so, the concern may apply to loops like the following:
    ```c#
    while(true)
    {
        Counts counts = m_counts;
        if(InterlockedCompareExchange(&m_counts, counts + 1, counts) == counts)
            break;
    }
    ```
    Where if the 3rd argument is replaced by the compiler with m_counts, it may yield an incorrect result. It's an invalid optimization but anyway that can't be done in the type of loops used in this code because the initial value used for subsequent loop iterations is the result of the compare-exchange operation and not the memory location.
- No need to decrement the count of waiters woken upon timeout. I had initially copied that part of the code from SemaphoreSlim, which needs it because it uses a Monitor that doesn't provide the same guarantees as the wait objects used here. It's not needed here.
- No change to perf on x64, I plan on testing arm as part of issue https://github.com/dotnet/coreclr/issues/14067 once I have some time and get a machine.

Avoid TaskWaitBegin/End events in Task.Wait on already completed task

We are unnecessarily firing the TaskWaitBegin/End EventSource event when the task has already completed if it's faulted/canceled.  We should only fire it if the task hasn't completed by the time we check.

Merge pull request #14493 from alpencolt/armel-cross-build

[RyuJIT/armel] cross build armel on x86 host

Merge pull request #14334 from BruceForstall/AddTestEnvToWindows

Fix corefx testing invocation

move ReplaceWith to cpp

to make DEBUG_DESTROY_NODE visible.

Refactor CopyFrom

add check that each tree in the method has its own unique gtTreeId.

extract fgDebugCheckStmtsList

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

Update CoreFx to preview1-25817-02 (master)

Merge pull request #14311 from ViktorHofer/SerializationExceptions

Make coreclr exceptions serializable and add typeforwards

Update CoreFx to preview1-25817-02

[RyuJIT/ARM32] Add the optimization case on CreateDictionaryLookupHelper (#13933)

* Implement optimization case for CreateDictionaryLookupHelper

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>
* Reenable mainv1/mainv2 tests

[RyuJIT/armel] cross build armel on x86 host.

Compile ARM soft-fp version of libclrjit.so on x86 platform.
For running cross-jit execute as:
COMPlus_AltJit=* COMPlus_AltJitName=libprotononjit.so ./corerun ...

protect strlen from nullptr (#14533)

Merge pull request #14350 from CarolEidt/LsraInfoCleanup

Cleanup of Lowering & LsraInfo

Merge pull request #14527 from stephentoub/timer_partition

Reduce Timer lock contention

Update BuildTools to prerelease-02116-01 (#14530)

Merge pull request #14529 from adityamandaleeka/r2r_exception_race

Fix deadlock in ReadyToRun scenarios between EH and code heap deletion

Merge pull request #14501 from briansull/FitsIn

Use FitsIn8 when setting GT_INT_CNS costs

Fix optEarlyPropRewriteTree (#14480)

Return ans use the right tree from optEarlyPropRewriteTree
Forbid large dst nodes.

* fix typo

Fixed stack probing for RHEL 6.9 (Linux kernel 2.6.32) (#14481)

Fixed stack probing for RHEL 6.9 (Linux kernel 2.6.32) (#14481)

Updated GenTree::IsIntCnsFitsInI32() tio use FitsInI32()

Fix deadlock caused by R2R method info lookups during code heap
deletion.

Fix stepping with tiered jitting

Fix #14426. Added a more flexible IL master breakpoint that can:
   a) bind to native offset 0 of each jitted code body
   b) use a MethodDescFilter so that only jitted code for one generic instance receives breakpoints

The remaining change is simply to switch step-in and trace stepping to use that new breakpoint binding behavior instead of the code version specific binding behavior that was used before. This ensures that even if the code version changes after the trace occurs we will still have a breakpoint on the alternate code versions and complete the stepping operation.

Merge branch 'master' into FitsIn

Merge pull request #14522 from dotnet/wtgodbe-patch-1

Parameterize PublishFlatContainer in x86 build

Review feedback changes

spill GT_RET_EXPR inside fat calli candidates. (#14520)

spill ret expr

Delete dead code (#14521)

PR Feedback

Optimize native build of coreclr repo - 3x less memory , 11% build time improvement  (#14509)

* Fix native build overparallelism

* Fix native build parallelization problems

* Allow overriding all build workaround parameters

* Fix cleanup ommisions and local environment variable naming

* Address code review feedback

* Fix nul spelling and add standard message prefix

* Simplify settings for CL compiler parallelism

* Address code review feedback - remove whitespace

Address PR feedback

Adjusting field name changes in native code

Change Task.Delay to use TimerQueueTimer instead of Timer

Timer is just a wrapper for a TimerHolder which is just a wrapper for a TimerQueueTimer.  We don't actually want the behavior provided by TimerHolder (to allow the underlying timer to be collected while it's in use) and explicitly work around that, and for that pleasure we're paying two additional allocations.  Just skip directly to the inner type.

Queue TimerQueueTimer directly to ThreadPool

Eliminate a wrapper ThreadPoolWorkItem that's unnecessary.  Saves an allocation ever time a Timer fires.

Reduce Timer lock contention

Timer currently uses a global lock to protect a single "queue" of Timers, and any operation to create/change/fire/delete a Timer takes that same lock.  This leads to a scalability problem for code that operates on lots of timers.

This change partitions this single queue into N queues, which all operate independently.  In the .NET Framework implementation, there's already logically such a split with one queue/lock per AppDomain, and this change utilizes the same underlying support in the runtime.  As such, contention is distributed across the N locks, helping scalability.

Merge branch 'master' into FitsIn

Parameterize PublishFlatContainer in x86 build

This was missed in my last change to parameterize PublishFlatContainer across the board, and is what was causing the failures in today's pipe builds. We only want to publish non-flat for Release builds, but were hardcoding PublishFlatContainer=true for all builds in the x86 definition.

Update CoreFx to preview1-25816-01 (#14516)

Merge pull request #14519 from adiaaida/fixThroughputScript

Fix call to submission.py in run-throughput-perf.py

Fix call to submission.py in run-throughput-perf.py

In run-throughput-perf.py, we were calling submission.py with --arch,
but submission.py requires -arch or --architecture. This recently made
submission.py start failing in throughput jobs.

This change also fixes run-throughput-perf so that if submission.py or
upload.py fail, the script fails so we see these jobs as failing in the
lab.

Forbid to clone GT_RET_EXPR (#14495)

* Forbid to clone GT_RET_EXPR

It potentially creates non-unique trees.

Environment.SystemDirectory: Avoid StringBuilder overhead (#14513)

* Environment.SystemDirectory: Avoid StringBuilder overhead

Use a stack allocated buffer, with fallback to a char array.

Update CoreFx to preview1-25815-02 (#14511)

Delete contract annotations (dotnet/corert#4722) (#14507)

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

Update BuildTools, CoreFx, PgoData to prerelease-02113-01, preview1-25814-02, master-20171012-0049, respectively (#14476)

Delete EnvironmentAugments.GetEnvironmentVariables (#14500)

CoreFX has been using EnumerateEnvironmentVariables for a while now.

Remove temp unsafe stackalloc workarounds for Span (#14503)

Delete unnecessary Debug.Assert (#14504)

Fix ILLink perf benchmark

This commit implements the following two changes to fix the
failure in ILLink perf test.

1) Fix Roslyn benchmark

Use ILLink.Tasks package to build the Roslyn linked benchmark,
now that Roslyn can target netcoreapp2.0.

2) Disable Musicstore temporarily

Disabling MusicStore build temporarily, since there's a build failure.

Merge pull request #14465 from sdmaclea/PR-ARM64-DUP-DV_2D

[ARM64] Fix INS_dup DV_2D encoding

Add EventSource event for dropped async state machines (#14497)

One of the main bugs we've seen with async methods in production are cases where an async method awaits something that's never completed and is then just dropped.  This in turn allows the async state machine to be collected, such that subsequent dumps have no usable information to determine what hung and where.

To help diagnose such issues, this commit adds an EventSource event that fires when an async state machine is collected without having been completed.  When that happens, it dumps a textual representation of itself into a string and sends that to an event.

Doing this depends on the recent changes that were made to how we box async state machines to the heap.  Now that we do so with an explicit strongly-typed "box" instead of a normal runtime box, when the event source event is enabled, we can instead instantiate a derived and finalizable box, where the finalizer is responsible for firing the event, and we then suppress finalization when the async method completes.  The only async method state machines that are then finalized are those that are allowed to be collected before they're completed.

Use IsIntCnsFitsIn8

When classifying the size of integer constants use new method FitsInI8() and existing method FitsInI32

Disabling legacy banckend perf CQ and scenario runs (#14468)

- Recent changes to enforce data validation uncovered bug on the legacy backend runs. Now, we do not allow to upload tests with no performance data.
  CQ run bug tracked by: #14463
  JitBench run bug tracked by: #14477

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

Merge nmirror to master

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

Updates to the jit bench harness (#14467)

Updates to the jit bench harness

Introduce a number of new options:
* use-existing-setup to reuse a previously set up environment
* iterations to control number of iterations
* tiering to enable tiered jit
* disable-r2r to disable loading of R2R images
* disable-ngen to disable loading of NGEN images
* minopts to force jit to generate minopts code

Also start tracking average response time. This is not yet "uploaded"
since the underlying data reported by the app needs some updates.

Optimize type casts (#14420)

JIT: optimize type casts

Implement the jit interface compareTypesForEquality method
to handle casts from known types to known types, and from
shared types to certain interface types.

Call this method in the jit for castclass and isinst, using
`gtGetClassHandle` to obtain the from type. Optimize sucessful
casts and unsuccessful isinsts when the from type is known
exactly.

Undo part of the type-equality based optimization/workaround
in the AsyncMethodBuilder code that was introduced in #14178
in favor of interface checks. There is more here that can
be done here before this issue is entirely closed and I will
look at this subsequently.

This optimization allows the jit to remove boxes that are
used solely to feed type casts, and so closes #12877.

Fix handling of SIMD12

It turns out that it is problematic to change the type of a SIMD12 PUTARG_STK operand. Instead, at codegen time, use the number of slots on the PUTARG_STK to identify the SIMD12 case.
Also, add some tests extracted from the corefx System.Numerics.Vectors tests.

Merge pull request #14498 from wtgodbe/CloudDropAccountName

Unify CloudDropAccountName

Unify CloudDropAccountName

Merge pull request #14496 from wtgodbe/WorkingDir

Correctly set working folder in publish

Correctly set working folder in publish

Fix compile error with crossgen CreatePDB on Unix (#14488)

- Fix wchar string literals
- Fix init order of class fields

Allow not to build SOS.NETCore.dll (#14404)

* Allow not to build SOS.NETCore.dll

* Automatically enable SkipSOS when a mscorlib option is given

Merge pull request #14466 from sdmaclea/PR-Fix-14464

Fix JIT/HardwareIntrinsics/IsSupported.cs

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

Update BuildTools, CoreFx, PgoData to prerelease-02111-02, preview1-25812-01, master-20171009-0050, respectively (master)

Merge pull request #14440 from briansull/needs-reloc

Refactor some code using opts.compReloc

Merge pull request #14457 from wtgodbe/FixTestAndPublish

Fix native binary syncing & myget publish

Fix Linux x86 break (#14461)

Fix #14418 (#14419)

Adds the appropriate handling of the default ILCodeVersion in DacDbiInterfaceImpl::GetILCodeVersionNode

Fix JIT/HardwareIntrinsics/IsSupported.cs

Add more debug logging to tiered compilation manager (#14425)

* Add more debug logging to tiered compilation manager

* Fixed format specifiers for pointer sized arguments

* A few additional tieredcompilation logging improvements

Fix #14427 (#14429)

Enumerate all of the jitted instances of a method using the tiered compilation manager

[ARM64] Fix INS_dup DV_2D encoding

Merge branch 'master' into needs-reloc

Fix native binary syncing & myget publish

Adjust whitespace for clang format

Fixed references to images. (#14443)

Fixed formatting typo (#14452)

Fixed Typo / C++ Syntax Highlighting (#14450)

Fixed incorrect (escaping?) characters '_' in code blocks.
Enabled C++ Syntax Highlighting.

Fixed typo (#14449)

Fixed formatting typo. (#14448)

Generate xorps instead of xorpd (#14385)

xorps is one byte shorter when AVX is not used. Also saves a call to genGetInsForOper.

Delete System.Runtime.Intrinsics.X86 package reference

Simplify SSE2 check for RyuJIT/x86 (#14432)

Update BuildTools, CoreFx, PgoData to prerelease-02111-02, preview1-25812-01, master-20171009-0050, respectively

Refactor code for constant handles that reference opts.compReloc to use ImmedValNeedsReloc()

Merge pull request #14439 from wtgodbe/CrossbuildFlat

Specify PublishFlat in Crossbuilds

Specify PublishFlat in Crossbuilds

Merge pull request #14438 from wtgodbe/PublishFixes

Fix final issues with publish in pipeline

Fix final issues with publish in pipeline