Fixing the public key token that gets used to find System.Uri in System.Runtime during marshaling.

[tfs-changeset: 1451527]

Merge pull request #683 from janvorli/refactor-rctopo-awk

Refactor the windows text resources processing

Merge pull request #687 from zhangkaizhao/master

C++11 requires a space between literal and identifier

Merge pull request #649 from richlander/rich-dnx

Add instructions to create custom DNX with CoreCLR

Add instructions to create custom DNX with CoreCLR

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

Merge changes from TFS

When marshaling System.URI we to and from WinRT we use need to access System.URI members(get_OriginalString, ctor). We currently use TypeName::GetTypeFromAsmQualifiedName to load the type and specify System.dll. The change is to simply use System.Runtime instead.

[tfs-changeset: 1451164]

C++11 requires a space between literal and identifier

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

Merge changes from TFS

Merge pull request #686 from jkotas/persian

Add CAL_PERSIAN to portable System.Globalization

Add CAL_PERSIAN to portable System.Globalization

Merge pull request #685 from jkotas/prebuilt

Update prebuilt ETW headers

Update prebuilt ETW headers

Refactor the windows text resources processing

This change refactors the windows text resource processing into
a parsing script and a script with platform specific function
responsible for writing to the output file.

Merge pull request #684 from libengu/GCTest

Add JIT GC tests

Add JIT GC tests

Selected working JIT GC test cases from old test bed before ToF
for porting into CoreCLR.

Related proj, config, and sln files added and updated. Passed
build, buildtest, runtest.

Cleaned up the source code. Updated comments to remove internal
information such as reference to internal bug numbers etc, removed
some not useful comment. Code is re-formatted for consistent style.
Copyright added. Scrub tool SSS ran. Pragma warning about parenthesis
ignored.

Merge pull request #674 from janvorli/fix-finally-issue

Fix one more exception handling issue

Fix one more exception handling issue

This change fixes exception handling issue that resulted in a finally block being invoked multiple
times in case of a rethrown exception in some specific cases.
The problem was caused by the ExceptionTracker::StackRange::CombineWith that was not handling
correctly the case when the current stack range was empty. It was copying only the upper limit
from the previous range and leaving the lower limit at the default (max) value.
This case happens in the interleaved exception handling when processing the first managed
frame after a block of native frames was unwound.
There is one additional fix in the UnwindManagedExceptionPass1 where we were popping one
frame from the thread's frame list before performing the native frames unwind. This was
not correct since in some cases, there can be more frames that need to be removed.

Merge pull request #647 from josteink/bsd_fpregs

PAL: Preliminary work on FPU/SSE-registers for FreeBSD x86_64.

Merge pull request #675 from janvorli/remove-timezone-from-pal

Remove timezone usage from PAL

Merge pull request #671 from pgavlin/lldbfix

Fix a casing problem in lldbplugin/CMakeLists.txt

Merge pull request #669 from josteink/freebsd-fixes

Fix various FreeBSD build-breakers.

Remove timezone usage from PAL

This change removes all references to the timezone variable, all the functions that
transitively depended on it from PAL and all the related tests.
None of these was used outside of PAL and the tests.

Various FreeBSD fixes.

Merge pull request #666 from richlander/rich-profile

Add Implementing Profilability BOTR doc

Merge pull request #672 from Djuffin/dbi-linux

Use new asm implementation of RtlRestoreContext for EE/debugger

Fix typo in the comment

Preliminary work on FPU/SSE-registers for FreeBSD x86_64.

This partially addresses https://github.com/dotnet/coreclr/issues/594

Use new asm implementation of RtlRestoreContext for EE/debugger on POSIX system

Merge pull request #668 from janvorli/rtl-restore-context-asm

Implement RtlRestoreContext in asm

Fix a casing problem in lldbplugin/CMakeLists.txt

On Linux, liblldb is installed by default as 'liblldb'. lldbplugin's CMake
file is currently looking for libLLDB, which is necessary for OS X. This
change allows either casing.

Merge pull request #663 from janvorli/fix-issue-631

Fix hardware exception handling in interleaved handling case

Merge pull request #664 from janvorli/refactor-pal-seh-registers-access

Refactor context register access in seh-unwind.cpp

Update code files to point to profiler doc

Implement RtlRestoreContext in asm

This change implements the RtlRestoreContext in asm to get consistent and
reliable behavior on all platforms.

Add Implementing Profilability BOTR doc

Refactor context register access in seh-unwind.cpp

This change refactors the native context register access to use the
MCREG_xxx macros so that it can compile on FreeBSD too.

Merge pull request #660 from janvorli/fix-issue-633

Fix exception handling issue (#633)

Fix hardware exception handling in interleaved handling case

The issue is that HandleHardwareException doesn't actually throw an exception, but it calls
directly the DispatchManagedException. In case the exception propagation crosses managed to
native boundary, the exception dispatching code uses throw; to unwind the native frames.
But in this particular case, there was no exception thrown and so the rethrow aborts.
The fix is to add try / catch to the HandleHardwareException that throws the PAL_SEHException
that it got as a parameter, catches it right away and then calls the DispatchManagedException
from the catch handler.

Merge changes from parent branch

[tfs-changeset: 1448103]

Merge pull request #634 from josteink/exceptions

Make FreeBSD handle exceptions like Linux.

Merge pull request #655 from josteink/freebsd-include

PAL: Use /usr/local/include for includes on FreeBSD.

Merge pull request #129 from Knagis/master

StreamReader: Cache a single instance of StringBuilder for each reader to improve performance

Fix exception handling issue (#633)

This change fixes exception handling issue #633. The problem was caused by the
fact that in interleaved exception handling (when exception propagates over
one or more native / managed stack frame boundaries), we were preserving the
m_ScannedStackRange and m_sfResumeStackFrame members of the exception tracker
after unwinding a sequence of native frames and moving on into next block of
managed ones. The m_ScannedStackRange in that case contained addresses of
frames that were already unwound and in case of a rethrown exception, there
could have been new frames in that range. That lead to improper detection
of already processed frames in some cases.
The proper solution is to start with an empty scanned stack range after the
partial unwind that unwinds managed and the adjacent native frames.

Merge pull request #651 from mmitche/update-coreclr-nuspec-version

Update version of coreclr packages

Merge pull request #656 from Djuffin/dbi-linux

Improve comments and fix Realloc in executable debugger heap

Improve comments and fix Realloc in executable debugger heap

PAL: Use /usr/local/include  for includes on FreeBSD.

Merge pull request #643 from DickvdBrink/profiler-path

Added CoreClr profiler path for 32 and 64bit

Update version of coreclr packages

Update to match the version that is consumed in corefx.  This allow's for easier corefx validation using the localpublic.props file.

Make FreeBSD handle exceptions like Linux.

Addresses https://github.com/dotnet/coreclr/issues/626
.

Added CoreClr profiler path for 32 and 64bit
fixes #601

Merge pull request #621 from Djuffin/dbi-linux

Make managed debugging on Linux possible

Merge pull request #627 from josteink/ptrace-freebsd

Fix configuration of ptrace on FreeBSD.

Merge pull request #624 from josteink/clang35-freebsd

Detect clang35 on FreeBSD.

Merge pull request #628 from stephentoub/indexofordinal_empty

Tweak IndexOfOrdinal on Unix for empty value

Fix configuration of ptrace on FreeBSD.

This closes https://github.com/dotnet/coreclr/issues/625

Tweak IndexOfOrdinal for empty value

A System.Runtime test was failing because IndexOf should return the startIndex rather than 0 when passed an empty search value.

Merge pull request #622 from stephentoub/fix_quirks

Enable UseLatestBehaviorWhenTFMNotSpecified for Unix corerun

Detect clang35 on FreeBSD.

clang-3.5 is required to build correctly, but is not detected on FreeBSD
where it is aliased as clang35 and clang++35, not clang-3.5 and clang++-3.5.

This closes https://github.com/dotnet/coreclr/issues/615

Enable UseLatestBehaviorWhenTFMNotSpecified for Unix corerun

While root causing some of our test failures on Linux, I found that
quirks were enabled, causing tests expecting new behavior to fail when
they encountered the old behavior.  This commit modifies the Unix
corerun to include the "UseLatestBehaviorWhenTFMNotSpecified" AppDomain
compat switch (as the Windows version does) so that execution on Unix
picks up the latest behavior rather than falling back to the old quirked
behavior.

Merge pull request #618 from cmckinsey/coreclr55

CQ fix for generic handle lookup in GenericHandleWorker

Merge pull request #620 from mikem8361/dbgshim

Get the dbgshim module to build and enabled all the public APIs.

Fix Windows build.

Get the dbgshim module to build and enabled all the public APIs with simple/dummy implementation for Linux.

Rename CMakeLists.txt file.

Make managed debugging on Linux possible (attach, load, bp, exceptions, stacktrace)

Fixes that were required to achieve that
1. Funnel SIGTRAP into EE debugger.
2. Making that memory allocated by EE debugger is executable.
3. Disable metadata reading optimizations which are not working on Linux.
4. Workaround RtlRestoreContext not properly working with EFlags.
5. Avoid calls to ShimRemoteDataTarget.GetContext on Linux, it is not
implemented and not needed for pure managed debugging.
6. Adjust IP value after breakpoint SIGTRAP to meet debuggers expectations.

Merge pull request #617 from jkotas/capturecontext

Avoid implicit forward labels in context2.S

CQ fix for generic handle lookup in GenericHandleWorker

When looking up the runtime handle in the generic handle cache for a methodtable
we get the declaring methodtable in the hierarchy and then lookup the handle in
the cache from that methodtable. When found we should insert the handle back into
the table using the original methodtable as the key and not the declaring MT
so that later lookups are faster. Inserting the handle back into the table under
the original key was a noop.

Desktop DDR and JITSH testing clean. The test case in Github issue #55 is now
3x faster. Benchmarked roslyn performance which shows no change in performance.

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

Merge changes from TFS

Avoid implicit forward labels in context2.S

Fix x86 build break

[tfs-changeset: 1444911]

Merge pull request #614 from jkotas/lasterror

Clear last error for SetLastError=true P/Invoke

Merge pull request #612 from mmitche/usage-exit-code

Usage message exit code

Clear last error for SetLastError=true P/Invoke

There isn't always a way to know the system function has failed without checking last error, in particular on Unix. Moreover, CLR does not guarantee that the last error is preserved while the managed code is running so this issue cannot be worked around calling SetLastError.

This change adds clearing of last error for P/Invokes with SetLastError=true. The cost of doing so is negligible, it makes error handling for P/Invokes of the above system functions possible, and all the other P/Invokes more robust.

Merge pull request #613 from janvorli/fix-sstring-formatmessage

Fix FormatMessageW in PAL

Fix FormatMessageW in PAL

The function was not properly accessing the Arguments list when the
FORMAT_MESSAGE_ARGUMENT_ARRAY flag was set. The va_list on Linux is
a struct with four members, so indexing Arguments array without
casting it first to a WCHAR** gets completely wrong results.

Merge branch 'master' into usage-exit-code

Conflicts:
build.cmd

Displaying usage message should cause build/test scripts to exit with non-zero code

Merge pull request #599 from janvorli/warnings-cleanup-2a

Fix next round of warning types

Fix next round of warning types

This change fixes the following warnings:
1) Assignment in a condition should be wrapped in ()
2) Priority of && / || should be indicated by parentheses.
3) Unknown #pragma optimize ifdefed out for non MSVC
4) Unused functions deleted or put under #ifdef
5) Extra tokens warning disabling moved to the CMakeLists.txt in the src/inc
6) Self assignment of a member or local variable
7) Assigning ~0 to a bitfield member that was just 8 bit wide

It also fixes a bug in the STRESS_LOGxx macro invocation in exceptionhandling.cpp and
stackwalk.cpp related to recent adding the DBG_ADDR macro usage. This macro expanded
a single parameter into two expressions to extract high / low 32 bits separately.
But the STRESS_LOGxx parameters are passed to the StressLog::LogMsg method as follows:
(void*)(size_t)(data1)
That means that the expanded pair x, y would be inserted as data 1 and that leads
to ignoring the x due to the comma operator.

Merge pull request #608 from stephentoub/globalization_def

Ensure FEATURE_COREFX_GLOBALIZATION is defined for the runtime

Ensure FEATURE_COREFX_GLOBALIZATION is defined for runtime

A bunch of our System.Runtime.Tests.dll, System.Xml.XDocument.Properties.dll,
and other tests were failing with formatting errors.  The runtime tests were
also triggering a runtime assert at shutdown.  It appears that all of these
stem from a mismatch between some data structure layouts in mscorlib and
the runtime, in particular with NumberFormatInfo.

Several fields throughout NumberFormatInfo are included conditionally based
on the  FEATURE_COREFX_GLOBALIZATION flag not being defined.  This flag is
being set for the compilation of mscorlib, but it appears not to be set when
compiling all relevant parts of the runtime, such that the native structure has
extra fields not present in the managed structure.  This is resulting in fields
being misinterpreted by the runtime when using the structure for formatting
(and presumably it's also leading to some potential corruption).

This fix simply adds a define for FEATURE_COREFX_GLOBALIZATION to
the makefile.  The tests that were failing now pass, and the runtime assertion
is no longer firing.

Merge pull request #604 from stephentoub/fix_lastindexofordinal_zerovalue

Special-case value.Length == 0 in LastIndexOfOrdinal

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

Merge changes from TFS

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

Merge changes from TFS

[tfs-changeset: 1443700]

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

Merge changes from TFS

Special-case value.Length == 0 in LastIndexOfOrdinal

With https://github.com/dotnet/coreclr/pull/579, I added to IndexOfOrdinal a check for value.Length == 0, to avoid returning bad data in that case.  But in doing so, and in neglecting to add a similar check to LastIndexOfOrdinal (which is currently implemented in terms of IndexOfOrdinal), I introduced an infinite loop into LastIndexOfOrdinal when value.Length == 0.  Fixing by adding a similar special-case.

Merge pull request #597 from mmitche/coreclr-paths

Change paths and os names to match corefx

Merge pull request #493 from AndreyAkinshin/master

Make _gtCostEx and _gtCostSz in gentree.h public

Change paths and os names to match corefx

* binaries is now bin, intermediates is now obj
* modifying unixmscorlib build to instead be linuxmscorlib + osxmscorlib.
* Change OS=Unix to OS=Linux/OS=OSX and modified properties to mimic corefx's TargetsUnix.

Disable gtCostEx and gtCostSz reporting in compiler.cpp

If you change line

in jit.h and set MEASURE_NODE_SIZE=1, you will get a build error like this:

coreclr\src\jit\compiler.cpp(888): error C2248: 'GenTree::_gtCostEx' : cannot access private member declared in class 'GenTree' [coreclr\binaries\intermediates\Windows_NT.x64.debug\src\jit\ClrJit.vcxproj]
coreclr\src\jit\compiler.cpp(889): error C2248: 'GenTree::_gtCostSz' : cannot access private member declared in class 'GenTree' [coreclr\binaries\intermediates\Windows_NT.x64.debug\src\jit\ClrJit.vcxproj]

Solution: disabling gtCostEx and gtCostSz reporting in compiler.cpp

See also: https://github.com/dotnet/coreclr/pull/493#issuecomment-83070694

Port GC fixes from NetfxDev1:

1121801 X64: Assert mark_stack_array[entry].len == 0 || mark_stack_array[entry].len >= Align(min_obj_size), vm\gc.cpp, Line: 13747 (OriginCS: 1420599)

[tfs-changeset: 1442449]

Port the following fixes to GC from netfxDev1:

1118110 [GC] Assert "chosen_power2 == 0", ndp\clr\src\vm\gc.cpp, Line: 8566 (Origin CS 1410546 - required for the next change)
1119047 ARM heap corruptions found in assert in gc_heap::plan_phase (Origin CS 1414858)

[tfs-changeset: 1442444]

Merge pull request #586 from stephentoub/corerun_cwd

Fix corerun to support argv[0] that's not a full path

Merge pull request #596 from stephentoub/init_culture_with_invariant

Copy all of invariant culture's data in stubbed out globalization

Merge pull request #595 from janvorli/fix-issue-566

Fix GC exception related issue

Merge pull request #591 from cmckinsey/coreclr474

Fix some of the issues found by Viva64 static analyser

Copy all of invariant culture's data in stubbed out globalization

We've been piecemeal copying parts of the invariant culture over to our temporary culture implementation.  But as I've hit additional pieces needed, I decided to just copy over everything so that hopefully we don't bump up against this again.

Fix GC exception related issue

This change fixes a problem that has occured when GC was called from
an exception handler during split exception handling. The problem
manifestated itself as a crash in the HasFrameBeenUnwoundByAnyActiveException
function due to the pInitialExplicitFrame being NULL.
The cause of this issue was caused by a combination of two issues.
First, the m_ScannedStackRange was not reset in the partial 2nd pass to
the state at the beginning of the corresponding partial 1st pass.
And second, the 'unwind has started' flag was not reset after
restoring exception flags in the new exception tracker after switching
from the 2nd back to the 1st pass during the split exception handling.

There was one additional issue, several additional members of the exception
tracker needed to be propagated to the new tracker. These members are
needed by the HasFrameBeenUnwoundByAnyActiveException function.

Since the number of state members that are propagated from the old to the
new exception tracker has grown, I've refactored the code to keep those
in a separate class.