Add asserts for the types of the Span indexers. (dotnet/coreclr#21037)

Also, add a test for dotnet/coreclr#20958

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

Fix search for headers and libraries via LLVM_HOME (dotnet/coreclr#21084)

Commit b89e2305 (PR dotnet/coreclr#18873) renamed WITH_LLDB_LIBS to LLDB_LIB_DIR and
WITH_LLDB_INCLUDES to LLDB_INCLUDE_DIR. But it forgot to update the
usage of these variables in some find_library and find_path calls.

As a result, setting LLVM_HOME is no longer enough to compile coreclr on
platforms with non-standard locations for lldb. This commit fixes that.

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

Fix Getting Max Japanese Era (dotnet/coreclr#21087)

We were using the ICU API ucal_getLimit and askin gto get the maximum value of the Japanese eras. it looks this API is just return the era matching the current system clock which prevent returning any future defined era in the system (which ICU 63.1 can support with some environment variable).
We raised the issue to the ICU owners but would be better just to get the era of Gregorian year 9999 which always should return the max era anyway.

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

Disable stale sockets test (dotnet/coreclr#21083)

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

Cleanup RtType.cs code

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

Fix Type.GetInterface for null namespace and non-null interface namespace

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

Add headers to crossgen, cee_crossgen, corzap_crossgen vcxproj files (dotnet/coreclr#20082)

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

Make type comparisons more general purpose (dotnet/coreclr#20940)

This has two parts:

## Part 1
CoreRT represents native type handles differently from CoreCLR - on CoreCLR, `RuntimeTypeHandle` is a wrapper over `RuntimeType` and RyuJIT is aware of that. On CoreRT, `RuntimeTypeHandle` wraps the native type handle, not a `RuntimeType`.

The knowledge is hardcoded in importer when importing the sequence "ldtoken foo / call Type.GetTypeFromHandle" - importer just removes the call and bashes the result of ldtoken to be a reference type. CoreRT had to avoid reporting `Type.GetTypeFromHandle` as an intrinsic because of that.

I'm adding another helper that lets RyuJIT avoid hardcoding that knowledge. Instead of just bashing the return type, we swap the helper call.

## Part 2

Native type handle equality checks need to go through a helper, unless the EE side says it's okay to compare native type handles directly.

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

Single pass Vectorize Span<byte>.IndexOfAny(..., ReadOnlySpan<byte>) for 2,3 lengths (dotnet/coreclr#20738)

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

Add headers to all md*, v3binder*, and ceefgen vcxproj (dotnet/coreclr#20163)

Work toward dotnet/coreclr#14884

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

Add ICustomMarshaler tests (dotnet/coreclr#19195)

* Add ICustomMarshaler tests

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

 Fix incorrect exception thrown in Marshal.ReleaseComObject(null) (dotnet/coreclr#21072)

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

Removing the legacy helper intrinsics and adding tests for their replacements (dotnet/coreclr#20994)

* Removing helper intrinsics from the x86 APIs in S.P.Corelib

* Removing JIT support for the removed x86 helper intrinsics

* Removing the x86 HardwareIntrinsics tests for the removed helper APIs

* Fixing up existing usages to no longer use the removed x86 helper intrinsics

* Skip CoreFX tests dependent on the removed x86 helper intrinsics

* Adding a GenerateTests.csx and templates for the new shared helper intrinsics

* Generating the new shared helper intrinsics from their templates

* Disabling some tests for arm and arm64 that are failing due to an assert

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

Force Unicode in test_location parameter in delete_existing_wrappers() (dotnet/coreclr#21056)

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

Optimization: Further reduce allocations in GetCustomAtttributes() paths (dotnet/coreclr#20795)

* Remove PseudoCustomAttribute allocations

Inside CustomAttribute, PseudoCustomAttribute is the mechanism used to return Attribute instances for what are really flags and such underneath in most cases. This is just an implentation detail, but this detail creates an intermediate array just for passing things around inside CustomAttribute...that's not necessary. This replaces the Attribute[] allocations with the ListBuilder<Attribute> struct for accumulating and relaying this attributes to parent methods.

Many overloads had a lot of duplicate coalescing code on this path as well...DRYed up that series of methods.

This change reduces allocation overhead for all calls by 40 bytes. In the case of a miss where none would be returned (e.g. .GetCustomAttributes<Attribute>() on a non-attributed class), this reduces allocations down to zero.

* CustomAttribute: split GetCustomAttributes into total and additive use cases

The split here was simple, but allocated an extra intermediate array for each parent type when searching for attributes in the inheritance case. Instead of doing that, we now move that to the second use case exclusively (which uses/returns the array created). The other use case adds to the ListBuilder<object> struct to accumulate through the inheritance hierarchy crawl (rather than an after-the-fact copy).

Note some parameter removals on the filter side...those weren't used already (and got in the way of this change).

* CustomAttributes: Cleanup of unused ops

Just removing the cruft that isn't actually used in here anymore (some of it in quite some time). Trivial amounts of speed gain, but every little bit helps!

* Param typo and red passing for GetCombinedList()

* Assume zero pseudo attributes by default (the most likely case)

Thsi eats the resize on the > 1 case, but that's so rare it's a net win to start with 0 here.

* Pseudo attributes: use out instead of return

A bit if a performance boost from the struct copy removal

* Pseudo attributes: switch to out everywhere and remove uneeded code paths

In relation to dotnet/coreclr#20795, these old calls seem to be for consistency but they're not likely to change and a comment (I think) accomplishes the same goal with less code...so here's a commit demonstrating it.

* Comment spacing for pseudo attribute fetching

Make it clear that pseudo attributes are a distinct thing (e.g. why not in *this* method) than the return line.

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

Exclude GC stress failure test. (dotnet/coreclr#21059)

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

Exclude tests from GCStress failing due to 21044 (dotnet/coreclr#21060)

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

Merge pull request dotnet/coreclr#21004 from BruceForstall/UpdateRuntestHelp

Update help output of runtest.cmd/sh

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

Add PInvoke/Decimal tests (dotnet/coreclr#19301)

* Add PInvoke/Decimal Tests

* Remove tests that fail on both .NET Framework and .NET Core.

* Refactor away the Windows-only headers.

* Make Decimal tests build on Linux.

* Remove some dead code.

* Clean up formatting.

* Don't run Decimal <--> Currency tests off-Windows.

* Make calling conventions and parameter/return marshalling more explicit

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

Delete unnecessary StackCrawlMarks in RtFieldInfo (dotnet/coreclr#21054)

Left over from Code Access Security

Fixes dotnet/coreclr#20965

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

Fix unloadability races (dotnet/coreclr#21024)

* Fix LoaderAllocator::AllocateHandle

When another thread wins the race in growing the handle table, the code
was not refreshing the slotsUsed local to the new up to date value. This
was leading to overwriting / reusing a live handle.
This change fixes it.

* Embed ThreadLocalBlock in Thread

Instead of allocating ThreadLocalBlock dynamically, embed it in the
Thread. That solves race issue between thread destruction and
LoaderAllocator destruction. The ThreadLocalBlock could have been
deleted during Thread shutdown while the LoaderAllocator's destruction
would be working with it.

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

Fix Formatting Japanese First Year on Linux (dotnet/coreclr#21039)

When formatting the first year with Gannen symbol we are checking if the year is followed by single quote which this is the case on Windows but it is not neccessay the case on Linux

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

Add workaround for https://github.com/dotnet/coreclr/issues/21051

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

Add explicit signatures for Span indexer

Fixes dotnet/coreclr#21032 and dotnet/coreclr#20958

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

Eliminate CMAKE dependency when not required (dotnet/coreclr#21040)

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

Add simple tests for ArrayWithOffset marshaling. (dotnet/coreclr#21013)

* Add simple tests for ArrayWithOffset marshaling.

* PR Feedback.

* Fix visibility

* Fix DllImport library name

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

Add Interop/PInvoke/Attributes tests (dotnet/coreclr#19107)

* Add Interop/PInvoke/Attributes tests

* Fix PR comments

* Fix compile warnings

* Remove comment

* Clean up test files to match current build system.

* Fix up test and make it windows only (as FEATURE_LCID is only enabled on Windows)

* Fixed string const issues.

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

Add PInvoke/CriticalHandles tests (dotnet/coreclr#19297)

* Add PInvoke/CreticalHandles tests

* Bring up to current infrastructure.

* Fix ReverseTest.

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

Delete HAS_FLS_SUPPORT and related code (dotnet/coreclr#21035)

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

Fixed signatures of ref readonly methods in JIT ETW events (dotnet/coreclr#20981)

* fixed ETW method signature of ref readonly methods

* added type token

* extract method for printing classes

* fixed optional_modifier, use CorSigUncompressToken_EndPtr

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

Add PInvoke/SafeHandles tests (dotnet/coreclr#19330)

* Add PInvoke/SafeHandles tests

* Fix compile warnings - include header file with relative path

* Run SafeHandle tests only on Windows. The tests are very Windows-centric, so we'd have to do quite a bit of a re-write to get them working xplat.

* Fix CMake project references.

* Fix interface CMakeLists project reference.

* Add casts for string literals.

* Use LPOLESTR and the W macro instead of OLECHAR* cast and L prefix.

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

Add Tests for VBByRefStr marshalling (dotnet/coreclr#20982)

* Add tests for VBByRefStr marshalling.

* Compile native for VBByRefStr on all platforms.

* Fix encoding.

* PR Feedback.

* Fix PInvokeDefs file.

* Fix visibility

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

Add tests for marshalling as VARIANT_BOOL or VARIANT (dotnet/coreclr#20856)

* Add native side impl for VARIANT_BOOL test.

* More bool native code.

* Add VariantBool marshalling tests.

* Add infrastructure for tests for object -> VARIANT marshalling.

* Add variant tests for all built-in types.

* Clean up Variant tests.

* Add test for passing custom structure as object.

* Add test for VARIANT_BOOL byref.

* Added byref tests for VARIANT marshalling.

* Add field marshalling tests for VARIANT.

* Move ByValue tests to not have two sources of truth.

* Make managed code the one source of truth.

* Clean up formatting/unused methods.

* Clean up some whitespace issues.

* Add tests for BStrWrapper and CurrencyWrapper marshalling.

* Add UnknownWrapper tests.

* Add a test for VariantWrapper and PInvoke.

* PR Feedback.

* Remove ToString.

* Fix visibility

* Fix MSBuild import path.

* Clean up Variant tests.

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

Update BuildTools, CoreClr, CoreFx, CoreSetup, IbcData, PgoData to preview1-03415-01, preview-27115-03, preview.18565.4, preview-27115-04, master-20181115-0045, master-20181115-0044, respectively (dotnet/coreclr#21005)

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

Add PInvoke/SizeParamIndex tests (dotnet/coreclr#19348)

* Add PInvoke/SizeParamIndex tests

* Fix compile warnings

* First pass cleaning up native code.

* Get ParamSizeIndex tests working xplat (excluding the BSTR array tests)

* Fix incorrect definition of ULONG off-windows.

* Disable reverse-PInvoke throwing test off-Windows.

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

Stop using Hostx86/arm crossgen in crossgen_comparison scenario (dotnet/coreclr#21021)

* And also create Linux arm64 crossgen_comparison scenarios in netci.groovy

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

Add PInvoke/DllImportpath tests (dotnet/coreclr#19270)

* Add PInvoke/DllImportpath tests

* Refactor test to current toolset and remove Windows-only Path API searching.

* Remove explicit calling convention from managed and add correct convention to test.

* Get environment test passing on Unix.

* Update test to pass off-Windows.

* Include feature/platform defines in tests.

* Fix environment test (again).

* Remove zero-width character that's breaking CI

* Fix zero-width character I missed.

* Remove invalid parens around LD_LIBRARY_PATH

* Ensure the path added to the LD_LIBRARY_PATH exists before the test starts.

* Combine all DllImportPath CMakeLists.txt files into one file.

* Include clr.coreclr.props when also including clr.defines.targets so we get the feature defines correctly as well.

* Disable unicode test for our OSX 10.12 CI builds.

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

Add tests for ANSI BSTRs (dotnet/coreclr#20985)

* Add tests for ANSI BSTRs

* Fix signed/unsigned mismatch in comparisons. Add xplatform include

* Use regular memcpy

* Enable ANSI BSTR off-windows.

* Also enable UnmanagedType.TBStr off-Windows since both BStr variants are available off-Windows.

* Make sure the AnsiBSTR marshaller is hooked up off-Windows.

* Remove a few more ifdefs I missed.

* remove ifdefs around binder definitions and the C# stub helpers.

* PR Feedback.

* Add xplat ARRAYSIZE macro

* Move xplat ARRAYSIZE to platformdefines.h

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

Merge pull request dotnet/coreclr#21008 from briansull/new-toolset

Fix warnings from the new toolset

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

Fixes for Azure DevOps official build (dotnet/coreclr#21017)

* Fix endpoint for internal jobs

 Fix external vs internal queues

 Properly quote scenarios argument to msbuild

 Increase queue timeout since most P1 jobs couldn't finish in time

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

Allow rune enumeration from string and ROS<char> (dotnet/coreclr#21007)

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

Add PInvoke/Array tests (dotnet/coreclr#19266)

* Add PInvoke/Array tests

* Fix compile warnings

* Get tests passing on Windows again.

* Change error_t type to work xplat.

* Change error_t in types.h as well

* Clean up build slightly.

* PR feedback and native test code refactoring.

* Clean up header.

* Add a xplat implementation of _itoa_s for our tests.

* CHAR -> char

* Make calling convention cdecl to avoid stdcall mangling.

* Make return test native code cdecl as well (missed it before).

* Fix bug in platformdefines _itoa_s

* Return early in TP_itoX_s when num == 0

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

Update CrstTypeTool to produce spaces (dotnet/coreclr#21029)

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

Changed back to use m_filtered

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

Code Review feedback

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

Combine ExcludeList in AltJit scenarios for both host and altjit archs (dotnet/coreclr#21018)

* Add -altjitarch command line argument in tests/runtest.cmd and -altjit_arch in tests/runtest.py

* Specify altjitarch in for x86_arm_altjit and x64_arm64_altjit jobs in netci.groovy

* Use AltJitArch MSBuild variable in tests/issues.targets to exclude tests based on either BuildArch or AltJitArch since both two JITs are running in altjit scenarios

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

Add PInvoke/Miscellaneous tests (dotnet/coreclr#19326)

* Add PInvoke/Miscellaneous tests

* Clean up HandleRef test.

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

Update Unicode data and optimize CharUnicodeInfo indexes (dotnet/coreclr#20983)

* Update Unicode data and optimize CharUnicodeInfo indexes

* Add new GenUnicodeProp to tools

* Add licence headers

* Add big-endian support

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

Allow supressing exceptions in diamond inheritance cases (dotnet/coreclr#20458)

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

Re-enable optimizations in interop helper functions. (dotnet/coreclr#21016)

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

Fix invalid tests for COM native server (dotnet/coreclr#21012)

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

Disable COM tests being CrossGen'd (dotnet/coreclr#21010)

* Disable running CrossGen on COM tests since the CrossGen tool doesn't support app manifests on Windows.

* Update ObjectStackAllocation test to use MSBuild property to disable CrossGen support.

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

Add test for marshalling ArgIterators on Windows (dotnet/coreclr#20888)

* Add (windows-only) tests for __arglist and ArgIterator marshalling.

* Add explicit calling convention notation to pinvokes.

* Add license header

* Update VarargsTest.cs

* Fix calling conventions.

* PR Feedback

* Add explicit calling convention.

* Update VarargsNative.cpp

* Don't reference System.Private.Corelib directly

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

Use correct printf type specifier

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

Jit changes from the desktop Toolset Update

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

Update CoreFX-in-CoreCLR testing instructions (dotnet/coreclr#21002)

Update to reflect changes in the repo, and fix various formatting issues.

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

Fix arm64 PUTARG_STK(contained OBJ(contained LCL_FLD_ADDR)). (dotnet/coreclr#20995)

* add a repro test

* Fix the issue.

Do not mark `GT_OBJ` as containned for arm64.
We already do the same in `NewPutArg`.

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

Fix warnings because of Bool incompatibality

Fix a compile warning because of bool vs BOOL incompatibality.
Fixes a build break in the new VS compiler.

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

Update help output of runtest.cmd/sh

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

Cleanup on the new Math APIs (dotnet/coreclr#20993)

* Updating the cached HAVE_COMPATIBLE_ILOGB0_EXITCODE and HAVE_COMPATIBLE_ILOGBNAN_EXITCODE to 1

* Fixing MaxMagnitude and MinMagnitude to correctly handle the case when ax and ay are equal

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

Update CoreClr, CoreFx, CoreSetup, IbcData, PgoData to preview-27113-06, preview.18563.5, preview-27114-01, master-20181114-0045, master-20181114-0044, respectively (dotnet/coreclr#20950)

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

Merge pull request dotnet/coreclr#20961 from briansull/const-eval-fix

Improve CanEvalForConstantArgs

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

Undo a workaround added in dotnet/coreclr#20442.

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

Fix for bug 20499.

When the jit is copying a struct-typed return to the return local in a synchronous method on arm64,
it ends up invoking an importer utility outside the importer, where impTreeLast is not set.

The call sequence is
fgMorphBlocks --> gtNewTempAssign --> impAssignStruct --> impAssignStructPtr --> impAppendTree

When impAssignStructPtr sees GT_COMMA src nodes, it unwraps them and inserts additional statements.
The fix is to pass an insertion point statement through this call chain to prevent impAssignStruct
and impAssignStructPtr from calling impAppendTree outside of the importer.

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

Disable R2R for object stack allocation tests.

Object stack allocation currently doesn't work in R2R mode
because we lower R2R allocations to helper calls in the importer.
That will be fixed but for now we need to disable R2R for these tests.

Fixes dotnet/coreclr#20977.

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

Initial commit for System.Text.Rune (dotnet/coreclr#20935)

This type represents a Unicode scalar value ([ U+0000..U+D7FF ], inclusive; and [ U+E000..U+10FFFF ], inclusive). The primary scenario is for having a consistent representation of Unicode data regardless of the underlying input encoding type, including abstracting away surrogate code points.

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

Fix CanEvalForConstantArgs(VNFunc vnf)

Change the default return value for new GenTree nodes to false

This allows new nodes to be added without immediately implementing
the compile time folding logic.

Added method header comment for CanEvalForConstantArgs

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

n_heaps needs to be initialized before init_static_data is called (dotnet/coreclr#20968)

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

Specify version of xunit.console.dll during the custom build (dotnet/coreclr#20975)

* Use stable Xunit v2.4.1
* Use xunit.console.dll with specified assembly version to avoid issues with Ready2Run jobs
* Update comment regarding building xunit with specified version in tests/runtest.py

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

Add tests for AsAny Marshalling (dotnet/coreclr#20963)

* Copy over tests from .NET Framework.

* Convert tests to use CoreCLRTestLibrary. Clean up tests that weren't self-consistent.

* Pass in context for non best-fit-mapping tests.

* Move around/group declarations.

* Clean up typedef names/usage.

* Fix signedness.

* Use ptrdiff_t and size_t instead of C99 types.

* Fix printf specifiers

* Remove unneeded now duplicate typedef

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

Change GetAppDomain to return AppDomain from the global static (dotnet/coreclr#20910)

* Change GetAppDomain to return it from the global static

The current implementation of the GetAppDomain takes it from the TLS for
the current thread. But we only have one AppDomain in the system, so we
can change it to return just that one.
I have still left the ThreadLocalInfo.m_pAppDomain and its setter
present, because SOS uses that to access the AppDomain and the SOS needs
to be runtime versino agnostic.
This makes it to perform better for Unix where accessing TLS is not
trivial.

* Move the AppDomain instance pointer to own static

To enable access to the one and only AppDomain without unnecessary
indirections, I have moved the pointer out of the SystemDomain class.

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

Initial support for normalized 'naked' R2RDump output (dotnet/coreclr#20875)

* Initial support for normalized 'naked' R2RDump output

This change introduces a new option "--naked" that takes output
normalization even further - it intentionally leaves out any
position information to make the output easier to diff between
CPAOT and Crossgen.

One other new option is "--entrypoints" which dumps a plain
list of JITted methods in the R2R executable. This can be used
for comparisons between CPAOT and Crossgen and / or for static
analysis of what methods were actually emitted by the compiler.

* Addressed Zach's PR feedback

1) Added argument consistency check for the invalid combination
"--naked" + "--raw".

2) Added dump of multi-dimensional array lower bounds and sizes
when available.

Thanks

Tomas

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

Unify ThrowHelper style (dotnet/coreclr#20974)

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

Increase FreeBSD job time allowance

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

Optimize vtable calls (dotnet/coreclr#20696)

* Implementation of R2R vtable call thunks. These thunks will fetch the target code pointer from the vtable of the input thisPtr, and jump to that address.
This is especially helpful with generics, since we can avoid a generic dictionary lookup cost for a simple vtable call.
Overall, these thunks cause the CPU to have less branch mispredictions, and give a small performance boost to vtable calls.

These stubs are under VirtualCallStubManager so that the managed debugger can handle stepping through them.

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

Handle generics in methodimpls for default interface methods (dotnet/coreclr#20404)

The existing logic looking at MethodImpls to find the default interface implementation was not correct in the presence of generics. The MethodImpl records we build during type load only contain inexact MethodDescs for the declMethod that are not suitable for comparisons.

The fix is to pipe through the token and resolve the declaring method from it at the time of dispatch.

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

Fixing Utf8Parser.Number to not modify the value of 'c' (dotnet/coreclr#20967)

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

Exclude failing System.Memory.Tests (dotnet/coreclr#20958) on Linux/ARM and Linux/ARM64 (dotnet/coreclr#20962)

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

Passing the CONTEXT in ICorDebugManagedCallback4::DataBreakpoint (dotnet/coreclr#20960)

* Passing the CONTEXT in ICorDebugManagedCallback4::DataBreakpoint
* ifdef out the message sending on configurations that does not support data breakpoint

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

Simplify build.sh command line arguments for Linux/arm cross build (Part 1) (dotnet/coreclr#20836)

* Don't use crosscomponent|-crosscomponent command line args in build.sh
* Don't use CROSSCOMPONENT environment variable in build.sh src/pal/tools/gen-buildsys-clang.sh
* Remove mentioning -crosscomponent from usage() in build-test.sh
* Don't use -crosscomponent in buildpipeline
* Don't use -crosscomponent in Groovy files
* Remove mentioning crosscomponent in Documentation/building/linux-instructions.md
* Stop building Hostx86/armel crossgen in build.sh
* Disable CrossGen-ing of System.Private.CoreLib on Linux/armel

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

Lower fast tail call wasn't patching control expression (dotnet/coreclr#20740)

Lower fast tail call can replace local variables (holding Caller stack arguments) with new temps in order to set up Callee stack arguments correctly. This involves patching corresponding LCL_VAR and LCL_VAR_ADDR nodes and replacing them with the location of a new temp.

This was not done for control expression which continued pointing to the old location and could contain a Callee argument.

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

Disable stress random inlining for object stack allocation tests.

ObjectStackAllocationTests are sensitive to inlining so we need
to disable stress random inlining.

Fixes dotnet/coreclr#20944.

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

Moving the Utf8Formatter and Utf8Parser into S.P.Corelib (dotnet/coreclr#20934)

* Moving the Utf8Formatter and Utf8Parser into S.P.Corelib

* Doing some minimal cleanup to lineup types and get the Utf8Parser/Utf8Formatter building

* Updating the Utf8 Float Parser to have different buffers for Single vs Double

* Fixing the Utf8Parser to track trailing zero digits and to properly mark the end of the buffer

* Fixing a couple of issues in Utf8Parser.Number

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

Update tests/arm64/corefx_linux_test_exclusions.txt (dotnet/coreclr#20206)

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

Remove SSE4.2 STTNI APIs for .NET Core 3.0 (dotnet/coreclr#20887)

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

enable build-id on FreeBSD (dotnet/coreclr#20945)

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

LoadLibrary refactoring (dotnet/coreclr#20841)

* Refactor LoadLibrary Methods

This change refactors the code in DllImport in preparation
for implementing the new NativeLibrary API here:
dotnet/corefxdotnet/coreclr#32015

The two main changes are:

1) A change in the semantics of the internal LoadLibrary helper functions.

When a native library is loaded, there are two categories of callers
expecting different return values:

External callers like AssemblyNative::InternalLoadUnmanagedDllFromPath()
and the upcoming System.Runtime.Interop.Marshall.LoadLibrary()
need the raw system handle
Internal callers like LoadLibraryModule() need the PAL registered handle
This change modifies the internal LoadLibraryModule* methods to work
in terms of native system handles, so that external callers can obrain
them directly. Methods requiring PAL-handles can register them explicitly.

There is no change in external signature of DllImport class, or the
native Dll cache in AppDomain class.

2) Differentiate HMODULE and NATIVE_LIBRARY_HANDLE

This change defines NATIVE_LIBRARY_HANDLE type to represent
raw system handles to native libraries that are not registered
with the PAL (On Unix systems).

The types on PAL and DlImport methods are adjusted to make
this semantic distinction explicit.

*
Fix loading LibC via PAL_LoadLibraryDirect()

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

Fix CoreRT build breaks

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

Allow jit to examine type of initonly static ref typed fields (dotnet/coreclr#20886)

The jit incorporates the value of integer and float typed initonly static
fields into its codegen, if the class initializer has already run.

The jit can't incorporate the values of ref typed initonly static fields,
but the types of those values can't change, and the jit can use this knowledge
to enable type based optimizations like devirtualization.

In particular for static fields initialized by complex class factory logic the
jit can now see the end result of that logic instead of having to try and deduce
the type of object that will initialize or did initialize the field.

Examples of this factory pattern in include `EqualityComparer<T>.Default` and
`Comparer<T>.Default`. The former is already optimized in some cases by via
special-purpose modelling in the framework, jit, and runtime (see dotnet/coreclr#14125) but
the latter is not. With this change calls through `Comparer<T>.Default` may now
also devirtualize (though won't yet inline as the devirtualization happens
late).

Also update the reflection code to throw an exception instead of changing the value
of a fully initialized static readonly field.

Closes dotnet/coreclr#4108.

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

Some cleanup of the Math functions from dotnet/coreclr#20788 (dotnet/coreclr#20912)

* Fixing Max, MaxMagnitude, Min, and MinMagnitude for Math/MathF to be IEEE compliant

* Disabling the System.Math.Max/Min tests

* Adding the new c_runtime PAL tests to the CMakeLists to ensure they actually get run.

* Fixing the casing of IlogB to ILogB

* Fixing the new PAL tests to match the correct/expected outputs

* Fixing up PAL_ilogb to correctly handle 0 and NaN

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

Update CoreClr, CoreFx, IbcData, PgoData to preview-27112-01, preview.18562.1, master-20181112-0045, master-20181112-0043, respectively (dotnet/coreclr#20929)

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

Verify the application group container directory exists in Mac Sandbox (dotnet/coreclr#20916)

* Verify the application group container directory exists in Mac Sandbox

Added an additional check to verify that the shared files directory based on the application group ID exists when running in a Mac sandbox. If it doesn't then the initialization will fail.

As part of this change, also refactored the logic the sets the shared file path into a separate method.

* Changed bool to BOOL

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

Keep delegate fields alive across a full native call (dotnet/coreclr#20896)

* Repurpose CleanupWorkList to also preserve delegate references in structs across the full native call.

* Change CleanupWorkListElement to abstract base class instead of interface.

* Make CleanupWorkList a singlely linked list.

* PR Feedback.

* Remove CleanupWorkList and make CleanupWorkListElement be able to represent the full list.

* Add back throw in SafeHandle field marshalling.

* PR feedback.

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

Remove VersionBlock struct for profiler attach (dotnet/coreclr#20889)

* Remove VersionBlock struct for profiler attach

* Change pipe/event names to not match the Desktop version pipe/event names

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

Fix invalid out of range exception thrown in ArrayPool trimming (dotnet/coreclr#20932)

The trim method had the wrong logic to get the relevant bucket size, resulting in InvalidOutOfRangeException exceptiosn being thrown and eaten on the finalizer thread.

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

Add PInvoke/BestFitMapping tests (dotnet/coreclr#19269)

* Add PInvoke/BestFitMapping tests

* Fix warning

* Make native side of tests xplat.

* Clean up managed side of the test build.

* Disable BestFitMapping tests off Windows.

* Individually disable tests.

* Disable the two tests that I missed last time.

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

Update IbcData, PgoData to master-20181110-0042, master-20181110-0040, respectively (dotnet/coreclr#20925)

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

Update BuildTools, CoreClr, CoreFx, CoreSetup, IbcData, PgoData to preview1-03407-01, preview-27110-01, preview.18560.1, preview-27109-05, master-20181109-0043, master-20181109-0044, respectively (dotnet/coreclr#20790)

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

Add back support for deserialization of BinaryFormatted resources (dotnet/coreclr#20907)

* Add back support for deserialization of BinaryFormatted resources

This adds back support for using BinaryFormatter to deserialize resources when loaded
from assembly.

We conditionally load BinaryFormatter and will throw if asked to deserialize binary formatted
resources and cannot find it.

* Apply code-review feedback

Changes lazy initialization to use LazyInitializer

Remove special handling of missing serialization assembly.

Clean up comments,

* Use named parameter when specifying permitDeserialization

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

Delete dead/unreachable code related to remoting (dotnet/coreclr#20880)

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