Merge pull request #7075 from tarekgh/DisableZh-CHS-CHT

Avoid creating zh-CHS/zh-CHT durng culture enumeration

Exclude smallFrame.sh test for ARM (#7080)

JIT/Methodical/tailcall_v4/smallFrame/smallFrame.sh contains
"tail.call, pop, ret" pattern. The pattern is not allowed in ECMA and
is only supported on x64 for compatibility. (issue #7036)

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

Correct the virtual overload of Stream.CopyTo (#7082)

Task<T> Exceptions to ThrowHelper (#7076)

Avoid creating zh-CHS/zh-CHT durng culture enumeration

zh-CHS and zh-CHT are old names that is not really supported in the standards.
when we enabled CultureInfo.GetCultures() there was some code that create cultures
for zh-CHS and zh-CHT. The fix here is disable that otherwise we'll have to enable
a good chunk of legacy code to support it while it never supported by coreclr and in
same time are not names used in standards

Make Stream.CopyTo virtual (#7070)

Merge pull request #7022 from lemmaa/prevent-build-output-contamination

[Script] Prevent build output contamination while building test overlay

Merge pull request #7064 from JosephTremoulet/MakeCse

Fix GTF_ flag collision

Merge pull request #6922 from gkhanna79/WinArm32

Enable Arm32 Test Build

Array<T> exceptions to ThrowHelper (#7059)

[Linux][LLDB][GDB/JIT] Add support for lldb 'breakpoint set' for jitted code (#6956)

* Add support for lldb 'breakpoint set' for jitted code

* Updated after review

* Enlarge buffer for line program

Move genIncRegBy() to codegenclassic.h (#7063)

We don't have to keep genIncRegBy() in codegen.h anymore,
since genIncRegBy() is defined in codegenlegacy.cpp and only used in LEGACY_BACKEND code.

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

[ARM] Remove ARM_HAZARD_AVOIDANCE (#7019)

remove `ARM_HAZARD_AVOIDANCE` as it is a rare case.

For #7003

Enable FEATURE_JIT_METHOD_PERF always (even when FEATURE_CORECLR is on). (#7028)

Fix GTF_ flag collision

Define flag `GTF_IS_IN_CSE` (which is used only to communicate in a call to
`gtNodeHasSideEffects` that the caller wants to disregard the side-effect
of running a .cctor) as `GTF_BOOLEAN` rather than `GTF_MAKE_CSE` -- as the
comment at the definition of `GTF_IS_IN_CSE` indicates, the only
requirement on its value is that it not conflict with one of the
side-effect flags, but `gtNodeHasSideEffects` does in fact check for
`GTF_MAKE_CSE`, so the current value conflicts.

Also update the code in `gtNodeHasSideEffects` that is dictated by
`GTF_IS_IN_CSE` to check just for that bit, instead of checking that the
caller passed in exactly `GTF_PERSISTENT_SIDE_EFFECTS_IN_CSE` -- no need
to entangle conceptually independent parameters.

Fixes #7042.

Add proper support for custom installed llvm tool chain. (#7050)

* Add LLVM_HOME enviroment variable support

fix helpers' flag: ULDiv, ULMod can throw Div-by-zero exception (#7052)

ArraySegment<T> exceptions to ThrowHelper (#7058)

Exclude two more tests for ARM (#7062)

According to issue #6217, following three tests fails
, because FEATURE_FASTTAILCALL is not supported in ARM.

  JIT/Directed/tailcall/tailcall/tailcall.sh
  JIT/Methodical/tailcall_v4/hijacking/hijacking.sh
  JIT/Regression/JitBlue/devdiv_902271/DevDiv_902271/DevDiv_902271.sh

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

Prevent build output contamination while building test overlay

While building test overlay, all files in `$mscorlibDir/bin` are
copyed to `$mscorlibDir` where usually `$coreclrBinDir`.
It contaminates build output. This commit prevent it by moving
copy commend to function `create_core_overlay`.

Allow mapping LCID and culture names mapping for neutral culture (#7049)

This change is to pass the flag LOCALE_ALLOW_NEUTRAL_NAMES to the OS API to allow mapping
LCID to/from culture names.

Improve ThrowHelper (#7048)

Merge pull request #7027 from CarolEidt/1stClassStructBlkOps

1st Class Struct Block Assignments

Merge pull request #7015 from ramarag/fixup_precode

Implement FixupPrecode for Arm64

Implement FixupPrecode for Arm64

More PR comments

Reduce item format allocations in AppendFormat (#4702)

Merge pull request #6976 from kvochko/prof_elt_callbacks

Enable FunctionEnter/FunctionLeave callbacks on ARM

Merge pull request #7041 from mmitche/add-help-job

Add help job

Add help job

Restore the change to gtNewBlkOpNode

LTO build: fix perf regression from strict alias violation and incorrect inline assembly (#7018)

Fix inline assembly implementation of getextcpuid() and strict aliasing violations in its callers

The implementation writes to the unsigned char result array, so
the clobber "memory" is needed.

As well its callers have strict aliasing violations from reading the
result array as a DWORD array. The fix is to memcpy the resulting
buffer onto the DWORD array.

Enabled FunctionEnter/FunctionLeave hooks on ARM (#6939)

More PR Feedback

Improve StringBuilder ctor(), ctor(int), and ToString() performance. Consistent changes in other functions. (#7029)

Merge pull request #7032 from mmitche/make-triggers-non-default

Changing new perf.groovy triggers to be non-default

Changing new perf.groovy triggers to be non-default

Address PR Feedback

Improve ToUpper/ToLower perf (#7026)

Make Lazy initialization of TextInfo::m_IsAsciiCasingSameAsInvariant thread safe (#7021)

Replace bool? with Tristate enum to make it thread safe. Also use Tristate instead of TurkishCasing enum

Merge pull request #6978 from adiaaida/fixFormat

Fix format.py

Fix format.py

In format.py, we were downloading bootstrap.sh/cmd to the current working
directory, but assuming that it was in coreclr/bootstrap, which is
incorrect. This change downloads it directly to coreclr/bootstrap, so that
we download and run to the same location.

This change also modifies where we install the newer version of dotnetcli
needed for jitutils to the coreclr/dotnetcli-jitutils directory. This
change is necessary because build.cmd->run.cmd->init-tools.cmd removes the
Tools directory, including our dotnetcli directory, so we don't want our
version of the cli to be trashed.

We also clean up our work at the end of the script by removing the
created jitutils and dotnetcli directories, and the bootstrap script.

Finally, this change adds some comments to tell the user what it's doing.

1st Class Struct Block Assignments

Change block ops to assignments, with block nodes (GT_BLK, GT_OBJ and GT_DYN_BLK) as the lhs, and with GT_STORE_* in the backend.
For this initial change, existing behavior is preserved as much as possible, with a few differences in lea generation for SIMD types.
This causes pessimization in some areas; those, as well as the additional opportunities that can be enabled after this change, have all been marked TODO-1stClassStructs.

Globalization update from CoreRT - part 2 (#7020)

- Persian calendar date formatting
- Fix IsPrefix handling of empty strings
- Formatting changes

Merge pull request #7025 from dotnet/revert-6994-replace_which_with_hash

Revert "Avoid using 'which' for verifying prereqs"

Revert "Avoid using 'which' for verifying prereqs"

Merge pull request #7024 from pgavlin/RunTestFix

Pass /p:Platform to msbuild in runtest.cmd.

Merge pull request #7014 from pgavlin/TestBuildSpeedup

Stop groveling for project.json files when building every test.

Pass /p:Platform to msbuild in runtest.cmd.

Otherwise, the tests will use the ambient setting of the Platform
variable, which might not match the architecture specified on the
command line. This results in the wrong build of the tests running
against the correct runtime, which e.g. causes failures in tests
with native components.

Merge pull request #7013 from swaroop-sridhar/PtrStruct

GcInfo: Report Struct containing one GcPointer correctly

GcInfo: Report Struct containing one GcPointer correctly

This change fixes a bug in the the GcInfo header,
where the ReturnKind is reported incorrectly for certain structs.

Structs that contain only one GCPointer or ByRef, can be returned
in a single register (Windows/Unix/OSX). The ReturnKind for these
struct-returns should be RT_Object/RT_ByRef, but they are currently
incorrectly reported as RT_Scalar. This change fixes the problem.

Fixes: dotnet/corefx#11260

Merge pull request #6807 from lemmaa/add-options-for-arm-target-test

[Linux/ARM/Script] Add some features help for arm target test

Update Unix System.Globalization implementation from CoreRT (#7004)

Stop groveling for project.json files when building every test.

This information is only needed by build.proj. Do not compute it in dir.props,
which is included by every test project.

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

Merge changes from TFS

Merge pull request #6772 from mellinoe/fix-build-packages

Specify packages.builds by default in build-packages command

Merge pull request #6552 from pgavlin/RemovePreRationalizeLiveness

Do not run local var liveness before rationalize.

Merge pull request #5884 from mikedn/x86-addsub-long

Fix add/sub decomposition

Merge pull request #7009 from BruceForstall/PrintErrorForFuncInfoFailure

Print an error, in DEBUG, if we fail to open the JitFuncInfoLogFile

Merge pull request #6943 from mellinoe/fedora-24

Add fedora24 native packages

Fix funceval of a function returning 16 byte value type. (#6997)

On Linux and OS X, structs less or equal to 16 bytes (instead of 8 bytes
on Windows) are enregistered. Only 8 bytes is being passed back to
debugger during a funceval of a property or method that returns a 16
byte value type.

To fix this, the 16 byte return value (which is only used for enregistered
structures of that size on xplat) needed to be plumbed from CallTargetWorker
through the MethodDescCallSite macros to the func eval code.

The func eval code needed to also deal with 16 byte results.

NUMBER_RETURNVALUE_SLOTS is the number of ARG_SLOTs that will contain the
maximum enregistered return value.

CordbEval:m_result is now ARG_SLOT[NUMBER_RETURNVALUE_SLOTS].

CallTargetWorker is now passed a pointer and the count of bytes to/of the
return value buffer.

Minor fix to SOS SymbolReader function.

Avoid using 'which' for verifying prereqs (#6994)

* Use hash instead of which in build.sh

* Use hash instead of which in init-tools.sh.

* Use hash instead of which in gen-buildsys-clang.sh.

Fix #6893. (#6932)

Aggregate arguments to calls (i.e. arguments that are made up
of multiple, independent values such as those to be passed in
multiple registers) are represented by GT_LIST nodes that are
not distiguishable from normal GT_LIST nodes without the extra
context of the call. GT_LIST nodes, however, are disallowed in
LIR ranges. The intersection of these two design choices causes
problems for code that operates on LIR but wants to observe
aggregate arguments as a whole (rather than piecewise). In the
case of #6893, this caused lowering to attempt to insert a
PUTARG_STK node after the GT_LIST node that represented the
aggregate argument that was being forced to the stack, which
caused an assertion in the LIR utilities to fail (the GT_LIST
node was passed as the insertion point to `LIR::Range::InsertAfter`,
which requires that the insertion point be a part of the given
range).

Conceptually, the fix is simple: the nodes that represent
aggregate arguments must be allowed to be part of LIR ranges.
The implementation, however, is complicated somewhat by the
need to distinguish the nodes used for aggregate arguments from
normal GT_LIST nodes. These changes implement a short-term
fix that distiguished these nodes by adding a node-specific flag,
`GTF_LIST_AGGREGATE`, to indicate that a GT_LIST node is in fact
an aggregate argument. In the long term, a more palatable
implementation would be the introduction of a new node and operator
type to represent aggregate arguments, but this is likely to
require broader changes to the compiler.

Merge pull request #6989 from adiaaida/updateCodingConventions

Update coding conventions

Fix the build break

[tfs-changeset: 1625162]

Do not run local var liveness before rationalize.

Neither rationalize nor lowering depend on liveness being run. The assertion mentioned
in the comment was caused by `fgExtendDbgLifetimes` inserting HIR nodes (GT_ASG nodes,
specifically) into LIR blocks.

Print an error, in DEBUG, if we fail to open the JitFuncInfoLogFile

Merge pull request #7001 from BruceForstall/CleanUpX86LegacyIssuesFile

Clean up test issues files

Clean up test issues files

Re-enable some tests that pass. Rearrange x86_legacy_backend_issues.targets
to more closely resemble issues.targets so it is easier to see the
difference.

Update coding conventions

Add a note to the jit coding conventions that jit-format changes supersede
the conventions in the doc. Add a link to jit-format's documentation.

Merge pull request #6988 from adiaaida/shiftLong

Implement DecomposeShift

Merge pull request #6917 from svick/fixing-alltestprojects

Fixed AllTestProjects.sln

Add some features help for arm target test

* Add `--build-overlay-only` option to help building test package for arm target.
  You can use this option on host after cross building, to make a
  test package that will run on target device.
* Make `setup-runtime-dependencies.sh` called on `x64` architecture only.
  Because dependent packages are not supported on other architectures now.
* Add `testsUnsupportedOnARM32.txt` file for list up test exclusions.
* minor style fix.

Update CoreClr, CoreFx, External to beta-24431-01, beta-24431-01, beta-24430-00, respectively (#6879)

Remove obsolete add/sub NYI

Fix the srcCount for an unused GT_LONG. (#6996)

GT_LONG nodes that do not have a corresponding use need to
consume their operands. There was already code in lowerxarch
that attempted to do so, but this code was no longer valid in
LIR.

Remove System.Reflection.AssemblyNameProxy and fix System.Reflection.Pointer constructor (#6987)

Update CoreClr, CoreFx, External to beta-24431-01, beta-24431-01, beta-24430-00, respectively

Implement DecomposeShift

This change adds support for GT_LSH, GT_RSH, and GT_RSZ for x86 longs.
These nodes are implemented via helpers, so in decompose, we must remove
the shift node from the tree and replace it with a call to the helper.
Additionally, these helpers require that the long operand is in EAX:EDX,
so rather than adding a GT_LONG to the call arg list like we would
normally, we split the high and low parts of the arg and mark them as
non-standard args in gtMorphArgs to force the low part to be in EAX and
the high part to be in EDX. The shift amount is already defaulted to ECX,
which is where the helper expects it to be.

Remove System.Reflection.AssemblyNameProxy and fix System.Reflection.Pointer constructor

Merge pull request #6993 from tarekgh/AddMissingSystemTextAPIs

Expose missing Global/Encoding APIs in coreclr and remove empty stubs

Merge pull request #6983 from BruceForstall/ReformatIssuesFiles

Reformat test issues files so they are more easily comparable

Fix the srcCount for an unused GT_LONG.

GT_LONG nodes that do not have a corresponding use need to
consume their operands. There was already code in lowerxarch
that attempted to do so, but this code was no longer valid in
LIR.

Port String reorganization from CoreRT (#6985)

GC update from CoreRT (#6984)

https://github.com/dotnet/corert/tree/master/src/Native/gc 97dd6d2cbb50901cc3651f41a53a4ff9a1052391

Skipped letter in a word (#6991)

Expose missing Global/Encoding APIs in coreclr and remove empty stubs

This change is to expose all missing globalization and Encoding APIs in corelib
to have the complete exposed list for such namespaces.
Also we are removing the empty methods stubs for Windows implementation
And have the actual implementation in-place

Merge pull request #6992 from dotnet/suse421-buildbadge

Add openSUSE 42.1 build badge to README.md

Add openSUSE 42.1 build badge to README.md

Merge pull request #6990 from smile21prc/coreclr-perf

Create coreclr perf jobs for Linux.

Create coreclr perf jobs for Linux.

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

Merge changes from TFS

Reformat test issues files so they are more easily comparable

This change reformats the issues files, and sorts the excluded tests
by name, so it is possible to easily compare the baseline issues.targets
file against the x86_jit32_issues.targets and x86_legacy_backend_issues.targets
files.

Delete non-existing method

[tfs-changeset: 1624957]

Merge pull request #6973 from BruceForstall/MoveTailcallTestExclusion

Move TailcallVerifyWithPrefix test to "baseline x86" exclusion section

Merge pull request #6867 from rahku/returnHijack

Enable return address hijacking for arm64

Merge pull request #6980 from smile21prc/coreclr-perf

More fixes.

More fixes.

Merge pull request #6979 from smile21prc/coreclr-perf

Fixed wrong env vars.

Fixed wrong env vars.