Merge pull request #8697 from sivarv/lockAddFix

Fix GT_LOCKADD register specification.

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

Update CoreClr, CoreFx to beta-24821-03, beta-24821-01, respectively (master)

Remove FEATURE_BLC_FORMATTING (#8699)

This feature is always defined and the PAL functions that would be used
if the feature was not defined are not in PAL anymore.

Update CoreClr, CoreFx to beta-24821-03, beta-24821-01, respectively

Fix GT_LOCKADD register specification.

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

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

Merge pull request #8693 from JosephTremoulet/ValueNumberField

Use field type value-numbering local field stores

Use field type value-numbering local field stores

Method `VNPairApplySelectorsAssign` takes the type of the value being
assigned, so when processing a store to a field of a local struct, pass
the type of the field rather than the type of the local; failure to do so
was blocking propagation of the value number to subsequent loads of the
same field due to the type mismatch.

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

Change wrong indentations in GetSecureInvoke() codes (#8688)

In the before #8592 merged PR, several indentations in modified GetSecureInvoke method were wrong.
So I request additional fixed PR.

1. 2 space indentations -> 4 spaces
2. tab indentation -> 4 spaces

ARM: do not use unpredictable sub.w sp, rN, #imm (#8636)

[x86/Linux] Implement libunwind-based unwindLazyStack (#8686)

* [x86/Linux] Implement libunwind-based unwindLazyStack

This commit implements libunwind-based unwindLazyStack for x86/Linux
in order to fix #8625.

Fix the bug that Secure Delegate Stubs are compiled every time. (#8592)

* Fix the bug that Secure Delegate Stubs are compiled every time.

I found that Secure Delegate stubs are compiled every time during application execution, which has a negative impact on execution performance. #8554

Like the GetMulticastInvoke() method, GetSecureInvoke() checks the hashtable when the method is executed and uses it if it already exists.

* Fix pDelMT bug and change file rights

Merge pull request #8681 from gkhanna79/PNBBuildPkg

Update build-packages.sh to support portableLinux

Update build-packages.sh to support portableLinux

Add Encoding.GetBytes(string, offset, count) (#8651)

Fix DllImport of IdnToAscii & IdnToUnicode (#8666)

Fix an issue found in OneCoreUAP testing. According to MSDN,
the official exporting DLL for IdnToAccii and IdnToUnicode
is normaliz.dll, not kernel32.dll. While most Windows SKUs
export these functions from both normaliz.dll and kernel32.dll,
recent tests revealed that some Windows SKUs export them from
normaliz.dll only.

Merge pull request #8674 from AndyAyersMS/BiggerBlockFlag

Widen basic block flag field to 64 bits

[Linux][GDB-JIT] Fix bugs in gdbjit that break lldb stepping (#8637)

* Fix .text and .thunk symbols overlapping

When current method calls itself, a __thunk* symbol might be generated with the same address
as the method symbol in .text section. Avoid generating such __thunk* symbol.

* Do not create DWARF line table entries with the same address

* For each HiddenLine assign a zero line number in DWARF

Allow LLDB to to skip HiddenLines when stepping.

* Fix __thunk symbols containing garbage

Fix a bug when __thunk* symbols of previously compiled methods cause
generation of __thunk* symbols for currently compiled method without
filling symbol info.

* Fix missing check for the end of list of compiled methods

* Remove unnecessary check for zero prevLine in gdbjit

[x86/Linux] Add UMThunkStub (#8627)

Add UMThunkStub method with logic from that of AMD64

Fixing up the arm subsystem versions (#8676)

* Adding arm64 and updating default subsystems

Remove Read/WriteProcessMemory from PAL. (#8655)

Ifdef more unused code that uses ReadProcessMemory. Move the current
memory probing in the transport to PAL_ProbeMemory. Add PAL_ProbeMemory
to dac PAL exports.

PAL_ProbeMemory may be changed to use write/read on a pipe to
validate the memory as soon as we make it perform as well as
the current code.

Remove ReadProcessMemory tests and add PAL_ProbeMemory pal tests.

Use ExecutionPolicy ByPass to execute probe-win.ps1 script during the build (#8673)

Port of https://github.com/dotnet/corert/issues/2377

Widen basic block flag field to 64 bits

Flag field is currently full, and I need at least one more bit to identify
the start of a cloned finally.

Widen to 64 bits and update a few uses. Also removed an unused copy.

Merge pull request #8668 from gkhanna79/PNBRid

Packaging support for portable Linux binaries.

Packaging support for portable Linux binaries.

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

Update CoreClr to beta-24816-04 (master)

Avoid Unsafe.AsRef in Span<T> implementation (#8657)

The JIT is not able to inline the current implementation of Unsafe.AsRef because of it has type mismatch on return. Change the the corelib Span to call Unsafe.As instead since fixing the type mismatch is not easy in the internal corelib version of Unsafe.AsRef.

Dictionary.GetValueOrDefault (#8641)

* Dictionary.GetValueOrDefault

* Fixed IDictionary.TryGetValue

* remove extensions

* // Method similar to TryGetValue that returns the value instead of putting it in an out param.

* public TValue GetValueOrDefault(TKey key) => GetValueOrDefault(key, default(TValue));

Update CoreClr to beta-24816-04

Fix buildsystem for linux cross-architecture component build (#8646)

* Fix buildsystem for linux cross-architecture component build

* refactoring build.sh, bug fix and typo fix

* Update build.sh

Makes CultureInfo.get_Parent thread safe (#8656)

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

Update CoreFx to beta-24816-02 (master)

Update CoreFx to beta-24816-02

[Linux][GDB-JIT] Add try/catch blocks to methods in DWARF (#8650)

* Add try/catch blocks to DWARF info

Use info about exception handling from IL and map it to
native code as try/catch blocks in DWARF.

* Improve locals naming convention consistency in gdbjit

* Drop pointer to line info from FunctionMember after it is dumped

Make it easier to use StringComparison & StringComparer with GetHashCode (#8633)

* Make it easier to use StringComparison & StringComparer with GetHashCode

* model.xml

Switch GCSample to the canonical GCToOSInterface implementation (#8653)

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

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

Merge pull request #8642 from CarolEidt/Fix359736

Correctly sequence fgMorphModToSubMulDiv

Update CoreClr, CoreFx to beta-24815-03, beta-24815-03, respectively

Remove API Set dependency (#8624)

Merge pull request #8207 from CarolEidt/StreamlineResolution

Streamline LSRA resolution

Merge pull request #8329 from litian2025/Fix_SIMDScalarMoveEncoding

Fix SIMD Scalar Move Encoding: VEX.L should be 0

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

Update CoreFx to beta-24815-01 (master)

Update CoreFx to beta-24815-01

Merge pull request #8546 from alsemenn/debuggertests_generate_scripts_win

Add script generator and generate test scripts to adapt debuggertests…

Merge pull request #8640 from pgavlin/VSO359734

Fix ref count adjustment in `fgMorphBlockStmt`.

Fix SIMD Scalar Move Encoding: VEX.L should be 0

For SIMD Scalar Move instructions such as vmovlpd, vmovlps, vmovhps,
vmovhps and vmovss on AVX system, JIT should ensure that those
instructions are encoded with VEX.L=0, because encoding them with
VEX.L=1 may encounter unpredictable behavior across different
processor generations. The reason of VEX.L is encoded with 1 is
because JIT calls compiler->getSIMDVectorType() which returns
EA_32BYTE, and it is been passed into emitter AddVexPrefix() which
ends up encoded VEX.L=1, the fix is to pass target type and base
type for those instructions to ensure that VEX.L=0 at emitter
AddVexPrefix()

Fix #8328

Correctly sequence fgMorphModToSubMulDiv

This method was creating a temp, but the final result was a GT_SUB with
a use of the temp as its op1, and it was not setting GTF_REVERSE_OPS.
This led to a liveness assert in LSRA.

Add a regression test.

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

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

Fix ref count adjustment in `fgMorphBlockStmt`.

LclVar ref counts must be incremented before attempting to remove the
morphed statement, since doing so decrements ref counts (and thus
requires refcounts to be conservatively correct).

Fixes VSO 359734.

[Linux][GDB-JIT] Add simple C++ mangling of method names in GDBJIT DWARF (#8638)

* Add simple C++ mangling of method names in GDBJIT DWARF

Example:
    Namespace1.Class1.Method -> void Namespace1_Class1::Method()

* Do not convert a name during mangling if target buffer is NULL

Merge pull request #8601 from pgavlin/gh7963

Fix consume-order checking in codegen.

Add support for R2R ldvirtftn helpers (#8608)

The codegen for the non-readytorun path (used on CoreCLR) requires the
runtime to support general purpose virtual method resolution based on a
method handle. CoreRT doesn't have such helpers.

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

Move RUNTIME_FUNCTION__BeginAddress into clrnt.h (#8632)

RUNTIME_FUNCTION__BeginAddress is defined in corcompile.h for x86, but
is defined in clrnt.h for all the other architectures.

This commit moves RUNTIME_FUNCTION__BeginAddress defines for x86 into
clrnt.h to make it consistent.

[x86/Linux] Adds Dummy Exception Handler (#8613)

This commit adds a dummy exeption handler to enable x86/Linux build.

This commit also reverts 7b92136d5ee to make it easy to enable
WIN64EXCEPTIONS in x86/Linux.

[ARM32/Linux] Initial bring up of FEATURE_INTERPRETER (#8594)

* Bring up FEATURE_INTERPRETER for ARM32/Linux
* Add a disclaimer message for GC preemption workaround

Adding method implementations to fast span/readonlyspan  (#8607)

[x86/Linux] Enforce 16-byte stack alignment (#8587)

Clang (and GCC) requires 16-byte stack alignment, but the current
implementation of CallDescrInternal and ThePreStub does not provide any
guarantee on stack alignment.

This commit adds 16-byte stack alignment adjust code inside these functions.

[x86/Linux] Fix incorrect __fastcall definition (#8585)

In x86/Linux, __fastcall is defined as __stdcall which causes stack
corruption issue for the following functions:
 - HelperMethodFrameRestoreState
 - LazyMachStateCaptureState

This commit removes __fastcall definition as clang recognize __fastcall.

[x86/Linux] Fix "Bad opcode" assert in unwindLazyState (#8609)

* [x86/Linux] Fix "Bad opcode" assert in unwindLazyState

This commit suppresses "Bad opcode" assert while runing "Hello, World" example.

This commit address the following three code patterns discovered while
digging the assert failure:
 - and $0x1, %al
 - xor $0xff, %al
 - stack protection code:
   mov %gs:<off>, <reg>
   cmp <off>(%esp), <reg>
   mov <reg>, <off>($esp)
   jne <disp32>

This commit revises LazyMachState::unwindLazyState to handle the first two patterns,
and revises compile options not to emit the third pattern.

Merge pull request #8617 from hqueue/arm/test20161210/IsContainableImmed

Ryujit/ARM32 Initial Lowering::IsContainableImmed for ARM

[x86/Linux] Port jithelp.asm (#8491)

* [x86/Linux] Port jithelp.asm

This commit ports jithelp.asm for x86/Linux

The following Tailcall helpers are marked as NYI:
 - JIT_TailCall
 - JIT_TailCallReturnFromVSD
 - JIT_TailCallVSDLeave
 - JIT_TailCallLeave

* Revise macro and indentation

Fix the ARM32 build.

Remove no-op file security (#8611)

Deletes FileSecurityState and pulls redundant methods. Also
removes DriveInfo, which isn't in use in Core.

Fix consume-order checking in codegen.

The switch to LIR invalidated the correspondence between a node's
sequence number and the order in which it must be consumed with respect
to other nodes. This in turn made the consume-order checks during code
generation incorrect.

This change introduces a new field, `gtUseNum`, that is used during code
generation to check the order in which nodes are consumed. Re-enabling
these checks revealed a bug in code generation for locked instructions
on x86, which were consuming their operands out-of-order; this change
also contains a fix for that bug.

Fixes #7963.

Merge pull request #8560 from AlexRadch/Deconstruction

Supporting C# 7 deconstruction of KeyValuePair and DictionaryEntry

Merge pull request #8621 from mikedn/cmp-cast-drop

Fix incorrect compare narrowing in TreeNodeInfoInitCmp

Fix incorrect compare narrowing in TreeNodeInfoInitCmp

TreeNodeInfoInitCmp attempts to eliminate the cast from
`cmp(cast<ubyte>(x), icon)` by narrowing the compare to ubyte. This should
only happen if the constant fits in a byte so it can be narrowed too,
otherwise codegen produces an int sized compare. (or a byte sized compare
with a truncated constant if we try to use GTF_RELOP_SMALL).

Enable interop debugging for Windows amd64 and x86. (#8603)

Found sos portable pdb problem on x86.  Fixed interop problems between the native sos and SOS.NetCore managed helper assembly.

Synchronize src\mscorlib\corefx with the CoreRT fork (#8619)

Remove managed environment cache (#8604)

The PAL already caches- no need to do it in managed.
Fold the code back into Environment.cs.

Delete HostProtection attributes (#8610)

Ryujit/ARM32 Initial Lowering::IsContainableImmed for ARM

Initial implementation of IsContainableImmed for ARM.

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>

Merge pull request #8520 from hqueue/arm/test20161208/lowercast

Ryujit/ARM32: Implement Lowering::LowerCast for ARM

Ryujit/ARM32 Implement Lowering::LowerCast for ARM

Simple integer to interger type conversion is passed.
Add comment for LowreCast with C++ comment style.

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>

Merge pull request #8510 from DrewScoggins/CommitNameFix

Add ability to give a name to a manual PR run

When using the PR leg of the Jenkins performance jobs it is now possible to specify a name that will be used when inserting the data into the perf database.

Add ability to give a name to a PR run

Rename Contract.Assert to Debug.Assert (#8600)

[x86/Linux] implement TheUMEntryPrestub (#8589)

Initial code for x86 TheUMEntryPrestub, UMThunkStub

Fix longname DAC to enable arm[64] symbol packages (#8574)

Merge pull request #8532 from sivarv/fixedRegFix

Fix to issue 8286.

Merge pull request #8598 from adiaaida/disableGetGeneration

Disable GetGenerationWR2 for GCStress on x86

Merge pull request #8541 from briansull/vso-287671

Fix missing flags on GT_DYN_BLK node

// BLOCKED (do not add now): [EditorBrowsable(EditorBrowsableState.Never)]

Disable GetGenerationWR2 for GCStress on x86

Disable this test for GCStress on x86 until the cause for its failure
can be investigated.

Merge pull request #8421 from hseok-oh/enable_arm_linux_nupkg

First step to create nuget packages for ARM32/Linux

Span<T> api update (#8583)

* Changing method/property order to match CoreFX impl

To make diffing the files easier

* Added other missing methods to match CoreFX impl

Added:
- public void CopyTo(Span<T> destination)
- public static bool operator ==(Span<T> left, Span<T> right)
- public static bool operator !=(Span<T> left, Span<T> right)
- public override bool Equals(object obj)
- public override int GetHashCode()

Also removed 'public void Set(ReadOnlySpan<T> values)' and it's no
longer part of the Span<T> public API, see

https://github.com/dotnet/apireviews/tree/master/2016/11-04-SpanOfT#spantset

[x86/Linux][SOS] Don't include utilcode.h in gcdumpx86.cpp and gcdump.cpp (#8475)

[x86/Linux][SOS] Add CALLBACK (aka stdcall) to function declarations (#8476)

Local GC: Decouple write barrier operations between the GC and EE (#8568)

* Decouple write barrier operations between the GC and EE

* Address code review feedback

* Address code review feedback

* Repair the standalone GC build

Fix typo in clang-format directive and reformat end of flowgraph.cpp (#8573)

Last 4K or so lines of flowgraph.cpp were not being formatted because
the clang-format on directive had a typo.

Fix the typo and reformat the latter part of the file.

Merge pull request #8581 from stephentoub/cwt_unsafecast

Use JitHelpers.UnsafeCast in ConditionalWeakTable

Use JitHelpers.UnsafeCast in ConditionalWeakTable

We know the types and can use UnsafeCast when reading out the objects as TKey and TValue.  This improves reading speed by ~30%.

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

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

Improve ConditionalWeakTable.Remove (#8580)

Clear the key of the deleted entry to allow GC collect objects pointed to by it

Fixes #8577

TimeZoneInfo: Avoid cloning privately-created ArgumentRule[] arrays (#8575)

TimeZoneInfo currently always creates a defensive copy of the specified
ArgumentRule[] array when created. This makes sense for the public
static factory methods. However, there's no need to create a defensive
copy of arrays created privately as part of its implementation (e.g.
reading the rules from the registry/disk). This change avoids the
unnecessary cloning.

Fix typos and grammer in coreclr README.md (#8561)

* Fix typeos and grammer in README.md

* Fix a small grammar issue and remove a comma