Simplify IsWow64Process interop (#39059)

Fixes #38910

Put back missing EditorBrowsableState.Nevers on PlatformID (#39065)

Delete NetEventSource.Enter/Exit events (#38874)

They're super verbose, very inconsistent, costly, and not helpful (they appear to be trying to provide tracing of the full flow of calls, but they fail to do even that).

[wasm] Enable System.Threading.Channels.Tests (#38849)

* [wasm] Enable System.Threading.Channels.Tests and skip hangs

* [wasm] Skip failing tests in System.Threading.Channels.Tests

Co-authored-by: Mitchell Hwang <mitchell.hwang@microsoft.com>

[interp] Add debug option which reports the hottest methods (#39045)

* [interp] Reenable computed goto on desktop

Which I accidentaly removed recently.

* [interp] Add debug option which reports the hottest methods

[wasm] Implement pinvoke callback support. (#38932)

* [wasm] Implement pinvoke callback support.

* Allow void return types.

* Fix name mangling for assembly names.

* Handle MonoPInvokeCallbackAttribute as well for now.

[wasm] Modify System.Net.NetworkInformation to throw PNSE (#38928)

Add OSPlatform entries for iOS/tvOS/watchOS/Android (#36704)

* Add OSPlatform entries for iOS/tvOS/watchOS/Android

Implements the non-controversial new OSPlatform members from https://github.com/dotnet/runtime/issues/33331

* Add tests for new OSPlatform entries

[browser][wasm][tests] Deactivate Map tests due to CI failures (#39061)

Remove cross targeting for TraceSource and TextWriterTraceListener (#39043)

* stop cross targeting

Allow XmlSerializer.Generator to run on Core > 2.x (#39013)

* Allow XmlSerializer.Generator to run on Core > 2.x

* Use repo local dotnet toolchain to build Generator

Use the repo local dotnet toolchain to generate the sgen output.
Adding a test to verify that the repo local and the live built outputs
are identical by size.

WASM enable System.Collections.NonGeneric.Tests (#39020)

Define Debugger.IsSupported feature switch to allow trimming of debugger only code. (#37288)

Implementations of new HttpContent sync methods. (#38635)

addressing some missing feedback Relating to Browser (#38968)

* addressing some missing feedback

* Apply suggestions from code review

Co-authored-by: Santiago Fernandez Madero <safern@microsoft.com>
* remove odbc32 from ref

* fix stuff

Co-authored-by: Santiago Fernandez Madero <safern@microsoft.com>

WASM Re-enable Threading.Channels serialization tests (#39016)

* WASM Re-enable Threading.Channels serialization tests

* PR Feedback

* Use ConditionalFact

Delete tools-local\scripts (#39051)

Revert changes to test breaking outter loop build (#39046)

Include member name when constructing ValidationResult in CompareAttribute (#38867)

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

Fix mark overflow processing (#38969)

A recent refactoring PR #1688 has regressed GC processing of mark overflow.
If GC heap segments in the segment linked list are not ordered in an ascending
order by their addresses, the mark overflow processing misses segments on
lower addresses if they follow segments on higher addresses.
This leads to some objects that are alive to not to be reported and freed.

This change fixes the problem by making sure the segment order doesn't matter.

Cleanup references to centos 6 (#39006)

* Remove Dockerfile

* Remove references to dockerfiles in runtime. These exist in dotnet-buildtools-prereqs.

Move tailcall dispatcher into corelib (#38938)

Fix #35559

[mono] Add support for UnmanagedCallersOnlyAttribute (#38728)

* [marshal] Add mono_method_has_unmanaged_callers_only_attribute

* [marshal] Allow calls to mono_marshal_get_managed_wrapped without a delegate class

   In that case, create a wrapper based on the signature of the method itself.

* [aot] Allow decode_method_ref to decode NATIVE_TO_MANAGED wrappers without a delegate class

   Bump the AOT file format

* [interp] ldftn will return a native-to-managed wrapper to UnmanagedCallersOnly methods

* aot: emit byte when we don't expect a class

* jit: create wrapper creation for ldftn in method-to-ir

   Do it at IR generation of the caller, not every time the ldftn is executed

* jit: don't create a jump trampoline for ldftn of a native-to-managed wrapper

   The wrapper might be called from a thread that's not attached to the runtime, and the jump trampoline will look at TLS vars that are not initialized

* interp: transform LDFTN into LDC of a create_method_pointer for UnmanagedCallersOnly method

* marshal: throw invalid program exception for instance and generic methods

* Emit IPE throw instead of aborting JIT or interp compilation for bad UnmanagedCallersOnly methods

   Instead of throwing while JITing (or transforming), throw when the LDFTN is executed.

* disallow delegate constructor calls on UnmanagedCallersOnly methods

* throw IPE if UnmanagedCallersOnly method has non-blittable args

* disallow DllImport and UnmanagedCallersOnly together

   throw NotSupportedException

Co-authored-by: Ryan Lucia <ryan@luciaonline.net>

WASM enable System.Collections.Specialized.Tests (#39012)

[wasm] Add beginnings of support for building with ICU. (#38977)

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

[wasm] Modify System.Drawing.Common to throw PNSE on Browser (#38999)

Co-authored-by: Santiago Fernandez Madero <safern@microsoft.com>

Add a repro test for `IsCompatibleType(cseLclVarTyp, expTyp)` (#38972)

* Add a repro test for #35724

* move it to pri1.

[browser][wasm][tests] Add tests for Map (#38862)

* [browser][wasm][map] Add tests for Map

* Address review comments

Add and use Environment.ProcessId (#38908)

* Add and use Environment.ProcessId

* Split newly added tests

Fix GC heap dump (#38893)

#36932 had a typo that caused types to never be logged

fix the condition in assert (#38965)

Make DSAKeyValueTests.LoadXml accept seed/counter may not roundtrip. (#38952)

Disable HandlerInvokedForSigQuit test on macOS (#39003)

WASM Enable System.Collections tests (#38976)

Update CadesIssuerSerial.xml (#38436)

remove extra white space

More comparisons (#37768)

HttpStress timeout increased to 60 seconds (#38669)

Silence ARM inline asm warning in libunwind (#38971)

The libunwind headers define unw_getcontext in the libunwind headers for ARM
as inline assembly with stmia instruction storing SP and PC, which clang
complains about as deprecated. However, it is required for atomic restoration
of the context, so disable that warning.

This change silences that warning.

Fix standalone GC on ARM32 (#38970)

* Fix standalone GC on ARM32

The standalone build of GC on ARM32 was missing the FEATURE_64BIT_ALIGNMENT,
which caused aligment failures at runtime for objects that require align
on 8 byte boundary and were not getting it.

This change fixes that. I had to modify the copy of MethodTable that the
standalone GC uses so that the RequireAlign8 flag can be checked.

* Move the FEATURE_64BIT_ALIGNMENT setting to gcenv.object.h

* Fix debug build where an assert required MethodTable::IsValueType

Remove local copying of emsdk_env.sh (#39004)

Recent versions of emsdk stopped supporting the construct_env argument: https://github.com/emscripten-core/emsdk/commit/819e95cd995147d8b16c600d64e16c880cc407df
Sourcing the normal isn't that slow anyway, it takes about 300ms on my machine

WASM: don't compile brotli sources into System.IO.Compression.Native (#38990)

Brotli isn't supported so we don't need it.

[wasm] Enable System.Threading.Tasks.Extensions tests (#38815)

* [wasm] Enable System.Threading.Tasks.Extensions tests

* Remove synchronous blocking from System.Threading.Tasks.Extensions tests

* [wasm] ManualResetValueTaskSourceTests remove unnecessary and add necessary conditionalfacts

* [wasm] ValueTaskTests clean up ConditionalFacts

* [wasm] Amend ManualResetValueTaskSourceTests tests asserting distinct threads

* [wasm] Skip hanging System.Threading.Tasks.Extensions tests with ActiveIssue

Co-authored-by: Mitchell Hwang <mitchell.hwang@microsoft.com>
Co-authored-by: Stephen Toub <stoub@microsoft.com>

Improve performance of DateTime.Kind property. (#38501)

* Improve performance of DateTime.Kind.

* Remove the unused field.

* Fixing whitespaces.

Co-authored-by: Ben Adams <thundercat@illyriad.co.uk>
* Revert the changes and add AgressiveInlining

* Put the attribute on the getter.

* Update src/libraries/System.Private.CoreLib/src/System/DateTime.cs

Co-authored-by: Ben Adams <thundercat@illyriad.co.uk>
Co-authored-by: Stephen Toub <stoub@microsoft.com>

Various createdump fixes and improvements to help diagnosing problems (#38940)

Better MacOS createdump thread suspend

Remove extra logging

Add trace_printf and flush diag logging

Add process id/name spew

Add stdout flushing to CrashInfo::Trace

Fix issue https://github.com/dotnet/diagnostics/issues/1305

Trim NetEventSource when EventSource.IsSupported is false (#38828)

* Trim NetEventSource when EventSource.IsSupported is false

Follow up to #38129. NetEventSource code was still left in even when EventSource.IsSupported is false, since all the usages of NetEventSource are keying off its own static property: NetEventSource.IsEnabled.

Remove NetEventSource.IsEnabled so the linker can trim NetEventSource code when EventSource.IsSupported is false.

* Remove HttpTelemetry.IsEnabled static property so the linker can trim EventSource code when EventSource.IsSupported is false.

* Fix fake NetEventSource to match what is in the product.

Set memory cache logger field as readonly (#37289)

[Wasm] Skip System.IO.Compression.Brotli tests b/c the library is PNSE (#38941)

Use mono_runtime_delegate_invoke in EventPipeInternal::CreateProvider. (#38989)

Due to issues on some platforms related to mono_delegate_to_ftnptr_impl,
switch to use mono_runtime_delegate_invoke without need for delegate
wrapper generation currently not supported on all platforms.

[CBOR] Make Half Read/Write methods public and add unit tests (#38946)

* Make Cbor Half methods public and add unit tests

* address feedback

Improve terminology used in code (#38960)

* Improve terminology used in code

* Address PR feedback

* Update src/libraries/System.Private.CoreLib/src/System/Resources/RuntimeResourceSet.cs

Co-authored-by: Michal Strehovský <MichalStrehovsky@users.noreply.github.com>
Co-authored-by: Michal Strehovský <MichalStrehovsky@users.noreply.github.com>

extend existing test that validates the sent and received bytes with a usage of sync method that uses BufferPtrSendOperation internally (#38935)

[master] Update dependencies from dotnet/arcade mono/linker Microsoft/vstest dotnet/llvm-project dotnet/xharness (#38880)

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
Co-authored-by: Alexander Köplinger <alex.koeplinger@outlook.com>

Add feature switch for diagnostics handler in System.Net.Http (#38765)

Fix a few more license headers (#38973)

* Fix a few more license headers

I missed a few directories outside of src.

* Update Directory.Build.props

Delete unreachable code in stackwalker (#38978)

Standardize on same illink descriptor name (#38985)

Update windows-instructions.md

Delete WebHeaderCollection's event source (#38871)

The logging is verbose, arbitrary, expensive, and duplicative of logging that happens when headers are actually read/written by the underlying networking stack.  Delete.

Add browser tfm to test projects that used to run when it derived from Unix (#38954)

* Add browser tfm to test projects that used to run when it derived from Unix

* Don't build System.Net.Mail.Unit.Tests as they are already PNSE

WASM: Enable System.Runtime.Serialization.Formatters tests (#38948)

* WASM: Enable System.Runtime.Serialization.Formatters tests

We need to preserve some assemblies the BinaryFormatter needs and also skip some test cases that would cause PNSE on WebAssembly.

* Disable IsolatedStorage case too

It throws PNSE now.

Build refs only for applicable BuildTFMs (#38936)

* Build refs only for applicable BuildTFMs

Remove global properties for CoreLib P2P (#38937)

* Remove global properties for CoreLib P2P

[wasm] Add pinvokes for System.IO.Compression.dll. (#38925)

remove pragma warning from ref source files (#38907)

[metadata] Move rarely used bits out of MonoMethod and give 1 bit back to MonoMethod:slot (#38735)

The `MonoMethod:slot` width was accidentally changed in https://github.com/dotnet/runtime/commit/8465be73eb40a9c409dc8974b9d6b235a1b5158f

To compensate, we move 3 bits out to image properties
   * `verification_success` - rarely used nowadays, protected by the image lock (like all image properties)
   * `is_reabstracted`, `is_covariant_override_impl` - infrequently used features of newer C# language features (default interface methods and covariant returns).  Setting these assumes the loader lock is held.

Fix remaining license headers (#38953)

* Fix license in remainder of .cs files

* Fix license header in .vb files

* Fix license header in .py files

* Fix license headers in .c, .cpp, .h

* Fix licensing on .dat files

* Fix license header in generators

* Fix license headers in .asm

* Fix license headers in .xml

* Fix remainder of license headers

* Add license default to .editorconfig

* Add some missing headers

* Fix Doundation to Foundation in some license headers

* Add missing license headers under libraries

Fix TOC in CI Health and investigation (#38914)

Fix case where EventPipeInternal::Enable is called with NULL output file. (#38927)

Remove commented libraries windows helix queues (#38947)

* Remove commented libraries windows helix queues

* Remove accidental empty line addition.

Delete dead code related to finalization (#38906)

Add tizen-arm64 RIDs (#38832)

* Add tizen-arm64 RIDs

Since tizen 6.0.0, arm64 will be supported

* Add arm64 arch to all tizen versions

Run CLRTools unittests in PR jobs (#38694)

-Add new job to existing runtime pipeline to run the unit tests that are part of the clr.tools subset
-Avoid publishing anything other than test results from that job
-Run only on Linux-X64 as that's the platform with the most spare availability. These tests do not need to be run on a broad swath of platforms, as they are unittests of pure C#

Fix calling a ThisCall delegate via Reverse P/Invoke. (#38829)

Delete ClientWebSocket's event source (#38870)

It's raising a single event for failed connects, and actual networking-related failures for failed connects will already be logged by the underlying API used.

By getting rid of the catch and rethrow, we can also avoid an extra layer of async state machine on connect.

Throw with ParamName=value instead of Encoding (#38820)

* Throw with ParamName=value instead of Encoding

* Update WebClientTest.cs

SocketHttpHandler: include host+IP information in HttpRequestException (#38131)

Fix #1326 by appending host:port info to HttpRequestException's message when connection fails.

Didn't change the inner SocketException, since it would require subclassing SocketException, which would add unnecessary complexity here.

ignore loopback NoDelay failure on OSX (#38900)

* ignore loopback NoDelay failure on OSX

* remove extra file

Fix live libraries build config for the runincontext pipeline (#38926)

[Wasm] Modify System.IO.IsolatedStorage to throw PNSE (#38898)

Console.Unix tests: don't run tests when SIGINT/SIGQUIT disposition is ignored (#38334)

* Console.Unix tests: don't run tests when SIGINT/SIGQUIT disposition is ignored

* Add SkipOnMono to HandlerInvokedForSigQuit test

Fix inconsistent copyright for C# files in Mono's corelib (#38664)

* Fix inconsistent copyright for C# files in Mono's corelib

* Update src/mono/netcore/System.Private.CoreLib/src/Mono/RuntimeMarshal.cs

Co-authored-by: Stephen Toub <stoub@microsoft.com>
* Use 2-line version of copyright

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

[wasm][debugger] DebugDirectoryEntryType = Reproducible means that has debug information (#38869)

Considering DebugDirectoryEntryType = Reproducible as an assembly that has debug information.

Fixes mono/mono#20075

Co-authored-by: thaystg <thaystg@users.noreply.github.com>

Remove unnecessary codes (#38918)

[wasm] Implement support for pinvoke callbacks in the interpeter. (#38839)

Co-authored-by: vargaz <vargaz@users.noreply.github.com>

Update license headers (#38793)

Add iOS/tvOS/Android to AdditionalBuildTargetFrameworks (#38922)

Saw these missing while reviewing https://github.com/dotnet/runtime/pull/37944.
As far as I can see this is only used when building inside VS but still good to add for consistency.

Avoid unnecessary CultureInfo.CurrentCulture accesses in Enum (#38866)

* Avoid unnecessary CultureInfo.CurrentCulture accesses in Enum

Enum can only be backed by primitive numerical types, and using the IConvertible interface implementations to convert to numerical types won't pay any attention to culture, so just as there's no need to pass through the supplied provider, there's no need to access CultureInfo.CurrentCulture.

* Address PR feedback

Unignore some Makefiles in src/mono (#38923)

The global .gitignore ignores `Makefile` since it's typically just a build artifact of CMake/automake.
However we have a few real Makefiles in src/mono that were excluded by this, resulting in VSCode's search not looking in them.

[interp] Remove usage of some gotos (#38813)

* [interp] Remove usage of some gotos

Their usage resulted in worse code on wasm.

* [interp] Rename method

Seems like something is failing in eglib-remap test

Make browser peer for windows_nt and unix (#37944)

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

July infra rollout: Move runtime tests out of the coreclr folder (#38058)

* Mechanical move of tests from src/coreclr/tests/src to src/tests

* Minimum changes to make CoreCLR tests build in the new location

(*) Path changes in the test build scripts;
(*) Modify runtime.yml filtering based on Nathan's and Santi's feedback;
(*) Fix runtime pipeline filtering clauses per Santi's PR feedback;
(*) Fix path to Coreclr.TestWrapper.csproj;
(*) Pass unprocessed build args to test wrapper creation;
(*) Fix missing $(TestRoot) on groups in Pri0 test build mode.

Thanks

Tomas

Reenable fixed tests after noRetyping and similar changes. (#38855)

* Reenable GitHub_26491.

Closes #13355

* Reenable crossgen2 tests failing with old retyping/

They were fixed both with and without retyping.
Closes #37883.

* Reenable HVA merge cases.

Closes #37341, closes #37880.

* Reenable GitHub_35821.

Closes #36206, closes #36418.

The issue was fixed by #37499.

* Delete extra lines that are no longer needed.

#37506 was fixed in #38241.

* delete a throwing init.

Prevent shutdown being blocked by finalization (#38909)

The finalization queue can be long or constantly growing when the finalization thread is not able to
keep up with finalizable object allocation rate. This can lead to shutdown being blocked for a long
time or indefinitely.

The fix is stop the finalization loop once we enter shutdown instead of trying to empty the finalization
queue.

Fixes #314

Use the latest base register while accessing the stack (#38834)

* logging

* the fix

* revert lclvars.cpp changes

* Revert "revert lclvars.cpp changes"

This reverts commit d39af7084687e8fe5e6d4f71674ec84d36a88340.

* wip

* revert lclvars.cpp changes

* deleted inst_RV_ST()

* removing logging, added some asserts

* jit-formatting

* add back case of INS_add and some more asserts

* reset lclvars.cpp

* delete comments and cleanup code

* revert changes inside common.il

* Revert "Disable failing Windows arm32 tests (#38844)"

This reverts commit 311fd810053bcfc4933d1a371323306f8860b3eb.

* review comments

* added standard function header

fix failing revocation & ssl tests (#38827)

* fix failing tests

* fix casing

* feedback from review

* style update

* retire EndEntityIsServer

Fix LookupPromotedStructDeathVars out param initialization (#38770)

Disable 2 System.Drawing tests for TailcallStress (#38891)

Issue: https://github.com/dotnet/runtime/issues/38889

Disable DisableAiaOptionWorks for TailcallStress (#38888)

Issue: https://github.com/dotnet/runtime/issues/38887

Disable pauseonstart test (#38883)

Issue: https://github.com/dotnet/runtime/issues/38847

Remove illegal `delete` of `Instr` objects (#38831)

`Instr` objects come from a static pool and should not be deleted
with `delete`. Instead, they are marked no longer used by setting
their `opcode` field to -1.

Fixes #12433