Merge pull request #19927 from adityamandaleeka/fix_arm64_emitIns

Fix emitIns_S_S_R_R not to encode SP to ZR before an add may be emitted

Adding the ARM floating point registers to the cordebug.idl file (#19934)

Address msbuild issues with cross compiled tests in helix (#19901)

* Address msbuild issues with cross compiled tests in helix

* Fix test warning about duplicate types

Fix emitIns_S_S_R_R not to encode SP to ZR before an add may be emitted.

Implement Thread.GetApartmentState/TrySetApartmentState (#6323)

Fixes #5776

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

Revert changes that are causing buildbreaks of OOB packages

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

Update AsyncMethodBuilder.cs (#19909)

Changed commentary for range of AsyncCache Tasks with Int32 type.

Fix GC for collectible classes (#19892)

There is a subtle bug in the GC that causes access violation when a
LoaderAllocator of a collectible class is pushed to the mark stack and
that class also has partial sub-objects. One example is an array of byte
arrays. In that case, the mark stack is supposed to have the reference
to the instance of the class stored right before the related partial
sub-objects references. But due to the bug, the slot where the instance
of the class was expected to be located was not set to anything.

CallSites for MemoryExtensions.Contains (CoreCLR) (#19874)

* Update additional callsites for MemoryExtensions.Contains

* One more callsite

* More callsites

* CR fixes

Fix optimization of CAST(int)(LSH(long)<<CNS_INT 32+ with side effects). (#19899)

* add a repro test

* Fix

Move more registry files to shared partition

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

Merge pull request #19885 from brianrob/runtimeeventsource_tests

Fix Intermittent RuntimeEventSource Test

JIT: use bit vector for identifying jump targets (#19884)

When doing the initial IL scan to locate block boundaries the jit really
only needs to track one bit per offset.

Merge pull request #19867 from echesakovMSFT/BuildCrossBitnessCrossGenAsWell

  Build both Hostx86 and Hostx64 arm32 CrossGens on Linux

Move registry interop to shared CoreLib partition (#19886)

Basic implementation for testing of COM activation of a .NET class (#19760)

* Rough outline of managed implementation for COM activation in SPCL

* Add property for finding interop common
Add property to exclude default assertion file
Display exe ExeLaunchProgram class is going to launch

* Add a native client for the NETServer
Consume the ExeLauncherProgram.cs file as a wrapper for the native test

* Update COM Server contracts to use 'int' instead of 'long'

* Complete symmetric testing coverage for .NET server and native client.

* Block EXE launch from running on non-Windows machines

* Disable COM testing in helix since it has issues on Windows Nano and there
is no way to determine that is the platform.

* Update tests based on CLSID mapping manifest approach.

Merge pull request #19276 from CarolEidt/FixUxArgSize

Fix Unix preallocatedArgCount

Fix intermitted RuntimeEventSource test.

Build both Hostx86 and Hostx64 arm32 crossgens. Continue to use Hostx86 crossgen to crossgen System.Private.CoreLib.dll

Build Hostx86/arm32 coreclrpal when CROSSCOMPILE is "1" and Hostx64/arm32 coreclrpal otherwise in configurecompiler.cmake

Build Hostx86/arm32 crossgen when CROSSCOMPILE is "1" and Hostx64/arm32 crossgen otherwise in configurecompiler.cmake

Allow Hostx64/arm32 combination in configurecompiler.cmake

ToPointerFix - VS debugger needs the ToPointer() methods to inspect a pointer, marking it as DependencyReductionRoot to make sure it is not optimized away.

[tfs-changeset: 1713493]

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

Add test cases for Pclmulqdq intrinsic

Implement PCLMULQDQ intrinsic

Add GetCacheSizePerLogicalCpu and GetCurrentProcessCpuCount to gcenv for windows (#19879)

* Add GetCacheSizePerLogicalCpu and GetCurrentProcessCpuCount to gcenv for windows

* Remove dead code, switch to __cpuid instaed of asm helper

* code review feedback

Fix Unix preallocatedArgCount

This has been an existing ToDo for some time.
This results in 19 bytes reduction in code size for pmi diffs of tests and frameworks.

Merge pull request #19881 from vitek-karas/EventSouceCommentTypos

Fix multiple typos in the EventSource.cs design comment.

Merge pull request #19844 from CarolEidt/Enable19397Arm64

Re-enable GitHub_19397 test

Fix multiple typos in the EventSource.cs design comment.

Merge pull request #19870 from nattress/debuginfo

Add DebugInfo dumping support to r2rdump

Update BuildTools, CoreClr, CoreFx, CoreSetup, PgoData to preview1-03207-02, preview1-26907-04, preview1-26907-04, preview1-26907-05, master-20180907-0057, respectively

Add DebugInfo dumping support to r2rdump

Display formatted debug info for each runtime function

Fix AVX2 Gather Intrinsic failture with JITStress

delete superpmi-shared/compileresult.cpp  assert on x86. (#19841)

* fix emitOutputAM for x86.

* Revert "fix emitOutputAM for x86."

This reverts commit fe5ade83e7c8e1ac1cb4442defdfee1d3a3be3f0.

* delete an assert

* add an assert about slotNum

MemoryExtensions.Contains (#19863)

Fixes towards making the runtime compiled without FEATURE_PREJIT defined (#19864)

Merge pull request #19866 from dotnetrt/fix-generatetests-script

Remove all Sse2 Pack* and Unpack* tests from GneerateTests.csx script

Import signature parsing and general improvements in R2RDump (#19813)

* Import signature parsing and general improvements in R2RDump

GcInfo - fixed a bug where the machine architecture was stored in
the instance field later than it was used in the calculation of
SlotTable which subsequently ended up with the wrong register set
(ARM64 instead of AMD64).

GcTransition - changed register set selector to a switch as the
previous implementation was incorrect for AMD64.

UnwindInfo - bug fix: the 4-alignment padding should come before
the personality routine, not after it. I have also slightly
simplified the calculation of the padding size.

CoreDisTools - added support for annotating indirect calls with
symbol information for the import cells; fixed formatting of
short relative jumps and added some code comments.

DisassemblingTypeProvider - bumped up some visibilities and made
a few generalizations letting me use this helper in the general
signature parser.

R2RImportSection - replaced binary signature samples with the
actual parsed representation.

R2RReader - I removed special-casing for eager import signatures
that seems wrong - I don't see any equivalent code in the
CoreCLR runtime. Please let me know if anyone is aware of a reason
why this was put here in the first place.

TextDumper and elsewhere - I have removed all tabs from the
R2RDump tool output.

R2RConstants - I have merged the various CoreCLR and R2R constants
in this file - previously they were interspersed in multiple
classes - and I added a bunch of new stuff needed by the signature
parser.

R2RSignature - comprises the newly added R2R signature parser.

Thanks

Tomas

* Slight modification in signature formatting to make it easier to read

I have swapped the ordering of the signature content with the
signature name - based on analyzing actual disassembly produced
by the code I came to the conclusion that the parsed signature
is generally more important than the cell fixup type which is
often obvious just based on the code context.

Thanks

Tomas

* Add dump of unwind info and RVA translation for near jumps / calls

* Add code comment on GetInstruction return value per PR feedback

Merge pull request #19857 from CarolEidt/Fix19674

Morph ByRef args of SIMD field assign

Add Basic Out-of-Process EventPipe Control and Heap Snapshot Capture (#19720)

Fix the slashes when setting the ZapBBInstrDir (#19858)

In OpenMethodProfileDataLogFile, we try to set the directory and path for the .ibc files using windows slashes (\). This causes this code to fail on Linux, which uses forward slashes. This is particularly a problem when setting COMPlus_ZapBBInstrDir, which takes that environment variable and attempts to find the name of the file using wcsrchr(assemblyPath, '\'). This causes a crash on linux when collecting IBC counts. The fix is to ifdef it for linux to use the correct path separator.

This change also includes a fix to change the code for checking if IbcTuning is set to use == instead of -eq which was causing a failure in the build that was ignored.

Remove all Sse2 Pack* and Unpack* tests from GneerateTests.csx script

Due te earlier error data for initiating generation of Sse2 Pack*
and Unpack* HW intrinsics tests were left in the file. Code generated
from this script will not compile as several required changes are missing.
Removal of this tests data restores full functionality of script.

Fix for bug 12398: Lowering is inconsistent in checking safety of RegOptional. (#19740)

This fixes an inconsistency in lowering where it fails to make an
operand contained because IsSafeToContainMem() returns false yet
it marks it regOptional, which may cause a problem if the operand
will be loaded at the point of use.

I also fixed a case where an operand was marked RegOptional even though
there is a type size mismatch.

There are 7 places that are affected, I added repro cases for 4 of them.
I wasn't able to construct repros for the 3 places that deal with
floating point operands but decided to fix those places anyway.

Fixes #12398.

Merge pull request #19748 from briansull/evalop-specialized

Rework the EvalOp generic-template methods

Merge pull request #19823 from echesakovMSFT/CrossBitnessValueNumbering

Merge pull request #19830 from echesakovMSFT/CrossGenAllocateMethodDescChunkPerMethodDesc

During CrossGen allocate MethodDescChunk per MethodDesc

Merge pull request #19855 from dotnetrt/fix-arm-tstlst

Remove deleted Sse2 tests from Test.lst files for arm and arm64 Windows targets

Fix ICU load on OpenSUSE Tumbleweed (#19812)

For some reason, OpenSUSE Tumbleweed has decided to prefix the
version number in the SO name of the ICU libraries by "suse".
That prevents our ICU version detection code from working.

This change adds scanning for SO names that contain that prefix too.

Enable IJW Native calling managed (#19750)

* Adds back code required to make IJW native->managed calls (if the runtime is already started) and includes a simple test.

Morph ByRef args of SIMD field assign

Fix #19674

Merge pull request #19296 from mikedn/cast-cg

Refactor integer cast codegen

Add logging to file when CrossGen System.Private.CoreLib.dll on Unix (#19848)

Disabling the failing CoreFX tests related to ToString for negative zero

Removing i386/fptext.asm in favor of using the shared implementation in number.cpp

Updating Number.Formatting to properly print -0

Clean CodeGen::genEmitCall (#19804)

* delete isProfLeaveCB from arm signature

The previous implementation was done many years ago and I do not why it was done that way.

* extract GetSavedSet

* add isNoGCHelper

* delete isNoGC arg

* move declarations closer to their uses

* delete isGc from genEmitCall

* delete unused method declaration.

* add emitNoGChelper that accepts CORINFO_METHOD_HANDLE

* fix missed switch cases

* add function headers

* Fix feedback

* Fix feedback2

Add native vararg test (#17707)

* Add native vararg testing

This is a desktop test. It will be excluded for coreclr; however, decided it is worth adding
to the test tree in case we ever need the coverage.

* Fix several errors, and address feedback

* Split long lines

* Update vararg test

* Fix excludes

* Refernce private_corelib

* Correctly build vararg test

* Fix incorrect addition

* Fix build break unix

* Fix native build on unix

* Fix entry point

* Correctly pass doubles instead of floats

* Correct vararg test, pass double to pinvoke methods

* Remove printf tests

* Fix cmake for varargnative.c

* Fix cdel calling convention

* Correctly pass success back to the main method

* Explicitely undef cdecl

* Fix some excludes and exclude the new test on x86

* Add unix exclude

* Disable echo_short and correct header

Remove deleted Sse2 tests from Test.lst files for arm and arm64 Windows targets

Fixes #19853

Remove unused GenTree flags (#19840)

GTF_IND_ARR_LEN was set by the importer in minopts/debug mode and used only by value numbering, which does not run in minopts/debug mode.

GTF_FLD_NULLCHECK was also set by the importer and not used anywhere. fgMorphField has its own opinion about when an explicit null check is needed.

add a regression test for Linux GC segfault. (#19820)

* add test

* move the test to pri1 and reduce number of iterations

Make cast classification arch independent

Get rid of Lowering::getCastDescription

Cleanup LinearScan::BuildCast

Improve ARM genIntCastOverflowCheck

Refactor genIntToIntCast on ARM

Extract cast overflow check ARM codegen to genIntCastOverflowCheck

Refactor genIntToIntCast on XARCH

Add support for collectible types to SOS (#19842)

* Add support for collectible types to SOS

Collectible types indirectly reference managed LoaderAllocator via
pointer to native AssemblyLoaderAllocator stored in their MethodTable.
GC uses this relation when scanning object graph to determine which
objects are rooted and which ones are not.
The gcroot command in SOS doesn't understand this relation and so it
is unable to find all roots for LoaderAllocator.
This change fixes it.

* PR feedback

Make the failure to get the collectible info non-fatal to make it
compatible with older runtimes.

[Unix|Arm64] Correct transition block for unix arm64 (#19822)

* Correct unix transition block macro

Don't build and publish libcoreclr libeventprovider libeventpipe as crosscomponents on ARM (#19782)

* Use _install to stop copying sosdocsunix.txt to crosscomponents directory in src/ToolBox/SOS/Strike/CMakeLists.txt
* Use _install for coreclrpal in src/pal/src/CMakeLists.txt
* Use _install for eventprovider in src/scripts/genLttngProvider.py
* Unconditionally use add_library_clr and _install for eventpipe in src/scripts/genEventPipe.py

Re-enable GitHub_19397 test

Updating the ilproj-sdk to support the `-RESOURCES` switch

Merge pull request #19719 from dotnet/MattGal-patch-1

Fix typos

Convert all Sse2.Compare*** tests to template based

Implement AVX2 Gather intrinsic in JIT

Add test for AVX2 gather intrinsic

Add managed non-const fall-back for AVX2 gather intrinsic

Rename CarryLessMultiply to CarrylessMultiply

Fix ServiceController name population perf (dotnet/corefx#32072)

* Fix ServiceController name population perf

* Split tests

* Remove dead field

* Remove new use of DangerousGetHandle

* SafeHandle all the things!

* VSB #1

* VSB #2

* Fix GLE

* Initialize machineName in ctor

* Test for empty name ex

* Null names

* Inadvertent edit

* Unix build

* Move interop into class

* Reverse SafeHandle for HAllocGlobal

* Fix tests

* Disable test for NETFX

* CR feedback

* Pattern matching on VSB

* Direct call

* typo

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

Fix UAP build break (dotnet/corert#6287)

Kick registry related files out of UAP builds.

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

During CrossGen allocate MethodDescChunk for every MethodDesc in MethodTableBuilder::AllocAndInitMethodDescs in src/vm/methodtablebuilder.cpp

Replace VNForPtrSizeIntCon(addrValue) with VNForHandle(addrValue, GTF_ICON_FTN_ADDR) for function address in Compiler::fgValueNumberHelperCallFunc in src/jit/valuenum.cpp

Replace ssize_t with target_ssize_t in GenTree::ParseArrayAddress GenTree::ParseArrayAddressWork in src/jit/gentree.cpp src/jit/gentree.h

JIT: add option to dump jit-generated debug info (#19808)

Add dump option for debug info (like the ones we have for gc info,
eh, unwind, etc) so we can more readily verify jit changes that are
might impact debug info generation.

Virtual AttributeType property and signature generic types (#19818)

* Virtual AttributeType property and signature generic types

https://github.com/dotnet/corefx/issues/31614

1. This will allow Reflection providers the option
to supply the attribute type without building
an entire constructor.

https://github.com/dotnet/corefx/issues/31798

2. This will permit other Reflection providers
to support Type.MakeGenericMethodParameter()
in their implementations.

* More robust argument validation.

* Change parameter name

Deal with cross-bitness compilation warnings Pt.2 (#19781)

* Disable ASMCONSTANTS_C_ASSERT in cross-bitness scenario in src/vm/ceeload.cpp

* Adjust MAXFIELDMARSHALERSIZE for cross-bitness scenario in src/vm/arm/cgencpu.h

* Make ALLOC_ALIGN_CONSTANT host specific in src/inc/stdmacros.h

* Make PRECODE_ALIGNMENT host specific in src/vm/arm/cgencpu.h

* Disable unreachable code in src/vm/arm/stubs.cpp

* Adjust CorDBIPC_BUFFER_SIZE for cross-bitness scenario in src/debug/inc/dbgipcevents.h

* Disable warning C4359 in src/vm/arm/cgencpu.h

* Deal with warning C4267: 'initializing': conversion from 'size_t' to 'int' in src/vm/stublink.cpp

* Deal with warning C4267: 'initializing': conversion from 'size_t' to 'int' in src/vm/callingconvention.h

* Disable unreachable REGDISPLAY constructor in src/inc/regdisp.h

Add modules to the heap only dump in order to have: (#19761)

+ types in with sos.dumpheap -stat instead of UNKNOWN
  + complete stack with sos.dumpstack On a dual core machine, full=11GB, withheap=226MB, withheap(+patch)=269MB

i386: Fixed definition with declaration in eetoprofinterfaceimpl.cpp (#18792)

Fix callKillSet for CORINFO_HELP_ASSIGN_BYREF on x64. (#19780)

* Fix callKillSet for CORINFO_HELP_ASSIGN_BYREF.

on x64.

* Fix typos.

Rework the EvalOp generics template methods
replaced EvalOpIntegral with EvalOpSpecialized

Add Avx2.Permute4x64 template based tests

Implement Avx2.Permute4x64 intrinsics

Fix build-test.sh wrapper build (#19779)

* Fix build-test.sh wrapper build

In addition this change creates a json file with the build info so that the wrapper build
may be skipped if it was built on the same os/arch/buildtype.

* Address pr feedback

Revert "Remove not necessary type forwarded from attributes when type comes from mscorlib in netfx" (#19784)

This reverts commit 4fb001d431e0f44df15709f02f776e565fec14c7.

Fix build break in CoreRT/ProjectN

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

Merge pull request #19772 from fiigii/insxor

Fix target register false dependency of lzcnt, tzcnt, and popcnt

JIT: don't allocate promoted struct death vars map for clear or lookup (#19753)

Only allocate this map if we're adding something. Refactor clearing and
lookup to do minimal work if the map hasn't been allocated (and it won't
have been in debug / minopts).

Saves a tiny bit of throughput and memory.

IndexOf(char, Ordinal) use fast path (#19788)

Enable gc_profiling for local gc (#19777)