Merge pull request #11215 from wtgodbe/ActuallyDoCrossgen

Set RunCrossGen=true for non-Windows R2R runs

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

* Move DynamicHelperFrame::UpdateRegDisplay into cgenx86.cpp

* Fix x86/Windows link error

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

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.

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

Merge pull request #11148 from wtgodbe/DisableCoreMangLib

Disable DynMethodJumpStubTests test

Disable DynMethodJumpStubTests test

Merge pull request #11117 from briansull/zapsig-fix

ReadyToRun/Crossgen: Disable cross module generic instantiations from profile data

Merge pull request #11147 from dotnet/ReEnableHashCodeTest

Re-enable Hashcode Test

Re-enable Hashcode Test

Merge pull request #11144 from wtgodbe/DisableSomeTests

Temporarily disable failing tests in Helix

Temporarily disable failing tests in Helix

Update jit RunBenchmarks test to no longer use XPath

Rewrite to use linq-to-xml instead. Verified at least that this
can still parse the adjoining xml file. Also now able to crossgen
this test case.

Closes #11082.

Merge pull request #11138 from wtgodbe/WrongCrossgen

Copy Runtime-specific Crossgen to Core_Root

Copy Runtime-specific Crossgen to Core_Root

Merge pull request #11055 from jorive/dev/xUnit-Performance-API

Improving robustness of the run-xunit-perf.cmd script.

[x86/Linux] Correctly initialize nonVolRegPtrs in LazyMachState::unwindLazyState (#10888)

Mark CharEnumerator as [Serializable] (#11124)

Fixes https://github.com/dotnet/corefx/issues/18704

[x86/Linux] Fix pow of tryrun (#11099)

x86/Linux has compatible math pow()
This will fix CoreFX System.Tests.MathFTests.Pow() failures

Add Marshal.GetExceptionPointers (#11125)

* Add Marshal.GetExceptionPointers

* Remove SecurityCritical

[ARM32/RyuJIT] Argument passing for multireg struct (#10846)

* [Linux/ARM32] Enable fgMorphMultiregStructArgs

Enable fgMorphMultiregStructArgs: morph struct argument to GT_FILED_LIST when size <= 4
Cannot handle split struct that using register and stack

* Fix formatting

Fix formatting error

* Fix typo and commit with not added file

- Fix typo
- Commit with not added file: src/jit/compiler.h

* Fix formatting and build error

* Fix comment and block using NYI macro

- Fix comment
- Block using NYI when struct passed by value can be split between registers and stack

[x86/Linux] Relax inst_JMP assert condition (#11062)

* [x86/Linux] Do NOT use exception-throwing block on ESP framed method

* Fix format error

* Relax assert condition

* Fix format error

* Revert changes in codegencommon.cpp

* Add parentheses

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

Update CoreClr, CoreFx to preview1-25221-01, preview1-25221-01, respectively (master)

Update CoreClr, CoreFx to preview1-25221-01, preview1-25221-01, respectively

Update StringBuilder.AppendJoin to approved API  (#11059)

* Updated StringBuilder.AppendJoin to approved API

* Organized StringBuilder.AppendJoin overloads

* Added ThrowHelper, removed Contract.Ensures

* Inline array accesses

Enable corerun to select Globalization Invariant mode (#11097)

* Enable corerun to select Globalization Invariant mode

* Support 'true' also

Fix R2R workaround for Diasymreader (#11116)

Inline List.Clear (#11104)

When we have profile data we can have cross module references in our ZapSig's when building ReadyToRun images.
We should return a null TypeHandle to indicate failure so that we don't attempt to add generic types/methods
that have cross module references.  (i.e. MyList<System.Guid> )

Fixing Delegate's hash code's distribution (#11019)

Delegate's GetHashCode just returns the hash code of the delegate type. For a scenario where delegates are used as keys in a dictionary, this leads to obvious performance problems. We should look at coming up with a better GetHashCode that properly factors in the target object and method, in all of the various forms a delegate can take.

Merge pull request #11115 from wtgodbe/readytoruntests

Move readytorun tests into directory one level deeper

Merge pull request #11033 from adityamandaleeka/handle_table_local_gc_store

Move handle object manipulation functions to interface

Move readytorun tests into directory one level deeper

Merge pull request #11020 from adiaaida/hardenPerf

Strip quotes out of BenchviewCommitName

Rename CompareAndSwap to InterlockedCompareExchange.

Merge pull request #11086 from wtgodbe/SecurityPermissions

Add dependency on System.Security.Permissions

Add dependency on System.Security.Permissions

Add the Linux minidump support documentation (#11087)

Add the Linux minidump support documentation

One minor fix to createdump.

Use C++ ranged 'for' for AllSuccessorIter use

[x86/Linux] Use CDECL as STDAPICALLTYPE (#10969)

* [x86/Linux] Use CDECL as STDAPICALLTYPE

* Use system default calling convention for internal functions

Strip quotes out of BenchviewCommitName

When we pass the name of the pr to submission-metadata.py, we need the
name of the pr to not include quotes, or submission-metadata.py will fail
because it has the "wrong" parameters (because parts of the name are no
longer quoted). This change strips quotation marks out of the benchview
commit name to harden against this kind of failure.

[x86/Linux] fix a problem that stack was broken by changing stdcall to cdecl in case of readytorun. (#11063)

* [x86/Linux] fix a problem that stack was broken by changing stdcall to cdecl in case of readytorun.

add DynamicHelperArgsStub and change jmp to call.

* [x86/Linux] set cfi_def_cfa_offset of DynamicHelperArgsStub to 16.

Merge pull request #11083 from AndyAyersMS/FixNonDevUseOfCheckProjects

Update CheckProjects to not fail for non-dev CORE_ROOT

Refactoring: move some BasicBlock functions to block.cpp

(where they belong)

Delete unused containsStatement().

Merge pull request #11081 from mikedn/static-noway

Convert some noway_assert to static_assert

Allow to run all benchmarks and even upload, but fail the job.

Add LinkBench to CoreCLR BenchView

This change adds Linker tests as scenario tests tracked in
BenchView for CoreCLR benchmarks.

The Benchmarks added are:
HelloWorld, WebAPI, CoreFx, MusicStore (MSIL and ReadyToRun), Roslyn.

The metrics are:
MSIL size before and after ILLINK
Total publish-dir size before and after ILLINK
% Reduction in MSIL and Total dir-size after linking.

Move ResetOBJECTHANDLE and the associated holder to the VM side.

Merge pull request #11077 from pgavlin/gh10940

Spill side-effects in impAssignMultiRegTypeToVar.