Merge pull request dotnet/coreclr#20409 from lol768/patch-1

Add a note to clarify debugging w/ dotnet is fine

Commit migrated from https://github.com/dotnet/coreclr/commit/d73e20f6cb12d7cf4c9909ff70a8920c978a1ce3

Updated "Viewing JIT Dumps" for .NET Core 3.0 (dotnet/coreclr#21687)

* Updated "Viewing JIT Dumps" for .NET Core 3.0

plus some minor wording change

* Incorporated changes from https://github.com/dotnet/coreclr/pull/21859

Commit migrated from https://github.com/dotnet/coreclr/commit/2d2caa80996562936b146fbf81ab228a21b1f7b7

Update CoreClr, CoreFx to preview-27308-03, preview.19058.1, respectively (dotnet/coreclr#21877)

Commit migrated from https://github.com/dotnet/coreclr/commit/937fa6ca4323f7ea6a8eb7a9aaf1667ea48a5a21

Merge pull request dotnet/coreclr#21880 from BruceForstall/FixTestReportsArchiving

Update test reports archiving

Commit migrated from https://github.com/dotnet/coreclr/commit/9037c3d8e224b5121bc103383d20b67be846a14a

Update test reports archiving

Two things:
1. Turn "echo" back on after runtests.cmd, so we see in the log file
the testReports.zip compression.
2. Change the archiving of testReports.zip to not occur even if the
test run fails (this is actually the most interesting case), and don't
fail if there is nothing to archive.

Commit migrated from https://github.com/dotnet/coreclr/commit/491c4ad85ba6cdfde84a45ee00b7b1ea823cd7a3

Fix issue dotnet/coreclr#20585 createdump explicitly uses /tmp. (dotnet/coreclr#21866)

Changed to PAL's GetTempPathA to get the temp path.

Commit migrated from https://github.com/dotnet/coreclr/commit/bd5ea1c46ca21631194610f88f2ba420e8982d5b

Fixed potential memory leak (dotnet/coreclr#21872)

Commit migrated from https://github.com/dotnet/coreclr/commit/d6bf5ed160e99b9a66087dcc8738f22a9a370121

Update CoreClr, CoreFx, PgoData to preview-27308-01, preview.19057.3, master-20190108-0040, respectively (dotnet/coreclr#21830)

Commit migrated from https://github.com/dotnet/coreclr/commit/7e5e65d212ef727918e7ad329f65ccf4ae38a865

Attempt to fix paths with spaces again (dotnet/coreclr#21837)

Commit migrated from https://github.com/dotnet/coreclr/commit/b3498fd9788651659b478ce4aad7c9b6bf4eafc2

Fix path length 255 (dotnet/coreclr#21838)

* Fix path length 255

* Fix assert

* Dead code

Commit migrated from https://github.com/dotnet/coreclr/commit/c885564832facf8a9cdcbe9baa1c7a18321cb9bd

Update ceemain.cpp (dotnet/coreclr#21869)

Commit migrated from https://github.com/dotnet/coreclr/commit/13c3af5bae229d426cfcd607e1bfecfb071df4a1

CoreRT updates for files moved to shared partition

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
Commit migrated from https://github.com/dotnet/coreclr/commit/b7f67c967a1acb311e7873b515edc92f74b658af

Moves ComTypes to shared partition (dotnet/coreclr#21861)

Commit migrated from https://github.com/dotnet/coreclr/commit/0a2ceb3fd146d9171b8c362aa8679d996d3e5bde

Fixing ContainCheckHWIntrinsic to ensure that scalar integer operands are the appropriate size (dotnet/coreclr#21641)

* Fixing ContainCheckHWIntrinsic to ensure that scalar integer operands are the appropriate size

* Adding a regression test for issue 21625

* Fixing IsContainableHWIntrinsicOp to use the containing node type (rather than the simd base type) for Scalar intrinsics

* Fixing the containment check for `Sse41.Insert(V128<float>, V128<float>, byte)`

* Cleaning up the isContainableHWIntrinsicOp logic in lowerxarch.cpp

* Restrict containment to genActualType(baseType)

* Formatting

* Removing some comments and simplifying the supportsContainment checks for various HWIntrinsics that take a scalar operand

* Applying formatting patch

Commit migrated from https://github.com/dotnet/coreclr/commit/ccaf35e1e0dc6bbfb86e4fe6b8c24fe8fe346e55

Check GenerateGuidForType argument for null (dotnet/coreclr#21851)

* Check GenerateGuidForType argument for null

* Also check for RuntimeType

Commit migrated from https://github.com/dotnet/coreclr/commit/c2ac0f95ae6d453634dbe42159ad1f740cbe0958

Collect Windows test run logs into testReports.zip file (dotnet/coreclr#21813)

* Collect Windows test run logs into testReports.zip file

And upload these to Jenkins, so we can see how the tests ran.

* Archive test Reports even if the test run fails

* Need to use "call" when invoking runtest.cmd batch file

Commit migrated from https://github.com/dotnet/coreclr/commit/59a2e255348ede17f1b57a88abc9fa04eeb2b556

Fix CharSet on StringBuilder hidden null terminator test (dotnet/coreclr#21853)

Commit migrated from https://github.com/dotnet/coreclr/commit/fee8d6ad6fc27073f88755dbc066d1aea8da5517

Actually fix issue dotnet/coreclr#21484 reported by the customer. (dotnet/coreclr#21850)

GC heap globals like ephemeral_heap_segment and finalize_queue are
null/invalid for a server GC. Add a check to skip the workstation GC
memory enumeration if server. The server memory enumeration already
skips if workstation GC.

Commit migrated from https://github.com/dotnet/coreclr/commit/101c99fdfe6c0c348fea615f6388d23a6e397fc6

JIT: encapsulate general checks for optimization (dotnet/coreclr#21822)

* JIT: encapsulate general checks for optimization

Add methods that answer the general question of whether or not
the jit is optimizing the code it produces.

Use this to replace composite checks for minopts and debug
codegen (the two modes where the jit is not optimizing).

* inline keyword cleanup

Commit migrated from https://github.com/dotnet/coreclr/commit/7d54c590315ad682ed74f9491654720d83532ae7

Respect the instrumented code maps passed by the profiler in a rejit (dotnet/coreclr#21786)

We currently do not respect the rejit provided instrumentation maps in DebuggerJitInfo::SetBoundaries, this change updates it to use the rejit map if one exists.

Commit migrated from https://github.com/dotnet/coreclr/commit/e8b934f0b52ecc019fee8ffaa48e658451a84996

Improve CustomAttribute (dotnet/coreclr#21832)

* Reduce local scopes in AddCustomAttributes

* CustomAttribute.cs tidy up

* Less struct copies

Commit migrated from https://github.com/dotnet/coreclr/commit/e5cb6c35fc18dcf2221f56ecb6a78f194c7fcd0c

Remove invalid test b21296 (dotnet/coreclr#21507)

This test was doing in-place modification of a managed string using
unsafe code which breaks string interning.

Commit migrated from https://github.com/dotnet/coreclr/commit/b3d8246ed6618dd5b291acc9347b07c0ed3a27da

Publish symbol packages (dotnet/coreclr#21769)

Commit migrated from https://github.com/dotnet/coreclr/commit/364ea875bbfb9065b15b1496e710c1baca03c1e8

Moves StackTrace to shared partition (dotnet/coreclr#21757)

* Moves StackTrace to shared partition

* Remove obsolete comment

* StackFrame::GetFileName behaves like property, remove SecurityException

* Adjust CoreRT ifdefs, fix names for consistency

Commit migrated from https://github.com/dotnet/coreclr/commit/d5c10c97f827354a83ef2915d31ac4e1c212662f

Fix missing project error in official build (dotnet/coreclr#21767)

In response to https://github.com/dotnet/arcade/pull/1567.
This uses an empty project to work around the new behavior that requires a project file even for restore operations.

Commit migrated from https://github.com/dotnet/coreclr/commit/c0d778417449b2e44ba2fedacd3046f1af755744

Fix incorrect assignment of READYTORUN_HELPER_[Dbl|Flt][Rem/Round] (dotnet/coreclr#21836)

Turns out there's a long-standing typo in CoreCLR that reverses
the helper enumeration values for DBL and FLT. This doesn't seem
to be a problem in the CoreCLR repo as such because it solely
uses the legacy identifiers CORINFO_HELP_DBLREM et al. We have
however ported the incorrect helper enumeration values into R2RDump
and ILCompiler. This change immediately fixes R2RDump as it resides
in the same repo, I'll send out the ILCompiler change in
a separate PR.

Commit migrated from https://github.com/dotnet/coreclr/commit/bcbb84df02d73a89dd13ea7fa2257f542202dd81

Remove CAS era security checks around resource loads (dotnet/coreclr#21825)

Commit migrated from https://github.com/dotnet/coreclr/commit/ee8bc3272391e46862dd8d27d3e086dc3bf953cf

Revert "Fix ilasm exec on paths with spaces (dotnet/coreclr#21819)" (dotnet/coreclr#21835)

This reverts commit dotnet/coreclr@b188e9d354e37e512c5b95bec38f0f06efb4161e.

Commit migrated from https://github.com/dotnet/coreclr/commit/c3a4a5b292d516ea214006ca1b00feb04e40ec1d

Delete redundant S.R.InteropServices.PInvokeMap and S.R.InteropServices.PInvokeMarshal (dotnet/coreclr#21831)

Commit migrated from https://github.com/dotnet/coreclr/commit/fcf2805fd887857ea727b7dd2658848427daaf9c

Remove redundant casts in TypeBuilder.CreateTypeNoLock (dotnet/coreclr#21824)

Commit migrated from https://github.com/dotnet/coreclr/commit/c239af7f3cfe1a62137a1c24a4228a69dd2df654

Maintain left.Equals(right) order for equality for compat (dotnet/coreclr#21829)

Commit migrated from https://github.com/dotnet/coreclr/commit/dda6df6ceaf0785c7bfd52fce3084b7f157c9588

Update BuildTools, CoreClr, CoreFx, PgoData to preview1-03603-01, preview-27305-02, preview.19055.1, master-20190105-0042, respectively (dotnet/coreclr#21749)

Commit migrated from https://github.com/dotnet/coreclr/commit/da2fe0efcb1f022087108abb9c6000561aee8751

(Cdotnet/coreclr#7) Use pattern matching `is` rather than `as` with null check (dotnet/coreclr#21828)

* Use pattern matching `is` rather than `as` with null check

Commit migrated from https://github.com/dotnet/coreclr/commit/d0cdf8813ef099deec3b6940e34f20d1bba054b8

inline keyword cleanup

Commit migrated from https://github.com/dotnet/coreclr/commit/a24b7a75d2f0b262564c10d69d0db27e3062c61a

Move Native Library APIs to NativeLibrary class (dotnet/coreclr#21821)

Move APIs that manipulate native libraries from
System.Runtime.InteropServices.Marshal class to the new
System.Runtime.InteropServices.NativeLibrary class.

In API review: dotnet/coreclr#32015,
The LoadLibrary APIs were originally Approved to live in
System.Runtime.InteropServices.Marshal class.

However, recently the decision was changed such that the APIs are
in a new NativeLibrary class.

Commit migrated from https://github.com/dotnet/coreclr/commit/110835b1b818b333f27ab76db3f223a03027698a

Remove redundant unsafe contexts (dotnet/coreclr#21826)

Commit migrated from https://github.com/dotnet/coreclr/commit/dcf9f36c21022b8fce374559ec5fea8d11a0852b

Fix ilasm exec on paths with spaces (dotnet/coreclr#21819)

Commit migrated from https://github.com/dotnet/coreclr/commit/b188e9d354e37e512c5b95bec38f0f06efb4161e

JIT: encapsulate general checks for optimization

Add methods that answer the general question of whether or not
the jit is optimizing the code it produces.

Use this to replace composite checks for minopts and debug
codegen (the two modes where the jit is not optimizing).

Commit migrated from https://github.com/dotnet/coreclr/commit/4c68562cbd2f7a33bad60705db9b11e3121d40c2

Fix misspelling (dotnet/coreclr#21640)

Commit migrated from https://github.com/dotnet/coreclr/commit/83bd6ca494971c399b3fbebee4623d9bf294731d

Remove stack marks from GetSatelliteAssembly. (dotnet/coreclr#21812)

Commit migrated from https://github.com/dotnet/coreclr/commit/6e986f5ed76701109f83d50df1da53ddb23624ab

Delete unused fFullReset argument (dotnet/coreclr#21814)

Commit migrated from https://github.com/dotnet/coreclr/commit/2301baeda250ecbc557430dca7ea70afdc968d52

Avoid passing stack crawl mark unnecessarily deep in the call stack. (dotnet/coreclr#21783)

* Avoid passing stack crawl mark unnecessarily deep in the call stack.

* Move stack crawl from TypeName::GetTypeManaged to RuntimeTypeHandle::GetTypeByName.

Commit migrated from https://github.com/dotnet/coreclr/commit/2ee2cb3a35fe1fa8d2d3b54c6839368ab8c89b73

If the debugger is stopped on a garbage collection event, resuming the threads during detach should simply be setting the event (dotnet/coreclr#21764)

Commit migrated from https://github.com/dotnet/coreclr/commit/509419356160fc419a6bbf1b70f5efb8e1dc4d10

Merge pull request dotnet/coreclr#21806 from BruceForstall/CheckTestEnvCall

Check result of call to TestEnv script

Commit migrated from https://github.com/dotnet/coreclr/commit/0234d6bcfe3c7cf98ba6dbd9d23cdbdcdd0d4a04

Support faster null checks (dotnet/coreclr#21765)

Commit migrated from https://github.com/dotnet/coreclr/commit/4e82a5801bd804bff0bf6294d7135bbb63df92f6

JIT: don't optimize struct copies for call args in debug or minopts (dotnet/coreclr#21792)

The jit will opportunistically optimize away copies for some struct call
arguments, if that argument is a "last use" for the struct (and some other
conditions apply).

In cases like dotnet/coreclr#21544 this leads to confusing debug experiences as inputs
to a call appear to be modified by the call.

Also we really should not be optimizing the code this way in debug or in
minopts codegen modes. So, block this optimization for debug and minopts.

Closes dotnet/coreclr#21544.

Commit migrated from https://github.com/dotnet/coreclr/commit/df4a1854aa42c1e97874b00fe49339e216e30af9

Fix Windows arm32/arm64 testing (dotnet/coreclr#21809)

* Fix Windows arm32/arm64 testing

After the conversion to runtest.py, we were not getting any
JIT or GC stress mode testing:
1. The generated test env batch file didn't have a ".bat" extension,
so Windows wouldn't execute it.
2. The Python generated temp file was locked, so couldn't be read.

Added more logging.

Commit migrated from https://github.com/dotnet/coreclr/commit/f0c969a072c05c0d602d16d68bc6e130fd17be88

Specify value of __RuntimeId for non-portable Linux test builds in build-test.sh (dotnet/coreclr#21777)

Commit migrated from https://github.com/dotnet/coreclr/commit/3eb581cf79f0cf195d6395fafd5a8b355a602242

Add StringExtensions.Concat to fix Microsoft.IO.Redist assembly

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
Commit migrated from https://github.com/dotnet/coreclr/commit/c90d7ec04e9167cb48d0eb4e49f6b5b6828b2d3a

EventPipePayloadDecoder clean-up (dotnet/coreclr#21716)

* Remove ReadUnalignedGuid

* Make EventPipePayloadDecoder unconditionally little-endian

Commit migrated from https://github.com/dotnet/coreclr/commit/606e36b862a1fa74c04abdd38b8b19752bd29280

Removes AppDomain dependency from TPLETWProvider (dotnet/coreclr#21709)

which always returns 1, to make code sharing easier

Commit migrated from https://github.com/dotnet/coreclr/commit/07f1fcce9999990763638d9b545b4c78eb79bd61

Skip additional cast from common EC.Run calls (dotnet/coreclr#21789)

* Skip additional cast from common EC.Run calls

* Unsafe.As

Commit migrated from https://github.com/dotnet/coreclr/commit/027a9105ae2783a7fffd6038bf07a5cf67ba287b

Reflection performance tweaks & cleanup (dotnet/coreclr#21737)

- Delete internal "Unsafe" field getters/setters. They performance was very close (within 10%) of the regular getters/setters.
- Made a few performance tweaks in reflection to make reflection faster overall. The regular field getters/setters are faster than what the unsafe ones used to be with this change.

Commit migrated from https://github.com/dotnet/coreclr/commit/a6269391cf04c656e9d472e1101e1ec670ecd605

Use ValueStringBuilder instead of StringBuilderCache in PathInternal.Windows.cs

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
Commit migrated from https://github.com/dotnet/coreclr/commit/7d67f73b7223620b709b2c7e33e0f6e3cef5a7e8

Add test for StringBuilder null terminator implementation detail (dotnet/coreclr#21800)

* Remove some commented out code.

* Add test verifying that we put a null terminator 2-past the end of the native buffer allocated for a StringBuilder.

Commit migrated from https://github.com/dotnet/coreclr/commit/08019ac94eb8cd57abad4d03c803371290cfe210

Merge pull request dotnet/coreclr#21803 from BruceForstall/FixTypo

Fix batch script errors running JitOptimizationSensitive tests

Commit migrated from https://github.com/dotnet/coreclr/commit/9f446067444fb0218b4504e2ab6921fd7492d4a2

Check result of call to TestEnv script

If you give an illegal, not executable TestEnv script, then this
will not fail silently.

Commit migrated from https://github.com/dotnet/coreclr/commit/118383661a30bfad6f6c45f2cae034ea05ec3e61

Fix batch script errors running JitOptimizationSensitive tests

Simple typo: REN instead of REM, leading to:

"The syntax of the command is incorrect."

Commit migrated from https://github.com/dotnet/coreclr/commit/b68b45451271c270c643ba4d96acf2a72e331647

Use Kernel32 instead of api-ms-win-core-* in DllImports in SafeFileHandle.cs

Commit migrated from https://github.com/dotnet/coreclr/commit/90a295c24f7f8764ea56670955340e2ddc764b00

Eliminate some bounds checks in Path (dotnet/coreclr#21721)

Commit migrated from https://github.com/dotnet/coreclr/commit/0a294abdcff0281da1ad8eeb450b4e812030b269

Adds portable version of EncodingTable (dotnet/coreclr#21735)

* Adds portable version of EncodingTable

Most of the implementation is extracted from CoreRT

* Use string comparer directly

* Remove no longer used COMNlsInfo

* Adds localization support

* Removes FeatureCoreFxGlobalization configuration

* Remove redudant encodings look up from GetEncoding

* Keep Hashtable for nameToCodePage as it does not lock on read

* Replace locked dictionary lookup with short switch

* Include comment with msbuild task link used to generate the data file

Commit migrated from https://github.com/dotnet/coreclr/commit/8d2f4ed72a9d48164a4c8147fe7911e62a5d9594

Merge pull request dotnet/coreclr#21619 from CarolEidt/FixDesktopSIMD

Fix desktop SIMD failures

Commit migrated from https://github.com/dotnet/coreclr/commit/10c3e60c863d01f638daa240af957c3c5791cda5

Merge pull request dotnet/coreclr#21776 from fiigii/fixAs

Fix Vector128/256 As<U> intrinsic recognition and exception

Commit migrated from https://github.com/dotnet/coreclr/commit/4f704d172863b50492a4a4254691fe1374a55b62

Improve performance of Path.ChangeExtension (dotnet/coreclr#21766)

In the common case where it need to replace a non-empty extension with a non-empty extension, it currently incurs a substring without the original extension prior to then concatenating with the new extension.  This PR avoids that.

(As the Path implementation is in corelib, this uses string.FastAllocateString and then formats into it with a span; if we wanted to avoid that, string.Create could also be used.)

* Add internal String.Concat overloads for spans

* Use span-based string.Concat overloads in several places

Commit migrated from https://github.com/dotnet/coreclr/commit/8f91ac8f374d30cd1a1122f78a90c35d7827a64a

Merge pull request dotnet/coreclr#21774 from hoyosjs/juhoyosa/event_size

Add payload size case for GC events

Commit migrated from https://github.com/dotnet/coreclr/commit/48b94195efc8bc607660ec957c481bc7e13ccf62

Fix Vector128/256 As<U> intrinsic recognition and exception

Commit migrated from https://github.com/dotnet/coreclr/commit/ffce0825f794bd7d849a014ff30d71b0f2388781

Fix GC for collectible array of pure value types (dotnet/coreclr#21782)

For collectible arrays, we enter the mark_object_simple1 method even if
the array contains no pointers so that the reference to LoaderAllocator
can be walked. But we were missing skipping the call to go_through_object
which is both not necessary and it also doesn't work correctly for
objects without pointers (the loop over the fields reports the first 8
bytes of each array element as if it was a pointer).
The fix is to skip the call to go_through_object for objects without
pointers in the mark_object_simple1 and background_mark_simple1.

Commit migrated from https://github.com/dotnet/coreclr/commit/03ab0ca0f81313319f30cfd75ee56dc59d71390c

Reduce allocation in Index/Range.ToString (dotnet/coreclr#21755)

* Reduce allocation in Index/Range.ToString

* Address PR feedback

Commit migrated from https://github.com/dotnet/coreclr/commit/88eb93abba27e97d5997ad2d6c04b54aeaff5e8f

Remove vc-runtime package dependency (dotnet/coreclr#21643)

Commit migrated from https://github.com/dotnet/coreclr/commit/b20917f552cfc6f502648f9342dbdfd7ac0ad91e

Bring Debug CRT DLLs when build IJW tests (dotnet/coreclr#21699)

Commit migrated from https://github.com/dotnet/coreclr/commit/8cf62cb77230389c4744b91f1c6ddd60331674ad

Removes the code moved to shared partition

Commit migrated from https://github.com/dotnet/coreclr/commit/6eecaafdeb8d5870f06f99905b57f260b3c09131

Moves Attribute instance bits to shared location (dotnet/corertdotnet/coreclr#6747)

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
Commit migrated from https://github.com/dotnet/coreclr/commit/8e36b4aba1d63fda1597c365adf9b7b440cf1370

Disable the [Intrinsic] check for desktop SIMD

Commit migrated from https://github.com/dotnet/coreclr/commit/c5a4a390e53cb30cc5b42f9dc4fb80c5ca579534

Fix desktop SIMD failures

Fix ISA checking to work with desktop.
Also fix Value Numbering jitdump to not assert for struct types without a handle.

Commit migrated from https://github.com/dotnet/coreclr/commit/de287410d790bc93668d4d01a202494087af8885

Cleanup current culture handling in the unmanaged runtime (dotnet/coreclr#21706)

Large portion of the current culture handling in the unmanaged runtime inherited from desktop has been no-op. The nativeInitCultureAccessors QCall that it used to depend on desktop got (almost) never called in CoreCLR.

- Delete resetting of current culture on threadpool threads. It was needed in desktop because of a very tricky flow of current culture between appdomains. It is superseded by the flowing the current culture via AsyncLocal in CoreCLR.
- Comment out fetch of managed current culture for unmanaged resource lookup. It has number of problems that are not easy to fix. We are not localizing the unmanaged runtime currently anyway, so it is ok to just comment it out.
- Fix the rest to call CultureInfo directly without going through Thread.CurrentThread

Commit migrated from https://github.com/dotnet/coreclr/commit/eff427c02a89e135c38a96032feb3c9a6a13cf5b

Remove substring allocation in MethodBase.AppendParameters (dotnet/coreclr#21770)

Commit migrated from https://github.com/dotnet/coreclr/commit/faf2acefa7d88a446f561b38740a72d8838440a5

Remove unnecessary propagation of stack crawl marks. (dotnet/coreclr#21781)

Commit migrated from https://github.com/dotnet/coreclr/commit/6f9121bf9e51472780e25d7f50b8c9e46898ad50

Merge pull request dotnet/coreclr#21669 from kvochko/arm_cfi_code_offsets

[Linux/ARM] Fix CFI code offsets in funclet prolog

Commit migrated from https://github.com/dotnet/coreclr/commit/acbfc3d9b0729cc5c2957d5536a894c07b1c4af8

Refactor Send to Helix step in Azure DevOps (dotnet/coreclr#21700)

* Move all the test job submission logic into separate file eng/send-to-helix-step.yml.

* Pass IsExternal=true and Creator for public submissions.

* Pass IsExternal=false and HelixAccessToken for internal submissions.

* Pass the parameters to MSBuild via environment.

* Make all the properties names passed to MSBuild "from outside" starting with underscore.

* Specify HelixArchitecture and HelixConfiguration properties.

Commit migrated from https://github.com/dotnet/coreclr/commit/0d128bc36e4de05da32aacca017574be812066df

Avoid some unnecessary allocation in MethodBuilder.ToString (dotnet/coreclr#21758)

This method already allocates a ton, and it's not going to be hot-path, but it's annoying me showing up in queries for string.Concat usage, so I'm fixing it :)

Commit migrated from https://github.com/dotnet/coreclr/commit/472181f672b21774109f8d3cdcd077556d7393b2

Replace string.Format(SR. with SR.Format(SR. (dotnet/coreclr#21763)

We use SR.Format in almost all places where we format resource strings (without a format provider), but there were a few stragglers.

Commit migrated from https://github.com/dotnet/coreclr/commit/369f53ea121798a65b81e6878c739c6bdcaae895

Change a couple string.Replace("a", "b") to string.Replace('a', 'b') (dotnet/coreclr#21768)

The latter is functionally the same but less overhead.

Commit migrated from https://github.com/dotnet/coreclr/commit/922971c5e2a0e74841218dbd8e00f6354ab2b98f

Use Alpine container with Node.js in Azure DevOps (dotnet/coreclr#21568)

* Use Alpine container with Node.js in azure-pipelines.yml
* Enable the dependsOn build_Linux_musl_x64_release in azure-pipelines.yml

Commit migrated from https://github.com/dotnet/coreclr/commit/9f108aad51eb94c3a0c3118481e687543989fa59

Use StringBuilderCache in PathInternal.NormalizeDirectorySeparators (dotnet/coreclr#21760)

When we do need to normalize, we're currently allocating the StringBuilder/char[] for the full path length.  As long as the path length is less than the max cacheable size (360), we'll now use a cached builder.

Commit migrated from https://github.com/dotnet/coreclr/commit/9d159eceee051b9536a26467adb489a6a5c0fa7a

Additional ifdef changes which are ES_BUILD_STANDALONE specific for System.Diagnostics.Tracing (dotnet/coreclr#21748)

Commit migrated from https://github.com/dotnet/coreclr/commit/e2a576a7130978dfeca4c100868789eee4adc403

Replace Array.Copy(src, dst, int) calls with Array.Copy(src, 0, dst, 0, int) (dotnet/coreclr#21756)

The former calls GetLowerBound(0) on both src and dst, which is unnecessary when the arrays are T[] and thus always have a lower bound of 0.

Commit migrated from https://github.com/dotnet/coreclr/commit/6911791e4614e900fa756ce6fd5348071278229f

Remove dummy globalization implementation. (dotnet/corertdotnet/coreclr#6756)

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
Commit migrated from https://github.com/dotnet/coreclr/commit/e7ef2ecadc31b1da4196fdb66a5f0485d1bfdf2c

Avoid boxing allocation and object[] allocation in BuildTraceFileName (dotnet/coreclr#21752)

Commit migrated from https://github.com/dotnet/coreclr/commit/0c5ab47942683e87ddc0997636f427d8bd9eb3ef

Temporary remove OSX.1012.Amd64.Open and OSX.1012.Amd64 Helix queues in eng/platform-matrix.yml (dotnet/coreclr#21642)

Related issue: https://github.com/dotnet/core-eng/issues/4856

Commit migrated from https://github.com/dotnet/coreclr/commit/3b6d990eef152bac4134318940b08b050982c0f0

Delete lstrlen from Unix PAL (dotnet/coreclr#21745)

strlen/wcslen works just fine.

Commit migrated from https://github.com/dotnet/coreclr/commit/a082f4702540309e832c2ebdf86b15dc798b136b

Remove second 'license' word for readability (dotnet/coreclr#21747)

The license name already has 'license' in the end, the second one is unnecessary.

Commit migrated from https://github.com/dotnet/coreclr/commit/d26e31a51d3a97ac925240c134d08418911c457d

Simplify and improve integer overflow checks in Interop (dotnet/coreclr#21732)

- Delete unnecessary CheckStringLength calls for result of string.Length. Managed strings are guaranteed to be under 2GB bytes, so these checks were unnecessary.
- Add `checked(...)` around buffer size computations that may hit potential integer overflow. It does not look like any of these would cause a bug that would lead to buffer overrun, but it is better to catch these early.

Commit migrated from https://github.com/dotnet/coreclr/commit/a5b1c68d4bb8a14042e93acb8f1032db4703b943

Update CoreClr, CoreFx to preview-27302-01, preview.19051.1, respectively (dotnet/coreclr#21674)

Commit migrated from https://github.com/dotnet/coreclr/commit/ccaba224b69cc3c334e73226e592dde5864afc14

AppContext moved to shared partition - CoreRT specific changes

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
Commit migrated from https://github.com/dotnet/coreclr/commit/0de73d34e08f767b118e95fe88e8e98b1fa31379

strlen to managed code and vectorize (dotnet/coreclr#21729)

Commit migrated from https://github.com/dotnet/coreclr/commit/4a7a2049a1a985b02fe5983e7e2a8d480270a72b

Cleanup ArgumentNull_Obj references in unmanaged runtime (dotnet/coreclr#21738)

Commit migrated from https://github.com/dotnet/coreclr/commit/6a3f7e1903c503165b1fbf2e82d2e777453ca8e5

Tweaks permissions ifdefs in System.Diagnostics.Tracing (dotnet/coreclr#21733)

Commit migrated from https://github.com/dotnet/coreclr/commit/c6c48260173ab7d22d599e60f7ca98e0cfc06ec4

Moves AppContext to shared location (dotnet/coreclr#21734)

Commit migrated from https://github.com/dotnet/coreclr/commit/0a03aa9b81eeac93a6c6fa238d4123a60009d0f6

Delete useless ArgumentNullException message (dotnet/coreclr#21728)

"Object cannot be null" does not add anything over the default "Value cannot be null"

Commit migrated from https://github.com/dotnet/coreclr/commit/a1b757c70f47dbac8a4ca04d2d7861a8a0442e56

Remove EventPipeController static constructor (dotnet/coreclr#21718)

* Remove EventPipeController static constructor

* Don't cache DefaultProviderConfiguration

Commit migrated from https://github.com/dotnet/coreclr/commit/73d7d224ae6fb9b13bb641440c06606f426215cb