Fix performance problem introduced in previous change. (#7783)

Merge pull request #7782 from sandreenko/fix-tfs-build

fix tfs build crash

fix tfs build crash

ifdef EE version statements were forgotten in two files.
Braket was on the wrong line.

Merge pull request #7778 from ianhays/binaryreadwrite_buffers

Add some extra checks to BinaryReader/Writer buffers

Add back call to BitArray::ClearAll (#7779)

Merge pull request #7706 from mjsabby/lcgjitprofcallbacks

Add LCG JIT Compilation Profiler Callbacks

Merge pull request #7776 from sivarv/lsraStatFix

Fix the condition to dump a basic block stats.

PR feedback for BinaryReader/Writer buffers

Add some extra checks to BinaryReader/Writer buffers

Merge pull request #7775 from ianhays/cfg_arm64

Add CFG flag to ARM64 build

Merge pull request #7772 from jkotas/gcinfo-cleanup

GCInfoDecoder cleanup

Fix the condition to dump a basic block stats.

Add CFG flag to ARM64 build

Merge pull request #7720 from Dmitry-Me/fixBrokenComparison

Fix broken range check

Replace LPVOID with void*

Simplify GCINFODECODER_CONTRACT

Delete DISABLE_EH_VECTORS

Delete _TARGET_SET_ macro

Fixed month calculating issue of calendars(#7666) (#7769)

Fixed the issue introduced by misunderstanding the precedence of '+' and
'>>'.

fix #7666

Fix Binder based PInvoke resolution for dynamic assembly (#7770)

Fixing the StackOverflowException error message. (#4786)

Fixed typos and format issues on clr-code-guide doc(#7767) (#7768)

There're several typos and format issues. Just fixed them.

Fix issue #7767

Port GC changes from CoreRT (#7764)

https://github.com/dotnet/corert/pull/2064: Add back limit for maximum object size
https://github.com/dotnet/corert/pull/2061: Fix retail build break

Fix broken #elif (#7756)

Add Profiling API callbacks for R2R methods (#7732)

Merge pull request #7749 from sivarv/VectorIntDot

Recognize Vector<int>.Dot on AVX as a JIT intrinsic.

Improve ArraySegment's override of GetHashCode (#4654)

* Improve ArraySegment's override of GetHashCode

* Use new implementation suggested by @VSadov

* Respond to PR feedback

* Remove the workaround for object.GetHashCode

* Respond to HashHelpers changes

* Fix compile errors.

Merge pull request #7739 from pgavlin/DisableGCStressIncompatibleTests

Mark tests GCStress-incompatible for x86.

Merge pull request #7760 from pgavlin/VSO279832

Properly initialize the varargs base lclVar.

Merge pull request #7759 from pgavlin/VSO278367

Initialize long return type descs in `gtNewCallNode`.

Initialize member variable (#7737)

Optimize Vector<int>.Dot on AVX.

Merge pull request #7755 from dotnet-bot/from-tfs

Merge changes from TFS

Merge pull request #7754 from ramarag/Threading

Exposing Some Threading Apis

Adding Isafeserializationdata interface

Address CR feedback

Properly initialize the varargs base lclVar.

The code that initializes this lclVar was not using the correct helper
to determine whether or not a lclVar was enregistered at the beginning
of a method.

Merge pull request #7672 from sandreenko/helper-for-corert

Ready to run helper for static base lookups in shared generic code

Merge pull request #7743 from sandreenko/frozen_strings

fix value numbering for frozen strings.

Initialize long return type descs in `gtNewCallNode`.

All long-returning calls (including helper calls) must have a properly
initialized ReturnTypeDesc. Previously, this ReturnTypeDesc was only
initialized by the importer for a subset of long-returning calls
(essentially user calls), which caused asserts whenother long-returning
calls were created by the compiler (e.g. for field accessors). This
change moves the necessary ReturnTypeDesc initialization into
`gtNewCallNode`.

Add LCG JIT Compilation Profiler Callbacks

Methods that contain no metadata (e.g. of sources are IL Stubs,
DynamicMethod, Expression Trees, etc.) also known as LCG methods are not
reported to profilers via the Profiling API. LCG, introduced in .NET 2.0
timeframe is unique in that it doesn't require the method to be hosted
in an assembly > module > type heirarchy and is GCable in of itself.

This change adds new APIs that notify the profiler of such methods but
since there is no metadata to go lookup, it provides some useful pieces
of information that the profiler author may want to expose to the
profiler user.

In the compilation start method we provide a className (always
dynamicClass), a methodName that can be a set of few predetermined names
like (ILStub_COMToCLR, etc.) or if the user has set the name for the LCG
method that can show up here. For example, when using the Expression
Trees API, the user can specify a friendly name which would be returned
here.

In the jit completed callback we provide information for the native code
start address and size. This is particularly useful to get more accurate
accounting of what the (previously unidentified) code is. At least the
user would know it is JITTed if nothing more (but most likely more
information like what kind of stub).

Furthermore, since this is going to be a profiler callback, the profiler
can initiate a stackwalk and give more contextual information to its
users beyond the pieces of information we can provide here that could
identify what they're encountering.

Finally, there is also the case that today the profiling APIs
underreport JITTed code in the process. Considerable amount of LCG code
can now be present in the program and in security-sensitive environments
where tracking JITTed code for security reasons is important the
profiling apis fall short. In such environments there is also often
restrictions on running with elevated privileges, so procuring this data
through other means (like ETW) may pose a challenge.

[GDBJIT] Generate more complete debug information (#7617)

* Implement support for classes
* Add support for 'this' argument type
* System.String debug info
* Functions are generated as class members in DWARF, use TypeKey as type key.
* Initial support of references
* Support 'this' artificial argument in gdbjit
* Resolve issue with recursive types in gdbjit
* DWARF classes are terminated even if they don't have children
* Fix System.Char encoding.
* Add support static fields for classes
* Add support for single dimension arrays.
* Simplify signature parsing.
* Preliminary support for multidimensional arrays.
* Add support of value types

Merge pull request #7736 from hseok-oh/fix_7649

GC for large object allocation when fail to get new segment

Merge pull request #7724 from adiaaida/implementDecomposeRotate

Implement DecomposeRotate

Merge pull request #7745 from dotnet-bot/from-tfs

Merge changes from TFS

Exposing Some Threading Apis

Merge pull request #7727 from AndyAyersMS/LateGCStructBailOut

Inliner: avoid inlining callees with GC structs on cold paths

GC for large object allocation when fail to get new segment

Delete unnecessary line

Updating the macro redefinition for _isnanf and _copysignf in floatsingle.cpp. This is required for the scenario when the system CRT header files are used instead of the newer MSVCRT header files.

[tfs-changeset: 1634419]

Fix build break

[tfs-changeset: 1634415]

fix value numbering for frozen strings

As Pat found it was value numbering issue.

Add comments describing why these tests are disabled.

Implement DecomposeRotate

fgRecognizeAndMorphBitwiseRotation can potentially morph a tree with a
TYP_LONG return type on x86 if the rotate amount is a GT_CNS_INT. This
change implements DecomposeRotate for those rotate instructions. There are
5 cases:

1) Rotate by 0: do nothing.
2) Rotate by 32: swap hi and lo.
3) Rotate by < 32: produce a shld/shld (for GT_ROL) or a shrd/shrd (for
GT_ROR).
4) Rotate by >= 32: swap hi and lo, subtract from the rotate amount 32, and
do option 3.
5) Rotate by >= 64: do rotateAmount % 64 to get the rotate amount between
0 and 63, then do one of options 1-4.

This change also updates CodegenBringUpTests\Rotate.cs to exercise these
paths.

Merge pull request #7734 from JeremyKuhne/ArrayPool

Move ArrayPool to System.Private.CoreLib

Merge pull request #7730 from dotnet-bot/from-tfs

Merge changes from TFS

Mark tests GCStress-incompatible for x86.

These tests time out when GC stress is enabled on x86.

Fixes #7692 and #7695.

return add with offset, add version if-defs

Fix Attribute implementations of Equals and GetHashCode to properly deal with derived types. #6232 (#6240)

Add cross-reference from test case to inspiring issue

Fixing the reference declarations for System.MathF. (#7735)

Remove misleading check (#7654)

Call JIT compiler's shutdown logic from crossgen. (#7663)

* Call JIT compiler's shutdown logic from crossgen.

Merge pull request #7733 from rahku/_appdomain

fix assembly.Load bug

Move ArrayPool to System.Private.CoreLib

Copy ArrayPool source from CoreFX and expose in System.Private.CoreLib.

fix assembly.Load bug

Merge pull request #7725 from CarolEidt/Fix278372

RyuJIT/x86: handle must-init multi-reg vars

Merge pull request #7726 from dotnet-bot/from-tfs

Merge changes from TFS

Add comment to test case and run it through codeformatter.

Remove the UseLatestBehaviorWhenTFMNotSpecified compat switch (#5708)

* Default to latest behavior when a TFM is not specified

Instead of requiring that a compatibility switch is set in order to get the
latest behavior for all AppContext switches that are based on TFM, have that
behavior on by default.

* Remove the UseLatestBehaviorWhenTFMNotSpecified compat switch

Better Tuple hashing, avoid boxing in Equals/GetHashCode (#6767)

Merge pull request #7612 from BruceForstall/ELT

Enable Enter/Leave/Tailcall hooks for RyuJIT/x86

Fix arm32 build break

[tfs-changeset: 1634155]

cross/build-rootfs.sh: fix default UbuntuPackages for default BuildArch (#7644)

* better default UbuntuPackages for default BuildArch
* Remove multiple sets of _UbuntuPackages to default

Inliner: avoid inlining callees with GC structs on cold paths

If an inline introduces a local or temp GC struct, the initialization
of that struct is done in the root method prolog. This can hurt
performance if the inline call site is cold. Detect this during
importation and update the inline policy to back out of the inline if
the inline is an always or discretionary candidate.

Jit diffs shows size wins in the core library with no regressions.
```
Total bytes of diff: -8789 (-0.29 % of base)
    diff is an improvement.
Total byte diff includes 0 bytes from reconciling methods
        Base had    0 unique methods,        0 unique bytes
        Diff had    0 unique methods,        0 unique bytes
Top file improvements by size (bytes):
       -8789 : System.Private.CoreLib.dasm (-0.29 % of base)
1 total files with size differences.
Top method improvements by size (bytes):
        -320 : System.Private.CoreLib.dasm - FrameSecurityDescriptor:CheckDemand2(ref,ref,long,bool):bool:this
        -224 : System.Private.CoreLib.dasm - RuntimeConstructorInfo:CheckCanCreateInstance(ref,bool)
        -214 : System.Private.CoreLib.dasm - RuntimeType:GetMethodBase(ref,long):ref
        -150 : System.Private.CoreLib.dasm - CultureInfo:GetCultureInfoByIetfLanguageTag(ref):ref
        -146 : System.Private.CoreLib.dasm - GC:RegisterForFullGCNotification(int,int)
103 total methods with size differences.
```
Desktop testing shows similar wins in a number of places and only
a few sparse small regressions.

Added a perf test. Thanks to @hypersw for noticing this.

Closes #7569.

Updating the new `System.MathF` calls which already existed in `System.Math` to call the `System.Math` implementation. (#7721)

* Updating the new `System.MathF` calls which already existed in `System.Math` to call the `System.Math` implementation.
* Removing the FCALL entry for MathF.Abs

RyuJIT/x86: handle must-init multi-reg vars

In the RyuJIT backend, lclVars can occupy multiple registers, and not have a stack location. These should be handled for must-init as for lvRegister. The existing code was doing this oinly for _TARGET_64_BIT_ when it should be !LEGACY_BACKEND.
This fixes DevDiv VSO bug 278372

Merge pull request #7708 from pgavlin/StackParamUntracked

Always treat incoming on-stack parameters as GC-untracked.

Merge pull request #7718 from dotnet-bot/from-tfs

Merge changes from TFS

Enable Enter/Leave/Tailcall hooks for RyuJIT/x86

fix compilation compilation error without ready to run.

Merge pull request #7714 from gkhanna79/CleanupArm32

Cleanup arm32

fix arm64 build break

[tfs-changeset: 1634120]

Merge pull request #7709 from wtgodbe/publishProj

Only build TargetingPack when building on Windows

Fix broken range check

Support "offline" initializtion of Tools (#7715)

This change allows init-tools to function in an "offline" mode where
tools are picked up from standalone folders. Specifically, it introduces
support for two new environment variables:

- DOTNET_TOOLSET_DIR
- BUILD_TOOLS_TOOLSET_DIR

If either is set, instead of downloading toolsets, we copy an already
existing one from the folder.  The TOOLSET_DIR is a folder with sub
directories for every version of the tool in question.

For buildtools, we expect a published toolset (sans the "dotnetcli"
folder) not just a set of nuget packages (i.e. the layout of Tools/
after running ./init-tools.sh in "online" mode).

The above varibles are useful for situations where we want to carry
multiple toolsets with us, but are less helpful for places where a
developer has produced their own toolset by hand (since the resulting
folder structure contains extra version information). For these cases,
I've added

- DOTNET_TOOL_DIR
- BUILD_TOOLS_TOOL_DIR

Which work like the above but don't require the nested folder structure.

Fix passing struct with four floats in registers via reflection (#7716)

This change fixes a bug in the code that copies a struct into the transition
frame. When it contains four floats, the first two are put to the right place,
but the following two are placed to an address that's offset by 8 instead of
by 16.

It also adds regression test for this problem as Pri 1 test.

Fix build break

[tfs-changeset: 1634099]

Merge pull request #7712 from dotnet/revert-7494-CoreLib

Revert "Delete mscorlib from CoreCLR targeting pack"

Cleanup external dependency details

Merge pull request #7681 from rahku/_appdomain

Expose apis in System.Reflection.Assembly

Enable DateTimeFormatInfo netstandard 1.7 APIs on Linux (#7713)

* Enable DateTimeFormatInfo netstandard 1.7 APIs on Linux

* Remove un-necessary return

Expose apis in System.Reflection.Assembly

format fix

helper address is already known

Cleanup External Dependency references

Remove Arm32 dependency from closed build

format fix

Revert "Delete mscorlib from CoreCLR targeting pack"

Merge pull request #7700 from wtgodbe/1.1

Upgrade build-against-packages frameworks to NetStandard1.1

Only publish TargetingPack when building on Windows