Merge pull request #589 from josteink/bsd_configure

Correctly set HAVE_BSD_REGS_T flag during configure.

Merge pull request #590 from josteink/bsd_regs

PAL: Add register-mapping for FreeBSD x86_64.

Merge pull request #587 from mikem8361/dbi1

Enabling the ClrStack command ICorDebug option (-i) to use to test DBI.

PAL: Add register-mapping fore FreeBSD x86_64.

Correctly set HAVE_BSD_REGS_T flag during configure.

Fix BSD-regs detection. Struct name is 'reg' not 'regs'.

This fixes a bunch of issues when trying to build on FreeBSD.

Merge pull request #569 from brianrob/perfdoc

Add Cross-Platform Performance and Eventing Design Document

Enabling the ClrStack command ICorDebug option (-i) to use to test DBI.

Fix sos on windows. Changed the name of the debug info resource to just "CLRDEBUGINFO".  This is what debugshim is looking for when searching for the coreclr module.

Merge pull request #585 from dotnet/revert-584-bsd_configure

Revert "Correctly set HAVE_BSD_REGS_T flag during configure."

Revert "Correctly set HAVE_BSD_REGS_T flag during configure."

Updates from PR comments.

Merge pull request #584 from josteink/bsd_configure

Correctly set HAVE_BSD_REGS_T flag during configure.

Correctly set HAVE_BSD_REGS_T flag during configure.

Currently the test sets a flag which is found nowhere else in the code.

    $ grep -R " BSD_REGS" * | wc -l
    0

This fixes a bunch of issues when trying to build on FreeBSD.

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

Merge changes from TFS

Merge pull request #578 from briansull/use-lea-for-mul359

Use lea for multiply by 3, 5 or 9

ExecutionContext.IsDefaultFTContext incorrectly checks the "current" context

Fixed: ExecutionContext.IsDefaultFTContext incorrectly checks the thread's current context instead of the context that is it called on. This prevents Overlapped from running it's callback on the captured execution context, and hence, async locals don't flow for overlapped callbacks. There are no tests with this change, because the only thing that is broken currently is Overlapped. I've written new tests for the new overlapped APIs that were broken *before* this change, and pass after this change, these will be checked in with the overlapped feature.
Changed: Marked ExecutionContext.Default private, as it was accidently made public.

[tfs-changeset: 1440483]

Merge pull request #581 from sergiy-k/gccrash-ipinmodule

Fix an issue in ExceptionTracker::FindParentStackFrameHelper that resulted in an AV during GC stack root scanning.

When choosing an instruction for a GT_MUL we will now use an LEA instruction for a MUL by an immediate 3, 5 or 9

The multiply has to be a non overflow checking multiply as an LEA does not set condition codes.
In Lowering we also must make sure that we don't 'contain' an indirection as the source operand.

Fix an issue in ExceptionTracker::FindParentStackFrameHelper that resulted in an access violation during GC stack root scanning.

This change fixes the second issue reported in https://github.com/dotnet/coreclr/issues/566. The problem is that ExceptionTracker::FindParentStackFrameHelper uses IsIPInModule to determine whether an IP address belongs to VM or managed code but this function is not implemented when FEATURE_PAL is defined.
We should not really need the actual implementation of IsIPInModule on Unix. It is sufficient to just differentiate between managed and other code using ExecutionManager::IsManagedCode on Unix.

Merge pull request #580 from Djuffin/dbi-linux

Removing static PAL from SOS and DBI

Merge pull request #579 from stephentoub/indexof_empty

Fix IndexOfOrdinal on Unix for empty strings

Make DBI and SOS use PAL from DAC

Make it possible to write stress log from a module that doesn't have statically liked PAL

Fix IndexOfOrdinal on Unix for empty strings

IndexOfOrdinal searching for an empty string should return 0.  It's currently returning -1.

Merge pull request #575 from josteink/ptrace

PAL: Fix ptrace-invocation on FreeBSD.

Fix ptrace-invocation on FreeBSD.

Unlike in Linux[1] FreeBSD's ptrace does not use an enum[2].

[1] http://linux.die.net/include/sys/ptrace.h
[2] http://www.freebsd.org/cgi/man.cgi?query=ptrace

This fixes compiler errors like this one:

    /home/josteink/build/coreclr/src/pal/src/arch/i386/context.cpp:305:21: error: use of undeclared identifier '__ptrace_request'
            if (ptrace((__ptrace_request)PT_GETREGS, processId, (caddr_t) &ptrace_registers, 0) == -1)

Merge pull request #577 from josteink/runelocale

Fix missing RuneLocale on FreeBSD.

Merge pull request #576 from janvorli/warnings-cleanup

Fix about 12 kinds of warnings over the codebase

Merge pull request #573 from josteink/master

Define PAL_CS_NATIVE_DATA_SIZE for FreeBSD x86_64

PAL: Improve portability of FreeBSD RuneType-fix.

Fix missing RuneLocale on FreeBSD.

Without this patch the build breaks with errors like this:

    /usr/include/xlocale/_ctype.h:52:1: error: unknown type name '_RuneLocale'
    _RuneLocale     *__runes_for_locale(locale_t, int*);
    ^

Fix about 12 kinds of warnings over the codebase

This change fixes:
1) Member initiazalization order in class constructor didn't correspond to the member order in some classes
2) Objects with vtables were copied via memcpy. While this is intentional, cast of the pointers to void*
is required to silence clang warning
3) Switch case statements containing values of different enums
4) Casting int to pointer
5) Missing return value in methods with _ASSERTE
6) Class name classifier on methods in the .h
7) Invalid position of the DECLSPEC_ALIGN at few places - it had no effect
8) Invalid position of the 'used' attribute
9) Issue with comparing holders to NULL
10) Operator 'new' returning NULL without being marked with throw() or noexcept
11) Variables marked as 'extern' at the declaration / initialization place
12) Data structure declared as struct at one place and forward declared as class at another

Some disabled warnings were no longer happening, so options to disable them were removed too

Merge pull request #574 from janvorli/jit-warnings-cleanup

Fix accidentally removed initialization of m_VNFunc4Map

Fix accidentally removed initialization of m_VNFunc4Map

During my last change, I've accidentally left out the m_VNFunc4Map
while reordering the initialization list.

Merge pull request #572 from stephentoub/add_encodings

Update EncodingTable.Unix to support additional encodings

Define PAL_CS_NATIVE_DATA_SIZE for FreeBSD x86_64

Based on patch from @Aesthetikx which uses the program
[defined here](https://github.com/dotnet/coreclr/issues/60#issuecomment-73670124)
to determine the correct native size.

Update EncodingTable.Unix to support additional encodings

A variety of tests, in particular XML tests, are using some of the built-in encodings beyond UTF8, e.g. UnicodeEncoding, and are validating information about them, e.g. that an XML document encoded using UnicodeEncoding includes "utf-16" rather than "utf-8" in the output.  This commit augments the temporary globalization implementation with basic knowledge of UTF7, UTF16, UTF16BE, and UTF32.

Merge pull request #570 from Djuffin/cordb_platform_fix

Fix CORDB_PLATFORM mismatch between DAC and SOS

Fix CORDB_PLATFORM mismatch between DAC and SOS

Merge pull request #558 from josteink/master

Fix compilation errors on FreeBSD.

Merge pull request #561 from janvorli/jit-warnings-cleanup

Fix warnings in the jitter code

Fix warnings in the jitter code

This change fixes some of the warnings in the jitter code on Linux. With this change
combined with an upcoming change in the rest of the codebase, 16 warning disabling
options can be removed.

The issues in the jitter were:
1) Incorrect typedefs for const_reference and const_pointer (the const was ignored)
2) Member initiazalization order in class constructor didn't correspond to the member order in some classes
3) Objects with vtables were copied via memcpy. While this is intentional, cast of the pointers to void*
   is required to silence clang warning
4) Comparing values of different enums - there are cases of two enums containing the same values.
5) Casting int to pointer - the cast was legal (forming a handle), adding cast to size_t in between used to fix the warning
6) "static struct" - removed the static keyword
7) Missing return value in methods with _ASSERTE
8) Class name classifier on methods in the .h
9) Specialized template member functions need to be defined out of the class

Merge branch 'perfdoc' of https://github.com/brianrob/coreclr into perfdoc

Added blank lines before and after headers.

Merge pull request #554 from russellhadley/AdditionalInterfaceHeaders

Add additional headers to the CoreCLR package to describe its interface.

Merge pull request #564 from stephentoub/fix_lastindexof_unix

Fix LastIndexOfOrdinal on Unix

Updated scenario priorities.

Fix LastIndexOfOrdinal on Unix

The temporary implementation of LastIndexOfOrdinal has some bugs causing ArgumentOutOfRangeExceptions and other fun things. This should fix them.

Merge pull request #567 from stephentoub/fix_etw_ifdefs

Fix EventSource listeners on Unix

Fix debug-pal on FreeBSD.

This define is required to enable getline() on FreeBSD.

Add additional headers to the CoreCLR package to describe its interface.

Adds headers:
- opcode.def and openum.h to describe the MSIL opcodes.

Merge pull request #563 from CarolEidt/cleanupClsVar

Fix #497: Identical expressions to the left and right of the ||...

Fix #497: Identical expressions to the left and right of the || operator

This was very old code that was needed before the rationalization
of assignments (should have been GT_CLS_VAR and GT_IND).
GT_CLS_VAR should now never be seen post-rationalizer.

Adding Cross Platform Performance and Eventing Design Doc.

Fix EventSource listeners on Unix

Some recent changes to EventSource misplaced an #endif, causing EventListeners to not receive any messages when FEATURE_MANAGED_ETW is not set (it's not set on Unix).

Merge pull request #562 from jkotas/cppmunge

Delete cppmunge

Delete cppmunge

We have used this tool to boostrap the Unix build, but it is not used anymore.

Merge pull request #543 from janvorli/native-resources

Add native resources compilation on Linux

Propagate bindtextdomain status to the caller

The bindtextdomain can fail due to OOM. Propagate the status out of the
PAL_BindResources.

Merge pull request #557 from josteink/master

build.sh: Fix CPU-count detection on FreeBSD.

Merge pull request #556 from janhenke/freebsd-0325

Detect AMD64 Architecture on FreeBSD

Detect AMD64 Architecture on FreeBSD

In contrast to Linux and Darwin, FreeBSD identifies the AMD64 architecture with
the "amd64" string. Extend the checks in CMakeLists to accept either "x86_64"
or "amd64" as AMD/Intel 64bit architecture.

build.sh: Fix CPU-count detection on FreeBSD.

Merge pull request #553 from Djuffin/dbi-linux

Various DBI related fixes that make it possible to load DBI on Linux

Make code comments more accurate

Get rid of EventRedirectionPipeline() call on Unix

Various DBI related fixes that make it possible to load DBI on Linux and attach to a living CoreCLR process

Merge pull request #551 from cmckinsey/ldnull-ldelem-ver-fix

Normalize array element type of ldnull to stack size for verification

Merge pull request #548 from cmckinsey/nulltype

Change null constant type to be TYP_REF for isinst nullcheck expansion

Merge pull request #550 from janhenke/freebsd-buildsupport

Force /usr/local/include as search path on FreeBSD

TF#1144615 - Normalize array element type of ldnull to stack size for verification purposes.

Stephen Toub reported a RyuJIT assertion failure in System.Linq.Expressions.Tests.dll on unix.
The debug assertion was firing to ensure that the verification element type is normalized to
the stack slot sized type for verification purpose in later code that expects this convention.

Testing: DDR_And_JitSelfHost_diffs al clean with no asm diffs.
Performance: 0 asm diffs so no impact

Force /usr/local/include as search path on FreeBSD

CMake does not find headers installed in /usr/local/include by default. On
FreeBSD this is the default location for libunwind.h though. Define
CMAKE_REQUIERED_INCLUDE to include both /usr/local/include and /usr/include.
This change only affects FreeBSD. Also FreeBSD (like Darwin) does not work with
_DEFAULT_SOURCE or _POSIX_C_SOURCE defined. Closes #547

Merge pull request #546 from stephentoub/expand_env_vars

Implement ExpandEnvironmentVariables on Unix

Add native resources compilation on Linux

This change adds compilation of native resources into a gettext binary format and
loading of string resources in coreclr on Linux.
On OSX, the resource string still contains just a formatted resource id.

Implement ExpandEnvironmentVariables on Unix

The Environment type exposes ExpandEnvironmentVariables, but the corresponding Win32 function isn't available in the PAL, and thus calling Environment.ExpandEnvironmentVariables fails with an EntryPointNotFoundException for ExpandEnvironmentVariables from libcoreclr.dll.

This commit provides a basic implementation of the function in the managed ExpandEnvironmentVariables method on Unix.  It passes all of the corresponding tests in System.Runtime.Extensions.Tests.dll.

Merge pull request #544 from stephentoub/fix_textinfo

Initialize TextInfo's m_textInfoName in stubbed out globalization on Unix

Initialize TextInfo's m_textInfoName in stubbed out globalization

Some System.Runtime.Extensions tests are failing due to IsAsciiCasingSameAsInvariant trying to use a null m_textInfoName.

Change null constant type to be TYP_REF for isinst nullcheck expansion

In the importer when inlining the fast expansion sequence for
isinst/castclass, the code was typing the constant 0 source of the
compare with TYP_I_IMPL instead of TYP_REF. Not only does this create
an inconsistently typed compare but it causes value-numbering of
the same constants with different types to not be equal. This blocks
later assertion propagation that propagates null-checks and prevents
many cases from being eliminated.

This change changes the type of the constant gtNode to be TYP_REF.
With this change there are many good diffs in the frameworks and
other assemblies.

Ran testing in desktop branch including SuperPMI asm diffs. Asm
diffs show hundreds of methods with improvements in framework
, Roslyn, and other internal assemblies. 1% code size reduction in
the methods that have changes. All diffs are elimination of redundant
null-checks.

Merge pull request #525 from Djuffin/thread_id

Clean up PLA thread id usage.

Merge pull request #524 from ellismg/stat-in-pal

Add Stat like methods to CoreFX PAL

Prefer stat64 if present

Probe for the existence of stat64 and use it over stat if it
exists.  Aditionally, update the check around having a birthtime to be
more portable.

Refactoring similar pieces of code into a function

Add Stat like methods to CoreFX PAL

CoreFX would like to be able to call stat(2) and fstat(2) but directly
pinvoking to them from managed code is a bit of a chore, since the calls
depend on both the platform and architecture of the target platform.

This change adds stable versions of these APIs that the managed code can
interact with, internally the runtime calls the APIs using the platform
specific APIs and structures.

Long term it might be a good idea to move some of this stuff out of the
runtime and to a native helper library in CoreFX, so we have one less
dependency on the runtime and the ABI between managed and native code
for these methods can version withour reving the runtime, but for now we
put them in the runtime, like our other CoreFX PAL functions, since we
don't have a great way of building native code (let alone platform
specific native code) in CoreFX.

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

Merge changes from TFS

Merge pull request #539 from jkotas/glob

More stubbed out globalization

More stubbed out globalization

Merge pull request #531 from mikem8361/name2ee

Enable DumpLog and Name2EE sos commands

Merge pull request #538 from sergiy-k/context-flags

Fix CONTEXT_CaptureContext to correctly store the processor flags

Merge pull request #527 from jkotas/macunwind

Fix reporting of callee saved registers with nongnu libuwind

Fix compilation of context.cpp

Clean up PLA thread id usage.

CPalThread::GetThreadId() was inconsistent with GetCurrentThread() that sometimes led to issues like CreateThread returning incorrect thread id.
This changes makes a clean distinction between thread id and pthread handle (pthread_t), both get stored in CPalThread and both can be appropriately used when necessary.
General direction is that gettip() is used for communication with the outer world, and pthread_self() is used for internal work with pthread API.

Fix CONTEXT_CaptureContext to correctly store the processor flags

The CONTEXT_CaptureContext function did not correctly captured the value of EFlags. The problem was that there were calls to the test instruction before reading EFlags. This change fixes it.

Incorporated codereview feedback

Merge pull request #535 from Djuffin/stress_log

Make stresslog work for libraries other than CoreClr (I need it for DBI)

Merge pull request #532 from jkotas/slotassert

Fix invocation of shared generic multicast delegate

Merge pull request #537 from stephentoub/comparison_fixes

Fixes to string comparison in stubbed out Unix globalization

Replace PAL_GetCoreClrModuleBase() with PAL_GetPalModuleBase()

Make stresslog work for libraries other than CoreClr (I need it for DBI)

Introducing of PAL_GetPalModuleBase removes dependency on CoreCLR name for
stress log module address.

Fix string.ToUpper/ToLower to use the computed result

Amazing the difference a single character can make :)

Fix CompareStringOrdinalAscii to support !ignoreCase

CompareStringOrdinalAscii is using the ignoreCase argument to decide whether to normalize the input characters to upper case.  But the function then mistakenly ignores the result of this and always normalizes.  This commit just changes the implementation to use the chars the resulted from deciding whether to normalize or not.

Merge pull request #533 from jkotas/glob

Initialize number separators in stubbed out globalization