Merge pull request #768 from libengu/TestGenericsArrays

Add Generics/Arrays test cases for JIT

Merge pull request #765 from DickvdBrink/llvm_locate_fix

Fixed locating llvm tools on fedora

Add Generics/Arrays test cases for JIT

Add test cases in directory JIT/Generics/Arrays. Related proj, config, and sln files
added and updated. Passed build, buildtest, runtest.

Merge pull request #766 from libengu/cmov

Add test cases in directory Directed/cmov

Add test cases in directory Directed/cmov

Add test cases in directory Directed/cmov. Logical operator
tests where many of the operations emit cmov instruciton on
x86\x64. Related proj, config, and sln files added and updated.
Passed build, buildtest, runtest.

Merge pull request #752 from CarolEidt/PrecedenceFixes

Issue #609 & #610: Fix precedence issues

Merge pull request #754 from jkotas/unnecessaryfile

Delete unnecessary .gitmirror file

Delete unnecessary .gitmirror file

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

Merge changes from TFS

Use more appropriate mirror setting for src\corefx

[tfs-changeset: 1457001]

Merge pull request #745 from bartonjs/crypto-interop-library

Create native interop library System.Security.Cryptography.Native

Fixed locating llvm tools on fedora

The locate_llvm_exec function was missing the $llvm_prefix

Merge pull request #746 from libengu/TestCheckedCtor

Add CSharp Base/Peer Constructor test cases

Create native interop library System.Security.Cryptography.Native

Add CSharp Base/Peer Constructor test cases

Add CSharp Base/Peer Constructor test cases. Related proj, config,
and sln files added and updated. Passed build, buildtest, runtest.

Issue #609 & #610: Fix precedence issues

Near line 6089 in importer.cpp, enforcing the precedence indicated by
the indentation resulted in no diffs, which at first seemed surprising.
However, this is because we never encounter the other mask values
without calli.

Near line 10483, this is where we convert (dup, stloc) to (stloc, ldloc).
This was clearly intended only for non-debug mode, but the original code
in the importer was like this:

     if (!opts.compDbgCode &&
         (nextOpcode == CEE_STLOC)   ||
         (nextOpcode == CEE_STLOC_S) ||
         ((nextOpcode >= CEE_STLOC_0) &&
         (nextOpcode <= CEE_STLOC_3)))
     {
         insertLdloc = true;

Due to the fact that the parens don't support the apparent intended
precedence, the condition is equivalent to:

    if ((!opts.compDbgCode && (nextOpcode == CEE_STLOC)   ||
       (nextOpcode == CEE_STLOC_S)                        ||
       ((nextOpcode >= CEE_STLOC_0) && (nextOpcode <= CEE_STLOC_3)))

Changing it the apparent original meaning caused regressions.  This is
because it is often cheaper to load a local (i.e. the insertLdloc case)
than to do a dup, which may 1) cause a temp to be created to store the
value on the stack, or 2) result in a tree (e.g. a large constant) which
is cloneable but which is more expensive in code bytes than a ldloc.

The original code clearly intended to do this only in the non-debug case,
but with the incorrect parens it was doing it for any short stloc.  I'm
not sure that it is 100% OK to do this transformation on dbg code, but
we've apparently been doing it for some time (it's the same way in
jit32).

Adding the apparently intended parens caused regressions (158392 bytes
over all the MCH files). The change in this shelveset has a net
improvement (3848 bytes over all jitAsmDiffs).  (Doing this
transformation in all cases, i.e. just eliminating the !opts.compDbgCode,
causes zero diffs, indicating that we don't have any instances of dups
followed by non-short stlocs.)

Fix #609
Fix #610

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

Merge changes from TFS

Enable build crossgen.exe from build.cmd.

Adds and modifies CMakeLists.txt files to enable building of crossgen.exe from build.cmd for x64 processor on Windows. Also adds a step in build.cmd to generate native image for mscorlib.

[tfs-changeset: 1456454]

Merge pull request #736 from ellismg/hardcode-utf8-in-pal

Hardcode CP_ACP to UTF-8 on Linux

Merge pull request #727 from richlander/rich-docs

Add Book of the Runtime docs

Merge pull request #733 from stephentoub/temp_path

Make GetTempPathA/W first check TMPDIR

Merge pull request #738 from libengu/AddJITCSETests

Add JIT CSE tests

Add JIT CSE tests

Add JIT CSE (common sub-expression elimination) optimization tests.
Related proj, config, and sln files added and updated. Passed build,
buildtest, runtest. Code went through Microsoft internal process
to meet open-source requirements.

Merge pull request #737 from janvorli/fix-div-overflow-exception

Ensure we differentiate between division by zero and overflow

Ensure we differentiate between division by zero and overflow

This change implements parsing of the IDIV instruction operand so that
we can find whether an integer division error was a division by zero or
an integer overflow.
It replaces the previous functionality that was implemented for OSX only
and was incomplete (it didn't handle memory operands) and partially incorrect
(not handling R8..R15 properly).

Add DAC Notes Book of the Runtime doc

Add Threading Book of the Runtime doc

Simplify a bit of logic in GetTempPathA

Hardcode CP_ACP to UTF-8 on Linux

Previously, on Linux, the PAL would attempt to detect the current codepage via
setlocale and maintained a mapping from LC_TYPES to Win32 locale names.
On OSX, we didn't do any of this, instead we always used UTF-8 as the
current codepage.

This change moves to this model on Linux as well, something we want to
do because it will enable us to use Ansi marshalling during PInvoke to
marshall UTF-8 data to native code (which is common when interoping with
existing native libraries on Linux) and allows us to remove a bunch of
code we don't want to carry forward from the PAL.

Merge pull request #730 from josteink/freebsd-linking

CoreClr: Fix FreeBSD linking-issues.

Add Profiling Book of the Runtime doc

Add Exceptions Book of the Runtime doc

Add Type System Book of the Runtime doc

Merge pull request #734 from josteink/freebsd-fpregs

Fix up final FPU-registers for FreeBSD x86_64.

Fix up final FPU-registers for FreeBSD x86_64.

FreeBSD now builds without compilation errors. For FreeBSD and glory!

                ,        ,
               /(        )`
               \ \___   / |
               /- _  `-/  '
              (/\/ \ \   /\
              / /   | `    \
              O O   ) /    |
              `-^--'`<     '
             (_.)  _  )   /
              `.___/`    /
                `-----' /
   <----.     __ / __   \
   <----|====O)))==) \) /====
   <----'    `--' `.__,' \
                |        |
                 \       /       /\
            ______( (_  / \______/
          ,'  ,-----'   |
          `--{__________)

This closes https://github.com/dotnet/coreclr/issues/594#issuecomment-93995183

Make GetTempPathA/W first check TMPDIR

Today GetTempPathA/W are hardcoded to return /tmp/.  This commit makes them first consult the TMPDIR environment variable, using its value if it exists.

This also overhauls the tests for GetTempPathA/W, which were out-of-date with regards to the existing implementation and which were excluded from PAL test runs.

CoreClr: Fix FreeBSD linking-issues.

This fix brings the build almost up to 100% (from 35%), where it fails due to PIC-errors.

Merge pull request #729 from mikem8361/clrstackfix

The ICorDebug version of sos ClrStack command stopped working

Merge pull request #723 from janvorli/fix-dangling-else

Fix dangling else warnings

The ICorDebug version of sos ClrStack command ("sos ClrStack -i") stopped working.  The existing was enumerating

all the modules in the target process to find the coreclr module, but under linux we just picked the first module found
because given just the base address of the module, it isn't easy to determine correct module given the context and
constrains the debugshim code ran.  We were lucky and picked the correct module but now the order has changed
which broke this command.

The fix is to just get the address for coreclr and create the ICorDebug interface from that instead of enumerating all the modules.

Merge pull request #701 from janvorli/fix-freebsd-rc-issue

Fix .rc processing issue on FreeBSD

Merge pull request #726 from janvorli/fix-copyright-headers

Add missing MIT license headers

Fix the return macro on Windows

There seems to be a bug in the VC++ compiler that causes the
following error to occur:
ex.cpp(492): fatal error C1061: compiler limit : blocks nested too deeply
There seems to be no nesting when I look at preprocessed source of the ex.cpp
and it builds fine on Unix.
To make the Windows build happy, I have left the return macro definition
for MSVC as it was before my change and only changed it for other compilers.

Merge pull request #721 from pgavlin/CorWarnings

Fix two sources of warnings in cor{,info}.h

Merge pull request #724 from ellismg/use-libcoreclr-for-some-apis

PInvoke to libcoreclr for three APIs

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

Merge changes from TFS

Add missing MIT license headers

.rc and .def files were missing the MIT license headers, so this
change adds them.

PInvoke to libcoreclr for three APIs

After pulling the PAL out of the runtime, we redirected all the PInvokes
which were going to libcoreclr to libcoreclrpal. It turns out that
CoTaskMemAlloc, CoTaskMemFree and CoCreateGuid are still exported by the
runtime itself and not the PAL.

Update the three PInvokes for these functions from managed code to point
at the runtime itself.

Fix dangling else warnings

This change fixes all dangling else warnings and removes the
compiler flag that was disabling their reporting.

Merge pull request #691 from josteink/freebsd-threading

PAL: Fix threading on FreeBSD.

Fix for the build break. I had moved UAP Host under a folder whcih is built only for x86. I'm changing the dirs.proj's to build the UAPHost for all architectures.
I verified that UAPHost gets built on x86coresys and armcoresys, whille VSIntegration and MSBuildTask folders gets built only on x86coresys.

[tfs-changeset: 1454192]

Fix two sources of warnings in cor{,info}.h

- cor.h was incorrectly casting a pointer-to-const to a
  pointer-to-non-const.
- corinfo.h contained a 0-sized array, which is not standard C++,
  and therefore generates a warning on compilers without MSVC
  extensions enabled. Using a 1-sized array (which is already done
  elsewhere in the same file) prevents this warning.

PAL: Fix threading on FreeBSD.

Merge pull request #719 from ellismg/preload-pal-from-corerun

Explicitly load libcoreclrpal in corerun

Explicitly load libcoreclrpal in corerun

libcoreclr has a dependency on libcoreclrpal, but libcoreclrpal is
usually not on the path the loader probes, since it is SxS with the
runtime itself.

Instead of forcing LD_LIBRARY_PATH to be set just have corerun load the
PAL from the same folder as the runtime before loading libcoreclr.

Fixes #709

Merge pull request #718 from janvorli/cleanup-safecrt

Cleanup safecrt in PAL

Cleanup safecrt in PAL

The safecrt code in PAL was heavily sprinkled with #ifdefs for POSITIONAL_PARAMETERS
(that we never define) and _SAFECRT_IMPL (which is always defined in the PAL code).
This change cleans that up by removing checks of those ifdefs.
Also, I've found that the src/pal/src/safecrt/crtdefs.h is not used anywhere,
so I am removing it as well.

Merge pull request #714 from richlander/rich-readme

Update README to accomodate wiki move

Merge pull request #716 from janvorli/fix-tailcall

Fix issue with the disabled tail calls via helpers

Fix issue with the disabled tail calls via helpers

This change fixes an issue that occured after I've disabled tail calls via
helpers. The problem was that the goto to NO_TAIL_CALL needs to happen
before the GTF_CALL_M_TAILCALL flag is set, otherwise the jitter
doesn't emit any call.

Merge pull request #715 from janvorli/remove-long-double

Remove long double support from safecrt routines in PAL

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

Merge changes from TFS

Remove long double support from safecrt routines in PAL

This change removes the long double support from the safecrt
printf style routines in PAL. The existing support was effectively
disabled anyways, we didn't have similar support in the non-safe
printf style functions and coreclr has no need to print long doubles.

Update README to accomodate wiki move

This change makes System.Runtime.InteropServices.FormatterSerives public and __DynamicallyInvokable to unblock serialization engine that reflects on this type during deserializatoin. This is import to unblock the F5 debugging using coreclr bits.

[tfs-changeset: 1453460]

Merge pull request #712 from bendono/dnx-doc

Fixed several grammar and spelling mistakes.

Fixed several grammar and spelling mistakes.

Merge pull request #662 from richlander/rich-wiki

Move subset of wiki files to coreclr repo

Merge pull request #707 from ellismg/fix-689

Use "x64" and not "amd64" in build.sh

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

Merge changes from TFS

Use "x64" and not "amd64" in build.sh

The help text for build.sh said that you should pass "x64" as a
paramter for the build architecture.  However, the script was looking
for amd64.  Update the script to look for x64 since that's what we use
everywhere else (and the behavior you got if you didn't pass an
argument to this script.)

Fixes #689

Moving UAP Host sources out from coreclr\hosts to where it'll be residing together with other UAP components

[tfs-changeset: 1452859]

Merge pull request #705 from ellismg/mscorlib-pal-rename

Update PAL name

Rename "CoreClrPal" to "coreclrpal"

Simply change the case of the library the PAL lives in so that is all
lowercase.

Update PAL name

As of 19d02824, the Win32 functions exported by the PAL are now in
libCoreClrPal instead of libcoreclr itself.  Update mscorlib's PInvokes
to match this change.

Merge pull request #704 from mikem8361/sharepal

Create shared PAL module and change all the modules to use it

Create shared PAL module and change all the debugger modules and coreclr to use it.

Rename and re-title wiki documents

Change pathing in documentation to match build update

Adding note about installing mono packages built from source and not from the official location

Update Windows instructions

Update instructions to match Linux

Updated Linux instructions

Updated Linux instructions

Rename OSX instructions

Updated Linux instructions

Improve readability of Linux instructions

Initial OS X instructions

Correct test instructions
- Correct running test instructions
- Add links for Linux and Mac OS X builds.

Updated Building and Running CoreCLR on Linux

Updated Building and Running CoreCLR on Linux

Created Testing across repos

Updated Building and Running CoreCLR on Linux

Created Building and Running CoreCLR on Linux

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

Merge changes from TFS

Merge pull request #703 from janvorli/implement-jit-tailcall

Disable JIT_TailCall invocation on Unix

Mirror subfolders for unixcorerun.

My previous change introduced a new subfolder to unixcorerun, but I forgot to update the .gitmirror file to a .gitmirrrorall file.

[tfs-changeset: 1452317]

Merge pull request #697 from mikem8361/dactablefix

Add back security class vtables to dac global table

Fix .rc processing issue on FreeBSD

This change fixes issue that occurs on FreeBSD when compiling the
.rc files into the gettext format. For some reason, the shell
expression evaluation returns the resulting resource id as a string
datatype. That leads to a failure when trying to format the result
as a hex number using sprintf.
The change ensures that if that happens, the type is changed to numeric.

Merge pull request #699 from jkotas/readonlycollections

Enable generic IReadOnlyCollection interfaces on array for Unix