Merge pull request #1248 from stephentoub/dllimport_prefix_suffix

Update DllImport name logic to support lack of "lib" prefix

Update DllImport logic to support lack of "lib" prefix

Today the runtime supports automatically appending an appropriate
suffix (e.g. ".so", ".dylib", etc.) for the library name provided
to DllImport.  This commit adds support for also prepending the
standard library prefix "lib".

Merge pull request #1246 from stephentoub/fix_unix_etw

Ensure FEATURE_MANAGED_ETW isn't set for TargetsUnix

Ensure FEATURE_MANAGED_ETW isn't set for TargetsUnix

Updated buildtools were brought in to coreclr, and the props file was
still using the $(OS) variable to control whether FEATURE_MANAGED_ETW was
set, but whereas that used to be used to mean what platform we were
targeting, updated buildtools use it to mean what platform we're building
on, with $(OSGroup) meaning what is the target platform.

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

Merge changes from TFS

Fix cross platform build break

impAssignTempGen is used outside the translation unit and should not
be marked as inline.  Doing so causes link errors in release cross
platform builds.

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

Merge changes from TFS

Fix definition/initalization order build break

Clang is more strict about definition and intialization orders
matching than MSVC.

Fix to Unnecessary use of JIT_Stelem_Ref helper call when storing null in arrays.

Repro program:
     static T Dequeue<T>(T[] array, int index)
     {
         var head = array[index];
         array[index] = default(T);
         return head;
     }

     When Dequeue is instantiated over a ref type, the second line logically expands to

     object temp;
     initobj(&temp, null, 8)
     array[index] = temp

     The array store results in call to CORINFO_HELP_ARRADDR_ST helper call even though we are storing a null and a write barrier is not needed.

Fix:
     After the args of helper call are morphed, (i.e. after constant propagation is done), we can check to see whether the 3rd arg (value being assigned) is null and if so morph that into an array-indexed assignment so that a proper null and bounds check still take place.

Fix #1160

[tfs-changeset: 1501294]

Fix to bug#1173115 and Issue#1 listed in bug#1177388.

On xarch, bin-ops like add/mul/div that write to its LHS operand (i.e. target of result) and hence lower indicates to LSRA to preference the result of such bin-ops to its target operand to avoid additional moves.  The issue is that when we have nested bin-op expression like GT_MUL(GT_ADD(a,b), c), LSRA will preference GT_MUL to its LHS operand only if there are further references of it.  Since there will no further references of LHS operand GT_ADD, GT_MUL is not preferenced to GT_ADD.  This results in an additional move.  Fix in lines 4289-4307 is meant to fix this by letting to use relatedInterval even if there are no further references of it.

     With the above fix alone, there were major CQ losses since GT_STORE_LCL_VAR(GT_ADD(a,b)) would require a mov if GT_ADD is not preferenced to its parent lcl var but to its target operand ('a' in this case).  To fix this lines 2977-2981 allow a non-lclvar source of a STORE_LCL_VAR to use its parent interval as related interval for preferencing.

     With the combination so the above two fixes, there is a overall 0.32% improvement in size of the methods that got impacted.  The regressions are all smaller and a result of register preferencing got changed due to the above two fixes.

Fix #1163
CQ persuite run indicates Fractals benchmark improved by 15%.

[tfs-changeset: 1501293]

Merge pull request #1219 from mmitche/package-json-2

Move CoreCLR to the modern build tools and dnx

Fix to issue- .Net 4.6 RC x64 is twice as slow as Legacy Jit64 and x86.

Perf regression is due to a stall caused by "cvtsi2sd xmm1, rax" that partially writes lower 8-bytes of xmm1 but keeps other bytes unmodified. Since there are further uses of xmm1, it will introduce a false dependency and cause stalls.

Fix is to emit "xorps targetReg, targetReg" before a cvtsi2ss/sd instruction.

Fix #993

[tfs-changeset: 1501252]

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

Merge changes from TFS

Merge pull request #1231 from justinvp/array_reverse

Perf: List<T>.Reverse changed to not use non-generic Array.Reverse

Improve the performance of List<T>.Reverse

List<T>.Reverse is currently implemented in terms of the non-generic
Array.Reverse. Array.Reverse includes a fast path for a known set
of primitive types via a call into the runtime (see TrySZReverse).
Otherwise, it falls back to slower code paths that involve boxing
for non-primitive value types.

This commit changes List<T>.Reverse to use a generic implementation
of reverse that does not have the performance issue for
non-primitive value types.

Don't reorder operands for Vector constructor

The code in simd.cpp that recognizes the constructor for fixed-size vectors (the InitN intrinsic) was incorrectly reordering the operands. Constructing the vector in reverse order is more efficient, but that can be done in codegen without change execution order.

This change results in numerous diffs, with more regressions than improvements. The diffs are due to somewhat different register pressure, due to the ordering change, and different cases of constant reuse.

Fix #1212

[tfs-changeset: 1500923]

Merge pull request #1216 from janvorli/fix-destructor-contracts

Add missing contracts to some destructors

Support for SIMD Exceptions

For the CopyTo methods, we need to throw ArgumentOutOfRangeException instead of IndexOutOfRangeException for the initial index value, and then ArgumentException if the number of elements exceeds the remaining elements in the array. To support this, the GenTreeBoundsChk node now has a gtThrowKind field which indicates the kind of throw block it will branch to if the check failes.

This requires new helper calls, which I added to the end of corinfo.h.

[tfs-changeset: 1499475]

Add missing contracts to some constructors

After adding virtual destructors to some classes, the contract scanning
tool started to complain about missing contracts in some of the
constructors that were not touched.
This change adds contracts to those.

Move CoreCLR to the modern build tools and dnx

This changes moves coreclr onto DNX (same version as corefx).  Theoretically, this should allow these tests to target the desktop CLR.  All of the old package.config files are gone and replaced with corresponding project.json files.  The up front restore behavior is retained.  Tests are now buildable individually, though not runnable in a similar fashion.

SIMD Exceptions Fix (TFS 1189814)

For the intrinsics that take arrays, we need to check the initial index (if given) to ensure that it is non-null.

[tfs-changeset: 1499422]

Fixing Databinding scenario  for KeyValuePair. Refactoring the databinding code for CLRIReferenceImpl/CLRIReferenceArrayImpl/ICustomPropertyProviderProxy

[tfs-changeset: 1499363]

Merge pull request #1215 from richardlford/rosfix

Modify conservative GC code for unaligned InlinedCallFrames

Modify conservative GC code for unaligned InlinedCallFrames

When making pinvoke calls the JITs allocate InlinedCallFrames
within their stack frames to record the location
of the managed stack frame. These InlinedCallFrames are
not necessarily 8-byte aligned. These were being used
as the topStack in src/vm/gcenv.cpp, in the method
GCToEEInterface::ScanStackRoots.

This change checks to see if a frame is an InlinedCallFrame.
If so then its GetCallSiteSP method is used to
get the real top of stack. This will always be aligned.

There were two problem before this change wwhen the
frame was misaligned (e.g. ending with 4 or c):

1. GCToEEInterface::ScanStackRoots starts at the
topStack and look at every 8-byte chunk as if it were a
pointer. But if topStack is unaligned it is not really
finding the pointers on the stack.

2. When it gets to the bottom the 8-byte access
read the first 4 bytes ok, but the last 4 bytes
go beyond mapped addresses and an access violation
results.

Merge pull request #1211 from mmitche/improve-build-tp

Improve the build throughput

Merge pull request #1217 from mikem8361/fixassert

Fix recursive asserts in coreclr.

CR feedback.

Fix recursive asserts in coreclr.

Remove assert in VM break handler to prevent recursive asserts. Fixed problem where the DebugBreak wasn't terminating the app on an assert causing double assert messages.  Restoring the SIGTRAP handler and returning was continuing the DebugBreak not terminating. Replaced this with an abort().

Cleanup the assert message formatting.

Add missing contracts to some destructors

This change adds missing contract annotation to several destructors that
were missing it.

Merge pull request #1196 from justinvp/docs_corefxtests

Fix markdown of the "Testing with CoreFX" documentation so it renders correctly on GitHub

Improve the build throughput

Especially with the builds of the tests, the diag logging significantly reduces TP in the build and seems to reduce parallelism a bit too.  Change this to normal.

Merge pull request #1209 from swaroop-sridhar/unused

Remove a few Unused Definitions

Merge pull request #637 from justinvp/stringsplit

String.Split optimization

Remove a few Unused Definitions

Remove three unused local variable definitions --
which caused warnings in the LLILC build.

Merge pull request #1124 from DickvdBrink/vs2015-doc-fix

Document Visual Studio 2015 support

Document Visual Studio 2015 support

Merge pull request #1200 from erozenfeld/CrossgenFix

Allow non-RIP-relative relocations in coreclr crossgen.

Merge pull request #1202 from mj1856/docs

Fix broken link in documentation

Fix broken link

Allow non-RIP-relative relocations in coreclr crossgen.

Merge pull request #1195 from mikem8361/dactable

Generate the dac table RVA using nm at build time.

Fix markdown so it renders correctly on GitHub

Fix Windows build.

Merge pull request #1194 from ellismg/intern-jit-packages-files

Speed up package resotre of JIT Test build

Generate the dac table RVA using nm at build time.

Remove the temporary PAL dac table file at runtime. Replaced it with dactablerva.h file generated at build time from the libcoreclr module. nm works across Linux, OSx and FreeBSD.

Merge pull request #1189 from MarcelGosselin/master

Fix broken Developer Guide documentation links

Fix broken Developer Guide documentation links

String.Split optimization

- Use EmptyArray<String>.Value instead of new String[0] on .NET Core
- Return early before allocating internal array(s) when count == 1

Speed up package resotre of JIT Test build

We spend a fair amount of time (six wall clock minutes on my machine)
doing package restore for the JIT tests. This is because each project
has its own packages.config file, so during the build we call nuget
restore for each project.

There are only four unique packages.config files for all these tests, so
this change moves them to tests/src/JIT/config and updates the project
files to consume them from there.

This means during the build the `EnsureDependencies` target does not
need to invoke nuget restore as often and the build is much faster as a
result.

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

Merge changes from TFS

Replace COMPLUS_SOCExtraSpew with COMPLUS_JitFunctionTrace, and enable it for all architectures in RyuJIT

[tfs-changeset: 1495487]

Merge pull request #1185 from bartonjs/x509chain

Add wrapper functions for building X509Chains

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

Merge changes from TFS

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

Merge changes from TFS

Add missing include directory

dbgshim.cpp wants to include debugshim.h but the directory that holds
this file was not on the include path when buidling with cmake.  Adding
this directory in line with what we had for the razzle driven internal
TFS build.

Fix arm/arm64 builds of dbgshim

[tfs-changeset: 1494755]

Make 64bit dump debugging possible on CoreCLR

[tfs-changeset: 1494731]

Merge pull request #1175 from mikem8361/palinit2

Some more pal init cleanup missed in #1174

Some more pal init cleanup missed in #1174

Merge pull request #1174 from mikem8361/palinit

Fixed the PAL_Initialize* order problem.

CR feedback from @janvorli

Fixed the PAL_Initialize* order problem.

Now it doesn't matter if PAL_InitializeDLL (which doesn't pass argc/argv) is called before PAL_InitializeCoreCLR or vice-versa.  The exe path and command line are updated for every PAL_Initialize or PAL_InitializeCoreCLR call.

Cleaned up the pal_module initialization to get and set the module name removing the need for PAL_InitializeCoreCLR to pass the coreclr path. This required changing the process attach DllInit to be done more on demand.

The signature for PAL_InitializeCoreCLR has changed to (two parameters removed):

PALIMPORT
DWORD
PALAPI
PAL_InitializeCoreCLR(
    const char *szExePath);

Fixed a problem in the ClrStack -i command where the SP/PC displayed wasn't correctly.

Merge pull request #1169 from pgavlin/splittests

Refactor JIT test projects.

Refactor JIT test projects.

This change replaces the template projects used for the ported JIT tests
with a single project file per test. This refactoring is intended to
clarify exactly which tests from the legacy testbed have been ported
and what their dependencies are; this should make later reasoning about
these tests more straightforward.

This change also disables a number of innocuous warnings in the ported
tests for hygenic purposes.

Add support for setting verification time and querying the root store directory

Merge pull request #1159 from mikem8361/bpmd

Enable bpmd command (even unjitted functions work)

Add methods for interacting with STACK_OF(X509)

Merge pull request #1156 from romansp/fix-typos

fix typo in objectnative.cpp comments

Enable bpmd command even unjitted functions work.

Preallocate JIT notification table on VM side (since ICLRDataTarget2->AllocVirtual isn't implemented under lldb) and implement WriteVirtual on debugger side to allow DAC to write entries.

Used the special set breakpoint on exception throw lldb command to catch the special exception the VM throws when some
thing is jitted. The debugger side finds the exception record by searching for the RtlpRaiseException function on the stack
when the bp is hit. It also assumes that the local variable is called ExceptionRecord.

Fix all the file path separators to use the platform independent defines.

Ifdef all the "/d" options in the supported commands because the / char isn't supported as a command line option and /d is always the DML option that isn't supported either.

Fix problem Prem run into running sos after the process stops.

Merge pull request #1136 from josteink/unwind-freebsd

Fix stack unwinding for FreeBSD.

Fix stack unwinding for FreeBSD.

Fix error in stack unwinding on FreeBSD as discovered by @saper.

This closes #1102 and #1113

fixed typo in objectnative.cpp comments

Merge pull request #1153 from mikem8361/ctrlc

Allow CTRLC to work when running under lldb using the SOS plugin

Allow CTRLC to work when running under lldb using the SOS plugin and debugger dlls
by only setting the SIGINT/SIGQUIT handlers if the signal handler thread is started.

Merge pull request #1107 from krk/master

ObjectDisposedException handled in test wrapper.

Merge pull request #1141 from mikem8361/dbgshim

Fully implement dbgshim so it doesn't depend on LD_LIBRARY_PATH (issue #650)

Merge pull request #1149 from rehrumesh/Fix_1148

Fixed #1148. Corrected links for Instructions

Merge pull request #1143 from jkotas/GetAssemblyName

Allow AssemblyLoadContext.GetAssemblyName for native images on CoreCLR

Fixed FreeBSD build.

Code review feedback.

Corrected links for Instructions

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

Merge changes from TFS

Allow AssemblyLoadContext.GetAssemblyName for native images on CoreCLR

Make sure dbgshim can attach to processes from other sessions

[tfs-changeset: 1488695]

Fully implement dbgshim so it doesn't depend on LD_LIBRARY_PATH (issue #650).

It finds and uses the path coreclr is in to load dbi/dac.

EnumProcessModules returns an array of module base addresses and GetModuleFileNameEx
takes the base address and returns the module name string.

Fixed directory separator in dbgshim.

Merge pull request #1140 from dotnet/blackdwarf-patch-1

Fix the ECMA standard link

Fix the ECMA standard link

On README.md, the intro to .NET Core had a wrong link.
Missing the directory portion.

Merge pull request #1134 from tvlooy/patch-2

Fix typo

Fix typo

Merge pull request #1130 from pgavlin/ILTests

Import the first batch of IL tests.

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

Merge changes from TFS

Merge pull request #1125 from mikem8361/removelibpath

Remove the dependency on LD_LIBRARY_PATH in the sos code (issue #820)

Import the first batch of IL tests.

This set of tests is comprised of IL tests that:
- only reference APIs in .Net Core's System.Runtime,
  System.Runtime.Extensions, and System.Threading contracts
- compile to a single executable,
- and expect no command line arguments at runtime.

Merge pull request #1120 from libengu/RestOfRegression

Add all remaining JIT/Regression test cases for JIT

Fix Exception Propagation issue for NetCoreForCoreCLR Non-AppX case

Currently in CoreCLR, We don't propagate managed exception from Managed->Native->Managed correctly for non-appx case--- if you throw a managed exception, then during M->N, the managed exception is stored in thread, later during N->M, we will try to fetch this stored exception if it is managed exception by checking IsManagedObject()

The logical for IsManagedObject should be just check its IUnknown slots first no matter whether it is appx or not; If doesn't match, try to QI IManagedObject.

[tfs-changeset: 1486094]

Merge pull request #1114 from dotnet/docs-subfolders

Add subfolders to Documentation folder

Fixing for comments in #1114

Add subfolders to Documentation folder

As raised in #1075 by @akoeplinger, the Documentation folder's file
list forces the user to scroll way down to see the README.md. This
PR fixes that by storing the documents in separate sub-folders one
per hierarchy of related documents. Also changed
Documentation\README.md and root README.md to take into account
the new paths.

Merge pull request #1127 from ronyb29/master

fixed typo in BufferedStream comments

Merge pull request #1126 from stephentoub/reflectionemit_writeline

Implement ILGenerator on Unix to use System.Console.dll