Merge pull request #12068 from omajid/master

Fix typo in CRL_CMAKE_WARNINGS_ARE_ERRORS

Remove relocations from SECTION_MethodDesc for ngened images for fields accessed from jit code for Linux ARM (#11963)

Call sched_yield directly for Sleep(0) (#12063)

Related to PR #12057

Fix typo in CRL_CMAKE_WARNINGS_ARE_ERRORS

Merge pull request #12064 from rartemev/fix_bash_script_issues

Fix path in bash script that runs test

fix procCount assertion (#12065)

Tiny typo in TaskCompletionSource xml comment (#12066)

Add support for not breaking *nix build on warnings (#12039)

Add a build flag to make -Werror optional and let the build continue
even in the presence of warnings.

This option is very useful for anyone compiling with a different
(version of the) compiler. A different (version of the) compiler may
produce a different set of warnings and a piece of code that compiles
without warnings may emit warnings with a different (version of the)
compiler.

Resolves https://github.com/dotnet/coreclr/issues/8586

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

Merge nmirror to master

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

Merge pull request #11699 from swgillespie/local-gc-branch-merge

Integration from dev/local-gc into master

attempt to fix r2r test failures

Merge pull request #11803 from BruceForstall/FixRuntest

Update runtest.cmd help to reflect reality

fix SleepEx(0,false) on Linux (#12057)

Return 0 on success from GC Reliability Framework scripts (#11990)

* Return 0 on success from GC Reliability Framework scripts

* Don't save standard out, Jenkins already does this

Merge pull request #11957 from sjsinju/spill_temp_size

[RyuJit/ARM32] Adjust alignment for TYP_DOUBLE in tmpPreAllocateTemps

Merge pull request #12003 from sjsinju/reg1_reg2_planB

[RyuJit/ARM32] Fix wrong behavior of UINT->CHAR casting

Add support for flat vtables in RyuJIT (#12041)

CoreRT needs one less indirection to get to the virtual method.

On CoreCLR a virtual method call resolution goes like:

```
this -> MethodTable -> chunk(?) -> slot
```

On CoreRT, it's more flat:

```
this -> EEType -> slot
```

Fix a bug in LoadNativeStringResource to honor the contract properly when the buffer is not big enough (#12051)

* Return last error properly

* adjust pcwchUsed if ERROR_INSUFFICIENT_BUFFER

* return hr

* ERROR_INSUFFICIENT_BUFFER is a win32 error....

Invoke jitShutdown during ProcessShutdownWork (#12054)

[x64/Linux] Fix FEATURE_INTERPRETER Release build (#12046)

Modify comments

Fix optdata restore functionality (#11935)

Fix optdata restore functionality (#11935)

Bring back optdata restore functionality following the move to 2.0
BuildTools and csproj-based CLI. Disable a harmless warning that broke
the Linux build when consuming PGO counts due to -Werr.

Fix #11796 for master

Ensure that platform is considered for LongPlatformName instead of PackagePlatform (#11900)

Support msbuildonunsupportedplatform on non-x64 architecture (#11966)

Fix 11991 (#12013)

Fix the case when the path doesn't have any '\'.
Fix clean_up.

Fix warnings on 32-bit platforms

Merge pull request #11959 from hqueue/arm/ryujit/11779

[RyuJIT/ARM32][LSRA] Fix restoring TYP_DOUBLE interval in ARM32

Merge pull request #11860 from rartemev/fix_tests_build_4

Fixed tests build on Linux

Merge pull request #11876 from hqueue/arm/ryujit/merge_lowerarmarch

Merge remaining common arm32/arm64 lowering functions

[Local GC] Remove uses of g_SystemInfo from the GC (#11628)

* [Local GC] Don't use g_SystemInfo to infer the number of processors

* [Local GC] Avoid using g_SystemInfo.dwAllocationGranularity from within
the GC.

In a few places, the GC aligns the size of a request memory reservation
to the allocation granularity before calling
GCToOSInterface::VirtualReserve. Upon inspection, this isn't necessary.

* [Local GC] Remove uses of g_SystemInfo from the handle table

* Add new API on GCToOSInterface for querying the total number of procs on the machine

[Local GC] Obtaining configuration information (#11379)

* [Local GC] Skeleton for GC configuration

* Initial tweaks after design feedback:
    1) Use string keys instead of enums. Upon receiving a string key,
    the EE looks at it and, if it's something that comes from startup
    flags, responds using the startup flag information. Otherwise, it
    forwards the string onto CLRConfig.
    2) Add a mechanism for getting string configuration values from the
    EE. This includes adding a RAII wrapper around strings so that they
    are freed correctly.

* Remove uses of g_pConfig from the GC and replace with GCConfig

* Use the GCConfig system for the GC log

* Fix poorly-named parameter

* Add documentation and caching of bool and int configs obtained from the EE

* Remove AppDomainLeaks as dead code

* Remove GC trace configs as dead code

* Repair unix build

* Fix an issue where we started the GC in the wrong latency mode

* Fix the unix build

* Pipe GCRetainVM configuration to the GC

* Dead code removal in the GC sample

* EEConfig -> GCConfig for heap verification constants in the GC

* Populate config information for bools and ints eagerly at startup

* Initialize g_theGCToCLR before initializing GCConfig

* Propegate HoardVM config to the GC

* Fix an incorrect comment

[Local GC] Add async pinned handle methods to the handle interface (#11569)

* [Local GC] Add async pinned handle methods to the handle interface

* Add a callback to HandleAsyncPinnedHandles for handle enumeration

* Rename HandleAsyncPinnedHandles -> EnumerateAsyncPinnedHandles

* Introduce typedef for function pointer type

[Local GC] Fix handle table interface violation (#11558)

[Local GC] Fix some handle table interface violations in COMDependentHandle (#11358)

Fix a build break in feature branch caused by obsolete error (#11335)

[Local GC] Scaffolding for loading a standalone GC (#11242)

* Configure the build system to build a CoreCLR capable of loading a standalone GC

* Proto-implementation of dynamic GC loading

* Build the GC with the VM's CMakeLists when doing a non-standalone build of the GC

* [Local GC] Introduce a new feature define, FEATURE_STANDALONE_GC_ONLY, to be used by the CI to explicitly test local GC dynamic loading code paths

* Fix the FEATURE_STANDALONE_GC_ONLY build for unix linkers

* Rebase against master

* Code review feedback: use the existing Unix exports file

gdbjit symbols exported (#11739)

* multiple export files as input for generate_exports_file

Now, generate_exports_file is able to process multiple input arguments.
This feature is useful when you want to export additional symbols depending on
compile time dependannt feature(-D option.)

* export file for gdbjit added

src/dlls/mscoree/gdbjit_unixexports.src file added
To enable gdbjit feature, libcoreclr.so should have __jit_xxx
symbols.
Current tizen build system removes .symtab section during rpm debuginfo
build. This patch moves __jit_xx symbols into .dynsym section.

[Arm64] Use inner shareable for instGen_MemoryBarrier

[Arm64] Use half barriers for IL Volatile implementation

Updating version of xUnit-Performance-Api (#11987)

- This update contains a fix for this issue: https://github.com/Microsoft/xunit-performance/issues/230
- Consolidate the Api version in two places: `dependencies.prop` and `PerfHarness.csproj`
- Move performance log files to root directory where Jenkins can archive them
- Fixed the output to console, so it is easier to see on the Jenkins job the sccript steps live and get an exact repro step if needed

Fixes deserializaing TimeZoneInfo by putting back the IDeserializationCallback.OnDeserialization method. (#12024)

[x64/Linux] Use correct argument registers in InterpreterStub (#12002)

* [x64/Linux] Use correct argument registers in InterpreterStub

* Adjust Linux/Windows ABI difference

* Use macros instead of array

* Follow naming convention

Merge pull request #11988 from stephentoub/fix_fds

Ensure several runtime file descriptors are CLOEXEC

Remove GCX_PREEMP call when allocating a buffer. (#12016)

Post-Serialization cleanup fixes to Reflection (#12010)

This is based on the following principles:

- We should not be making changes to instance/virtual methods
in the abstract base types for Reflection. These are apis
in themselves to third party classes that subclass these types.
Even if the Runtime chooses not to make its own implementations
of these type serializable, third party classes have the right
to make that choice for themselves and build off the base class
serialization infrastructure that was shipped before.

RuntimeAssembly and RuntimeModule still override with PNSE
so we'll get the desired exception for those cases.

- The Runtime's own implementations of these types will not be
serializable. Removing all [Serializable] attributes and
serialization interfaces. Runtime types are internal types and
thus have no duty to implement ISerializable if the public
abstract base type does not.

Delete useless file (#12021)

Disable inheritance of several common file descriptors

Default to portable build in dev and official builds (#11985)

Fix issues with RelativePointer instead of RelativeFixupPointer (#12004)

Expose Enumerator members out of EnvironmentAugments. (#11980)

Expose Enumerator members out of EnvironmentAugments.

[RyuJit/ARM32] Fix wrong behavior of UINT->CHAR casting

'JIT\Regression\CLR-x86-JIT\V1-M11-Beta1\b44879\b44879\b44879.cmd'

When UINT cast to CHAR, MOV instruction had been emitted with R0 to R0 for extending the type.
It is wrong behavior to cast.

In case of INT to BOOL casting, that use uxtb instruction to extend BOOL type.

So this PR would use a uxth instruction too insted of wrong mov instruction.

@dotnet/arm64-contrib @dotnet/arm32-contrib Please make sure this code is okay.

[RyuJIT/ARM32] Update restoring TYP_DOUBLE interval

- Introduce findAnotherHalfRegRec()
- Restore TYP_DOUBLE interval to valid double register

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>

[RyuJIT/ARM32] Applying review feedback

- Group cases for each target to improve readability

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>

[RyuJIT/ARM32] Applying review feedback

- Use canRestorePreviousInterval() for all target
- Update condition for restoring TYP_DOUBLE interval

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>

Merge pull request #11995 from ViktorHofer/SerializationFollowUp

Internal hashtable serialization attribute removed

Allowing larger segment size for SOH, in preparation for enabling it for NoGC region. (#11923)

This is just so that I can test with larger SOH seg size to make sure we are not getting AVs.
More perf changes will happen with the NoGC region changes.

Remove relocations from SECTION_MethodDesc for ngened images (#11394)

Internal hashtable serialization attribute removed

Coretype variables renamed back to netfx counterpart for reflection based serialization (#11910)

* Variables renamed for reflection based serialization

* Make EqualityComparers serialize like desktop

* add missing interfaces

* TimeZone serializable added

* Internal hashtable serializable

* Removed TimeZone as serializable type

* Remove Lazy<T>'s [Serializable] attribute for 2.0

For performance, Lazy was completely rewritten for .NET Core 2.0 and has an entirely different format than desktop; trying to get it to match the desktop serialization format would require either reverting or providing a complicated custom serialization/deserialization implementation to try to match.  Lazy can also wrap an Exception that occurred from trying to instantiate the object, and the only exception types that are serializable as of now in core are the base Exception and AggregateException.  As such, we're cutting it from the list of supported types in 2.0.  An easy workaround is simply to do what the implementation does: serialize lazy.Value instead of lazy.

* tiny fixes to equalitycomparer.cs

Merge pull request #11977 from mskvortsov/FixDoubleRegArgNext

[RyuJIT/arm32] Fix double argument passing

Merge pull request #11872 from hqueue/arm/ryujit/dodge_lowering_long_cmp

[RyuJIT/ARM32] Dodge lowering long compares

Skip cast nodes for the tail call check. (#11986)

Fix #11983

Fix build break

Moves windows specific field to Debug.Windows.cs file (dotnet/corert#3733)

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

Use FD_CLOEXEC instead of equivalent 1 in InternalCreatePipe

Remove Debian Armel (#11978)

RyuJIT/arm32: Fix double argument passing

Fix failures in ETW logging on 4GB aware 32 bit processes (#11941)

We incorrectly cast a 32 bit pointer to a 64 bit poitner using as a SIGNED integer.
If this is a 32 bit process that is using more than 2GB of memory this can result in
sign rather than zero extension.  This makes the poitner invalid at the OS level
and causes the OS API to fail.

We disovered this in Visual Studio when debugging large (Rosyln) scenarios.
There were numerous failures which causes sever slowdowns becasue the
EventSource logged OutputDebugString events when the OS API failed.

The fix is to use unsigned extension.   Note that I have confirmed that casting
from a IntPtr or a void* to a ulong does zero extension (that is it uses the target type
to determine whether to use sign or zero extension).

To be useful for Visual Studio, this needs to be ported to the desktop runtime.

Merge pull request #11972 from hseok-oh/ryujit/issue_11971

[RyuJIT/ARM32] Fix offset of struct argument when using stack

Merge pull request #11973 from hseok-oh/ryujit/issue_11837

[RyuJIT/ARM32] Fix assertion failed 'remainingSize == TARGET_POINTER_SIZE'

[RyuJIT/ARM32] Fix assertion failed 'remainingSize == TARGET_POINTER_SIZE'

Fix assertion failed by 'remainingSize == TARGET_POINTER_SIZE'
Fix to check register allocation correctly

[RyuJIT/ARM32] Fix offset of struct argument when using stack

Fix offset value when struct argument is passed on stack

[RyuJIT/ARM32] Fix restoring TYP_DOUBLE interval in ARM32

- Fix segmentation fault by defining nextRegRec when nextRegRec is not defined yet.

- Introduce canRestorePreviousInterval()
  This helper check whether previous interval can be restored considering TYP_DOUBLE.

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>

Make linux test build worked

Implement GetCurrentProcessCpuCount for Unix and merge NumberOfProcesors implementations. (#11742)

* Implement GetCurrentProcessCpuCount for Unix and merge NumberOfProcessors implementations.

* Split GetCurrentProcessCpuCount in PAL and non-PAL parts

* gcenv.unix.cpp: GetCurrentProcessAffinityMask: revert, GetCurrentProcessCpuCount: add implementation

* bugfix: #if instead of #ifdef HAVE_SCHED_GETAFFINITY

* Remove PAL_GetCurrentProcessCpuCount

* gcenv.unix: implement GetCurrentProcessAffinityMask, GetCurrentProcessCpuCount

* numa.cpp: initialize g_cpuToAffinity[i].Number when numa.h is missing

* cleanup

* mscorwks_unixexports.src: Add GetProcessAffinityMask

* PR feedback

* Add GetProcessAffinityMask to mscordac_unixexports.src

added DWARF exprloc for static value class fields (#11911)

[RyuJit/ARM32] Adjust alignment for TYP_DOUBLE in tmpPreAllocateTemps

In #11782,

Assertion failed 'spillTempSize <= lvaGetMaxSpillTempSize()' has been occurred in code generation phase.

Adjusting alignment for TYP_DOUBLE in tmpPreAllocateTemps is needed.

Fix build breaks

* Remove UnitySerializationHolder references
* Remove the fields of StrongNameKeyPair
* Initialize BadImageFormatException._fusionLog
* Remove MemberInfoSerializationHolder references

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

IDeserializationCallback cleanup

Throws PlatformNotSupportedException from IDeserializationCallback and IObjectReference implementations on types that are no longer serializable.

Fix tool initialization detection (#11912)

init-tools.cmd attempts to create a file in a directory that doesn't exist. File creation fails and next time the script is run initialization will be performed again. That wastes a lot of time.

Update CoreClr to preview1-25327-01 (#11949)

Update test coverage statements in workflow doc

Update license links

Update CoreClr, CoreFx to preview1-25326-02, preview1-25324-02, respectively (#11932)

Update contributor docs (#11944)

* Update contributor docs

* Update

* Added up-for-grabs and ports

* Update binary licensing

* Update\

* Simplify developer workflow doc

* Update per feedback

Enable linux perf runs (#11905)

- Removing `[assembly: MeasureInstructionsRetired]` from performance tests.
- Due to this issue: https://github.com/Microsoft/xunit-performance/issues/231, we are blocked from running on performance tests on Linux.
  The tag is not used by infrastructure and it is passed during runtime when profiling on Windows.
- Adding the option to execute CoreRun with the stability prefix.
- Moving file to be archived by Jenkins and updating groovy file.
- Setting the maximum number of iterations.

ISerializable cleanup (#11873)

Changes to throw PlatformNotSupportedException from ISerializable.GetObjectData and serialization constructors on non-Serializable types. Also removes private serialization constructors and some unneeded code that was used to support serializing non-serializable types. A few tests testing GetObjectData implementations are also removed. For Exception types, we call base instead of throwing from GetObjectData to be consistent with the many Exceptions that don't override GetObjectData

Move rundown to the same code-site where we do ETW rundown before EEShutdown is marked as started. (#11937)

Merge pull request #11920 from briansull/crossgen-tuning

Remove the check that requires an instrumented flags for SPC when collecting IBC data

Fix BitSetUint64, clean and improve the surrounding code. (#11721)

1. Add the functions that were not implemented in bitsetasuint64inclass/
2. BitSetUint64: make copy_constructor public
3. delete VARSET_INIT_NOCOPY and ALLVARSET_INIT_NOCOPY
4. delete VARSET_INIT and ALLVARSET_INIT.
5. Use the correct const type for VARSET_TP.

Merge pull request #11823 from hqueue/arm/ryujit/lsra_gc_helper

[RyuJIT/ARM][LSRA] Update register mask for GC helper

EETypeHashTable lookups can deadlock with GC during comparison with unrestored persisted TypeHandles (#11922)

Fix compile with disabled FEATURE_READYTORUN option (#11875)

- just remove FEATURE_READYTORUN definitions
in clrdefinitions.cmake to turn off option

Signed-off-by: Petr Bred <bredpetr@gmail.com>

[RyuJIT/ARM32] Update RMB for helper function

Make RBM_CALLEE_TRASH be consistent with CORINFO_HELP_STOP_FOR_GC helper
which is JIT_RareDisableHelper defined in vm/arm/asmhelpers.S

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>

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

Merge nmirror to master

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

Merge pull request #11917 from JosephTremoulet/RationalCall

Clear GTF_CALL from non-calls in rationalize

Apply review feedback to increase readability

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>

Remove the check that requires an instrumented flags for mscorlib when collecting IBC data
This is not needed for CoreCLR and blocks some valuable usage cases.

Merge pull request #11878 from CarolEidt/ContainLclVars

Make non-tracked lclVars contained

Merge pull request #11919 from pgavlin/XunitTestResults

Add XUnit results for Windows_NT x86 jobs.