Initial commit of checked in build definitions for CoreCLR

Revert EncodingForwarder for the sake of the Perf (#10740)

* Revert EncodingForwarder for the sake of the Perf

* Add missing optimization

[x86/Linux] PAL test in CI build (#10688)

* [x86/Linux] PAL test in CI build

Add PAL test in CI x86/Linux build
Change docker image: including rootfs

* Fix netci.groovy

Add PAL test's result

[ARM/CI] Update image for armel Tizen CI (#10730)

Update image for armel CI to sync with Tizen rootfs

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

Merge pull request #10742 from BruceForstall/RemoveUnusedDefine

Remove unused PREDICT_REG_RER_INDIRECT_PARAM define

Merge pull request #10752 from BruceForstall/FixBBFformatting

Clean up block.h formatting

Clean up and prepare FileLoadException.cs for move to Shared Partition. (#10764)

* String => string

* Replace fields with autoprops.

* Turn Type.FullName call to Type.ToString() in ToString().

Avoids MissingMetadataExceptions on CoreRT.

Assuming metadata exists, ToString() and FullName()
still generates the same string except in the weird
case where an app subclasses FileLoadException
with a generic exception type - in which case,
the ToString() version is actually more readable.

* Get rid of useless try-catch around FusionLog calls.

FusionLog is now just a normal property and an answer
to a trivia question - it won't throw SecurityException.

* Get rid of low-value SetMessageField() middleman.

* Split off the stuff we don't want to share.

* The complementary carwash...

* Don't conditionalize FusionLog serialization.

[x86/Linux] Clean up ZapUnwindData (#10750)

[Local GC] Fix a bad merge (#10748)

replace the linear time algorithm from lclvars with the constant one. (#10401)

add new block flag DOMINATED_BY_NOT_NORMAL_ENTRY

It allows to answer the question: "Is block dominated by non normal
entry" - in O(1).

gtLdftnResolvedToken is no longer necessary, because token is saved on the stack. (#10738)

The was an error in the assert because I checked pointers that were
obvious different.
The actual values could not be different because they are created from
the same token in the same place (importer.cpp DO_LDFTN:
impMethodPointer saved resolvedToken as gtLdftnResolvedToken ,
CORINFO_RESOLVED_TOKEN* heapToken = impAllocateToken(resolvedToken);
impPushOnStack(op1, typeInfo(heapToken)); saved the token on the stack.

[x86/Linux] 16-byte aligned HelperMethodFrameRestoreState (#10720)

* [x86/Linux] 16-byte aligned HelperMethodFrameRestoreState

* Use STACK_ALIGN_PADDING and update comment

Clean up block.h formatting

Hide a few things from clang-format

Merge pull request #10724 from mikedn/small-type-xchg

Use the correct type for interlocked instructions

Merge pull request #10615 from brianrob/profiler

Cross-Platform Sample-Based Profiler

Merge pull request #10739 from wtgodbe/GroovyFedora

Add Fedora24 to CI

Remove unused PREDICT_REG_RER_INDIRECT_PARAM define

[Local GC] Pull some logic using SystemDomain::System out of the GC (#10670)

* [Local GC] Pull some logic using SystemDomain::System out of the GC

* Address code review feedback: add comment about this (per the GitHub discussion) and rename API to ForceFullGCToBeBlocking

Merge pull request #10727 from hqueue/typo/doc_ryujit

Fix class name in ryujit-overview.md

Merge pull request #10211 from sdmaclea/PR-ARM64-CROSSGEN-DISCONTIG

[Arm64] Simplify trackedStackPtrsContig logic

Add Fedora24 to CI

Fix registry name enumeration (#10711)

1. Values can be removed after we've gotten the count
2. Allocations for names were  over 32K * enum call + 32K.

[Local GC] Move Weak Reference finalization out of the GC (#10676)

* [Local GC] Move Weak Reference finalization out of the GC

* Address two issues:
  1) Use GetGCSafeMethodTable instead of GetMethodTable, so that the
     mark bit is correctly masked off the object's method table pointer,
  2) Address code review feedback by re-inserting a missed call to
     GetCanonicalMethodTable and rename the new API to EagerFinalized to
     better illustrate its broader purpose.

* Repair the GC sample

Merge pull request #10574 from JosephTremoulet/FixTestProjectPath

Fix path to CoreCLRTestLibrary from SizeConstTest

Fix ThrowExceptionFromContextInternal for x86 Linux (#10728)

The function was missing restoring of some callee saved registers
which caused crashes in Checked build and hang in Release build.

[x86/Linux] Fix build error in nuget packaging (#10691)

* [x86/Linux] Fix build error in nuget packaging

Fix build error in nuget packaging
Modify condition for excluding libcoreclrtraceptprovider.so

* Update runtime.Linux.Microsoft.NETCore.Runtime.CoreCLR.props

Fix typo

Merge pull request #10112 from pgavlin/gh10106

Only build compatjit.dll when building JIT32.

Only build compatjit.dll when building JIT32.

In order to avoid confusion, do not build and package compatjit.dll
unless we are building JIT32.

Fixes #10106.

[x86/Linux] add three functions for _X86_TARGET_ (#10378)

* [x86/Linux] add three functions for _X86_TARGET_

There is no functions for _X86_TARGET_.
  - ZapUnwindData::GetAlignment()
  - DWORD ZapUnwindData::GetSize()
  - void ZapUnwindData::Save(ZapWriter * pZapWriter)

* [x86/Linux] remove personality routine on x86/linux.

remove creating personaly routine when assemblies is generated to ni.
add a function for getting size of UnwindDataBlob on x86/linux.

Signed-off-by: ragmani <ragmani0216@gmail.com>
* [x86/Linux] correct the unclearly fixed parts.

change Unwindinfo's size from sizeof(ULONG) to sizeof(UNWIND_INFO).
change Unwindinfo's alignment from sizeof(DWORD) to sizeof(BYTE).
remove unnecessary code.

* [x86/Linux] change alignment from sizeof(BYTE) to sizeof(DWORD).

Fix class name in ryujit-overview.md

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

Use the correct type for interlocked instructions

The interlocked operation type can, at least in theory, be TYP_VOID in the case of GT_LOCKADD. The first operand type is not relevant because it's the address of the memory location and not an indir.

The only option is to use the type of the second operand but that may be a small int type so genActualType needs to be used to get the correct type.

Merge pull request #10303 from hseok-oh/fix_10300

[ARM32/RyuJIT] Fix register allocation for null check in call instruction

Merge pull request #10637 from rahku/UrlString

Remove file system/security/util/URLString.cs

Remove file system/security/util/URLString.cs

Add a job for testing via IlLink for Linux

Add jobs to run CoreCLR tests on Ubuntu via ILLINK

Fix invocation of ILLink in linux testing

Fix the input to illink -a argument to not include the extension.
The current command line caused silent wrong code gneration
because of https://github.com/mono/linker/issues/56

[x86/Linux] 16-byte aligned ResolveWorkerAsmStub (#10655)

Merge pull request #10715 from wtgodbe/alpineFix

Fix Alpine build break in pipeline

Update module not found error message (#10692)

This message can be misleading, since it says "The specified module
could not be found", but it would also report case when the module
was found, but one of the module's dependencies were not found.
Multiple people in the past were hit by that when their module was
not loaded, they were thinking that there is some problem with
the path coreclr is loading modules from or something like that and
spent non-trivial amount of time trying to figure that out.
The fact that the module could have missing dependencies is really
not obvious.

So I am updating the message to reflect that fact.

Fix Alpine build break in pipeline

CoreRT: make CORINFO_HELP_READYTORUN_DELEGATE_CTOR optimization for verifiable sequences. (#10663)

delegate ctor optimization for CoreRT.

Merge pull request #10190 from sdmaclea/PR-ARM64-BACKPORT-9500

[Arm64/Windows] Backport #9500 changes

Merge pull request #10614 from sdmaclea/PR-ARM64-GENRET-GCINFO

[Arm64] Fix GCStress hole in genReturn

Merge pull request #10609 from alpencolt/ryu-arm-locallock

[RyuJIT/ARM32] Localloc implementation in codegen

Merge pull request #10699 from adityamandaleeka/fix_env_empty_var_handling

Fix handling of empty environment variables in PAL

Merge pull request #10681 from BruceForstall/LinuxArmAltjit

Add Linux host support for arm32/arm64 altjit

Fix handling of empty environment variables in PAL.

Removed Unused Local Random Instances (#10696)

* Removed Unused Local Random Instances

Discovered two cases of a `Random` instance that weren't being used
within their respective methods and removed them.

* Retained Earlier Comment

Retained an earlier comment for consistency

Use setMachineAffinity for arm64 (#10558)

[x86/Linux] Fix crash when GC triggered inside funclets (#10188)

* [x86/Linux] Fix crash when GC triggered inside funclets

* [x86/Linux] Make main function not to report

Pass crossbuild and linux-id to CMakeLists.txt using CMake arguments (#10686)

Remove in CMakeLists.txt
1) read cross-build from environment variable
2) read linux-id from file system
Instead, pass cross-build and linux-id from gen-buildsys-clang.sh using cmake arguments

[x86/Linux] Correctly unwind esp frames (#10685)

Merge pull request #10601 from rahku/c4996

1. Remove disable of C4996

Merge pull request #10682 from BruceForstall/CleanUpJitEEDefine

Remove unused code under typo define

Merge pull request #10580 from noahfalk/fitjit_tier0_flag

Add Tier0 jit flag

Update README.md (#10641)

In this file I’ve found a “project.json” occurrence that I've changed to ".csproj".

Move '#pragma warning(disable:21000)' to correct place (#10455)

Whilst I was writing a blog post ['A Hitchhikers Guide to the CoreCLR Source Code'](http://mattwarren.org/2017/03/23/Hitchhikers-Guide-to-the-CoreCLR-Source-Code/#top-10-lists) I noticed that a `#pragma warning(disable:21000)` was mis-aligned.

I'm pretty sure that having it in the wrong place doesn't cause a problem (i.e. this doesn't actually *fix* anything), because the other methods are smaller, so I understand if you don't want to take the PR.

Improve Guid.ToString (#10669)

* Improve Guid.ToString

* revert HexToChar

Merge pull request #10678 from briansull/contract-fix

Remove the SO_TOLERANT contact from CEEInfo::resolveVirtualMethodHelper

Debug.Assert FF (#10652)

Fix 'this' pointer in struct methods (#10668)

The 'this' method argument should always be a pointer.

Remove unused code under typo define

The code is under COR_JIT_EE_VER which never existed (it used to
be COR_JIT_EE_VERSION), thus the code has never been used.

remove fedora 23 (#10667)

Merge pull request #10629 from russellhadley/EHWriteThru-design-doc

Add initial design document for EH WriteThru

Fix InternalsVisibleTo when it references an assembly with some DebuggableAttribute flags (#10664)

Fixes #3541
- Mask out the DebuggableAttribute bits from when comparing assembly spec flags for matching an InternalsVisibleTo reference to an assembly

Merge pull request #10671 from wtgodbe/Redist

Publish VC Redist DLLs as part of CoreCLR Package

Remove the SO_TOLERANT contact from CEEInfo::resolveVirtualMethodHelper
that contract should only apply to CEEInfo::resolveVirtualMethod which uses the JIT_TO_EE_TRANSITION() transition macros.

Merge pull request #10672 from pgavlin/FixExceptionReference

Fix a reference to System.Exception in a JIT test.

Add Linux host support for arm32/arm64 altjit

Now, on Linux, the x64 build will build an arm64-targeting
altjit. The x86 build will build an arm32-targeting JIT.
The altjit is named libprotononjit.so/dylib. It can be
used for debugging or for generating asm diffs using jit-diff.

Publish VC Redist DLLs as part of CoreCLR Package

Merge pull request #10326 from jashook/arm64_test_update_20_march

Update the arm64 test drop - WIP

Fix a reference to System.Exception in a JIT test.

This bad reference was casuing a failure in the x64 pri1 R2R jobs.

Basic EventPipe functionality that supports writing sample profile events to JSON.  SampleProfiler that globally suspends the runtime every 1ms to walk stacks and write them to the EventPipe.

[x86/Linux] add the process of handling when entry point is funclet. (#10602)

* [x86/Linux] add the process of handling the entry point when it is funclet in case x86.

* [x86/Linux] enable only for TARGET_X86 && FEATURE_PAL.

Signed-off-by: ragmani <ragmani0216@gmail.com>

Update the arm64 test drop

[RyuJIT/ARM32] Localloc implementation in codegen

Algorithm is almost the same as for other platforms but code
is little bit refactored to improve readability.

Added test to cover almost all possible branches of codegen.
Since arrays allocation of int type is needed multiplication and overflow
checks new tests cover only byte arrays.

[RyuJIT/ARM32] GS Cookie check implementation.

Implementation of GS Cookie check for RyuJIT/ARM32 almost the same as
for ARM64. So this code was moved to codegencommon with.
Also added optimization to use tmp register for emitting helper call.

Cut down FormatterServices (#10653)

Fix Double.ToString performance on Linux and OSX (#10572)

* Fix Double.ToString performance on Linux and OSX

* some feedback fixes

* Remove the implementation depending on ecvt_r

Bring back kApplicationException (#10650)

Remove securitystate (#10649)

Move identical Globalization files to shared partition

Move identical files to shared partition

Move identical Interop files to shared partition

[Local GC] Move some EE-specific finalize-on-unload logic out of the GC (#10598)

* Move some EE-specific logic to a callback on GCToEEInterface

* Fix the sample

* Consistent style for pointers

* Code review feedback: Move app domain index check into EE callback and make ShouldFinalizeObjectForUnload always return true (due to CoreCLR not having app domains)

* Code review feedback: Fix a comment and add a TODO for bringing these changes to desktop

[Local GC] Remove static fields from GC interface (#10566)

* Remove max_generation from GC interface

* [Local GC] Clean up the GC interface by removing all static fields
from it:
  1) gcHeapType is replaced by g_heap_type (EE side) and g_gc_heap_type
     (GC side), each of which is set on initialization. g_heap_type is
     read by the DAC to determine what kind of heap is being used.
  2) maxGeneration is not necessary due to the GC DAC changes.

* Rebase against master

* Comments and cleanup

* Use a heap allocation instead of alloca

* Code review feedback: remove a redundant cast and allocate generation count table once, on first use

* Address code review feedback: cache some calls to GetMaxGeneration

Merge pull request #10584 from adityamandaleeka/handle_table_local_gc_init

Change how the VM calls ObjectFromHandle.

Add .gitmirror file

Merge pull request #10627 from briansull/profile-data

Added DisableInlining flag to the ProfileData.

Add initial design document for EH WriteThru

Document describes an initial approch to enregistering local variables
that live across EH flow.  Today the JIT forces all local vars that are live
across EH flow that kills all registers to live on the stack.  The transformation outlined ensures consistency on the stack across the EH kills but allows the enregistration with in either the normal flow of control or with in any EH handlers.

Refactor dotnet download code in init-tools.cmd (#10527)

* Refactor dotnet download code in init-tools.cmd
This addresses the improvements proposed in issue #10526.
Includes the init-tools.cmd script refactor and a new script called
dotnet-download.ps1 which includes the extracted code and logic

* Code review feedback changes

* Fix spacing

Jit: fix a few issues with single def local tracking (#10633)

Fixes for three issues that came up in desktop testing.

1. Don't track class types for locals when in import only mode,
since the jit may be looking at an uninstantiated generic method.
In these cases the jit's use of TYP_REF to represent CORINFO_TYPE_VAR
confuses the tracking code. In import only mode the jit is not going
to use the information, so there is no need to track it at all.

    Import only mode is tied to verification and CoreCLR runs in full
trust mode, so no test was added.

2. Allow `lvaUpdateClass` to be called more than once but assert
that the second and subsequent calls provide the exact same update
as the first call.

    This can happen for single def ref class locals whose definition
block is reimported. Reimportation is triggered by some tolerable
non-ref type mismatches at block joins and so ref type information
should be consistent for each importation attempt.

    Add a couple of IL test cases translated from destkop MC++ tests
that will trigger this kind of reimportation.

3. Bail out of devirtualization (for desktop only) if the base class
has precise initialization semantics, since desktop seems to trigger
class initialization for ref type callvirts (contrary to ECMA355 I.8.9.5).
See #4853 for some discussion. Since this is desktop only behavior
no test was added.

Merge pull request #10638 from AndyAyersMS/EmptyTryUpdateFinallyNestLevel

Jit: decrement finally nesting level when removing empty trys

Delete moved files

Merge pull request dotnet/corert#3175 from jkotas/nmirror-merge

Merge nmirror to master

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

Change how VM calls ObjectFromHandle.

Add way to get table for handle and ADIndex for handletable.

Add Tier0/1 jit flags

These flags provides a hook to change the JIT policy in the future and diverge tier0/tier1 compilation from min_opt/speed_opt respectively.

delete GetMethod2 (#10626)

* delete GetMethod2

* delete unused code

Jit: decrement finally nesting level when removing empty trys

For non-funclet EH models, the GT_END_LFIN statement tracks the
nesting level of the associated finally. When removing a try-finally
with an empty try we need to decrement this level for any try-finallys
nested within the associated finally.

Added a test case with several levels of empty try and nesting.

Closes #10621.