Address #10799 comments

Correct assert w.r.t meaning of lvRegister

Per recomendation of Pat Gavlin in #9898

Fixes #9898

Handle unsupported varargs tests on Unix gracefully. (#10794)

Some of mcc_i* tests caused segmentation faults on Unix. This commit make these tests exit by throwing a System.EntryPointNotFoundException exception instead of causing a segmentation fault.

Fix #9530

Bring back DefaultDllImportSearchPathsAttribute (#10786)

Add reporting swap file size to GlobalMemoryStatusEx on Unix (#10700)

The swap file size reporting is added using the Linux, OSX and FreeBSD
means. Also a new PAL test was added to exercise the function.

Delete duplicate files and fix build breaks

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

Merge nmirror to master

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

Call custom parameterless constructor on structs through Activator (#10778)

Fixes #6843
- Disabled caching struct types that have custom parameterless constructors in `ActivatorCache`
- Removed some things relevant to security, which don't apply to CoreCLR

remove fedora 23 (#10725)

* remove fedora 23

* Fedora rename

NS2.0 Prepare FileNotFoundException for move to shared partition. (#10775)

* String->string - get rid of the big comment block.

* Fields into autoprops.

* FullName => ToString() to avoid MME problems on N.

* Remove SecurityException guard around FusionLog

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

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

Merge nmirror to master

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

Merge pull request #10784 from briansull/ibc-shutdown-fix

Fix shutdown A/V issue when IBC logging with the Music Store app

Devirtualizable default encoding (#10783)

don't switch to COOP GC when logging CCW ref count (#10756)

Fix shutdown A/V issue when IBC logging with the Music Store app

This app continues to run managed code on other threads after the Main thread exits and enters shutdown.
We now block IBC logging during the time that we are writing out the IBC data.

Update ILLink build scripts

Update ILLINK build commands to reflect current CoreBuild of ILLinker.
Also remove the line that sets DoNotEmbedDescriptors,
since it is no longer necessary.

Merge pull request #10768 from mskvortsov/ryujit-arm32-lea

[RyuJIT/ARM32] Update genLeaInstruction

Add views for the jitstress and illink folders (#10773)

Make Encoding.Default return UTF8 without BOM emission (#10747)

* Make Encoding.Default return UTF8 without BOM emission

* Remove SL reference in the code

* Remove the static cache

Merge pull request #10697 from sdmaclea/PR-FIX-RUNTEST

Do not use declare -A

Merge pull request #9086 from mikedn/simd-cast

Add Vector.op_Explicit tests

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>

RyuJIT/ARM32: update GT_LEA codegen

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)

Add Vector.op_Explicit tests

[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)

Do not use declare -A

This broke runtest.sh on OSX 10.12

[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.