Merge pull request #18766 from CarolEidt/Fix18765

Build uses for atomic ops

Merge pull request #18716 from BruceForstall/RemoveTstJobsFromViews

Remove build-only and TST jobs from the Jenkins views

Update CoreClr, PgoData to preview1-26703-04, master-20180703-0030, respectively (#18761)

Build uses for atomic ops

Fix #18765

Use generic win-x86/win-x64 RIDs in stress_dependencies

Add back most CreateInstance APIs to AppDomain and Activator (#18751)

* add ObjectHandle.cs

* add signature placeholder

* move CreateInstance from netfx

* move CreateInstanceFrom from netfx

* nit: visibility, text formatting

Update CoreFX CI packages to NetcoreApp3 (#18760)

* Update to FX CI packages to NetcoreApp3

* Add Extra Dependencies

* Enable all tests fixed by change

Updating SSE_StaticCast and AVX_StaticCast to set the correct type on the returned node.

Fold away Sse.StaticCast and Avx.StaticCast in the importer

Adding an Avx.StaticCast test (modified from the Sse.StaticCast test)

SuperPMI: add ability to exclude failing method contexts from replays (#18721)

* format spmi

* rename mchFile to mchFileName

* refactor ProcessChildStdOut

* add a stub to exclude methods

Set the activation context for the CoreRun application to what is defined (#18728)

in the target managed assembly. This allows RegFree COM scenarios and ensures
the intended app manifest is used for the 'exe' scenario.

Debug registers cannot be copied in user mode (#18730)

Dispatch Runtime Events to EventListener (#18649)

Update BuildTools, CoreClr, PgoData to preview1-03002-01, preview1-26702-04, master-20180702-0047, respectively (#18731)

Move RuntimeIDArg initialization (#18747)

Fixing up the Sse41.Insert float HWIntrinsics

reenable fixed arm64 altjit tests (#18722)

Add back DefinePInvokeMethod (#18742)

Adding back the tests for Avx.MaskLoad

Updating the Avx.Extract/Insert methods to throw PNSE when IsSupported is false

a

Removing unnecessary `try/catch` blocks from the ExtractScalar and InsertScalar HWIntrinsic test templates

Tabs vs. spaces (#18740)

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

Fix some build breaks seen with the older VS we use to build with on desktop. (#18736)

Fix recursive inlining of PInvoke stubs (#18737)

PInvoke stubs can be inlined into a regular methods in CoreRT. PInvoke transition
has to be inlined as well when that happens. Otherwise, there is a risk of recursive
inlining in corner cases.

Pass CompAllocator by value (#15025)

Passing CompAllocator objects by value is advantageous because it no longer needs to be dynamically allocated and cached. CompAllocator instances can now be freely created, copied and stored, which makes adding new CompMemKind values easier.

Together with other cleanup this also improves memory allocation performance by removing some extra levels of indirection that were previously required - jitstd::allocator had a pointer to CompAllocator, CompAllocator had a pointer to Compiler and Compiler finally had a pointer to ArenaAllocator. Without MEASURE_MEM_ALLOC enabled, both jitstd::allocator and CompAllocator now just contain a pointer to ArenaAllocator. When MEASURE_MEM_ALLOC is enabled CompAllocator also contains a pointer but to a MemStatsAllocator object that holds the relevant memory kind. This way CompAllocator is always pointer sized so that enabling MEASURE_MEM_ALLOC does not result in increased memory usage due to objects that store a CompAllocator instance.

In order to implement this, 2 additional signficant changes have been made:
* MemStats has been moved to ArenaAllocator, it's after all the allocator's job to maintain statistics. This also fixes some issues related to memory statistics, such as not tracking the memory allocated by the inlinee compiler (since that one used its own MemStats instance).
* Extract the arena page pooling logic out of the allocator. It doesn't make sense to pool an allocator, it has very little state that can actually be reused and everyting else (including MemStats) needs to be reset on reuse. What really needs to be pooled is just a page of memory.

Since this was touching allocation code the opportunity has been used to perform additional cleanup:
* Remove unnecessary LSRA ListElementAllocator
* Remove compGetMem and compGetMemArray
* Make CompAllocator and HostAllocator more like the std allocator
* Update HashTable to use CompAllocator
* Update ArrayStack to use CompAllocator
* Move CompAllocator & friends to alloc.h

Update CoreClr, PgoData to preview1-26630-01, master-20180630-0049, respectively (#18729)

Fix build breaks with older MSVC compiler (#18725)

Fixing some Bmi1 HWIntrinsic method names (#18718)

Regex: reduce allocation slightly, add tests, code cleanup, add parser comments (#30632)

* RegexParser & optionsstack ref

* Add test coverage for Group.Synchronized

* Adjust options mode test case

* Add inline comment '#' test branch

* Add comments

* Replace manual ToLower calls by Span.ToLower

* Make applicable fields readonly in parser

* Change to Assert to reduce an if check in one branch

* Code formatting

* Avoid string allocation when IgnoreCase set

Prefix patterns which are passed to RegexBoyerMoore are already
lowercased by the parser. Remove the redundant ToLower() call and assert
the patterns lowercase state

* Add surrogate pair positive & negative tests

* Add test cases for rtl anchor

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

Merge pull request #18715 from BruceForstall/FixSignedComparison

Fix issue with signed/unsigned comparison

optimize fgMorphTree for GenTreeArgList (#18582)

* add fgMorphArgList

* hide the call inside fgMorphSmpOp

* fix grammar mistakes

Remove build-only and TST jobs from the Jenkins views

We only want non-PR, "top-level" jobs in the views. Build-only
and TST jobs are always "child" jobs of other flow jobs, which
will be in the views.

Fix issue with signed/unsigned comparison

On Mac (maybe on other platforms using clang?) the following set of
comparisons in the emitter were done as unsigned:
```
(val >= 0xFFFFFFFF80000000LL)
```
even though 'val' is signed. This led to assertion failures. On Windows,
the comparisons were signed.

To fix this, cast the constant to the signed `ssize_t` type (to match `val`).

Fixes #18341

Merge pull request #18693 from BruceForstall/FixLinuxX64FlowJobs

Stop creating Linux/x64 corefx flow jobs

Change CancellationTokenRegistration.Unregister visibility from internal to public (#18698)

Corrected a few typos in the documentation and comments (#18706)

* Corrected a few typos in the documentation and comments

* Corrected a few typos in the documentation and comments

* Corrected a few typos in the documentation and comments

* Corrected a few typos in the documentation and comments

* Corrected a few typos in the documentation and comments

* Corrected a few typos in the documentation and comments

Improve bool.TryFormat perf (#18711)

* Improve bool.TryFormat perf

Improves throughput by ~50%, by avoiding AsSpan().CopyTo and just writing out the characters one-by-one.  I experimented with playing the same tricks around storing the data in a ulong and blitting it out to the destination reinterpreted as a span of ulongs, but it didn't make a measurable improvement and increased the code complexity.

* Update Boolean.cs

Merge pull request #18696 from CarolEidt/Fix18362

Fix & test for #18362

JIT: fix regression when returning struct with no fields on SysV (#18708)

The runtime classifies such structs as being returned by reference, so we
need to follow suit in the jit. If the classifier says the value can't be
returned in a register, then so be it.

Add a test case.

Add public implementation CultureDataSupport (#18691)

Update BuildTools, CoreClr, CoreFx, PgoData to preview1-02929-01, preview1-26629-04, preview1-26628-03, master-20180629-0054, respectively (#18694)

Remove relocations for vtable chunks (#17147)

* Separate sections READONLY_VCHUNKS and READONLY_DICTIONARY

* Remove relocations for second-level indirection of Vtable in case FEATURE_NGEN_RELOCS_OPTIMIZATIONS is enabled.

Introduce FEATURE_NGEN_RELOCS_OPTIMIZATIONS, under which NGEN specific relocations optimizations are enabled

* Replace push/pop of R11 in stubs with
  - str/ldr of R4 in space reserved in epilog for non-tail calls
  - usage of R4 with hybrid-tail calls (same as for EmitShuffleThunk)

* Replace push/pop of R11 for function epilog with usage of LR as helper register right before its restore from stack

LSRA: fix multi-reg ops under FIELD_LIST

Need to build a use for each reg.
Also, dump the defList if it's not empty at end of block.

Add test for #18362

Also add to the arm & arm64 tests.lst

Stop creating Linux/x64 corefx flow jobs

Merge pull request #18689 from BruceForstall/UpdateArmBuildDocumentation

Update ARM/Linux build directions

Update Tizen CI docker image (#18567)

* Update Tizen rootfs generation scripts for Tizen 5.0 M1

* Update Tizen CI docker image

* Update Tizen CI RID

Update ARM/Linux build directions

Update BuildTools, CoreClr, CoreFx, PgoData to preview1-02928-01, preview1-26627-04, preview1-26627-04, master-20180627-0051, respectively (master) (#18588)

* Update BuildTools, CoreClr, CoreFx, PgoData to preview1-02928-01, preview1-26627-04, preview1-26627-04, master-20180627-0051, respectively

* Rename netcoreapp2.2 => netcoreapp3.0

* Disable test for CoreFX update

Avoid capturing ExecutionContext into CancellationTokenSource's Timer (#18670)

* Avoid capturing ExecutionContext into CancellationTokenSource's Timer

It's not needed, and it can keep unrelated state alive unnecessarily

* Address PR feedback

Fix alternate stack cleanup on MUSL (#18685)

The MUSL implementation of sigaltstack checks that the ss.ss_size is
larger or equal than the MINSIGSTKSZ even when the ss_flags is set
to SS_DISABLE even though Linux man page for sigaltstack states that
when this flag is set, all other ss fields are ignored.

We were not setting the ss_size in this case and it was causing a memory
leak for each thread that has terminated on MUSL based Linux distros
like Alpine.

Glibc implementation doesn't check the ss_size when the SS_DISABLE is set
so the problem was really MUSL specific.

Typo (#18684)

* approrpriate -> appropriate

* allignment -> alignment

* aquire -> acquire

* aquisition -> acquisition

* arbitraty -> arbitrary

* arcance -> arcane

* archetecture -> architecture

* Archicture -> Architecture

* architecures -> architectures

* argmuent -> argument

Typo (dotnet/corert#6029)

* aggresively -> aggressively

* algortithm -> algorithm

* anindex -> an index

* antecendent -> antecedent

* barnching -> branching

* becauase -> because

* behaivor -> behavior

* Boolen -> Boolean

* Browing -> Browsing

* calander -> calendar

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

JIT: fix bug returning small structs on linux x64 (#18563)

The jit was retyping all calls with small struct returns as power of two
sized ints when generating code for linux x64 and arm32/arm64. When the
results of those calls were assigned to fields the jit would use overly wide
stores which could corrupt neighboring fields.

The fix is to keep better track of the smallest power of two enclosing int type
size for the struct. If this exactly matches the struct size then the the
result of the call can be used in an arbitrary context. If the enclosing type is
larger, the call's result must be copied to a temp and then the temp can be
reinterpreted as the struct for subsequent uses.

Defer retyping inline candidates and tail call candidates.
Then handle deferred updates for cases where calls don't get inlined.

Add test cases for 6 byte structs showing the various situations the
jit must handle: callee is not an inline candidate, is an inline candidate
and gets inlined, or inline candidate that does not get inlined.

Add a 3 byte test case to get coverage on arm32. Add new tests to the
arm32/arm64 test lists.

Cross-bitness in ZapRelocs (#18665)

* Cast to UINT32 to avoid warnings on Windows in ZapBaseRelocs::WriteReloc in src/zap/zaprelocs.cpp

* Replace TADDR with DWORD in ZapInfo::recordRelocation IMAGE_REL_BASED_PTR in src/zap/zapinfo.cpp

* Replace sizeof(cell) with TARGET_POINTER_SIZE in src/zap/zapimport.cpp

* Replace TADDR with DWORD in ZapBaseRelocs::WriteReloc IMAGE_REL_BASED_PTR in src/zap/zaprelocs.cpp

* Define target_size_t type

* Replace TADDR with target_size_t in ZapInfo::recordRelocation in src/zap/zapinfo.cpp

* Replace SIZE_T PVOID with target_size_t in src/zap/zapimport.cpp

* Replace TADDR with target_size_t in src/zap/zaprelocs.cpp

* Rename target_size_t to TARGET_POINTER_TYPE

Modify netci definition (#18677)

Add Public implementation RemoveEventHandler in EventRegistrationTokenTable (#18671)

Windows CoreFX CI  (#18365)

* Add test list CL switch

* End-To-End Test Run on Windows

* Cleanup

* MAX_PATH Workaround

* Set Execution directory for CoreFX tests

* Add All CoreFX PR Tests

* Add test dependencies

* Add extra dependencies

* Add parallel test execution

* Disable OuterLoop tests and System.Data.SqlClient.*  tests

* Initialize maximum degree of parallelization to Environment.ProcessCount

* Remove unnecessary cli option

* Update Dependencies

* Add "enabled" property to tests

* Remove exclusions due to TestUtilities mismatch

* Add capability to run all tests for running Helix test lists directly

* Refactor build script to build testhost when skipping managed tests

* Disable failing System.Threading.Tests.EventWaitHandleTests.Ctor_InvalidMode

* Add switch to skip native test build

* Add testing documentation

* Don't run tests marked as "disabled" when running all available tests

* Add switch to build only testhost and remove Core_Root_Stage

* Clean up TopN.CoreFX.Windows.issues.json

* Refactor build-test.cmd

* PR feedback - build pipeline and documentation

* PR Feedback - Test Helper headers and comments

* Fix buildtesthost option for only building CoreFX test dependencies

* Disable intermittently failing test DrawBezier_PointFs

Merge pull request #18645 from tgiphil/readonly

Changed internal value to readonly for primitive types

Moved SafeWaitHandle and cancellationToken to shared (#18662)

* Moving SafeWaitHandle to shared

* Moved CancellationToken to shared

* _ remove from variable names

* Default change to Default(Token) and minor changes

* Fixing coreclr unix build

* moving semaphoreSlim to shared

* splitting safeWaitHandle to .windows and .unix

Remove conditional compilation of DotNETRuntimeEventSource.cs.

Fix sync.sh to call run.exe -p even if positional parameters are specified after --.

Merge pull request #18663 from fiigii/fixapis

Fix Aes and Ssse3.Shuffle intrinsic APIs

Merge pull request #18425 from acmyu/xml

R2RDump - Output in XML format

Merge pull request #18130 from sdmaclea/PR-ARM64-JIT-Use-LSE-Atomics

[Arm64] JIT generate LSE Atomics

Removes duplicate attributes (#18668)

Add VM support for Unsafe.AsRef(in T)

Applying PR feedback

More Dead String (#18664)

Removed any strings in \src\inc\corerror.xml that appeared only in one or more of the following:
\src\dlls\mscorrc\mscorrc.rc
\src\pal\prebuilt\corerror\mscorurt.rc
\src\pal\prebuilt\inc\corerror.h

Fix #18420

Fix bogus assert in ArraySortHelper (dotnet/corert#6018)

Fixes #6016.
Resolves #6015.

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

Simplify built-in types across corefx (dotnet/corefx#30656)

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

Fix Aes and Ssse3.Shuffle intrinsic APIs

Fix merge errors, replace spaces with tab, print unwind flag name

Dump disasm line by line, changes to dumping bytes to xml, fix
compilerIdentifier typo

Refactor into TextDumper and XmlDumper classes

Xml serialize section contents and gcinfo transitions

XMLSerialize gc transitions and section contents

Make unwindinfo and gcinfo xml serializable

Output in xml format with XmlSerializer

Add public to classes and set accessor to properties needed for serialization

Merge pull request #18435 from acmyu/unwind

R2RDump - Use symbolic names for opcodes

Fix value numbering when selecting a constant (#18627)

When applying selectors, constants were special-cased to not require any
type casts. However this is wrong if a narrowing needs to be performed.

Fix #18235

Delete slack link, standardize on gitter (#18644)

Contributes to https://github.com/dotnet/coreclr/issues/18643

Changed internal value to readonly to all the primitive types

Merge pull request #18585 from BruceForstall/FixArmOffsets

Clean up ARM reserved register handling

Remove commentary on non-existent appx checks (#18631)

Remove commentary on non-existent appx checks

Simplify types in non-shared corelib (dotnet/corert#6008)

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

Cross-bitness regular and thread static fields offsets (#18636)

* Define OFFSETOF__DomainLocalModule__m_pDataBlob_ OFFSETOF__DomainLocalModule__NormalDynamicEntry__m_pDataBlob in src/vm/appdomain.hpp

* Define OFFSETOF__ThreadLocalModule__m_pDataBlob OFFSETOF__ThreadLocalModule__DynamicEntry__m_pDataBlob in src/vm/threadstatics.h

* Add assertions and offsets for dynamically allocated statics

* Compensate OffsetOfDataBlob difference

* Check target specific offsets in assertions in src/vm/methodtablebuilder.cpp

Don't close the JIT func info file on shutdown (#18060)

During shutdown there can be a race between closing/nulling `compJitFuncInfoFile` in `compShutdown` and a background thread trying to write to it after checking that it was not null.

Cross-bitness in instance fields placement and CORINFO structs (#18366)

* Replace sizeof(OBJECTREF*) with TARGET_POINTER_SIZE

* Define IN_TARGET_32BIT IN_TARGET_64BIT macros

* Replace IN_WIN32 IN_WIN64 with IN_TARGET_32BIT IN_TARGET_64BIT in src/vm/jitinterface.cpp src/vm/methodtablebuilder.cpp

* Define and use OFFSETOF__CORINFO_* constants in clrjit

* Define for all 64-bit targets

* Use unsigned __int32 to emphasize that this is 32-bit number

* Rename Array__u1Elems to Array__data

* Eliminate OFFSETOF__CORINFO_RefArray__length

* Rename OFFSETOF__CORINFO_RefAny__ to OFFSETOF__CORINFO_TypedReference__

* Fix OFFSETOF__CORINFO_TypedReference__dataPtr macro value

Fix nested-anon-types warnings (#18591)

Remove test logs (#18621)

Removed dead strings used in debug-only code (#18622)

Removed strings used in debug-only code in ex.cpp

Fix #18420

Update to latest released dotnet SDK (#18626)

non shared changes and disabling tests

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

Simply type names in Corelib (#18623)

Use correct basic block to check legality of PInvoke callsite for inlining (#18620)

It is the same logic as used in other similar places

Clean up ARM reserved register computations

Tighten up and correct the math in compRsvdRegCheck.

Merge pull request #18600 from BruceForstall/FixArmFPFrameAddressLimits

Expand ARM local variable R11-based addressing