Fixed method name in comment (#18571)

Merge pull request #18573 from CarolEidt/ExplicitLayoutTest

Add another test for #17971

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.

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>

Merge pull request #18357 from acmyu/sections

R2RDump - Dump CompilerIdentifier and ImportSection contents

Parse entries of import section

Avoid two unnecessary string.Substring calls in PunycodeDecode (#18546)

fixes for  sos on FreeBSD (#18479)

* fixes for  sos on FreeBSD

* refactore unwind detection to common block for BSD and Linux

* missing newline

* do not do libunwind detection for OSX

Fix struct promotion check for SIMD field (#18548)

A struct can have `lvIsRegArg` true but have multiple SIMD fields if it is passed by reference.

Typo (#18550)

* accesible -> accessible

* acheive -> achieve

* addtional -> additional

* affars -> affairs

* approch -> approach

* archtecture -> architecture

* aritmetic -> arithmetic

* arquired -> acquired

* assigments -> assignments

* assmblies -> assemblies

Dump bytes with offset instead of rva

Parse and dump compiler identifier and import sections, move dump section contents to own function

Dump contents of NativeArray and NativeHashtable

Use GetEntryPointIdFromOffset to get EntryPointId of generic methods

Output formatting changes for unwindInfo and section contents

Merge pull request #18500 from acmyu/liveness

R2RDump - Output GcSlot liveness interweaved with disasm instructions

Save name of dll in constant, use tab instead of spaces

Update CoreClr to preview1-26619-04 (#18540)

PInvoke calli support for CoreRT (#18534)

* Ifdef out NGen-specific PInvoke calli inlining limitation for CoreCLR

This limitation seems to be a left-over from effort to eliminate JITing with fragile NGen.

* Delete dead partial-trust related code

* Allow PInvoke stub inlining

* Add convertCalliToCall JIT/EE interface method

* Update superpmi

Update BuildTools, CoreClr, CoreFx, PgoData to preview1-02918-02, preview1-26619-01, preview1-26617-01, master-20180619-0054, respectively (#18518)

Removed dead strings from corerror.xml

Remove dead strings mscorrc.rc

genPutArgStk needs to check for FIELD_LIST first (#18499)

When a struct is passed on the stack using `FIELD_LIST`, the type of the `FIELD_LIST` is the type of its first field. If that type was a struct type (i.e. a SIMD type), `genPutArgStk` would assume that it was the non-`FIELD_LIST` case.

Fix #18497

Delete PinnableBufferCache (dotnet/corert#5950)

Port https://github.com/dotnet/coreclr/pull/18360 to CoreRT

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

Correctly return early if the test fails (#18529)

Add public implementation WindowsRuntime ExceptionSupport (#18494)

Improve Intel hardware intrinsic APIs (#17637)

* Improve Intel hardware intrinsic APIs

* Simplify Avx.Extract non-const fallback

temporarily disable pthread based named mutexes on FreeBSD (#18480)

Document span lifetime issue in CreateSpan (dotnet/corefx#30490) (#18528)

The language rules around span safety that C# and F# adhere to assume
there is no way to create a `Span<T>` wrapper over a `ref` local /
parameter. This means `ref` inputs into a method are not considered when
calculating the allowed lifetime of a returned `Span<T>`. Hence both
CreateSpan and CreateReadOnlySpan will be assumed to have heap lifetime
even when provided stack based inputs. Example:

``` c#
Span<int> Example() {
  int i = 42;
  Span<int> span = MemoryMarshal.CreateSpan(ref i, length: 1);
  return span; // C# and F# will allow this
}
```

In this case the actual lifetime of `span` is that of `i`. Yet the
compiler doesn't consider the `ref i` input and hence believes this must
be heap based and hence safe to return out of the method.

This is okay as these methods are unsafe. But want to explicitly
document that fact.

More information on the safety rules can be found in the [span safety
proposal](https://github.com/dotnet/csharplang/blob/master/proposals/csharp-7.2/span-safety.md)

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

Typo (#18520)

* Acutal -> Actual

* addtional -> additional

* agressive -> aggressive

* agument -> argument

* Alignement -> Alignment

* alredy -> already

* MAnaged -> Managed

* analagous -> analogous

* aquire -> acquire

* arbitary -> arbitrary

Re-Enable StructABI test for xplat (#18496)

mach_absolute_time as the primary clock source on macOS (corefx#30391) (#18505)

macOS 10.12+ supports clock_gettime (HAVE_CLOCK_MONOTONIC is defined)
However, mach_absolute_time has better resolution and should be used
as the primary clock source.

Update CoreClr, CoreFx to preview1-26616-03, preview1-26616-02, respectively (#18503)

Update BuildTools, CoreClr to preview1-02915-01, preview1-26616-01, respectively (#18495)

Add missing registers enum

Interweave gcslot liveness with disasm

Use DumpInstruction instead of code block

Merge pull request #18474 from acmyu/gc

R2RDump - Get GC slot liveness code offset

Merge pull request #18379 from acmyu/disasm

R2RDump - Print disassembly to text buffer

Delete bitrotten appdomain agility checks (#18489)

Fixes #18484

Use correct field offset in genPutArgStkFieldList (#18483)

* Use correct field offset in genPutArgStkFieldList

Fix #18482

* formatting

* Add the new test to the arm and arm64 test lists

Avoid some virtual dispatch in Encodings (#18487)

- Seal some internal overrides
- In the more common encodings, add overrides of GetBytes/GetByteCount/GetChars/GetCharCount to avoid another layer of virtuals from the base class (they can call to an internal non-virtual).

Use IntPtr instead of long for pointers

Fix bugs from trying to get transitions for untracked slots when it's not supposed to

Adjust method comment

Ensure AdjustmentRule.DaylightDelta is within [-12,12] (#18477)

* Modulo AdjustmentRule.DaylightDelta

* fix typo

Update BuildTools, CoreClr, CoreFx, PgoData to preview1-02914-01, preview1-26615-04, preview1-26615-01, master-20180615-0040, respectively (#18472)

Fix handling of generating relative path to parent (#18460)

Fixes #30263

clean up list of disabled warnings. (#18318)

* delete warnings that do not longer exist

For example C4171 was deleted after VS 6.0 (https://docs.microsoft.com/en-us/previous-versions/visualstudio/visual-studio-6.0/aa233011(v=vs.60))

* delete C4206 fromm the list

because its default value is 4, so this line is useless.

* reenable warning as error.

* enable warning C4430

and fix places that trigger it.

* fix C4334

* format the list

* fix ssize_t

Make gcInfoTypes global instead of passed as param

fix the desktop build break (#18475)

* fix desktop build break

after #18346

* add dstCount

Fixed some bugs with gc transitions

Get gc slot liveness code offset

[Windows|Arm64|Vararg] Add FEATURE_ARG_SPLIT (#18346)

* [ARM64|Windows|Vararg] Add FEATURE_ARG_SPLIT

Enable splitting >8 byte <= 16 byte structs for arm64 varargs
between x7 and virtual stack slot 0.

* Force notHfa for vararg methods

* Correctly pass isVararg

* Correct var name

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