Fix JIT32_GCENCODER-related build errors

Remove lazy initialization of Task.CompletedTask (#8846)

Remove lazy initialization of Task.CompletedTask

[Local GC] Move Software Write Watch's write barrier updates to GCToEEInterface::StompWriteBarrier (#8605)

* Move Software Write Watch's write barrier updates to use the new
GCToEEInterface::StompWriteBarrier to stomp the EE's write barrier.

* Address code review feedback, move SetCardsAfterBulkCopy to EE side of the interface

Merge pull request #8835 from pgavlin/RemoveAsgdLclVar

Remove the `asgdLclVar` analysis from liveness.

Merge pull request #8808 from rahku/cleanup

Hide types implemented in corefx

Update CLI to 1.0.0-preview2-1-003182, BuildTools to 1.0.27-prerelease-01205-03 (#8829)

* Update CLI to 1.0.0-preview2-1-003182, BuildTools to 1.0.27-prerelease-01205-03

* Add BuildVersion.targets and required properties to test directory's dir.props

Simplify lclNum fetching and a few assertions.

Hide types implemented in corefx

Merge pull request #8832 from RussKeldorph/corefxtests

Fix archival and test result steps for corefx tests

Swap wget and curl to fix init-tools capability check (#8831)

Remove the `asgdLclVar` analysis from liveness.

This analysis is not necessary for correctness and does not appear to
have any impact on the generated code: in particualr, internal assembly
diffs show no changes with this analysis disabled.

For example, consider the following statement:

    `(asg (lclVar V0) (add (lclVar V0) (icon 4)))`

Under the `asgdLclVar` analysis, the visit of the lclVar node that is an
operand to the add would mark the lclVar node that is the target of the
assignment with `GTF_VAR_USEDEF` and exit without adding V0 to the
current set of used lclVars. Instead, the visit of the lclVar node that
is the target of the assignment would add V0 to both the set of used
lclVar nodes and the set of defined lclVar nodes.

With this analysis disabled, the visit of the lclVar node that is an
operand to the add will add V0 to the set of used lclVar nodes and the
visit of the lclVar node that is the target of the assignment will add
V0 to the set of defined lclVar nodes. In both cases, the result is the
same.

Merge pull request #8825 from adityamandaleeka/fix_jit_dasm

Fix check for JIT dasm

Delete unused InteropServices types (#8765)

Fix archival and test result steps for corefx tests

These archival and testResult groovy calls were using absolute paths, which is
apparently incorrect.  Change then to use relative paths and make things
consistent between Windodws and Ubuntu runs.

Update CoreClr, CoreFx to beta-24906-01, beta-24906-02, respectively (#8826)

Use POSIX "command" instead of non-standard "which" (#8823)

GNU which might not be installed on a minimal GNU/Linux installation.

Refs #6994 and #7025.

Merge pull request #8820 from pgavlin/RemoveVarTmp

Remove `BasicBlock::bbVarTmp`.

Fix check for JIT dasm.

Merge pull request #8781 from hqueue/arm/rootfs/tizen

Add Tizen rootfs for armel

Merge pull request #8818 from SedarG/sedarg

Revert "Adding Microsoft.NETCore.Native package (#8791)"

Remove `BasicBlock::bbVarTmp`.

This field was never used.

Merge pull request #8813 from dotnet-bot/master-UpdateDependencies

Update CoreClr, CoreFx to beta-24904-03, beta-24905-02, respectively (master)

Revert "Adding Microsoft.NETCore.Native package (#8791)"

This reverts commit eea7b4691c0a489435814e48a257a9d047475006.

fix link

Fix https://github.com/dotnet/coreclr/issues/8816

Update CoreClr, CoreFx to beta-24904-03, beta-24905-02, respectively

Use x86 crossbuild for cross-architecture component on x64-linux (#8722)

* Use x86 crossbuild for cross-architecture component on x64-linux

Fix build.sh and build system generator to use x86 cross-build system on x64-linux
- x86-host/ARM32-target cross-architecture component build on x64 linux
- use x86 root file system and cross buildtool

* Modify build.sh

- define new function: build_cross_arch_component
- fix typo

* Additional fix: build.sh

add usage description
- environment variable: CAC_ROOTFS_DIR
- argument: crosscomponent

Update cmake file for armel

Enable build using Tizen rootfs

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

Add a script to fetch tizen packages

Fetch packages for build RootFS by XML information in download.tizen.org

Signed-off-by: pius.lee <pius.lee@samsung.com>

Add Tizen rootfs for armel

- This rootfs can be used for CoreCLR, CoreFX and Core-Setup.

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

Use armel instead of arm-softfp (#8771)

* Use armel instead of arm-softfp for CoreCLR

- Update scripts for rootfs, build, build system generation and CI
- Update managed build project
- Rename rootfs directory
- Fix bug in rootfs script related to lldb package
  lldb package only usable lldb-3.5-dev for jessie/arm-softfp
  We cannot get lldb-3.6-dev or lldb-3.8-dev from jessie/arm-softfp repository

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>
Singed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>
* Update ARM32 CI script

We don't need a patch anymore, because we fixed cmake file.

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

Fix unsigned_abs function on Unix (#8805)

The unsigned_abs function in jit.h is implemented using labs function.
However labs has 32 bit argument / result, while the unsigned_abs expects
it to be 64 bit.
The correct function to use for FEATURE_PAL is llabs. But since we now have
abs overload for __int64 in pal.h, we can just remove the #ifdef for FEATURE_PAL
and use the same implementation for both Unix and Windows.

Merge pull request #8809 from pgavlin/FixJitStressFailure

Add a missing ref count bump in DecomposeLclFld.

Merge pull request #8711 from sivarv/simdAbsOpt

Accelerate Abs intrinsic even in cases where there is no direct instruction support available.

Fix VSO 359733. (#8803)

* Fix VSO 359733.

This bug was an assertion when extracting side effects that failed when
checking that the presence or absence of value numbers on the head of
the side effect list matched the presence or absence of value numbers on
the node being added to the list. This condition does not hold when
remorphing, as remorphing may create nodes without value numbers. In
this case, the new comma node will have no value numbers for
conservative correctness. The failing assertion was weakened to allow
the presence/absence of value numbers on the head of the list and the
node being added to differ during remorphing.

Merge pull request #8804 from adiaaida/devdiv_362706

Fix DevDiv 362706

Add a missing ref count bump in DecomposeLclFld.

DecomposeLclFld transforms a single lclFld node into two lclFld nodes
that reference the same lclVar, but was missing a corresponding ref
count bump. This was causing a failure in JIT/Methodical/fp/exgen/10w5d_cs_do
under JITStress=1 and JITStress=2.

Fix NULL string printing in printf (#8800)

The functions implementing various printf flavors in PAL were not handling
correctly the case when a string pointer passed to them for %S and %s formats
was NULL. Instead of printing `(null)`, they were crashing.

This change fixes the problem and also adds PAL tests to verify it is working
properly.

Fix DevDiv 362706

The issue here was that when we morphed a mod into a sub mul div, we would
ultimately decrease the ref counts to the numerator of the original mod
without performing a corresponding increase. The fix is to increase the
ref count when morphing into the sub mul div.

Accelerate Abs intrinsic even in cases where there is no direct instruction support available.

Merge pull request #8802 from adiaaida/fixCorefxGroovy

Do not supply fx_root to run-corefx-tests

Do not validate fx_root in run-corefx-tests

Users are able to supply an fx_root that does not yet exist and
run-corefx-tests.py will create that directory. However,
run-corefx-tests.py checks to confirm that fx_root already exists when
we validate. This causes the script to exit with an error when fx_root
does not yet exist. This change modifies run-corefx-tests.py so that we
do not validate the fx_root path.

This change also fixes the path to WORKSPACE for ubuntu corefx testing
in netci.groovy.

Adding Microsoft.NETCore.Native package (#8791)

* "Adding Microsoft.NETCore.Native package"

* Adding generic linux configuration to Microsoft.NETCore.Native package

* Fixing the OSX pkgproj of Microsoft.NETCore.Native to include S.Globalization.Native.dylib and not .so

Merge pull request #8719 from jkotas/native-buffer

Use ArrayPool for PathHelper

Merge pull request #8796 from dotnet-bot/master-UpdateDependencies

Update CoreClr, CoreFx to beta-24904-02, beta-24904-01, respectively (master)

Path normalization performance improvements

- Use const variant of path separators in the Path implementation
- Add fast paths for common cases to hot parsing routines
- Avoid redundant caches for MaxPath on Unix

Delete unused file

Use ArrayPool for PathHelper

Replace special unmanaged memory pool used by PathHelper by shared ArrayPool. Custom memory pools are not good for global system performance, and the particular implementation of the unmanaged memory pool was also problematic for CoreRT port because of its dependencies.

[x86/Linux] Implement ResolveWorkerChainLookupAsmStub (#8780)

Replace ResolveWorkerChainLookupAsmStub with actual implementation
- code is copied from Win32 version
- replaced some parts to compile in linux

[x86/Linux] Implement SinglecastDelegateInvokeStub (#8799)

Implement SinglecastDelegateInvokeStub for x86/Linux
- Code copied from that of AMD64 and some translation
- Fixes LargeObjectAlloc.exe unit test case and maybe others

Update CoreClr, CoreFx to beta-24904-02, beta-24904-01, respectively

Remove un-necessary ITuple constraint (#8793)

Redo some argument name corrections (#8727)

* TraceLogging
* SignatureHelper
* RuntimeReflectionExtensions
* RuntimeHandles - Public
* SemaphoreSlim - Public
* Directory - Public
* ModuleBuilder - Public
* EncoderFallback - Public - Test Failures
* StringBuilder - Public - Test Failures

Merge pull request #8757 from JosephTremoulet/FixHeapLiveIn

Fix heap live-in calculation

Merge pull request #8789 from pgavlin/VSO359737

Fix DevDiv bug 359737.

Add a comment per code review.

Merge pull request #8788 from JohnChen0/apiset

Fix a typo that caused AVX2 flag to be unset

Merge pull request #8751 from mellinoe/corelibraries-unix

Add CORE_LIBRARIES to the TPA list if it's set in unixcoreruncommon

Remove unnecessary `canEndWithoutSeparator` parameter (#8786)

TimeZoneInfo.StringSerializer can be a struct (#8784)

Adjust ref counts during EH normalization (#8713)

EH normalization can leave incorrect ref counts. This caused asserts in a few
cases with the forthcoming finally cloning, as these EH related blocks were
exposed to flow optimizations.

Fix DevDiv bug 359737.

This bug was an assertion during CSE that ensures that all CSE
candidates have value numbers. This assertion tripped because an earlier
CSE in the same tree as the faulty candidate re-morphed modulus by a
constant into a divide and subtract; the resulting tree was left without
value numbers. This change disables the problematic transform when
remorphing as part of CSE.

Fix a typo that caused AVX2 flag to be unset

Fix issue #8736, a regression caused by PR #8624. A typo in
the code causes the AVX2 flag to be always unset.

TimeZoneInfo.StringSerializer: Avoid some unnecessary allocations (#8783)

Instead of using multiple calls to `string.Replace` to escape reserved
chars before appending to a `StringBuilder`, do the replacing while
appending the chars directly to the `StringBuilder`.

Also, append numbers/dates directly instead of running it through the
escape method, as these Invariant-formatted strings will not contain any
chars that need to be escaped.

Only add CORE_LIBRARIES to TPA list if it is different from clrFilesAbsolutePath

This avoids placing duplicate entries on the TPA list.

Merge pull request #8785 from dotnet-bot/master-UpdateDependencies

Update CoreClr, CoreFx to beta-24903-03, beta-24903-02, respectively (master)

Update CoreClr, CoreFx to beta-24903-03, beta-24903-02, respectively

Merge pull request #8777 from dotnet-bot/master-UpdateDependencies

Update CoreClr, CoreFx to beta-24903-02, beta-24903-01, respectively (master)

ArrayPool fixes (#8774)

* Use Array.Empty<T>() to make the code identical to CoreRT copy
The internal EmptyArray<T> does not exist in CoreRT.
* Remove unnecessary helper type for lazy initialization
All generic types are beforefieldinit by design.
* Use | for clarity

Update CoreClr, CoreFx to beta-24903-02, beta-24903-01, respectively

TimeZoneInfo: Reduce intermediate allocations in GetTimeZoneIds on Unix (#8769)

Avoid intermediate underlying array allocations as items are added to
the resulting List<string> by specifying the capacity. Also, change
the signature of the private method to return List<string> instead of
IEnumerable<string> to avoid the enumerator allocation when enumerating
the ids.

Update CoreClr, CoreFx to beta-24902-03, beta-24902-02, respectively (#8745)

[Linux/ARM32] duplicated definition in crosscomp.h (#8729)

Some structs in crosscomp.h are already defined in pal.h or palrt.h
Fix to avoid duplicated definition

[x86/Linux] Fix UMThunkStub calling convension (#8714)

This fixes reverse P/Invoke segment faults by handling cdecl from UM to
fastcall for Managed code, which corrects many CoreFX unit test cases including CharMinValue.exe.
First and second parameter(s) are set to ECX, EDX and rest to the stack.

Delete unused Diagnostics files (#8754)

Delete unused Threading types/members (#8766)

Delete unused SymbolStore files (#8767)

Delete unused IO files (#8753)

Merge pull request #8755 from JosephTremoulet/DeadParam

Remove dead parameter in value-numbering

Fix heap live-in calculation

Update the code in liveness that sets the `fgCurHeapUse` flag (which needs
to identify whether the current block has an upwards-exposed use of the
heap) to set this unconditionally for opcodes that use the heap.
Previously, this code was avoiding setting the flag if the block has a
prior def of the heap, but in the absence of a guarantee to the contrary,
we must assume that the heap use may not alias the heap def, and so still
be upwards-exposed and cause the heap to be live-in to the block.

Also remove the OptRepeat workaround for the lack of this, in
`ResetOptAnnotations`.

Fixes #7846.

Remove dead parameter in value-numbering

Parameter `newVNsForPhis` of method `fgValueNumberBasicBlocks` is unused;
its presence and comment are confusing/misleading.  What actually happens
is that `fgValueNumberBlock` itself checks whether the value numbers for
incoming Phi args' SSA defs are defined yet or not.

Add CORE_LIBRARIES to the TPA list if it's set in unixcoreruncommon

Merge pull request #8743 from sandreenko/an-additional-shift-test

an additional shift test. fix #7216 .

Typo in documentation (#8740)

Update CoreClr, CoreFx to beta-24828-03, beta-24828-03, respectively (#8741)

an additional shift test. fix #7216 .

Test long and ulong shifts by 0, 32 and 64 constants.

Merge pull request #8738 from dotnet-bot/master-UpdateDependencies

Update CoreClr to beta-24828-02 (master)

Fix order of raising AssemblyLoadContext.Unloading and AppDomain.ProcessExit (#8737)

The AssemblyLoadContext.Unloading event must be raised before the AppDomain.ProcessExit event.
AssemblyLoadContext.OnUnloading now subscribes to AppContext.Unloading.
Then AppContext.OnAppContextUnloading and AppContext.OnProcessExit, in that order, subscribe to AppDomain.ProcessExit.
Part of fix for dotnet/corefx#14566

Fix Bitposition funtion (#8739)

Fix Bitposition(value) function implementation to check platform, not target.
- cross-architecture component build error for ARM32/linux

Update CoreClr to beta-24828-02

Merge pull request #8734 from justinvp/tzi_files

TimeZoneInfo: Move nested types and Win32/Unix implementations to separate files

Update issue guide (#8728)

* Revert "Update IssuesFeedbackEngagement.md"

This reverts commit 124754f2f5382f2781db8f132e3c89d21b56600b.

* Update wording

Remove ifdefs from TimeZoneInfo

Move Win32 and Unix -specific implementation details to their own files.

Merge pull request #8712 from dotnet-bot/master-UpdateDependencies

Update CoreClr, CoreFx to beta-24827-03, beta-24827-01, respectively (master)

Update CoreClr, CoreFx to beta-24827-03, beta-24827-01, respectively

Move nested TimeZoneInfo types to their own files

Remove files related to legacy build system (#8723)

TimeZoneInfo Cleanup (#8720)

Update TimeZoneInfo to use CoreFX coding style and other minor cleanup.

Improve ArrayPool core affinity (#8716)

- Move the ExecutionId to non-generic type so that it can be shared by all ArrayPool instances.
- Add logic to refresh it periodically. It avoids pathological cases where the OS scheduler ends up reassigns the preferred cores and multiple active threads start competing over the same buckets.
- Removed flushing of ExecutionId on LockedStack lock contention since it was not very effective

Remove all usage of vsnprintf (#8709)

This change removes all usages of vsnprintf and modifies runtime to not to use
vsnprintf or _vsnprintf

I've also fixed two issues in PAL TRACE function string format parameters that
caused crashes when I was trying to run all PAL tests with PAL tracing enabled.

[Linux] Fix cross-architecture component build configuration for ARM32-target (#8687)

* fix CMakeLists.txt for palrt

* Modify CMakeLists.txt for cross-architecture component configuaration

Add coreclrpal project to fix a dependency issue when CMake configuration

* Allow only x64-host/arm-target for linux

allow x64-host/arm-target only, because pal is not stable for x86 now

* Update build.sh

delete NumProc=1 (inserted for debugging)

* Change x64-host to x86-host for arm-target

JIT correctness problem,
and fix CMake configuration for x86 in pal

* Fix type

* Update CMakeLists.txt

* fix style

* Set x86 when arm-target/x64-host

* fix typo