Jan Vorlicek [Tue, 7 Apr 2015 23:50:27 +0000 (01:50 +0200)]
Merge pull request #655 from josteink/freebsd-include
PAL: Use /usr/local/include for includes on FreeBSD.
Eric StJohn [Tue, 7 Apr 2015 22:55:36 +0000 (15:55 -0700)]
Merge pull request #129 from Knagis/master
StreamReader: Cache a single instance of StringBuilder for each reader to improve performance
Matt Mitchell [Tue, 7 Apr 2015 14:56:02 +0000 (07:56 -0700)]
Merge pull request #651 from mmitche/update-coreclr-nuspec-version
Update version of coreclr packages
Jan Vorlicek [Tue, 7 Apr 2015 12:02:35 +0000 (14:02 +0200)]
Merge pull request #656 from Djuffin/dbi-linux
Improve comments and fix Realloc in executable debugger heap
Eugene Zemtsov [Tue, 7 Apr 2015 11:15:19 +0000 (04:15 -0700)]
Improve comments and fix Realloc in executable debugger heap
Jostein Kjønigsen [Tue, 7 Apr 2015 08:40:05 +0000 (08:40 +0000)]
PAL: Use /usr/local/include for includes on FreeBSD.
noahfalk [Mon, 6 Apr 2015 23:38:37 +0000 (16:38 -0700)]
Merge pull request #643 from DickvdBrink/profiler-path
Added CoreClr profiler path for 32 and 64bit
Matt Mitchell [Mon, 6 Apr 2015 23:14:14 +0000 (16:14 -0700)]
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.
Dick van den Brink [Sun, 5 Apr 2015 10:53:06 +0000 (12:53 +0200)]
Added CoreClr profiler path for 32 and 64bit
fixes #601
Eugene Zemtsov [Sat, 4 Apr 2015 06:20:15 +0000 (23:20 -0700)]
Merge pull request #621 from Djuffin/dbi-linux
Make managed debugging on Linux possible
Sergiy Kuryata [Sat, 4 Apr 2015 05:45:44 +0000 (22:45 -0700)]
Merge pull request #627 from josteink/ptrace-freebsd
Fix configuration of ptrace on FreeBSD.
Sergiy Kuryata [Sat, 4 Apr 2015 05:39:57 +0000 (22:39 -0700)]
Merge pull request #624 from josteink/clang35-freebsd
Detect clang35 on FreeBSD.
Stephen Toub [Fri, 3 Apr 2015 16:48:41 +0000 (12:48 -0400)]
Merge pull request #628 from stephentoub/indexofordinal_empty
Tweak IndexOfOrdinal on Unix for empty value
Jostein Kjønigsen [Fri, 3 Apr 2015 11:03:26 +0000 (11:03 +0000)]
Fix configuration of ptrace on FreeBSD.
This closes https://github.com/dotnet/coreclr/issues/625
Stephen Toub [Fri, 3 Apr 2015 12:44:39 +0000 (08:44 -0400)]
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.
Stephen Toub [Fri, 3 Apr 2015 10:52:58 +0000 (06:52 -0400)]
Merge pull request #622 from stephentoub/fix_quirks
Enable UseLatestBehaviorWhenTFMNotSpecified for Unix corerun
Jostein Kjønigsen [Fri, 3 Apr 2015 10:22:48 +0000 (10:22 +0000)]
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
Stephen Toub [Fri, 3 Apr 2015 01:46:26 +0000 (21:46 -0400)]
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.
Jan Kotas [Thu, 2 Apr 2015 23:46:23 +0000 (16:46 -0700)]
Merge pull request #618 from cmckinsey/coreclr55
CQ fix for generic handle lookup in GenericHandleWorker
Mike McLaughlin [Thu, 2 Apr 2015 23:36:28 +0000 (16:36 -0700)]
Merge pull request #620 from mikem8361/dbgshim
Get the dbgshim module to build and enabled all the public APIs.
Mike McLaughlin [Thu, 2 Apr 2015 22:58:43 +0000 (15:58 -0700)]
Fix Windows build.
Mike McLaughlin [Fri, 27 Mar 2015 23:34:05 +0000 (16:34 -0700)]
Get the dbgshim module to build and enabled all the public APIs with simple/dummy implementation for Linux.
Rename CMakeLists.txt file.
Eugene Zemtsov [Sat, 28 Mar 2015 11:42:11 +0000 (04:42 -0700)]
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.
Jan Kotas [Thu, 2 Apr 2015 21:15:29 +0000 (14:15 -0700)]
Merge pull request #617 from jkotas/capturecontext
Avoid implicit forward labels in context2.S
Chris McKinsey [Thu, 2 Apr 2015 20:09:44 +0000 (13:09 -0700)]
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.
Jan Kotas [Thu, 2 Apr 2015 20:04:51 +0000 (13:04 -0700)]
Merge pull request #616 from dotnet-bot/from-tfs
Merge changes from TFS
Jan Kotas [Thu, 2 Apr 2015 19:25:01 +0000 (12:25 -0700)]
Avoid implicit forward labels in context2.S
Jan Kotas [Thu, 2 Apr 2015 19:19:47 +0000 (12:19 -0700)]
Fix x86 build break
[tfs-changeset: 1444911]
Jan Kotas [Thu, 2 Apr 2015 16:44:37 +0000 (09:44 -0700)]
Merge pull request #614 from jkotas/lasterror
Clear last error for SetLastError=true P/Invoke
Matt Mitchell [Thu, 2 Apr 2015 15:53:33 +0000 (08:53 -0700)]
Merge pull request #612 from mmitche/usage-exit-code
Usage message exit code
Jan Kotas [Thu, 2 Apr 2015 03:12:34 +0000 (20:12 -0700)]
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.
Jan Vorlicek [Thu, 2 Apr 2015 00:41:02 +0000 (02:41 +0200)]
Merge pull request #613 from janvorli/fix-sstring-formatmessage
Fix FormatMessageW in PAL
Jan Vorlicek [Wed, 1 Apr 2015 23:31:53 +0000 (01:31 +0200)]
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.
Matt Mitchell [Wed, 1 Apr 2015 23:07:45 +0000 (16:07 -0700)]
Merge branch 'master' into usage-exit-code
Conflicts:
build.cmd
Matt Mitchell [Wed, 1 Apr 2015 23:03:37 +0000 (16:03 -0700)]
Displaying usage message should cause build/test scripts to exit with non-zero code
Jan Vorlicek [Wed, 1 Apr 2015 20:14:24 +0000 (22:14 +0200)]
Merge pull request #599 from janvorli/warnings-cleanup-2a
Fix next round of warning types
Jan Vorlicek [Fri, 27 Mar 2015 22:30:10 +0000 (23:30 +0100)]
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.
Stephen Toub [Wed, 1 Apr 2015 17:59:01 +0000 (13:59 -0400)]
Merge pull request #608 from stephentoub/globalization_def
Ensure FEATURE_COREFX_GLOBALIZATION is defined for the runtime
Stephen Toub [Tue, 31 Mar 2015 23:11:05 +0000 (19:11 -0400)]
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.
Jan Vorlicek [Wed, 1 Apr 2015 08:33:00 +0000 (10:33 +0200)]
Merge pull request #604 from stephentoub/fix_lastindexofordinal_zerovalue
Special-case value.Length == 0 in LastIndexOfOrdinal
Jan Kotas [Wed, 1 Apr 2015 06:29:07 +0000 (23:29 -0700)]
Merge pull request #607 from dotnet-bot/from-tfs
Merge changes from TFS
Jan Kotas [Wed, 1 Apr 2015 05:57:24 +0000 (22:57 -0700)]
Merge pull request #606 from dotnet-bot/from-tfs
Merge changes from TFS
Jan Kotas [Wed, 1 Apr 2015 05:41:38 +0000 (22:41 -0700)]
[tfs-changeset: 1443700]
Jan Kotas [Wed, 1 Apr 2015 05:20:59 +0000 (22:20 -0700)]
Merge pull request #600 from dotnet-bot/from-tfs
Merge changes from TFS
Stephen Toub [Wed, 1 Apr 2015 03:20:44 +0000 (23:20 -0400)]
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.
Matt Mitchell [Tue, 31 Mar 2015 21:31:39 +0000 (14:31 -0700)]
Merge pull request #597 from mmitche/coreclr-paths
Change paths and os names to match corefx
Jan Kotas [Tue, 31 Mar 2015 17:57:10 +0000 (10:57 -0700)]
Merge pull request #493 from AndreyAkinshin/master
Make _gtCostEx and _gtCostSz in gentree.h public
Matt Mitchell [Thu, 26 Mar 2015 16:07:26 +0000 (09:07 -0700)]
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.
Andrey Akinshin [Wed, 18 Mar 2015 09:57:19 +0000 (11:57 +0200)]
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
Sedar Gokbulut [Tue, 31 Mar 2015 00:48:13 +0000 (17:48 -0700)]
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]
Sedar Gokbulut [Tue, 31 Mar 2015 00:44:46 +0000 (17:44 -0700)]
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]
Jan Vorlicek [Mon, 30 Mar 2015 19:10:07 +0000 (21:10 +0200)]
Merge pull request #586 from stephentoub/corerun_cwd
Fix corerun to support argv[0] that's not a full path
Stephen Toub [Mon, 30 Mar 2015 17:56:52 +0000 (13:56 -0400)]
Merge pull request #596 from stephentoub/init_culture_with_invariant
Copy all of invariant culture's data in stubbed out globalization
Jan Vorlicek [Mon, 30 Mar 2015 16:53:09 +0000 (18:53 +0200)]
Merge pull request #595 from janvorli/fix-issue-566
Fix GC exception related issue
Chris McKinsey [Mon, 30 Mar 2015 16:06:22 +0000 (09:06 -0700)]
Merge pull request #591 from cmckinsey/coreclr474
Fix some of the issues found by Viva64 static analyser
Stephen Toub [Mon, 30 Mar 2015 13:28:55 +0000 (06:28 -0700)]
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.
Jan Vorlicek [Mon, 30 Mar 2015 10:06:12 +0000 (12:06 +0200)]
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.
Chris McKinsey [Sat, 28 Mar 2015 18:38:33 +0000 (11:38 -0700)]
Merge pull request #593 from cmckinsey/replay450
Replay the fix for #450
Chris McKinsey [Sat, 28 Mar 2015 18:21:51 +0000 (11:21 -0700)]
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.
Chris McKinsey [Thu, 26 Mar 2015 08:04:05 +0000 (01:04 -0700)]
Fix some of the issues found by Viva64 static analyser
Eliminate some unnecessary null check.
Remove null-checks on this and guard instance calls with null-checks.
Collapse a redundant then/else case.
Stephen Toub [Sat, 28 Mar 2015 04:52:24 +0000 (00:52 -0400)]
Fix corerun to support argv[0] that's not a full path
Unix's corerun tries to parse a path out of argv[0], and blows up
in std::string if no '/' is found. This commit adds a check to make
sure that there was a '/' found, after first expanding argv[0] to an
absolute path.
Jan Kotas [Fri, 27 Mar 2015 21:27:05 +0000 (14:27 -0700)]
Merge pull request #589 from josteink/bsd_configure
Correctly set HAVE_BSD_REGS_T flag during configure.
Jan Kotas [Fri, 27 Mar 2015 21:26:41 +0000 (14:26 -0700)]
Merge pull request #590 from josteink/bsd_regs
PAL: Add register-mapping for FreeBSD x86_64.
Jan Kotas [Fri, 27 Mar 2015 20:51:52 +0000 (13:51 -0700)]
Merge pull request #587 from mikem8361/dbi1
Enabling the ClrStack command ICorDebug option (-i) to use to test DBI.
Jostein Kjønigsen [Fri, 27 Mar 2015 20:45:19 +0000 (20:45 +0000)]
PAL: Add register-mapping fore FreeBSD x86_64.
Jostein Kjønigsen [Fri, 27 Mar 2015 16:19:36 +0000 (16:19 +0000)]
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.
Brian Robbins [Fri, 27 Mar 2015 18:52:04 +0000 (11:52 -0700)]
Merge pull request #569 from brianrob/perfdoc
Add Cross-Platform Performance and Eventing Design Document
Mike McLaughlin [Tue, 24 Mar 2015 00:56:09 +0000 (17:56 -0700)]
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.
Jan Kotas [Fri, 27 Mar 2015 17:56:45 +0000 (10:56 -0700)]
Merge pull request #585 from dotnet/revert-584-bsd_configure
Revert "Correctly set HAVE_BSD_REGS_T flag during configure."
Jan Kotas [Fri, 27 Mar 2015 17:56:38 +0000 (10:56 -0700)]
Revert "Correctly set HAVE_BSD_REGS_T flag during configure."
Brian Robbins [Fri, 27 Mar 2015 17:50:23 +0000 (10:50 -0700)]
Updates from PR comments.
Jan Kotas [Fri, 27 Mar 2015 17:15:21 +0000 (10:15 -0700)]
Merge pull request #584 from josteink/bsd_configure
Correctly set HAVE_BSD_REGS_T flag during configure.
Jostein Kjønigsen [Fri, 27 Mar 2015 16:19:36 +0000 (16:19 +0000)]
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.
Jan Kotas [Fri, 27 Mar 2015 00:15:30 +0000 (17:15 -0700)]
Merge pull request #582 from dotnet-bot/from-tfs
Merge changes from TFS
Jan Kotas [Thu, 26 Mar 2015 23:18:25 +0000 (16:18 -0700)]
Merge pull request #578 from briansull/use-lea-for-mul359
Use lea for multiply by 3, 5 or 9
David Kean [Thu, 26 Mar 2015 23:16:11 +0000 (16:16 -0700)]
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]
Jan Kotas [Thu, 26 Mar 2015 22:36:34 +0000 (15:36 -0700)]
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.
Brian Sullivan [Thu, 26 Mar 2015 00:26:56 +0000 (17:26 -0700)]
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.
Sergiy Kuryata [Thu, 26 Mar 2015 21:00:26 +0000 (14:00 -0700)]
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.
Sergiy Kuryata [Thu, 26 Mar 2015 20:43:10 +0000 (13:43 -0700)]
Merge pull request #580 from Djuffin/dbi-linux
Removing static PAL from SOS and DBI
Stephen Toub [Thu, 26 Mar 2015 20:22:18 +0000 (13:22 -0700)]
Merge pull request #579 from stephentoub/indexof_empty
Fix IndexOfOrdinal on Unix for empty strings
Eugene Zemtsov [Thu, 26 Mar 2015 19:12:12 +0000 (12:12 -0700)]
Make DBI and SOS use PAL from DAC
Eugene Zemtsov [Thu, 26 Mar 2015 19:02:55 +0000 (12:02 -0700)]
Make it possible to write stress log from a module that doesn't have statically liked PAL
Stephen Toub [Thu, 26 Mar 2015 18:49:46 +0000 (11:49 -0700)]
Fix IndexOfOrdinal on Unix for empty strings
IndexOfOrdinal searching for an empty string should return 0. It's currently returning -1.
Jan Kotas [Thu, 26 Mar 2015 15:51:44 +0000 (08:51 -0700)]
Merge pull request #575 from josteink/ptrace
PAL: Fix ptrace-invocation on FreeBSD.
Jostein Kjonigsen [Thu, 26 Mar 2015 07:51:43 +0000 (07:51 +0000)]
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)
Jan Kotas [Thu, 26 Mar 2015 14:23:24 +0000 (07:23 -0700)]
Merge pull request #577 from josteink/runelocale
Fix missing RuneLocale on FreeBSD.
Jan Kotas [Thu, 26 Mar 2015 14:04:16 +0000 (07:04 -0700)]
Merge pull request #576 from janvorli/warnings-cleanup
Fix about 12 kinds of warnings over the codebase
Jan Kotas [Thu, 26 Mar 2015 13:46:37 +0000 (06:46 -0700)]
Merge pull request #573 from josteink/master
Define PAL_CS_NATIVE_DATA_SIZE for FreeBSD x86_64
Jostein Kjønigsen [Thu, 26 Mar 2015 12:21:42 +0000 (12:21 +0000)]
PAL: Improve portability of FreeBSD RuneType-fix.
Jostein Kjønigsen [Thu, 26 Mar 2015 09:38:04 +0000 (09:38 +0000)]
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*);
^
Jan Vorlicek [Thu, 26 Mar 2015 07:19:22 +0000 (08:19 +0100)]
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
Jan Vorlicek [Thu, 26 Mar 2015 07:20:32 +0000 (08:20 +0100)]
Merge pull request #574 from janvorli/jit-warnings-cleanup
Fix accidentally removed initialization of m_VNFunc4Map
Jan Vorlicek [Thu, 26 Mar 2015 06:52:10 +0000 (07:52 +0100)]
Fix accidentally removed initialization of m_VNFunc4Map
During my last change, I've accidentally left out the m_VNFunc4Map
while reordering the initialization list.
Stephen Toub [Thu, 26 Mar 2015 06:45:12 +0000 (23:45 -0700)]
Merge pull request #572 from stephentoub/add_encodings
Update EncodingTable.Unix to support additional encodings
Jostein Kjonigsen [Thu, 26 Mar 2015 06:43:17 +0000 (06:43 +0000)]
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.
Stephen Toub [Thu, 26 Mar 2015 05:56:12 +0000 (22:56 -0700)]
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.
Sergiy Kuryata [Thu, 26 Mar 2015 00:24:41 +0000 (17:24 -0700)]
Merge pull request #570 from Djuffin/cordb_platform_fix
Fix CORDB_PLATFORM mismatch between DAC and SOS
Eugene Zemtsov [Wed, 25 Mar 2015 22:07:01 +0000 (15:07 -0700)]
Fix CORDB_PLATFORM mismatch between DAC and SOS
Jan Kotas [Wed, 25 Mar 2015 23:54:02 +0000 (16:54 -0700)]
Merge pull request #558 from josteink/master
Fix compilation errors on FreeBSD.