Merge pull request #1346 from fffej/master

Fix links that are now in /coding-guidelines

Merge pull request #1348 from ganeshran/api-mswin-dll-missing

Adding Fix for the api-win-ms dll missing error in documentation.

Workaround for the api-ms-win-* dll missing error

This change adds a workaround for a known issue of api-ms-win-* dll missing error while debugging the coreclr solution.

fix links

Merge pull request #1336 from tony/freebsd-lldb

Update FreeBSD installation docs for LLDB support

Merge pull request #1347 from fffej/teh-to-the

Fix typos of teh

Fix typos of teh

Update FreeBSD installation docs to the new devel/llvm37 package.

- Instructions updated for building with llvm37
- Remove freebsd-install-lldb.sh script
- Add syntax-specific highlighting to code blocks
- Create and link to note on other Clang / LLDB versions

Merge pull request #1342 from jkotas/early-mscorlib-references

Fix crash when using OpenCover with mscorlib

Merge pull request #1327 from mikem8361/settid

Add work around a bug in lldb on a core dump

Fix crash when using OpenCover with mscorlib

OpenCover injects self referencing assembly refs to mscorlib. The fix avoids crash when the assembly loader tries to resolve them early during startup.

Fix #1340.

Merge pull request #1326 from ganeshran/MovingPackageFolderCreationSection

Moving Folder creation section above Nuget.exe Download

Moving Folder creation section above Nuget.exe Download

The Download Nuget.exe section assumes that the ~/coreclr-demo/packages folder already exists. However the command to create it is given in the section after that. In other platform instructions file the sequence appears correctly

And Removing the phrase 'Place all the Nuget packages together' as per PR comment

Merge pull request #1325 from mikedn/winx86

Enable the Windows x86 build

Enable the Windows x86 build
This allows building the Windows x86 version of CoreCLR by using "x86" for the BuildArch parameter of build.cmd. Note that CMAKE_SYSTEM_PROCESSOR is no longer used in Windows builds to set IS_64BIT_BUILD.
This change is enough to get CoreCLR to build but more changes are required for it to actually run correctly. In particular, the JIT compiler support for x86 is pretty limited at this point.

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

Merge changes from TFS

Allow x86 version of prebuilt mscoree.h to be used for all architectures

[tfs-changeset: 1509924]

Merge pull request #1332 from benpye/arm-regression-fix

Fix ARM build regression

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

Merge changes from TFS

Fix build break on Windows ARM

[tfs-changeset: 1509827]

Merge pull request #1337 from dotnet/revert-1334-from-tfs

Revert "Merge changes from TFS"

Revert "Merge changes from TFS"

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

Merge changes from TFS

Fix ARM build regression

Merge pull request #1331 from justinvp/streamreader_cachedencodings

StreamReader: Use the static cached Encoding instances

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

Merge changes from TFS

Use x86 version of prebuilt mscoree.h

It matches what we do in the legacy build environment. The x64 version of the header file is causing build breaks on x86 as @mikedn found out.

[tfs-changeset: 1509675]

Merge pull request #1330 from dobe/master

fixed typo

fixed typo

Added a "setsostid" command to workaround a bug in lldb on a core dump where the system ids (TIDs) are wrong.

First find the managed thread os id is:

(lldb) sos Threads
                                                                                                                                                           Lock
       ID OSID ThreadOBJ           State GC Mode     GC Alloc Context                  Domain           Count Apt Exception
   1    1 3787 00000000006547F8    20220 Preemptive  00007FFFCC0145D0:00007FFFCC015FD0 00000000006357F8 0     Ukn
   6    2 3790 0000000000678FB8    21220 Preemptive  0000000000000000:0000000000000000 00000000006357F8 0     Ukn (Finalizer)

(lldb) thread list
Process 0 stopped
* thread #1: tid = 0x0000, 0x00007f01fe64d267 libc.so.6`__GI_raise(sig=6) + 55 at raise.c:55, name = 'corerun', stop reason = signal SIGABRT
  thread #2: tid = 0x0001, 0x00007f01fe7138dd libc.so.6, stop reason = signal SIGABRT
  thread #3: tid = 0x0002, 0x00007f01fd27dda0 libpthread.so.0`__pthread_cond_wait + 192, stop reason = signal SIGABRT
  thread #4: tid = 0x0003, 0x00007f01fd27e149 libpthread.so.0`__pthread_cond_timedwait + 297, stop reason = signal SIGABRT
  thread #5: tid = 0x0004, 0x00007f01fe70f28d libc.so.6, stop reason = signal SIGABRT
  thread #6: tid = 0x0005, 0x00007f01fe70f49d libc.so.6, stop reason = signal SIGABRT

Then use the new command "setsostid" to set the current thread using the "OSID" from above:
(lldb) setsostid 3790 6
Set sos thread os id to 0x3790 which maps to lldb thread index 6

Now ClrStack should dump that managed thread:
(lldb) sos ClrStack

To undo the affect of this command:
(lldb) setsostid

Added setclrpath command that allows the path that sos/dac/dbi are loaded from to be changes instead of using
the coreclr path. This may be needed if loading a core dump and the debugger binaries are in a different directory
that what the dump has for coreclr's path.

Merge pull request #1320 from ganeshran/1313-GitSetupSection

Adding instructions to clone both corefx and coreclr in the Git Setup…

Merge pull request #1285 from benpye/linux-arm-stack-unwinding

Add unwinding for the ARM Linux platform

Modify unwinder to provide RtlVirtualUnwind.

Use PAL_VirtualUnwind on Linux ARM

Add assembler annotation for unwinding

Adding instructions to clone both corefx and coreclr in the Git Setup section for Linux and FreeBSD. The corefx clone is needed on Linux/FreeBSD to build the native components. Ref #1313

Merge pull request #1319 from kangaroo/aarch64-hfa-fix

[aarch64] Correct the checking of return class type to check against CORINFO_TYPE_FLOAT

[aarch64] Correct the checking of return class type to check against CORINFO_TYPE_FLOAT

Fix more build breaks on Windows arm64

[tfs-changeset: 1508010]

Fix build break on Windows arm64

[tfs-changeset: 1507960]

Merge pull request #1317 from nguerrera/corefx-native-docs

Add instructions for building corefx native components

Add instructions for building corefx native components

Fix #1313

Merge pull request #1292 from kangaroo/aarch64

Initial aarch64/linux support

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

Merge changes from TFS

[aarch64] Address code review feedback and disable EmitGetThreadInlined on PAL for now

Merge pull request #1309 from schellap/copyprop

Conservatively mark variable as killed if there is an embedded comma …

Fix prefast warnings in CLR code

[tfs-changeset: 1507366]

Conservatively mark variable as killed if there is an embedded comma assignment

GitHub Issue: Another RyuJIT optimization bug #1299
https://github.com/dotnet/coreclr/issues/1299

The underlying problem is there are two loops:

The first loop does copy prop
The second loop updates the stack of live definitions in the currently
traversed CFG path.

This approach works fine only if assignments are atomic statements, but
when there is an assignment embedded using a comma in a statement:
The first loop does nothing about this and continues to propagate
copies, i.e., without updating the liveness stack. This leads to using a
stale VN for comparison rather than using an updated VN due to this
definition.

Only when the second loop runs, this live definition is added to the
stack.

The right fix is to merge the two loops into a single loop that does
copy prop on the tree and at the same time updates the liveness on the
stack right after the call. But this is causing a lot of ASM diffs which
would affect stability.

I am making a conservative fix of temporarily marking the variables that
have embedded intervening defs as "killed", until the second loop runs
to update liveness.

There are no SuperASM diffs with this change.

Merge pull request #1298 from mmitche/tail-call-fix

Fix for potential bad tailcall code generation

Fix for potential bad tailcall code generation

The JIT can potentially pass an incorrect parameter to a callee when optimizing a tail call.  While updating a GT_LCL* node with lcl var num of temp passed to the tail call, we also need to change its opererand to GT_LCL_VAR.

[aarch64] lsra wa not properly setting the killMask for initBlk, copyBlk or copyObj

[pal] PAL_printf_arg_remover was not properly handling formats like %*s

Merge pull request #1269 from Sridhar-MS/osx-tickcount

Fix to get the correct value for Environment.TickCount in OSX

[aarch64] Enable the interpreter on linux as well

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

Merge changes from TFS

[aarch64] Initial aarch64/linux bring up

Fix to bad codegen issue while tail calling virtual stub dispatched call that requires runtime lookup.

This is a case of Virtual Stub Dispatch (VSD) call that requires runtime lookup.  For such a call JIT needs to pass the address of the indirection cell in R11 (which is a callee trash register).  For the F# repro, VSD call is marked with ".tail" prefix and a result RyuJIT attempted to dispatch through a tail call helper.  But CLR tail call helper implementation doesn't preserve R11 nor make any such guarantees and a result indirection cell address passed in R11 gets trashed leading to an AV in VM during call dispatch.  In fact all of the calls that have non-standard args passed in callee trash registers cannot be tail called via helper for the same reason.  Note that it is fine to fast tail call though.

     Fix: During call morphing reject tail calls with non-stand args to be dispatched via tail-call helper.

[tfs-changeset: 1506345]

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

Merge changes from TFS

Add new file to the old msbuild makefiles

[tfs-changeset: 1506251]

Fix to return the right frequency (in ticks per second) and counter (in terms of ticks) in OSX.

Reference: https://developer.apple.com/library/ios/qa/qa1643/_index.html

Merge pull request #1295 from benpye/windows-unix-hosting-api

Expose Unix hosting API on Windows

Expose Unix hosting API on Windows

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

Merge changes from TFS

Fix 1203471: Incorrect rationalization of unused SIMD localVar

The Rationalizer was not correctly handling a case of an unused SIMD expression involving a localVar or temporary value, where the SIMD expression is returning a non-SIMD value, and the expression is sufficiently complex (e.g. a call to vector * scalar which is inlined but not an intrinsic).
The ldobj of the localVar is not eliminated, because it involves an indirection, and therefore appears potentially unsafe to eliminate. However, when we transform the ldobj into a plain localVar during the Rationalizer, we need to correctly handle the case where it has no parent.

[tfs-changeset: 1505976]

Merge pull request #1239 from sergey-raevskiy/proftoeeinterfaceimpl-doc-update

Update link to documentation

Use the cached Encoding instances in StreamReader

The cached Encoding instances can be used instead of always creating
new instances. The same constructor arguments are used.

Merge pull request #1262 from apanda/master

Fix build to work with binutils v2.25.0 on Linux

Build fails when using binutils version 2.25.0

On Linux, when using binutils version 2.25.0, XMM instructions (e.g., movdqa) requires specifying the size of the
operand (e.g., specifying xmmword ptr). With the current code it gives an error: invalid operand for instruction.

This commit changes movdqa occurances to fix the problem.

Move 's_TimebaseInfo' intialization to PAL intialization and nanosecond resolution support for StopWatch in OSX.

OSX provides nanosecond timer capabilities. But the PAL was not using it and falling back to microsecond-level resolution.

Merge pull request #1278 from eerhardt/master

Adding TimeZoneInfo.Local and FindSystemTimeZoneById support on Linux.

Adding TimeZoneInfo.Local and FindSystemTimeZoneById support on Linux.

The local time zone is found by using the TZ environment variable or the "localtime" files on the system.  I needed to add some code that searched through the zoneinfo directory to get the correct Id of the local time zone, since this information is only stored in the file name, and not in the file itself.

FindSystemTimeZoneById uses the id as a file path to find the correct tzfile on the system.

Also added a few #ifs of dead code on Linux.

Replace the usage of MACH monotonic function 'clock_get_time' with 'mach_absolute_time' in GetTickCount64.

Monotonic clock function 'clock_get_time' was causing perf regression in 'Environment.TickCount'. So replacing it with 'mach_absolute_time' which gives considerable performance gains. Here are the perf numbers
Using 'clock_get_time' (without caching host ports) - Average time 2.86 microseconds.
Using 'clock_get_time' (with caching host ports) - Average time 0.82 microseconds.
Using 'gettimeofday' (original implementation) - Average time 0.05 microseconds.
Using 'mach_absolute_time' - Average time 0.04 microseconds.

Merge pull request #1277 from erozenfeld/RODataAlignment

Add an option to request 16 bytes alignment for read-only data.

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

Merge changes from TFS

Add an option to request 16 bytes alignment for read-only data.

llilc jit needs this to support LLVM floating-point constant pools.

Merge pull request #1210 from benpye/linux-arm

CoreCLR building on ARM

Fix FILECanonicalizePath test.

The string buffer in the test did not allow for the null terminating
byte.

After calls to vprintf (and similar) call va_end

Add the arg remover so that the arguments handled by the native vprintf
functions are removed from the arg list.

Add ARM target for CoreCLR on Linux.

c_runtime/vprintf/test1 is disabled as casting NULL to va_list is
against the C specification.

Fix SetFilePointer tests on 32 bit platforms.

Define _FILE_OFFSET_BITS=64 so that we have long file support on 32 bit
platforms.

Implement context capture/restore for ARM.

Link libgcc_s before libunwind on ARM so C++ exceptions work.

Translate armasm to gas syntax.

Specify Thumb, VFPv3, ARMv7 for the ARM target.

Add ARM configuration to mscorlib build

Implement GetLogicalProcessorCacheSizeFromOS in PAL.

Set UNWIND_CONTEXT_IS_UCONTEXT_T from configure check.

Merge pull request #1280 from nguerrera/remove-corefx-crypto

Remove System.Security.Cryptography.Native from coreclr build

Update ETW allocation sampling shutdown method contract.

[tfs-changeset: 1504772]

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

Merge changes from TFS

Fix recursive assert in utilcode/debug.cpp when VM code, etc. hits an assert.

The fix was to rearrange the code so the assert isn't necessary and removed it. Fixes TFS bug 1199569.

[tfs-changeset: 1504713]

Merge pull request #1242 from david-mitchell/update-osx-instructions

Update OS X build instructions.

Fix Mach host port leaks in OSX

- Add return value check for "host_get_clock_service".
- Fix host port leaks in GetTickCount64 and GlobalMemoryStatusEx.

Remove System.Security.Cryptography.Native from coreclr build

This library now builds in corefx. The coreclr/src/corefx folder
is left behind with an empty project list to leave room for
Sytem.Globalization.Native which will need to stay in coreclr
due to being depended upon by mscorlib.

Remove the SecuritySafeCritical from the ParseTargetFrameworkName.

This attribute was added as part of the initial implementation and is no longer needed in the current version. What this attribute does it allows us to call SecurityCritical methods from our code. However, we no longer have the need to do that. Additionally, having this attribute prevents libraries that are explicity SecurityTransparent from compiling this code in their assembly (SecAnnotate would complain).

[tfs-changeset: 1504639]

Fix to get the correct value for Environment.TickCount in OSX

On linux monotonic clock API clock_gettime is available. But on OSX the PAL implementation falls back to gettimeofday. This always retured the time elapsed since Unix Epoch (1970/01/01). I fixed it by using the mach API clock_get_time on OSX to get the time elapsed since system start.

Fixes #2441
https://github.com/dotnet/corefx/issues/2441

Merge pull request #1263 from adityamandaleeka/contextAlignment

Fix alignment issue when converting to/from native context.

Merge pull request #1266 from mikem8361/debugdoc

Add directions for debugging CoreClr on Linux

Add directions for debugging CoreClr on Linux.

Fix alignment issue when converting to/from native context.

When building in release mode, the code in CONTEXTFromNativeContext and
CONTEXTToNativeContext was using the movaps instruction to move the float
and XMM register values to/from the native context. This is because the
macros FPREG_St and FPREG_Xmm were casting unaligned pointers to an
aligned struct type (M128A). This change adds a type M128U which is the
same as M128A but without the alignment guarantee. Using this type in
those macros prevents the compiler from trying to use movaps on
potentially unaligned memory. This bug manifested when trying to call
these functions with native contexts that were passed into a real-time
signal handler.

Merge pull request #1247 from mmitche/skip-tests-option

Add option to skip test build in build.cmd

Add option to skip test build in build.cmd

Add help information regarding skiptestbuild option

Merge pull request #1255 from benpye/coreconsole-crash-fix

Allow coreconsole to be called without extension

Allow coreconsole to be called without extension.

Merge pull request #1199 from judemelancon/patch-1

Remove incorrectly duplicated comment

Merge pull request #1227 from mikedn/shrdis

Fix SHR operand diassembly in RyuJIT

Merge pull request #1250 from david-mitchell/fix-locals

Declare locals in correct order.

Merge pull request #1245 from janvorli/hosting-api-extension

Extend the Unix hosting API

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

Merge changes from TFS

Merge pull request #1252 from ViIvanov/fix/assert-in-lazy-value/issue-1186

Fix #1186: wrong Contract.Assert in Lazy<>::Value getter.