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

Merge pull request #18615 from A-And/CIFXStaging

CoreFX CI - Correct testResults path

Change string.Compare(...) == 0 occurrences to string.Equals(...) (#18616)

Moved EventWaitHandle.cs to shared (#18612)

* Removed AccessControl Unused arguments

* moving common code to CreateEventCore and using constructor chaining

* Moving Functions to eventwaithandles.windows

* EventWaitHandle matched with corert

* eventwaithandle.windwos matched with corert

* Addding interop functions

* moved files to shared

* Minor changes

* fixing build for corert unix

* Removing Comment

Pass RID to crossgen (#18592)

Fix incorrect usage of ILCodeVersion::AsNode (issue #18602) (#18606)

When the debugger is querying the active rejit IL for an IL method that has not been rejitted it incorrectly creates a VMPTR_ILCodeVersionNode for a code version that shouldn't have one.

Correct testResults path

Expand ARM local variable R11-based addressing

When addressing a local with negative offsets from R11 (if it can't
be reached from SP), allow the full range of negative offsets allowed
in the instructions. Floating-point load/store especially has a much
bigger range than what was previously allowed.

Fix incorrect script path (#18613)

Move Amd64 specific class to it's own folder and subnamespace

Non Shared Changed And feedback

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

Remove trigger phrases and add testhost building (#18608)

Add public implementation MarshalerSupport (#18530)

Start moving testing documentation into a single location (#18598)

Start moving testing documentation into a single location
Update `test-configuration.md` with more details on creating a test and requirements

Update package version to 3.0.0

This doesn't make much practical difference since CoreCLR's packages aren't
intended for any consumers other than core-setup, but we might as well try
to be consistent with our peer repos in case it provides a useful hint when
something goes wrong.

Merge pull request #18581 from CarolEidt/Reenable18295

Re-enable tests after fixing 18295

Merge pull request #18595 from BruceForstall/FixArmEncoderLimits

Fix ARM encoder limits

Moved Semaphore.cs and Semaphore.Windows.cs to shared (#18597)

* TryOpenexistingworker and createsemaphore moved to semaphore.windows.cs

* Common code moved to verifyCounts and CreateMutexCore and then moved to semaphore.windows.cs

* CreateSemaphore merged with CreateSemaphore core

* Moving interop functions from win32Native to inteop.kernel32

* Moving the files to shared

* Minor Formating

Update behaviors of new TextWriter StringBuilder overloads (#18578)

Two issues addressed:
- CancellationToken was being ignored; the overloads should check whether cancellation was requested.  This applies to the existing Write{Line}Async overloads that were added in 2.1.
- The other overloads support inputs (string, char[], etc.) being null, and just treat that the same as an empty string.  We should do the same for StringBuilder rather than throwing.

Merge pull request #18570 from acmyu/bugs

R2RDump - Bug fixes

Windows CoreFX CI Staging Commit (#18388)

* Netci

* Add fake test switch

* Add branch PR trigger

* Add Checked corefx_innerloop jobs

Fix unset ZapRelocationType for fixup (#18589)

Fix ARM encoder limits

The ARM encoder was using slightly incorrect limits when deciding whether
a particular constant offset was allowed in an addressing mode.

Change the limits to match the instruction definitions.

Merge pull request #18031 from kbaladurin/fix-hfa-tests

[JIT/ARM] Fix Compiler::lvaFrameAddress

Update PgoData to master-20180621-0050 (#18587)

Undo accidental 2.0 -> 2.1 renames (dotnet/corert#5981)

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

Rev netcoreapp2.0 to netcoreapp2.1 (dotnet/corert#5974)

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

Update CoreClr, CoreFx to preview1-26621-01, preview1-26621-01, respectively (#18584)

Merge pull request #18552 from dotnet-maestro-bot/master-UpdateDependencies

Update BuildTools, CoreClr, CoreFx, PgoData to preview1-02920-01, preview1-26620-04, preview1-26620-03, master-20180620-0039, respectively (master)

Fixed method name in comment (#18571)

Merge pull request #18573 from CarolEidt/ExplicitLayoutTest

Add another test for #17971

Re-enable tests after fixing 18295

Update BuildTools, CoreClr, CoreFx, PgoData to preview1-02920-01, preview1-26620-04, preview1-26620-03, master-20180620-0039, respectively

An UnusedValue still requires a target reg (#18561)

* An UnusedValue still requires a target reg

The BuildSimple method wasn't creating a def for an unused value. Although (in this case) the code is dead, the code generator must still be able to generate code for it.

* Add test case for #18295 to arm/arm64 tests.lst

added concurrent access detection to Remove() (#18524)

Add another test for #17971

Remove MaskLoad tests from arm and arm64 lst files (#18569)

This will disable the test from being run with smart in our windows arm
testing. This corresponds to the tests being deleted in #17637.

Make sections optional, throw more descriptive error when rva can't be converted to offset

Avoid extra allocation in IDNA (#18551)

* Avoid extra allocation in IDNA

* cache the indexd value in StringBuilder

Fix instruction groups offset on the border between cold/hot code. (#17775)

* add an additional debug printing.

* add additional checks

* Clean-up checks a bit.

* fill unused allocated chunks

* clean up checks

* fix EMITTER_STATS

* optimize padding filling

call to emitCurCodeOffs(cp) is not free.

Fix Complus_JitMinOpts for the first method (#18555)

Remove StringBuilderCache usage from System.Text.RegularExpressions (#30474)

* Remove StringBuilderCache usage

* fixed buffer size, manual reverse copying & local textinfo

* Fast path for not to encode string

* avoid one span slice

* code formatting & bounds check opt

* Avoid string allocation in char class

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

Make unwind info specific to amd64

Use Registers for FrameRegister

Save frameOffset in 16bits

Dump unwind codes based on opcode

Rename enums with Pascal casing

Use symbolic names for opcodes, get register name for opinfo

Fix bugs

Merge pull request #18357 from acmyu/sections

R2RDump - Dump CompilerIdentifier and ImportSection contents