ARM: increase small reg set for jitStressRegs=3 (#18462)

We may need two registers (base + offset) to set up the target register for a virtual call.

Fix #18228

add overloads (#18458)

Write and WriteAsync overloads for StringBuilder. (#18281)

* Write and WriteAsync overloads for StringBuilder.

Addresses issue https://github.com/dotnet/corefx/issues/30048

Note that tests will come as a separate checking as part of the coreFX repo.

* Review feedback

* Review feedback

* Added WriteLine(StringBuilder overloads (review feedback)

* review feedback

* Fix configureAwait

* bug fix

* Bug fix

* review feedback

Update CoreClr, CoreFx, PgoData to preview1-26614-04, preview1-26614-04, master-20180614-0041, respectively (#18461)

Adds original error info to NotSupportedException during FileStream initialization (#18455)

Skip version (dotnet/corefx#28835)

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

Added ISOWeek to System.Globalization (#18456)

* Added ISOWeek to System.Globalization

* Revert DateTime changes. Use constants from GregorianCalendar instead.

* Add more comments

* Also allow 7 as value for day of week

* Add note about ISO week-numbering year parameters

* Add note about allowing 7 for day of week

[Windows|Arm64|VarArgs] Correctly pass HFA arguments (#18364)

* Fix passing HFA of two floats to vararg methods

Previously, the type would be reported as HFA and enregistered; however,
this is not correct, as arm64 varargs abi requires passing using
int registers.

* Address linux build issue

* Apply final format patch

* Add _TARGET_WINDOWS_

Merge pull request #18438 from BruceForstall/FixKeepAlive

Fix GC KeepAlive test

Fix enregistered lclFld bug (#18418)

* Fix enregistered lclFld bug

In `impFixupStructReturnType()`, don't transform to `GT_LCL_FLD` if we have a scalar lclVar.
Also, to avoid future bad codegen, add verification and recovery code to Lowering.

Fix #18408

* Extract the full conditions for whether a lclVar is a reg candidate, so it can be called from the assert in Lowering.

* Review feedback

Merge pull request #18442 from BruceForstall/DisableB65423

Disable test b65423, tracked by #18441

moving ARM HW Intrinsics files to shared partition (#18451)

Fix allocation methods at few places in PAL (#18457)

In the utf8.cpp and process.cpp, PAL was incorectly using the
global new operator. This results in calling this operator's
definition in coreclr runtime, which is wrong. In the case
of the utf8 stuff, a customer has reported a crash happening
due to that when the path from which the PAL was initialized
contained chinese characters.

THe fix is to use InternalNew / InternalDelete functions instead.

Added CallerArgumentExpressionAttribute (corefx issue 21809) (#18404)

* Added CallerArgumentExpressionAttribute

* Modified CallerAtgumentExpressionAttribute

Removed ArgumentNull check from constructor and reordered members to
be more consistent with other attribute classes.

Update BuildTools, CoreClr, CoreFx, PgoData to preview1-02912-01, preview1-26613-04, preview1-26613-07, master-20180613-0058, respectively (#18189)

Update unshared Reflection.Emit files

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

Merge nmirror to master

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

 Move x86 HW intrinsics files to shared (#18427)

* Switch platfom amd64 to x64

* Move x86 intrinsics file to shared

* Adding condition so corert always picks up X.PlatformNotSupported.cs

Related to: #15922

Merge pull request #18433 from BruceForstall/DisableLongRunningGCStressTests

Disable HugeArray test in GCStress modes

Disable test b65423, tracked by #18441

Fix GC KeepAlive test

We need to determine that an object finalizer has not been run
*before* the KeepAlive call. After the call, it is perfectly
valid for the finalizer to be called, which can happen under
GCStress.

Add public implementation WindowsRuntimeResourceManagerBase (#18417)

Disable HugeArray test in GCStress modes

This test takes too long under GCStress, and causes timeout failures.
It was already disabled for x86. Disable it for all platforms, to
avoid unexpected failures in the CI. E.g., especially for ARM, which
is slow.

JIT: finally cloning -- handle fall through out of try (#18416)

When searching for a handler to clone, handle the case where
control can fall out of the try into a callfinally pair.

Closes #18353.

Unify struct arg handling (#18358)

* Unify struct arg handling

Eliminate unnecessary struct copies, especially on Linux, and reduce code duplication.
Across all targets, use GT_FIELD_LIST to pass promoted structs on stack, and avoid
requiring a copy and/or marking `lvDoNotEnregister` for those cases.

Unify the specification of multi-reg args:
- numRegs now indicates the actual number of reg args (not the size in pointer-size units)
- regNums contains all the arg register numbers

Fix unwinding of funclet with no epilog on x86/Linux (#17458)

Adding containment support to more x86 HWIntrinsics (#18349)

* Adding containment support for the Sse42.Crc32 intrinsic

* Adding containment support for the x86 Compare*OrderedScalar and Compare*UnorderedScalar HWIntrinsics

* Adding containment support to several intrinsics that were marked NoContainment

Fix Decimal.NearNegativeZero/NearPositiveZero (#18424)

Reword invalid program exception message (#18409)

* Reword invalid program exception message

* Reword

* Update msg

skip empty regMask in verifyRegistersUsed (#18415)

Standardize on PLATFORM_WINDOWS instead of PLATFORM_UNIX (#18411)

There are a bunch of uses of PLATFORM_WINDOWS in System.Private.CoreLib, and just a few uses of PLATFORM_UNIX.  Standardize on the former, removing all occurrences of the latter.

Fix UTF8 encoding fallback index (#18387)

* Fix UTF8 encoding fallback index

When falling back in UTF8 decoding, we’ll call the DecoderFallbackBuffer.InternalFallback method which takes a pointer to the source bytes pointer which point to where we have progressed inside the source bytes. The problem is before calling DecoderFallbackBuffer.InternalFallback we call GetBytesUnknown which can adjust the source byte pointer and DecoderFallbackBuffer.InternalFallback need to be called with the pointer before the adjustment. The fix is simply storing the original pointer value of the source byte pointer before calling GetBytesUnknown and then pass the stored original pointer to DecoderFallbackBuffer.InternalFallback instead of the adjusted source byte pointer

* Fix the comment

WinRT attribute Cleanup (#18412)

Remove WinRTSynchronizationContextFactoryBase (#18385)

(Re)Enable NativeVarargs for CoreCLR (#18373)

* Enable NativeVarargs for CoreCLR

This will allow coreclr run programs which have native varargs.
It also re-enables the ArgIterator type for managed to managed native
vararg calls. This will allow the use of the __arglist keyword.

Limitations:

NYI statements have been added for all Unix Targets.

With this change __arglist (native varargs) will be supported, but not
yet tested on:

Amd64 Windows
x86 Windows
Arm32 Windows
Arm64 Windows

This change does not re-enable native vararg tests. This will be done
in a separate change.

Adds the CoreLib IL to packages (#18372)

* Adds the CoreLib IL to packages

The purpose of this is to enable code coverage runs to include CoreLib in CI - for that it is necessary to make the IL version available.

Fix PCODE not having thumb bit set when comparing two addresses (#18247)

Removed length restrictions on named synchronization primitives lengths on Windows (#18402)

* Removed Path Length check

* waitHandleNameMax removed

Update Buildtools (#18380)

Avoid NativeOverlapped pinning by allocating unmanaged memory for it (#18360)

It makes PinnableBufferCache unnecessary

Use nameof of Encoding classes (#18395)

Merge pull request #18394 from BruceForstall/Fix10115

Re-enable tests for Windows arm64

Merge pull request #18363 from sdmaclea/PR-waitany-debug

Waitany test debug and cleanup

Some extra changes needed in shared

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

Remove Substring in TimeZoneInfo parsing (#18378)

* Remove Substring in TimeZoneInfo parsing

Convert Substring to Slice to eliminate string allocations while parsing Posix daylight savings rules.

Use cached strings for values 0 to 9 (#18383)

When doing ToString on single-digit integers, just return a cached string.  This helps avoid string allocation/formatting costs for this very common case, while only costing one comparison for other cases.

Remove ConcurrentDictionary from CoreLib (#18308)

Re-enable tests for Windows arm64

These tests don't fail anymore. Re-enable them.

Fixes #10115

Merge pull request #18382 from BruceForstall/MakeArmGcStress3Clean

Disable 3 tests incompatible with GCStress=3

Fix EventPipe EventHandle Caching for TraceLogging (#18355)

Fix coredistools download script now that Packages layout has changed

Fixes #18321

Moved Mutex.cs From src to shared (#18320)

* Moved Mutex.cs to shared

* Reduced #ifdef in mutex.cs

* Moved windows specific Implementation and renamed mincore to kernal32

* safeWaitHandle changed to _waitHandle

* moved using statement out of buffer namespace

Expose public implementation AsyncCausalitySupport to S.R.WindowsRuntime (#18376)

Merge pull request #18287 from sdmaclea/TryClearGCStressInstructionUpdate

Use atomic ops in CommitGCStressInstructionUpdate()

Disable 3 tests incompatible with GCStress=3

The following fail in CI with arm Windows GCStress=3, and are now marked
`GCStressIncompatible`.

1. GC\Coverage\smalloom\smalloom.cmd
2. Issue #17480: tracing\tracevalidation\rundown\rundown\rundown.cmd, tracing\tracevalidation\jittingstarted\JittingStarted\JittingStarted.cmd

Fix threading gcStress fails

Mark GCStressIncompatible ond/or fix synchronization logic

Merge pull request #18326 from acmyu/gc

R2RDump - GcInfo and slot table

Workaround for  compiler.hpp (1848) - Assertion failed 'lvRefCnt' (#18292)

* add test

* clean fgRemoveStmt a bit

* fix the issue (more like a workaround).

Disable failing CoreFx tests on Windows/arm and Linux/arm (#18347)

* System.Dynamic.Runtime.Tests on Windows/arm and Linux/arm

* System.Linq.Expressions.Tests on Windows/arm (JitStress=2)

* System.Runtime.Numerics.Tests on Windows/arm (JitStress=1/JitStress=2)

JIT: Fix bug in finally cloning caused by unsound callfinally reordering (#18348)

We need to make sure that if we reorder a callfinally during finally cloning
that the callfinally is actually the one being targeted by the last block in
the try range.

Closes #18332. Linked issue has some more detailed notes.

Fixed and optimized Decimal.GetHashCode (#18288)

Add missing #ifdef.

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

Interop Copy tests now all return 101 during fail rather than crash (#18310)

* Interop Copy tests now all return 101 during fail rather than crash

* Remove unused Assert APIs

Merge pull request #18354 from sdmaclea/PR-eventsourcetrace

Mark eventsourcetrace GCStressIncompatible

Merge pull request #18340 from mikedn/ins-desc

Increase instrDesc::_idIns width on XARCH

Merge pull request #18356 from CarolEidt/Fix_GT_START_NONGC

GT_START_NONGC has no kill set

GcInfo belongs to method, not runtime function

Waitany test debug and cleanup

Skip old time zones adjustments with offsets higher than 14h (#18305)

* Skip old time zones with offsets higher than 14h

* apply offline feedback

* fix braces

GT_START_NONGC has no kill set

It was mistakenly added to the `GT_PROF_HOOK` case in the recent LSRA changes.

Mark eventsourcetrace GCStressIncompatible

Allow ILCodeVersion to fallback to default IL (#18322)

For compat with profilers that used our APIs in unexpected ways we can allow
the ILCodeVersion to fallback to the default IL code when no IL was explicitly
given.

Moves CurrentSystemTimeZone, ICastable, RegistryView and RegistryValueKind to shared (#18337)

* Moves CurrentSystemTimeZone to shared completely

* Moves ICastable to shared

* Moves RegistryView and RegistryValueKind to shared

* Moves ICastableHelpers to its own file

Force short date pattern to use yyyy on Linux (#18316)

* Force short date pattern to use yyyy on Linux

The default pattern we get is using yy which causes the years  to be displayed  as 2 digits. This is not acceptable for many users. The fix here is to force 4-digit year as a default and still keep the original pattern which has 2-digits year in the optional list

* Address the review feedback

Remove dead case from string.Compare (#18345)

Guard Windows code under PLATFORM_WINDOWS (#18344)

Use atomic ops in CommitGCStressInstructionUpdate()

Increase instrDesc::_idIns width on XARCH

XARCH exceeded the 512 instruction limit but it still has room for instruction formats (only 77 at the moment) so a bit was taken from _idInsFmt.

Also, cleaned up some tiny while instrDesc traces while trying to fix "Space taken up to here:" kind of comments.

Fixed a couple of comments about ArgumentException on timeout parameters (#18335)

Remove a dead branch from __DTString (#18334)

All call sites to the ctor validate that the DateTimeFormatInfo has been created.

Delete FormatterConverter from CoreLib (#18336)

It's not used in the library.  The type is implemented/exposed from corefx.

Cross-bitness support in JitInterface (#18329)

* Replace sizeof(tailCallType) / sizeof(tailCallType[0]) with _countof(tailCallType)

* Remove TransparentProxyObject and RealProxyObject

* InlinedCallFrame::GetEEInfo should not be used for R2R compilation

* Hardcode constants used in CEEInfo::getEEInfo

* Thread fields should not be used for R2R compilation

* Zero pEEInfoOut->inlinedCallFrameInfo when compiling R2R code

* Rename OFFSETOF__PtrArray__m_Array to avoid compilation conflicts with asmconstants

* Add assertions enabled in every non-crossbitness scenario

* Define and use CROSSBITNESS_COMPILE macro

Fix ARM cast codegen (#18063)

* Fix ARM cast codegen

ARM cast codegen is rather convoluted and sometimes does the wrong thing by applying GTF_UNSIGNED to the destination type even though this flag is only about the source type.

* Add more conversion tests

These tests are intended to support casts with contained operands. They're also useful to catch issues caused by load nodes having small types and generally improve test coverage for casts.

Adding containment support for more x86 hardware intrinsics (#18297)

* Adding containment support to one-operand scalar HWIntrinsics (x86)

* Adding containment support to two-operand imm HWIntrinsics (x86)

* Adding containment support to three-operand imm HWIntrinsics (x86)

* Updating hwintrinsiccodegenxarch to properly mask Sse41.Insert for TYP_FLOAT

* Updating the Sse41.Insert tests for TYP_FLOAT

* Adding containment support for Sse2.CompareLessThan and BlendVariable (Sse41/Avx/Avx2)

* Fixing `genHWIntrinsic_R_RM_I` to call `emitIns_SIMD_R_R_I`, rather than `emitIns_R_R_I`

* Updating emitOutputSV to not modify the code for IF_RWR_RRD_SRD_CNS

* Cleaning up some of the emitxarch code.

* Moving roundps and roundpd into the IsDstSrcImm check

Refactoring gcInfo, added verbose option to cli

Fix error cause by merge

Fix unwindInfo bug caused by missing alignmentPad, dump raw byes of unwindInfo and gcInfo

Use single variable for multiple tabs

Read slot table, dump GcInfo

Extract other fields from gcinfo

Extract gcinfo header, return kind and code length. Use code length as size of the runtime function

Extract gcinfo from image

Avoid passing R2RReader to R2RMethod and RuntimeFunction constructors

Use typeDef.Namespace to get full namespace, move to separate function

R2RDump - Unwind Info (#18248)

* Extract unwind info from image

* Use typeDef.Namespace to get full namespace, move to separate function

* Avoid passing R2RReader to R2RMethod and RuntimeFunction constructors

* Dump unwind info

* Changes to unwindInfo output format

Fix issue in Compiler::impImportStaticFieldAccess (#18328)

When CORINFO_FIELD_STATIC_SHARED_STATIC_HELPER accessor is being processed for
structs, the method creates a GT_IND node instead of GT_OBJ node. For structs
that can be normalized to a specific type (like TYP_SIMD8), it causes an assert
in Compiler::impNormStructVal in the switch case for GT_IND.

This change modifies the node creation to work the same way as in the default
case for accessors, creating GT_OBJ for structs and GT_IND otherwise.

JIT: implement assembly name dissassembly filtering (#18306)

Add `COMPlus_JitDisasmAssemblies` to specify a whitelist of assemblies
whose methods will be disassembled (also filters out order, EH, GC, and
unwind info dumps too).

Intended use is for PMI asm diffing. Since PMI runs in process, current
disassembly will dissassemble various methods in PMI itself as well
as framework methods that did not get prejitted.

With this new config option we can restrict disassembly to just the assemblies
that we're processing via PMI.

Moves FastResourceComparer and TaskAwaiter to shared (#18283)

* Reduces diff in BinaryReader

* Moves FastResourceComparer to shared

* Reduces diff in FileBasedResourceGroveler

* Minor comment diff in ManifestBasedResourceGroveler

* Reducing diff by removing extra blank lines in ResourceReader

* Reduces diff by renaming private fields in AssemblyName

* String to string and Object to object in AssemblyName

* Reduces diff in TaskAwaiter

* Moves TaskAwaiter to shared

* Apply code review feedback

* Fixes failure in corert

* Revert Renaming fields in AssemblyName

Increase timeout and log the result for some WaitAny tests that expect AbandonedMutexException (#18312)

Follow-up to https://github.com/dotnet/coreclr/pull/13298, missed a few cases.

Add instructions for making idl file changes (#18188)