Merge pull request #7530 from ramarag/StrongNameKeyPair

Strong name key pair

Exposing StrongNameKeyPair

Merge pull request #7543 from JosephTremoulet/NoRngChecks_Rel

Stop making JitNoRngChecks debug-only

Fix buffer length usage in UTF-8 to Unicode conversion (#7540)

These changes are ported from desktop CLR.

Fix various CustomAttributeBuilder bugs (#7206)

Fix various CustomAttributeBuilder bugs

Stop making JitNoRngChecks debug-only

JitNoRngChecks is a flag, only available in special builds, that removes
all array bounds checks from generated code, which exists solely to
facilitate experiments measuring the cost of bounds checks.  Commit
0aebfbef0 added support for it to RyuJit, but accidentally did so only for
debug builds; release `FEATURE_ENABLE_NO_RANGE_CHECKS` builds are
currently broken.

This change makes the jit flag's definition available in release builds
with `FEATURE_ENABLE_NO_RANGE_CHECKS` defined as well.  The corresponding
code in clrconfigvalues.h already works that way.

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

Update CoreClr to beta-24609-02 (master)

Update CoreClr to beta-24609-02

Add test to validate ByRef type unification (#7534)

Make sure that `ldtoken SomeType&` leads to the same type as `ldtoken
SomeType` followed by `MakeByRefType`.

Implement CultureNotFoundException id-based ctors (#7536)

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

Update CoreClr, CoreFx to beta-24609-01, beta-24608-01, respectively (master)

Update CoreClr, CoreFx to beta-24609-01, beta-24608-01, respectively

Standardize on _echo name (#7532)

Merge pull request #7521 from vkvenkat/master

Fixed #7519 - Dotnet crashes while saturating multiple CPU groups on Windows

Return null on allocation failure due to OOM instead of throwing a exception from within the GC (#7501)

Disable CER code that is causing access violation

Resolves issue dotnet/corefx#12412.

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

Update CoreClr, CoreFx to beta-24607-03, beta-24607-05, respectively (master)

Merge pull request #7520 from pgavlin/PutStructArgStk

Refactor genPutStructArgStk for clarity.

Fix formatting.

Update CoreClr, CoreFx to beta-24607-03, beta-24607-05, respectively

Fixed #7519 - Dotnet crashes while saturating multiple CPU groups on Windows

Refactor genPutStructArgStk for clarity.

This also contains a minor CQ improvement: instead of always
incrementing the source and destination pointers when copying a struct
argument to the stack, we now only do so if there are more slots to
copy.

Merge pull request #7515 from BruceForstall/FixMerge

Fix bad merge

Merge pull request #7509 from sandreenko/master

Set missed default value for coreclr runtest.cmd #7488.

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

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

Merge pull request #7466 from adiaaida/fix7224

Force byteable registers for indir op source

Force byteable registers for indir op source

When we have a GT_STOREIND, we need to not only make sure that the
source and dest of the storeind are marked as requiring byteable
registers, but also the indirect op's source. For example:

```
GT_STOREIND(GT_ADD(TYP_INT, mem of TYP_BYTE, reg of TYP_INT or TYP_BYTE))
```

We need to make sure that the source reg of the GT_ADD is byteable.

This change fixes #7224.

Update CoreClr, CoreFx to beta-24607-02, beta-24607-03, respectively

Enable netstandard1.7 collations APIs (#7502)

* Enable netstandard1.7 collations APIs

This change is adding the netstandard 1.7 APIs to CompareInfo class and adding support to SortKey and SortVersion classes
The change include some support in cultureinfo and culturedata for locale Id's too which is used by CompareInfo class

* optimize the globalization data so we'll have faster initialization

* Remove security attributes

Restore some changes lost in a merge conflict (#7512)

Fix bad merge

A previous checkin backed-out a change that moved one test out of
the RyuJIT/x86 section. I'm reverting that, and putting the test
exclusion back where it should be.

At the same time, now that there are no RyuJIT/x86 test exclusions,
I'm deleting the RyuJIT/x86 section.

Merge pull request #7490 from BruceForstall/EnableLegacy

Enable COMPlus_useLegacyJit=1 for CoreCLR for x86

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

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

Merge pull request #7504 from pgavlin/NormalizeSpills

Always normalize stores when spilling lclVars.

Set missed default value for coreclr runtest.cmd.

__MSBuildBuildArch is set to x64 by default like __BuildArch. Fix 7488.

Enable legacy JIT fallback for CoreCLR on Windows x86

This is to support the eventual case where RyuJIT/x86 becomes the default
x86 JIT, but we support fallback to the older, legacy JIT32/x86 (which
will be renamed compatjit.dll). This uses the same mechanism that was
built for .NET 4.6 when RyuJIT/x64 was shipped, and allowed fallback
to JIT64 as compatjit.dll.

However, we use a different configuration name to avoid conflicting
with the .NET 4.6 COMPlus_useLegacy name (and unintentionally causing
users to fall back to JIT64 with their .NET 4.6 apps).

The COMPlus variable is COMPlus_UseWindowsX86CoreLegacyJit=1. For the
dotnet.exe host, you can set `"System.JIT.UseWindowsX86CoreLegacyJit": true`
in the "configProperties" section of the app.runtimeconfig.json file.

There is a new COMPlus_RequireLegacyJit=1 option to aid testing JIT
fallback.

Fix a typo and a formatting issue.

Always normalize stores when spilling lclVars.

In order to support the semantics of the MSIL evaluation stack, RyuJIT
normalizes all "small" lclVars (i.e. lclVars with a non-struct
type that is smaller than an int32) from the actual type of the lclVar
to int32. This normalization may be performed in one of two ways:
- the lclVar's may be normalized when the lclVar is used (load
  normalization)
- at any point at which the value is defined, the value being stored to
  the lclVar is normalized (store normalization)

Store normalization must not be used if the lclVar is aliasable, since
the contents of the upper bytes for that lclVar's slot may be modified
by some other alias. Load normalization must be used if the lclVar is
a method parameter, as there is no guarantee that the caller has
properly nomralized the arguments it has passed. As such, any lclVar
that is either aliasable or is a method parameter is load-normalized,
and any other lclVar is store-normalized. lclVars that are
store-normalized are assumed to be represented as properly zero- or
sign-extended 4-byte values at all times, and so can simply be loaded
as int32s.

This picture becomes somewhat more subtle after register allocation,
however: once a lclVar has been enregistered, the contents of its
register are guaranteed to be properly zero- or sign-extended for the
purposes of store normalization, and in order for the lclVar to have
been enregistered in the first place, it must not have been aliasable.
As such, all enregistered variables may be store-normalized (as well as
load-normalized, if they represent method parameters). This has a
particularly significant impact on spills, as it implies that even if
the lclVar being spilled is not normally considered store-normalized, it
may be considered so for the purposes of the spill. This is advantageous
for architectures such as x86, as it allows a small lclVar to be
successfully spilled from a register that is not addressable at the size
of the lclVar's type. This change switches both LSRA and the code
generator to always use the actual type of a spilled lclVar for the
corresponding store rather than making this behavior conditional on
whether or not the lclVar is store-normalized.

Fixes #7236.

Update CoreClr, CoreFx to beta-24606-03, beta-24606-02, respectively

Fix incorrect index for localsDebug array (#7495)

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

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

Update CoreClr, CoreFx to beta-24606-02, beta-24606-01, respectively

Merge pull request #7486 from CarolEidt/Fix7008

Support double-aligned frames for RyuJIT/x86

Merge pull request #7500 from mikedn/and-cmp-to-test

Fix and-cmp to test recognition

Support double-aligned frames for RyuJIT/x86

This factors out the analysis for when to use a double-aligned frame from the legacy backend.
Doing so exposed an assert for a case where we turn an intrinsic into a call in the Rationalizer, but don't update the frame requirements.
There may be some remaining tuning opportunities, given that the (pre-existing) algorithm doesn't attempt to create a double-aligned frame if we need an EBP-based frame for other reasons.

Fix #7008

Fix and-cmp to test recognition

Containment is used by TreeNodeInfoInitCmp to tell genCompareInt that a "and-cmp to test" pattern has been recognized.

Before LSRA added "reg optional" the result of isContained was solely determined by TreeNodeInfoInit. Now LSRA can make operands contained and then that happens genCompareInt wrongly believes that TreeNodeInfoInitCmp recognized the pattern.

To fix this genCompareInt needs to recognize (x and c) == 0 in addition to checking isContained.

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

Update CoreClr to beta-24605-03 (master)

Update CoreClr to beta-24605-03

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

Update CoreClr to beta-24605-02 (master)

Delete mscorlib from CoreCLR targeting pack (#7494)

Update CoreClr to beta-24605-02

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

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

Merge pull request #7481 from pgavlin/RyuJITCrossgen

Add an option to crossgen S.P.CoreLib using the alt jit.

Add an option to crossgen S.P.CoreLib using the alt jit.

This also changes the x86/RyuJIT CI job to pass this option to
build.cmd. This leg will now use RyuJIT when crossgen'ing S.P.CoreLib
and when running tests.

Merge pull request #7436 from giuliohome/issue-7346

Issue 7346 ConstructorArguments from CustomAttributes for null array of enum

Merge pull request #7484 from adiaaida/assertOnNYI

Set default x86 test run to assert on nyi

Issue 7346 ConstructorArguments with null array of enum

From http://stackoverflow.com/a/39521919
In structure of the blob where the custom attribute is specified,
an enum argument is represented using the byte 0x55
followed by a string specifying the name and assembly of the enum type.
A null array is represented using a length of -1.

Unfortunately, what happens if you pass an array of enum as null is that
enum name is lost.

Fix #7346

Set default x86 test run to assert on nyi

Now that we have eliminated all of the NYIs from the base test run, set
the default to be to assert on NYI, incase we expose new NYIs.

Update CoreClr, CoreFx to beta-24604-03, beta-24604-02, respectively

Merge pull request #7476 from jkotas/security-annotations

Correct security annotations

[Linux][GDB-JIT] Fix incorrect frame location when we have a lot of locals (#7451)

* Fix incorrect frame location when we have a lot of (more than ten) locals or arguments

Add PGO GENPROFILE support to coreclr and clrjit (#7423)

* Add PGO GENPROFILE support to coreclr and clrjit

Update the cmake build system to enable support for Profile Guided
Optimization (PGO) on Windows, and enable this feature for two target
binaries (coreclr and clrjit).

With this change, toggle between instrumented and profile-optimized
settings for target binaries by passing pgoinstrument argument to the build.cmd
Assume profile-optimized mode by default. Fall back to regular non-PGO
optimized builds if profile data is not available.

API: Move Synchronized from StreamReader to TextReader (#7469)

Correct security annotations

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

Merge changes from TFS

Remove coreclr checks from AppDomain apis (#7471)

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.

Mark new UnmanagedMemoryAccessor methods as SafeCritical.

[tfs-changeset: 1630680]

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.