Merge pull request #2393 from EgorBo/fix-mono-sample

Add RunSample target to mono.proj

Optimize `box + isinst + unbox.any` as `nop` (#1817)

Extend box pattern match to cover `box + isinst + unbox.any` .

Bump version to 6.13 (#31715)

Co-authored-by: Jo Shields <directhex@apebox.org>

Libraries code should not mutate string instances (#31700)

Update src/mono/mono.proj

Co-Authored-By: Alexander Köplinger <alex.koeplinger@outlook.com>

Update src/mono/mono.proj

Co-Authored-By: Alexander Köplinger <alex.koeplinger@outlook.com>

Remove AppDomain usage from stack walk and related places (#31708)

This is yet another cleanup removal of the AppDomain.

Address feedback

[interp] Remove goto for code_type unknown | interp | compiled. (#31667)

[interp] Remove goto for code_type unknown | interp | compiled.

The goto kinda implies a loop, but it is really just handling one case
before the other, but the source misorders the checks, perhaps for perf,
which maybe, can be done equivalently with G_UNLIKELY, and also remove
some possible extra memory reads.

Co-authored-by: Jay Krell <jay.krell@cornell.edu>

Implement Socket.DuplicateAndClose() for Windows (#1858)

* reimplement socket duplication, fix #1760

* additional cleanup

* ungroup PNSE tests, no RemoteExecutor timeout

* address review findings

* fix path for Interop.WSADuplicateSocket.cs

* remove swallowing in Socket(SocketInformation)

* review suggestions

* make sure duplicate socket is not inheritable

* remove debug code leftover

* harden SocketPal.CreateSocket()

* blittable WSAPROTOCOL_INFOW

* additional naming fixes,

default System.Net.Sockets.sln configurations to Windows_NT again

* disposal on errors, improve comments, more tests

* nits

* handle GetSockName() error

Update ILLinkTasksVersion dependency (#2334)

Clean up

When the interpreter does recurse, use a different function (#31635)

for each call site, so you can tell from a WebAssembly stack
which case it is. i.e. with function names but no line numbers.

With this, we can see that call_varargs contributes to the problem
here: https://github.com/mono/mono/issues/18646

Also, do not inline the interpreter, it probably/hopefully never was.

Co-authored-by: Jay Krell <jay.krell@cornell.edu>

Don't require Checked config for coreclr tests

[runtime] Fix the --enable-minimal=aot build. (#31684)

Fixes https://github.com/mono/mono/issues/18675.

Remove remaining libraries references to string.Copy (#31706)

Revert "Revert "Optimize typeof(T1).IsAssignableFrom(typeof(T2))" (#31643)" (#31705)

This reverts commit 729276d30fbafdc28cd9813f74c0bb49702d775c.

Merge pull request #31699 from AndyAyersMS/Fix31665

JIT: remove assert from postorder return value optimizations

Remove unused locals in System.ServiceProcess.ServiceController (#31704)

* Remove unused locals in System.ServiceProcess.ServiceController

* Inline status variable

* Apply suggestions from code review

Co-Authored-By: Stephen Toub <stoub@microsoft.com>
Co-authored-by: Stephen Toub <stoub@microsoft.com>

[interp]Fix the type of clause_args. (#31651)

Co-authored-by: Jay Krell <jay.krell@cornell.edu>

Fix logic in compareTypesForCast (#31681)

We weren't careful enough with `__Canon` in some cases, which lead to unsafely
returning `MustNot` when the cast outcome was not determined at jit time.

Add an extra check, update comments, and add some test cases.

Addresses the failures seen in #1195 (which was reverted).

Fix issue 1871 exception test failing (#31696)

Don't adjust the offset by 1 instead of the architecture instruction size (STACKWALK_CONTROLPC_ADJUST_OFFSET).

Allow null IV for Create{Encryptor,Decryptor}.

The IV is permitted to be null for most ciphers when in ECB
mode.

Add missing nullability annotation to string ctor (#31702)

set THUMB_CODE bit (#21180)

[aot] Avoid inflating gparams with byreflike types during generic sharing. (#31670)

Fixes https://github.com/mono/mono/issues/18676.

[Arm64] Update execution characteristics for crc32b, crc32h, facge, facgt (#8500)

* Add execution characteristics for Crc32 instructions in emitarm64.cpp

* Update execution characteristics for DV_3B instructions in emitarm64.cpp

* Add execution characteristics for DV_3D instructions in emitarm64.cpp

* Update comment in emitIns_R_R_R in emitarm64.cpp

Save stack arguments size in InlinedCallFrame.m_Datum on x86 to handle callee's popping of arguments (#26834)

* Save stack arguments size in InlinedCallFrame.m_Datum on x86 to handle callee's popping of arguments

* Move stack arguments sizes logic to Windows-only

Fix package testing run name prefix (#31690)

JIT: remove assert from postorder return value optimizations

We were trying to assert that we weren't losing side effects when folding
return value trees. But this seems to be more trouble than it is worth as
the set of side effects can also change.

Also re-enable the disabled tests, and make one of them Pri0.

Fixes #31665.

Disable two JIT tests failing with known issue tracked under #31665 (#31674)

* Disable two JIT tests failing with known issue tracked under #31665

* Address Andy's PR feedback - make the exclusion clauses global

Delete unnecessary netstandard builds (#31641)

* Delete unnecessary netstandard builds and NuGet packages of inbox components

- Delete netstandard builds and NuGet packages for System.Reflection.Emit.*
- Delete netstandard builds and NuGet packages for System.Reflection.TypeExtensions
- Delete netstandard build for System.Buffers\ref
- Delete clrcompression UAP  build support

Contributes to #2138

* Remove packages from the baseline

Merge pull request #31694 from DrewScoggins/anscoggi/FixArch

* Update Arch and add RunKind variable

* Fix up Linux script as well

Fix up Linux script as well

Add Single and Double overloads to BinaryPrimitives (#6864)

* Add Single and Double overloads to BinaryPrimitives

Fix #2365

Remove popped trees more aggressively. (#31677)

When processing CEE_POP the importer sometimes created trees
like
ASG(tmp1, expr)
followed by
COMMA(ADDR(tmp1), NOP))
and that was causing some CQ issues.

LocalAddressVisitor was then marking tmp1 as address-exposed
and we never get rid of the dead ASG(tmp1, expr). What's worse, we
then were adding a zero-initialization for tmp1 in the prolog.

The fix here is to create a NOP instead of a COMMA is there are
no side-effects in the first operand of the COMMA.

This addresses the first example in #2325.

Update Arch and add RunKind variable

Merge branch 'master' of https://github.com/dotnet/runtime

Initial Crossgen2 fixes for composite R2R support (#31652)

With initial local changes to enable composite R2R I have finally
been able to compile a CoreCLR unit test in this mode. It doesn't
yet work at runtime, these changes just fix initial compiler issues
I hit during the build:

1) Delegate constructor signature was missing some comparison logic
that ended up asserting in the node comparer in the composite
build.

2) I have removed two seeminly superfluous parameters to
CreateMethodEntrypointNodeHelper.

3) I have removed an assertion related to multi-file RVA fields
that I believe to be no longer relevant after the refactorings
to copy the RVA fields around properly.

Thanks

Tomas

Link System.Globalization.Native into coreclr (#6928)

* Link System.Globalization.Native into coreclr

* Add more paths to coreclr subset filters

* Fix linux build

* PR Feedback

* PR Feedback 2

* Fix build (pass -DREPO_ROOT_DIR only on coreclr atm), System.IO.Ports.Native needs to be built first

* Remove version header file from static libs

* Remove CORECLR_BUILD variable definition

handle GetSockName() error

[mini] Add method-triggered printing for runtime stats (#2355)

This work extends the `--stats` command to take in an optional method name. When a matching method is compiled or transformed, the stats will be printed out at that point in addition to after program execution. This is useful for products looking to measure startup time and activity, as well as for ourselves to track any loader performance regressions.

Once this lands, we should see about running it on a cadence, ideally on both desktop and Android. Additionally, more work needs to be done to the counters to expose them in a way more friendly to consumption on Android/wasm (including a callback). We should also update the website to document this additional behavior.

Co-authored-by: Ryan Lucia <ryan@luciaonline.net>
Co-authored-by: Kyle Markland <v-kymark@microsoft.com>
Co-authored-by: Rene Garcia <v-regarc@microsoft.com>

Libraries native renames (#31668)

Match coreclr & cross compilation naming conventions

Rename BIT64                -> TARGET_64BIT   in src/libraries/Native
Rename BIT32                -> TARGET_32BIT   in src/libraries/Native
Rename FEATURE_PAL          -> TARGET_UNIX    in src/libraries/Native
Rename PLATFORM_UNIX        -> TARGET_UNIX    in src/libraries/Native
Rename PLATFORM_WINDOWS     -> TARGET_WINDOWS in src/libraries/Native
Rename PLATFORM_OSX         -> TARGET_OSX     in src/libraries/Native
Rename _ARM_                -> TARGET_ARM     in src/libraries/Native
Rename _ARM64_              -> TARGET_ARM64   in src/libraries/Native
Rename _AMD64_              -> TARGET_AMD64   in src/libraries/Native
Rename _X86_                -> TARGET_X86     in src/libraries/Native
Rename ARM                  -> TARGET_ARM     in src/libraries/Native
Rename ARM64                -> TARGET_ARM64   in src/libraries/Native
Rename AMD64                -> TARGET_AMD64   in src/libraries/Native
Rename X86_                 -> TARGET_X86     in src/libraries/Native

nits

Catch ObjectDisposedException in SmtpClient.Abort (#22867)

* Catch ObjectDisposedException in SmtpClient.Abort

* Address PR feedback

* Apply suggestions from code review

Co-Authored-By: Cory Nelson <phrosty@gmail.com>
Co-authored-by: Cory Nelson <phrosty@gmail.com>

Clean up

Fix x86 InlinedCallFrame popping during EH (#2215)

* Fix x86 InlinedCallFrame popping during EH

There is a bug in InlinedCallFrame popping on x86 during EH with R2R
compiled code. The code is popping this frame if it is at the top of
the explicit frames stack before executing catch handler. But it
is missing a check to see if the frame was actually in the unwound part
of the stack. So in some corner cases, it can end up popping a frame
that belongs to an active pinvoke somewhere up the call chain.

The fix is to do the popping only if the InlinedCallFrame is located at
an address smaller than the ESP of the caller of the method that
contains the exception handler.

* Fix typos in a comment close to my change
* Remove workaround for the original issue

Annotate System.ComponentModel.Composition for nullable (#1842)

* Annotating System.ComponentModel.Composition for nullable

Clean up

Make fixup decoding lazy (#31646)

Update mono.proj

Fix RunCoreClrTest on *nix

Nullable annotate System.Security.Principal.Windows (#2347)

* annotate System.Security.Principal.Windows

* enable nullable for ref assembly

* Update src/libraries/System.Security.Principal.Windows/src/System/Security/Principal/SID.cs

Co-Authored-By: Stephen Toub <stoub@microsoft.com>
* Update src/libraries/System.Security.Principal.Windows/src/System/Security/Principal/SID.cs

Co-Authored-By: Stephen Toub <stoub@microsoft.com>
* address feedback

* address feedback

* remove dead code

* remove ComVisibleAttribute

* Make domainSid optional in Sid constructor

* add omitted nullable annotation

* address feedback

* work around annotation issue

* try fixing build

* Update src/libraries/System.Security.Principal.Windows/src/System/Security/Principal/WindowsIdentity.cs

Co-Authored-By: Stephen Toub <stoub@microsoft.com>
* Update src/libraries/System.Security.Principal.Windows/src/System/Security/Principal/WindowsIdentity.cs

Co-Authored-By: Stephen Toub <stoub@microsoft.com>
* address feedback

Co-authored-by: Stephen Toub <stoub@microsoft.com>

Add missing method dependency to Mono SPC (#2338)

* Add missing method dependency to Mono SPC

Fixes #2227

* Reenable ILLinkTrimAssembly

* Fix entries in linker xml that used wrong names

* Add more entries to linker xml from CoreCLR

Co-authored-by: Alexander Köplinger <alex.koeplinger@outlook.com>

disposal on errors, improve comments, more tests

Add run-sample-coreclr rule

Improve RunCoreClrTests target

[build] Check for Python 3 in configure.ac (#2372)

* [build] Check for Python 3 in configure.ac

I fear this causing packaging problems, but as-is we require Python 3 for most of our Python files but don't actually check for that in configure. We should be consistent about this. Some projects (notably wasm) also use the Python version detected here, so bumping this will also bump the version used in those cases (a good thing, since there are some known bugs when using the scripts with Python 2).

* Update docker images to reflect py3 inclusion

* Update centos-7 image

Co-authored-by: Ryan Lucia <ryan@luciaonline.net>
Co-authored-by: Alexander Köplinger <alex.koeplinger@outlook.com>

Address feedback

fix build

Address feedback

[interp] Fix enum constrained calls on netcore (#31618)

We were failing to dereference a managed pointer to the enum, due to incorrect checks, when doing a constrained call.

Fixes https://github.com/mono/mono/issues/18530

Add RunCoreClrTests target

Fix ppc64 build by fixing -mminimal-toc more correctly. (#31636)

The original fix back at mono/mono#18554 did not work quite right, which was missed for lack of CI.
Reworked this check to instead test for `-mminimal-toc` on all powerpc hosts, and to only do the ELF version check on FreeBSD.

Fixes mono/mono#18554 mono/mono#18578 current CI build failures on AIX

Co-authored-by: Phil Jaenke <prj@rootwyrm.com>

Add RunCoreClrTest target

Revert "Optimize typeof(T1).IsAssignableFrom(typeof(T2))" (#31643)

Split binder tracing tests into separate test projects (#3855)

* Split binder tracing tests into separate test projects

* Stop using specific time when waiting for events

* Don't run tests in a separate process under GC stress

Skip EqualityComparer<TKey>.Default creation in Dictionary .ctor (#31629)

* Skip EqualityComparer<TKey>.Default creation in Ditctionary .ctor

* Update src/libraries/System.Private.CoreLib/src/System/Collections/Generic/Dictionary.cs

Co-Authored-By: Stephen Toub <stoub@microsoft.com>

Fix build

Fix build

Merge branch 'master' of github.com:dotnet/runtime into fix-mono-sample

Clean up

Move more things from Makefile to mono.proj

Eliminated info creation for non-public properties (#2278)

Optimize typeof(T1).IsAssignableFrom(typeof(T2)) (#1195)

* Optimize Type.IsAssignableFrom

* Add TypeIntrinsics.IsAssignableFrom to TypeIntrinsics.csproj

* Formatting

* Add License header

* Remove assert

* fix "mustexpand assert"

* Add tests for Arrays of vt

Merge branch 'master' of github.com:dotnet/runtime into fix-mono-sample

Adjust mono_class_init_internal to no longer bypass cleanup when a fa… (#21964)

…ilure is detected.

This fixes: https://issuetracker.unity3d.com/issues/unity-physics-collisions-do-not-work-and-errors-are-thrown-when-entering-play-mode

<!--
Thank you for your Pull Request!

If you are new to contributing to Mono, please try to do your best at conforming to our coding guidelines http://www.mono-project.com/community/contributing/coding-guidelines/ but don't worry if you get something wrong. One of the project members will help you to get things landed.

Does your pull request fix any of the existing issues? Please use the following format: Fixes #issue-number
-->

Co-authored-by: Alex Thibodeau <alexthibodeau@unity3d.com>

Allow type mismatch for struct single field (#6102)

On Arm64, we can have an HVA or HFA with a single scalar or SIMD field. When it is the destination of an assigment to a call result, the assignment may be re-typed to the non-struct type. It may be ideal in the long run to defer this re-typing until Lowering, but allow this case for now.

Fix #2182

Loosen assert

[llvm] Enable jump threading with LLVM JIT. (#25687)

See https://github.com/mono/mono/issues/16243. This will allow elision of redundant bounds checks. This pass is already used during AOT (as part of opt -O2).

Here's `FirstOrDefault` (from mono/mono#16243) with this change:
```
0000000000000000 <gram_FirstOrDefault__int___>:
<BB>:1
   0: 31 c0                 xor    %eax,%eax
   2: 48 85 ff              test   %rdi,%rdi
   5: 74 09                 je     10 <gram_FirstOrDefault__int___+0x10>
   7: 83 7f 18 01           cmpl   $0x1,0x18(%rdi)
   b: 7e 03                 jle    10 <gram_FirstOrDefault__int___+0x10>
   d: 8b 47 24              mov    0x24(%rdi),%eax
  10: c3                    retq
***
```

Co-authored-by: imhameed <imhameed@microsoft.com>

Highlight directory (#31625)

JIT: fold trees after inline return expression updates (#1751)

Aggressively fold as we substitute inline return value trees in for the return
value placeholders. Notice when this folding leads to branch simplification,
and make the associated flow graph update.

The recently added early flow opt pass will then transitively remove any
newly unreachable code.

Resolves dotnet/coreclr#27395.

Add tests for Arrays of vt

Merge branch 'master' of github.com:dotnet/runtime into type-isassignedfrom

Revert "Improve call counting mechanism (#1457)" (#30105)

* Revert "Revert "Disable test based on #129 (#130)" (#2310)"

This reverts commit 427cd918a027aed167b1c4bedb52c7ca50731aba.

* Revert "Fail FuncEval if slot backpatching lock is held by any thread (#2380)"

This reverts commit fc06054a774e28a5a47bbe862adcf03251abb43c.

* Revert "Improve call counting mechanism (#1457)"

This reverts commit 3a457cb4b552d9b32fbf844389ad2a08bcb2a7a6.

Merge branch 'master' of github.com:dotnet/runtime into type-isassignedfrom

# Conflicts:
# src/coreclr/tests/src/JIT/Intrinsics/TypeIntrinsics.cs
# src/libraries/System.Private.CoreLib/src/System/Type.Helpers.cs

Emit DebugDirectory section and all DebugDirectory entries (#2398)

* Emit DebugDirectory section and all DebugDirectory entries

- Add new RELOC type for raw data points
- Emit native DebugDirectory entry with proper MD5 hash of the output image
- Fix determinism issue: timestamp has to be copied from input IL image

Cleanup NETCOREAPP ifdef usage in System.Runtime.Tests (#22224)

Fixes #21827

Disable macOS 10.15 from fullMatrix runs

This OS was just enabled a couple of days ago, and isn't testing so hot.

Turning it off until it can come back on in a passing state.

Fix UpperVector save asserts (#2268)

Not all helper calls have standard kill sets.
Fix #2181

 Rename cross compilation related defines (#2256)

* Rename identifiers to be more consistent

This patch was a machine generated search and replace of
identifiers.  The replacement was done in the order listed
for the path specified.

A regex was used to find the identifiers.

s/(^|[^A-CE-Za-z0-9_])${find}($|[^A-Za-z0-9_])

The 'D' character was specifically allowed a s a prefix
to catch add_definitions(-D${find}).

The patch also reverts all managed code changes after replacement.

    _ARM_                -> HOST_ARM       in src/coreclr
    _ARM64_              -> HOST_ARM64     in src/coreclr
    _AMD64_              -> HOST_AMD64     in src/coreclr
    _X86_                -> HOST_X86       in src/coreclr
    _HOST_UNIX_          -> HOST_UNIX      in src/coreclr
    _HOST_AMD64_         -> HOST_AMD64     in src/coreclr
    _HOST_ARM64_         -> HOST_ARM64     in src/coreclr
    _HOST_ARM_           -> HOST_ARM       in src/coreclr
    _HOST_X86_           -> HOST_X86       in src/coreclr
    DBG_TARGET_AMD64     -> TARGET_AMD64   in src/coreclr
    DBG_TARGET_ARM64     -> TARGET_ARM64   in src/coreclr
    DBG_TARGET_ARM       -> TARGET_ARM     in src/coreclr
    DBG_TARGET_X86       -> TARGET_X86     in src/coreclr
    _TARGET_AMD64_       -> TARGET_AMD64   in src/coreclr
    _TARGET_ARM_         -> TARGET_ARM     in src/coreclr
    _TARGET_ARM64_       -> TARGET_ARM64   in src/coreclr
    _TARGET_X86_         -> TARGET_X86     in src/coreclr
    _TARGET_ARMARCH_     -> TARGET_ARMARCH in src/coreclr
    _TARGET_XARCH_       -> TARGET_XARCH   in src/coreclr
    _HOST_64BIT_         -> HOST_64BIT     in src/coreclr
    BIT64                -> HOST_64BIT     in src/coreclr
    _TARGET_64BIT_       -> TARGET_64BIT   in src/coreclr
    DBG_TARGET_64BIT     -> TARGET_64BIT   in src/coreclr
    HOST_IS_WINDOWS_OS   -> HOST_WINDOWS   in src/coreclr
    PLATFORM_UNIX        -> HOST_UNIX      in src/coreclr/*.cmake
    PLATFORM_WINDOWS     -> HOST_WINDOWS   in src/coreclr/*.cmake
    _TARGET_MAC64        -> TARGET_OSX     in src/coreclr
    FEATURE_PAL          -> TARGET_UNIX    in src/coreclr
    _TARGET_UNIX_        -> TARGET_UNIX    in src/coreclr
    _TARGET_WINDOWS_     -> TARGET_WINDOWS in src/coreclr
    PLATFORM_UNIX        -> TARGET_UNIX    in src/coreclr
    PLATFORM_WINDOWS     -> TARGET_WINDOWS in src/coreclr

* Miscellaneous manual fixes to fix compilation

Remove unused defines

Remove BIT32
Remove DBG_TARGET_AMD64_UNIX
Remove DBG_TARGET_ARM64_UNIX
Remove DBG_TARGET_ARM_UNIX
Remove DBG_TARGET_32BIT

Fixes for HOST_<arch> rename

Move TARGET_<Arch> and TARGET_<bit>

Move from clrdefinitions.cmake
to configurecompiler.cmake so it is used globally.

More jit.h

* Whitespace by clang-format

crossgen2 ARM support misc (#21109)

Fix handling of missing ConfigurationGroup property (#24077)

* Account for missing ConfigurationGroup

Our logic for setting the CoreCLR and Mono runtime configurations didn't
properly account for cases where only `$(Configuration)` was passed down
as a build property. It expects that both `$(Configuration)` and
`$(ConfigurationGroup)` are passed down as a pair.

This change guards against us accidentially setting the runtime
configuration to the current library definition of configuration which
is more like a target framework than Debug or Release.

Related to #23800

* Test for missing ConfigurationGroup

Abort cmov optimization, when one of the out blocks is try block (#2373)

Fixes mono/mono#17733

Co-authored-by: Fan Yang <52458914+fanyang-mono@users.noreply.github.com>

Add profiler APIs for writing EventPipe events (#2173)

* Profiler APIs for writing events

* Code review feedback

* Change EVENTPIPE_EVENTID to EVENTPIPE_EVENT to avoid confusion with EventID

* have the runtime pack the arguments instead of the profiler

* Add more checks in asserts

New pgo format for crossgen2 (#2314)

* Adjust compilation model to support trace based profile guided optimization data in a new format
- Add parser for new format
- Adjust program startup and options parsing to make it simpler for the profiler guided logic to be aware of version bubbles
- Use CompilationModuleGroup and version bubble in parsing and handling of profile data

* Handle malformed and incorrect IBC data

Referred GC Benchmarking Infra in the GC Performance testing docs (#5532)

import thunks for ARM just from zapper (#21072)

Key should be non null (#5072)

Add runtime flavor to test run name for libraries outerloop runs (#12715)

* Add runtime flavor to test run name for libraries outerloop runs

* Bring Windows_x64 back

move more rules from mono/mono