Remove some allocation at "hello world" startup (#44469)

* Specialize `EqualityComparer<string>.Default`

Removes ~80 allocations at startup.

* Avoid loading Encoding.Unicode just for CodePage

* Use fixed instead of GCHandle.Alloc in EventSource.Initialize

* Remove lock / lock object from EncodingProvider.AddProvider

* Remove lock object from AppDomain.cs

* Lazily allocate EventSource's m_createEventLock

It's only used on an error path.  We don't need to allocate it for each EventSource that's created.

* Avoid unnecessary CultureInfo access in derived TextWriters

SyncTextWriter already overrides FormatProvider, in which case the t.FormatProvider passed to the base will never be used, so this call is incurring a virtual dispatch for no benefit.  And NullTextWriter needn't access InvariantCulture and force it into existence if it isn't yet, as the formatting should never actually be used, and if it is, its FormatProvider override can supply the culture.

* Avoid allocating AssemblyLoadContext's dictionary if no direct interaction with ALC

AssemblyLoadContext.OnProcessExit gets called by EventSource, which in turn forces s_allContexts into existence in order to lock on it in order to enumerate all active contexts, and if there's been no interaction with AssemblyLoadContext, there won't be any to enumerate.  So delay allocate the object.

* Address PR feedback

* Call EventListener.DisposeOnShutdown from AppContext.OnProcessExit

Avoids the need to register with AppContext.ProcessExit, avoiding an EventHandler allocation, and avoids the need in the common case to fire AppContext.ProcessExit, which in turn avoids allocating an AppDomain and EventArgs if they weren't otherwise created, plus it avoids the delegate invocation.

* Update src/libraries/System.Private.CoreLib/src/System/Diagnostics/Tracing/EventSource.cs

Update dependencies from https://github.com/mono/linker build 20201110.1 (#44499)

Microsoft.NET.ILLink.Tasks
 From Version 6.0.0-alpha.1.20559.3 -> To Version 6.0.0-alpha.1.20560.1

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>

Reduce StreamWriter allocation (#44495)

* Lazily-allocate StreamWriter's byte[] buffer

The byte[] buffer is used just when flushing to the underlying stream, to store the intermediate bytes as generated from the characters by the encoding.  Currently the byte[] buffer is allocated in the constructor, but for relatively small strings written/flushed synchronously (such as those often used with Console.Write), we can avoid the buffer entirely via stack allocation (in Flush); for all other cases, we can allocate it on demand.

(I considered using ArrayPool to rent/return in each Flush{Async} call, but opted not to in this change.  Someone could investigate that subsequently.)

* Undo use of static async helpers

In .NET Framework, there was a non-trivial penalty due to StreamWriter deriving from MarshalByRefObject and the impact that had on async methods accessing lifted locals on the state machine (it made them significantly more expensive).  The workaround was to make the async methods be statics and pass in all of the relevant instance state needed.  That workaround is no longer necessary on core, where MarshalByRefObjects are nops, leaving behind a workaround that is not just clunkier, but actually makes things more expensive because more state needs to be stored on the state machine objects (all of the arguments, which can instead just be accessed off of `this`).  Undo that whole change.

* Address PR feedback

Use corlib instead of System.Private.CoreLib as a name prefix in AOT symbol names. (#44235)

WASM Bindings optimizations and fixes (#41808)

* Fix MarshalTypedArrayByte and re-enable it. Re-enable TestFunctionApply

* Re-enable MarshalTypedArray

* Detect when the managed wrapper for a JS object has been collected and create a new one

* Remove debugging code

* Maintain a small pool of temporary root instances to minimize GC pressure. Improve benchmark

* Don't use release_roots in call_method due to varargs overhead

* Various call_method optimizations

* Checkpoint: Don't rely on finally block for teardown in call path, because it has a performance cost

* Checkpoint: Unboxing fast path for primitives

* Checkpoint: Fix unboxing fast path

* Update bindings to use bound static methods instead of call_method in various places

* Address PR feedback

* Revert sample and add separate proj for benchmark

* Fix benchmark

* Revert test change

* Fix passing mono object ptrs to bound functions
Fix passing strings across the boundary
Fix JS strings being truncated at the first null when passed to mono

* Implement unboxing for chars
Slightly optimize the unboxing slow path

* Don't allocate a root buffer for arguments if nothing needs to be rooted. Reuse scratch native buffer across calls unless re-entrant to avoid per-invoke malloc

* Fix whitespace damage from merge

* Tweaks to try and prevent boxing/gc

* Fix typo

* Add some tests

* Fix test failures

* Add more error handling and diagnostic messages
Fix a couple broken tests

* Repair merge damage

* Remove bindings benchmark

* Use TypedArray.fill 3-argument version to zero memory

* Checkpoint: Introduce format strings

* Fix interpolated strings

* Test refactoring

* Checkpoint: Add more test coverage for bindings and interop

* Checkpoint: Enum marshaling works

* Improve test coverage

* Checkpoint: Unify unboxing of primitive types

* Checkpoint: Unify unboxing of primitive types

* Checkpoint: Restore fn to satisfy runtime-test.js

* Checkpoint: Unify boxing for primitives

* Remove now-unused box methods

* Don't store names for null method IDs

* Fix indentation damage

* Add test

* Satisfy CI

* Accept weaker promises

Co-authored-by: Larry Ewing <lewing@microsoft.com>

JIT: preliminary version of profile-based inline policy (#44427)

Add a new inline policy that can be used when a method has profile data.

It uses the call site frequency to boost profitability. Size and per-call
benefit are currently using the estimates done by the model policy.

Not on by default. Set COMPlus_JitInlinePolicyProfile=1 to enable.
Add testing to weekly experimental runs.

Implement internal ISpanFormattable on {U}IntPtr (#44496)

Allows nint and nuint to participate in the same optimizations in string building / formatting as do the other primitives.

Add libs.native subset when using the runtime helper target in the wasm makefile (#44450)

Ensures libSystem.Native.a/libSystem.IO.Compression.Native.a end up in the runtime pack.

Revert "Use Dictionary for underlying cache of ResourceSet (#44104)" (#44482)

This reverts commit a6835438581730da23fb3d4eea7ea7c4af1b00a3.

Fix R2RDump test in CG2 pipelines (#44460)

Our scripts normally assume that all test projects are at least
two nesting levels deep under $(TestBinDir); the r2rdump test was
violating that and, in doing so, it was triggering a failure in
CG2 compilation.

After I decided to fix this by moving the r2rdump folder under
"readytorun" I found out that there already is a r2rdump folder
there - apparently Amy during her initial R2RDump implementation
authored a basic test suite even though it's disabled right now.

I have resolved this by moving the original test under
"readytorun/r2rdump" to "readytorun/r2rdump/BasicTests" and I moved
Andrew's new test under "readytorun/r2rdump/FrameworkTests".
I'm not super happy about the naming but I don't have any better
ideas at the moment, I'll be happy to improve this based on PR
feedback.

Thanks

Tomas

[mono][jit] Emit profiler enter after jit attach; leave before detach (#44345)

* [jit] Emit profiler enter after jit attach; leave before detach

profiler enter code (such as mono_trace_enter_method) must not be
called before a thread attaches to the runtime - otherwise calls like
`mono_domain_get()` will return NULL unexpectedly and then crash.
When detaching, we should call the profiler leave code before the
detach, and suppress it on return.

Simple repro (that relies on pal_signal.c SignalHandlerLoop - which is
a background thread from System.Native that calls back into managed
when there's a SIGCHLD): compile and run this program with `MONO_ENV_OPTIONS=--trace`

```csharp
using System;
using System.Diagnostics;

namespace Repro
{
    internal class Program
    {
        private static void Main(string[] args)
        {
            using (Process myProcess = new Process())
            {
                    myProcess.StartInfo.UseShellExecute = true;
                    myProcess.StartInfo.FileName = "echo";
                    myProcess.StartInfo.Arguments = "hello from shell";
                    myProcess.StartInfo.CreateNoWindow = true;
                    myProcess.Start();
                    myProcess.WaitForExit();
            }
            Console.ReadKey ();
        }
    }
}
```

Update System.Drawing.Common stable versions to fix harvesting (#44481)

Bump Helix SDK so that iOS simulators don't get rebooted before/after every Helix job (#44455)

Use commit parents instead of branch names to evaulate changed paths. (#44474)

[runtime] Add some support for stack walks for AOT code on wasm. (#44431)

For methods which need to show up during stack walks, push/pop an LMF
frame and have mono_arch_unwind_frame () handle it.

This is needed to be able to support methods which do stack walks like
Type.GetType () or Assembly.Load () in the future.

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

[RyuJIT] Fix get_ManagedThreadId(get_CurrentThread) optimization (convert to NI) (#41213)

* Convert get_ManagedThreadId(get_CurrentThread) optimization to NamedIntrinsics

* drop CORINFO_HELP_INTERNALTHROW and CORINFO_HELP_SEC_UNMGDCODE_EXCPT

[master] Update dependencies from mono/linker (#44408)

* Update dependencies from https://github.com/mono/linker build 20201109.1

Microsoft.NET.ILLink.Tasks
 From Version 6.0.0-alpha.1.20556.1 -> To Version 6.0.0-alpha.1.20559.1

* Update dependencies from https://github.com/mono/linker build 20201109.2

Microsoft.NET.ILLink.Tasks
 From Version 6.0.0-alpha.1.20556.1 -> To Version 6.0.0-alpha.1.20559.2

* Update dependencies from https://github.com/mono/linker build 20201109.3

Microsoft.NET.ILLink.Tasks
 From Version 6.0.0-alpha.1.20556.1 -> To Version 6.0.0-alpha.1.20559.3

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>

Initial changes to support Mac Catalyst (#44127)

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

Make internal API ThrowIfDeserializationInProgress internal (#44147)

* Make internal API ThrowIfDeserializationInProgress internal

also avoid doing costly argument checks

* Make methods public again

* Update src/libraries/System.Private.CoreLib/src/System/Runtime/Serialization/SerializationInfo.SerializationGuard.cs

Co-authored-by: Levi Broderick <GrabYourPitchforks@users.noreply.github.com>
Co-authored-by: Levi Broderick <GrabYourPitchforks@users.noreply.github.com>

Disable 2 failing coreclr tests until they are fixed. (#44420)

* another failing test

* exclude another failure,

Resolve warnings for assignment or explicit cast of possibly null type parameter value (#43925)

New event to track memory allocation for JIT code (#44030)

* Added MethodJitMemoryAllocatedForCode event

* Renane the event fields

* Include cold code size as well

* Replace methodName with methodId

* Update the condition that fires the event

Removing `g_hmodCoreCLR` and `g_hThisInst`  (#43735)

* g_hmodCoreCLR is Windows-only

* Use 42 in MarkAsThrownByUsWorker()

* replaced GetCLRModule with GetClrModulePathName

* Addded PAL_GetPalHostModule to mscordac_unixexports

* replaced a few cases of g_hThisInst where used to get module path. We can uise `GetClrModulePathName` instead.

* more cases where used to get module path

* unused `GetModuleInst()`

* not using HMODULE in StressLog

* removed mscoree GetModuleInst

* Deleted both  `g_hThisInst` and `g_hmodCoreCLR`

* fix Linux build

* rename `GetModuleBase` ->` GetClrModuleBase`,  it was confusing with other `GetClrModuleBase`

* Fix Linux build

* PR feedback

* Removed hMod variables in Debugger areas

* standalone GC

* cache GetClrModuleBase

* Prevent double-read of `pImageBase`

* fix linker issue

* couple more link order fixes.

* Remove `g_thisModule` in daccess.cpp. Fix typos.

Improve nullable annotation on Volatile.Read (#44410)

Make it clear to the compiler that Volatile.Read doesn't write to the ref argument, such that if the input was non-null before the call, it'll still be non-null after the call.

Fix genEventing.py to generate correct EventEnabled functions (#44384)

[iOS] Fix build for Xcode 12.1 (#44311)

Remove unused project references (#44425)

Remove some unnecessary GetTypeInfo usage (#44414)

Improve crossgen2 comparison jobs (#44119)

- Fix compilation on unix platforms
  - Wrap use of wildcard in quotes
- Print better display name into log
- Fix X86 constant comparison handling
- Add ability to compile specific overload via single method switches

Change async void System.Text.Json test to be async Task (#44418)

Disable a failing test. (#44404)

Fix C++ build for mono/metadata/threads.c (#44413)

`throw` is a reserved keyword in C++.

Move some more UnmanagedCallersOnly tests to IL now that they're invalid C# (#43366)

Add files need for wasm executable relinking/aot to the wasm runtime pack. (#43785)

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

Update glossary (#44274)

Co-authored-by: Juan Hoyos <juan.hoyos@microsoft.com>
Co-authored-by: Stephen Toub <stoub@microsoft.com>
Co-authored-by: Günther Foidl <gue@korporal.at>

Fix/remove TODO-NULLABLEs (#44300)

* Fix/remove TODO-NULLABLEs

* remove redundant !

* apply Jozkee's feedback

* address feedback

Delete stale comment about dotnet/roslyn#30797

Update dotnet/roslyn issue link

Clean up the samples (#44293)

change using in ConnectCallback_UseUnixDomainSocket_Success (#44366)

add missing constructor overloads (#44380)

Co-authored-by: Geoffrey Kizer <geoffrek@windows.microsoft.com>

LongProcessNamesAreSupported: make test work on distros where sleep is a symlink/script (#44299)

* LongProcessNamesAreSupported: make test work on distros where sleep is a symlink/script

* PR feedback

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

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

Disable RunThreadLocalTest8_Values on Mono (#44357)

* Disable RunThreadLocalTest8_Values on Mono

It's failing on SLES

* fix typo

Update dependencies from https://github.com/mono/linker build 20201106.1 (#44367)

Microsoft.NET.ILLink.Tasks
 From Version 6.0.0-alpha.1.20555.2 -> To Version 6.0.0-alpha.1.20556.1

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>

clarify http version test (#44379)

Co-authored-by: Geoffrey Kizer <geoffrek@windows.microsoft.com>

Improve performance of Enum's generic IsDefined / GetName / GetNames (#44355)

Eliminates the boxing in IsDefined/GetName/GetValues, and in GetNames avoids having to go through RuntimeType's GetEnumNames override.

Bootstrapping a test for R2RDump (#42150)

Reflect test path changes in .gitattributes; remove nonexistent files (#44371)

Add slash in Windows SoD tool build (#44359)

* Add slash in Windows SoD tool build

* Update SoD search path to match output dir

* Fixup dotnet version

* Remove merge commit headers

* Disable PRs

Co-authored-by: Drew Scoggins <andrew.g.scoggins@gmail>

Match CoreCLR behaviour on thread start failure (#44124)

Co-authored-by: Aleksey Kliger (λgeek) <akliger@gmail.com>

Fix Send_TimeoutResponseContent_Throws (#44356)

If the client times out too quickly, the server may never have a connection to accept and will hang forever.

[master] Update dependencies from dotnet/arcade dotnet/llvm-project dotnet/icu (#44336)

[master] Update dependencies from dotnet/arcade dotnet/llvm-project dotnet/icu

 - Merge branch 'master' into darc-master-2211df94-2a02-4c3c-abe1-e3534e896267

Fix ELT profiler tests (#44285)

Add threshold on number of files / partition in SPMI collection (#44180)

* Add check for files count

* Fix the OS check

* decrese file limit to 1500:

* misc fix

* Do not upload to azure if mch files are zero size

Tighten argument validation in StreamConformanceTests (#44326)

[master] Update dependencies from mono/linker (#44322)

* Update dependencies from https://github.com/mono/linker build 20201105.1

Microsoft.NET.ILLink.Tasks
 From Version 6.0.0-alpha.1.20527.2 -> To Version 6.0.0-alpha.1.20555.1

* Update dependencies from https://github.com/mono/linker build 20201105.2

Microsoft.NET.ILLink.Tasks
 From Version 6.0.0-alpha.1.20527.2 -> To Version 6.0.0-alpha.1.20555.2

* Disable new optimization for libraries mode (it cannot work in this mode)

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
Co-authored-by: Marek Safar <marek.safar@gmail.com>

Simple GT_NEG optimization for #13837 (#43921)

* Simple arithmetic optimization with GT_NEG

* Skip GT_NEG optimization when an operand is constant. Revert bitwise rotation pattern

* Fixed Value Numbering assert

* Cleaned up code and comments for simple GT_NEG optimization

* Formatting

Co-authored-by: Julie Lee <jeonlee@microsoft.com>

Simplify catch-rethrow logic in NetworkStream (#44246)

A follow-up on #40772 (comment), simplifies and harmonizes the way we wrap exceptions into IOException. Having one catch block working with System.Exception seems to be enough here, no need for specific handling of SocketException.

Use Dictionary for underlying cache of ResourceSet (#44104)

Tweak StreamConformanceTests for cancellation (#44342)

- Avoid unnecessary timers
- Separate tests for precancellation, ReadAsync(byte[], ...) cancellation, and ReadAsync(Memory, ...) cancellation

[mono] Change CMakelists.txt "python" -> Python3_EXECUTABLE (#44340)

Debian doesn't install a "python" binary for python3.

Add an implicit argument coercion check. (#43386)

* Add `impCheckImplicitArgumentCoercion`.

* Fix tests with type mismatch.

* Try to fix VM signature.

* Allow to pass byref as native int.

* another fix.

* Fix another IL test.

[tests] Re-enable tests fixed by #44081 (#44212)

Fixes
https://github.com/mono/mono/issues/15030 and
fixes https://github.com/mono/mono/issues/15031 and
fixes https://github.com/mono/mono/issues/15032

[sgen] Add Ward annotations to sgen_get_total_allocated_bytes (#43833)

Attempt to fix https://jenkins.mono-project.com/job/test-mono-mainline-staticanalysis/

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

Use targetPlatformMoniker for net5.0 and newer tfms (#43965)

* Use targetPlatformMoniker for net5.0 and newer tfms

* disabling analyzer, update version to 0.0, and use new format.

* update the targetFramework.sdk

* removing supportedOS assembly level attribute

* fix linker errors and addressing feedback

* making _TargetFrameworkWithoutPlatform as private

Remove silly ToString in GetCLRInstanceString (#44335)

Update SharedPerformanceCounter assert (#44333)

Pick libmonosgen-2.0.so from cmake install directory instead of .libs (#44291)

This aligns Linux with what we already do for all the other platforms.

Forbid `- byref cnst` -> `+ (byref -cnst)` transformation. (#44266)

* Add a repro test.

* Forbid the transformation for byrefs.

* Update src/coreclr/src/jit/morph.cpp

Co-authored-by: Andy Ayers <andya@microsoft.com>
* Update src/coreclr/src/jit/morph.cpp

* Fix the test return value.

WriteLine is just to make sure we don't delete the value.

* improve the test.

avoid a possible overflow and don't waste time on printing.

Co-authored-by: Andy Ayers <andya@microsoft.com>

Use the new async Run method from xharness and implement a test (#44046)

* Use the new async Run method from xharness and implement a test

* Remove PumpThreadPool as it is no longer used

* Update xharness for async support

Set fgOptimizedFinally flag correctly (#44268)

- Initialize to 0 at compiler startup
- Set flag when finally cloning optimization kicks in

Fixes non-deterministic generation of nop opcodes into ARM32 code

Implementing support to Debugger::Break. (#44305)

[master][watchOS] Add simwatch64 support (#44303)

Xcode 12.2 removed 32 bits support for watchOS simulators, this PR helps to fix https://github.com/xamarin/xamarin-macios/issues/9949, we have tested the new binaries and they are working as expected


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

File.Exists() is not null when true (#44310)

* File.Exists() is not null when true

* Fix compile

* Fix compile 2

ConcurrentQueueSegment allows spinning threads to sleep. (#44265)

* Allow threads to sleep when ConcurrentQueue has many enqueuers/dequeuers.

* Update src/libraries/System.Private.CoreLib/src/System/Collections/Concurrent/ConcurrentQueueSegment.cs

Co-authored-by: Stephen Toub <stoub@microsoft.com>
* Apply suggestions from code review

Co-authored-by: Stephen Toub <stoub@microsoft.com>
Co-authored-by: AMD DAYTONA EPYC <amd@amd-DAYTONA-X0.com>
Co-authored-by: Stephen Toub <stoub@microsoft.com>

Remove unused QCall for WinRTSupported (#44278)

JIT: minor inliner refactoring (#44215)

Extract out the budget check logic so it can vary by inlining policy.
Use this to exempt the FullPolicy from budget checking.

Fix inline xml to dump the proper (full name) hash for inlinees.

Update range dumper to dump ranges in hex.

Add stream conformance tests for TranscodingStream (#44248)

* Add stream conformance tests for TranscodingStream

* Special-case 0-length input buffers to TranscodingStream.Write{Async}

The base implementation of Encoder.Convert doesn't like empty inputs.  Regardless, if the input is empty, we can avoid a whole bunch of unnecessary work.

Bump xharness workitem timeout for Android as well (#44301)

Previously we only did it for iOS/tvOS

Fix illumos native build (#44251)

* Fix illumos native build

* Address PR feedback

Http version test cleanup (#44283)

* remove dead code

* simplification

* more

* fix to still use stream wrapper

* combine tests

* more

* get rid of ClientDisconnectOk overload

* fix

* simplify DribbleStream

Co-authored-by: Geoffrey Kizer <geoffrek@windows.microsoft.com>

Ctor_SafeHandle_BasicPropertiesPropagate_Success: improve raw packet case. (#44294)

Fix expected value for ProtocolType when a new Socket is created from
an existing raw packet socket Handle. This assert isn't checked unless
the test runs as root.

And use a proper value for the ProtocolType when the Socket is created.

Workaround xunit bug for disabling FileStreamStandaloneConformanceTests (#44295)

Uncomment `pr: none` in perf.yml (#44302)

Correctly set dylib version for libmonosgen-2.0.dylib (#44290)

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

fix stream conformance tests for mock provider (#44282)

Co-authored-by: Geoffrey Kizer <geoffrek@windows.microsoft.com>

More consistent cache key validation in Remove (#44286)

Add R2RDump support for lazy GC info (#44261)

This change makes it such that GC info is not automatically parsed
for each method, we only store a delegate performing the parsing
that gets called on demand. In practice this also helps us with
working around a bug in GC info previously crashing R2RDump when
emitting PDB in ARM release build mode.

Thanks

Tomas

Omit ErrorInformation from ManagementException binary serialization (#44112)

* Omit ErrorInformation from ManagementException binary serialization
Fixes #2116

* Serialize MgmtException.errorObject as NULL for .NET Framework compat

Allow embedded nulls in the result of mono_wasm_invoke_js (). (#44237)

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

Split SslStream stream conformance tests for TLS1.2/1.3 (#44245)

Move dotnet version to hardcoded version to workaround TFM change (#44224)

* Move dotnet version to hardcoded version to workaround TFM change

* Change 'windows' -> 'Windows_NT'

* Move osGroup checks back to 'windows'

* Update scenario yml

* Add crossgen path for SoD tests

Disable DOUBLY_LINKED_FL (#44262)

Fixes #44250

 Use shared lock for files opened without FileAccess.Write on Linux (#44185)

* Use shared lock for readonly files on Linux

* Add suggested tests

* Exclude test from OSX

* Add RemoteExecutor.IsSupported condition to tests

Update IpcStreamFactory state machine to handle being started on a thread that ends (#43711)

Handles the case on Windows where Async IO is cancelled due to the thread coreclr started on ending.

Remove MNADeprecationWorkaround in host test assets (#44259)

JIT: simple redundant compare optimization (#43811)

For a relop, walk up the dominator tree to see if any dominating
block has a similar compare. If so, and there's just one path from
that block to the relop, the relop's value is known.

Closes #11909.

Use SetFileInformationByHandle on FileStream.SetLength (#44170)

* Use SetFileInformationByHandle on FileStream.SetLength

* Remove Interop.SetEndOfFile since is no longer used

* Add manual test

* Address feedback

Add debugging a ci dump documentation template and script to fill info and include it on azdo test tab (#43937)

* Add debugging a ci dump documentation template and script to generate it on helix

* PR Feedback and fix helix workitems

* Fix python script and -buildid argument

* Upload doc only if dumps were found

* Pass down templatedir

* PR feedback

Update buildtriage.md (#44196)

* Update buildtriage.md