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

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.

[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.

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

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

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

Fix Link (#7699)

Merge pull request #7698 from CarolEidt/Fix278371

Allow non-constant size for rep stos InitBlk on x86

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

Incoming parameters that are passed on the stack are never reported as
tracked to the GC, but `lvaIsGCTracked` (which is used to determine
whether or not to report liveness for a lclVar's stack slot) still
returned true if the corresponding lclVar was tracked. This caused
problems on x86 when dealing with incoming GC stack parameters in ESP
frames: in this case, writes to a GC stack parameter could incorrectly
change the liveness for a lclVar at the same offset from ESP as the
parameter's offset from EBP.

Fixes #7696.

Merge pull request #7701 from rahku/arm64fixes

_isnanf is not defined for arm64 use isnan instead

header for getRuntimeContextTree

return break to the previous position,

_isnanf is not defined for arm64 use isnan instead

Upgrade build-against-packages dependencies to NetStandard1.1

Allow non-constant size for rep stos InitBlk on x86

Fixes DevDiv VSO bug 278371, and adds a test case.

Merge pull request #7683 from sivarv/VectorByteFix

Vector<T>.ctor(T val) is not recognized for ubyte/byte/short/ushort when the arg is a local of small int type (e.g. method args).

Fix issue #7470 (Regression with enabled GDB JIT interface) (#7688)

Adding single-precision math functions. (#5492)

* Adding single-precision math functions to floatsingle
* Adding single-precision math functions to the PAL layer.
* Adding single-precision math tests to the PAL layer.
* Adding single-precision math functions to mscorlib.
* Adding single-precision math function support to the vm.
* Updating floatsingle.cpp to define a _isnanf macro for Windows ARM.

Merge pull request #7684 from mellinoe/fix-alpine-pgks

Fix placeholder pdb file path in alpine nuget packages

Merge pull request #7648 from jkotas/fix-docs

Update instructions to use local dotnet tool build

Add missing Arg_BadDecimal resource to corelib (#7673)

Fixes #7671

Merge pull request #7676 from pgavlin/EmitKillGCRefRRWSH

Update GC info when emitting IF_RRW_SHF.

Fix placeholder pdb file path in alpine nuget packages

Include project.json template in the instructions

Merge pull request #7608 from pgavlin/MoveGSCookieCheck

Generate GS cookie checks before consuming call args.

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

Merge changes from TFS

Fix memcpy undef (#7675)

This change fixes a memcpy #undef in the palinternal.h. It was undefed only for
non-debug builds, but it needs to be undefed for debug build as well. The non-debug
undef covers the case where memcpy is defined as DUMMY_memcpy, but doesn't cover
the debug case where memcpy is defined as PAL_memcpy.

Change the name buildtools semaphore temporarily

Update instructions to use local dotnet tool build

Vector<T>.ctor(T val) is not recognized for ubyte/byte/short/ushort
when the arg is a local of small int type (e.g. method args).

ifdef for EE version

Merge pull request #7628 from wtgodbe/HelixFinal

Build tests against packages & produce Core_Root for arbitrary OS

Merge pull request #7667 from kouvel/ApiThreadCleanup

Expose RuntimeThread.Yield, fix warning

Adding SafeSerializationEventArgs to netstandard2.0

[tfs-changeset: 1633663]

Generate GS cookie checks before consuming call args.

This is necessary in order to correctly calculate the liveness of the
call arguments. Generating this check after consuming the arguments
causes errors in the GC info s.t. GC ref arguments are not considered
live past the call instruction: instead, they are considered dead
before the call instruction.

Fixing PAL_TRY use in BOTR (#7658)

In the Exceptions Section it gives an example use of `PAL_TRY`,
but the parameters to the `PAL_TRY` need to be passed as an argument,
and cannot be implicit.  This commit brings the change inline with
the programming model.

Enable NumberFormatInfo netstandard 1.7 APIs (#7659)

* Enable NumberFormatInfo netstandard 1.7 APIs

* Remove ComVisible attributes

* Remove more ComVisible attributes

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

Merge changes from TFS

Update GC info when emitting IF_RRW_SHF.

Instructions with this format should never produce a GC reference, and
should kill any GC reference that was in the destination register.

This situation can arise with IR such as the following:

    t1347 =    &lclFld   byref  V00 loc0         [+8] Fseq[m_b] REG ecx

            /--*  t1347  byref
            *  st.lclVar int    V271 tmp270      d:3 ecx REG ecx RV

    t8942 =    lclVar    int    V271 tmp270       ecx REG ecx RV

    t8937 =    const     int    0 reuse reg val REG eax RV

            /--*  t8942  int
            +--*  t8937  int
    t8943 = *  gt_long   long   REG NA

    t8939 =    const     int    4 REG NA

            /--*  t8943  long
            +--*  t8939  int
    t8944 = *  <<Hi      int    REG eax

    t6676 =    lclVar    int    V271 tmp270      u:3 ecx (last use) REG ecx RV $6b6

    t8940 =    const     int    4 REG NA

            /--*  t6676  int
            +--*  t8940  int
    t8941 = *  <<        int    REG ecx

For this IR, we generate the following x86 assembly:

    IN04b3: 001218 8D4DE8       lea      ecx, bword ptr [ebp-18H]
    IN04b4: 00121B 0FA4C804     shld     eax, ecx, 4
    IN04b5: 00121F C1E104       shl      ecx, 4

Where the `lea` corresponds to the `&lclFld` node, the `shld`
corresponds to the `<<Hi` node, and the `shl` corresponds to the `<<`
node. Because the `&lclFld` node is typed as a byref, the `lea`
instruction marks `ecx` as a byref register. Normally, the `st.lclVar`
would update the gc-ness of its destination, but in this case it is
elided because its source register matches its destination register. As
a result, `ecx` is still considered to contain a byref value at the
point of the `shl`.

It is very likely that it is also a bug that the elided `st.lclVar` does
not udpate the gc-ness of its destination, though it remains to be seen
whether or not this can cause SBCG in other cases.

The typing of the `&lclFld` node as a byref is also questionable, as
these nodes are guaranteed to point to the stack (and are therefore
never heap references). This typing is conservatively correct, however,
and AFAICT only causes us to over-report registers or stack slots in the
generated GC info.

Merge pull request #6679 from parjong/fix/issue_6569

Revises isSingleFloat32Struct

Merge pull request #7661 from pgavlin/FixInteropTestCC

Fix calling conventions in ByValArray test.

Merge pull request #7668 from mellinoe/alpine-rids

Add native package projects for alpine.3.4.3-x64

helper for corert

Helper to get ready to run, shared, generic, static base.

Merge pull request #6613 from wtgodbe/packagebuilds

Use packages.builds when generating nuget packages

helper for corert

Helper to get ready to run, shared, generic, static base.