platform/upstream/dotnet/runtime.git
20 months agoAdd and use a MakeSrcRegOptional that validates IsSafeToContainMem was called (#77895)
Tanner Gooding [Mon, 7 Nov 2022 15:11:41 +0000 (07:11 -0800)]
Add and use a MakeSrcRegOptional that validates IsSafeToContainMem was called (#77895)

* Add and use a MakeSrcRegOptional that validates IsSafeToContainMem was called

* Applying formatting patch

* Revert the MakeSrcRegOptional in ContainCheckDivOrMod due to an assert

* Moving TryMakeSrcContainedOrRegOptional into the lower.cpp file

20 months agocheck the right return value for mmap (#77952)
Manish Godse [Mon, 7 Nov 2022 14:44:03 +0000 (06:44 -0800)]
check the right return value for mmap (#77952)

20 months agoRemove unnecessary encoding allocations in System.Private.DataContractSerialization...
Krzysztof Wicher [Mon, 7 Nov 2022 10:11:20 +0000 (11:11 +0100)]
Remove unnecessary encoding allocations in System.Private.DataContractSerialization (#76366)

* Remove unnecessary encoding allocations in System.Private.DataContractSerialization

* Address PR feedback

21 months agoControl if invalid configuration values are swallowed or thrown (#77708)
Steve Dunn [Sun, 6 Nov 2022 23:08:52 +0000 (23:08 +0000)]
Control if invalid configuration values are swallowed or thrown (#77708)

21 months agoCorrect typo in README (#77958)
Jon Galloway [Sun, 6 Nov 2022 22:31:19 +0000 (14:31 -0800)]
Correct typo in README (#77958)

21 months agoInclude dependently promoted fields in SSA (#77238)
SingleAccretion [Sun, 6 Nov 2022 16:58:03 +0000 (19:58 +0300)]
Include dependently promoted fields in SSA (#77238)

* Remove quirks

* Early prop fix for call asgs

* Include all tracked locals in SSA

* Remove a (now incorrect) comment

* Fix a typo

* Encoding fixes

 1) The definition of SIMPLE_NUM_COUNT was wrong.
 2) SsaNumInfo::Composite, in the compact case, did not clear the old value.
 3) SsaNumInfo::Composite, in the outlined case, did not copy the already
    (compactly) encoded simple names.

* Fix store numbering

The load path needs to use the offset relative to
the store's target location.

* Clarify 'fieldValueOffset' calculations

21 months agoFix mono arm assert with large return structs (#77854)
Bill Holmes [Sun, 6 Nov 2022 16:02:30 +0000 (11:02 -0500)]
Fix mono arm assert with large return structs (#77854)

* Fix mono arm assert with large return structs

When the return struct is large the store instruction offset is too
large. When this happens, put the large offset in a register.

* Adding JIT test for large return structs

21 months agoUpdate dependencies from https://github.com/dotnet/arcade build 20221104.2 (#77940)
dotnet-maestro[bot] [Sun, 6 Nov 2022 12:38:35 +0000 (04:38 -0800)]
Update dependencies from https://github.com/dotnet/arcade build 20221104.2 (#77940)

Microsoft.DotNet.ApiCompat , Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.Build.Tasks.Archives , Microsoft.DotNet.Build.Tasks.Feed , Microsoft.DotNet.Build.Tasks.Installers , Microsoft.DotNet.Build.Tasks.Packaging , Microsoft.DotNet.Build.Tasks.TargetFramework , Microsoft.DotNet.Build.Tasks.Templating , Microsoft.DotNet.Build.Tasks.Workloads , Microsoft.DotNet.CodeAnalysis , Microsoft.DotNet.GenAPI , Microsoft.DotNet.GenFacades , Microsoft.DotNet.Helix.Sdk , Microsoft.DotNet.PackageTesting , Microsoft.DotNet.RemoteExecutor , Microsoft.DotNet.SharedFramework.Sdk , Microsoft.DotNet.VersionTools.Tasks , Microsoft.DotNet.XUnitConsoleRunner , Microsoft.DotNet.XUnitExtensions
 From Version 8.0.0-beta.22552.1 -> To Version 8.0.0-beta.22554.2

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
21 months ago[main] Update dependencies from dotnet/icu dotnet/runtime-assets dotnet/emsdk dotnet...
dotnet-maestro[bot] [Sun, 6 Nov 2022 12:37:56 +0000 (04:37 -0800)]
[main] Update dependencies from dotnet/icu dotnet/runtime-assets dotnet/emsdk dotnet/roslyn-analyzers dotnet/roslyn (#77829)

* Update dependencies from https://github.com/dotnet/runtime-assets build 20221102.2

Microsoft.DotNet.CilStrip.Sources , System.ComponentModel.TypeConverter.TestData , System.Drawing.Common.TestData , System.Formats.Tar.TestData , System.IO.Compression.TestData , System.IO.Packaging.TestData , System.Net.TestData , System.Private.Runtime.UnicodeData , System.Runtime.Numerics.TestData , System.Runtime.TimeZoneData , System.Security.Cryptography.X509Certificates.TestData , System.Text.RegularExpressions.TestData , System.Windows.Extensions.TestData
 From Version 7.0.0-beta.22531.1 -> To Version 7.0.0-beta.22552.2

* Update dependencies from https://github.com/dotnet/emsdk build 20221102.9

Microsoft.NET.Workload.Emscripten.net7.Manifest-8.0.100
 From Version 8.0.0-alpha.1.22510.1 -> To Version 8.0.0-alpha.1.22552.9

* Update dependencies from https://github.com/dotnet/roslyn-analyzers build 20221103.1

Microsoft.CodeAnalysis.Analyzers , Microsoft.CodeAnalysis.NetAnalyzers
 From Version 3.3.4-beta1.22551.1 -> To Version 3.3.4-beta1.22553.1

* Update dependencies from https://github.com/dotnet/icu build 20221103.1

Microsoft.NETCore.Runtime.ICU.Transport
 From Version 8.0.0-alpha.1.22531.1 -> To Version 8.0.0-alpha.1.22553.1

* Update dependencies from https://github.com/dotnet/runtime-assets build 20221103.1

Microsoft.DotNet.CilStrip.Sources , System.ComponentModel.TypeConverter.TestData , System.Drawing.Common.TestData , System.Formats.Tar.TestData , System.IO.Compression.TestData , System.IO.Packaging.TestData , System.Net.TestData , System.Private.Runtime.UnicodeData , System.Runtime.Numerics.TestData , System.Runtime.TimeZoneData , System.Security.Cryptography.X509Certificates.TestData , System.Text.RegularExpressions.TestData , System.Windows.Extensions.TestData
 From Version 7.0.0-beta.22531.1 -> To Version 7.0.0-beta.22553.1

* Update dependencies from https://github.com/dotnet/roslyn-analyzers build 20221103.2

Microsoft.CodeAnalysis.Analyzers , Microsoft.CodeAnalysis.NetAnalyzers
 From Version 3.3.4-beta1.22551.1 -> To Version 3.3.4-beta1.22553.2

* Update dependencies from https://github.com/dotnet/roslyn build 20221103.5

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.5.0-1.22528.14 -> To Version 4.5.0-1.22553.5

* Update dependencies from https://github.com/dotnet/emsdk build 20221104.1

Microsoft.NET.Workload.Emscripten.net7.Manifest-8.0.100
 From Version 8.0.0-alpha.1.22510.1 -> To Version 8.0.0-alpha.1.22554.1

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
21 months agoFix rerunning crossgen2 tests (#77903)
Jakob Botsch Nielsen [Sat, 5 Nov 2022 22:22:58 +0000 (23:22 +0100)]
Fix rerunning crossgen2 tests (#77903)

21 months agoOptimize XxHash3 on ARM platform (#77881)
Alexandre Mutel [Sat, 5 Nov 2022 15:05:12 +0000 (16:05 +0100)]
Optimize XxHash3 on ARM platform (#77881)

* Optimize XxHash3 on ARM platform

* Extract code to MultiplyWideningLower

* Update src/libraries/System.IO.Hashing/src/System/IO/Hashing/XxHash3.cs

Co-authored-by: Stephen Toub <stoub@microsoft.com>
Co-authored-by: Stephen Toub <stoub@microsoft.com>
21 months agoSwitch custom attribute hash from using CClosedHashEx to SHash (#77933)
Elinor Fung [Sat, 5 Nov 2022 13:28:30 +0000 (06:28 -0700)]
Switch custom attribute hash from using CClosedHashEx to SHash (#77933)

* Switch custom attribute hash from CClosedHashEx to SHash

* Remove CClosedHashEx

Co-authored-by: Jan Kotas <jkotas@microsoft.com>
21 months ago[JIT] X64 - More replacement sequences for integer multiplication by a constant ...
Will Smith [Sat, 5 Nov 2022 03:01:26 +0000 (20:01 -0700)]
[JIT] X64 - More replacement sequences for integer multiplication by a constant (#77137)

* Using 3 instruction sequence for x64 multiply

* Do not do this in morph. Do it in codegen now.

* Fixing codegen

* Only allow values under 127 and do not skip mov - correctness testing

* Try to fix tests

* cleanup

* Moving to Lowering

* Quick fix

* Fully works in lowering now

* Account for all ints

* Take into account codegen opts

* Minor cleanup

* Minor cleanup

* Fixed test

* Added int multiply disasm checks. Fixed SuperFileCheck namespace bug. Made SuperFileCheck anchors more likely to match.

* Update comments

* Update comments

* Update comments

* Update comments

* Formatting

* Fixing build

* Fixing build again

* minor rename

* Moving x64 multiply codegen optimizations to lowering

* Using ReplaceWithLclVar

* Feedback. Removed use of FULL-LINE as it is more readable not strictly necessary. Forgot to add an additional instruction to a disasm test.

* Minor fix

* Formatting

* Adding codegen opts back, but disabling lowering opts if min-opts is enabled

* Little more changes

* Formatting

* Remove codegen opts

* Update comments

* Added test case for address-exposed

* Added TryLowerLshWithConstant

* Cleanup

* Removed TryLowerLshWithConstant

* Handle GT_LSH to emit lea if const is 3

* Added tests.

* Updated comments

* Revert LowerShift

* Remove IndexWithScale

* Fix build

* Update comment

* Formatting

* Update comments. Fix disasm test.

* Another minor test change

* Fix test

* Fix test

* Feedback

* Update IntMultiply.cs

* Update codegenxarch.cpp

* Update codegenxarch.cpp

21 months agoDisable Microsoft.NETCore.Runtime.JIT.Tools for source build (#77928)
Jan Kotas [Sat, 5 Nov 2022 01:37:46 +0000 (18:37 -0700)]
Disable Microsoft.NETCore.Runtime.JIT.Tools for source build (#77928)

Fixes #77920

21 months agoAdding EVEX encoding logic for RR/AM pathways (#77419)
DeepakRajendrakumaran [Sat, 5 Nov 2022 00:34:58 +0000 (17:34 -0700)]
Adding EVEX encoding logic for RR/AM pathways (#77419)

* Cleaning up TODO-XARCH-CLEANUP on EA_8BYTE size.

* Adding EVEX encoding pathways for emitOutputRR().

* Adding EVEX encoding support for AM paths.

* Moving 'JitStressEvexEncoding' under Debug flag and other review cleanup.

21 months agoRemove nonexistent tests from PAL test run files (#77917)
Aaron Robinson [Fri, 4 Nov 2022 23:34:44 +0000 (16:34 -0700)]
Remove nonexistent tests from PAL test run files (#77917)

21 months agoWork around invalid IR with field morphing temps (#77848)
SingleAccretion [Fri, 4 Nov 2022 21:06:14 +0000 (00:06 +0300)]
Work around invalid IR with field morphing temps (#77848)

* Add a test

* Use unique temps for zero-offset field morphing

To avoid running into invalid IR issues.

21 months agoSuppress warnings for deprecated keychain APIs
Kevin Jones [Fri, 4 Nov 2022 20:53:27 +0000 (16:53 -0400)]
Suppress warnings for deprecated keychain APIs

21 months agoVectorize {Last}IndexOfAny{Except} for ASCII needles (#76740)
Miha Zupan [Fri, 4 Nov 2022 20:02:45 +0000 (13:02 -0700)]
Vectorize  {Last}IndexOfAny{Except} for ASCII needles (#76740)

* Vectorize {Last}IndexOfAny{Except} for ASCII needles

* Add a ShouldUseSimpleLoop helper

* Review feedback

21 months agoAdd FrozenDictionary/Set (#77799)
Stephen Toub [Fri, 4 Nov 2022 19:49:15 +0000 (15:49 -0400)]
Add FrozenDictionary/Set (#77799)

* Import original frozen collections source with updates to successfully compile

Co-authored-by: Martin Taillefer <mataille@microsoft.com>
Co-authored-by: Stephen Toub <stoub@microsoft.com>
* Overhaul frozen collections (API, implementation, tests)

* Address PR feedback

Co-authored-by: Martin Taillefer <mataille@microsoft.com>
21 months agoAdd System.IO.Hashing ref csproj (#77856)
Stephen Toub [Fri, 4 Nov 2022 19:48:51 +0000 (15:48 -0400)]
Add System.IO.Hashing ref csproj (#77856)

* Add System.IO.Hashing ref csproj

* Fix System.IO.Hashing ref source and add sln

21 months agoIf conversion should not convert PHI nodes (#77886) (#77888)
Alan Hayward [Fri, 4 Nov 2022 18:59:29 +0000 (18:59 +0000)]
If conversion should not convert PHI nodes (#77886) (#77888)

* If conversion should not convert PHI nodes (#77886)

* Add regression test

21 months agoImprove JIT bit sets (#77876)
Bruce Forstall [Fri, 4 Nov 2022 17:07:53 +0000 (10:07 -0700)]
Improve JIT bit sets (#77876)

1. `AllVarBitSetTraits::GetArrSize()` was allocating far too much memory:
one size_t for every bit in the vector.
2. Cache the array size for `BitVecTraits::GetArrSize()` -- this function was
showing up hot in a perf run (x86 SPMI replay).
3. Stop passing `sizeof(size_t)` to `GetArrSize()` -- we always assume bitsets
work in size_t units.

21 months agoJIT: Propagate only BBF_COPY_PROPAGATE flags during inliner substitution (#77845)
Jakob Botsch Nielsen [Fri, 4 Nov 2022 16:27:34 +0000 (17:27 +0100)]
JIT: Propagate only BBF_COPY_PROPAGATE flags during inliner substitution (#77845)

21 months agoAdding issue #77889 to GenerateRuntimeGraphTests. (#77890)
Jan Dupej [Fri, 4 Nov 2022 15:59:23 +0000 (16:59 +0100)]
Adding issue #77889 to GenerateRuntimeGraphTests. (#77890)

21 months agosource-build: set the properties to build for mono runtime. (#77786)
Tom Deseyn [Fri, 4 Nov 2022 15:33:34 +0000 (16:33 +0100)]
source-build: set the properties to build for mono runtime. (#77786)

21 months agoUpdate api-review-process.md (#77860)
Dan Moseley [Fri, 4 Nov 2022 15:25:42 +0000 (09:25 -0600)]
Update api-review-process.md (#77860)

* Update api-review-process.md

* Update docs/project/api-review-process.md

Co-authored-by: Stephen Toub <stoub@microsoft.com>
Co-authored-by: Stephen Toub <stoub@microsoft.com>
21 months agoRemove SharedFxLookup tests (#77858)
Elinor Fung [Fri, 4 Nov 2022 14:53:15 +0000 (07:53 -0700)]
Remove SharedFxLookup tests (#77858)

These were a roundabout/confusing way of testing that we roll forward
patch/minor for framework resolution by default and how we resolve
dependencies based on assembly version when the dependency is included
in multiple frameworks. We have existing tests that are clearer and
lighter weight that do this.

21 months agoRemove `palrt` from JIT and remove `memcpy` from PAL (#77556)
Aaron Robinson [Fri, 4 Nov 2022 14:22:18 +0000 (07:22 -0700)]
Remove `palrt` from JIT and remove `memcpy` from PAL (#77556)

* Remove palrt from JIT and remove memcpy from PAL

The compiles and runs without palrt.
The memcpy in the PAL was a thin wrapper and
didn't appear to add any additional value. The
PAL export was removed and the memcpy macro
was removed in most cases.

21 months agoMove LibraryImports to Common (#77674)
Robin Lindner [Fri, 4 Nov 2022 14:21:38 +0000 (15:21 +0100)]
Move LibraryImports to Common (#77674)

21 months agoAdd readme for System.Linq (#77672)
Krzysztof Wicher [Fri, 4 Nov 2022 13:47:49 +0000 (14:47 +0100)]
Add readme for System.Linq (#77672)

* Add readme for System.Linq

* Add readmes for .Parallel, .Queryable

* remove remark about not accepting refactoring changes due to new language features

* PR feedback

* static -> extension

21 months ago[wasm] WasmAppHost docs (#77265)
Marek Fišera [Fri, 4 Nov 2022 09:36:03 +0000 (10:36 +0100)]
[wasm] WasmAppHost docs (#77265)

Co-authored-by: Ankit Jain <radical@gmail.com>
21 months agoUse backport workflow from arcade repo (#77844)
Alexander Köplinger [Fri, 4 Nov 2022 09:11:54 +0000 (10:11 +0100)]
Use backport workflow from arcade repo (#77844)

* Use backport workflow from arcade repo

The arcade repo now includes the backport workflow as a reusable workflow so we can reference it from there and cleanup code duplication.

* Preserve custom PR description template

21 months agoAdd BinaryPrimitives.ReverseEndianness for spans of elements (#76339)
Stephen Toub [Fri, 4 Nov 2022 08:57:47 +0000 (04:57 -0400)]
Add BinaryPrimitives.ReverseEndianness for spans of elements (#76339)

21 months agoStandardize test script naming to CLRTestBa{tc,s}hP{re,ost}Commands (#76483)
Mark Plesko [Fri, 4 Nov 2022 08:30:33 +0000 (01:30 -0700)]
Standardize test script naming to CLRTestBa{tc,s}hP{re,ost}Commands (#76483)

Main change is BashCLRTestPreCommands -> CLRTestBashPreCommands. As part of this I noticed that wasm tests overwrite the Bash precommands, and fixing that breaks them so it isn't touched here. Also, there are several non-Commands variables that start with Ba{tc,s}hTest that aren't touched here.

Remove check on RunCrossGen2 in disasm checks as that is intended as a shell environment variable, not an msbuild variable.

Convert a few more tests to the new environment variable format.

21 months ago[wasm] browser profiler (again) (#77779)
Pavel Savara [Fri, 4 Nov 2022 07:26:56 +0000 (08:26 +0100)]
[wasm] browser profiler (again) (#77779)

* 0e24ea7 again
* fix support for shells
* report only when enabled

21 months agoDesign to support ByRefLike types in Generics (#67129)
Aaron Robinson [Fri, 4 Nov 2022 02:40:20 +0000 (19:40 -0700)]
Design to support ByRefLike types in Generics (#67129)

* Design to support ByRefLike types in Generics

* Trim whitespace

* Review feedback

* Remove words

* Update based on feedback

* Add F# as being impacted.

* Feedback and clarify wording for constrained calls.

* Add well-known IL sequences for boxing
Record APIs where compiler constraint analysis should be suppressed.
Defined analysis suppression attribute.

* Minor nits.

* Update docs/design/features/byreflike-generics.md

Co-authored-by: Jared Parsons <jaredpparsons@gmail.com>
* Review feedback

* Update to design doc

* Misspelling

* Update byreflike-generics.md

Co-authored-by: Jared Parsons <jaredpparsons@gmail.com>
21 months agoRemove "wprintf" from PAL (#77852)
Aaron Robinson [Fri, 4 Nov 2022 02:24:58 +0000 (19:24 -0700)]
Remove "wprintf" from PAL (#77852)

The last usage was in FormatMessageW, which
only used wprintf syntax for messages that used
embedded printf format between two "!". We do
not use this feature in any of our messages so
this code path is technically unused.

Remove _snwprintf_s from PAL

Remove _vsnwprintf_s from PAL

_woutput_s was not removed because it is intertwined
with the implementation for printf. The linker will just
prune the implementation anyways.

FormatMessageW errors if embedded formatting is used
Update PAL tests for FormatMessageW

21 months agoOptimize XxHash3 (#77756)
Alexandre Mutel [Thu, 3 Nov 2022 20:45:56 +0000 (21:45 +0100)]
Optimize XxHash3 (#77756)

* Optimize XxHash3

* Update src/libraries/System.IO.Hashing/src/System/IO/Hashing/XxHash3.cs

* Fix var

* Fix compilation errors for < .NET 7.0

* Fix issue on scalar code path

Co-authored-by: Stephen Toub <stoub@microsoft.com>
21 months agoFix op_Inequality for `Vector64<T>` (#77815)
David Wrighton [Thu, 3 Nov 2022 20:45:03 +0000 (13:45 -0700)]
Fix op_Inequality for `Vector64<T>` (#77815)

Recent changes to the implementation of the `Vector64<T>` type caused a regression to the fallback path for inequality. In particular it changed from inequality to checking of all elements were unequal, which is a different operation.

21 months agoUpdate superpmi-collect queues (#77825)
Jakob Botsch Nielsen [Thu, 3 Nov 2022 19:36:23 +0000 (20:36 +0100)]
Update superpmi-collect queues (#77825)

21 months agoUpdate dependencies from https://github.com/dotnet/arcade build 20221102.1 (#77828)
dotnet-maestro[bot] [Thu, 3 Nov 2022 19:05:56 +0000 (14:05 -0500)]
Update dependencies from https://github.com/dotnet/arcade build 20221102.1 (#77828)

Microsoft.DotNet.ApiCompat , Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.Build.Tasks.Archives , Microsoft.DotNet.Build.Tasks.Feed , Microsoft.DotNet.Build.Tasks.Installers , Microsoft.DotNet.Build.Tasks.Packaging , Microsoft.DotNet.Build.Tasks.TargetFramework , Microsoft.DotNet.Build.Tasks.Templating , Microsoft.DotNet.Build.Tasks.Workloads , Microsoft.DotNet.CodeAnalysis , Microsoft.DotNet.GenAPI , Microsoft.DotNet.GenFacades , Microsoft.DotNet.Helix.Sdk , Microsoft.DotNet.PackageTesting , Microsoft.DotNet.RemoteExecutor , Microsoft.DotNet.SharedFramework.Sdk , Microsoft.DotNet.VersionTools.Tasks , Microsoft.DotNet.XUnitConsoleRunner , Microsoft.DotNet.XUnitExtensions
 From Version 8.0.0-beta.22528.1 -> To Version 8.0.0-beta.22552.1

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
21 months agoUpdate dependencies from https://github.com/dotnet/linker build 20221102.1 (#77830)
dotnet-maestro[bot] [Thu, 3 Nov 2022 18:52:34 +0000 (13:52 -0500)]
Update dependencies from https://github.com/dotnet/linker build 20221102.1 (#77830)

Microsoft.NET.ILLink.Tasks
 From Version 7.0.100-1.22551.1 -> To Version 7.0.100-1.22552.1

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
21 months agoUse correct container definitions in perf_slow.yml (#77774)
Alexander Köplinger [Thu, 3 Nov 2022 18:40:34 +0000 (19:40 +0100)]
Use correct container definitions in perf_slow.yml (#77774)

Fallout from https://github.com/dotnet/runtime/pull/75473, container handling happens in platform-matrix.yml now.

21 months agoMake ResourceManager.BaseName work as documented. (#75497)
madelson [Thu, 3 Nov 2022 17:58:25 +0000 (13:58 -0400)]
Make ResourceManager.BaseName work as documented. (#75497)

* Make ResourceManager.BaseName work as documented.

This property is documented to return the "qualified namespace name and
the root resource name of a resource file". However, previously when
constructing a ResourceManager with a Type the BaseName property would
instead return the name of the type without its namespace.

fix #74918

* Remove _locationInfo field

* Add additional test cases for ResourceManager.BaseName.

Address feedback from https://github.com/dotnet/runtime/pull/75497#issuecomment-1262913704.

* Fix BaseName test case in response to https://github.com/dotnet/runtime/pull/75497#issuecomment-1295500698

21 months agoUnify RunCrossGen2 environment variable to be set to 1 instead of "true" (#77808)
Jakob Botsch Nielsen [Thu, 3 Nov 2022 17:15:40 +0000 (18:15 +0100)]
Unify RunCrossGen2 environment variable to be set to 1 instead of "true" (#77808)

Various places were setting this to "true" and various other places to
"1". Always use "1" in the environment variable and "true" in the
msbuild property. This fixes local runs of crossgen2 tests (using
src/tests/run.cmd runcrossgen2tests) -- these were not skipping
FileCheck tests before as the test wrapper script only checks for "1".

21 months agoDo not add null checks for boxed statics (#77796)
SingleAccretion [Thu, 3 Nov 2022 17:05:03 +0000 (20:05 +0300)]
Do not add null checks for boxed statics (#77796)

* Do not add some null checks

* Fix ARR_ELEM exceptions

21 months agoAvoid some unnecessary MemoryStream.ToArray calls (#77832)
Stephen Toub [Thu, 3 Nov 2022 16:49:57 +0000 (12:49 -0400)]
Avoid some unnecessary MemoryStream.ToArray calls (#77832)

21 months agoMono workloads: Fix building non-mono project when `RunAOTCompilation=true` (#77762)
Ankit Jain [Thu, 3 Nov 2022 15:41:51 +0000 (11:41 -0400)]
Mono workloads: Fix building non-mono project when `RunAOTCompilation=true` (#77762)

This manifested in a case where a non-mono project was passed a mono specific property `RunAOTCompilation=true`, but instead of it getting ignored, the build failed with:

```
/usr/local/share/dotnet/sdk/7.0.200-preview.22521.7/Sdks/Microsoft.NET.Sdk/targets/Microsoft.NET.Sdk.ImportWorkloads.targets(38,5):

error NETSDK1147: To build this project, the following workloads must be installed: macos [/private/tmp/c/c.csproj]
error NETSDK1147: To install these workloads, run the following command: dotnet workload restore [/private/tmp/c/c.csproj]
```

And this is because the `MonoAOTCompiler.Task` is imported with:

https://github.com/dotnet/runtime/blob/82aa87f9cdf9ac20f0f685016648c36247a76ff1/src/mono/nuget/Microsoft.NET.Workload.Mono.Toolchain.Manifest/WorkloadManifest.targets.in#L57-L57

- This happens for net6, and net8 projects.

Fixes https://github.com/dotnet/runtime/issues/77707 .

cc @steveisok @lewing

21 months ago[wasm] Bump tests timeout 15mins to 30mins, for .. (#77819)
Ankit Jain [Thu, 3 Nov 2022 15:40:57 +0000 (11:40 -0400)]
[wasm] Bump tests timeout 15mins to 30mins, for .. (#77819)

.. `xharness`.

Some library tests, like `System.Text.RegularExpressions.Tests` can take more than 15mins.

- Also, fix building `console-v8` sample with `make`

21 months agoUse more spans in `System.Reflection.Metadata` et. al. (#76574)
Theodore Tsirpanis [Thu, 3 Nov 2022 15:29:21 +0000 (17:29 +0200)]
Use more spans in `System.Reflection.Metadata` et. al. (#76574)

* Use Unsafe.As in ImmutableByteArrayInterop.

Code quality increases.

* Optimize methods in the BlobUtilities class.

* Include more information in exceptions thrown from catch blocks.

* Use spans in BlobContentId, avoiding ImmutableByteArrayInterop.

* Use spans in BlobWriter, reducing pinning and ImmutableArrayInterop.

* Use spans in BlobBuilder, reducing pinning and ImmutableArrayInterop.

* Remove NoInlining from the throw helpers.

It prevents the JIT from looking into the method's body, realizing it's a throw helper, and doing what's best (such as considering it cold).

* Inline `BlobUtilities.Read(Immutable)?Bytes`.

* Avoid a length check introduced by the use of `BinaryPrimitives`.

* Remove `DisallowNull` from `ImmutableArrayInterop.DangerousCreateFromUnderlyingArray`.

* Remove an early bail-out if the buffer was empty.

* Stop using value tuples.

* Fix tests and expose writing a span to BlobWriter.

* Fix stack overflows.

* Shorten BlobUtilities methods that write integers.

* Optimize some methods in `MemoryBlock`.

21 months ago#76881 internal properties in GetRuntimeProperties are not shown (#77169)
Meri Khamoyan [Thu, 3 Nov 2022 15:03:31 +0000 (19:03 +0400)]
#76881 internal properties in GetRuntimeProperties are not shown (#77169)

Fixes https://github.com/dotnet/runtime/issues/76881

21 months agoFixes typos and formatting in comments (#76975)
Michael Aranda [Thu, 3 Nov 2022 14:20:52 +0000 (08:20 -0600)]
Fixes typos and formatting in comments (#76975)

21 months agoImplement Environment.IsPrivilegedProcess (#77355)
Ilya [Thu, 3 Nov 2022 14:13:51 +0000 (19:13 +0500)]
Implement Environment.IsPrivilegedProcess (#77355)

* Implement Environment.IsPrivilegedProcess

* Address feedback

* Use one static

* Fix mono wasm

* Address feedback 2

* Address feedback 3

* Fix typo.

* Fix wasm

21 months agocorrect code snippets (#77581)
RaymondY [Thu, 3 Nov 2022 14:13:15 +0000 (22:13 +0800)]
correct code snippets (#77581)

Let them be valid C# code. Uniform code style as well.

21 months agoFix ThunkGenerator build break (#77816)
Jan Kotas [Thu, 3 Nov 2022 14:04:02 +0000 (07:04 -0700)]
Fix ThunkGenerator build break (#77816)

Apply all style auto-fixers on ThunkGenerator

21 months agoFix wrong metadata name in test (#77806)
Jakob Botsch Nielsen [Thu, 3 Nov 2022 10:37:34 +0000 (11:37 +0100)]
Fix wrong metadata name in test (#77806)

21 months agoUpdate tar test data version (#77785)
Adeel Mujahid [Thu, 3 Nov 2022 09:01:15 +0000 (11:01 +0200)]
Update tar test data version (#77785)

21 months agoJIT: Enable phi-based jump threading when SSA updates aren't needed (#77748)
Andy Ayers [Thu, 3 Nov 2022 04:18:41 +0000 (21:18 -0700)]
JIT: Enable phi-based jump threading when SSA updates aren't needed (#77748)

Leverage the new SSA accounting to look for cases of phi-based jump threading
that will not require SSA updates. In particular cases where the phis are all
locally consumed.

Also update documentation on the SSA checker implementation (which aims to
ensure that the SSA accounting we're relying on here is accurate).

21 months agoRemove empty BEGIN/END_ENTRYPOINT_* macros (#77810)
Elinor Fung [Thu, 3 Nov 2022 03:26:36 +0000 (20:26 -0700)]
Remove empty BEGIN/END_ENTRYPOINT_* macros (#77810)

21 months ago[mono] Avoid returning true from RuntimeType.IsActualEnum for generic parameters...
Zoltan Varga [Thu, 3 Nov 2022 00:48:15 +0000 (20:48 -0400)]
[mono] Avoid returning true from RuntimeType.IsActualEnum for generic parameters. (#77763)

Related: https://github.com/dotnet/runtime/pull/71685.

21 months agoSystem.Threading.RateLimiting de-queue canceled request (#77182)
Ali Khalili [Wed, 2 Nov 2022 21:17:08 +0000 (21:17 +0000)]
System.Threading.RateLimiting de-queue canceled request (#77182)

* Dequeue canceled request before making decision for processing queue’s requests

21 months agoUpdate assembly version from hardcoded to MajorVersion (#74157)
Carlos Sanchez [Wed, 2 Nov 2022 20:28:48 +0000 (13:28 -0700)]
Update assembly version from hardcoded to MajorVersion (#74157)

* Update assembly version from hardcoded to MajorVersion

* Bump version to 8 in XsdDataContractExporterTests

* Retrieve major version dynamically for expected message in TypesTest_Negative

* Retrieve major version dynamically for expected message in SurrogateProvider_MemberData

* Manually generate the TestData.resources.
The dotnet exec command generated by the msbuild Target had to be manually executed in the cmdline with the target framework folders for RemoteExecutor hardcoded to net7.0-windows so the resources file could be generated.

* Avoid hardcoded 8 in additional SurrogateTests line

* Fix Diagnostics.EventLog supression message

* Update assembly version from hardcoded to MajorVersion

* Bump version to 8 in XsdDataContractExporterTests

* Retrieve major version dynamically for expected message in TypesTest_Negative

* Retrieve major version dynamically for expected message in SurrogateProvider_MemberData

* Manually generate the TestData.resources.
The dotnet exec command generated by the msbuild Target had to be manually executed in the cmdline with the target framework folders for RemoteExecutor hardcoded to net7.0-windows so the resources file could be generated.

* Avoid hardcoded 8 in additional SurrogateTests line

* Fix Diagnostics.EventLog supression message

* Extract the LocalEchoServer bits from the projects

.. so that the helix bits are in a LocalEchoServer.helix.targets ,
instead of being special-cased in sendtohelix-wasm.targets .
- And this is setup and used by the test projects by importing one file.

* Build the LocalEchoServers specifically without using artifacts .

- The RemoteLoopServer, and NetCoreServer are projects used as aspnetcore
middleware, and loaded by xharness.
- These are built against the live artifacts, same as other projects.
    - But this can be a problem when the libraries in `runtime` are on a
    newer assembly version (say `8.0.0`), but xharness is still built with
    `7.0.0` libraries.
    - In that case, xharness fails to load the middleware:

```
Application startup exception: System.Reflection.ReflectionTypeLoadException: Unable to load one or more of the requested types.

...

Could not load file or assembly 'System.Runtime, Version=8.0.0.0, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'. The system cannot find the file specified.

   at System.Reflection.RuntimeModule.GetTypes(RuntimeModule module)
   at System.Reflection.Assembly.GetTypes()
   at Microsoft.DotNet.XHarness.CLI.CommandArguments.TypeFromAssemblyArgument`1.GetLoadedTypes()+MoveNext() in /_/src/Microsoft.DotNet.XHarness.CLI/CommandArguments/Arguments/TypeFromAssemblyArgument.cs:line 29
   at Microsoft.DotNet.XHarness.CLI.Commands.WebServer.<>c__DisplayClass0_0.<Start>b__9(TestWebServerOptions options) in /_/src/Microsoft.DotNet.XHarness.CLI/Commands/WebServer.cs:line 60
   at Microsoft.Extensions.Options.OptionsFactory`1.Create(String name)
   at Microsoft.Extensions.Options.OptionsCache`1.<>c__3`1.<GetOrAdd>b__3_0(String name, ValueTuple`2 arg)
   at System.Collections.Concurrent.ConcurrentDictionary`2.GetOrAdd[TArg](TKey key, Func`3 valueFactory, TArg factoryArgument)
   at Microsoft.Extensions.Options.OptionsCache`1.GetOrAdd[TArg](String name, Func`3 createOptions, TArg factoryArgument)
   at Microsoft.DotNet.XHarness.CLI.Commands.WebServer.TestWebServerStartup.Configure(IApplicationBuilder app, IOptionsMonitor`1 optionsAccessor) in /_/src/Microsoft.DotNet.XHarness.CLI/Commands/WebServer.cs:line 126
   at System.RuntimeMethodHandle.InvokeMethod(Object target, Void** arguments, Signature sig, Boolean isConstructor)
   at System.Reflection.MethodInvoker.Invoke(Object obj, IntPtr* args, BindingFlags invokeAttr)
   at System.Reflection.RuntimeMethodInfo.Invoke(Object obj, BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture)
   at Microsoft.AspNetCore.Hosting.ConfigureBuilder.Invoke(Object instance, IApplicationBuilder builder)
   at Microsoft.AspNetCore.Hosting.ConventionBasedStartup.Configure(IApplicationBuilder app)
   at Microsoft.AspNetCore.Hosting.WebHost.BuildApplication()
```

- Build the project in isolation from rest of the repo, so that it is
built with references only from the SDK. The built assembly can then be
deployed for use with xharness, just like before.

* add missing files

* [wasm] fixup extensions targets for LocalEchoServer

* [wasm] Use eng/targetingpacks.targets instead of wasm specific targets

* [wasm] Avoid duplicate imports, and don't use the targetingpacks.targets

.. for runtime tests since the paths are set explicitly in the project

* [wasm] Workaround for https://github.com/dotnet/runtime/issues/77707

The middleware projects are now being built separately, so they can
target a different TFM. But when `RunAOTCompilation=true` is set, like
on CI/AOT builds, the build breaks with:

```
To build this project, the following workloads must be installed: macos [/__w/1/s/Build.proj]
To install these workloads, run the following command: dotnet workload restore [/__w/1/s/Build.proj]
```

This is described in https://github.com/dotnet/runtime/issues/77707 .
And the issue is present in `7.0 rc1` also, which is currently used for
building the repo. To workaround it, we unset some properties which
aren't needed anyway for the middleware projects builds.

Co-authored-by: carlossanlop <carlossanlop@users.noreply.github.com>
Co-authored-by: Ankit Jain <radical@gmail.com>
21 months agoFix missing dispose on rate limiting (#76784)
Ali Khalili [Wed, 2 Nov 2022 20:26:19 +0000 (20:26 +0000)]
Fix missing dispose on rate limiting (#76784)

21 months agoAvoid locking during host trace flush when not necessary (#77765)
Elinor Fung [Wed, 2 Nov 2022 20:09:05 +0000 (13:09 -0700)]
Avoid locking during host trace flush when not necessary (#77765)

21 months ago[main] Update dependencies from 7 repositories (#77594)
dotnet-maestro[bot] [Wed, 2 Nov 2022 17:57:48 +0000 (17:57 +0000)]
[main] Update dependencies from 7 repositories (#77594)

[main] Update dependencies from 7 repositories

21 months agoFix triple slash documentation for System.Runtime.Intrinsics (#76687)
Drew Kersnar [Wed, 2 Nov 2022 17:02:30 +0000 (12:02 -0500)]
Fix triple slash documentation for System.Runtime.Intrinsics (#76687)

* Add missing type param documentation for Vector64.Create<T>

* Add summaries to the Vector types

* Fix typeparamref

* Address PR feedback

21 months ago[main] Update dependencies from dotnet/linker (#77595)
dotnet-maestro[bot] [Wed, 2 Nov 2022 16:56:10 +0000 (12:56 -0400)]
[main] Update dependencies from dotnet/linker (#77595)

* Update dependencies from https://github.com/dotnet/linker build 20221027.4

Microsoft.NET.ILLink.Tasks
 From Version 7.0.100-1.22525.3 -> To Version 7.0.100-1.22527.4

* Update dependencies from https://github.com/dotnet/linker build 20221031.1

Microsoft.NET.ILLink.Tasks
 From Version 7.0.100-1.22525.3 -> To Version 7.0.100-1.22531.1

* Update dependencies from https://github.com/dotnet/linker build 20221031.2

Microsoft.NET.ILLink.Tasks
 From Version 7.0.100-1.22525.3 -> To Version 7.0.100-1.22531.2

* Update dependencies from https://github.com/dotnet/linker build 20221101.1

Microsoft.NET.ILLink.Tasks
 From Version 7.0.100-1.22525.3 -> To Version 7.0.100-1.22551.1

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
21 months agoUse SR.Format in a few more places (#77615)
Adeel Mujahid [Wed, 2 Nov 2022 16:55:56 +0000 (18:55 +0200)]
Use SR.Format in a few more places (#77615)

21 months agoAvoid reporting empty debug info ranges to the vm (#77289)
Brian Bohe [Wed, 2 Nov 2022 16:41:08 +0000 (09:41 -0700)]
Avoid reporting empty debug info ranges to the vm (#77289)

* Removing empty variable live ranges

The debugger is not using empty variable live ranges.
We are reporting them because they can get extended
later if the variable becomes alive in the immediately
next emitted instruction. If an empty live range is
not getting extended, which we can realize after
emitting all the code or creating a new live range
for the same variable, we can remove it.

* Extending variable live ranges in more cases

When the emitter moved to the next group but has not
emitted any instruction, and the variable died and
becomes alive again, we would like to extend its range.

* Avoiding creating a new debug range when previous is empty

* Updating check for empty debug ranges

* Updating print

* Avoiding printing twice variable live range

* Avoiding reporting empty variable ranges to the vm

* Revert "Avoiding printing twice variable live range"

This reverts commit 4e1cf47dd6cdf9d45ce4a51eaa05b3ec6e4b3b41.

* Revert "Updating print"

This reverts commit 7b79b0d955daa4dc9770b604d6780591b51c9ee1.

* Revert "Updating check for empty debug ranges"

This reverts commit e8b102d489d79028750068decf4f427ad8e5f69f.

* Revert "Avoiding creating a new debug range when previous is empty"

This reverts commit a11fd5d0ffaa98631d731bec1f2619f1d108d33a.

* Revert "Extending variable live ranges in more cases"

This reverts commit 609605a1ca7cf1c0c843dbaf353432ec9ed846e2.

* Revert "Removing empty variable live ranges"

This reverts commit 66d18e031f83c8efe864981b35c0548af49e0714.

* Freeing vm memory when there is no debug info

* Persisting JIT-EE contract on empty debug info

* Update src/coreclr/jit/ee_il_dll.cpp

Co-authored-by: Jakob Botsch Nielsen <Jakob.botsch.nielsen@gmail.com>
Co-authored-by: Brian Bohe <brianbohe@microsoft.com>
Co-authored-by: Jakob Botsch Nielsen <Jakob.botsch.nielsen@gmail.com>
21 months agoMark and expose additional Vector functions as Intrinsic (#77562)
Tanner Gooding [Wed, 2 Nov 2022 16:25:40 +0000 (09:25 -0700)]
Mark and expose additional Vector functions as Intrinsic (#77562)

* Mark and expose additional Vector functions as Intrinsic

* Applying formatting patch

* Ensure Vector64.CreateScalarUnsafe for long/ulong/double is handled

* Ensure we use the right Count

* Ensure small types don't sign extend up to int

* Ensure reflection invocation is correct for Vector<T> bitwise ops

* Adding tests and fixing a couple small typing issues

* Fixing the name of the One property

* Assert GT_RSH for simd isn't TYP_LONG on xarch

* Ensure the correct operation is picked for vector ShiftRightArithmetic

* Fixing StoreAlignedNonTemporal on Arm64

21 months ago[hot_reload] implement param reflection (#77563)
Aleksey Kliger (λgeek) [Wed, 2 Nov 2022 16:21:30 +0000 (12:21 -0400)]
[hot_reload] implement param reflection (#77563)

* Add new test ReflectionAddNewMethod

* FIXME: get_param_names, get_marshal_info and custom_attrs need work

* WIP - add a method param reverse lookup

* look up params from added methods

* Remove FIXMEs and unused field

* remove writelines from test

* fix test on coreclr

* why does coreclr have 2 attributes here??

* There should be 2 attributes on the 4th param

* one more place that looks at params

* A couple more places where we look at the Params table

* Check default values on params on added method

* fix lookup if table is empty

* add a gratuitious typeof assert

otherwise the CancellationToken type is trimmed on wasm

* Add a single mono_metadata_get_method_params function

remove duplicated code

21 months agoJIT: Support profiles with only histograms (#77721)
Jakob Botsch Nielsen [Wed, 2 Nov 2022 09:35:40 +0000 (10:35 +0100)]
JIT: Support profiles with only histograms (#77721)

Split up "we have a profile" and "we have profile weights" questions.

21 months ago[mono] Remove Vector128 checks for amd64 APIs (#77534)
Matous Kozak [Wed, 2 Nov 2022 09:24:23 +0000 (10:24 +0100)]
[mono] Remove Vector128 checks for amd64 APIs (#77534)

* Remove Vector128 checks for amd64 APIs

* refactor emit_sum_vector (amd64)

21 months agoAdd RIDs for Tizen 7.0 (#76600)
Jay Cho [Wed, 2 Nov 2022 09:00:19 +0000 (18:00 +0900)]
Add RIDs for Tizen 7.0 (#76600)

21 months ago[mono][wasm] Handle delegates decorated with [UnmanagedFunctionPointe… (#77709)
Zoltan Varga [Wed, 2 Nov 2022 04:14:20 +0000 (00:14 -0400)]
[mono][wasm] Handle delegates decorated with [UnmanagedFunctionPointe… (#77709)

* [mono][wasm] Handle delegates decorated with [UnmanagedFunctionPointer] in
the interp-to-native generator.

Fixes https://github.com/dotnet/runtime/issues/76930.

* Update src/tasks/WasmAppBuilder/PInvokeTableGenerator.cs

Co-authored-by: Ankit Jain <radical@gmail.com>
* Update src/tasks/WasmAppBuilder/PInvokeTableGenerator.cs

Co-authored-by: Ankit Jain <radical@gmail.com>
* Update src/tasks/WasmAppBuilder/PInvokeTableGenerator.cs

Co-authored-by: Ankit Jain <radical@gmail.com>
Co-authored-by: Ankit Jain <radical@gmail.com>
21 months agoAdd option for truncated stream detection (#75671)
mfkl [Wed, 2 Nov 2022 04:13:38 +0000 (11:13 +0700)]
Add option for truncated stream detection (#75671)

* Add option for truncated stream detection

fix https://github.com/dotnet/runtime/issues/47563

* Use RemoteExecutor

move the test to concrete classes as abstracted classes are not supported by RemoteExecutor

* review feedback

* use same error text message

* cache appcontext getswitch

* fix failing test

* slice byte array for assertion

* renaming

* add missing RemoteExecutor.IsSupported

* fast check first

21 months agoRevert "[wasm] browser profiler (#77449)" (#77759)
Ankit Jain [Wed, 2 Nov 2022 02:01:45 +0000 (22:01 -0400)]
Revert "[wasm] browser profiler (#77449)" (#77759)

This reverts commit 0e24ea7c2a0436a8f2bf83e8f5981ec035518b99.

It broke all the perf pipeline wasm builds. It happens with v8 `10.9.130`:

```
Error: Invalid 'startMark' argument: No numeric 'startTime' field
    at pe (/home/helixbot/work/B2E7090F/w/A520093B/e/performance/artifacts/bin/for-running/MicroBenchmarks/659adab0-5c86-4e74-8913-672bd56fd58e/bin/net7.0/browser-wasm/AppBundle/dotnet.js:3:12994)
    at /home/helixbot/work/B2E7090F/w/A520093B/e/performance/artifacts/bin/for-running/MicroBenchmarks/659adab0-5c86-4e74-8913-672bd56fd58e/bin/net7.0/browser-wasm/AppBundle/dotnet.js:5:95874
exiting due to exception: Error: Invalid 'startMark' argument: No numeric 'startTime' field
```

- It works fine with older versions like `10.7.193` (current stable).
- It wasn't caught in the CI checks because those are using an older version of v8.

21 months agoJIT: increase edge weight slop tolerance some more (#77747)
Andy Ayers [Wed, 2 Nov 2022 00:31:27 +0000 (17:31 -0700)]
JIT: increase edge weight slop tolerance some more (#77747)

Allow up to 1/50 variance between the expected and computed values.

Fixes #77450.

21 months agoFix Activity.OperationName when Activity is created with null value (#77746)
Tarek Mahmoud Sayed [Tue, 1 Nov 2022 22:53:48 +0000 (15:53 -0700)]
Fix Activity.OperationName when Activity is created with null value (#77746)

21 months agoFix TimeZoneInfo test failure (#77745)
Tarek Mahmoud Sayed [Tue, 1 Nov 2022 22:50:01 +0000 (15:50 -0700)]
Fix TimeZoneInfo test failure (#77745)

21 months agoMigrate docker containers to latest tags (#76630)
Michelle McDaniel [Tue, 1 Nov 2022 21:19:49 +0000 (14:19 -0700)]
Migrate docker containers to latest tags (#76630)

* Move docker tags to -latest

As part of https://github.com/dotnet/arcade/issues/10123, we have added a floating -latest tag to all currently in-support docker container images. This change moves all container reference to the -latest version so runtime can get all of the latest updates to the containers.

* Change all of the centos-8-rpmpkg images to centos-7-rpmpkg

CentOS 8 was EOL and has been removed as a supported docker image in dotnet-buildtools-prereqs-docker

* Replace -latest tags with new tag schema

* Move tests off eol docker containers

* Update the images in the new infra

21 months agoUpdate the Nuget Config used for Maui net6 apps. (#77739)
Parker Bibus [Tue, 1 Nov 2022 20:00:32 +0000 (13:00 -0700)]
Update the Nuget Config used for Maui net6 apps. (#77739)

21 months agoAdd readmes for configuration packages, part 1 (#77733)
MSDN.WhiteKnight [Tue, 1 Nov 2022 19:29:00 +0000 (00:29 +0500)]
Add readmes for configuration packages, part 1 (#77733)

21 months agoFixing VarLocInfo on JITDUMP (#77700)
Brian Bohe [Tue, 1 Nov 2022 18:43:57 +0000 (11:43 -0700)]
Fixing VarLocInfo on JITDUMP (#77700)

Co-authored-by: Brian Bohe <brianbohe@microsoft.com>
21 months agoUpdate mask for lookup of `this` in CCW `ComWrappers` scenario (#77302)
Aaron Robinson [Tue, 1 Nov 2022 18:41:39 +0000 (11:41 -0700)]
Update mask for lookup of `this` in CCW `ComWrappers` scenario (#77302)

* Expand ComWrappers mask for vtable lookup on 64-bit

21 months ago[wasm] browser profiler (#77449)
Pavel Savara [Tue, 1 Nov 2022 18:20:59 +0000 (19:20 +0100)]
[wasm] browser profiler (#77449)

* wip

* wip

* cleanup

* feedback

* feedback

21 months agoUpdate SuperPMI artifact logging (#77685)
Aaron Robinson [Tue, 1 Nov 2022 18:07:42 +0000 (11:07 -0700)]
Update SuperPMI artifact logging (#77685)

* Update SuperPMI artifact logging

21 months ago[wasm] Fix analyzer support in templates (#77704)
Ankit Jain [Tue, 1 Nov 2022 17:48:32 +0000 (13:48 -0400)]
[wasm] Fix analyzer support in templates (#77704)

- Add
  `[assembly:System.Runtime.Versioning.SupportedOSPlatform("browser")]`
  to the browser, and console templates. This would allow the analyzers,
  if enabled, to treat the assembly as one that will run only on
  browser.

- Populate `@(SupportedPlatform)` with only `browser`, for *wasm*
  projects, similar to https://github.com/dotnet/sdk/blob/fef8cedfb6b4ac85a7e135f3e4f155e29cdcbdf1/src/BlazorWasmSdk/Targets/Microsoft.NET.Sdk.BlazorWebAssembly.5_0.targets#L75-L79

- This also fixes the firefox CI build which regressed recently

21 months agoRun dotnet-format for ThunkGenerator (#77715)
Egor Bogatov [Tue, 1 Nov 2022 16:32:17 +0000 (17:32 +0100)]
Run dotnet-format for ThunkGenerator (#77715)

21 months agoImprove WoA workflow documentation (#77691)
Kevin Jones [Tue, 1 Nov 2022 16:17:48 +0000 (12:17 -0400)]
Improve WoA workflow documentation (#77691)

* Recommend using Visual Studio's ninja

21 months agoUpdate some documentation (#77711)
Jakob Botsch Nielsen [Tue, 1 Nov 2022 16:15:55 +0000 (17:15 +0100)]
Update some documentation (#77711)

* Update references to Ngen variants of JIT environment variables
* Update crossgen to crossgen2
* SPC instead of mscorlib

21 months agoUpdate dependencies from https://github.com/dotnet/arcade build 20221028.1 (#77713)
dotnet-maestro[bot] [Tue, 1 Nov 2022 16:04:58 +0000 (09:04 -0700)]
Update dependencies from https://github.com/dotnet/arcade build 20221028.1 (#77713)

Microsoft.DotNet.ApiCompat , Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.Build.Tasks.Archives , Microsoft.DotNet.Build.Tasks.Feed , Microsoft.DotNet.Build.Tasks.Installers , Microsoft.DotNet.Build.Tasks.Packaging , Microsoft.DotNet.Build.Tasks.TargetFramework , Microsoft.DotNet.Build.Tasks.Templating , Microsoft.DotNet.Build.Tasks.Workloads , Microsoft.DotNet.CodeAnalysis , Microsoft.DotNet.GenAPI , Microsoft.DotNet.GenFacades , Microsoft.DotNet.Helix.Sdk , Microsoft.DotNet.PackageTesting , Microsoft.DotNet.RemoteExecutor , Microsoft.DotNet.SharedFramework.Sdk , Microsoft.DotNet.VersionTools.Tasks , Microsoft.DotNet.XUnitConsoleRunner , Microsoft.DotNet.XUnitExtensions
 From Version 8.0.0-beta.22524.5 -> To Version 8.0.0-beta.22528.1

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
21 months agoFirst pass at adding a test to reproduce the problem. (#77693)
Steve Dunn [Tue, 1 Nov 2022 14:55:36 +0000 (14:55 +0000)]
First pass at adding a test to reproduce the problem. (#77693)

21 months agoAvoid running R2RDump if crossgen2 failed (#77705)
Andrew Au [Tue, 1 Nov 2022 14:50:31 +0000 (07:50 -0700)]
Avoid running R2RDump if crossgen2 failed (#77705)

21 months agoUpdate a few dependencies (#77678)
Jakob Botsch Nielsen [Tue, 1 Nov 2022 13:15:18 +0000 (14:15 +0100)]
Update a few dependencies (#77678)

Fix some component governance warnings.

21 months agoArm64: Add If conversion pass (#73472)
Alan Hayward [Tue, 1 Nov 2022 12:40:41 +0000 (12:40 +0000)]
Arm64: Add If conversion pass (#73472)

* Arm64: Add If conversion pass

* Minor review fixups

* Return a PhaseStatus

* Fix formatting

* Check for side effects on NOPs

* Add function block comments for the phase

* Remove creation of AND chains from if conversion pass

* Update middleBlock flow

* Check for order side effects

* Remove COLON_COND check

* Remove flag toggling

* Move the conditional assignment to the JTRUE block

* Fix formatting

* Allow conditions with side effects

* Fix formatting

* Correct all moved SSA statements

* Add size costing check

* Only move middle block ssa defs

* Fix formatting

* Fewer SSA assumptions

* Use implicit func for value numbering

* Update header for gtFoldExprConditional

* Cost based on speed

* Add Stress mode for inner loops

* Rework costings

* Check for invalid VNs

* Ignore compares against zero

* Ensure float compares are contained

* Allow if conversion of test compares

* Do not contain test compares within compare chains

* Add float versions of the JIT/opt/Compares tests

* Fix formatting

* Compare chains use CmpCompares, selects use Compares

* Fix flow checking for empty blocks

* Fix to contexts setting JitStdOutFile

* Fix attr and reg producing in select generation

21 months agoDelete a bit of dead code (#77655)
SingleAccretion [Tue, 1 Nov 2022 10:15:31 +0000 (13:15 +0300)]
Delete a bit of dead code (#77655)

Mostly I was after the FP morphing, but removed some other things along the way as well.

No diffs except a nice TP boost as expected.

21 months agoFold bound checks for static readonly arrays/strings (#77593)
Egor Bogatov [Tue, 1 Nov 2022 08:41:52 +0000 (09:41 +0100)]
Fold bound checks for static readonly arrays/strings (#77593)

Co-authored-by: Jan Kotas <jkotas@microsoft.com>
Co-authored-by: SingleAccretion <62474226+SingleAccretion@users.noreply.github.com>
21 months agoSwitch CoreCLR WeakReference to unified managed implementation (#77196)
Vladimir Sadov [Tue, 1 Nov 2022 06:58:35 +0000 (23:58 -0700)]
Switch CoreCLR WeakReference to unified managed implementation (#77196)

* Delete managed CoreClr WR

* all but com works

* COM tests are passing

* HandleTagBits const for NativeAot

* Exclusive Set

* fix

* Use sign bit

* Platforms not supporting COM can mask only one bit.

* new approach

* fix mono build

* check for FEATURE_COMWRAPPERS too

* stub NativeAOT support (NYI).

* current

* moved handle tags on the managed side to one location

* Getter optimizations

* Optimizations for Setter

* accessibility of some members

* ensure identity of the rehydrated RCW

* make ComWeakRefToObject a QCall

* delete unused pWeakReferenceOfTCanonMT and pWeakReferenceMT

* byte-aligned

* cleanup unreachable code

* renamed WeakReferenceObject::m_Handle -> WeakReferenceObject::m_taggedHandle

* Apply suggestions from code review

Co-authored-by: Aaron Robinson <arobins@microsoft.com>
* some PR feedback

* GetWeakHandle no longer cares about inlining.

* turn ObjectToComWeakRef into a QCall

* revert changes under coreclr\gc

* added a note to eventually remove HNDTYPE_WEAK_NATIVE_COM

* Update src/coreclr/gc/gcinterface.h

Co-authored-by: Maoni Stephens <Maoni0@users.noreply.github.com>
Co-authored-by: Aaron Robinson <arobins@microsoft.com>
Co-authored-by: Maoni Stephens <Maoni0@users.noreply.github.com>