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.

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

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.

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)

[local gc]  move DacNotifyGCMarkEnd and AnalyzeSurvivorsRequested to GCToEEInterface  (#19774)

Updating runtest.py so that it works with Python 3

Add startup hook in System.Private.CoreLib (#19486)

* Add startup hook in System.Private.CoreLib

ProcessStartupHooks can be called from the host before the user's Main
entry point. It receives a list of dlls and types containing
Initialize() methods that will be called, making it possible to inject
managed code early during startup.

* Allow ! in assembly path for startup hook and other changes

Also:
- Report full assembly path when startup hook assembly is not found
- Remove unnecessary assert
- use Type.Delimiter instead of "."

* Use C# 7 tuple syntax and remove assert

* Improve error handling

Throw MissingMethodException only when there aren't any Initialize
methods at all.

When there are Initialize methods with incorrect
signatures (parameters, return type, visibility, or instance methods),
throw invalid signature error.

This should improve diagnosability of this feature.

* Remove eager check for missing startup hook assemblies

* Require full assembly path and use Split(char) overload.

* Remove startup hook type syntax

The type is now required to be "StartupHook" (in the global
namespace).

* Add assembly path to startup signature exception

With a hard-coded type name, printing the type.method of the startup
hook in the exception will no longer be much of an aid in debugging
startup hook signature issues. Adding the assembly path makes it clear
which startup hook had the problem.

* Use const strings

* Call startup hook inside ExecuteMainMethod

This way it will be called when the application is executed, but not
during other uses of hosting apis that go through
coreclr_create_delegate. This change will ensure that the threading
state is set based on attributes in the main method, before the
startup hooks run.

* Run startup hooks after setting root assembly and other fixes

- Run startup hooks after setting the appdomain's root
  assembly (visible in Assembly.GetEntryAssembly()
- Make the class static
- Remove debug output
- Don't allocate an empty ARG_SLOT array

* Allow non-public Initialize method, adjust coding style

* Remove overly-specific assert

Update existing COM test assets (native/managed) to share CLSIDs with… (#19756)

* Update existing COM test assets (native/managed) to share CLSIDs with other projects.

* Update Interop.settings.targets

None of these test changes apply to Ubuntu ARM

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

Fix CoreCLR debug launch (#19730)

JIT: don't compute edge weights if not optimizing (#19736)

We were spending a bit less than 1% of jit time computing edge weights
for minopts an debug codegen, but then never using them.

Also when optimizing we wanted to just compute edge weights but ran
through a bit more code than that.

Restructure the code so it's clearer what runs when, and bypass the
edge weight computations for minopts and debug codegen.

typo fix (#19763)

Fix test folder register (#19749)

rename test

Delete code that tracks stack level in morph. (#19703)

* call fgCheckArgCnt only from stackLevelSetter

* delete changing fgPtrArgCntMax from codegencommon

* delete fgPtrArgCntCur

* reset write phase only once

* delete gtStkDepth

* add headers for the new fucntions

* fix comments

fix merge conflict in build-test.sh (#19744)

Define T_RUNTIME_FUNCTION in cross-bitness crossgen ARM32 (#19718)

* Allow 'Non-ARM Host managing ARM related code' scenario in src/inc/crosscomp.h
* Restrict RtlUnwindEx RtlVirtualUnwind to X86 || FEATURE_PAL in src/inc/clrnt.h
* Define PTR_RUNTIME_FUNCTION via T_RUNTIME_FUNCTION in src/inc/daccess.h

Merge pull request #19742 from CarolEidt/Disable19397Arm64

Disable Failing test on Arm64

Merge pull request #19727 from echesakovMSFT/ConstantPropImmedValNeedsReloc

Do not Constant-Prop immediate values that require relocation on ARM32

Update floatdouble and floatsingle to use `floatcontrol(precise, value push)` and `floatcontrol(precise, pop)`

Updating floatdouble and floatsingle to work around an incorrect result under /fp:fast

Merge pull request #19726 from briansull/vn-dumping

Value Numbering dumping changes and documentation on normal and excSet value numbers

Typos (#19737)

* testsing -> testing

* reclaming -> reclaiming

* oppurtunity -> opportunity

* implemenation -> implementation

* indicaates -> indicates

* enregisterd -> enregistered

* compuations -> computations

* convinient -> convenient

* intution -> intuition

* Communicaton -> Communication

Fix typo

Remove not necessary type forwarded from attributes when type comes from mscorlib in netfx

Disable Failing test on Arm64

Value Numbering dumping changes and documentation on normal and excSet value numbers

Support the dumping of exception set
Added full method header comments for VNNormVal and VNWithExc
Removed extra spew for fieldSeq
Removed several unnecessary fgCurMemoryVN assigned printf
Added noway_asserts when dealing with a GT_ASG for a LHS that is a GT_PHI_ARG, GT_BLK or GT_OBJ

using struct for VectorPacket in PacketTracer benchmark

Merge pull request #19695 from CarolEidt/Fix19397

Handle multiReg COPY

Merge pull request #19205 from vancem/EventSourceUIntFix

Event source uint fix

Enable ibc official build (#19586)

Enable IBCOptimize for x86 and x64 Release builds

Improve struct promotion for 256-bit SIMD fields

Fix IJW test failing on Linux (#19729)

Disables the IJW test on Linux and ARM64

Cast gtIconVal to target_ssize_t in src/jit/assertionprop.cpp

Do not Constant-Prop immediate values that require relocation on ARM32

Merge pull request #19619 from echesakovMSFT/Mov32RelocatableDisplacementARM32

[ARM32] Split IF_T2_N3 into relocatable and non-relocatable cases

Merge pull request #19053 from riverar/riverar/r2r_size_unavail

Set function size to -1 if sizing info unavailable

Update BuildTools, CoreClr, CoreFx, CoreSetup, PgoData to preview1-03127-01, preview1-26827-04, preview1-26827-04, preview1-26827-01, master-20180827-0150, respectively (master) (#19543)

* Update BuildTools, CoreClr, CoreFx, CoreSetup, PgoData to preview1-03127-01, preview1-26827-04, preview1-26827-04, preview1-26827-01, master-20180827-0150, respectively

* Undo buildtools update

* Use older version of CoreFx.Private.TestUtilities

* Disable outdated test

* Disable failing test against #19537

Enable mixed mode assembly loading (#19542)

Bring back functionality for loading IJW assemblies and calling managed->native. Also add workaround to test case for the C++ compiler inserting calls to mscoree.

Merge pull request #19712 from briansull/remove-unused-arg

Removed unused argument 'evalAsgLhsInd' in Value Numbering

Merge pull request #19716 from BruceForstall/Fix19715

Fix warning C4723 potential divide by zero

return NETClientPrimitives to the exluce list for all platforms (#19717)

Replacing the IsDstSrcSrcAVXInstruction jump table with a flag.

Replacing the IsDstDstSrcAVXInstruction jump table with a flag.

Break into debugger on assertion failures (#19702)

* Break into debugger on assertion failures

Assertion failures terminated the process by default that made them hard to debug. Changed them to
break into debugger or trigger fail fast when the debugger is not attached. This should make the day-to-day
CoreCLR developer experience better and it is simular to what we had on .NET Framework in the past.

* Fix Unix build break

Add RaiseFailFastException to Unix PAL

Fake PR to test CI

Merge pull request #19701 from BruceForstall/FixLinuxNonJitBuild

Fix Linux/x86 altjit

Handle multiReg COPY

On x86, `MUL_LONG` wasn't considered a multi-reg node, as it should be, so that when it gets spilled or copied, the additional register will be correctly handled.
Also, the ARM and X86 versions of genStoreLongLclVar should be identical and shared (neither version were handling the copy of a `MUL_LONG`).
Finally, fix the LSRA dumping of multi-reg nodes.

Fix #19397

Merge pull request #19709 from acmyu/docs

R2RDump - Add tables describing which architectures have been tested

Fix warning C4723 potential divide by zero

Appears in VS 15.8 for arm64 builds.

Fixes #19715

Removed unused argument 'evalAsgLhsInd' in Value Numbering
It was previously used for Assign-Op forms of assignment
which we no longer use in the RyuJIT.

fix build break when STUB_LOGGING is defined (#19707)

Add missing license headers. (#19693)