Merge pull request dotnet/coreclr#6340 from wtgodbe/tPack

Stop building targetingPack package on non-windows

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

Merge pull request dotnet/coreclr#6346 from dotnet-bot/UpdateDependencies20160720052232

Updating External dependencies to beta-24320-00

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

Merge pull request dotnet/coreclr#6354 from dotnet/revert-6344-UndoNetCIChange

Revert "Revert: Pull crash dump links"

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

Revert "Revert: Pull crash dump links"

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

Make PAL_SEHException smaller (dotnet/coreclr#6267)

The PAL_SEHException is quite large due to the fact that it contains context
and exception records. This causes a problem when we try to throw it and
system gets out of memory. The C++ runtime can throw exceptions even in that
case, but only if they are smaller than certain threshold. So in our case,
it just aborts the process.

This change separates the context and exception records from the exception,
which ensures that the above mentioned mechanism in the C++ runtime can
kick in and the exception can be thrown even when malloc fails to allocate
it.

I have also modified HandleHardwareException to return BOOL indicating whether
the execution should continue at the possibly modified exception context
instead of using RtlRestoreContext. In that case, we return all the way back
to the signal handler, update the ucontext in there and return from it.

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

Fix mistyped instruction in linux-instructions.md (dotnet/coreclr#6332)

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

Fix for CoreCLR 6269 - Trigger formal binding if domain-wide cache does not have entry for an assembly in custom load context. (dotnet/coreclr#6335)

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

Merge pull request dotnet/coreclr#6337 from AndyAyersMS/CountObservations

Inliner: improve arg observations

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

Merge pull request dotnet/coreclr#6338 from swgillespie/revert-weakreference-tests

Re-enable two tests testing GC.GetGeneration(System.WeakReference)

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

Updating External dependencies to beta-24320-00

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

Merge pull request dotnet/coreclr#6344 from AndyAyersMS/UndoNetCIChange

Revert: Pull crash dump links

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

Revert: Pull crash dump links

Ubuntu testing is failing with a groovy scripting error.
Matt asked me to revert this change for now.

This reverts commit dotnet/coreclr@7fc5cb7e52c5db53a5696d398e281c0f6f6a1588.

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

Remove duplicate code from string.GetHashCode (dotnet/coreclr#4696)

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

Improve performance, drastically reduce misfires for new string(char*) on 64-bit (dotnet/coreclr#6125)

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

Better performance for string.CompareOrdinalHelper (dotnet/coreclr#6119)

- For 64-bit, process a `long` at a time (and 3 longs per iteration of the loop), since it's the largest word size of the processor. This is what other functions, like `EqualsHelper` or `StartsWithOrdinalHelper` do.

- Make sure we perform aligned reads on 64-bit platforms

  - Strings (objects in general?) are allocated on an aligned word boundary (that is, divisible by 4/8). However, the offset to the chars (`OffsetToStringData`) is 12 on 64-bit platforms, meaning that if we read a long from the first char it will *not* be aligned. To fix this, first check for differences in the first/second chars and start the loop from there.

- Used `goto` within the loop code, so we don't have to keep making jumps in the common case.

I also removed a comment above the `String` class that seems to be outdated (seeing as there is no string constructor that accepts another string).

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

Merge pull request dotnet/coreclr#6293 from kyulee1/altjit

ARM64: Cross-Target Jit

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

Stop building targetingPack package on non-windows

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

Re-enable two tests testing GC.GetGeneration(System.WeakReference) that has now been exposed

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

Inliner: improve arg observations

Two related changes to better capture cases where args or constant args
are seen at inline call sites.

On the observation side, the inliner's stack modelling of the callee is
crude and will often overestimate the evaluation stack depth. So when
looking at an opcode that takes just one stack operand (eg BRFALSE) the
inliner's check should be whether the stack has at least one element
instead of checking whether the stack has exactly one element. For
compatibility reasons, the check for exactly one element is still used
when the inline is evaluated via the LegacyPolicy.

On the policy side, instead of keeping a bool flag to note that there
were interesting arg cases, count the number of observations.
LegacyPolicy continues to act as before, checking if count is zero or
nonzero instead of whether the flag was false or true. The count is
available for use in other heuristics and is reported in inline data.

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

Merge pull request dotnet/coreclr#6336 from mmitche/crash-dump-links

Pull crash dump links

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

Pull crash dump links
Pulls crash dump links from dumplings.txt in the workspace into the build report

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

Make coreclr buildable on 32bit Windows 10. (dotnet/coreclr#6316)

Automatically determine the OS architecture(x86 or x64) and download the
correct dev tool.

Fix dotnet/coreclr#6308

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

Merge pull request dotnet/coreclr#6331 from dotnet-bot/UpdateDependencies20160719051717

Updating External dependencies to beta-24319-00

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

Fix self-process identification for FreeBSD (dotnet/coreclr#6314)

FreeBSD does not come with procfs enabled by default, and should use
sysctl() for this purpose.

While it has similarities with NetBSD's implementation, there are a
few subtle differences, which justifies leaving this implementation
under its own guard.

It's also worth noting that on FreeBSD sysctl.h MUST be present, which
is unlike NetBSD. Therefore the HAVE_SYS_SYSCTL_H define is not
checked for or used.

This commit fixes https://github.com/dotnet/coreclr/issues/6184.

This commit is based on the following commit from core-setup:
https://github.com/dotnet/core-setup/commit/dotnet/coreclr@d5ce08014a174b006a3b409b8bb93d003ae583a0

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

Merge pull request dotnet/coreclr#6227 from parjong/revert-6021-LessConservativeGtObj

Revert "Less conservative gt obj"

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

Updating External dependencies to beta-24319-00

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

Merge pull request dotnet/coreclr#6324 from AndyAyersMS/ReplayCapturesProfitModel

Inliner: capture profit model estimate during replay

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

Merge pull request dotnet/coreclr#6322 from wtgodbe/packageFixes

Fix package build in build-packages.sh to use dotnet instead of corerun

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

Fix package build in build-packages.sh to use dotnet instead of corerun

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

Fix libc path for macOS El Capitan (dotnet/coreclr#6302)

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

Add to model.xml several members we wish to expose in the FX (dotnet/coreclr#6310)

* Add to model.xml several members we wish to expose in the FX.
Done by pulling lines selectively from a model.xml created from the full
s.p.corelib.dll.

* Add parameter to build.cmd to build both mscorlib and its native image, then stop. Also dump any errors to console for CI.
* Fix up to date check of BCLRewriter: should build when model.xml is changed

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

Add Lazy<T>(T value) constructor (dotnet/coreclr#6304)

Fix dotnet/corefxdotnet/coreclr#2578

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

Merge pull request dotnet/coreclr#6315 from kyulee1/disemail

ARM64: Temporarrily disable email notification

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

ARM64: Temporarrily disable email notification

Currently all Arm64 boxes are dead.
They need to be upgraded.

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

Provide guidance for common build-failure scenario. (dotnet/coreclr#6164)

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

Add ICloneable back to several globalization types on Unix (dotnet/coreclr#6307)

The globalization implementation used for corefx on Unix came from corert, which didn't have ICloneable, and thus the globalization types that implement ICloneable in desktop and coreclr on Windows don't on Unix.  Now that ICloneable is coming back, we need these types to implement ICloneable appropriately.

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

Minimize buffer allocations in Stream.CopyTo for seekable streams (dotnet/coreclr#4540)

The current implementation of  Stream.CopyTo  allocates a giant, 81920-byte buffer if no  bufferSize  parameter is passed. This is incredibly wasteful if the stream we're copying from can seek, because then we can use the  Length  and  Position  properties to determine how many bytes are left and allocate a buffer of that size.

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

Revert "Less conservative gt obj"

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

GC update from CoreRT (dotnet/coreclr#6305)

https://github.com/dotnet/corert/tree/master/src/Native/gc dotnet/coreclr@bc9671edca37502794c01647215519d4254d3ad4

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

Merge pull request dotnet/coreclr#6299 from mairaw/patch-1

fix small typos on contributing guide

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

fix small typos

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

Merge pull request dotnet/coreclr#6296 from swgillespie/vscode-gitignore

Add VS Code local settings folder to gitignore

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

Inliner: capture profit model estimate during replay

When using ReplayPolicy and dumping data, capture the profitability
estimate made by the ModelPolicy, so we can more easily cross-validate
it against externally measured profitability.

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

Merge pull request dotnet/coreclr#6295 from wtgodbe/initToolsFix

Stop chmoding nonexistant corerun file in Tools dir

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

Stop chmoding nonexistant corerun file in Tools dir

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

Merge pull request dotnet/coreclr#6037 from sejongoh/linux_struct_arg

Unecessary Linux stack argument copy

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

Merge pull request dotnet/coreclr#6249 from prajwal-aithal/devel/arm-ci-documentation

Documentation: Add documentation for building coreclr for emulator

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

ARM64: Cross-Target Jit

Fixes https://github.com/dotnet/coreclr/issues/6279
Fixes https://github.com/dotnet/coreclr/issues/6280

- This builds x64_arm64 clrjit.dll
- crossgen is not statically linked to jit anymore. It needs clrjit.dll
  dynamically
- Adding this cross-component binary into Jit pacakge.

So, clrjit.dll (native-target) is consumed by coreclr or crossgen
(native-target)
clrjit.dll (cross-target) is consumed by crossgen (cross-target).
Likewise, later this cross-target clrjit.dll can be used for corert targeting arm64
so that we can generate arm64 code on host machine (x64).

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

Merge pull request dotnet/coreclr#6287 from MichalStrehovsky/removeAssertOnMustExpand

Remove assert validating intrinsic expansion on mustExpand

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

Merge pull request dotnet/coreclr#6050 from prajwal-aithal/devel/arm-ci-runtests

ARM-CI: Add tests to CI script

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

Remove assert validating intrinsic expansion on mustExpand

The assert is incorrect. Consider expansion of
CORINFO_INTRINSIC_RTH_GetValueInternal, where the RuntimeTypeHandle was
created through a generic lookup helper call. The intrinsic expansion of
this sequence becomes the generic lookup helper call on it's own.

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

Remove unnecessary copies for stack struct argument except:

  - struct is promoted to registers
  - fgMakeOutgoingStructArgCopy is called
  - tree pattern of obj - addr - simd operator

    *  call      void   foo
    ...
    +--*  obj       simd16
    |  \--*  addr      byref
    |     |  /--*  lclVar    simd16 V05 loc4
    |     \--*  simd      simd16 int -
    |        \--*  lclVar    simd16 V08 tmp1
    ...

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

Merge pull request dotnet/coreclr#6256 from wtgodbe/buildToolsVersion

Update to latest version of buildtools

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

Update to latest version of buildtools

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

Merge pull request dotnet/coreclr#6276 from BruceForstall/LocAllocCleanup

Clean up localloc implementation

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

Clean up localloc implementation; improve AMD64 codegen for initialized locallocs

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

Merge pull request dotnet/coreclr#6274 from hqueue/arm/xor_swap

Introduce XOR swap to legacy codegen for ARM32

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

Introduce XOR swap to legacy codegen for ARM32

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>
Commit migrated from https://github.com/dotnet/coreclr/commit/11f6a0d41a57bb1e80a5e63165de257624fdfb80

docs: Add building coreclr for emulator

Added documentation for building coreclr for the Linux ARM Emulator.

Signed-off-by: Prajwal A N <an.prajwal@samsung.com>
Commit migrated from https://github.com/dotnet/coreclr/commit/3c277086192d6a600f81e752239dd0ec2d02ae83

Merge pull request dotnet/coreclr#6263 from stephentoub/remove_serializable

Add #if FEATURE_SERIALIZATION around customized [Serializable]s

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

Merge pull request dotnet/coreclr#6257 from rahku/crossgen

ARM64:Similar to arm, in order to preserve volatile semantics MemoryB…

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

Make StubHelpers::ProcessByrefValidationList NOTHROW (dotnet/coreclr#6258)

This change fixes StubHelpers::ProcessByrefValidationList so that it doesn't throw.
While there was a EX_TRY / EX_CATCH, the catch handler calls FormatValidationMessage
to format the message before aborting the process and that function can throw.

The fix is to wrap the inside of the EX_CATCH in one more EX_TRY / EX_CATCH and
abort the process without message if there is an exception in the FormatValidationMessage.

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

Merge pull request dotnet/coreclr#6205 from stephentoub/environmentaugments

Add EnvironmentAugments to coreclr

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

Merge pull request dotnet/coreclr#6182 from wateret/fix-6051

[ARM-softfp/Linux] Get precise type for struct

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

Add #if FEATURE_SERIALIZATION around non-functional [Serializable]

We're adding BinaryFormatter to corefx.  At present, ISerializable and friends are declared in corefx rather than being exported from System.Private.Corelib.  As a result, there are a fair number of types in mscorlib that try to customize their serialization, but because they're doing so using interfaces that aren't exported (and in some cases, aren't even currently compiled in), a formatter outside of Corelib sees a type as serializable but then tries to serialize it using the default rules and fails.

This commit ifdefs the [Serializable] attribute on types that try to customize their serialization.  If in the future we decide we want to expose ISerializable and friends from Corelib in order to enable the serialization of these types (which we'll likely want to), e.g Delegate, Exception, etc. we'll need to turn on FEATURE_SERIALIZATION for coreclr, and these will all come back automatically.

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

Merge pull request dotnet/coreclr#6255 from wtgodbe/targetingpack

Build the TargetingPack package as part of build-packages scripts

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

Merge pull request dotnet/coreclr#6254 from pgavlin/RemoveRenameUpdatedVars

Remove RenameUpdatedVars.

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

Conditionally compile EnvironmentAugments.cs for coreclr

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

Add EnvironmentAugments to coreclr

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

Merge pull request dotnet/coreclr#6251 from BruceForstall/Localloc

Implement localloc for RyuJIT/x86

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

Implement localloc for RyuJIT/x86

Fixes dotnet/coreclr#4182

Change RyuJIT to generate localloc code for x86 the same way legacy JIT does.
E.g., use a push/dec loop for initialized memory. And avoid using "sub esp"
to keep the emitter from tracking stack pointer adjustments.

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

Merge pull request dotnet/coreclr#6224 from BruceForstall/Fix4177

Fix dotnet/coreclr#4177: RyuJIT/x86 NYI related to lvaArgType()

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

Fix dotnet/coreclr#4177: RyuJIT/x86 NYI related to lvaArgType()

For x86, this function can simply return the variable type. There is
no need to map struct types. The function actually will never be
called on x86 for struct types because it is only called for register
arguments.

This NYI was only hit in the case of JMP calls. I fixed two cases
in the JMP implementation where FEATURE_VARARGS code assumed AMD64, due
to the Windows AMD64 ABI convention to pass float varargs values in both
integer and floating point registers. I put that under `#ifdef _TARGET_AMD64_`.

This removes 426 NYI.

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

Merge pull request dotnet/coreclr#6253 from stephentoub/stop_stripping_serializable

Stop BclRewriter from trimming serializable metadata

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

ARM64:Similar to arm, in order to preserve volatile semantics MemoryBarrier instructions needs to be inserted. Otherwise writes to memory locations can be re-ordered. For eg. In function leave_spin_lock() we set holding_thread to -1 and then set pSpinLock->lock to -1. Without memorybarrier it is possible for different threads to see pSpinLock->lock new value of -1 and old value of holding_thread. This causes assertion pSpinLock->holding_thread == (Thread*)-1 in function enter_spin_lock() to fire

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

Merge pull request dotnet/coreclr#6247 from ellismg/add-back-exception-members

Add back some surface area to thinned corelib

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

build the TargetingPack package as part of build-packages scripts

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

Stop BclRewriter from trimming serializable metadata

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

Remove RenameUpdatedVars.

The variable renaming performed by rationalize was originally intended
(according to its comments) to allow assignments that were embedded in
subtrees to be hoisted out into a preceding statement. It appears that
this logic has been removed, so the rename pass is no longer needed.

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

[ARM-softfp/Linux] Get precise type for pointer-size struct

Compiler::getPrimitiveTypeForStruct may return TYP_FLOAT but is off on softfp.
So we need to check that in another way rather than `IsHfa()`.
Otherwise struct types with single float32 will make inconsistency in gentree.
Which ends up with assert failure "Bad IL: Illegal assignment of float into integer!".

Check is done in Compiler::getPrimitiveTypeForStruct, by isSingleFloat32Struct.

Fix dotnet/coreclr#6051

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

Merge pull request dotnet/coreclr#6246 from rahku/crossgen

Arm64: remove gcstress_fail tag from certain tests in tests.lst

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

Add back some surface area to thinned corelib

We are working on expsosing some existing exception types from
System.Runtime in CoreFX. The implementations will facade over
System.Private.CoreLib, but we need to make sure all the members we expect
to expose from the contract are present in the implementation assembly.

Expose the missing members for ExecutionEngineException,
NotFiniteNumberException and ExternalException.

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

Merge pull request dotnet/coreclr#6244 from AndyAyersMS/FixDataCollection

Inliner: Update data collection by targeting a single inline

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

Arm64: remove gcstress_fail tag from certain tests in tests.lst

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

Merge pull request dotnet/coreclr#6226 from rahku/crossgen

ARM64: In GetDelegateTarget a portion of the _WIN64 code was under _T…

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

Suppress Interactive Shell when running PS

On recent builds of Windows/PowerShell it seems like PS wants to write a
prompt after invoking the script, which causes a problem because that puts
extra gunk into the generated source file from h2inc, which causes
downstream failures.

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

Merge pull request dotnet/coreclr#6225 from pgavlin/RationalizeCleanup

Refactor top-level comma rewriting.

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

Add a comment at the use of RewriteTopLevelComma.

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

Inliner: Update data collection by targeting a single inline

The inliner currently will record detailed data about the last successful
inline performed (given a build with DEBUG or INLINE_DATA defined).
However, for purposes of inline profitability analysis we might be more
interested in the data from an earlier inline.

This change creates a mechanism where the replay log can flag one inline
per method as the target of data collection. The inliner checks for this
attribute during replay and captures that inline's data.

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

Refactor top-level comma rewriting.

Top-level commas were being handled in three separate parts of
rationalize: TreeTransformRationalization, RewriteSimpleTransforms,
and DoPhase. Furthermore, the transform performed by the latter
was distinct from the transforms performed by the former two parts,
as those parts called out to RewriteTopLevelComma in order to do
the necessary rewrite. This change collects the two transforms
in RewriteTopLevelComma and centralizes the comma handling in
RewriteSimpleTransforms.

With the comma handling centralized in RewriteSimpleTransforms,
TreeTransformRationalization was not doing much of anything, so
it has been removed and RewriteSimpleTransforms renamed to
TreeTransformRationalization.

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

active exception regression fix (dotnet/coreclr#6117)

This patch fixes regression caused by 597e160 commit.
Previous PAL_VirtualUnwind does not set CONTEXT_UNWOUND_TO_CALL
properly. In this patch, the flag is added for non-signaled exception.

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

Fix !PrintException (pe) -lines bug where line number/source info is not being displayed. (dotnet/coreclr#6241)

The argument bLineNumbers to FormatGeneratedException was being used a local flag.

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

Expose missing members of Array to prep for dotnet/corefxdotnet/coreclr#9998.  (dotnet/coreclr#6230)

* Expose missing members of Array to prep for dotnet/corefxdotnet/coreclr#9998. modelgen.exe is helpful.

* Remove dead code

* Implement Array.LongLength.

* Revert "Implement Array.LongLength."

This reverts commit dotnet/coreclr@fdf7e96172d3b816551b827b6a45fbc692b0551d.

* Revert "Remove dead code"

This reverts commit dotnet/coreclr@432087c2d6f826db93fa30a07476dc2decb12633.

* Expose Array.GetLongLength from VM

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

ARM64: In GetDelegateTarget a portion of the _WIN64 code was under _TARGET_AMD64_ whereas at other places (like NDirectStubLinker::DoNDirect & InlinedCallFrame::HasFunction ) related code is under _WIN64. Make the code in GetDelegateTarget to be in sync with other functions and remove ifdef _TARGET_ADM64_. Without this change for arm64 InlinedCallFrame::m_datum stores unmanaged target pointer but StackWalker assumes it to be a NDirectMethodDesc and tries to dereference it and crashes.

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

Merge pull request dotnet/coreclr#6137 from papaslavik/int_softfp_fix

Fix for dotnet/coreclr#5877: pre-spill just the floating arguments

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

Merge pull request dotnet/coreclr#5966 from mikedn/x86-neg-long

Implement GT_NEG decomposition

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

Merge pull request dotnet/coreclr#6198 from briansull/multireg-refactor

Refactor and cleanup work for passing and returning struct types

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

Merge pull request dotnet/coreclr#5948 from jyoungyun/fix/issue_5947

Add testsRunningInsideARM.txt

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

Fix exceptions in GC wave 2 (dotnet/coreclr#6220)

I've removed blocks with PAL_TRY that were under CATCH_GC. I also had to
fix contract problem in GCToEEInterface::GcStartWork that was marked as
THROWS due to StubHelpers::ProcessByrefValidationList being marked as
THROWS, but the StubHelpers::ProcessByrefValidationList in fact doesn't
throw since it has body wrapped in EX_TRY / EX_CATCH.

This also fixes a problem that started to appear after my previous exceptions
fix change.

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

Implement GT_NEG decomposition

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