Merge pull request #7835 from DrewScoggins/coreclr-perf

Adding CoreCLR Perf support for Jenkins

Merge pull request #7872 from jkotas/revert-tuple-hash

Revert changes in tuple hashing algorithm

Add comment about F# dependency on hashing algorithm

Revert "Better Tuple hashing, avoid boxing in Equals/GetHashCode (#6767)"

This reverts commit 361e6eb85c74b57691955d97dda801d725cd2a59.

Conflicts:
src/mscorlib/mscorlib.shared.sources.props

C++ conformance. (building with /permissive-) (#7855)

These issues were found when building with the /permissive- flag in the
latest version of MSVC. No tests were added/modified because this does not
change any behavior.
There are a few types of language conformance issues fixed in this:

1) Strict string conversion (this is also covered by the /Zc:strictStrings
flag)

The 'const' is not implicitly dropped by string literals, which means the
following is not allowed:
char str = "const string literal"; //error: cannot convert a 'const char'
to a 'char*'
This fix to to make str 'const char*'. (This can have a domino effect
depending on where str is used)

2) Fully qualified inline declarations members inside class

struct A {
void A::f() { } // Error: illegal qualified name in member declaration,
remove redundant 'A::' to fix
};

3) MSVC by default will allows name lookup in a dependent base. This is
disabled by /permissive-

template <class T> struct B {
  void f();
};

template <class T> struct D
 : public B<T> //B is a dependent base because its type depends on the type of T in D<T>.
{
    //One possible fix is to uncomment the following line.  If this
    //were a type we should have 'using typename'...
    //using B<T>::f;

    void g() {
       f(); //Error: identifier not found, one possible fix is change it to 'this->f();'
    }
};

void h()
{
   D<int> d;
   d.g();
}

4) Warning 4800 has been removed in version 19.1 (1910) of the compiler.

For backwards compatability, surround the usage of 4800.
This is not related to C++ conformance.
 #if _MSC_VER <= 1900
 // 'BOOL' forcing value to bool 'true' or 'false'
 #pragma warning(disable: 4800)
 #endif

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

Merge changes from TFS

Add missing std::move to one exception throw (#7865)

We had the std::move missing in one of the exception throws. For some reason,
the current clang didn't complain about it even though the copy constructor was
deleted.

Update broken link to RIDs (#7850)

Use memset instead of ZeroMemory (#7851)

`CORINFO_HELP_READYTORUN_STATIC_BASE` and
`CORINFO_HELP_READYTORUN_GENERIC_STATIC_BASE` both use a mostly/fully
zeroed out `resolvedToken`, the common practice is to zero out with
`memset` in this codebase.

Fix exception.StackTrace race condition.  Issue #12467. (#7856)

Fix conversion from CorJitFlag2 enum to JitFlag. (#7857)

fix typo

rollback CS1635584

[tfs-changeset: 1635595]

Expose System.Security.Principal.PrincipalPolicy (required for AppDomain apis)

[tfs-changeset: 1635584]

Merge pull request #7837 from BruceForstall/JitFlags4

Introduce new CORJIT_FLAGS type

Add verification of System.Globalization.Native.so symbols (#7848)

This change adds check of undefined dependencies of the System.Globalization.Native.so
to the build.sh. The recent change that made the ICU APIs resolving driven by the
shared library itself creates a potential for someone adding usage of an ICU API that
is not defined in the icushim.h.
With this change, such problem will be caught during the build and cause it to fail.

Introduce new CORJIT_FLAGS type

The "JIT flags" currently passed between the EE and the JIT have traditionally
been bit flags in a 32-bit word. Recently, a second 32-bit word was added to
accommodate additional flags, but that set of flags is definitely "2nd class":
they are not universally passed, and require using a separate set of bit
definitions, and comparing those bits against the proper, 2nd word.

This change replaces all uses of bare DWORD or 'unsigned int' types
representing flags with CORJIT_FLAGS, which is now an opaque type. All
flag names were renamed from CORJIT_FLG_* to CORJIT_FLAG_* to ensure all
cases were changed to use the new names, which are also scoped within the
CORJIT_FLAGS type itself.

Another motivation to do this, besides cleaner code, is to allow enabling the
SSE/AVX flags for x86. For x86, we had fewer bits available in the "first
word", so would have to either put them in the "second word", which, as
stated, was very much 2nd class and not plumbed through many usages, or
we could move other bits to the "second word", with the same issues. Neither
was a good option.

RyuJIT compiles with both COR_JIT_EE_VERSION > 460 and <= 460. I introduced
a JitFlags adapter class in jitee.h to handle both JIT flag types. All JIT
code uses this JitFlags type, which operates identically to the new
CORJIT_FLAGS type.

In addition to introducing the new CORJIT_FLAGS type, the SSE/AVX flags are
enabled for x86.

The JIT-EE interface GUID is changed, as this is a breaking change.

Merge pull request #7803 from AlexGhiondea/Fix_SB

Make sure that when the StringBuilder.Append method is called we firs…

Implement initThisClass for generic types in ReadyToRun (#7842)

Only implementing the RyuJIT portion for CoreRT purposes (CoreRT side at dotnet/corert#2085).
I can file an issue to add support for this in CoreCLR if there's interest. On CoreCLR this is currently a disqualification from ReadyToRun and causes fallback to JITting.

Merge pull request #7839 from rahku/_appdomain

expose events from AppDomain via AssemblyLoadContext

expose events from AppDomain via AssemblyLoadContext

Fix PDB creation for Ready-to-Run image of facade assemblies

[tfs-changeset: 1635538]

Properly spell class name in comments (#7821)

Fixed gcinfoencoder build errors. (#7826)

MEASURE_INFO switch now depends on _DEBUG switch. Ensure NumUntracked,
UntrackedSlotSize, NumUntrackedSize and EncPreservedSlots fields handled correctly in
GcInfoSize::operator+=() and GCInfoSize::Log().

fix #6008

Add the missing ICU APIs to the ICU shim (#7840)

there was a change done by the PR #7773 which removed the dependency on ICU version. and there is another PR #7811 which used more ICU APIs not listed in the shim.
the change here is to add these APIs to the shim

Use getLocationOfThisType instead of embedGenericHandle (#7813)

getLocationOfThisType does way less work and gives the same result (the part we care about).

[ARM/CI] Change the name of corefx project (#7820)

Merge pull request #7787 from sivarv/cmpOpt

Optimize 'test' instruction in case of op1 == 0 or op1 != 0 where op1 is known to set Zero flag.

Merge pull request #7828 from MichalStrehovsky/readyToRunGenericStaticBaseValueNum

Fix value numbering of ReadyToRunGenericStaticBase

Merge pull request #7746 from AndyAyersMS/LessGCLayout

Jit: use class attributes check to reduce GC layout asks

Move JIT perf tests to new infrastructure

Moved over the running of the JIT perf tests to use the same logic as the
perflab tests.  Had to make a few changes to what we save and where since
we reuse the sandbox folder.

Update performance tests

Add inner iterations to all performance tests in the perflab directory.
This is because we were measuring a time that was to small, and the
results were not useful.  This should also reduce the number of high level
iterations we run as well.

Move benchmarks to use netstandard1.4 instead of netcoreapp1.0 to fix desktop reflection issue

Make sure that when the StringBuilder.Append method is called we first check that we can insert as many characters as were requested before we actually start writting them into the StringBuilder.

This will prevent the case when we realize that there is not enough space in the StringBuilder (and we cannot grow it as much as it is requested) after we already wrote stuff in it.

Allow Ready-to-Run image to be loaded multiple times (#7809)

Address issue #6695 by porting the fix from desktop CLR.

Fix value numbering of ReadyToRunGenericStaticBase

Fixes dotnet/corert#2080.

Add Misc types to InteropServices

Add the following types to InteropServices:
- AutomationProxyAttribute
- ComAliasNameAttribute
- ComCompatibleVersionAttribute
- ComConversionLossAttribute
- ComRegisterFunctionAttribute
- ComUnregisterFunctionAttribute
- ICustomFactory
- ICustomMarshaler
- ImportedFromTypeLibAttribute
- LCIDConversionAttribute
- ManagedToNativeComInteropStubAttribute
- PrimaryInteropAssemblyAttribute
- ProgIdAttribute
- TypeLibImportClassAttribute
- TypeLibVersionAttribute

Optimize 'test' instruction for op1 == 0 and op1 != 0 where op1 is known to set flags.

Update CoreClr, CoreFx to beta-24625-03, beta-24625-02, respectively (master) (#7804)

* Update CoreClr, CoreFx to beta-24625-03, beta-24625-02, respectively

* Upgrade S.R.Serialization.Primitives to prerelease

Merge pull request #7808 from pgavlin/VSO280103

Clear side-effect flags on local reads in rationalize.

Merge pull request #7823 from adiaaida/checkformatlocation

Fix format.py to look for .bat on Windows

Fix format.py to look for .bat on Windows

https://github.com/dotnet/jitutils/pull/56 changed the file extension for
the script files on Windows to be .bat, so the format.py script needs to
be updated.

Enable CultureInfo netstandard 1.7 APIs on Linux (#7811)

* Enable CultureInfo netstandard 1.7 APIs on Linux

* fix spaces

* Use Array.Empty<CultureInfo> instead of CultureInfo[0]

Remove dependency of System.Globalization.Native.so on specific ICU version (#7773)

Remove dependency of System.Globalization.Native.so on specific ICU version

Merge pull request #7816 from ramarag/ThreadingOverlapped

Adding missing members of threading.overlapped

Merge pull request #7806 from sandreenko/fix-VSO-278365

RyuJIT/x86: fix VSO 278365

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

Merge changes from TFS

Adding missing members of threading.overlapped

Fix build break caused by git commit for AppDomain::GetAssemblies

[tfs-changeset: 1635147]

Merge pull request #7802 from pgavlin/VSO279829

Do not promote long-typed args in methods that use JMP.

fix VSO 278365

don't convert (int) ((double)Round(double))  to  (int)Round(double) on
RyuJit/x86

Merge pull request #7751 from ramarag/MemFailPoint

Making MemoryFailPoint functional on xplat

Add CoreCLR perf test support to Jenkins

Made changes to the perf.groovy to add support for running the CoreCLR
performance tests in the Perflab directory.  Also made changes to the
runner script for CoreCLR to allow it to run more than one type of test.

Merge pull request #7800 from BruceForstall/ChangeJITGuid

Change JIT-EE interface GUID

Making MemoryFailPoint functional on xplat
Expose GlobalMemoryStatusEx on Xplat

Clear side-effect flags on local reads in rationalize.

These operations are always side-effect free.

Merge pull request #7786 from rahku/_appdomain

Add implementation of AppDomain.GetAssemblies

Add implementation of AppDomain.GetAssemblies

Do not promote long-typed args in methods that use JMP.

On x86, methods that use JMP must ensure that all incoming arguments are
located in the appropriate stack slots/registers. To ensure that this is
true for long-typed arguments, do not promote these arguments in such
methods.

Merge pull request #7799 from adiaaida/fixRotateFlagCheck

Fix flag check when morphing into rotate

Merge pull request #7798 from AndyAyersMS/FixObservationImpact

Inliner: fix RARE_GC_STRUCT observation impact

Change JIT-EE interface GUID

This should have been changed with #7672. However, changing it now
will help prevent unexpected bad interactions with SuperPMI and other
tools that expect the old interface.

Fix flag check when morphing into rotate

When we morphed a tree into a rotate, we assumed that the parent's flags
would be the same as its operands' flags, when in actuality, the parent's
flags should be a superset of its operands' flags. This change fixes the
flag check.

Fixes VSO 278374.

Check index before access (#7795)

Add more lldbplugin/libsosplugin tests (#5606)

* Add more libsosplugin tests, update framework

+ Introduce test.assert* functions in testutils.py
+ Print test results as a table with failed assertions highlighted
+ PEP8!

* Add more lldbplugin/libsosplugin tests

Inliner: fix RARE_GC_STRUCT observation impact

Closes #7792.

Since this observation isn't always fatal, change its impact to INFORMATION.
Also, refresh the assertion checking around fatal observations so that the
EnhancedLegacyPolicy (which is the current default) will catch this kind of
error going forward.

Do not materialize CAS PseudoCustomAttribute (#7794)

Do not materialize CAS permissionsets as custom attributes in reflection. We are just providing the CAS types in corefx to make things compile, but nothing else.

Fix https://github.com/dotnet/corefx/issues/12759

Add few Interop Marshal APIs (#7742)

Add the following interop Marshal APIs:
M:System.Runtime.InteropServices.Marshal.GenerateGuidForType(System.Type)
M:System.Runtime.InteropServices.Marshal.GenerateProgIdForType(System.Type)
M:System.Runtime.InteropServices.Marshal.GetComObjectData(System.Object,System.Object)
M:System.Runtime.InteropServices.Marshal.GetHINSTANCE(System.Reflection.Module)
M:System.Runtime.InteropServices.Marshal.GetIDispatchForObject(System.Object)
M:System.Runtime.InteropServices.Marshal.GetTypedObjectForIUnknown(System.IntPtr,System.Type)
M:System.Runtime.InteropServices.Marshal.PtrToStringAuto(System.IntPtr)
M:System.Runtime.InteropServices.Marshal.SetComObjectData(System.Object,System.Object,System.Object)
M:System.Runtime.InteropServices.Marshal.StringToCoTaskMemAuto(System.String)
M:System.Runtime.InteropServices.Marshal.StringToHGlobalAuto(System.String)

Fix typos in windows build instructions (#7789)

Fix performance problem introduced in previous change. (#7783)

Merge pull request #7782 from sandreenko/fix-tfs-build

fix tfs build crash

fix tfs build crash

ifdef EE version statements were forgotten in two files.
Braket was on the wrong line.

Merge pull request #7778 from ianhays/binaryreadwrite_buffers

Add some extra checks to BinaryReader/Writer buffers

Add back call to BitArray::ClearAll (#7779)

Merge pull request #7706 from mjsabby/lcgjitprofcallbacks

Add LCG JIT Compilation Profiler Callbacks

Merge pull request #7776 from sivarv/lsraStatFix

Fix the condition to dump a basic block stats.

PR feedback for BinaryReader/Writer buffers

Add some extra checks to BinaryReader/Writer buffers

Merge pull request #7775 from ianhays/cfg_arm64

Add CFG flag to ARM64 build

Merge pull request #7772 from jkotas/gcinfo-cleanup

GCInfoDecoder cleanup

Fix the condition to dump a basic block stats.

Add CFG flag to ARM64 build

Merge pull request #7720 from Dmitry-Me/fixBrokenComparison

Fix broken range check

Replace LPVOID with void*

Simplify GCINFODECODER_CONTRACT

Delete DISABLE_EH_VECTORS

Delete _TARGET_SET_ macro

Fixed month calculating issue of calendars(#7666) (#7769)

Fixed the issue introduced by misunderstanding the precedence of '+' and
'>>'.

fix #7666

Fix Binder based PInvoke resolution for dynamic assembly (#7770)

Fixing the StackOverflowException error message. (#4786)

Fixed typos and format issues on clr-code-guide doc(#7767) (#7768)

There're several typos and format issues. Just fixed them.

Fix issue #7767

Port GC changes from CoreRT (#7764)

https://github.com/dotnet/corert/pull/2064: Add back limit for maximum object size
https://github.com/dotnet/corert/pull/2061: Fix retail build break

Fix broken #elif (#7756)

Add Profiling API callbacks for R2R methods (#7732)

Merge pull request #7749 from sivarv/VectorIntDot

Recognize Vector<int>.Dot on AVX as a JIT intrinsic.

Improve ArraySegment's override of GetHashCode (#4654)

* Improve ArraySegment's override of GetHashCode

* Use new implementation suggested by @VSadov

* Respond to PR feedback

* Remove the workaround for object.GetHashCode

* Respond to HashHelpers changes

* Fix compile errors.

Merge pull request #7739 from pgavlin/DisableGCStressIncompatibleTests

Mark tests GCStress-incompatible for x86.

Merge pull request #7760 from pgavlin/VSO279832

Properly initialize the varargs base lclVar.

Merge pull request #7759 from pgavlin/VSO278367

Initialize long return type descs in `gtNewCallNode`.

Initialize member variable (#7737)