Updating `JitEEVersionIdentifier` GIUD. A follow up for removing ArrayTypeDesc. (#1330)

Optimize AdjustMaxWorkersActive (#1103)

@ninedan reported this function as expensive in one of the server scenarios
he monitors. The actual time is spent in `ThreadStore::GetAllThreadList`.
Moving the call to `GetTotalWorkerThreadPoolCompletionCount` should reduce this
cost.

Fixes #1079

Copy change to XsdDateTime (#1319)

I copied the change from dotnet/coreclr#7769 to XsdDateTime which eliminates extraneous iterations of the days array.

Add documentation guidelines when adding new APIs (#1335)

* Add documentation guidelines when adding new APIs

* Update docs/coding-guidelines/adding-api-guidelines.md

Co-Authored-By: Dan Moseley <danmose@microsoft.com>
* Update docs/coding-guidelines/adding-api-guidelines.md

Co-Authored-By: Dan Moseley <danmose@microsoft.com>
* Update docs/coding-guidelines/adding-api-guidelines.md

Co-Authored-By: Maira Wenzel <mairaw@microsoft.com>
* Update docs/coding-guidelines/adding-api-guidelines.md

Co-Authored-By: Maira Wenzel <mairaw@microsoft.com>
* Update docs/coding-guidelines/adding-api-guidelines.md

Co-Authored-By: Maira Wenzel <mairaw@microsoft.com>
Co-authored-by: Dan Moseley <danmose@microsoft.com>
Co-authored-by: Maira Wenzel <mairaw@microsoft.com>

Remove STREAM_MEMORY_OVERLOADS_AVAILABLE (#1328)

Removed duplicates/unused code from src\jit\jitstd. (#1272)

make_signed/make_unsigned moved to clr_std/type_traits.
Removed pair.h and type_traits.h

Fix #27557

Fix reported result for incompatible version for default ALC (#1083)

FindInLoadContext and ApplicationAssemblies stages were reporting name
mismatch instead of incompatible version

CoreFX Renames in System.IO PathInternal.Tests (#1017)

Fix typo: availabe -> available (#1324)

Add CMAKE add_jit function and remove duplicated code for all JIT (#1292)

[ARM64] Change INS_bkpt to INS_brk for INS_BREAKPOINT (#892)

* [ARM64] Change INS_bkpt to INS_brk for INS_BREAKPOINT

https://github.com/dotnet/runtime/issues/606

* [ARM64] Change INS_bkpt to INS_brk for INS_BREAKPOINT

https://github.com/dotnet/runtime/issues/606

* [ARM64] Change INS_bkpt to INS_brk for INS_BREAKPOINT

https://github.com/dotnet/runtime/issues/606

* [ARM64] Change INS_bkpt to INS_brk for TARGET_UNIX

https://github.com/dotnet/runtime/issues/606

* [ARM64] Change INS_bkpt to INS_brk for TARGET_UNIX

https://github.com/dotnet/runtime/issues/606

* Fix formatting errors for INS_MULADD

Move LocalAddressVisitor to a separate file (#1306)

implement GetScopeId() on Linux (#1038)

Add asserts to catch unintentional use of generic overloads (#1200)

* Add asserts to catch unintentional use of generic overloads

* Delete unnecessary ifdefs

Move Corelib shared files under Common (#1307)

Contributes to #1232

Implement HttpContent and HttpClient cancellation APIs (#686)

* Implement HttpContent and HttpClient cancellation APIs

Implements APIs approved in dotnet/corefx#32615 and dotnet/runtime#576

* Use new CreateContentReadStreamAsync overload in MultipartContent

* Expose HttpContent.SerializeToStreamAsync overload with cancellation

* Remove HTTP_DLL preprocessor directives

* Rename WIN_HTTP_HANDLER to WINHTTPHANDLER_DLL

* Use more 'var' where type is obvious

Share compiler config code between libs and coreclr (#1072)

* Share compiler config code between libs and coreclr

Delete cmake_minimum_required from PAL tests (#1312)

It is sufficient to have it in the top level file only

Fix which platform should be excluded. (#1314)

Consolidate .netcoreapp.cs files because System.Numerics.Vectors project is no longer cross-compiled (#1310)

Disable failing jobs in PR testing. (#1283)

* Disable failing jobs in PR testing.

* Disable the same in runtime file.

Make sure we update nextByte when we decode NEXT_OPCODE in x86 GCinfo (#1299)

* Make sure we update nextByte when we decode NEXT_OPCODE in x86 GCinfo

* Update src/coreclr/src/tools/crossgen2/ILCompiler.Reflection.ReadyToRun/x86/InfoHdr.cs

Co-Authored-By: Jan Vorlicek <janvorli@microsoft.com>
Co-authored-by: Jan Vorlicek <janvorli@microsoft.com>

Contributing: renaming methods (#1303)

Fix GCRefMap for generic method calls via Unboxing stub (#1304)

Crossgen2 was generating GCRefMap for calls to generic methods via
unboxing stubs incorrectly. With unboxing stub, the caller doesn't pass
in an extra generic parameter since the reference to the boxed value
type passed to the stub serves that purpose. But the GetCallRefMap method
was generating GCRefMap entry for the extra generic argument, which was
leading to a crash when GC was scanning the arguments.

Fix DelayLoadHelper kind for delegate ctors and IsInstanceOf (#1302)

Crossgen2 was incorrectly setting the DelayLoadHelper kind to
DelayLoad_Helper instead of DelayLoad_Helper_Obj and
DelayLoad_Helper_ObjObj. This was causing GC holes.

Update the path to built runtime for `build -vs` (#1295)

* After 3b9abae, the script was not updated to reflect the changes and was still setting the old path. This resulted in VSTest failing with `It was not possible to find any compatible framework version...`. This commit updates the path (albeit hard-coded; though, this is done in a few other scripts as well) to fix this for now.

Fix Environment.UserInteractive on Windows (#1234)

* Implement Environment.UserInteractive

* typo

* rename

* Revert M.Win32.SE

* CR feedback

* CR feedback:

* typo

* Fix test

* GUOI declaration

* Another typo!

* Fix build break

* Update src/libraries/System.Private.CoreLib/src/System.Private.CoreLib.Shared.projitems

Co-Authored-By: Jan Kotas <jkotas@microsoft.com>
* Update src/libraries/System.Private.CoreLib/src/System.Private.CoreLib.Shared.projitems

Co-Authored-By: Jan Kotas <jkotas@microsoft.com>
* Update src/libraries/System.Private.CoreLib/src/System.Private.CoreLib.Shared.projitems

Co-Authored-By: Jan Kotas <jkotas@microsoft.com>
* CR feedback

Co-authored-by: Jan Kotas <jkotas@microsoft.com>

Removing ArrayTypeDesc (#1201)

* Reduce use of ArrayTpeDesc, somewhat

* remove `AsArray`. Fewer uses of ArrayTypeDesc

* remove `IsArrayType`

* remove AssertArrayTypeDescLoaded

* normalized use of GetElementType vs GetTypeParam vs GetArrayElementTypeHandle

* removed GetMethodTableOfElementType

* HasTypeParam  and GetTypeParam should work for TypeHandle when array is not a TypeDesc

* Some cleanup of `IsTypeDesc` use.

* Removed `ArrayTypeDesc`

* a few fixes

* another fix

* dealt with TODOs

* couple fixes from reviewing code

* PR feedback:
Removed `CORINFO_HELP_NEWARR_1_R2R_DIRECT`
Folded a `cmp` in x86 array Address.
Removed` METHOD__BUFFER__BLOCKCOPY` from mscorlib.h
`GetMethodTableOfRootTypeParam`
`GetRootTypeParam`
Straightened `Object::GetTypeHandle`
`AsMethodTable` changes from PR feedback

* Removed `ELEMENT_TYPE_NATIVE_ARRAY_TEMPLATE_ZAPSIG`

* `TypeDesc` is ok in `canInlineTypeCheckWithObjectVTable`

* trivial  `AsMethodTable` changes

*  `GetVarTypeForTypeHandle` should throw `unsupported` on TypeDescs

* not digging for the root element type in `GetGCSafeTypeHandleIfPossible`

* Last PR feedback

* Fix `TypeInfoTests.Guid()`

Revert "Revert "Support GetCultureInfo with predefinedOnly flag (#654)" (#1257)" (#1261)

This reverts commit 4f27799093dcc168384db1ee783e5164a1723df8.

Make corelib work in VS (#1284)

Make a copy of the image for PEReader (#1263)

[runtime] Arm64: Add ZipLow, ZipHigh, UnzipOdd and UnzipEven codegen and tests. (#1190)

Improve message for missing C++ workload (#1253)

* Improve message for missing C++ workload

* typo:

* Fix case of missing rootdir

Fix CompareInfo and SQL tests (#1254)

* Fix CompareInfo tests after Windows Regression With Kana

* Fix SQL test

* Address the feedback

Move most of CoreLib interop under Common (#1247)

Contributes to #1232

Escape vertical bar (`|`) in the echo message (#1160)

* When `--runtime-path` is not specified, this unescaped character causes the batch file to attempt executing `--runtime-path` (which obviously is not possible) giving out very confusing failures.

Arm64: Implement AbsoluteCompare* hardware intrinsics (#1223)

Implements the following set of hardware intrinsics:

* AbsoluteCompareGreaterThan
* AbsoluteCompareGreaterThanOrEqual
* AbsoluteCompareLessThan
* AbsoluteCompareLessThanOrEqual

Implements instructions facge and facgt.

Revert "Support GetCultureInfo with predefinedOnly flag (#654)" (#1257)

This reverts commit 5272637ab8ae41c164a51316d792c8dd316df35d.

Support GetCultureInfo with predefinedOnly flag (#654)

* Support GetCultureInfo with predefinedOnly flag

Clean workspace in global-build-job.yml (#1136)

More Code Cleanup for EH WriteThru (#1180)

* More Code Cleanup for EH WriteThru

These changes arose from the code reviews for the EH WriteThru work. Some are only tangentially related to EH write-thru, were suggested during that review.

* Revert change to hasEHBoundaryIn condition

* PR feedback

* More PR Feedback

* Fix comment

Allow folding of aligned loads when using the VEX encoding and optimizations are enabled (#376)

Generate struct LCL_VAR nodes in LocalAddressVisitor (#1084)

Converts trees like OBJ(ADDR(LCL_VAR)) into single struct LCL_VAR nodes. It turns out that these are pretty common, thanks to call args & impNormStructVal. Doing this transform before global morph allows local assertion propagation to catch more copies.

Combine pal_compiler.h definitions (#1240)

Implement Arm64 comparison hardware intrinsics (#1196)

This implements the following set of hardware intrinsics:

* CompareEqual
* CompareGreaterThan
* CompareGreaterThanOrEqual
* CompareLessThan
* CompareLessThanOrEqual
* CompareTest

[runtime] Arm64: Add AddAcross Codegen and Tests. (#1093)

Annotate System.Text.Json for nullable (#528)

Annotate System.Text.Json for nullable

Delete DumpIR code (#1202)

Fix order of arguments in message box call (#1214)

fixing wrong MOV asm instruction emitter for arm64 (#1216)

Fix typo in exception message (#1205)

Consolidate .netcoreapp.cs files in Common project (#1178)

Expose SocketAsyncEventArgs constructor for suppressing the execution context (#706)

Make execution context suppressing ctor public. Resolves #937.

More R2RDump refactoring to make it easier for ILSpy to consume the APIs (#1209)

Merge pull request #266 from ts2do/BigIntegerFix

Eliminate copy assignment of BigInteger

Remove duplicate definition (#1067)

Consolidate .netcoreapp.cs test files in System.Reflection (#1176)

* Consolidate .netcoreapp.cs files because System.Reflection project is no longer cross-compiled

* Move extensions to separate class

Remove some unnecessary console noise in the build (#1146)

* More nologo/nosummary

* Avoid CMAKE logging when skipping copy

Intrinsify typeof(T).IsValueType (#1157)

Remove early exit logic in GetSortKey-related code paths (#1078)

Avoid copying in ReadSectionHeaders (#894)

Apply FlagsAttribute to Characteristics (#837)

Revert changes to EncoderNLS/DecoderNLS.Convert (#752)

Improve ForEach micro-benchmark for ImmutableArray (#1183)

* Improve Foreach benchmark in ImmutableArray<Int32>

Analysis of the generated ASM code vs the same benchmark for Array shows
that the GetEnumerator call is not being inlined (the loop itself is).

In the case of the ValueType ImmutableArray.GetEnumerator method,
there's a call to ThrowNullRefIfNotInitialized for validation.

By adding MethodImplAttribute(MethodImplOptions.AggressiveInlining)
to both methods, we are able to force the JIT to inline the call and get
similar results in the benchmark.
Looking at the hardware counters collected in the benchmark, there are less
CacheMisses and BranchMispredictions/Op when the inlining happens.

Unfortunately, the same fix didn't seem to work for the other overloads of
GetEnumerator, for the explicit generic implementation. That still needs
more investigation.

* AggressiveInline in ThrowNullRefIfNotInitialized isn't needed to inline GetEnumerator

Cleanup unneeded codes regarding `unsafe` (#1188)

* Remove incorrect comments

* Remove unnecessary `unsafe` keywords

Replace GetDirectoryNameOfFileAbove with GetPathOfFileAbove where possible (#1175)

Remove XML dumper (#1184)

only set next segment if Sequence spans multiple segments (#276)

CoreFX Renames (#872)

* CoreFX Renames

* Update ReadMe.SharedCode.md

* CoreFX renames to libraries

* Update ReadMe.SharedCode.md

Additions to formatting.

Add up for grabs links to coreFX and coreCLR (#881)

* Add up for grabs links to coreFX and coreCLR

Add note under Up for grabs about issues in CoreCLR and CoreFX, until Issues are fully consolidated.
and to submit pull requests to the Runtime Repo.
Contributes to #304

* Update CONTRIBUTING.md

Co-authored-by: Dan Moseley <danmose@microsoft.com>

Organize some markdowns and fix links  (#1159)

* Move files into hierarchy

* fix more links

* Fix botr and features links

* Remove bad apostrophe

* spelling

* Apply suggestions from code review

Co-Authored-By: Youssef Victor <31348972+Youssef1313@users.noreply.github.com>
* Update docs/coding-guidelines/package-projects.md

Co-Authored-By: Jan Kotas <jkotas@microsoft.com>
Co-authored-by: Youssef Victor <31348972+Youssef1313@users.noreply.github.com>
Co-authored-by: Jan Kotas <jkotas@microsoft.com>

[doc] Add requirements for cross building (#1169)

Checked for clang building under Ubuntu 18.04 x64.

fix typo in Hashtable.cs (#1151)

Fix .cmd scripts help arguments (#1131)

Flow archType to build script in global live build (#1133)

Change bIsFree check in DacValidateMethodTable (#1086)

Some commands of SOS validate a method table in DacValidateMethodTable.
In the function, it checks whether a method table is FreeObjectMethodTable or not with GetClass() value.
However, GetClass() should not be NULL. (There is an assert in GetClass())
In this patch, it compares pMT address with g_pFreeObjectMethodTable address only.

more appropriate default rootfs-dir path (#1126)

crossbuilding of at least armel:tizen and arm64:xenial works fine
without specifying the ROOTFS_DIR environment variable (with default
path) only if changes of this PR are applied. (building coreclr using
src/coreclr/build.sh script on linux)

chmod +x on scripts building rootfs for Tizen (#1125)

get the execute rights to Tizen scripts building the rootfs

SequenceEqual picks the correct overload (#1113)

Consolidate .netcoreapp.cs files because System.Diagnostics.* projects is no longer cross-compiled (#1101)

Revert "Fix .cmd scripts help arguments (#1043)" (#1110)

This reverts commit 0a29c61468f60f815119e52e477d0154351a4abf.

Publish the official build to blob storage (#1092)

* Publish installers and checksums

- Use the arcade publishing process to publish the installers and checksums
- Remove old blob publishing
- Calculate the product version using an always-suffixed version
- Change the blob path that we should upload to to be Runtime/<version> instead of assets/core-setup/.. to line up with the desired paths.

* Updates for dotnet/runtime

Use InstallerTasksAssemblyPath.

Fail fast on RuntimeNupkgFile item duplicates.

Find some more symbol packages. AllConfigurations symbol nupkgs weren't detected due to different artifact arrangement. There may be more missing, however I did this to do unblock some quick local validation so I'm including it.

Co-authored-by: Matt Mitchell <mmitche@microsoft.com>

Allow R2RReader to reuse existing MetadataReader/PEReader objects (#1077)

Publish CoreCLR packages (#1090)

Add missing _BuildConfig. Pass configuration with Arcade style, at the end for win arg parsing.

Fix build-packages.sh binlog location.

Remove CoreCLR nupkg output redirection.

Initial global build PR pipeline (#986)

Fixes: #951

* Fix OSX build dependencies and container use per Santiago's advice

* Comment the purpose of the new pipeline per Jared's PR feedback

* Prune unused parameters of global-build-job

* Fix variable propagation

Merge pull request #784 from layomia/nested_dicts

Improve serialization support for nested dictionaries

Make test libraries configuration agnostic (#378)

* Make test libraries configuration agnostic

* PR Feedback, keep Debug define for tests

* More PR Feedback

* Last pr feedback

Restore "Start generating LCL_FLDs in LocalAddressVisitor #737" (#991)

* Revert "Revert "Start generating LCL_FLDs in LocalAddressVisitor (#737)" (#973)"

This reverts commit 9bf91f8d178e09bb81fb1749a82d588cbe8028cf.

* Add a test that produces an unused, top-level indir

* If an indir has no user then it's not a def

* Fix test return code

Merge pull request #993 from pgovind/RegEx

Parse RegEx such as "[^]"

Fix .cmd scripts help arguments (#1043)

* Fix .cmd scripts help arguments

* Support -? arg in .cmd scripts

Prevent method from being inlined to get correctly handled in try-catch blocks (#1066)

Fix dependencies.props import (#833)

* Fix dependencies.props import

* Fix props by sanitizing variables

* Don't use ProjectDir variable in script

Merge remote-tracking branch 'upstream/master' into RegEx

Add binder resolution events (#604)

Handle GenericParameterDesc's in validation (#1063)

Add Pri1 Crossgen2 pipeline (#1033)

Use goto to exit in build.cmd so that error code propagates out of cmd (#990)

* Use goto to exit in build.cmd so that error code propagates out of cmd

* Add comment

* Fix windows format job

Consolidate .netcoreapp.cs test files in System.Runtime.* (#999)

* Consolidate .netcoreapp.cs files because System.Runtime.* projects is no longer cross-compiled

Add official signed build pipeline (#1016)

* Add official signed build

* Don't attempt test signing during PR validation

* Add build trigger include/exclude paths

* Remove unnecessary conditions (config, success)

* Add issue link for internal tool restore

* Add issue link for test signing

* Fix consolidation TODO formatting, links for all

Add a test to throw when Ecmascript is not specified