Merge pull request #7468 from CarolEidt/Fix7460

Allow GT_FIELD_LIST as legal arg for RyuJIT/x86

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

Merge changes from TFS

Add support for Alpine Linux (#7440)

This change enables build of CoreCLR on Alpine Linux. Here is the list
of changes:
- Disable asserts checking RSP in arbitrary threads against cached stack limit
  for the respective thread. The stack on Alpine obviously grows over the limit
  reported by the pthread functions.
- Disable using XSTATE. This should be re-enabled after MUSL gets the _xstate,
  _fpx_sw_bytes and related data structures added to the signal.h header.
- Disable setting rlimit of RLIMIT_NOFILE to the max value, since it breaks
  debugging for some reason.
- Add skipping over the hardware signal trampoline in the PAL_VirtualUnwind.
  While we were not trying to walk over it in a simple case, in a case where
  an exception was thrown from a catch handler of a hardware exception, we
  still attempted to walk over it and it fails on Alpine.
- Fix detection of Alpine Linux in the PAL's CMakeLists.txt so that it works
  in Docker containers too.
- Modified PAL_VirtualUnwind to make the check for unwinding past the bottom
  of the stack unconditional. We had a long list of platforms where we were
  doing this check and it doesn't hurt to do it on platforms where it is not
  needed. I have done that rather than adding a check for Alpine Linux as
  another platform that needs it.

Allow GT_FIELD_LIST as legal arg for RyuJIT/x86

The condition in GenTree::IsValidCallArgument() was not correct for
GT_FIELD_LIST, which is used for passing a struct on the stack in
RyuJIT/x86.

Fix #7460

Merge pull request #7464 from pgavlin/RyuJITPInvokeFrame

Fix P/Invoke call and method {pro,epi}logs in RyuJIT/x86.

Remove FirstChanceExceptionEventArgs from BCL folder. The file has been moved into mscorlib folder

[tfs-changeset: 1630635]

Add comment anchors.

Fix P/Invoke call and method {pro,epi}logs in RyuJIT/x86.

The contract for methods that contain P/Invokes differs between 64- and
32-bit targets.

In the former case, the JIT must:
- In the method prolog, initialize the P/Invoke frame with a call to
  CORINFO_HELP_INIT_PINVOKE_FRAME and push the frame on to the current
  thread's frame list if the method is an IL stub.
- Before/after each call, if the method is *not* an IL stub, push/pop
  the frame (respectively).
- In the method epilog, if the method is an IL stub, pop the P/Invoke
  frame.

In the latter case, CORINFO_HELP_INIT_PINVOKE_FRAME pushes the P/Invoke
frame on to the current thread's frame list, and the frame need only
be popped in the method epilog.

This change adjusts P/Invoke lowering s.t. the 64-bit discipline is
only followed for 64-bit targets.

Add Math.Clamp overloads to mscorlib.cs (#7462)

Merge pull request #7445 from dotnet-bot/master-UpdateDependencies

Update CoreClr to beta-24603-03 (master)

[Linux] [SOS.NETCore] Quick fix for Path.GetFileName (#7441)

* Quick fix for Path.GetFileName

* Updated after review

Fix DWARF linetable info provided by GDBJIT (#7444)

Extend PC to the end of function in DWARF linetable commands.
This allows the debugger to show source lines for all addresses
inside a function.

Update CoreClr to beta-24603-03

Fix a bug where the DAC read four more bytes than necessary (#7430)

to retrieve gcHeapType, which failed on crash dumps.

Remove WaitHandleExtensions, as it is implemented in CoreFX (#7431)

Merge pull request #7361 from vancem/MessageInLttng.9-26-16

Add the serialization of the Message field when writting LTTng events on Linux

Merge pull request #7359 from vancem/EventSOurceNullRefInLevelProperty.9-26-16

Fix Null Reference in accessing Level property in EventSource failure msgs

Initial support of local variables and method arguments in GDB JIT interface (#7400)

Merge pull request #7437 from dotnet-bot/master-UpdateDependencies

Update CoreClr, CoreFx to beta-24601-02, beta-24601-02, respectively (master)

Update CoreClr, CoreFx to beta-24601-02, beta-24601-02, respectively

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

Merge changes from TFS

Merge pull request #7438 from pgavlin/gh7090

Properly initialize lvaTrackedVarSet.

Merge pull request #7432 from pgavlin/gh6920

Call `gtEffectiveVal` in `gtWalkOp`.

Merge pull request #7388 from rahku/_appdomain

Expose AppDomain apis

Properly initialize lvaTrackedVarSet.

The lack of initialization when `lvaCount == 0` was causing failures
under JITStressRegs=0x80. When this initialization is not performed,
it is initialized s.t. every bit is set, including bits that represent
non-existent lclVars. This caused LSRA to AV when attempting to access
the lclVar table using a lclNum derived from an iteration over
`lvaTrackedVarSet`.

Fixes #7090.

Port changes to UnmanagedMemoryStream from corefx (#7418)

* Port changes to UnmanagedMemoryStream from corefx

dotnet/corefx#12170

* Add PrepareConstrainedRegions

* Add Back Read<T> and Write<T> methods

* Replace MemoryCopy with Memcpy

Expose AppDomain apis

Call `gtEffectiveVal` in `gtWalkOp`.

This function was using an ad-hoc, open-coded version of
`gtEffectiveVal` that was missing the effective value of `GT_NOP` nodes.
This change replaces these implementations with calls to
`gtEffectiveVal` and makes `gtEffectiveVal` non-recursive.

Fixes #6920.

Merge pull request #7428 from adiaaida/audit_flags

Copy GTF_ALL_EFFECTS to hiResult in decomp

Merge pull request #7427 from CarolEidt/Fix7197

Arm64 StoreBlk Lowering Fix

Merge pull request #7365 from dotnet-bot/master-UpdateDependencies

Update CoreClr, CoreFx, External to beta-24529-03, beta-24530-02, beta-24523-00, respectively (master)

Merge pull request #7429 from jashook/jit32_oss_devdiv255294

Exclude DevDiv_255294.cmd for jit32 due to timeout

Copy GTF_ALL_EFFECTS to hiResult in decomp

We need to make sure that the hiResult in decompose gets the same effect
flags as the original tree. This change updates decompose methods to pass
those flags from the tree to the hiResult in the cases that they weren't
being passed before.

Exclude DevDiv_255294.cmd for jit32 due to timeout

Update CoreClr, CoreFx, External to beta-24529-03, beta-24530-02, beta-24523-00, respectively

Explicitly specify feature test macro for glibc earlier than 2.19 (#7397)

Arm64 StoreBlk Lowering Fix

With the block assignment changes, the Arm64 version of
Lowering::TreeNodeInfoInitBlockStore was not using the correct node
for the source address.

Fix #7197

Fix change to GenericEqualityComparer.Equals from #5804 (#7419)

Merge pull request #7420 from tarekgh/EnableCalendarsAPisOnLinux

Enable added Calendar APIS on Linux

Add missing System.IO APIs (#7385)

This is the last chunk of APIs missing from System.IO that will be included in netstandard20 and that live in mscorlib.

Merge pull request #7407 from sivarv/simdOpt

Optimize codegen when SIMD (in)Equality that produces bool result is compared against true/false.

Enable added Calendar APIS on Linux

Merge pull request #7399 from CarolEidt/LinearCodegenRefactor

Refactor common codegen code

Optimize codegen when SIMD (in)Equality that produces bool result is compared against true/false.

Correct preprocessor conditional termination in case BACKGROUND_GC is not defined. (#7411)

This patch ensures that the code remains valid even when BACKGROUND_GC is not defined.
Before this patch, the endif that terminates the conditional compilation block
misses the terminating curly brace. That means there will be an extra, erroneous
closing brace the moment BACKGROUND_GC isn't defined.

Remove some unreachable code that was causing the ProjectK build to fail.

[tfs-changeset: 1630090]

Set THUMB bit for RUNTIME_FIXUP_HELPER functions (#7373)

* Appends THUMB bit for RUNTIME_FIXUP_HELPER address
* Revise GetEEFuncEntryPoint (for ARM) and use it to set thumb bit
* Uses GetEEFuncEntryPoint instead of GFN_TADDR

Fix ifdefs used to select the symbol reader dll name (#7406)

Refactor common codegen code

Extract the main code generator loop, the register management methods, and some debug info code.

Fix a typo in jit.settings.targets.

[tfs-changeset: 1630047]

Merge pull request #7404 from tarekgh/AddNormalizationToLinux

Add Normalization functionality to Linux

Add option to measure time spent inside calls to the CLR. (#7357)

* Add option to measure time spent inside calls to the CLR (off by default).

[tfs-changeset: 1629999]

Add Normalization functionality to Linux

This functionality will be used in different places (e.g. string class)

Expose required S.Resources members on corelib (#7402)

Fixes dotnet/corefx#12012

Merge pull request #7368 from pgavlin/SideEffects

Fix lowering's containment analysis.

Fix lowering's containment analysis.

This fixes a silent bad code generation issue that arose during internal
testing. The original repro is a test failure under COMPlus_JitStress=2.
Due to explicit null check insertion, we (eventually) end up with the
following LIR:

    t6096 =    lclVar    ref    V86 cse10         <l:$4ad, c:$1b5>

            /--*  t6096  ref
            *  st.lclVar ref    V41 tmp29        d:26

    t2733 =    lclVar    ref    V41 tmp29        u:26 <l:$4ad, c:$1b5>

            /--*  t2733  ref
            *  nullcheck byte   <l:$4b8, c:$58a>

    t2736 =    lclVar    ref    V41 tmp29        u:26 (last use) <l:$4ad, c:$1b5>

    t2737 =    const     long   20 field offset Fseq[y] $107

            /--*  t2736  ref
            +--*  t2737  long
    t2735 = *  +         byref  <l:$2ad, c:$2ac>

    t6081 =    lclVar    ref    V83 cse7          <l:$4bd, c:$1b7>

            /--*  t6081  ref
            *  st.lclVar ref    V41 tmp29        d:27

    t2762 =    lclVar    ref    V41 tmp29        u:27 <l:$4bd, c:$1b7>

            /--*  t2762  ref
            *  nullcheck byte   <l:$583, c:$58f>

    t2765 =    lclVar    ref    V41 tmp29        u:27 (last use) <l:$4bd, c:$1b7>

    t2766 =    const     long   20 field offset Fseq[y] $107

            /--*  t2765  ref
            +--*  t2766  long
    t2764 = *  +         byref  <l:$2af, c:$2ae>

            /--*  t2764  byref
    t2763 = *  indir     int    <l:$54e, c:$1ed>

    t2767 =    lclVar    int   (AX) V07 loc4          $1ee

            /--*  t2763  int
            +--*  t2767  int
    t2738 = *  +         int    <l:$554, c:$553>

            /--*  t2735  byref
            +--*  t2738  int
            *  storeIndir int

During lowering, we attempt to form an RMW add rooted at the final
storeIndir. The pattern matching that attempts to form RMW operations,
however, does not consider whether or not it is safe to perform the
code motion involved in making the destination and source addresses
for the operator contained. In this case, lowering moves the evaluation
of the address (i.e. the dataflow tree rooted at the add that produces
t2735) into the storeIndir. This moves a use of tmp29 across a def of
the same and causes the program to store a value to an incorrect
address.

There are many variations on this pattern. For example, given the
following C#:

static int T(C[] a, C c)
{
    return a.Length != c.M() ? 100 : 0;
}

The evaluation of a.Length (including the necessary null check) should
occur before the call to c.M(). The lack of correct checks for safe
code motion that caused the original repro, however, cause the JIT to
generate bad code in this case as well: the null check for a is folded
into the load of a.Length, which is then made contained by the compare.
This results in the call to c.M() executing before the null check, which
causes the program to behave incorrectly in the case that a is null.

In order to fix the code motion analysis, this change introduces a new
type, `SideEffectSet`, that can be used to summarize the side effects
of a set of nodes and check whether or not they interfere with another
set of side effects. This change then uses the new type to ensure that
it is safe to perform the code motion necessary to make an operand
contained before doing so.

Merge pull request #7403 from ramarag/addisvolatile

Adding back isVolatile to mscorlib facade as C# compiler uses it

Adding back isVolatile to mscorlib facade as C# compiler uses it

Merge pull request #7356 from bartonjs/new_macos_gettingstarted

Fix stale OSX getting started to stop using `brew link`

Merge pull request #7392 from kouvel/TestFix

Work around issue in #7365 by referencing System.Reflection.Extension…

Fix System.Diagnostics.StackTrace version in stack trace helper code (#7395)

Issue #7381

Merge pull request #7306 from hseok-oh/fix_6667_2

Fix 6667: Constant folding for unsigned long divide operation

Work around issue in #7365 by referencing System.Reflection.Extensions directly in some test projects

Merge pull request #7379 from gkhanna79/SimplePkgVer

Centralize versioning and stabilization of nuget packages.

Invoke 'copy_test_native_bin_to_test_root' inside 'create_core_overlay' (#5676)

This commit tries to fix #5675.

Specialize IndexOf, LastIndexOf for the enum equality comparers (#5804)

Merge pull request #7367 from sivarv/simdeq

Optimize SIMD codegen for (in)equality check against zero that produces bool result.

Centralize APISet and WinArm32 package versions as well.

Centralize versioning and stabilization of nuget packages.

Move dlstack test exclusion to all configurations instead of (#7364)

just ryujit/x86, since it fails in other configurations as well.

Optimize SIMD codegen for (in)equality check against zero that produces bool result.

remove comments

Fix arm64 build issue #7355. (#7362)

Fix build break.

Introduce "IGCToCLR" and use it within the GC to interface with the EE (#7295)

* Introduce a feature define FEATURE_STANDALONE_GC that allows
the GC to communicate with the EE through a virtual interface.

* Split GCToEEInterface definition out of gc.h into its own header

* Address code review feedback

* Address code review feedback

Fix one more place where Writing EventSource messages did not initialize all meta-data properly

Fix for issue 6768 NullRef when access Level when there are EventSource messages.

See https://github.com/dotnet/coreclr/issues/6768 for mor details on the bug.

The issue is that EventSource is firing events (to report errors) before the EventSource
is completely initialized.   We dealt with many of the issues associated with this but
not all of them.   This change makes these special eventSource events (with ID of 0
 take the same codepath as for TraceLogging events (with negative IDs) when the
EventWrittenArgs properies are accessed.

Add the serialization of the Message field when writting LTTng events on Linux.

Fix stale OSX getting started to stop using `brew link`

Even with --force the `brew link` command will not link openssl any longer.
This updates the instructions to provide a less invasive version of what we
needed `brew link` to do: make the runtime versioned ABI resolvable, and make
the header files discoverable (via pkg-config).

Check PAL first, and then Windows Phone (#7354)

Add missing System.Security ns types. (#7352)

Add the IsSecurityXX properties omitted from Type base class (#7349)

Add Math.Clamp methods (#7242)

Micro-optimization for IList.Contains array implementation (#7347)

Merge pull request #7328 from ramarag/removemarkers

Adding MemoryFailPoint and  Removing some markers

Merge pull request #7337 from tarekgh/FixCollationNames

Fix the Collation name returned from CompareInfo.Name and TextInfo.Name

Remove CoreCLR's dependency on the diasymreader.dll installed as part of the full .NET Framework on Desktop (#7336)

* Remove CoreCLR's dependency on the diasymreader.dll installed as part of the full .NET Framework on Desktop

Issue #5922

* Code review feedback.

Merge pull request #7332 from JosephTremoulet/InlineVolatile

Allow inlining with volatile actual argument exprs

Fix the Collation name returned from CompareInfo.Name and TextInfo.Name

this was a regression #7004 and caused the issue #1139

Adding MemoryFailPoint Removing some markers

update JIT_MemSet/MemCpy, Buffer::BlockCopy and Buffer::InternalBlock… (#7198)

* update JIT_MemSet/MemCpy, Buffer::BlockCopy and Buffer::InternalBlockCopy

* add header comments

Merge pull request #7327 from JeremyKuhne/fixshortpath

Fix 8.3 UNC expansion

Merge pull request #7329 from sivarv/SimdGetItemFix

Optimize codegen for SIMDIntrinsicGetItem when SIMD vector is a memory-op.

Optimize codegen for SIMDIntrinsicGetItem when SIMD vector is a memory-op.

Remove vestigial assertions

The inliner used to have its own import path, which required calling
special inline versions of the spill helpers; these asserts were intended
to ensure that these regular spill helpers are not invoked during inlining,
which they do by asserting against `fgGlobalMorph`, since that flag was set
during inlining in the old code.  The inliner has since been updated to
re-use the main import code, and also to run when `fgGlobalMorph` is not
set anyway.  So these helpers currently run successfully during inlining,
despite the stale comment; remove the bogus assertions.

Allow inlining with volatile actual argument exprs

When presented with an argument marked `GFT_ORDER_SIDEEFF`, mark the
argument `argHasSideEff` and allow inlining.  Remove the vestigial code to
disallow inlining in these cases; it corresponded to a prior inliner
implementation with a different import path that has since been removed;
the inliner now re-uses the main import path, which can handle the
necessary stack spilling in the presence of `GTF_ORDER_SIDEEFF`.

Fixes #7054.

Remove the dependency of using config.json in cwd (#7330)

Currently the config.json file is looked for in the current working
directory. This change forces run.exe to use the relative path of
the file relative to the build script.

Expose some types/members (#7320)

* Expose some types/members

Expose some types/members needed for dotnet/corefx#11631, dotnet/corefx#11729, and dotnet/corefx#11231

Split MORPH into multiple phases. (#7314)

* Split MORPH into multiple phases.