Give AOT mono package arch identifiers (#49893)

Revise UnnamedOptionsManager to avoid allocation in ctor (#49883)

Enable building repo on Windows ARM64 (#49864)

Support open delegate for Nullable<> (#42837)

* Support open delegate for Nullable<>

* Disable new tests on Mono

Co-authored-by: David Wrighton <davidwr@microsoft.com>

[crossgen2] Fix memory leak in _corInfoImpls. (#49764)

* [crossgen2] Implement Dispose in Compilation.

_corInfoImpls elements keeps _compilation reference which keeps reference to whole table _corInfoImpls. The circular referene together with issue #12255 potentionally leads to memory leak in crossgen2 if that table is created more than once. Dispose() method clears the table and prevents memory leak.

Signed-off-by: Timur Mustafin <t.mustafin@partner.samsung.com>
* Update src/coreclr/tools/aot/crossgen2/Program.cs

Co-authored-by: Jan Kotas <jkotas@microsoft.com>

Regression test for #49826 (#49870)

This can only be merged in after #49854 fixing the bug is merged in.

Thanks

Tomas

Crossgen2-related CoreCLR test infra fixes (#49867)

(*) When compiling tests with Crossgen2, use optimizations by default
like Crossgen1 does.

(*) Don't use large version bubble when Crossgen2-compiling
the framework. While at some point this had value as preparatory
testing for composite mode, it's not a shipping scenario right now
and it was hiding the codegen bug causing the issue

https://github.com/dotnet/runtime/issues/49826

in System.Text.Json.

Thanks

Tomas

[browser][wasm] Fix provision-wasm build error (#49863)

```
cd emsdk && ./emsdk install `cat emscripten-version.txt`
cat: emscripten-version.txt: No such file or directory
```

[AndroidCrypto] Implement Root (read-only) and CurrentUser/My certificate stores (#48862)

Remove unused variables in test build scripts (#49837)

Remove unnecessary suppressions of IL2060  (#49843)

* Remove unnecessary suppressions of IL2060

These suppressions are no longer necessary with the latest trimmer version.

Fix #49486

Resolving ILLink warnings on System.Private.Xml (Part 1) (#49413)

* Resolving ILLink warnings on System.Private.Xml (Part 1)

* Only annotate the unsafe Load overload methods from XslCompiledTransform type

* Address feedback and fix one more warning

* Update src/libraries/System.Private.Xml/src/System/Xml/Xsl/Runtime/EarlyBoundInfo.cs

Co-authored-by: Eric Erhardt <eric.erhardt@microsoft.com>
* PR Feedback

Co-authored-by: Eric Erhardt <eric.erhardt@microsoft.com>

Introduce `iossimulator` RIDs, and convert as needed (#49305)

Previously, we have had four iOS RIDs:

iOS-arm
iOS-arm64
iOS-x86
iOS-x64

Apple has never shipped an iOS device with an x86 or x64 processor. Instead, the x86/x64 RIDs have meant "iOS simulator with these arches" as opposed to "iOS with these arches". Amongst other things, that means building against a DIFFERENT SDK, iPhoneSimulator.platform vs iPhoneOS.platform

In the Apple Silicon future, the iOS simulator is now an ARM64 binary - so we need:

iOS-arm
iOS-arm64
iOS-arm64, but built against the simulator SDK not the device SDK
iOS-x86
iOS-x64
Clearly, there's a problem.

The solution is to move the simulators to a different RID, to avoid the overloading issue:

iOS-arm
iOS-arm64
iOSSimulator-arm64
iOSSimulator-x86
iOSSimulator-x64

This PR introduces the new entries in the RID graph, moves our existing iOS-x{86,64} to iOS-sim-x{86,64}, adds a new iOS-arm64.

The above also applies for tvOS, with a smaller set of OSes:

tvOS-arm64
tvOSSimulator-arm64
tvOSSimulator-x64

Ref: #48216

Avoid use of incorrect instantation argumens when checking for constraint satisfaction (#47258)

- The constraint processing logic in the runtime conflates the idea of constraint checking on open, closed over concrete types, and closed over non-concrete types
- This change adds a tweak to avoid using an instantiation context that isn't related to the type variable being instantiated

Fixes issue #45600 and adds a regression test from the customer.

Fix signature generation for devirtualization (#49854)

- Crossgen2 computes its signatures from slightly different data than the VM does
- Tweak the jit interface to specify that the resolved token comes from devirtualization to detect this special case
  - The extra tokenType information  is ignored by the VM
- Pass the devirtualization specified owning type through when necessary
  - Check that it meets the requirements of being defined on a a derived type from the method target

Use arm jit for armel in crossgen2, and build the matching bits on all architectures (#49734)

- Use arm jit for armel in crossgen2, and build the matching bits on all architectures

struct improvement, part1: create more LCL_FLD (#48377)

* Add additional tests.

* Use LclFld when copy to/from promoted from/to an unpromoted struct/block.

* Response review.

Slimmer IOptions<T> (#49852)

* Slimmer IOptions<T>
- Use a Lazy instead of the IOptionsCache (which is a concurrent dictionary)

[main] Update dependencies from dotnet/xharness (#49865)

Microsoft.DotNet.XHarness.CLI , Microsoft.DotNet.XHarness.TestRunners.Xunit
 From Version 1.0.0-prerelease.21165.2 -> To Version 1.0.0-prerelease.21169.1

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

Big-endian fix: ResourceReader.AllocateStringForNameIndex (#49811)

* Byte-swap characters read from memory-mapped resources
  on big-endian systems

Big-endian fix: System.Reflection.Metadata BlobBuilder/BlobWriter (#49812)

* BlobBuilder: do not use Encoding.Unicode.GetBytes but simple
  byte-swaps on big-endian systems (fixes a semantic difference
  to little-endian code w.r.t. surrogate char handling)

* BlobWriter: add big-endian WriteUTF16 code path like BlobBuilder

Big-endian fix: Base64Encoder (#49814)

* Take byte order into accout when packing characters into an int

Add clr.iltools subset (#49545)

* Add new subset to build only the IL tools (ilasm/ildasm)
  (enabled by default)

* New -skipiltools argument to src/coreclr/build-runtime.{cmd,sh}
  (skips building the IL tools if present)

* Actually make -skipjit argument work to not build the JIT & VM

Big-endian fix: GUID handling (#49813)

* Fix Utf8Formatter.Guid on big-endian systems
  (first three fields are stored in native byte order, not always
  little-endian)

* Use Guid constructor in BlobContentId.FromHash to remove
  endian-dependent code accessing the structure directly

Big-endian test case fixes: Xoshiro hash (#49709)

* Skip Xoshiro_AlgorithmBehavesAsExpected on big-endian systems

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

Big-endian fix: PEBuilder.CalculateChecksum (#49816)

* Always use little-endian encoding to compute checksum

Big-endian fix: UnmanagedMemoryAccessor.Write (decimal) (#49815)

* Use TryGetBits to avoid dependency on accessing internal
  fields of decimal type (which would be endian-dependent)

Use latest DiaSymReader and support ARM64 host in crossgen2 (#49841)

Disable looking for PaxCtl on non Linux/BSD distros (#49746)

Fix SuperPMI replay flags (#49850)

Before this fix, when doing replay (not asmdiffs) of a set of
MCH files, the `-f` arguments would accumulate, one for every
MCH file run so far.

Enable the latest managed pgo data (#49793)

- Implementation is parallel to existing ibc handling, so that it can be toggled on/off by adjusting the `UsingToolIbcOptimization` property
- Use the same data for all assemblies produced in current build
- Apply data to release builds only
- Disable mismatch assertions in jit for current state where il mismatches are common

Fix failures on Windows when more warnings are enabled. (#49194)

* Fix warning for System.IO.Compression.Native when more analysis is enabled.

* Fix warnings in corehost when more analysis is enabled.

Fix or disable warnings on latest VS dogfood (#49799)

* Fix or disable warnings on latest VS dogfood

* Delete JIT local warning disables

Disable android arm64 tests on PRs (#49844)

System.Text.Encodings.Web refactoring and code modernization (#49373)

- Unify workhorse implementations across all inbox encoders
- Refactor most unsafe code from TextEncoder workhorse routines into standalone helpers
- Fix bounds check logic in workhorse routines
- SSSE3-optimize central workhorse routine
- Remove vestigial code from the library and unit test project
- Add significant unit test coverage for the workhorse routines and unsafe helpers
- Ref: CVE-2021-26701 (MSRC 62749)

Update ActivityUserGuide.md (#49818)

Refer activity guide to the official docs

https://github.com/dotnet/docs/pull/23313 is bringing more enhancements shortly

Improve the speed of using COMPlus_JitTimeLogCsv (#49798)

* Improve the speed of using COMPlus_JitTimeLogCsv

I noticed that setting this and running a superpmi replay
was super slow. (Btw, note that you must use `--sequential`
in the scenario to avoid parallel writes to the log file).

This change fixes this in a number of ways:
1. The `CycleTimer::CyclesPerSecond()` function does not-trivial
computation to compute its answer. So cache the result.
2. It's expensive to constantly open and close the output file
(once per function). Partially, this is because anti-virus seems to
get involved. So open it once, and keep it open until the JIT
process is shut down.
3. Avoid calling `eeGetMethodFullName()` in DEBUG builds. (Note that
JitTimeLogCsv collects interesting information beyond just per-phase
timings that are also useful for extracting from a CSV file.)

* Fix CachedCyclesPerSecond logging

* Fix clang build break

[main] Update dependencies from dotnet/arcade dotnet/icu (#49820)

* Update dependencies from https://github.com/dotnet/arcade build 20210317.3

Microsoft.DotNet.XUnitExtensions , Microsoft.DotNet.VersionTools.Tasks , Microsoft.DotNet.CodeAnalysis , Microsoft.DotNet.Build.Tasks.TargetFramework.Sdk , Microsoft.DotNet.Build.Tasks.Packaging , Microsoft.DotNet.Build.Tasks.Installers , Microsoft.DotNet.Build.Tasks.Feed , Microsoft.DotNet.XUnitConsoleRunner , Microsoft.DotNet.Build.Tasks.Archives , Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.ApiCompat , Microsoft.DotNet.GenAPI , Microsoft.DotNet.GenFacades , Microsoft.DotNet.RemoteExecutor , Microsoft.DotNet.Helix.Sdk , Microsoft.DotNet.SharedFramework.Sdk
 From Version 6.0.0-beta.21166.5 -> To Version 6.0.0-beta.21167.3

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

Microsoft.NETCore.Runtime.ICU.Transport
 From Version 6.0.0-preview.3.21165.1 -> To Version 6.0.0-preview.3.21167.1

* Revert changes to publish-using-darc.ps1

Workaround still needed for https://github.com/dotnet/arcade/issues/6987

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

Only override and set EMSDK_PATH if it is not already set. (#49810)

Add PipeReader.Create from ReadOnlySequence<byte> (#48369)

Add new factory method to PipeReader to create a PipeReader from a ReadOnlySequence<byte>

[mbr] Return -1 if relative index is not in given generation (#49795)

Logically after an update is applied, tokens refer to rows in tables that are a
concatenation of all the added rows from every update.  (ie: if the baseline
table 0x23 had 2 rows and then generation 1 an added 3 more rows, an index like 0x05
refers to table 0x23 row 3 from gen 1).

The way the lookup works is that the EnC map table records the logical tokens
of the update.  So for table 0x23 the 3 additions in gen 1 will have entries
like encmap row 5 = 0x23000003, row 6 = 0x23000004, row 7 = 0x23000005, and
then rows for the next table and so on.  The relative index in gen1 of token
0x23000005 then, is the distance of its row (ie row 7) from the first token for
the table (ie row 5), plus 1 (since tokens are 1-based).  So in this case token
0x23000005 corresponds to the 7-5+1 = 3rd row of table 0x23 in the gen1 dmeta image.

The problem is that previously we returns this index-base+1 computation even in
cases where we walked either to the end of the encmap table or past the last
row for the table we care about.

In the case where we walked to the end of the encmap table, the index-base+1
could sometimes return a value that looked like a valid token.  The upshot is
that we looked up an incorrect AssemblyRef (0x23 table) from the wrong
generation.

The updated code returns -1 for all cases where we didn't find an encmap row
for our given token.

Example: https://gist.github.com/lambdageek/cd7ea06bf5004ba884e7979f28623da5

Update branch from master -> main (#49835)

Add ActiveIssue to failing System.Numerics.Tests.Vector3Tests.Vector3DistanceTest (#49825)

Add 3.1 to package index / validation (#49792)

* Add 3.1 inbox assembly versions to package index

* Include netcoreapp3.1 for package validation

Enable macOS-arm64 libraries tests on runtime-staging rolling builds (#49445)

* Disable failing Apple Silicon Libraries tests

-  Use tests.proj to disable catastrophically failing libraries tests
-  Add ActiveIssues where XUnit didn't crash

* Enable CoreCLR lib tests for Apple Silicon in Runtime-Staging

[wasm][AOT] Fix System.Buffers.Tests path (#49785)

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

Enhance DynamicallyAccessedMembers Attribute (#49778)

* eh fix

* test change that inadvertently got checked in earlier

* Suppresses the trimmer warning on TypeAnalysis ctor

* Incorporating FB

* Update src/libraries/System.Diagnostics.DiagnosticSource/src/System/Diagnostics/DiagnosticSourceEventSource.cs

Co-authored-by: Eric Erhardt <eric.erhardt@microsoft.com>
* Update src/libraries/System.Diagnostics.DiagnosticSource/src/System/Diagnostics/DiagnosticSourceEventSource.cs

Co-authored-by: Eric Erhardt <eric.erhardt@microsoft.com>
* Update src/libraries/System.Private.CoreLib/src/System/Diagnostics/Tracing/EventSource.cs

Co-authored-by: Eric Erhardt <eric.erhardt@microsoft.com>
* Fix DynamicDependency as per PR feedback

* an earlier change got reverted

* fixed proj file netcore app condition check

* fixed NETCORE_ENGINEERING_TELEMETRY build failures

* fixeing another NETCORE_ENGINEERING_TELEMETRY build failures

* Adding RequiresUnreferencedCode to TypeAnalysis ctor instead of suppressing the warning to get FB, not fully fixed

* PR FB and suppressing warnings for safe calls

* propagated the warning all the way up

* CI build break fix for one file

* excluding NativeRTEventSrc from being build in a project

* Missed couple of supppressions on NativeRTEventSrc

* build break fixes

* Enhancement to DynamicallyAccessedMembersAttribute

Co-authored-by: Eric Erhardt <eric.erhardt@microsoft.com>

Add Mono AOT Performance runs (#49766)

* Add AOT perf support

* Fix up dependency information

* Fix zipping behavior

* Specify between mono and mono aot

* Fix in another spot

* Cleanup archive

* Last mono cleanup

* Change archive extension to tar.gz

* Fix AOT paths

* Fix mkdir

* Add -p to mkdir

* Add recursive copy

* Fix logic around selecting AOT

* Remove comments

* Add Arm64

* Remove unused variable

FileStreamStrategies refactor (#49750)

* move all strategy-related types to Strategies subfolder and namespace

* make FileStreamHelpers a partial class, move some cross-platform helper methods there

* rename config switch

* address code review feedback

Clarify host testing instructions (#49797)

Print less confusing message for Mono's FailFast abort (#49802)

Remove arcade packaging workaround (#49613)

[System.Memory.Data] Add `Empty` to `BinaryData` (#49777)

This is useful for APIs which return `BinaryData` and want to use a
singleton instead of allocating a new `BinaryData` each
time. Previously, developers would need to create their own static
copy of an empty binary data, now they can just use `Empty`

Fixes #49670

Update aka.ms links (#48924)

* Update aka.ms links

* Temporarily remove the symbols, nethost and badge links

Allow for specify return value on System.Linq.Enumerable.*OrDefault methods (#48886)

* Fix #20064

* Add API to ref assembly

* Make overloads with defaultValue not nullable

* Add unit tests, simplify implementation

* Add LastOrDefault tests

* Add Queryable tests

* Additional tests. Reformatting TryGet Methods.

* Update src/libraries/System.Linq.Queryable/tests/FirstOrDefaultTests.cs

* Apply suggestions from code review

Co-authored-by: Eirik Tsarpalis <eirik.tsarpalis@gmail.com>
* Fix ref methods

* Further adjust nullability

* Fix more nullables

* fix failing tests

* Restore coding style

Co-authored-by: Eirik Tsarpalis <eirik.tsarpalis@gmail.com>

Stop building extra installer bundles during PGO build (#49779)

* Select only the runtime-internal package and turn off installers

* Remove R2R modifications

[main] Update dependencies from dotnet/arcade dotnet/icu dotnet/llvm-project dotnet/runtime-assets (#49752)

* Update dependencies from https://github.com/dotnet/arcade build 20210316.5

Microsoft.DotNet.XUnitExtensions , Microsoft.DotNet.VersionTools.Tasks , Microsoft.DotNet.CodeAnalysis , Microsoft.DotNet.Build.Tasks.TargetFramework.Sdk , Microsoft.DotNet.Build.Tasks.Packaging , Microsoft.DotNet.Build.Tasks.Installers , Microsoft.DotNet.Build.Tasks.Feed , Microsoft.DotNet.XUnitConsoleRunner , Microsoft.DotNet.Build.Tasks.Archives , Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.ApiCompat , Microsoft.DotNet.GenAPI , Microsoft.DotNet.GenFacades , Microsoft.DotNet.RemoteExecutor , Microsoft.DotNet.Helix.Sdk , Microsoft.DotNet.SharedFramework.Sdk
 From Version 6.0.0-beta.21155.1 -> To Version 6.0.0-beta.21166.5

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

Microsoft.NETCore.Runtime.ICU.Transport
 From Version 6.0.0-preview.3.21151.1 -> To Version 6.0.0-preview.3.21165.1

* Update dependencies from https://github.com/dotnet/llvm-project build 20210315.1

runtime.win-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools , runtime.win-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk , runtime.osx.10.12-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools , runtime.osx.10.12-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk , runtime.linux-arm64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk , runtime.linux-arm64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools , runtime.linux-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk , runtime.linux-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools
 From Version 9.0.1-alpha.1.21158.1 -> To Version 9.0.1-alpha.1.21165.1

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

System.ComponentModel.TypeConverter.TestData , System.Drawing.Common.TestData , System.IO.Compression.TestData , System.IO.Packaging.TestData , System.Net.TestData , System.Private.Runtime.UnicodeData , System.Runtime.TimeZoneData , System.Security.Cryptography.X509Certificates.TestData , System.Windows.Extensions.TestData
 From Version 5.0.0-beta.21159.1 -> To Version 6.0.0-beta.21162.1

* re-add publishing workaround

workaround for https://github.com/dotnet/arcade/issues/6987

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

[mono] Add all instances from the aot profile when using aot profiles… (#49758)

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

[interp] Actually extend the return value when transitioning back to interp from aot/jit (#49789)

[AndroidCrypto] Avoid printing JNI exceptions in cases where they are expected (#49530)

runtime-linker-tests: revert earlier change which added browser offse… (#49774)

Remove top-level range check nodes (#49271)

Refactored optRemoveRangeCheck to handle top-level standalone range check nodes.

Update Microsoft.DiaSymReader.Native package (#49739)

Annotate System.Diagnostics.Process APIs throwing PNSE on iOS/tvOS (#49354)

* Process.Start

* Remove redundant annotations

* Remove iOS since it's in the default list of supported platforms

* Process.Kill

* Make Process.Kill throw PNSE on iOS/tvOS

* Annotate windows-related implementation of Kill method

[main] Update dependencies from mono/linker (#49577)

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
Co-authored-by: Eric Erhardt <eric.erhardt@microsoft.com>
Co-authored-by: Marek Safar <marek.safar@gmail.com>
Co-authored-by: Alexander Köplinger <alex.koeplinger@outlook.com>

Add runtime MVID checks for composite images (#49668)

* Crossgen2 implementation of new MVID R2R section
* Runtime checks for assembly MVIDs against native images
* Basic R2RDump support for dumping the manifest assembly MVID table.
* R2R minor version increment; format spec update

Thanks

Tomas

Fix incorrect merge suppressing my change to publish CG2 framework (#49666)

As David discovered, I apparently made a merge bug in rebasing my
CG2 framework switch-over change prior to checking in that actually
suppressed CG2 publishing due to losing the property
PublishReadyToRunUseCrossgen2. This change fixes that deficiency.

Thanks

Tomas

fix typo (#49763)

More tests for FileStream edge cases (#49754)

* add StreamConformanceTests to the solution file so we can read source code from VS

* add FileOffsetIsPreservedWhenFileStreamIsCreatedFromSafeFileHandle tests

* add WriteAsyncStartsWherePreviousReadAsyncHasFinished test

[wasm] Add a mono_wasm_print_stack_trace () helper function usable during runtime debugging. (#49759)

Include "simple" UTF-8 validation and transcoding logic for interpreted and low-footprint scenarios (#49372)

Address issue where stress mode testing may not work for this more exhaustive than usual test (#49736)

Update area-owners.md (#49769)

Added myself to hosting areas.

Allow jit to disregard PGO; PGO changes for SPMI and MCS (#49551)

* COMPlus_JitDisablePGO will now block the jit from using profile data. Useful
for investigations and (someday) bug triage. Also disable GDV if we're disabling PGO

* `-base_jit_option` and `-diff_jit_option` can be used to pass options to
superpmy.py asmdiffs.

* Fix SPMI file name computation to better handle cases where the directory is
a relative path.

* Enhance `mcs -jitflags` output to describe how many method contexts have
PGO data, and which kinds of data they hold. Useful for validating that an
SPMI collection done via dynamic PGO has actually collected an interesting
set of jit requests.

[wasm] Enable wasm relinking by default for Release config (#49251)

* [wasm] Set default value for $(WasmBuildNative) based on:

- If $(WasmBuildNative) is already set (eg. by user project) to true, then error if EMSDK_PATH is unset
- if unset, and$(PublishTrimmed) == false, then set to false
- if unset, and $(Configuration) == "Release", then set to true
- If it is $(WasmBuildNative) == true after above steps, and emsdk is missing
    => then emit a warning, and set it to false

commits:

* [wasm] Set WasmBuildNative=true for Release config

- But set to `false` if `$(PublishTrimmed)` is not true

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

* dotnet-linker-tests need mono-aot-cross

* [wasm] Cleanup setting emcc flags

* Address review feedback from @lewing

.. and update tests to track the changes.

* Correctly address review feedback:

Reorganize span parse tests (#47678)

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

[wasm] Avoid error-ing out in AOT only mode if an AOT image is not found since apps (#49318)

can load assemblies which were not AOT-ed, if they don't attempt to execute code from them.

Adds support for building Wasm on Windows (#45545)

* First pass at building wasm on windows

* Adds support for building wasm on Windows

Emscripten and EMSDK_PATH must be set up externally.  This differs from linux and macos
where we optionally can do it for you.

* Cleanup

* Feedback

* Rename conflicting target

* Made sure current master still works

* auto provision when EMSDK_PATH is not set and try to see if a browser build works on windows

* Add call in front of batch scripts. All of a sudden it's necessary

* Use RunWithEmSdkEnv task

* Override arch for Browser OS

Similar to what we do in build.sh

* Update machine certificates

To avoid this error:

    Installing tool 'node-12.18.1-64bit'..
      Installation failed!
      Error: Downloading URL 'https://storage.googleapis.com/webassembly/emscripten-releases-builds/deps/node-v12.18.1-win-x64.zip': <urlopen error [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1045)>
      Warning: Possibly SSL/TLS issue. Update or install Python SSL root certificates (2048-bit or greater) supplied in Python folder or https://pypi.org/project/certifi/ and try again.
      Warning: The SDK/tool 'node-12.18.1-64bit' cannot be activated since it is not installed! Skipping this tool...

The script comes from runtimelabs repo https://github.com/dotnet/runtimelab/blob/d0cda961aeda26faae17c474e26d822cd1e45438/eng/pipelines/runtimelab/update-machine-certs.ps1

* Use msbuild to write the emcc version file

* Put emscripten version in text file

And use it in Makefile and mono.proj

* Try to fix Browser_wasm_Windows detection

* Install native dependencies

I think we are missing cmake and failing with:

      configure: cmake -DENABLE_WERROR=1 -DCMAKE_INSTALL_PREFIX=D:\workspace\_work\1\s\artifacts\obj\mono\Browser.wasm.Debug\out -DCMAKE_INSTALL_LIBDIR=lib -DCMAKE_BUILD_TYPE=Debug -DGC_SUSPEND=preemptive -DENABLE_MINIMAL=jit,portability,sgen_major_marksweep_conc,sgen_split_nursery,sgen_gc_bridge,sgen_toggleref,sgen_debug_helpers,sgen_binary_protocol,logging,shared_perfcounters,interpreter,threads,eventpipe,qcalls -DENABLE_INTERP_LIB=1 -DDISABLE_ICALL_TABLES=1 -DDISABLE_CRASH_REPORTING=1 -DENABLE_ICALL_EXPORT=1 -DENABLE_LAZY_GC_THREAD_CREATION=1 -DENABLE_LLVM_RUNTIME=1 -DENABLE_METADATA_UPDATE=1 "-DCMAKE_C_FLAGS= -fexceptions -ID:\workspace\_work\1\s\.packages\microsoft.netcore.runtime.icu.transport\6.0.0-preview.3.21151.1/runtimes/browser-wasm/native/include" "-DCMAKE_CXX_FLAGS= -fexceptions" D:\workspace\_work\1\s\src\mono\ -DCMAKE_TOOLCHAIN_FILE=D:\workspace\_work\1\s\src\mono\wasm\emsdk\upstream\emscripten\cmake\Modules\Platform\Emscripten.cmake -DCMAKE_CROSSCOMPILING_EMULATOR="D:/workspace/_work/1/s/src/mono/wasm/emsdk/node/12.18.1_64bit/bin/node.exe" -G Ninja
      Traceback (most recent call last):
        File "D:\workspace\_work\1\s\src\mono\wasm\emsdk\upstream\emscripten\\emcmake.py", line 33, in <module>
          sys.exit(run())
        File "D:\workspace\_work\1\s\src\mono\wasm\emsdk\upstream\emscripten\\emcmake.py", line 26, in run
          building.configure(sys.argv[1:])
        File "D:\workspace\_work\1\s\src\mono\wasm\emsdk\upstream\emscripten\tools\building.py", line 339, in configure
          run_process(args, stdout=stdout, stderr=stderr, env=env, **kwargs)
        File "D:\workspace\_work\1\s\src\mono\wasm\emsdk\upstream\emscripten\tools\shared.py", line 94, in run_process
          ret = subprocess.run(cmd, check=check, input=input, *args, **kw)
        File "D:\obj\windows-release\37amd64_Release\msi_python\zip_amd64\subprocess.py", line 472, in run
        File "D:\obj\windows-release\37amd64_Release\msi_python\zip_amd64\subprocess.py", line 775, in __init__
        File "D:\obj\windows-release\37amd64_Release\msi_python\zip_amd64\subprocess.py", line 1178, in _execute_child
      FileNotFoundError: [WinError 2] The system cannot find the file specified
    D:\workspace\_work\1\s\src\mono\mono.proj(410,5): error MSB3073: The command "call D:\workspace\_work\1\s\src\mono\wasm\emsdk\emsdk_env.bat && emcmake cmake -DENABLE_WERROR=1 -DCMAKE_INSTALL_PREFIX=D:\workspace\_work\1\s\artifacts\obj\mono\Browser.wasm.Debug\out -DCMAKE_INSTALL_LIBDIR=lib -DCMAKE_BUILD_TYPE=Debug -DGC_SUSPEND=preemptive -DENABLE_MINIMAL=jit,portability,sgen_major_marksweep_conc,sgen_split_nursery,sgen_gc_bridge,sgen_toggleref,sgen_debug_helpers,sgen_binary_protocol,logging,shared_perfcounters,interpreter,threads,eventpipe,qcalls -DENABLE_INTERP_LIB=1 -DDISABLE_ICALL_TABLES=1 -DDISABLE_CRASH_REPORTING=1 -DENABLE_ICALL_EXPORT=1 -DENABLE_LAZY_GC_THREAD_CREATION=1 -DENABLE_LLVM_RUNTIME=1 -DENABLE_METADATA_UPDATE=1 -DCMAKE_C_FLAGS=" -fexceptions -ID:\workspace\_work\1\s\.packages\microsoft.netcore.runtime.icu.transport\6.0.0-preview.3.21151.1/runtimes/browser-wasm/native/include" -DCMAKE_CXX_FLAGS=" -fexceptions"  D:\workspace\_work\1\s\src\mono\" exited with code 1.
    ##[error]src\mono\mono.proj(410,5): error MSB3073: (NETCORE_ENGINEERING_TELEMETRY=Build) The command "call D:\workspace\_work\1\s\src\mono\wasm\emsdk\emsdk_env.bat && emcmake cmake -DENABLE_WERROR=1 -DCMAKE_INSTALL_PREFIX=D:\workspace\_work\1\s\artifacts\obj\mono\Browser.wasm.Debug\out -DCMAKE_INSTALL_LIBDIR=lib -DCMAKE_BUILD_TYPE=Debug -DGC_SUSPEND=preemptive -DENABLE_MINIMAL=jit,portability,sgen_major_marksweep_conc,sgen_split_nursery,sgen_gc_bridge,sgen_toggleref,sgen_debug_helpers,sgen_binary_protocol,logging,shared_perfcounters,interpreter,threads,eventpipe,qcalls -DENABLE_INTERP_LIB=1 -DDISABLE_ICALL_TABLES=1 -DDISABLE_CRASH_REPORTING=1 -DENABLE_ICALL_EXPORT=1 -DENABLE_LAZY_GC_THREAD_CREATION=1 -DENABLE_LLVM_RUNTIME=1 -DENABLE_METADATA_UPDATE=1 -DCMAKE_C_FLAGS=" -fexceptions -ID:\workspace\_work\1\s\.packages\microsoft.netcore.runtime.icu.transport\6.0.0-preview.3.21151.1/runtimes/browser-wasm/native/include" -DCMAKE_CXX_FLAGS=" -fexceptions"  D:\workspace\_work\1\s\src\mono\" exited with code 1.

* Try use newer cmake

* Revert "Try use newer cmake"

This reverts commit bd95f346f9a3c5828924201c79df3b356043c11c.

* Normalize path

* Remove excessive space char

* Try newer emscripten version

To try if it helps with the cmake issue

* Try trim end '\' from directory passed to cmake

Also revert emscripten version back

* Provision emscripten only when targeting wasm

* Do not set $os, it is already set above

* Revert "Remove excessive space char"

This reverts commit b1c4da12a0c39e78b0cf02d1da1332a55cedf772.

* Provision emscripten only on windows

Co-authored-by: Steve Pfister <steve.pfister@microsoft.com>
Co-authored-by: Radek Doulik <radekdoulik@gmail.com>

Fix nested Multipart when generating MailMessage (#47403)

* Fix nested Multipart when generating MailMessage

When sending a mail with alternate views and attachments, attachments were nested a second time. Doing so made the generated mail unintelligible for some clients (Like Outlook).

Fix dotnet#47379

* Add test for MailMessage serialization

Co-authored-by: MihaZupan <mihazupan.zupan1@gmail.com>

Fix card marking stealing for regions (#49704)

Essentially the problem was that for regions, we have several lists of segments, one for each generation. Thus, the card_marking_enumerator logic needs provisions to restart enumerating with another segment list.

The changes fall into 3 categories:

- new method switch_to_segment to start enumerating a new segment list - called for a new generation
- changes to make the enumerator state consistent across one segment list ending and another starting
- added dprintf instrumentation so we can later reconstruct how different threads enumerated chunks.

Remove kTLSProtocol13_ForwardDef (#48762)

Use a single approach to detect presence of pseudo handle support. (#48623)

Update Microsoft.DotNet.Helix.Sdk to 6.0.0-beta.21166.5 (#49747)

Includes https://github.com/dotnet/arcade/pull/7103

Add obsolete attribute to DisablePrivateReflectionAttribute (#49550)

* Add obsolete attribute to DisablePrivateReflectionAttribute

* Attribute added to ref file

* fix ref file and attribute

* Add new diagnostic ID

* Fix obsolete attribute

* Fix obsolete attribute in ref file

* Add new obsoletion ID to documentation

* Removed all references of DisablePrivateReflection in src/coreclr

* Fixed error message of obsoletion

* Fixed documentation

* Removed reference of DisablePrivateReflection

* Add NoWarn for new obsoletion ID

* Add pragma to tests

* Fix error message of new obsoletion

* Fix new obsoletion description in documentation

Big-endian fixes for ilasm/ildasm (#49685)

* Add byte swap when accessing PublicKeyBlob in AsmMan::EndAssembly

* Fix access beyond end of pszString in DumpUnicodeString

* Remove unneccessary assert in _FillMDDefaultValue

Add msbuild task to generate binary runtimeconfig format (#49544)

* Add msbuild task to generate binary runtimeconfig format

* Update property name due to naming conversion.

* Fixed more formatting issue

* Fixed one more naming convention

* Update src/tasks/RuntimeConfigParser/RuntimeConfigParser.cs

Co-authored-by: Ryan Lucia <ryan@luciaonline.net>
* Update src/tasks/RuntimeConfigParser/RuntimeConfigParser.cs

Co-authored-by: Ryan Lucia <ryan@luciaonline.net>
* Update src/tasks/RuntimeConfigParser/RuntimeConfigParser.cs

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

* Update src/tasks/RuntimeConfigParser/RuntimeConfigParser.cs

Co-authored-by: Dan Moseley <danmose@microsoft.com>
* Fix error handling

Co-authored-by: Ryan Lucia <ryan@luciaonline.net>
Co-authored-by: Dan Moseley <danmose@microsoft.com>

[QUIC] Fix TestConnect (#49660)

* Mark all failing tests with ActiveIssue

* Fix TestConnect

Better windows only guard which can be removed by linker (#48594)

Also, remove superfluous test poking into an internal state to mimic
non-Windows test environment

Simplify ASCIIUtility test utility class (#49720)

* Simplify ASCIIUtility test utility class

The existing code seems to be just an elaborate way to do the same thing.

* Update ASCIIUtilityTests.cs

Handle RsaVerificationPrimitive failure (non-internal error) (#49741)

Update dependencies from https://github.com/dotnet/xharness build 20210315.2 (#49745)

[main] Update dependencies from dotnet/xharness

FileStream rewrite part II (#48813)

* introduce WindowsFileStreamStrategy

* introduce SyncWindowsFileStreamStrategy

* introduce AsyncWindowsFileStreamStrategy

* only DerivedFileStreamStrategy needs to have a reference to FileStream

remove finalizer from FileStream, keep it only in DerivedFileStreamStrategy and LegacyFileStreamStrategy

* use the new strategies when buffering is not enabled

* introduce BufferedFileStreamStrategy

* use the Legacy strategy by default for now, add tests for the other implementation

* Apply suggestions from code review

Co-authored-by: David Cantú <dacantu@microsoft.com>
Co-authored-by: Stephen Toub <stoub@microsoft.com>

Fix stress-http and stress-ssl builds on Linux (#49706)

CollectionsMarshal.GetValueRef(Dictionary) (#49388)

Big-endian fix for JsonClassInfo.GetKey (#49708)

* Big-endian fix for JsonClassInfo.GetKey

* Skip assertion in JsonClassInfo.GetKey on big-endian
  (code will not ensure the properties tested here)

* Update src/libraries/System.Text.Json/src/System/Text/Json/Serialization/JsonClassInfo.Cache.cs

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

Fix build on platforms without gsharedvt (#49710)

A `MonoMemoryManager*` parameter was added to `mono_arch_get_gsharedvt_call_info()` in https://github.com/dotnet/runtime/commit/34c68c7edf6d61c8ff3d2631003ed86353415479 that was missed in arch-stubs.c

ReadOnlySequence: Don't leak the flags that are on _startInteger and _endInteger. (#47969)

* ReadOnlySequence: Don't leak the flags that are on _startInteger and _endInteger.

* Typo fix.

* Added more tests + review fixes.

System.IO.Pipelines.PipeReader: Fix TryRead summary (#49603)

Add missing 'from' to the summary for TryRead.

Unify corerun test runner (#49529)

* Remove unused files.

* Rewrite corerun so that it runs on all platforms.
Remove the unixcorerun implementation but fold in
those its features.

Attach debugger is supported on Windows and macOS.
CORE_ROOT can now be set the same way on all platforms:
  --clr-path, CORE_ROOT, corerun dir.

* Dump the computed configuration for runtime if a failure to
  initialize or execute the entry assembly occurs.
Remove -v flag.

* Add debugger attach support for systems with procfs.

* Remove the corerun option for setting System.Globalization.Invariant.

rework ProxySetViaEnvironmentVariable_DefaultProxyCredentialsUsed test (#49348)