platform/upstream/coreclr.git
7 years agoEnable build pipeline for Alpine Linux 3.6 (#14587)
Jan Vorlicek [Sun, 22 Oct 2017 13:05:37 +0000 (15:05 +0200)]
Enable build pipeline for Alpine Linux 3.6 (#14587)

7 years agoMerge pull request #14632 from sdmaclea/PR-ARM64-SIMD-SIMD.CPP
Bruce Forstall [Sat, 21 Oct 2017 16:43:49 +0000 (09:43 -0700)]
Merge pull request #14632 from sdmaclea/PR-ARM64-SIMD-SIMD.CPP

[Arm64] SIMD simd.cpp

7 years agoMerge pull request #14630 from sdmaclea/PR-ARM64-SIMD-LOWERING
Bruce Forstall [Sat, 21 Oct 2017 16:35:34 +0000 (09:35 -0700)]
Merge pull request #14630 from sdmaclea/PR-ARM64-SIMD-LOWERING

[Arm64] Basic SIMD lowering

7 years agoMerge pull request #14627 from sdmaclea/PR-ARM64-SIMD-EMITTERS
Bruce Forstall [Sat, 21 Oct 2017 16:30:58 +0000 (09:30 -0700)]
Merge pull request #14627 from sdmaclea/PR-ARM64-SIMD-EMITTERS

[Arm64] Add SIMD emitters

7 years agoMerge pull request #14638 from BruceForstall/FixSimdMinOpts
Bruce Forstall [Sat, 21 Oct 2017 15:07:08 +0000 (08:07 -0700)]
Merge pull request #14638 from BruceForstall/FixSimdMinOpts

Allow GT_CALL as BYREF operand for SIMD intrinsics

7 years agoUpdate profiling API status (#14644)
Noah Falk [Sat, 21 Oct 2017 04:44:41 +0000 (21:44 -0700)]
Update profiling API status (#14644)

We've been making progress testing APIs and fixing issues. The description in this doc had gotten well out of date.

7 years agoMerge pull request #14620 from CarolEidt/RegSelectFix
Carol Eidt [Sat, 21 Oct 2017 04:16:47 +0000 (21:16 -0700)]
Merge pull request #14620 from CarolEidt/RegSelectFix

Fix Register selection refactor bugs

7 years agoRevert "Reenable PGO on Linux Release builds" (#14645)
Jan Kotas [Sat, 21 Oct 2017 02:57:33 +0000 (19:57 -0700)]
Revert "Reenable PGO on Linux Release builds" (#14645)

7 years agoMerge pull request #14625 from BruceForstall/FixDynBlkAssert
Bruce Forstall [Fri, 20 Oct 2017 23:31:22 +0000 (16:31 -0700)]
Merge pull request #14625 from BruceForstall/FixDynBlkAssert

Remove incorrect assert for DynBlk codegen

7 years agoRename MemoryHandle PinnedPointer to Pointer and add property HasPointer. (#14604)
Ahson Ahmed Khan [Fri, 20 Oct 2017 22:48:55 +0000 (15:48 -0700)]
Rename MemoryHandle PinnedPointer to Pointer and add property HasPointer. (#14604)

7 years agoAllow GT_CALL as BYREF operand for SIMD intrinsics
Bruce Forstall [Fri, 20 Oct 2017 21:42:47 +0000 (14:42 -0700)]
Allow GT_CALL as BYREF operand for SIMD intrinsics

This is an extension of #13965: in the MinOpts case, we don't
have GT_RET_EXPR -- we have GT_CALL instead.

We now see IR like:
```
[000429] --C-G--N----              |  |  \--*  BLK(32)   simd32
[000413] --C-G-------              |  |     \--*  CALL      byref  System.Runtime.CompilerServices.Unsafe.AsRef
[000410] ------------              |  |        |     /--*  CNS_INT   int    2
[000411] ------------              |  |        |  /--*  MUL       int
[000409] ------------              |  |        |  |  \--*  CAST      int <- int
[000408] ------------              |  |        |  |     \--*  CNS_INT   int    16 Vector<T>.Count
[000412] ------------ arg0         |  |        \--*  ADD       int
[000407] ------------              |  |           \--*  LCL_VAR   int    V01 arg1
```

Whereas in the optimizing case, we see:
```
[000060] ------------              *  STMT      void  (IL   ???...  ???)
[000058] I-C-G-------              \--*  CALL      byref  System.Runtime.CompilerServices.Unsafe.AsRef (exactContextHnd=0x028FAFF8)
[000055] ------------                 |     /--*  CNS_INT   int    2
[000056] ------------                 |  /--*  MUL       int
[000054] ------------                 |  |  \--*  CAST      int <- int
[000053] ------------                 |  |     \--*  CNS_INT   int    16 Vector<T>.Count
[000057] ------------ arg0            \--*  ADD       int
[000052] ------------                    \--*  LCL_VAR   int    V01 arg1

...

[000076] --C----N----              |  |  \--*  BLK(32)   simd32
[000065] --C---------              |  |     \--*  RET_EXPR  byref (inl return from call [000058])
```

Fixes #14301

7 years ago[Arm64] SIMD simd.cpp
Steve MacLean [Fri, 20 Oct 2017 20:51:24 +0000 (16:51 -0400)]
[Arm64] SIMD simd.cpp

7 years ago[Arm64] Basic SIMD lowering
Steve MacLean [Fri, 20 Oct 2017 20:47:22 +0000 (16:47 -0400)]
[Arm64] Basic SIMD lowering

7 years ago[Arm64] Add SIMD emitters
Steve MacLean [Fri, 20 Oct 2017 20:36:28 +0000 (16:36 -0400)]
[Arm64] Add SIMD emitters

7 years ago[Arm64] Add more SIMD instructions
Steve MacLean [Thu, 19 Oct 2017 21:43:11 +0000 (17:43 -0400)]
[Arm64] Add more SIMD instructions

7 years agoRemove incorrect assert for DynBlk codegen
Bruce Forstall [Fri, 20 Oct 2017 19:51:53 +0000 (12:51 -0700)]
Remove incorrect assert for DynBlk codegen

The assert specified a particular register for the `size` argument
to the memset helper call. However, it preceded the genConsumeBlockOp()
function which ensured that would be the case.

Fixes #14544

7 years ago[RyuJit/arm32] Do nothing if double is on stack. (#14603)
Sergey Andreenko [Fri, 20 Oct 2017 18:32:16 +0000 (11:32 -0700)]
[RyuJit/arm32] Do nothing if double is on stack. (#14603)

7 years agoFix Register selection refactor bugs
Carol Eidt [Fri, 20 Oct 2017 18:31:12 +0000 (11:31 -0700)]
Fix Register selection refactor bugs

Fix #14617
Fix #14618

7 years agoMerge pull request #14606 from CarolEidt/Fix14591
Carol Eidt [Fri, 20 Oct 2017 18:20:17 +0000 (11:20 -0700)]
Merge pull request #14606 from CarolEidt/Fix14591

LSRA Arm64 consistent reg sets

7 years agoMerge pull request #14609 from hseok-oh/ryujit/fix_14377_linux
Bruce Forstall [Fri, 20 Oct 2017 17:35:21 +0000 (10:35 -0700)]
Merge pull request #14609 from hseok-oh/ryujit/fix_14377_linux

[RyuJIT/ARM32] Remove NYI: struct return from multi-reg GT_CALL

7 years ago[RyuJIT/ARM32] Fast tail call: code generation (#14445)
Hyeongseok Oh [Fri, 20 Oct 2017 17:29:44 +0000 (02:29 +0900)]
[RyuJIT/ARM32] Fast tail call: code generation (#14445)

* Codegen for fast tail call

Codegen call and epilog for fast tail call

* Implementation for GT_START_NONGC

This implementation removes two NYI_ARM
Code generation for GT_START_NONGC which is used to prevent GC in fast tail call

* Define fast tail call target register and mask on ARMARCH

Define REG_FASTTAILCALL_TARGET and RBM_FASTTAILCALL_TARGET on ARMARCH
Modify lsra init and codegen to use these definition

* Merge genFnEpilog

Merge genFnEpilog for ARM32 and ARM64

* Fix bug in getFirstArgWithStackSlot

Fix bug in getFirstArgWithStackSlot: AMD64 and X86

7 years ago[Local GC] Add a Standalone GC loader design document (#14435)
Sean Gillespie [Fri, 20 Oct 2017 16:49:06 +0000 (09:49 -0700)]
[Local GC] Add a Standalone GC loader design document (#14435)

* Add a Standalone GC loader design document

* First round of feedback:
        1. Remove some stuff from the preamble that doesn't belong in a
        design document
        2. Simplify the three-variable loading approach to just use one
        variable that contains the path of a GC to load.
        3. Clean up the wording in a few places.
        4. Remove the "state machine" section and reference to state
        machines in general.

7 years agoUpdate comments in TieredCompilationManager (#14610)
Noah Falk [Fri, 20 Oct 2017 08:15:16 +0000 (01:15 -0700)]
Update comments in TieredCompilationManager (#14610)

We've continued plugging away and the status in the comment was out-of-date.

7 years ago[RyuJIT/ARM32] Remove NYI: struct return from multi-reg GT_CALL
Hyeongseok Oh [Fri, 20 Oct 2017 05:39:43 +0000 (14:39 +0900)]
[RyuJIT/ARM32] Remove NYI: struct return from multi-reg GT_CALL

Remove useless NYI

7 years agoMerge pull request #14600 from BruceForstall/FixArmBadGCInfoForCpObj
Bruce Forstall [Fri, 20 Oct 2017 03:51:26 +0000 (20:51 -0700)]
Merge pull request #14600 from BruceForstall/FixArmBadGCInfoForCpObj

Fix ARM bad GC info for CpObj

7 years agoMerge pull request #14602 from benaadams/jit-formatting
Bruce Forstall [Fri, 20 Oct 2017 03:42:04 +0000 (20:42 -0700)]
Merge pull request #14602 from benaadams/jit-formatting

Fix jit fromatting

7 years agoLSRA Arm64 consistent reg sets
Carol Eidt [Fri, 20 Oct 2017 03:24:33 +0000 (20:24 -0700)]
LSRA Arm64 consistent reg sets

tryAllocateFreeReg() uses the RegOrder array to iterate over the available registers. This needs to be consistent with the available registers of the given type. Otherwise, allocateBusyReg() will assert when it finds a free register that should have been allocated in tryAllocateFreeReg().
Fix #14591

7 years agoFix jit fromatting
Ben Adams [Fri, 20 Oct 2017 00:44:50 +0000 (01:44 +0100)]
Fix jit fromatting

7 years agoFix ARM bad GC info for CpObj
Bruce Forstall [Thu, 19 Oct 2017 23:30:46 +0000 (16:30 -0700)]
Fix ARM bad GC info for CpObj

In the case where the dst lives on the stack, after the first
gcref/byref was copied, we never set the type back to non-GC
for subsequent copies using the same temp register.

7 years agoMerge pull request #14597 from dotnet-bot/from-tfs
Brian Sullivan [Thu, 19 Oct 2017 23:18:28 +0000 (16:18 -0700)]
Merge pull request #14597 from dotnet-bot/from-tfs

Merge changes from TFS

7 years agoMerge pull request #14576 from fiigii/knobs
Bruce Forstall [Thu, 19 Oct 2017 22:31:48 +0000 (15:31 -0700)]
Merge pull request #14576 from fiigii/knobs

Add configuration knobs to control "IsSupported" and fix a bug

7 years agoMake ContractException public in impl assembly to support corert typeforwarding ...
Viktor Hofer [Thu, 19 Oct 2017 21:23:14 +0000 (23:23 +0200)]
Make ContractException public in impl assembly to support corert typeforwarding (#14589)

* Make ContractException public in impl assembly to support corert typeforwarding

7 years agoFix condition for mustExpand flag (#14590)
Jan Kotas [Thu, 19 Oct 2017 21:20:03 +0000 (14:20 -0700)]
Fix condition for mustExpand flag (#14590)

The flag was getting overwritten when both CORINFO_FLG_INTRINSIC and CORINFO_FLG_JIT_INTRINSIC are set

7 years agofix clang format complaint
Brian Sullivan [Thu, 19 Oct 2017 21:14:54 +0000 (14:14 -0700)]
fix clang format complaint

[tfs-changeset: 1678578]

7 years agoAdd configuration knobs to control IsSupported
Fei Peng [Wed, 18 Oct 2017 22:20:31 +0000 (15:20 -0700)]
Add configuration knobs to control IsSupported

7 years agoFix Desktop build break - warning for unreachable code
Brian Sullivan [Thu, 19 Oct 2017 19:26:22 +0000 (12:26 -0700)]
Fix Desktop build break - warning for unreachable code

[tfs-changeset: 1678568]

7 years agoFix warning: potentially uninitialized local variable 'recentAssignedRef' used
Brian Sullivan [Thu, 19 Oct 2017 19:25:55 +0000 (12:25 -0700)]
Fix warning: potentially uninitialized local variable 'recentAssignedRef' used

[tfs-changeset: 1678567]

7 years agoMerge pull request #14582 from briansull/cleanup-iconhdl
Brian Sullivan [Thu, 19 Oct 2017 18:14:58 +0000 (11:14 -0700)]
Merge pull request #14582 from briansull/cleanup-iconhdl

Cleanup IconHandle nodes

7 years agoMerge pull request #14570 from sdmaclea/PR-ARM64-FLAG_FEATURE_SIMD
Bruce Forstall [Thu, 19 Oct 2017 16:28:58 +0000 (09:28 -0700)]
Merge pull request #14570 from sdmaclea/PR-ARM64-FLAG_FEATURE_SIMD

[Arm64] Support flag FEATURE_SIMD

7 years ago[RyuJIT/ARM32] Support for CFI unwind info (#14447)
sergey ignatov [Thu, 19 Oct 2017 16:28:20 +0000 (19:28 +0300)]
[RyuJIT/ARM32] Support for CFI unwind info (#14447)

* [RyuJIT/ARM32] Support for CFI unwind info

7 years agoUse ascii * instead of Chinese * character. (#14584)
Jim Ma [Thu, 19 Oct 2017 05:03:26 +0000 (13:03 +0800)]
Use ascii * instead of Chinese * character. (#14584)

Currently we have Chinese `*` characters at `src\classlibnative\bcltype\number.cpp` L289-294. That would cause a `C4819: Non-ASCII character in source` warning when building on Windows via Visual Studio (version: 15.4.0 (msvc 19.11.25547)).

This PR replaces the Chinese * characters with ascii *.

Fix #14556

7 years agoMerge pull request #14534 from CarolEidt/RegSelectCleanup
Carol Eidt [Thu, 19 Oct 2017 04:17:43 +0000 (21:17 -0700)]
Merge pull request #14534 from CarolEidt/RegSelectCleanup

Refactor register selection heuristics

7 years agoFix hill climbing float overflow (#14505)
Koundinya Veluri [Thu, 19 Oct 2017 03:51:30 +0000 (20:51 -0700)]
Fix hill climbing float overflow (#14505)

Fix hill climbing float overflow

- When hill climbing finds that it wants to decrease the thread count but can't because the thread count is already the minimum, it instead tries to increase the sampling interval by a factor of up to 10 depending on how much it wanted to decrease the thread count
- The ratio was being used incorrectly (used max instead of min), and sometimes the ratio can be so large that the conversion to int after the float math overflows
- If something in the process enabled floating point exceptions, it may also crash
  - There doesn't appear to be a clean way to disable hill climbing, added a config variable that disables it in case a workaround is necessary for some other reason in the future
- Fixed to avoid overflow in the math to what was probably intended
- There may be another bug in GetWaveComponent() that causes values of such high magnitude to be generated, I'll leave that investigation for when that in particular becomes a real issue

7 years agoMerge pull request #14573 from CarolEidt/Fix13751
Carol Eidt [Thu, 19 Oct 2017 03:49:56 +0000 (20:49 -0700)]
Merge pull request #14573 from CarolEidt/Fix13751

Fix multiple issues in LSRA for TYP_DOUBLE:

7 years agoJIT: optimize stelem (ref) like we optimize stelem.ref (#14580)
Andy Ayers [Thu, 19 Oct 2017 02:42:52 +0000 (19:42 -0700)]
JIT: optimize stelem (ref) like we optimize stelem.ref (#14580)

In particular, avoid the array store check when storing null or values from
the same array.

Addresses first part of #14574.

7 years agoRemoved hnd1 and hnd2 arguments to gtNewIconEmb*HndNode methods
Brian Sullivan [Thu, 19 Oct 2017 02:13:59 +0000 (19:13 -0700)]
Removed hnd1 and hnd2 arguments to gtNewIconEmb*HndNode methods

7 years agoClang format fixes
Brian Sullivan [Thu, 19 Oct 2017 01:58:20 +0000 (18:58 -0700)]
Clang format fixes

7 years agoIfdef out legacy uses of GT_ASG_op (#14384)
mikedn [Thu, 19 Oct 2017 01:34:55 +0000 (04:34 +0300)]
Ifdef out legacy uses of GT_ASG_op (#14384)

* Ifdef out legacy uses of GT_ASG_op

GT_ASG_op nodes are only generated when the legacy backend is used.

* Address feedback

* Cleanup gtOverflow/gtOverflowEx

7 years agoRemoved unused handle args
Brian Sullivan [Thu, 19 Oct 2017 01:12:36 +0000 (18:12 -0700)]
Removed unused handle args
from gtNewIconHandleNode and gtNewIcomEmbHndNode

7 years agoFix BinaryWriter/Reader span parameter names (#14577)
Stephen Toub [Thu, 19 Oct 2017 00:55:09 +0000 (20:55 -0400)]
Fix BinaryWriter/Reader span parameter names (#14577)

Per API review, they should match the corresponding overloads' existing parameter names.

7 years agoAllow both CORINFO_FLG_INTRINSIC and CORINFO_FLG_JIT_INTRINSIC to be set on potential...
Jan Kotas [Thu, 19 Oct 2017 00:54:18 +0000 (17:54 -0700)]
Allow both CORINFO_FLG_INTRINSIC and CORINFO_FLG_JIT_INTRINSIC to be set on potential intrinsics (#14566)

* Allow both CORINFO_FLG_INTRINSIC and CORINFO_FLG_JIT_INTRINSIC to be set on potential intrinsics

* Call getIntrinsicID for CORINFO_FLG_INTRINSIC only

7 years agoRemoved unused fields of Icon nodes
Brian Sullivan [Thu, 19 Oct 2017 00:51:37 +0000 (17:51 -0700)]
Removed unused fields of Icon nodes
The union of gtIconHdl1, gtIconHdl2 and gtIconCPX, gtOIconCls is not used

7 years agoMerge pull request #14572 from sdmaclea/PR-ARM64-SIMD-INSTRS
Bruce Forstall [Wed, 18 Oct 2017 23:01:27 +0000 (16:01 -0700)]
Merge pull request #14572 from sdmaclea/PR-ARM64-SIMD-INSTRS

[Arm64] Add instructions needed by SIMD

7 years agoFix multiple issues in LSRA for TYP_DOUBLE:
Carol Eidt [Wed, 18 Oct 2017 21:31:05 +0000 (14:31 -0700)]
Fix multiple issues in LSRA for TYP_DOUBLE:
- updateAssignedInterval() needs to handle the case where the previously assigned interval was TYP_DOUBLE.
- Similarly, at block boundaries, if we are setting the register for a double interval, and the assigned interval is either null or is TYP_FLOAT ,we also need to unassign the other half of the register.
- LSRA only considers the valid double registers when allocating. When setting the candidates for the source of a return, it should only set the bit for the valid double register.

Fix #13751

7 years ago[Arm64] Support flag FEATURE_SIMD
Steve MacLean [Wed, 18 Oct 2017 18:02:19 +0000 (14:02 -0400)]
[Arm64] Support flag FEATURE_SIMD

7 years ago[Arm64] Add instructions needed by SIMD
Steve MacLean [Wed, 18 Oct 2017 18:25:50 +0000 (14:25 -0400)]
[Arm64] Add instructions needed by SIMD

7 years agoMerge pull request #14569 from wtgodbe/eqNe
William Godbe [Wed, 18 Oct 2017 20:15:02 +0000 (13:15 -0700)]
Merge pull request #14569 from wtgodbe/eqNe

Replace ne with eq in publish to azure steps

7 years agoReplace ne with eq in publish to azure steps
wtgodbe [Wed, 18 Oct 2017 18:03:13 +0000 (11:03 -0700)]
Replace ne with eq in publish to azure steps

7 years agoMerge pull request #14554 from adiaaida/reenablePGOLinux
Michelle McDaniel [Wed, 18 Oct 2017 17:41:31 +0000 (10:41 -0700)]
Merge pull request #14554 from adiaaida/reenablePGOLinux

Reenable PGO on Linux Release builds

7 years agoImprove thread statics performance (#14560)
Jan Kotas [Wed, 18 Oct 2017 09:38:55 +0000 (02:38 -0700)]
Improve thread statics performance (#14560)

- Disable code to handle multiple appdomains
- Use more efficient object array accessor

7 years agoUpdate BuildTools, CoreClr, CoreFx to prerelease-02118-01, preview1-25818-02, preview...
dotnet-maestro-bot [Wed, 18 Oct 2017 05:42:06 +0000 (00:42 -0500)]
Update BuildTools, CoreClr, CoreFx to prerelease-02118-01, preview1-25818-02, preview1-25818-01, respectively (#14558)

7 years agoMerge pull request #14547 from CarolEidt/Fix14539
Carol Eidt [Wed, 18 Oct 2017 03:00:00 +0000 (20:00 -0700)]
Merge pull request #14547 from CarolEidt/Fix14539

Arm64: Use op2 type for LOCKADD

7 years agoformat spmi sources. (#14545)
Sergey Andreenko [Wed, 18 Oct 2017 00:12:24 +0000 (17:12 -0700)]
format spmi sources. (#14545)

format spmi sources.

7 years agoMerge pull request #14553 from briansull/fix-warn
Brian Sullivan [Tue, 17 Oct 2017 23:34:37 +0000 (16:34 -0700)]
Merge pull request #14553 from briansull/fix-warn

Fix for x86 desktop build break

7 years agoArm64: Use op2 type for LOCKADD
Carol Eidt [Tue, 17 Oct 2017 20:35:14 +0000 (13:35 -0700)]
Arm64: Use op2 type for LOCKADD

Fix #14539

7 years agoMerge pull request #14555 from wtgodbe/AzureBlobSymPkg
William Godbe [Tue, 17 Oct 2017 22:11:19 +0000 (15:11 -0700)]
Merge pull request #14555 from wtgodbe/AzureBlobSymPkg

Use C#-friendly path for indexing symbol packages

7 years agoUse C#-friendly path for indexing symbol packages
wtgodbe [Tue, 17 Oct 2017 22:10:21 +0000 (15:10 -0700)]
Use C#-friendly path for indexing symbol packages

7 years agoReenable PGO on Linux Release builds
Michelle McDaniel [Tue, 17 Oct 2017 22:06:34 +0000 (15:06 -0700)]
Reenable PGO on Linux Release builds

Now that all the machines have been updated to have llvm 3.9 with PGO
support, reenable PGO builds.

7 years agoFix for x86 desktop build
Brian Sullivan [Tue, 17 Oct 2017 22:03:22 +0000 (15:03 -0700)]
Fix for x86 desktop build
Fix warning C4389: '==' : signed/unsigned mismatch

7 years agoMerge pull request #14541 from stephentoub/taskbeginwait
Stephen Toub [Tue, 17 Oct 2017 21:37:57 +0000 (17:37 -0400)]
Merge pull request #14541 from stephentoub/taskbeginwait

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

7 years agoGetEnvironmentVariable: Avoid StringBuilder marshaling overhead (#14502)
Justin Van Patten [Tue, 17 Oct 2017 21:23:24 +0000 (14:23 -0700)]
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.

7 years agoCLRLifoSemaphore cleanup (#14535)
Koundinya Veluri [Tue, 17 Oct 2017 20:52:52 +0000 (13:52 -0700)]
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.

7 years agoAvoid TaskWaitBegin/End events in Task.Wait on already completed task
Stephen Toub [Tue, 17 Oct 2017 20:01:57 +0000 (16:01 -0400)]
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.

7 years agoMerge pull request #14493 from alpencolt/armel-cross-build
Bruce Forstall [Tue, 17 Oct 2017 18:28:23 +0000 (11:28 -0700)]
Merge pull request #14493 from alpencolt/armel-cross-build

[RyuJIT/armel] cross build armel on x86 host

7 years agoMerge pull request #14334 from BruceForstall/AddTestEnvToWindows
Bruce Forstall [Tue, 17 Oct 2017 18:21:15 +0000 (11:21 -0700)]
Merge pull request #14334 from BruceForstall/AddTestEnvToWindows

Fix corefx testing invocation

7 years agomove ReplaceWith to cpp
Sergey Andreenko [Tue, 17 Oct 2017 02:46:37 +0000 (19:46 -0700)]
move ReplaceWith to cpp

to make DEBUG_DESTROY_NODE visible.

7 years agoRefactor CopyFrom
Sergey Andreenko [Thu, 12 Oct 2017 01:28:35 +0000 (18:28 -0700)]
Refactor CopyFrom

7 years agoadd check that each tree in the method has its own unique gtTreeId.
Sergey Andreenko [Thu, 12 Oct 2017 00:55:25 +0000 (17:55 -0700)]
add check that each tree in the method has its own unique gtTreeId.

7 years agoextract fgDebugCheckStmtsList
Sergey Andreenko [Wed, 11 Oct 2017 21:50:23 +0000 (14:50 -0700)]
extract fgDebugCheckStmtsList

7 years agoMerge pull request #14537 from dotnet-maestro-bot/master-UpdateDependencies
Stephen Toub [Tue, 17 Oct 2017 16:48:45 +0000 (12:48 -0400)]
Merge pull request #14537 from dotnet-maestro-bot/master-UpdateDependencies

Update CoreFx to preview1-25817-02 (master)

7 years agoMerge pull request #14311 from ViktorHofer/SerializationExceptions
Viktor Hofer [Tue, 17 Oct 2017 15:25:39 +0000 (17:25 +0200)]
Merge pull request #14311 from ViktorHofer/SerializationExceptions

Make coreclr exceptions serializable and add typeforwards

7 years agoUpdate CoreFx to preview1-25817-02
dotnet-maestro-bot [Tue, 17 Oct 2017 14:23:43 +0000 (07:23 -0700)]
Update CoreFx to preview1-25817-02

7 years ago[RyuJIT/ARM32] Add the optimization case on CreateDictionaryLookupHelper (#13933)
Sujin Kim [Tue, 17 Oct 2017 13:33:51 +0000 (22:33 +0900)]
[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

7 years ago[RyuJIT/armel] cross build armel on x86 host.
Alexander Soldatov [Fri, 13 Oct 2017 16:14:35 +0000 (19:14 +0300)]
[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 ...

7 years agoprotect strlen from nullptr (#14533)
Sergey Andreenko [Tue, 17 Oct 2017 06:40:43 +0000 (23:40 -0700)]
protect strlen from nullptr (#14533)

7 years agoMerge pull request #14350 from CarolEidt/LsraInfoCleanup
Carol Eidt [Tue, 17 Oct 2017 05:07:15 +0000 (22:07 -0700)]
Merge pull request #14350 from CarolEidt/LsraInfoCleanup

Cleanup of Lowering & LsraInfo

7 years agoMerge pull request #14527 from stephentoub/timer_partition
Stephen Toub [Tue, 17 Oct 2017 02:22:35 +0000 (22:22 -0400)]
Merge pull request #14527 from stephentoub/timer_partition

Reduce Timer lock contention

7 years agoRefactor register selection heuristics
Carol Eidt [Tue, 17 Oct 2017 02:06:01 +0000 (19:06 -0700)]
Refactor register selection heuristics

This is in preparation for tuning these for both throughput and code quality.

7 years agoUpdate BuildTools to prerelease-02116-01 (#14530)
dotnet-maestro-bot [Tue, 17 Oct 2017 01:42:41 +0000 (20:42 -0500)]
Update BuildTools to prerelease-02116-01 (#14530)

7 years agoMerge pull request #14529 from adityamandaleeka/r2r_exception_race
Aditya Mandaleeka [Tue, 17 Oct 2017 01:36:58 +0000 (18:36 -0700)]
Merge pull request #14529 from adityamandaleeka/r2r_exception_race

Fix deadlock in ReadyToRun scenarios between EH and code heap deletion

7 years agoMerge pull request #14501 from briansull/FitsIn
Brian Sullivan [Tue, 17 Oct 2017 00:47:59 +0000 (17:47 -0700)]
Merge pull request #14501 from briansull/FitsIn

Use FitsIn8 when setting GT_INT_CNS costs

7 years agoFix optEarlyPropRewriteTree (#14480)
Sergey Andreenko [Tue, 17 Oct 2017 00:46:05 +0000 (17:46 -0700)]
Fix optEarlyPropRewriteTree (#14480)

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

* fix typo

7 years agoFixed stack probing for RHEL 6.9 (Linux kernel 2.6.32) (#14481)
Roman Artemev [Mon, 16 Oct 2017 23:20:45 +0000 (16:20 -0700)]
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)

7 years agoUpdated GenTree::IsIntCnsFitsInI32() tio use FitsInI32()
Brian Sullivan [Mon, 16 Oct 2017 22:33:37 +0000 (15:33 -0700)]
Updated GenTree::IsIntCnsFitsInI32() tio use FitsInI32()

7 years agoFix deadlock caused by R2R method info lookups during code heap
Aditya Mandaleeka [Mon, 16 Oct 2017 21:58:50 +0000 (14:58 -0700)]
Fix deadlock caused by R2R method info lookups during code heap
deletion.

7 years agoFix stepping with tiered jitting
Noah Falk [Mon, 16 Oct 2017 22:18:43 +0000 (15:18 -0700)]
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.

7 years agoMerge branch 'master' into FitsIn
Brian Sullivan [Mon, 16 Oct 2017 22:17:36 +0000 (15:17 -0700)]
Merge branch 'master' into FitsIn

7 years agoMerge pull request #14522 from dotnet/wtgodbe-patch-1
William Godbe [Mon, 16 Oct 2017 22:17:17 +0000 (15:17 -0700)]
Merge pull request #14522 from dotnet/wtgodbe-patch-1

Parameterize PublishFlatContainer in x86 build

7 years agoReview feedback changes
Brian Sullivan [Mon, 16 Oct 2017 22:16:31 +0000 (15:16 -0700)]
Review feedback changes

7 years agospill GT_RET_EXPR inside fat calli candidates. (#14520)
Sergey Andreenko [Mon, 16 Oct 2017 22:07:23 +0000 (15:07 -0700)]
spill GT_RET_EXPR inside fat calli candidates. (#14520)

spill ret expr