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

Fix EventSource to stop ignoring EventCommand.SendManifest (#848)

* Dont ignore EventCommand.SendManifest command in EventSource

* Add ETW manifest generation test

* Mark the ETW ManifestGeneration tests as privileged only

* Define IsProcessElevatedAndNotWindowsNanoServer

* Change undefined file path

Delete strong name cruft (#1006)

The amount of strong name support that CoreCLR needs is very small (really just a method to convert public key to public key token). It is not worth it to build a separate .lib for just this single method. Fold the strong name APIs into metadata and change the API to return HRESULT.

JsonNull return false when comparing against null (#842)

* JsonNull return false when comparing against null

I'ved modified JsonNulls comparison methods to return false when
compared against null, the matching test have also been updated
in change

Fix #820

* Updated xml comments to reflect my changes

I have updated the xml comments to reflect the changes
that was made in JsonNull and removed left over comment
in JsonNullTests

* Applied suggested change

Applied the suggested change to correct xml documentation

Co-Authored-By: Ahson Khan <ahkha@microsoft.com>
* Applied Suggested change

Applied the suggested change to correct xml documentation

Co-Authored-By: Ahson Khan <ahkha@microsoft.com>

Update dependencies from https://github.com/dotnet/arcade build 20191217.1 (#1013)

- Microsoft.DotNet.XUnitExtensions - 5.0.0-beta.19617.1
- Microsoft.DotNet.XUnitConsoleRunner - 2.5.1-beta.19617.1
- Microsoft.DotNet.VersionTools.Tasks - 5.0.0-beta.19617.1
- Microsoft.DotNet.ApiCompat - 5.0.0-beta.19617.1
- Microsoft.DotNet.Arcade.Sdk - 5.0.0-beta.19617.1
- Microsoft.DotNet.Build.Tasks.Configuration - 5.0.0-beta.19617.1
- Microsoft.DotNet.Build.Tasks.Feed - 5.0.0-beta.19617.1
- Microsoft.DotNet.Build.Tasks.Packaging - 5.0.0-beta.19617.1
- Microsoft.DotNet.Build.Tasks.SharedFramework.Sdk - 5.0.0-beta.19617.1
- Microsoft.DotNet.CodeAnalysis - 5.0.0-beta.19617.1
- Microsoft.DotNet.GenAPI - 5.0.0-beta.19617.1
- Microsoft.DotNet.GenFacades - 5.0.0-beta.19617.1
- Microsoft.DotNet.Helix.Sdk - 5.0.0-beta.19617.1
- Microsoft.DotNet.RemoteExecutor - 5.0.0-beta.19617.1

Fix to array validation in type loadability checks (#1004)

simplify SslStream.AuthenticateAs*Async() (#453)

* port from corefx

* add TestHelper.cs

* capture handshake exception

* feedback from review

* cleanup more apm and renegotiation

* fix unit test

* feedback from review

* feedback from review

* feedback from review

* fix test after merge

* feedback from review

* add back ProcessAuthentication as main entry point

Fix named cgroup handling in docker (#980)

While named cgroups work fine outside of docker container, they weren't
working when created and used inside of a docker container. The problem
was caused by the fact that the hierarchy root extracted from
/proc/self/mountinfo and the cgroup path extracted from /proc/self/cgroup
are not equal for named groups. They just share the same prefix.
The cgroups handling code was not epxecting this case and ended up building
the final cgroup path incorrectly (including the common part of the path).
This change fixes it by checking for matching prefix of the paths instead
of comparing the whole paths.

Switch over the remaining CoreCLR pipelines to live-live mode (#813)

I have also found out that my recent change to switch over the JIT
stress pipelines was incomplete - it was building the live libraries
but not actually using it due to a typo in the parameter name. I am
fixing that as part of this change.

Thanks

Tomas

Add tracing for default probing for managed assemblies (#740)

Re-enable symstore targets for System.Private.CoreLib (#1003)

FileSystemAclExtensions missing `this` nullchecks (#1001)

Set vtable offset as contained (#722)

- Can remove a machine instruction which adds vtable offset

add test for nested ssl authentication (#873)

* add test for nested ssl authentication

* feedback from review

Move System.Globalization.Native under libraries (#875)

- Delete Microsoft.NETCore.Native package
- Unify CMake files with the rest of libraries/native
- Fix pedantic warnings
- Comment out assert (https://github.com/dotnet/runtime/issues/946)
- Delete dlclose during shutdown. It is not safe to unload any libraries that the runtime depends on.
- Install ICU for libraries build in the CI

Fixes https://github.com/dotnet/coreclr/issues/22391

sq

Parse [^] when ecmascript is set

Merge pull request #984 from cshung/dev/andrewau/r2rdump-perf-fix

Improving performance of R2RDump

All related headers are marked as NonTrailing (#903)

Currently, some of the known headers disallowed in the message trailer are marked with HttpHeaderType.NonTrailing flag. However, there are others identified as disallowed by the related RFC missing this flag.
Fixes dotnet/corefx#42474

SslStress: add randomized delays (#977)

Remove CoreFX.issues.rsp (#985)

Improving performance of R2RDump

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

This reverts commit 4fa163852af62908948e8108a03f4044f96d2bc1.

[master] Update dependencies from Microsoft/vstest dotnet/arcade (#874)

* Update dependencies from https://github.com/microsoft/vstest build 20191216-02

- Microsoft.NET.Test.Sdk - 16.5.0-preview-20191216-02

* Update dependencies from https://github.com/dotnet/arcade build 20191216.4

- Microsoft.DotNet.XUnitExtensions - 5.0.0-beta.19616.4
- Microsoft.DotNet.XUnitConsoleRunner - 2.5.1-beta.19616.4
- Microsoft.DotNet.VersionTools.Tasks - 5.0.0-beta.19616.4
- Microsoft.DotNet.ApiCompat - 5.0.0-beta.19616.4
- Microsoft.DotNet.Arcade.Sdk - 5.0.0-beta.19616.4
- Microsoft.DotNet.Build.Tasks.Configuration - 5.0.0-beta.19616.4
- Microsoft.DotNet.Build.Tasks.Feed - 5.0.0-beta.19616.4
- Microsoft.DotNet.Build.Tasks.Packaging - 5.0.0-beta.19616.4
- Microsoft.DotNet.Build.Tasks.SharedFramework.Sdk - 5.0.0-beta.19616.4
- Microsoft.DotNet.CodeAnalysis - 5.0.0-beta.19616.4
- Microsoft.DotNet.GenAPI - 5.0.0-beta.19616.4
- Microsoft.DotNet.GenFacades - 5.0.0-beta.19616.4
- Microsoft.DotNet.Helix.Sdk - 5.0.0-beta.19616.4
- Microsoft.DotNet.RemoteExecutor - 5.0.0-beta.19616.4

remote *_i.h blacklist from .gitignore (#975)

Update NCL stress infra for live-live builds (#896)

R2RDump changes to allow separation of presentation from parsing (#267)

  Annotate System.Collections.Immutable for nullable (#367)

* Annotate System.Collections.Immutable for nullable

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

Unit test to repro

JIT: detect address of field as an invariant inlining arg (#845)

Update the check for arg invariance to include addresses of fields in local
structs. This allows the inliner to directly substitute more arguments into
the body of the inlinee.

Resolves dotnet/coreclr#27630.

Start generating LCL_FLDs in LocalAddressVisitor (#737)

* Start generating LCL_FLDs in LocalAddressVisitor

* Fix broken vararg offset computation

Since it's subtracted the local field offset also need to be subtracted so it actually ends up being added as needed.

* Introduce array/index temps for LCL_FLDs

Previously fgMorphArrayIndex was always creating temporaries for array/index FIELD trees. Since now LCL_FLDs are created before fgMorphArrayIndex this preserves the existing behavior.

* CR feedback

* Fix incorrect field sequence type check

* Fix GetTail formatting

Remove usage of TargetGroup where possible (#899)

In non-infra related paths we don't need to check on TargetGroup anymore
and can instead use the well-known TargetFramework property.

Merge pull request #257 from tannergooding/float-format

Fixing Dragon4 to take unbiased rounding into account.

Use default setting for UseSharedCompilation (#900)

UseSharedCompilation now defaults to true in the Microsoft.Net.Compilers.Toolset package. Previously we used the deprecated Microsoft.Net.Compilers package in which the property defaults to false.

Add SkipLocalsInitAttribute (#454)

This attribute supports a new compiler feature that allows a user to
specify that they don't want to emit the CLR `.locals init` portion of
the header in methods. This can sometimes produce a performance
improvement but, in some cases, can reveal uninitialized memory to the
application.

Fixes https://github.com/dotnet/corefx/issues/29026

Adjusting `GetCurrentProcessorId` caching to different environments. (#467)

* Adjusting `GetCurrentProcessorId` caching rate based on relative performance of `GetCurrentProcessorNumber` and `TheadStatic`

Adding a regression test for corefx#42576

Fixing Dragon4 to take unbiased rounding into account.

Use wildcard to resolve S.P.CoreLib from CoreCLRArtifacts (#878)

* Use wildcard to resolve S.P.CoreLib from CoreCLRArtifacts

* Probe 2 paths instead

Remove PackageReference to cijobs tool used to add it to Core_Ro… (#746)

* Remove PackageReference to cijobs tool.

* Remove jit-dasm and jit-analyze package references per feedback.

* Remove test_runtime.csproj and move the copying of the UCRT files to Core_Root to test_dependencies.csproj.

Fix incorrect merge of my framework crossgenning fix with Jeremy's local live-live change (#879)

I found out that in two places my fix for crossgenning the framework
incorrectly merged with Jeremy's live-live change. In particular,
as I was moving around the steps for patching CORE_ROOT with explicit
CoreFX while Jeremy was deleting them, my changes inadvertently
resurrected two such deleted sections.

Thanks

Tomas

Removing "netcoreapp"  and "netfx" TargetGroup (#457)

* netcoreapp -> netcoreapp5.0 in .sln files

* netcoreapp; -> netcoreapp5.0;  , netcoreapp- to netcoreapp5.0-

* netcoreapp- -> netcoreapp5.0 in .csproj

* 'netcoreapp -> 'netcoreapp5.0 in .csproj

* done with libraries sub folder

* missed semicolon

* minor changes

* netcoreapp -> netcoreapp5.0 in some extra files

* vNext target variable in configurations.props

* netcoreapp => netcoreapp5/0 in some other .props file

* netcoreapp -> vnext in .csproj

* making the netcorepap framework switch work

* fixing sendToHelix and linux build

* TargetFrameworkVNext -> netcoreappCurrent

* case correct netcoreappcurrent

* fixing send to helix zip path

* netfx -> net472 in .sln

* netfx; -> net472; in .props

* netfx- -> net472- in .props

* netfx - -> net472- in .csproj

* 'netfx-> 'net472 in .csproj

* netfx -> net472 manual change

* fixing ci build for netfx

* Not trying to change the global property

* Fixing build failure due to master changes in net https tests

* adding documentaiton and fixing filtering

* netcoreapp -> netcore5.0 in py file

* changing the path to use netcoreapp5.0 version

* feedback and override corefx path corrected in installers

* remaing netcoreapp5.0 changes done

* netcoreapp-> netcoreapp5.0 after live builds

* net472 -> netframeworkcurrent

Fix crossgenning of framework libraries on Windows (#747)

1) Fix crossgenning of framework libraries on Windows;

2) Enable crossgenning S.P.C with CG2, remove obsolete CoreCLR logic;

3) Fix framework build with legacy Crossgen due to a typo in my change;

4) The ordering of steps in build-test was incorrect - we need to
patch CORE_ROOT with live-live libraries build before Crossgenning
the framework therein, otherwise weird things happen (and the result
is incorrect in any case).

5) Removed misplaced >nul 2>nul - we agreed with JanV that it's
better to not throw diagnostic information away.

6) Applied JanV's suggestion for improvement - using a tighter filter
for reference assemblies - Microsoft.*.dll / System.*.dll /
mscorlib.dll instead of just *.dll that ended up picking tons of
garbage like clrjit.dll and the various Win32 API contracts, both
in the build-test script and in CLRTest.Crossgen.targets.

Thanks

Tomas

Fix JitEECallTimingInfo bit rot (#868)

Added new JIT/EE interface functions. Moved some around so
the order of functions is exactly the same order as in the
corinfo.h header file.

This functionality can be enabled by:
1. Changing the definition of `MEASURE_CLRAPI_CALLS` in jit.h
from `0` to `1`.
2. Building Release on x86 or x64 (arm32 and arm64 are currently unsupported)
3. Set environment variables:
```
set COMPlus_JitTimeLogFile=time.txt
set COMPlus_JitEECallTimingInfo=1
```
4. Run a program.

The `time.txt` file will contain a JIT phase time line item "CLR API calls"
for the sum total of all JIT-EE function call cost, plus a per-API breakdown
for each called API with number of calls, total/max/average time, and percentage
of total ABI call cost.

Ignore method name when doing `mcs -removeDup` (#602)

Thus, methods with different names but otherwise are identical will
match and only one will be retained.

This improves both Checked and Release, and fixes the Release
build removeDup regression introduced in #548.

Fix enterprise testing pipeline due to live-live changes (#843)

Revised script to build the libraries subset due to changes with CoreCLR and live-live build.

Add note about test generation depth

Clarify Lowering in ryujit-overview (#859)

tiny comment typo. (#846)