Stephen Toub [Sat, 29 Sep 2018 23:12:48 +0000 (19:12 -0400)]
Improve Guid parsing performance (dotnet/coreclr#20183)
* Improve Guid parsing performance
Significantly improves the performance of parsing all Guid number styles, primarily by avoiding using the StringToInt core helper that was used previously to parse each of the consistuent components, as well as avoiding some unnecessary searches of the strings in order to determine which format is employed.
I kept strong compatibility with the existing implementation, down to what exceptions are thrown when (even when they’re a bit strange). However, there are a few cases where the error messages in those exceptions differ from what they previously were, due to ambiguities, and IMO it not being worth making the implementation slower to try to maintain the exact same choice. For example, the string “{0xdddddddd, 0xdddd 0xdddd,{0xdd,0xdd,0xdd,0xdd,0xdd,0xdd,0xdd,0xdd}}” isn’t parsable, because it’s missing a comma between the second and third components, and with whitespace removed the parser will try to parse “0xdddd0xdddd” and fail to do so. Previously that would result in an error message “Additional non-parsable characters are at the end of the string”, and now it’ll result in an error message “Guid string should only contain hexadecimal characters.” Similarly, “(-ddddddd-dddd-dddd-dddd-
dddddddddddd)” would previously fail with “Unrecognized Guid format”, and now it’ll also fail with “Guid string should only contain hexadecimal characters.”
* Undo int->uint / short->ushort field changes
* Address PR feedback
Commit migrated from https://github.com/dotnet/coreclr/commit/
c04fee1611a7ad7075bc28d9c4e0546ccdad8b30
Jan Kotas [Sat, 29 Sep 2018 13:37:45 +0000 (06:37 -0700)]
Simplify SafeBuffer.Initialize implementation (dotnet/coreclr#20198)
Commit migrated from https://github.com/dotnet/coreclr/commit/
ecbbcf6edab9c4fb723bd59fba04c8177c977840
Jeremy Koritzinsky [Sat, 29 Sep 2018 06:37:34 +0000 (23:37 -0700)]
Unify argument verification for Marshal.GetHINSTANCE off-Windows (dotnet/coreclr#20130)
Commit migrated from https://github.com/dotnet/coreclr/commit/
87e96eafa84545335dd6ec02cb4e233211456677
Jan Kotas [Sat, 29 Sep 2018 06:33:00 +0000 (23:33 -0700)]
Delete redundant framework API tests (dotnet/coreclr#20187)
Contributes to dotnet/coreclr#12782
Commit migrated from https://github.com/dotnet/coreclr/commit/
21d2d302b3eec9d72493b28028784c39cc531a73
Sergey Andreenko [Sat, 29 Sep 2018 01:57:29 +0000 (18:57 -0700)]
Make `structType` optional in jitEEInterface method `getFieldType`. (dotnet/coreclr#20191)
* Make `structType` optional in `getFieldType`.
The declaration in corinfo.h says: "if 'structType' == 0, then don't bother the structure info". However, `getFieldTypeInternal ` did not check this case.
* Do not bother the structure info when we do not need it from `getFieldType`.
Commit migrated from https://github.com/dotnet/coreclr/commit/
f7f0929a468f9a4d115b51da0dc5b9328413352c
Stephen Toub [Sat, 29 Sep 2018 00:05:29 +0000 (20:05 -0400)]
Avoid allocation in Task.Yield() awaiter (dotnet/coreclr#20186)
Now that IAsyncStateMachineBox is an IThreadPoolWorkItem, we can queue it directly in Task.Yield's awaiter. This makes `await Task.Yield();` allocation-free when the default scheduler is used.
Commit migrated from https://github.com/dotnet/coreclr/commit/
7d14e3a60dd9eff95e60b919cf30fd9f306e0e75
Egor Chesakov [Fri, 28 Sep 2018 22:30:26 +0000 (15:30 -0700)]
Specify configuration to runtest.sh to make runtest.py use the correct build_type value (dotnet/coreclr#20188)
Commit migrated from https://github.com/dotnet/coreclr/commit/
c6ef88d88fe8c95323247393c04e6b161da3bb0e
Drew Scoggins [Fri, 28 Sep 2018 22:17:32 +0000 (15:17 -0700)]
Merge pull request dotnet/coreclr#20152 from DrewScoggins/UpdateIBCDeps
Update ibc deps
Commit migrated from https://github.com/dotnet/coreclr/commit/
be944fae4506939e7dbe322148c35826c4bc9dd3
Jan Vorlicek [Fri, 28 Sep 2018 20:09:24 +0000 (22:09 +0200)]
Enable delegate marshalling for collectible types (dotnet/coreclr#20158)
* Enable delegate marshalling for collectible types
This change is trivial - it just removes checks preventing the delegate
marshalling for collectible types. The unmanaged to managed thunks are
allocated from a global LoaderAllocator and they are released when the
corresponding managed delegates are collected. So for unloadability, we
don't need to change this behavior in any way.
* Disable CoreFX tests outdated by the change
Commit migrated from https://github.com/dotnet/coreclr/commit/
49209e181103006603e2cf4e05dccb564bef537d
Tanner Gooding [Fri, 28 Sep 2018 18:02:55 +0000 (11:02 -0700)]
Updating NumberToStringFormat to not print the sign if there are no digits being returned (dotnet/coreclr#20109)
Commit migrated from https://github.com/dotnet/coreclr/commit/
98aff4a23a2ecf8d823ece9b201045f52aa0624a
Luqun Lou [Fri, 28 Sep 2018 17:15:12 +0000 (10:15 -0700)]
Enable BSTR Marshaling Support for x-plat PInvoke (dotnet/coreclr#19766)
Commit migrated from https://github.com/dotnet/coreclr/commit/
f90b65858d7ca0604e7aa1a1d04e9e5261e7f7af
dotnet-maestro-bot [Fri, 28 Sep 2018 15:48:15 +0000 (08:48 -0700)]
Update CoreClr, CoreFx to preview1-26928-03, preview1-26928-03, respectively (dotnet/coreclr#20180)
Commit migrated from https://github.com/dotnet/coreclr/commit/
a67ce5980d6457f1af91b088fed7b95400aa7def
Alexander Soldatov [Fri, 28 Sep 2018 15:16:29 +0000 (18:16 +0300)]
Added transcript of ARMEnabled. (dotnet/coreclr#20181)
Commit migrated from https://github.com/dotnet/coreclr/commit/
c4bf09d95011c0b4f86be1001b228b337a11fdd4
dotnet-maestro-bot [Fri, 28 Sep 2018 11:25:33 +0000 (04:25 -0700)]
Update CoreClr, CoreFx, CoreSetup to preview1-26928-01, preview1-26928-01, preview1-26928-01, respectively (dotnet/coreclr#20175)
Commit migrated from https://github.com/dotnet/coreclr/commit/
c07b543de35c626db2c2b380fd1f4eeccaa5fce0
Stephen Toub [Fri, 28 Sep 2018 00:13:27 +0000 (20:13 -0400)]
Avoid AwaitTaskContinuation allocation in some awaits (dotnet/coreclr#20159)
In .NET Core 2.1, I added a bunch of optimizations to async methods that are based on reusing the async state machine object itself for other purposes in order to avoid related allocations. One of those optimizations was using the boxed state machine itself as the continuation object that could be queued onto a Task, and in the common case where the continuation could be executed synchronously, there would then not be any further allocations. However, if the continuation needed to be run asynchronously (e.g. because the Task required it via RunContinuationsAsynchronously), the code would allocate a new work item object and queue that to the thread pool to execute. This then also forced the state machine object to lazily allocate the Action delegate for its MoveNext method. This PR extends the system slightly to also cover that asynchronous execution case, by making the state machine box itself being queueable to the thread pool. In doing so, it avoids that AwaitTaskContinuation allocation and also avoids forcing the delegate into existence. (As is the case for other optimizations, this one is only employed when ETW logging isn't enabled; if it is enabled, we need to flow more information, and enabling that would penalize the non-logging case.)
Commit migrated from https://github.com/dotnet/coreclr/commit/
03ead514a3aa414b26246a4160d0593811fd1044
dotnet-maestro-bot [Thu, 27 Sep 2018 20:09:51 +0000 (13:09 -0700)]
Update CoreClr, CoreFx, CoreSetup, PgoData to preview1-26927-03, preview1-26927-03, preview1-26927-01, master-
20180926-0038, respectively (dotnet/coreclr#20153)
Commit migrated from https://github.com/dotnet/coreclr/commit/
883cc1a1d611727e47332507cbfbac01c36c6dec
Stephen Toub [Thu, 27 Sep 2018 18:58:18 +0000 (14:58 -0400)]
Fix CancellationTokenRegistration.Dispose racing with cancellation (dotnet/coreclr#20145)
CancellationTokenRegistration.Dispose is guaranteed to only return when the associated callback has already finished running or has been successfully removed such that it'll never run. This is to ensure that code following Dispose can be guaranteed that the callback isn't currently running and won't after that point, as if it did, it could potentially depend on mutable shared state or itself mutate shared state in a non-thread-safe manner.
However, significant optimizations introduced in .NET Core 2.1 impacted a specific case of this guarantee. It still behaves correctly if cancellation hasn't already been requested, if cancellation has already processed the associated callback, and even if cancellation is currently executing the associated callback. However, if cancellation is currently processing other callbacks and hasn't yet gotten around to processing the associated one, Dispose() may incorrectly return immediately, such that the callback may still end up getting invoked after Dispose() returns, which can violate assumptions of consuming code in very impactful ways.
This commit modifies how the callbacks are removed from the registration list in order to fix the issue. Previously, all of the callbacks were swapped out at once, which then left the list empty, which is what caused subsequent disposals to think the callback had already been processed or unregistered. With this change, we instead just remove each registration as it's being processed, such that a concurrent disposal can still successfully find the registration in the callback list if it's not yet been processed.
This change does have a small perf impact, but only on the case where cancellation is actually invoked, which is the less common case; most usage of CTS doesn't actually result in cancellation, and optimization is focused on registration and unregistration perf. Rather than taking and releasing the lock once when cancellation is requested, we now take and release the lock per callback when cancellation is requested.
Commit migrated from https://github.com/dotnet/coreclr/commit/
552f4bf7addc944d85f4604fd2fe228ac984412d
Egor Chesakov [Thu, 27 Sep 2018 18:28:17 +0000 (11:28 -0700)]
Enable Ubuntu arm Cross Release crossgen_comparison scenario by default (dotnet/coreclr#20160)
Commit migrated from https://github.com/dotnet/coreclr/commit/
c11f789873e46a53b42bb1c91ce81c3669ca7145
Egor Chesakov [Thu, 27 Sep 2018 16:01:39 +0000 (09:01 -0700)]
Merge pull request dotnet/coreclr#19960 from echesakovMSFT/UseCrossBitnessLinuxCrossGenByDefault
Use Hostx64/arm crossgen to crossgen System.Private.CoreLib on Linux
Commit migrated from https://github.com/dotnet/coreclr/commit/
9404b663a48979d3dc4b7545077ceb627d539dde
Michelle McDaniel [Thu, 27 Sep 2018 14:28:36 +0000 (07:28 -0700)]
Replace Crst* with CrstStatic for IBCLogger (dotnet/coreclr#20148)
Commit migrated from https://github.com/dotnet/coreclr/commit/
3d9eed70cfc299906c94e0f2722c7d39f777f27d
Viktor Hofer [Thu, 27 Sep 2018 04:43:48 +0000 (06:43 +0200)]
Maestro dependency update without buildtools (dotnet/coreclr#20143)
* Update BuildTools, CoreClr, CoreFx, CoreSetup, PgoData to preview1-03224-03, preview1-26925-05, preview1-26925-06, preview1-26925-01, master-
20180925-0031, respectively
* Enable RestoreOutputPath override
* Disable System.Security.Cryptography.Pkcs.Tests
To help unblock corefx, etc. update.
* Revert buildtools changes
* Disable buildtools auto updates
Commit migrated from https://github.com/dotnet/coreclr/commit/
0bc8c62cfb7514b91b4ebd5f073835391e0502a3
Egor Chesakov [Thu, 27 Sep 2018 02:52:44 +0000 (19:52 -0700)]
Enable Ubuntu arm crossgen_comparison Checked in each PR (dotnet/coreclr#20139)
* Run Ubuntu arm crossgen_comparison (Checked | Release) daily
* Enable Ubuntu arm Checked crossgen_comparison by default
Commit migrated from https://github.com/dotnet/coreclr/commit/
6dc2556a685674ecb9c716aa21720f2295ff8df0
Egor Chesakov [Thu, 27 Sep 2018 02:47:18 +0000 (19:47 -0700)]
Cross-bitness in VTable Calls (dotnet/coreclr#20137)
* Use TARGET_POINTER_SIZE in CEEInfo::getMethodVTableOffset in src/vm/jitinterface.cpp
* Use SIZEOF__MethodTable_ in src/vm/methodtable.h
Commit migrated from https://github.com/dotnet/coreclr/commit/
be18f2b4a7e0a4f7257df048e92d64a73035ed92
Sergey Andreenko [Wed, 26 Sep 2018 23:48:13 +0000 (16:48 -0700)]
Do not treat a custom layout as overlapping when trying to inline a struct method. (dotnet/coreclr#20141)
* Do not treat custom layout as overlapping when trying to inline struct method.
That hack was added 4 years ago with independent struct promotion for parameters.
I was not able to find any related issues.
This change produces asm diffs because it allows us to inline more (lvaCanPromoteStructVar is a multiplier for
inlineIsProfitable parameter).
For System.Private.CoreLib we have total bytes of diff: 294 (0.01% of base).
For example, we started to inline methods from 'System.Threading.Tasks.ValueTask' that has '[StructLayout(LayoutKind.Auto)]'.
* Always sort fields in lcCanPromoteStructType.
It will be optimized away in the future commits.
* Delete the argument that is no longer used.
* Fix variable name according to jit-coding-conventions.
Rename 'StructPromotionInfo' to 'structPromotionInfo'.
Commit migrated from https://github.com/dotnet/coreclr/commit/
24bbc37eb70895b6d7cc9cc95b8453742cdc0cae
Jacek Blaszczynski [Wed, 26 Sep 2018 23:47:14 +0000 (01:47 +0200)]
Remove deleted HW intrinsic tests from exclusion lists for Windows arm, arm64 targets (dotnet/coreclr#20151)
Fixes dotnet/coreclr#20149
Commit migrated from https://github.com/dotnet/coreclr/commit/
b1a9f1fdb0ef7a9dbf1659a419a0bf064c38106e
Bruce Forstall [Wed, 26 Sep 2018 23:09:10 +0000 (16:09 -0700)]
Merge pull request dotnet/coreclr#20098 from dotnetrt/fix-win-unix-format
[CI] Fix failures in Windows/Linux CI Formatting jobs by force killing all dotnet processess
Commit migrated from https://github.com/dotnet/coreclr/commit/
5addb2d0ca2e845d8bb512eab9b6f221092cafa0
Drew Scoggins [Wed, 26 Sep 2018 22:44:33 +0000 (15:44 -0700)]
Update BuildInfoPaths for PGO and IBC packages
Commit migrated from https://github.com/dotnet/coreclr/commit/
d79d5e08dc9d38f8464a3242e046c4058797eebc
Jeremy Koritzinsky [Wed, 26 Sep 2018 22:38:24 +0000 (15:38 -0700)]
Merge pull request dotnet/coreclr#20132 from jkoritzinsky/fixes/coreclr/7829
Fix possible overflow in SafeBuffer.Initialize.
Commit migrated from https://github.com/dotnet/coreclr/commit/
c7e9526b108c10360b71f7649e118831c54399ea
Drew Scoggins [Wed, 26 Sep 2018 20:34:21 +0000 (13:34 -0700)]
Update dependencies.props to fix typo
In the XmlUpdateStep item for the IBC data had a typo and was called
Data not IbcData. This was causing Maestro to not update the
IbcCurrentRef automatically.
Commit migrated from https://github.com/dotnet/coreclr/commit/
fc3e4cb86fd7effe7e76b6c4a3f29b762a113999
Bruce Forstall [Wed, 26 Sep 2018 18:17:16 +0000 (11:17 -0700)]
Merge pull request dotnet/coreclr#19939 from mikedn/visit-cmpxchg
Fix incorrect CMPXCHG visiting order
Commit migrated from https://github.com/dotnet/coreclr/commit/
da1ad5976a8002612cb8aef23aecd3a47f6453be
Jeremy Koritzinsky [Wed, 26 Sep 2018 17:57:18 +0000 (10:57 -0700)]
Remove duplicated test
Commit migrated from https://github.com/dotnet/coreclr/commit/
ae752f0e0a31784b30c7283c959c83c59ed2a154
Jeremy Koritzinsky [Wed, 26 Sep 2018 17:39:43 +0000 (10:39 -0700)]
Add outdated test to CoreFX.issues.json
Commit migrated from https://github.com/dotnet/coreclr/commit/
904e0f142ff5e48e563a0e22f422e758ffab6c9e
Jeremy Koritzinsky [Wed, 26 Sep 2018 17:31:58 +0000 (10:31 -0700)]
PR Feedback and exclude regression test on 64-bit processes.
Commit migrated from https://github.com/dotnet/coreclr/commit/
f211a76a9e45f2ed7e5f515810e0e04e97520ff2
Luqun Lou [Wed, 26 Sep 2018 16:32:26 +0000 (09:32 -0700)]
Use platformdefines through lib instead of cpp (dotnet/coreclr#19948)
* Use platformdefines through lib instead of cpp
* add a dummy export function
* Remove dummy export
* Compile as Static Library
* revert platformdefines.cpp file
* remove DLL_EXPORT
Commit migrated from https://github.com/dotnet/coreclr/commit/
e2a45dce9a853c3eead8aa635317f5615d60bdb6
Jan Kotas [Wed, 26 Sep 2018 06:07:00 +0000 (23:07 -0700)]
Delete dead .NET Framework activation code (dotnet/coreclr#20134)
Commit migrated from https://github.com/dotnet/coreclr/commit/
f5f73ff2758f5c308561716fe8fb8b7cf6bc3cda
Jeremy Koritzinsky [Tue, 25 Sep 2018 23:58:20 +0000 (16:58 -0700)]
Add regression test.
Commit migrated from https://github.com/dotnet/coreclr/commit/
9864e50091aebb4c0c5cca873d94e71f0dd84207
Jeremy Koritzinsky [Tue, 25 Sep 2018 23:45:04 +0000 (16:45 -0700)]
Simpler implementation from PR feedback.
Commit migrated from https://github.com/dotnet/coreclr/commit/
35dff1a2277d583b9d554c853bc4b9a31f6f79a0
Sung Yoon Whang [Tue, 25 Sep 2018 23:18:14 +0000 (16:18 -0700)]
Porting fix on Desktop for VirtualUnlock perf issue on WKS GC when we reset mem (dotnet/coreclr#20115)
Commit migrated from https://github.com/dotnet/coreclr/commit/
b67d2fa8fc8cf5b8c41f5c34b1abe31e704386c4
Jeremy Koritzinsky [Tue, 25 Sep 2018 21:54:26 +0000 (14:54 -0700)]
Fix possible overflow in SafeBuffer.Initialize.
Since a 0-length buffer is technically possible (though not very usable), have sizeOfEachElement==0 -> ByteLength == 0.
Commit migrated from https://github.com/dotnet/coreclr/commit/
0c288c367db3dc7c296e67a4bb2fff84f0f93c3f
Egor Chesakov [Tue, 25 Sep 2018 18:51:42 +0000 (11:51 -0700)]
Merge pull request dotnet/coreclr#19979 from echesakovMSFT/CrossGenComparisonSymbolFiles
CrossGen comparison - compare debugging files
Commit migrated from https://github.com/dotnet/coreclr/commit/
694d8d7fd2fb5d5cf0f664a185d94fea1166ae2f
Jacek Blaszczynski [Tue, 25 Sep 2018 17:11:26 +0000 (19:11 +0200)]
Add header files to ildasm, ilasm vcxproj (dotnet/coreclr#20052)
Commit migrated from https://github.com/dotnet/coreclr/commit/
659af58047a949ed50d11101708538d2e87f2568
Bruce Forstall [Tue, 25 Sep 2018 16:55:29 +0000 (09:55 -0700)]
Merge pull request dotnet/coreclr#20107 from dotnet/dev/unix_test_workflow
Improve Linux arm32/arm64 corefx test runs
Commit migrated from https://github.com/dotnet/coreclr/commit/
1496647a946279908acd8d03210ac3ad114204d2
Sergey Andreenko [Tue, 25 Sep 2018 16:38:42 +0000 (09:38 -0700)]
switch to netcoreapp2.0 (dotnet/coreclr#20120)
Allows to get rid of warning " CSC : warning CS1685: The predefined type 'MarshalByRefObject' is defined in multiple assemblies in the global alias;".
Commit migrated from https://github.com/dotnet/coreclr/commit/
337f0e0fc4ee05106909e3d881279dc4d494e36f
Jacek Blaszczynski [Tue, 25 Sep 2018 14:20:05 +0000 (16:20 +0200)]
Add headers to cordb{di, ee_dac, ee_wks}.vcxproj files to improve developer experience (dotnet/coreclr#20049)
Commit migrated from https://github.com/dotnet/coreclr/commit/
2e56748c414f9ead45b9730a0a7fb364f751ab83
Jan Vorlicek [Mon, 24 Sep 2018 23:51:44 +0000 (01:51 +0200)]
Fix hijacking for ARM/ARM64/x86 on Unix (dotnet/coreclr#20042)
* Fix hijacking for ARM/ARM64/x86 on Unix
We were not checking the case when we have interrupted a thread inside
of a function epilog for other architectures than amd64. When such an
interruption happens, GS cookie check in a stack walking has failed,
since we are unable to decode GS cookie location when the IP is in
epilog.
This fix implements IsIPInEpilog for all architectures and makes the
check unconditional.
Commit migrated from https://github.com/dotnet/coreclr/commit/
f3984776548d45f1ac06f791c8ee8658d7927fad
Davis Goodin [Mon, 24 Sep 2018 23:48:05 +0000 (18:48 -0500)]
Don't check for libintl.h on Darwin (OSX) (dotnet/coreclr#20094)
Commit migrated from https://github.com/dotnet/coreclr/commit/
4e4665f127d2ec8f361ad068690f79be20a63280
Sergey Andreenko [Mon, 24 Sep 2018 23:03:36 +0000 (16:03 -0700)]
clean code that counts promoted implict byref argument occurrences. (dotnet/coreclr#20113)
* clean code that counts promoted implict byref argument occurrences.
Move 3 copies of that code into one in `PreOrderVisit`. Run this check after we have made morph for fields and structs.
Commit migrated from https://github.com/dotnet/coreclr/commit/
bcb2a56138685748c5a8c1d8c973ff1a51c81883
Jan Kotas [Mon, 24 Sep 2018 21:19:05 +0000 (14:19 -0700)]
Simplify OSVERSIONINFOEX interop (dotnet/coreclr#20111)
Commit migrated from https://github.com/dotnet/coreclr/commit/
2f0c406fc9f56e9ad2ae6f7138b88c15e96d0716
谭九鼎 [Sun, 23 Sep 2018 18:52:46 +0000 (02:52 +0800)]
Remove unnecessary parenthesis (dotnet/corefxdotnet/coreclr#32419)
Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
Commit migrated from https://github.com/dotnet/coreclr/commit/
72ba9e20d488a85e8a7eac0ff62a01968a2f6be5
Bruce Forstall [Sat, 22 Sep 2018 06:10:55 +0000 (23:10 -0700)]
Improve Linux arm32/arm64 corefx test runs.
Add proper parallelism.
Prevent per-test output during parallel runs from interleaving
by saving output per process and displaying it after the
process has completed.
Improve summary test/failure output at end of log.
Change Linux arm32/arm64 corefx jobs to run tests in parallel.
Change Linux/arm64 corefx scheduled jobs to be daily, not weekly,
since they are very fast when run in parallel (<20 minutes).
Commit migrated from https://github.com/dotnet/coreclr/commit/
07b8ea2ca94fc519c68f211e1494ebc7ccd588f1
Bruce Forstall [Fri, 21 Sep 2018 22:50:17 +0000 (15:50 -0700)]
Move run-tests.sh from corefx
This is the script used by Jenkins to run arm32/arm64 corefx testing
in the coreclr repo.
Leave the original one around, in case anyone uses it.
Name it run-corefx-tests.sh, to more closely model what we have for
Windows arm/arm64 corefx testing.
Commit migrated from https://github.com/dotnet/coreclr/commit/
a08d45d8dab3e8a0154d2e9fbc51afd3dd4159df
Tanner Gooding [Sat, 22 Sep 2018 13:43:30 +0000 (06:43 -0700)]
Porting NumberToDouble to managed code. (dotnet/coreclr#20080)
* Porting NumberToDouble to managed code.
* Deleting bcltype/number.cpp and bcltype/number.h
* Fixing NumberToDouble to call Int64BitsToDouble, rather than DoubleToInt64Bits
* Some minor code cleanup in NumberToDouble for better readability.
* Some additional code cleanup in the Number.NumberToDouble.cs code
Commit migrated from https://github.com/dotnet/coreclr/commit/
09cc49e8cac72915b72240c766e25ada171e9fe7
Jacek Blaszczynski [Fri, 21 Sep 2018 23:30:49 +0000 (01:30 +0200)]
Fix failures in Windows/Linux CI Formatting jobs by killing all dotnet processess
Use shell commands to kill all processess with dotnet/dotnet.exe name before
attempting the final cleanup of the CI workspace
Commit migrated from https://github.com/dotnet/coreclr/commit/
29025ba85295cf0f1d3a1b45dcbc3be96ad74690
Egor Chesakov [Thu, 20 Sep 2018 22:56:12 +0000 (15:56 -0700)]
Address PR feedback - Cleanup temp dirs after usage
Commit migrated from https://github.com/dotnet/coreclr/commit/
223a88c02050887b5284ca334c5fbbef9180df9e
Bruce Forstall [Sat, 22 Sep 2018 01:17:24 +0000 (18:17 -0700)]
Merge pull request dotnet/coreclr#20095 from BruceForstall/Disable80373
Disable test b80373 for all non-Windows targets
Commit migrated from https://github.com/dotnet/coreclr/commit/
99bf5ce15efc1179b47d1e1d1522673a30ae8628
Ahson Khan [Sat, 22 Sep 2018 00:47:47 +0000 (17:47 -0700)]
Streamline MemoryExtension Trim and Trim(char) by removing calls to TrimStart/End and avoiding unnecessary Slice. (dotnet/coreclr#19959)
* Mark TrimStart and TrimEnd as Aggressively Inline to improve perf
* Remove inlining attribute and streamline Trim
Commit migrated from https://github.com/dotnet/coreclr/commit/
1fd5caec53c82d437d7c792d3dbf3e9f6a43e018
Billy O'Neal [Fri, 21 Sep 2018 23:02:58 +0000 (16:02 -0700)]
Add missing <cmath> header for fabs to fix Visual C++ vNext break, av… (dotnet/coreclr#20079)
* Add missing <cmath> header for fabs to fix Visual C++ vNext break, avoid narrowing to float from double
Commit migrated from https://github.com/dotnet/coreclr/commit/
a6474505115fe45fb1cef161e7ba71c30c62001a
Bruce Forstall [Fri, 21 Sep 2018 22:14:48 +0000 (15:14 -0700)]
Disable test b80373 for all non-Windows targets
Tracked by dotnet/coreclr#20024
Commit migrated from https://github.com/dotnet/coreclr/commit/
cadedd4bc656ec50f00658ac4c065ab55b82bb14
Brian Sullivan [Fri, 21 Sep 2018 21:01:11 +0000 (14:01 -0700)]
Merge pull request dotnet/coreclr#20033 from briansull/normalizeAssertionProp
Changes to use VNNormalValue in assertionProp
Commit migrated from https://github.com/dotnet/coreclr/commit/
48095cba91577ded6a440e05a969a81fa85b0b88
Andrew Arnott [Fri, 21 Sep 2018 17:13:43 +0000 (10:13 -0700)]
Fix copy-paste error in xml doc comment (dotnet/corefxdotnet/coreclr#32388)
Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
Commit migrated from https://github.com/dotnet/coreclr/commit/
53e92937a5a8cc454921b55c7a296d3ab7bedf96
Michelle McDaniel [Fri, 21 Sep 2018 16:48:31 +0000 (09:48 -0700)]
Move smoketests to new helix queue (dotnet/coreclr#20090)
Commit migrated from https://github.com/dotnet/coreclr/commit/
deb192495ca8e9f7b02786ffe31067b8fbcfd32e
Marek Safar [Fri, 21 Sep 2018 15:07:36 +0000 (17:07 +0200)]
Simplifies UTF32Encoding/UTF8Encoding/UnicodeEncoding ctors initialization flow (dotnet/coreclr#20072)
Commit migrated from https://github.com/dotnet/coreclr/commit/
6654fd3e291128f1d2dee9e0a1ee49d1fd612dff
Simon Nattress [Thu, 20 Sep 2018 20:47:09 +0000 (13:47 -0700)]
Fix decoding bug in r2rdump's native format reader
Sign extension is done based on a 32-bit int. We were incorrectly basing it off the size of a byte.
Commit migrated from https://github.com/dotnet/coreclr/commit/
6410f031da3dd7f25dce79ad29fa348733598df2
Fadi Hanna [Fri, 21 Sep 2018 05:06:48 +0000 (22:06 -0700)]
Performance fix for R2R: vtable calls (dotnet/coreclr#20061)
* Performance fix for R2R: use vtable-based codegen for virtual calls within the System.Private.CoreLib version bubble, avoiding the use of the VSD, or in the generics case, a dictionary lookup.
The CoreLib assembly will always be serviced along side the coreclr runtime, so special casing CoreLib to using vtable-based calls like the fragile NI case is ok.
Commit migrated from https://github.com/dotnet/coreclr/commit/
0149085e0156e357d7cfc2fe28c3f62355027167
Mike McLaughlin [Fri, 21 Sep 2018 04:28:27 +0000 (21:28 -0700)]
Fix issue dotnet/coreclr#20076 createdump faults. (dotnet/coreclr#20084)
Return E_FAIL instead of S_FALSE from ClrDataFrame::GetLocalSig().
Also issue https://github.com/dotnet/diagnostics/issues/61
Commit migrated from https://github.com/dotnet/coreclr/commit/
f1cf6e7172ae0f0fc3c4617c810d71c514c5e0ba
Simon Nattress [Wed, 19 Sep 2018 17:24:02 +0000 (10:24 -0700)]
Make EH table optional in R2RDump
Ready-to-run binaries don't always have an EH table. Fix r2rdump so it doesn't crash if the table isn't present.
Commit migrated from https://github.com/dotnet/coreclr/commit/
10e34eb55c3bb0a16969b45c97e12b275549ec20
Brian Sullivan [Fri, 21 Sep 2018 00:00:23 +0000 (17:00 -0700)]
Updated assertionprop.cpp
Commit migrated from https://github.com/dotnet/coreclr/commit/
9d49bf1ec6f102b89e5c2885e8f9d3d77f2ec144
Jacek Blaszczynski [Thu, 20 Sep 2018 23:09:36 +0000 (01:09 +0200)]
Convert Sse2 tests PackSignedSaturate, PackUnsignedSaturate, UnpackHigh and UnpackLow to template based (dotnet/coreclr#19670)
* Convert PackSignedSaturate, PackUnsignedSaturate, UnpackHigh, UnpackLow tests to template based
* Add generated Sse2 Pack***Saturate, UnpackHigh, UnpackLow tests, remove replaced tests
* Remove deleted Sse2 Pack** Unpack** tests from Test.lst files for arm and arm64 Windows targets
Commit migrated from https://github.com/dotnet/coreclr/commit/
4180182cae098b55cb6e2711cb2c2c43decaac0a
Andy Ayers [Thu, 20 Sep 2018 22:02:58 +0000 (15:02 -0700)]
JIT: Fix operand evaluation order for GT_INDEX_ADDR (dotnet/coreclr#20047)
We need to evaluate the array operand first, and it's op1. So evaluate
in that order, and don't allow reversal.
Closes dotnet/coreclr#20040.
Commit migrated from https://github.com/dotnet/coreclr/commit/
e30f187cda3767e1c50b870864de8d0eb5f8582a
Jan Kotas [Thu, 20 Sep 2018 21:14:48 +0000 (14:14 -0700)]
Move RegistryKey to shared CoreLib partition (dotnet/coreclr#20067)
* Move RegistryKey to shared CoreLib partition
- Cut down RegistryKey to just what CoreLib needs. I went back and forth on whether to share the corefx implementation with ifdefs or not. I have choosen to duplicate it at the end. The ifdefs were either too complex or there was too much cruft left behind that the IL linker was not able to remove.
- Move the internal CoreLib implementation of Registry to Internal.Win32 namespace to ensure that it is not confused with the public standlone one
Fixes dotnet/coreclr#10741 and dotnet/coreclr#17899
Commit migrated from https://github.com/dotnet/coreclr/commit/
4ca5cd260a1c04f74961faccfad3200fe865a249
Fei Peng [Wed, 19 Sep 2018 21:45:32 +0000 (14:45 -0700)]
Disable tests of BroadcastScalarToVector128
Commit migrated from https://github.com/dotnet/coreclr/commit/
0f4ecd9ccc1c7bad32d7ec8eb8bb335f67cfd432
Fei Peng [Wed, 19 Sep 2018 20:38:56 +0000 (13:38 -0700)]
Add pointer overloads for Avx2.BroadcastScalarToVector128
Commit migrated from https://github.com/dotnet/coreclr/commit/
eb4db9f0494b879b9a94ab1b4c3e1e8182d917aa
Fei Peng [Wed, 19 Sep 2018 23:29:01 +0000 (16:29 -0700)]
Add unsigned overloads for MultiplyLow
Commit migrated from https://github.com/dotnet/coreclr/commit/
fcebb9b70f60b2b0661363134675f046037eba0e
Fei Peng [Wed, 19 Sep 2018 22:57:35 +0000 (15:57 -0700)]
Add all integer overloads for AVX2/SSSE3 AlignRight
Commit migrated from https://github.com/dotnet/coreclr/commit/
3ccad6222e34b0b5d1fbd8f65ab93dff89e0179a
Fei Peng [Wed, 19 Sep 2018 22:48:35 +0000 (15:48 -0700)]
Add all integer overloads for Avx2/SSE4.1 BlendVariable
Commit migrated from https://github.com/dotnet/coreclr/commit/
0927f22958e6dd4fcda160fd87519008f7e19bd1
Brian Sullivan [Thu, 20 Sep 2018 20:29:34 +0000 (13:29 -0700)]
Added methods VNLiberalNormalValue and VNConservativeNormalValue
Commit migrated from https://github.com/dotnet/coreclr/commit/
854764f4837180046513449cd111349f0d82d8a5
Tanner Gooding [Wed, 19 Sep 2018 21:25:36 +0000 (14:25 -0700)]
Moving GetExponent/Mantissa and make BigInteger used fixed-sized buffer
Commit migrated from https://github.com/dotnet/coreclr/commit/
97e8b443d7da613e211659743dafe4dec870b4db
Tanner Gooding [Mon, 17 Sep 2018 22:15:43 +0000 (15:15 -0700)]
Removing BitScanReverse from pal.h
Commit migrated from https://github.com/dotnet/coreclr/commit/
cee8cc77e91f5a2c249da29b95f3ee0d10bbed7a
Tanner Gooding [Mon, 17 Sep 2018 21:29:49 +0000 (14:29 -0700)]
Fixing some naming conventions and removing dead code.
Commit migrated from https://github.com/dotnet/coreclr/commit/
77daa1a1b3aab5dba05893b7e3c216ff266823e2
Tanner Gooding [Mon, 17 Sep 2018 16:45:13 +0000 (09:45 -0700)]
Removing bcltype/fp.h from native code.
Commit migrated from https://github.com/dotnet/coreclr/commit/
6498dea67c7148accfea03b28e4168ed67627dff
Tanner Gooding [Mon, 17 Sep 2018 16:28:30 +0000 (09:28 -0700)]
Making Number.Grisu3.DigitGen slightly more efficient.
Commit migrated from https://github.com/dotnet/coreclr/commit/
ad3d6f89ac8270e64c4a5bd15137041ab07225b7
Tanner Gooding [Mon, 17 Sep 2018 07:03:26 +0000 (00:03 -0700)]
Removing the Grisu3 native implementation.
Commit migrated from https://github.com/dotnet/coreclr/commit/
8eb7f92283699d727dc4568e76ef5ae4afa0d90f
Tanner Gooding [Mon, 17 Sep 2018 06:55:18 +0000 (23:55 -0700)]
Porting the Grisu3 algorithm to managed code.
Commit migrated from https://github.com/dotnet/coreclr/commit/
f71a7739db60bf42f8c44037c8feca514bd6757d
Tanner Gooding [Mon, 17 Sep 2018 06:16:36 +0000 (23:16 -0700)]
Porting bcltype/diyfp.cpp to managed code as shared/System/Number.DiyFp.cs
Commit migrated from https://github.com/dotnet/coreclr/commit/
e90208629cd440346744127a5b169b4c4e2c5152
Tanner Gooding [Mon, 17 Sep 2018 05:35:26 +0000 (22:35 -0700)]
Removing the Dragon4 and DoubleToNumber native implementation.
Commit migrated from https://github.com/dotnet/coreclr/commit/
520d408e2ce6a243779db51661a869af199908d4
Tanner Gooding [Mon, 17 Sep 2018 05:34:40 +0000 (22:34 -0700)]
Porting the Dragon4 algorithm to managed code.
Commit migrated from https://github.com/dotnet/coreclr/commit/
b9e981c3e8fc1c718958ca3e0433ba6a6e3cf3cc
Tanner Gooding [Mon, 17 Sep 2018 05:30:56 +0000 (22:30 -0700)]
Porting bcltype/bignum.cpp to managed code as shared/System/Number.BigInteger.cs
Commit migrated from https://github.com/dotnet/coreclr/commit/
27cb9b9750ee324c259c8b8c15054bafdde5cb92
Michelle McDaniel [Thu, 20 Sep 2018 16:58:01 +0000 (09:58 -0700)]
Revert "Move smoketests to use Helix queue (dotnet/coreclr#20017)" (dotnet/coreclr#20058)
This reverts commit dotnet/coreclr@
d529163b2dbc880117f436d8a824d531b1506346.
Commit migrated from https://github.com/dotnet/coreclr/commit/
a72d196801aea97807aaf8e57861ee46cacd623f
Egor Chesakov [Thu, 20 Sep 2018 16:53:13 +0000 (09:53 -0700)]
Merge pull request dotnet/coreclr#20035 from echesakovMSFT/CrossBitnessLinuxPerfMap
Make Hostx64/arm crossgen /CreatePerfMap behave the same as Hostarm/arm crossgen
Commit migrated from https://github.com/dotnet/coreclr/commit/
3d2e9cce9619d4664c317edeabceaa9ffd81f00a
Bruce Forstall [Thu, 20 Sep 2018 15:27:58 +0000 (08:27 -0700)]
Merge pull request dotnet/coreclr#20071 from BruceForstall/FixViews
Fix arm/arm64 Jenkins views
Commit migrated from https://github.com/dotnet/coreclr/commit/
520e089230fe5897be167e1586f8d2abba13745f
Marek Safar [Thu, 20 Sep 2018 15:05:17 +0000 (17:05 +0200)]
Cleans up error message caching in DuplicateWaitObjectException (dotnet/coreclr#20073)
Commit migrated from https://github.com/dotnet/coreclr/commit/
b42022031a5a2883890845ff93c807a5a0ad10ff
Bruce Forstall [Thu, 20 Sep 2018 07:22:40 +0000 (00:22 -0700)]
Fix arm/arm64 Jenkins views
Filter out the non-flow jobs, especially the corefx builds that are
consumed by corefx flow jobs.
Commit migrated from https://github.com/dotnet/coreclr/commit/
3a77e02658e8dafbd34f4b49146f4720c9f6f610
Bruce Forstall [Thu, 20 Sep 2018 07:58:57 +0000 (00:58 -0700)]
Merge pull request dotnet/coreclr#20069 from dotnet/dev/unix_test_workflow
Refine Linux arm64 "limited hardware" designation
Commit migrated from https://github.com/dotnet/coreclr/commit/
3ef621a7614e5a5a41917f040f4c03503f32af57
Bruce Forstall [Thu, 20 Sep 2018 07:41:46 +0000 (00:41 -0700)]
Merge pull request dotnet/coreclr#20070 from BruceForstall/FixLimited
Exclude Windows from arm64 daily corefx job scheduling
Commit migrated from https://github.com/dotnet/coreclr/commit/
7b6ba62db21de37a7154ee56fc117f949061c663
Bruce Forstall [Thu, 20 Sep 2018 07:40:45 +0000 (00:40 -0700)]
Exclude Windows from arm64 daily corefx job scheduling
Commit migrated from https://github.com/dotnet/coreclr/commit/
ff8c9ea68f45b8620d36560bc4f4c18a3daf6e5f
Bruce Forstall [Thu, 20 Sep 2018 07:17:40 +0000 (00:17 -0700)]
Merge pull request dotnet/coreclr#20068 from BruceForstall/FixLimitedHardware
Refine Linux arm64 "limited hardware" designation
Commit migrated from https://github.com/dotnet/coreclr/commit/
b6d3e7078c89d17d9cea040a901b227920edd318
Bruce Forstall [Thu, 20 Sep 2018 06:51:50 +0000 (23:51 -0700)]
Refine Linux arm64 "limited hardware" designation
Add Linux arm64 to the limited hardware category, meaning non-PR jobs
only run in the master branch.
However, allow non-corefx JIT stress jobs to be scheduled daily,
as they are fast. If corefx jobs are changes to run in parallel
(and thus are faster), they can also possibly be made to run daily.
Commit migrated from https://github.com/dotnet/coreclr/commit/
962743bee85bee0b6b68443d70d3d569be05098c
Bruce Forstall [Thu, 20 Sep 2018 06:33:57 +0000 (23:33 -0700)]
Merge pull request dotnet/coreclr#20065 from BruceForstall/FixArm64Innerloop
Fix arm64 Linux innerloop triggered jobs
Commit migrated from https://github.com/dotnet/coreclr/commit/
5f0726b5991c8113e7b62c8f4edc5c11c20787dc
Bruce Forstall [Thu, 20 Sep 2018 00:27:07 +0000 (17:27 -0700)]
Fix arm64 Linux innerloop triggered jobs
Only trigger flow job, not build job.
Commit migrated from https://github.com/dotnet/coreclr/commit/
0848eb109ba07fcddc20a45538d651f499222170
Bruce Forstall [Thu, 20 Sep 2018 00:07:50 +0000 (17:07 -0700)]
Merge pull request dotnet/coreclr#20064 from BruceForstall/FixSpace
Remove extra space from verbose context strings
Commit migrated from https://github.com/dotnet/coreclr/commit/
4b9a9d71b06fbce71c7d000e8b2ed3623976b30c