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.

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

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.

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.

Move TailcallVerifyWithPrefix test to "baseline x86" exclusion section

This test fails for both legacy x86 JIT as well as RyuJIT. Make that more
clear by moving the exclusion to the "x86 baseline failures" section.

Merge pull request #6970 from BruceForstall/EnableStringArrTest

Re-enable stringarr test

Merge pull request #6953 from Maoni0/alloc_bytes

added GC.GetAllocatedBytesForCurrentThread API

Revises UMThunkStub unwindable in ARM/Linux (#6788)

In ARM/Linux, UMThunkStub currently pushes r0-r3 and r12 without .pad,
which results in #6787.

This commit revises the prolog and epilog of UMThunkStub to fix 6787.
(In addition, personality routine is addes as ARM64/AMD64 already does.)

Monitor.TryEnter asm timeout != 0 before spin (#6951)

* JIT_MonTryEnter_Slow timeout != 0 before spin
* JIT_MonTryEnter_InlineGetThread timeout != 0 before spin

added GC.GetAllocatedBytesForCurrentThread API

Merge pull request #6972 from smile21prc/coreclr-perf

Change dir to make sure the right project.json is used.

Change dir to make sure the right project.json is used.

Merge pull request #6971 from smile21prc/coreclr-perf

Add debugging code to find where Microsoft.DotNet.xunit.performance.r…

Add debugging code to find where Microsoft.DotNet.xunit.performance.runner.cli.dll is deployed.

Re-enable stringarr test

I can't get this to fail locally despite thousands of runs.

I fixed the test to detect failure when checking all indices, not
just when checking the last index of the first rank.

Fixes #4844.

Merge pull request #6969 from smile21prc/coreclr-perf

Add steps to install xunit performance packages

Add steps to install xunit performance packages

Merge pull request #6968 from smile21prc/coreclr-perf

Add debugging code.

Add debugging code.

Merge pull request #6967 from smile21prc/coreclr-perf

Fix wrong dir, and add debugging code.

Fix wrong dir, and add debugging code.

Merge pull request #6966 from smile21prc/coreclr-perf

Fixed wrong dir.

Fixed wrong dir.

Merge pull request #6964 from BruceForstall/Finish6940

Re-enable test DevDiv_255263

Enable return address hijacking for arm64.
And also fixes a bug in HijackFrame::GCScanRoots

Merge pull request #6963 from smile21prc/coreclr-perf

Rename the script and add debugging steps.

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

Merge changes from TFS

Merge pull request #6913 from adiaaida/formattingJob

Add Formatting job

Re-enable test DevDiv_255263

I forgot to re-enable this test when fixing #6940.

1. Rename the script to the shell type.
2. Added a few lines for debugging.

Fix file descriptor leak on PAL_IsDebuggerPresent (#6958)

Merge pull request #6946 from BruceForstall/Fix3597

Change RyuJIT/x86 GC encoder to handle LSRA flexibility

Cleanup/optimize some String.Compare overloads, part 2 (#6891)

* Change signature of CompareOrdinalEx and move arg validation to the call sites
* Change thrown param name from count -> length
* Rename nativeCompareOrdinalEx -> CompareOrdinalHelper

Merge pull request #6949 from BruceForstall/Fix6940

RyuJIT x86: Fix initblk with unrolled loop and constant fill

Merge pull request #6947 from BruceForstall/Fix6097

Fix #6097

Merge pull request #6948 from BruceForstall/Fix4992

Fix #4992

RyuJIT x86: Fix initblk with unrolled loop and constant fill

The computed fill value was only using the low 8 bytes of the
xmm register; we need it to fill the entire 16 byte register.

Fixes #6940

Fix #4992

These tests no longer fail, so re-enable them.

Fix #6097

The tests no longer fail (were previously fixed, possibly by LIR work).
Re-enable the related tests.