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

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

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)

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

Merge pull request #7325 from tarekgh/AddStringInfoMethods

Add the needed StringInfo methods to Linux implementation

Fix 8.3 UNC expansion

This ports the fix from
https://github.com/dotnet/corefx/pull/11984

Add the needed StringInfo methods to Linux implementation

Merge pull request #7303 from CarolEidt/Fix7195

DynBlk assert fixes for Arm64

Add System.Reflection.Pointer deserialization ctor to model.xml (#7322)

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

Merge changes from TFS

Fix Unable to step through Visual Studio remote debugger with coreclr executables on Nano Server (#7317)

Needed to add back a dummy version of ICorDebugLegacyNetCFHostCallbackInvoker_PrivateWindowsPhoneOnly interface so
VS doesn't get the QI failure and stop working by not calling the "Continue" method.

Issue #7316

Merge pull request #7319 from pgavlin/GTLongFieldList

Decompose GT_FIELD_LIST nodes that use a GT_LONG.

Merge pull request #7315 from tarekgh/DigitsValueUnicodecategory

Support Digit and Decimal values in CharUnicodeInfo

Fix formatting issues.

Merge pull request #7309 from CarolEidt/x86MkRefAny

Support mkrefany for x86

Support Digit and Decimal values in CharUnicodeInfo

This change adding the needed data for the digit and decimal values and the needed code to use the data and return the required values

Merge pull request #7294 from pgavlin/longLclFld

Implement `st.lclFld` decomposition and fix #6778.

Address PR feedback.

[GDB/JIT] Add support for lldb step-over command on managed code (#7273)

Decompose GT_FIELD_LIST nodes that use a GT_LONG.

These nodes are decomposed by splitting the appropriate field list
element into two elements: one each for the low and high halves of
the GT_LONG.

GcInfo V2 for Jit32 Desktop

This change switches Jit32 on Desktop CLR to use GcInfo V2 encodings.
The updates to the encoder are in this change, and the necessary changes
to the decoder in the VM are already in place.

Testing:
Standard DDR testing
X86 GCStress testing

[tfs-changeset: 1628772]

Merge pull request #7312 from adiaaida/improveFormattingVerbosity

Improve formatting job output

Improve formatting job output

To make it more clear where there are formatting errors, have format.py
print what commands the user needs to run to fix the formatting errors in
their PR.

Merge pull request #7311 from adiaaida/fixutilsformatting

Fix formatting in utils.cpp

Fix formatting in utils.cpp

Support mkrefany for x86

Use a GT_FIELD_LIST to pass the RefAny struct.

Fix #4183

Supports cross build configuration override (#7307)

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

Merge changes from TFS

Generate CrossGen Pdbs as part of bulid (#7298)

This change has two parts.  The first part changes build.cmd and the pkgproj
so that as part of generating a .NET Core package we also generate the PDBS
for the Native System.Private.Corelib.ni.dll.

The second part changes crossgen so that it marks the Native Images as having
a list of 0 exports.   This is important because we want the generated PDBs
to be naturally published on the symbol server, and currently the symbol server
skips DLLs what are 'resource only' (no imports or exports).   Because native
images don't use 'normal' PE imports and exports, they get optimized away
by the symbol server.  Making a empy list of exports fixes this.

Display histogram of total memory per method when MEASURE_MEM_ALLOC is on. (#7286)

* Display histogram of total memory per method when MEASURE_MEM_ALLOC is on.

Fix 6667: Constant folding for unslgined long divide operation

Merge pull request #7285 from sivarv/internalReg

 Support Lower to reserve internal register(s) different from targetReg.

Implement `st.lclFld` decomposition and fix #6778.

The problem that was blocking both of these issues was an issue in
liveness when analyzing `st.lclFld` nodes that store to an unpromoted
long-typed local variable or any long-typed field. The latter case--a
`st.lclFld` node that targets a long-typed field--is easy to handle.
Such stores are decomposed from:

```
         /--*  t0 int
         +--*  t1 int
    t2 = *  gt_long long

         /--*  t2
         *  st.lclFld long V00
```

To:

```
         /--* t0
         *  st.lclFld int V00 [+0]

         /--* t1
         *  st.lclFld int V00 [+4]
```

The two `st.lclFld` nodes that this transformation generates are marked
with `GTF_VAR_USEASG` to indicate that they are partial (rather than
complete) defs. This is necessary in order to correctly calculate
liveness.

Before this change, stores to unpromoted long-typed local variables
were also decomposed in the above fashion. These local vars can arise in
a number of situations:
- vars that are used to store multi-reg args or return values
- vars that represent long-typed fields of promoted struct-typed vars
- long-typed vars that are marked as unenregisterable before decomp

Unfortunately, the decomposition given above differs in its liveness
semantics when compared to the original IR: in the original IR, the
`st.lclVar` is considered a plain def of its destination, while in
the decomposed IR, each `st.lclFld` is considered both a partial def
and a use of its destination. This difference--namely, that in the
first case the destination is not used at the def and in the second
case it is--causes problems for any analysis that needs to take into
account whether or not a variable is used at a def (e.g. dead store
removal, live-in sets, etc.). In order to retain the original
semantics without complicating the IR, stores to unpromoted, long-typed
local variables are not decomposed. These stores are instead handled
during code generation.

Add missing contracts on GCHeapUtilities

[tfs-changeset: 1628632]

DynBlk assert fixes for Arm64

In the case where the block size is not a constant, there were some incorrect asserts. The size register is on the size node itself, the node is a store, and the check for whether the size is a handle used to be guarded by a check that it is GT_CNS_INT, but since it was really intended to assert that there are no gc pointers, according to the comment on the old code, it doesn't really seem to be a useful assert (we could never get a size node in that case, in the new IR form).

Merge pull request #7302 from adiaaida/moreFormatting

Fix formatting in ssabuilder

Support Lower to reserve internal register(s) different from targetReg.

Merge pull request #7299 from tarekgh/MarkCultureInfoMethodsAsPublic

Mark some CultureInfo methods as public

Fix formatting in ssabuilder

The formatting changes missed the new changes to ssabuilder. Format that
file.

Merge pull request #7291 from adiaaida/format

Reformat jit source base and turn on formatting jobs for PR

Merge pull request #7284 from gkhanna79/Ver120

Update Nuget package versions to 1.2.0

Mark some CultureInfo methods as public

We are exposing such APIs now so we need them to be public

Fix formatting job for Ubuntu

The change to move to the newer version of the CLI for coreclr causes
something to look for a config.json file in the current directory (rather
than the directory where the build.sh script resides), forcing build.sh to
be run from the base coreclr dir (you can't run it from any other
location), so changing directories to the jitutils bin directory would
cause the build to fail, failing the jit-format run. This change fixes the
format.py script to build the full path to the jit-format executable and
then run it from the coreclr directory, rather than changing to the utils
bin dir and running jit-format from there.

Make formatting jobs required for PR

Fix conflicts between nix and Windows formatting

Reformat jit code on OSX

Reformat jit code for Windows x64

Merge pull request #7191 from erozenfeld/SSAStackOverflow

Fix for StackOverflow in SsaBuilder::TopologicalSort.

Fix OS X build break with AppleClang 8.0.0 (#7292)

Fixes issue #7275

Merge pull request #7293 from CarolEidt/FixArmObj

Fix Arm64 codegen to use GT_FIELD_LIST

Merge pull request #7289 from CarolEidt/Fix7196

Fix Issue 7196: No extra internal regs for block nodes

Fix Arm64 codegen to use GT_FIELD_LIST

In #7252 changing GT_LIST to GT_FIELD_LIST when passing struct fields,
I omitted some required changes to codegenarm64.cpp. This caused the
crossgen of System.Private.CoreLib.dll to fail.

Fix for StackOverflow in SsaBuilder::TopologicalSort.

SsaBuilder::TopologicalSort allocated an array of bools on the stack.
In one of internal cases the array had 126,758 elements, which caused a
StackOverflow. This change switches SsaBuilder::TopologicalSort from
stack-allocated array of bools to BitVec.

TopologicalSortHelper used to be recursive. It was changed to use an iterative
algorithm in a previous change. I'm inlining TopologicalSortHelper into
TopologicalSort now.

I verified no measurable negative throughput impact from this change
and no asm diffs in SuperPMI.

Fix Issue 7196: No extra internal regs for block nodes

There is some possibly obsolete code that copies the internal registers
for a GT_IND onto the address node as well. With the struct IR changes,
the block nodes (e.g. GT_STORE_BLK) also return true for OperIsIndir(),
so they were getting the internal registers copied over from their address
node. This led to an assert on Arm64.

Fix #7196

Fix CrossGen error reporting when input is missing (#7287)

Currently, running "crossgen foo.dll" when foo.dll does not exist produces
a misleading error message "The image being compiled is not a .NET assembly".
The reason is CrossGen ignores the actual error code reported during
initial asssembly load and verification, and always reports the error as
NGEN_E_FILE_NOT_ASSEMBLY. This change removes that logic, since the original
error code is much more likely to be useful.

If out some dead code in System.Array for FEATURE_CORECLR (#7288)

Clean up GenTree node size dumping code. (#7278)

Clean up GenTreeXxxx struct size dump; fix 32-bit build when MEASURE_NODE_SIZE is enabled.

Merge pull request #7280 from JosephTremoulet/BoundsCheckDisable

Respect JitNoRangeChks flag in RyuJit