Merge pull request #1137 from hadibrais/patch-1

Add Table of Contents to clr-code-guide.md

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

Merge changes from TFS

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

Merge changes from TFS

We are incorrectly stripping away the Identity method from the EnumHelper type when building mscorlib.

[tfs-changeset: 1513902]

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

Merge changes from TFS

Build Toolbox\BinaryRewriting as part of the PK product group

We need this so we can get a signed build of BclRewriter.exe which is only produced by the offical build out of the ProjectK branch.

[tfs-changeset: 1513878]

Port the DateTime parser fix from the Desktop

This change is fixing the DateTime parser in cases of date/time strings for cultures which has the date and time separators are same. This happen with Norway, Serbia and Finland cultures

[tfs-changeset: 1513801]

Merge pull request #1329 from AndreyAkinshin/master

Fix a bug with disappeared cast to byte in numeric binary expressions

Fix a bug with disappeared cast to byte in numeric binary expressions

Lowering::IndirsAreEquivalent doesn't check if the 2 indirections have the same type and because of that a RMW style instruction is generated instead of the expected

movzx    rax, byte  ptr [7FFA61024742h]
xor      eax, 33
mov      word  ptr [7FFA61024742h], ax

This behavior was fixed by adding an additional check to IndirsAreEquivalent (implementation of the mikedn's approach: https://github.com/dotnet/coreclr/pull/1329#discussion_r36397171).

Fix #1323.

Merge pull request #1372 from eerhardt/master

Implement TimeZoneInfo.GetSystemTimeZones() on Linux

Implement TimeZoneInfo.GetSystemTimeZones() on Linux

Implementing GetSystemTimeZones on Linux by reading
the zone.tab file on the machine, parsing out the
valid TimeZone IDs from it, and using those Ids to read
all the time zones on the machine.

Merge pull request #1377 from jkotas/timer

Delete unused System.Threading.Timer.dll dependencies

Delete unused System.Threading.Timer.dll dependencies

Merge pull request #1375 from jkotas/vs2015-fixes

Assorted fixes for VS2015 build breaks

Disable debug info generation for test with very large method

Roslyn is hitting out of memory exception caused by internal PDB writer limitations (dotnet/roslyn/issues/3915)

Fix VS2015 test build breaks

Fix Windows SDK path for VS2015

Fix path to test build log

Merge pull request #1352 from eerhardt/master

TimeZoneInfo parse v2 tzfile and create AdjustmentRules

TimeZoneInfo parse v2 tzfile and create AdjustmentRules

TimeZoneInfo on Linux needs to parse V2 tzfile information and
correctly create AdjustmentRule instances based on the tzfile
information.

To create AdjustmentRules correctly, I introduced a new flag on
AdjustmentRule - NoDaylightTransitions, which means the adjustment
is fixed for the whole time the rule is in effect.  This allows
us to create an AdjustmentRule for each time the offset changed
in a time zone (whether the change was for Daylight Savings or
for some other reason).  I also needed to relax the rule that
DateStart and DateEnd needed to be DateTimeKind.Unspecified.
Instead, when we create NoDaylightTransitions adjustment rules
in Linux, we use DateStart and DateEnd to be UTC times the offset
transitioned.

Fix #2465.

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

Merge changes from TFS

Implement /disassemble nidump option for readytorun images

[tfs-changeset: 1512604]

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

Merge changes from TFS

Fix build break on desktop

[tfs-changeset: 1512590]

Merge pull request #1366 from sokket/timers

Removing old System.Threading.Timer tests and moving to CoreFX

Removing old System.Threading.Timer tests with the corresponding
move to CoreFX via CoreFX PR 2681

Merge pull request #1362 from mikem8361/exports1

Add GetCurrentThreadId to the Linux/Unix exports from coreclr.

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

Merge changes from TFS

Merge pull request #1363 from mmitche/fix-opensuse

Remove llvm-ranlib dependency

Fix BPMD SOS command for ReadyToRun

DAC was not getting notified about new method code

[tfs-changeset: 1512178]

Remove llvm-ranlib dependency

Appears to be unused, and not available on openSUSE.

Add GetCurrentThreadId to the Linux/Unix exports from coreclr.

Merge pull request #1324 from kouvel/kouvel-SetSignalCountAssertFix-1318

Fix PAL signal count upper limit issues.

Fix PAL signal count upper limit issues.

- Fix assert in SetSignalCount to allow int32 max as a valid value.
- Change ISynchStateController signal count functions to use LONG
  instead of DWORD to match the storage type for consistency.
- In ReleaseSemaphoreInternal, handle overflow of (oldCount +
  releaseCount) to return failure. Updated PAL semaphore tests to catch
  this case.

Fix #1318

Merge pull request #1254 from James-Ko/patch-1

mscorlib: Use Array.Empty where possible

Use Array.Empty where possible

Merge pull request #1354 from jkotas/vs2015-rtm

Fix build breaks on VS2015 RTM

Merge pull request #1357 from benpye/arm64-release-fix

Fix ARM64 compile break

Fix ARM64 compile break

Merge pull request #1343 from benpye/cross-compilation

Add cross compilation support for ARM and ARM64

Fix build breaks on VS2015 RTM

Most of the changes are about mismatches between printf formatting strings and argument types.

Fix #1294.

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

Merge changes from TFS

Fix bad msbuild properties so that conditions behave as expected.

[tfs-changeset: 1511107]

Merge pull request #1291 from justinvp/streamreader_preamble

StreamReader: Use a cached byte[] preamble for Encoding.UTF8

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

Merge changes from TFS

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.

Fix for build issue #764084, remove PK product group (it shouldn't have been added to begin with).

[tfs-changeset: 1510862]

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

Pick up security cookie definition from CRT headers

[tfs-changeset: 1510564]

Improve build-rootfs script.

Title:

   Binplace xplat binaries during official builds

Change Description:

   This is the initial implementation of having our official devdiv razzle builds binplace non-Windows xplat binaries.

   For now xplat binaries are binplaced only during official builds; this can be overridden by specifying /p:GetAllXPlatBinaries=true.  Binplacing is limited to CoreClr builds on amd64 as that's all we build for xplat.
   xplat binaries are pulled directly from Azure; if this becomes problematic we can look at having an internal cache.
   Binaries are pulled from Azure using a checked in copy of azcopy.exe.  For now I have embedded the source key; this will be removed in the future in favor of using a shared access signature.
   At present the build number for the xplat bits is fixed; this will be removed in a future change.
   The binaries are in zip format.  Once downloaded the contents are extracted in a directory under %_nttree% named after the platform; this puts them with the other xplat bits we build on Windows.

[tfs-changeset: 1510395]

Replace multiple .gitmirror files with .gitmirrorall .

Add missing packages in documentation.

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

Remove comments from tryrun cmake files

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.

Add cross compilation support for ARM and ARM64.

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]

Use a cached byte[] preamble for Encoding.UTF8

Avoids allocations associated with GetPreamble() for the default and
commonly used Encoding.UTF8 encoding.

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.