Merge pull request #11255 from mskvortsov/FixPutArgReg

[RyuJIT/ARM32] Update putarg_reg codegen

Merge pull request #11258 from mskvortsov/FixConsumeReporting

Do not report out-of-order consuming if a node wasn't consumed at all

Fix a double-free encountered when failing to initialize a handle table bucket (#11244)

* Fix a double-free encountered when failing to initialize a handle table bucket

* Don't free m_bucket in the destructor of HandleTableBucketHolder

Fix unittest `LPSTRTest` (#11096)

* Fix unittest `LPSTRTest`

The test case Interop.StringMarshalling.LPSTR.LPSTRTest
always returned exit code 100(pass) even if it actually failed.
This implements `ExitTest()` in the same manner with `LPTSTRTest`.

* Fix unittest `LPSTRTest` native

Merge pull request #11219 from sdmaclea/PR-ARM64-CpObj-Use-LDP-STP

[Arm64] Use ldp/stp in genCodeForCpObj()

Merge pull request #11243 from sdmaclea/PR-ARM64-LDP-ALLOW-GCREFS

[Arm64] Update emitOutputInstr to support GC refs in ldp forms

Do not report out-of-order consuming if node wasn't consumed at all

[x86/Linux] Disable USE_CALLER_SP_IN_FUNCLET in EH (#11247)

RyuJIT/ARM32: Update GT_PUTARG_REG codegen.

[Arm64/Unix] Revise JIT_MemCpy (#11143)

* [Arm64/Unix] Revise JIT_MemCpy

Use ldp/stp
Correctly handle short copy lengths
Simplify code & pseudo code
Use uint*_t to make pseudo code more readable

Fix building with only VS2017 installed. (#11182)

The fix was originally made in https://github.com/dotnet/corefx/pull/18700

Merge pull request #11240 from wtgodbe/SourceBranch

Use SourceBranch from pipebuild to define Helix Branch

Fix to respect readonly prefix for multidimensional array Address method (#11245)

Fixes #9414
- Added missing 'else' back from regression in https://github.com/dotnet/coreclr/commit/97b4ff0b438261ba11b357008630076054a6f25d#diff-edc46b80f57431489a82311948a8234dL6424, which has the effect of passing null to the Address method's instParam to indicate the readonly prefix in order to bypass the type check

Fix filename display in exception messages (#11238)

Update CoreClr, CoreFx to preview2-25226-02, preview2-25226-02, respectively (#11127)

Delete moved file

Move DllNotFoundException, EntryPointNotFoundException to CoreLib (#3415)

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

Fix timer callback time drift (#11220)

Fixes #6408:
- The change to S.T.Timer is the actual fix, but fixed the timer firing on the thread pool timer thread similarly as well

Fix comment

[Arm64] Update emitOutputInstr to support GC refs in ldp forms

Merge pull request #11235 from pgavlin/VSO424019

Require a frame pointer in methods with throw blocks for sources that…

Use SourceBranch from pipebuild to define Helix Branch

Fix and re-enable some GC finalizer tests (#11216)

* Fix and re-enable some GC finalizer tests

* Make the Usage test a little more robust

[Arm64] CpObj ldp/stp review feedback

Rework asserts
Add comments
Format

Merge pull request #11178 from sbomer/ilrt_factoring

Refactor ildasm round trip logic

Require a frame pointer in methods with throw blocks for sources that have mismatched stack depths.

This requirement effectively disables double-aligned frames (which are
effectively a special case of ESP frames). Without this change it is
possible for the JIT to generate a throw block with sources that have
mismatched offsets, which can lead to GC holes.

Fixes VSO 424019.

Merge pull request #11218 from AndyAyersMS/FixReturnTypeMismatch

JIT: fix return type mismatch inlining block

Lookup System.Runtime and mscorlib references by Assembly name

Rather than by alias.

Fixes #10595

Merge pull request #11229 from wtgodbe/CrossgenFix

Exit on failure of HelixPrep.proj

Exit on failure of HelixPrep.proj

Refactor ildasm round trip logic

This change factors the Ildasm round trip logic so that it should not
require a separate test build. The ildasmroundtrip flag is no longer
passed to build-test.cmd; instead the wrapper scripts are always
generated with a check for an environment variable that can be set from
runtests.

Also remove separate test build for ilrt in the ci scripts

Enable build with clang 4.0 (#11226)

This change enables build with clang 4.0 and fixes a bunch of new errors that
the stricter compiler was reporting.

Fix thread wait to be resilient to real time clock changes on Unix (#11213)

This change fixes a problem that happens when a thread uses a timed wait
and the real time clock change in the system (e.g. due to the user action,
time server sync, leap second, etc). In such case, the wait may end up waiting
for shorter or longer period of time.
It fixes the issue for Thread.Sleep and timed wait on a process local synchronization
primitives by using CLOCK_MONOTONIC instead of CLOCK_REALTIME for waiting on
cond vars.
Unfortunately, the waits on cross process synchronication primitives like
named mutexes always use CLOCK_REALTIME, so this fix works on process local primitives
only.

[x86/Linux] Stack alignment check in Hijack Helpers (#11221)

[x86/Linux] Use appropriate FCALL macro for DateMarshaler__ConvertToNative (#11194)

Merge pull request dotnet/corert#3429 from dotnet/nmirror

Merge nmirror to master

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

Comment explaining use of FlushProcessWriteBuffers in StompWriteBarrier. (#10924)

Fix few clang analyzer warnings (#11185)

This change fixes two issues reported by the clang analyzer. In both cases, uninitialized
variable value was used in an error code path.

Merge pull request #11107 from BruceForstall/NumSuccImprovement

Num succ improvement

Merge pull request #11204 from BruceForstall/FixArmAltjitGcinfo

Fix GCInfo library used by ARM/ARM64 protononjit builds

Merge pull request #11215 from wtgodbe/ActuallyDoCrossgen

Set RunCrossGen=true for non-Windows R2R runs

[Arm64] Use ldp/stp in genCodeForCpObj()

Move DynamicHelperFrame::UpdateRegDisplay into cgenx86.cpp (#11193)

* Move DynamicHelperFrame::UpdateRegDisplay into cgenx86.cpp

* Fix x86/Windows link error

JIT: fix return type mismatch inlining block

If caller passes a directly substitutable expression whose
type is compatible with but not the same as the callee argument type,
and that argument is the callee's return value, the jit may
reject inlining the method because of a return type mismatch.

Fix is to retype the expressions we use in the callee's body
as the callee's type.

Closes #11211.

Set RunCrossGen=true for non-Windows R2R runs

Merge pull request #11134 from pgavlin/VSO406163

Fix fgNewBBinRegion when inserting into filters.

Merge pull request #11110 from rartemev/issue_9880

Made behavior to conform to ECMA for 'tail.call, pop, ret' case

[RyuJIT/ARM32] Move genJumpKindsForTree into common part (#11103)

* Move genJumpKindsForTree() into codegenarmarch.cpp.

* Edit genJumpKindsForTree().

* Introduce genCodeForJumpTrue() for both ArmArch and XArch.

* Fix GT_JTRUE codegen for ARM32 by using genCodeForJumpTrue().

Fix GCInfo library used by ARM/ARM64 protononjit builds

Otherwise, using the normal GCInfo library, the altjits will
assert creating GC info.

Create new "cross-target" builds of the GCInfo library.

Merge pull request #11118 from rartemev/issue_10094

Fixed netci.groove scripts to make CentOS and Ubuntu R2R jitstressregs builds worked.

Made behavior to conform to ECMA for 'tail.call, pop, ret' case

Merge pull request #11206 from wtgodbe/ContainerName

Update R2R Container Names

Update R2R Container Names

Testing. DO NOT MERGE ME

Allow FEATURE_INTERPRETER to be enabled by command-line option (#11188)

* Allow FEATURE_INTERPRETER to be enabled by command-line option

* Set FEATURE_INTERPRETER as 0

Merge pull request #11187 from wtgodbe/NonWindowsR2R

Enable running R2R tests in Helix on non-Windows

Enable running R2R tests in Helix on non-Windows

Merge pull request #11203 from mskvortsov/FixLclFld

[RyuJIT/ARM32] Update lclFld codegen

[ARM32] clang 3.9 as a default for ARM cross build (#11064)

* [ARM32] clang 3.9 as a default for ARM cross build

Use clang 3.9 as a default compiler for ARM cross build

* Both arm and armel will be built using clang 3.9 as a default
* ARM CI will use clang 3.9 as a default
* ARM pipieline build will use clang 3.9 as a default

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>
* [ARM32] Update docs describing clang-3.9 and ARM cross build

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>
* [ARM] Check minimum version requirement of clang for ARM cross build

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

Fix fgNewBBinRegion when inserting into filters.

In order to ensure that the GC operates properly when handling
exceptions, the CLR ABI requires that control exits a filter region
throguh its terminal instruction. `fgNewBBinRegion` was violating this
invariant, however, which lead to GC stress failures when the GC was
invoked immediately after a filter finished executing but before control
entered any handlers. This change fixes the behavior of
`fgNewBBinRegion` when inserting new blocks into filter regions.

There are two cases to consider when inserting a new BB into a filter
region:
1. The filter region consists of multiple blocks
2. The filter region consists of a single block

The first case is straightforward: instead of inserting a new BB after
the last block of the filter, insert it before the last block of the
filter. Because the filter contains multiple blocks, the predecessor of
the filter must also be in the filter region.

The latter case requires splitting the single block, as inserting a new
block before the single block would otherwise change control flow (the
filter would begin with the new block rather than the single block). In
order to acheive this with minimal complexity, this change first inserts
a `BBJ_ALWAYS` block that transfers control to the existing single
block, then inserts a second new block immediately before the existing
single block. The second new block is then returned. To put it
visually, the transformation is from this:

```
    filter start: BBN (BBJ_EHFILTERRET)
```

to this:

```
    filter start: BBN + 1 (BBJ_ALWAYS -> BBN)
                  BBN + 2 (jumpKind)
                  BBN     (BBJ_EHFILTERRET)
```

and the function returns `BBN + 2`.

Fixes VSO 406163.

Merge pull request dotnet/corert#3418 from dotnet/nmirror (#11186)

Merge nmirror to master

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

Merge pull request #11191 from BruceForstall/FixFuncletAssert

Fix funclet begin assert

Merge pull request #11080 from BruceForstall/MeasureNoway

Add code to measure dynamic noway_assert usage

RyuJIT/ARM32: Update GT_LCL_FLD codegen.

[x86/Linux] Do not report untracked from filters (#11195)

Related Issue : #11101

Add code to measure dynamic noway_assert usage

Set COMPlus_JitMeasureNowayAssert=1 to enable measurement.
Set COMPlus_JitMeasureNowayAssertFile to a path/file to send
output to a file instead of stdout.

Code is under `#if MEASURE_NOWAY` which is enabled for DEBUG
only, though it can be manually enabled for non-DEBUG builds.

Fix funclet begin assert

Merge pull request #11196 from wateret/fix-formatting

Fix formatting fail

Merge pull request #11183 from sdmaclea/PR-FIX-11181

Fix 11181

[x86/Linux] Change CI docker image (#11098)

* [x86/Linux] Change CI docker image

Change CI docker image to support clang-3.9

* Update x86_ci_script.sh

change docker tag name

* Update x86_ci_script.sh

environment set correctly: ROOTFS_DIR

* Update x86_ci_script.sh

change docker image name

Fix formatting fail

Ensure using the given executable path. (#10525)

* Ensure using the given executable path.

Fix the bug that corerun trying to load the executable from current directory even if the user has specified a full path of the executable.

Fix #5631

* Revert "Ensure using the given executable path."

This reverts commit d237e6329f85132429176a0644cf6d93c9437ff4.

* Partially implemented.

* Ensure app context using correct IL file path.

Extracting simple name from the given file path and look up its value in tpa list. If the value does not equal to the given file path, we update the value with correct given file path.

Fix #5631

* Re-arrange the slashIndex to eliminate unneccessary code.

* A few performance improvement with a memory leak bug fixed.

* Added the absolute path of target assembly to tpa list.

Fix #5631

* Added the path of target assembly to tpa on Windows.

Fix #5631

Add "createdump" command to SOS. (#11161)

Uses the createdump utility code as a library.

Changed the "-m, --micro" option to "-t, --triage".

Lots of cleanup.

Merge pull request #11152 from AndyAyersMS/FixTailCallStressAssert

JIT: simple fix for the tail call stress assert

Merge pull request #11157 from mikedn/reg-encoding

Simplify emitter's RegEncoding

Merge pull request #11108 from BruceForstall/RangedForAllSuccessorIter

Use C++ ranged 'for' for AllSuccessorIter use

Merge pull request #11172 from mskvortsov/ArithOverflowChecks

[RyuJIT/ARM32] Enable arithmetic overflow checks

Support other toolsets in Win/ARM64 build

Fix 11181

Fix cmake warning about mismatched if/endif comment (#11176)

Remove PAL FileAlignment restriction (#10959)

* Remove PAL FileAlignment restriction

* Address PR #10959 feedback

* Fix amd64 crossgen error

* Respond to review feedback

* Fix arm32 regression

* Prepare to remove VIRTUAL_PAGE_* from map.cpp

Also simplify previous section code

* Rename function to GetVirtualPageSize()

[Local GC] Ensure that handle creation returns null on failure instead of throwing (#11092)

* [Local GC] Ensure that handle creation returns null on failure instead of throwing

* Fix some clang pedantry about jumping past variable initialization

* Throw OOM if initialization of handle store fails

* Perform the handle OOM check in each handle helper

do not use = {nullptr} initialization. (#11153)

It worked incorrectly on desktop.

Split NumSucc/GetSucc into two versions

One set takes Compiler*, one doesn't.

It's simpler, and hopefully faster.

Merge pull request #11169 from hseok-oh/ryujit/fix_11128

[ARM32/RyuJIT] Fix failure related with struct argument passing

Merge pull request #11091 from BruceForstall/BlockRefactor

Refactoring: move some BasicBlock functions to block.cpp

RyuJIT/ARM32: Enable arithmetic overflow checks.

Support FEATURE_DBGIPC override (#11164)

[x86/Linux] Remove unnecessary ResumeEsp field (#11167)

[ARM32/RyuJIT] Fix failure related with struct argument passing

Block using NYI_ARM when struct value split between registers and stack
Fix type assign error for struct slot

[x86/Linux] Fix unmatched NESTED_END in CallEHFilterFunclet (#11163)

11106: Use RuntimeHelpers.IsReferenceOrContainsReferences in Dictionary.Remove methods (#11158)

Simplify emitter's RegEncoding

VC++ does't recognize that `(c ? x - 16 : x) & 7` is just `x & 7` and we end up with 5 instructions instead of 1.

Mark GetGenerationWR2 as GCStressIncompatible across all platforms (#11151)

Remove destructor from GCEvent and instead rely on the OS to clean up (#11132)

* Remove destructor from GCEvent and instead rely on the OS to clean up

* Add a comment justifying the lack of destructor

* wording: many -> all

Two improvments for LCG jump stubs (dynamic methods): (#9160)

When requesting a jump stub block for LCG, ask for a block of 4 jump stubs instead of 32
    Enable caching and cache lookup for LCG jump stubs

JIT: simple fix for the tail call stress assert

If class handles match, the return types are compatible, no matter
what the jit types indicate.

Addresses #10047.

Move xplat minidump doc to botr. (#11149)

Change default minidump type to MiniDumpWithPrivateReadWriteMemory.

Remove language about uploading triage dumps.

Turn off diagnostic output.

Move CoreCLR to Preview2 (#11146)

Merge pull request #11140 from AndyAyersMS/FixRunBenchmarks

Update jit RunBenchmarks test to no longer use XPath