Start using CODE_OWNERS.TXT (#5027)

Enable Checksum csc param for mscorlib (#4996)

Enable Checksum csc param for mscorlib

Merge pull request #4954 from kvochko/sos_tests

Added tests for libsosplugin

Merge pull request #5023 from wtgodbe/pipeline

Add dev workflow scripts for build pipeline

Add dev workflow scripts for build pipeline

Merge pull request #4874 from gkhanna79/SPC_S1

Add support for System.Private.CoreLib to be core library

Merge pull request #4999 from sejongoh/avoid_ti_method_check

Add typeInfo::GetMehtod2 to avoid checking GetType == TI_METHOD.

Implementing the Unix calling convention for the generic dictionary access stubs in R2R (#4998)

Add typeInfo::GetMehtod2 to avoid checking GetType == TI_METHOD.

If FEATURE_CORECLR is enabled, the pointer type is not known to be a method pointer type when GetMethod is called in the importer.

Merge pull request #4789 from sejongoh/always_download_coredistools

Always download coredistools package

Merge pull request #5008 from yizhang82/dllimportpath-fix2

Enable DefaultDllImportSearchPathsAttribute in CoreCLR Windows. Host …

Merge pull request #5002 from CarolEidt/Long32Doc

Document about supporting long on 32-bit architectures.

Optimize Array.GetLength, GetUpperBound, GetLowerBound (#4993)

... if the argument is 0. As the rank will always be greater, this is safe.

Update StringObject comment in object.h (#5011)

* m_ArrayLength is no longer present (I did a search of the source and couldn't find it), so this comment is misleading.
* Also removing mis-leading comment about high two bits of field

Move notion of restricted physical memory behind the GCToOSInterface (#5007)

Merge pull request #4997 from RussKeldorph/fix3379

Rename some IL tests whose names conflicted with C# tests

Enable DefaultDllImportSearchPathsAttribute in CoreCLR Windows. Host NATIVE_DLL_SEARCH_DIRECTORIES has higher priority.

Merge pull request #5003 from jkotas/corert-gc

Update GC from CoreRT

Initial change to support System.Private.CoreLib.dll as Core Library.

Merge pull request #4975 from kspawa/ldiv

Replace ldiv with normal division for PAL

Always download coredistools package

runtest.sh/cmd currently downloads the coredistools package if COMPlus_GCStress variable is set by TestEnv or test-env options. This is a bit complicated and this doesn't work with a new option for GC stress testing in build/buildtest.cmd. This changeset moves downloading the coredistools package to buildtest.cmd where it takes gcstress option and downloads other packages.

Linux build doesn't have buildtest.sh, so package donwloading is still in
runtest.sh

Update GC from CoreRT

https://github.com/dotnet/corert/tree/master/src/Native/gc 854b9ea7accfbd24423a10382c1a5a2bd6753450

Rename some IL tests whose names conflicted with C# tests

For catchrettoinnertry, we appeared to have some redundant test projects.
I removed the redundant ones and cleaned up the remaining.

Fixes #3379

Document about supporting long on 32-bit architectures.

Merge pull request #4962 from CarolEidt/FixIssue3964

Fix CircleInConvex to check result within 1 ULP

Merge pull request #4620 from dagood/remove-infer-runtimes

Remove --infer-runtimes restore argument

Merge pull request #4985 from adiaaida/2442

Update Boxing\xlang projs with correct project includes

Merge pull request #4982 from LLITCHEV/Issue4853

Remove invalid tests.

Update sin projs with correct project includes

All of the xlang sin ilprojs were missing includes to the correct
references. This change updates the proj files with those
ProjectReferences. Additionally, this change updates the includes in the
il tests to match the name of sinlib assemblies.

Fixes #2442.

Merge pull request #4965 from adiaaida/disableRegPair

Disable RegPair for x86 Longs

Fix CircleInConvex to check result within 1 ULP

This test was checking for the result to be within epsilon of the
expected result, but this is not correct unless the algorithm is
evaluating all interim results to a larger precision.
It would also be more reasonable for the test to compute the expected
result (e.g. using an alternate algorithm), and it would also be nice
to have a method to compute ULP for a given value, but those are beyond
the scope of this fix.
Re-enable the test except for RyuJIT/x86.

Fix #3964

Merge pull request #4905 from JeremyKuhne/JeremyKuhne-buildupdate

Update windows instructions

Merge pull request #4978 from ellismg/add-fedora-23

Add Fedora 23 runs

Merge remote-tracking branch 'origin/master' into Issue4853

Conflicts:
tests/testsFailingOutsideWindows.txt

Merge pull request #4974 from RussKeldorph/xfails

Test cleanup (mostly non-Windows failures)

Disable RegPair for x86 Longs

We are not using regpairs for longs on x86 ryujit, so we need to turn off
CPU_LONG_USES_REGPAIR. This is the first step to getting var = call for
GT_CALLs with long return types on x86.

Add runtimes to each project.json in tests/src missing a runtimes section.

Remove --infer-runtimes restore arg: runtimes specified in project.json files.

Added a README with instructions on running and writing tests.

Merge remote-tracking branch 'origin/master' into Issue4853

Conflicts:
tests/issues.targets
tests/src/JIT/Methodical/cctor/misc/tail.il
tests/src/JIT/Methodical/cctor/misc/threads3.il
tests/src/JIT/Methodical/cctor/simple/precise3.il
tests/x86_legacy_backend_issues.targets

Merge pull request #4969 from gkhanna79/RelaxALC

Enable overriding assemblies from TPA

Remove invalid tests.

See comments at issue #4853 for the why these tests are invalid.

Other tests in the same directories are covering the valid cases, so
removing these.

Fixes #4853

Merge pull request #4979 from dotnet-bot/from-tfs

Merge changes from TFS

Enable overriding assemblies from TPA

Fix arm and x86 build breaks

[tfs-changeset: 1604843]

Add Fedora 23 runs

Remove unneeded INLINE_NDIRECT and INLINE_PINVOKE #ifdefs (#4977)

Merge pull request #4976 from dotnet-bot/from-tfs

Merge changes from TFS

Merge pull request #4967 from BruceForstall/Fix3235

Change referenced common.dll assembly to lower-case

Fix build breaks in arm64 and legacy internal amd64 builds

[tfs-changeset: 1604829]

Use different value for win32resource for System.Private.CoreLib. (#4964)

When building both mscorlib and System.Private.CoreLib we should use a different name to avoid conflicting on this file.

Replace ldiv with normal division for PAL

Test cleanup (mostly non-Windows failures)

Clean up the hodgepodge of failures in #2450.  Many of them are simply bad
exclusions for tests that don't exist either because they were deleted or
never actually ported.  The rest were fixed more or less as recommended by
@LLITCHEV.

Fixes #2450

Merge pull request #4972 from RussKeldorph/fix2235

Re-enable arrres.il test

Change referenced common.dll assembly to lower-case

Also, change ILPart.dll to lower-case.

Fixes a bunch of tests that were failing on Ubuntu. Re-enable those.

Fixes #3235.

Add support for reverse PInvoke callouts to the JIT (#4952)

Merge pull request #4927 from ramarag/ModifyBuildOnly

Removing Redundant  Projects that are BuildOnly:

Re-enable arrres.il test

arrres.il is a quesionable test case because it relies on finalizers being
run even though the JIT is allowed to extend the lifetime of objects
beyond the point where finalizer effects are checked.  The test could be
fixed, but for now I'm just marking it as requiring optimization, which is
what has been done previously.  I'm deleting the "dbg" project because it
is the same as the "rel" project except that it disables optimization.

Fixes #2330

Initial implementation of generic dictionary access for R2R. (#4519)

For now, only TypeHandle dictionary entry slots are supported and encoded in a R2R version resilient format (the rest to come soon).
Support is only limited for x64 Windows platforms (rest is still TODO)

The basic idea: each dictionary access is initally a call to a DynamicHelper R2R cell that computes the dictionary signature, and patches the R2R cell address with an assembly stub that performs the dictionary lookup.

Removing Redundant Projects that are BuildOnly :
RunOnly Projects should refer to only BuildAndRun projects
BuildOnly projects are to be used only for Non-Execution cases
and custom execution drivers

Port string.PadLeft and string.PadRight from CoreRT (#4968)

Merge pull request #4970 from dotnet-bot/from-tfs

Merge changes from TFS

Fix a PCH-related issue in the jitstdout change.

[tfs-changeset: 1604771]

Merge pull request #4901 from LLITCHEV/Issue4801

Add a issues target file with tests to be excluded on jit32 x86.

Add a issues target file with thests to be excluded on jit32 x86.

Merge pull request #4922 from adityamandaleeka/remove_getfilenamefromsymlink

Change how PAL file I/O functions treat symbolic links

Merge pull request #4958 from swgillespie/disable-gcstress

Exclude from GCStress a few tests that take an unreasonably long time to complete

Merge pull request #4942 from pgavlin/gh4832

Use a JIT-specific stdout `FILE*`.

Merge pull request #4957 from RussKeldorph/fix2235

Reenable passing tests

Merge pull request #4961 from AndyAyersMS/RandomInlineBudgetLimit

Add budget check to RandomPolicy

Merge pull request #4955 from briansull/fix-asmdiff

Fix issue with Arm64 AsmDiffs

Merge pull request #4956 from mmitche/gcstress-no-push

Run gcstress pri1r2r only a few times a week

Merge pull request #4959 from tijoytom/master

Renaming to StringToCoTaskMemUTF8

Add budget check to RandomPolicy

Prevents RandomPolicy from being overly aggressive with small methods
that can lead to excessive inlining.

Closes #4711. Undoes #4704.

Renaming the API StringToCoTaskMemUTF8 since we
need to be consistent with existing one.Also
fixing a merge conflict.

Exclude from GCStress a few tests that take an unreasonably long time to complete

Run gcstress pri1r2r only a few times a week

Currently this is set on push, but since it can take days to run this is a bad idea

Reenable passing tests

Many of the tests listed in #2235 are now passing since their dependencies
have been fixed.  b57367 is a notable exception; it's static dependency is
now satisfied, but at runtime it loads a type that is not present in .NET
Core.  I'm deleting it because I verified there are other tests that test
the same load/invoke pattern (on different types).

The rest of the tests listed in #2235 have already been deleted for other
reasons.

Fixes #2235

Fix issue with Arm64 AsmDiffs

Merge pull request #4911 from tijoytom/master

Marshaling helpers for MarshalAs LPUTF8Str.

Added tests for libsosplugin

Merge pull request #4813 from pgavlin/gh3058

Disable FEATURE_MERGE_JIT_AND_ENGINE on *nix.

Merge pull request #4921 from AndyAyersMS/InlineReplay

Inliner: introduce ReplayPolicy

Address code review feedback.

Flip the order of the PID and activation function checks in the activation
signal handler.

Chain activation signal handlers in the PAL.

The handler for the PAL's activation signal (`SIGRTMIN` on Linux) was
not properly chaining to the previous handler. This caused problems
with thread suspension on Linux in the case that multiple PALs were
loaded in a process: whichever PAL was loaded last would handle the
signal, and if that PAL was not configured to perform the appropriate
thread suspension, the runtime would hang.

Merge pull request #4941 from gkhanna79/Fix4938

Enable SHA256 to be used as hashing algorithm for compiler/assembler

Merge pull request #4945 from dotnet-bot/from-tfs

Merge changes from TFS

Stop using SYS_delete syscall for deleting files.

Add tests for file IO functions operating on symlinks.

Merge pull request #4932 from JohnChen0/r2r

Fix contract violations in Ready to Run code

Merge pull request #4933 from dotnet-bot/from-tfs

Merge changes from TFS

Inliner: introduce ReplayPolicy

The ReplayPolicy reads an external script to determine which inlines
to perform. The script is the same Xml syntax that's produced by
the inliner when JitInlineDumpXml is enabled. This format can be edited
by hand or tool to force particular inlining patterns to occur. Methods
or calls sites not mentioned in the script are considered as noinline.

There's a bunch of work still left to make this fully robust, but in
testing it works well enough for my immediate use case that I'll hold
off on further polish until it's needed. But, for future reference,
here's a laundry list:

* Need better ways to identify methods. Token and hash are not enough.
* Need better ways to identify call sites. Callee token is not enough.
* Consider preparsing or mapping the script into memory.
* Consider caching node positions in the InlineContexts.
* Make it robust for multithreading.
* Handle the prejit root case somehow.
* Possibly allow overriding of inline attributes.

Update first-class-structs.md

Update some progress and fix some formatting

Use a JIT-specific stdout `FILE*`.

Historically, the JIT has had issues with logging inside of processes
that change the output mode of stdout (e.g. crossgen). This change
replaces the previous solution (which relied on #ifdefs) by dup'ing
the stdout file and wrapping it in a new `FILE*` set to a known
output mode.

Enable SHA256 to be used as hashing algorithm for compiler/assembler

Small formatting fix

Merge pull request #4899 from swgillespie/gcsimulator-fixup

Enable the GCSimulator CI run to run on a regular basis

Marshaling helpers for MarshalAs LPUTF8Str.

Merge pull request #4915 from tarekgh/FixLowerAsciiCharChecks

Fix the optimization check to call ordinal

Fix contract violations in Ready to Run code