Sergiy Kuryata [Wed, 14 Oct 2015 16:36:12 +0000 (09:36 -0700)]
Merge pull request dotnet/coreclr#1763 from kouvel/PerfFix
Add string allocation fast path outside Windows
Commit migrated from https://github.com/dotnet/coreclr/commit/
ba3a1552fdacac4047681e45256edef2599c8b0b
Aditya Mandaleeka [Wed, 14 Oct 2015 00:48:03 +0000 (17:48 -0700)]
Merge pull request dotnet/coreclr#1758 from wtgodbe/failingTests
Removed working tests from failing tests file
Commit migrated from https://github.com/dotnet/coreclr/commit/
d1c27d62267d08a0c1c9be454e896984aa6b55fa
Matt Ellis [Wed, 14 Oct 2015 00:15:36 +0000 (17:15 -0700)]
Merge pull request dotnet/coreclr#1759 from ellismg/format-corefx-code
Format native CoreFX code to follow conventions
Commit migrated from https://github.com/dotnet/coreclr/commit/
a7b40d4017c6487281b36604d810dd4609ccfc20
Koundinya Veluri [Tue, 13 Oct 2015 22:49:03 +0000 (15:49 -0700)]
Add string allocation fast path outside Windows
- On Linux, a microbenchmark is 140% faster with the portable fast path
- On Windows with the asm fast path, the microbenchmark was 300% faster than on Linux before, and is now 65% faster with the portable fast path
- On Windows, the portable fast path is 5% slower than the asm fast path
Commit migrated from https://github.com/dotnet/coreclr/commit/
d7772f3feb5fd09ea841add320f6e2f25642686d
Jose Perez Rodriguez [Tue, 13 Oct 2015 23:12:59 +0000 (16:12 -0700)]
Merge pull request dotnet/coreclr#1669 from joperezr/UseNewBclRewriter
Using DotNetCore version of BclRewriter in mscorlib build
Commit migrated from https://github.com/dotnet/coreclr/commit/
c95df90677aee02bef314acaabaa9027fd4b2c1a
Koundinya Veluri [Tue, 13 Oct 2015 21:04:49 +0000 (14:04 -0700)]
Merge pull request dotnet/coreclr#1755 from kouvel/DisableTests
Disable a couple of tests that are not supported outside Windows
Commit migrated from https://github.com/dotnet/coreclr/commit/
327794d8eefc0692132df13906dc7109c4d1bf40
Matt Ellis [Tue, 13 Oct 2015 19:15:54 +0000 (12:15 -0700)]
Run format-code.sh
Commit migrated from https://github.com/dotnet/coreclr/commit/
22a9ef1feef0c639afe214237d38a0dd0c2d050b
Matt Ellis [Tue, 13 Oct 2015 19:11:50 +0000 (12:11 -0700)]
Add format-code.sh from CoreFX
Commit migrated from https://github.com/dotnet/coreclr/commit/
eb13823904d1a6d0167f101d63e5e68bba0b61b0
Jose Perez Rodriguez [Fri, 2 Oct 2015 16:52:22 +0000 (09:52 -0700)]
Using DotNetCore version of BclRewriter in mscorlib build
Commit migrated from https://github.com/dotnet/coreclr/commit/
cd49a9fada15fefeef6969d1f38fe6575c78533a
William Godbe [Tue, 13 Oct 2015 18:28:07 +0000 (11:28 -0700)]
Removed working tests from failing tests file
Commit migrated from https://github.com/dotnet/coreclr/commit/
e3c820017e959b0f0e6907d8231d6ea5e7f7d6a6
Matt Mitchell [Tue, 13 Oct 2015 18:34:03 +0000 (11:34 -0700)]
Merge pull request dotnet/coreclr#1756 from mmitche/add-arm-arm64-cross
Add arm64/arm cross builds to CI definition
Commit migrated from https://github.com/dotnet/coreclr/commit/
fec8f02328c50dc3143aca629305f8c8a77a635f
Matt Mitchell [Tue, 13 Oct 2015 18:23:48 +0000 (11:23 -0700)]
Add missing quote
Commit migrated from https://github.com/dotnet/coreclr/commit/
3671f922c175766b667fc94fc727bd9a6f69d22c
Matt Mitchell [Tue, 13 Oct 2015 17:44:58 +0000 (10:44 -0700)]
Use the correct comment style
Commit migrated from https://github.com/dotnet/coreclr/commit/
b1fdc32632ea856a47fe9cdd459cf7fd224b843b
Matt Mitchell [Tue, 13 Oct 2015 17:38:23 +0000 (10:38 -0700)]
Add arm64/arm cross builds to CI definition
Commit migrated from https://github.com/dotnet/coreclr/commit/
33298b80ef45790edfaa60f03437ddca95e3ebe8
Koundinya Veluri [Tue, 13 Oct 2015 17:29:10 +0000 (10:29 -0700)]
Disable a couple of tests that are not supported outside Windows
Commit migrated from https://github.com/dotnet/coreclr/commit/
b20924348aa8c40f271f2e58cdb02445662508b8
Jan Vorlicek [Tue, 13 Oct 2015 14:56:52 +0000 (16:56 +0200)]
Merge pull request dotnet/coreclr#1738 from janvorli/fix-cocreate-guid
Implement CoCreateGuid using uuid_generate on Unix
Commit migrated from https://github.com/dotnet/coreclr/commit/
eefbaaa9cfb84a48495199e334b9e8ea13b06646
Jan Vorlicek [Fri, 9 Oct 2015 18:56:43 +0000 (20:56 +0200)]
Implement CoCreateGuid using uuid_generate on Unix
We were generating GUIDs on Unix as random numbers. But that is not
correct since GUIDs have defined structure with bits having specific
meanings. For example, there are four bits that represent the type of
the GUID, which means whether the guid is randomly generated,
name based on SHA1, time based etc.
This change changes the CoCreateGuid to use the uuid_generate function
from the uuid library that should generate well-formed GUIDs on
Linux and OSX and the uuid_create on FreeBSD.
Commit migrated from https://github.com/dotnet/coreclr/commit/
dbd046be21faab7346db55f3dd638162a20f3941
Rama krishnan Raghupathy [Tue, 13 Oct 2015 05:27:21 +0000 (22:27 -0700)]
Merge pull request dotnet/coreclr#1753 from ramarag/lttng_check
Add checks for liblttng-ust-dev at build time
Commit migrated from https://github.com/dotnet/coreclr/commit/
2e70585b98e6371aae2d00f3fcf537de0a3cfdab
Rama [Mon, 12 Oct 2015 23:56:33 +0000 (16:56 -0700)]
Add checks for liblttng-ust-dev at build time
Commit migrated from https://github.com/dotnet/coreclr/commit/
a8a1b6500a316f388b12b8299608d8adaa630b89
Jan Vorlicek [Mon, 12 Oct 2015 22:55:35 +0000 (00:55 +0200)]
Merge pull request dotnet/coreclr#1749 from janvorli/improve-frame-destructor
Improve frame destructor performance
Commit migrated from https://github.com/dotnet/coreclr/commit/
1f43da21666a3310b478917e0e1d888913dfcca3
Koundinya Veluri [Mon, 12 Oct 2015 22:12:11 +0000 (15:12 -0700)]
Merge pull request dotnet/coreclr#1746 from kouvel/PerfFix2
Add some fast paths that are missing outside Windows
Commit migrated from https://github.com/dotnet/coreclr/commit/
9237f3722e767c282981990be8bd47ef188cc917
Jan Vorlicek [Tue, 6 Oct 2015 00:25:36 +0000 (02:25 +0200)]
Improve frame destructor performance
This change makes the Frame destructor access thread local storage much less often,
only in rare case when the Frame was not popped before the destructor is called.
Commit migrated from https://github.com/dotnet/coreclr/commit/
d6a18e3e12074fad765ed1ba055ef3861f85efaf
Koundinya Veluri [Fri, 9 Oct 2015 18:36:25 +0000 (11:36 -0700)]
Add some fast paths that are missing outside Windows
- Allocation fast path for arrays of object elements
- On Linux, a microbenchmark is 25% faster with the portable fast path
- On Windows with the asm fast path, the microbenchmark was 52% faster than on Linux before, and is now 22% faster with the portable fast path
- On Windows, the portable fast path is within 4% slower than the asm fast path
- Allocation fast path for objects
- On Linux, a microbenchmark is 200% faster with the portable fast path
- On Windows with the asm fast path, the microbenchmark was 325% faster than on Linux before, and is now 43% faster with the portable fast path
- On Windows, the portable fast path is within 1% slower than the asm fast path
- Skipped the Box fast path since that seems to be inlined into jitted code using the new object fast path. As a result of adding the new object fast path, boxing perf has also improved outside Windows similarly to above.
Commit migrated from https://github.com/dotnet/coreclr/commit/
2753a8e8f39aad2188d3dc056a4b971323bf4110
Jan Kotas [Mon, 12 Oct 2015 14:24:11 +0000 (07:24 -0700)]
Merge pull request dotnet/coreclr#1739 from jasonwilliams200OK/master
code,pal: Removes unused headers
Commit migrated from https://github.com/dotnet/coreclr/commit/
1a729f8c93d03e305a1d830c9e65ca05185adcd1
Rama krishnan Raghupathy [Mon, 12 Oct 2015 05:28:38 +0000 (22:28 -0700)]
Merge pull request dotnet/coreclr#1598 from ramarag/enablelttng
Enablelttng
Commit migrated from https://github.com/dotnet/coreclr/commit/
404ab59c799f02b25e3ec4f86e886371e89e51bc
Rama [Fri, 9 Oct 2015 01:53:59 +0000 (18:53 -0700)]
This enables Lttng Logging for CoreClr,
The Tracepoint Providers are built as a separate shared object called libcoreclrtraceptprovider and it is
available in the same directory as libcoreclr.so
By Default the ability of Tracing will not be present
To enable the ability of Tracing, the apps need to be run like:
LD_PRELOAD=libcoreclrtraceptprovider.so ./corerun HelloWorld.exe
For now to change Xplat Event Logging mechanism add any events to:
<root>/src/vm/ClrEtwAll.man
Then regenerate files by running :
<root>/src/inc/genXplatLtnng.pl
Conflicts:
Documentation/building/linux-instructions.md
Commit migrated from https://github.com/dotnet/coreclr/commit/
ff328b606c4edad13e9a211a8d89288340952f4c
Peter Jas [Sat, 10 Oct 2015 19:05:53 +0000 (19:05 +0000)]
code,pal: Removes unused headers.
`<typeinfo>` and `<errno.h>` were unused in `seh.cpp`.
On FreeBSD, the `errno` macro was conflicting due to these
stale inclusions. The exact cause is still unknown that how
it is building on CI and only failing locally on clean FreeBSD 10.2
installation with Clang 3.5.
Fixes dotnet/coreclr#1091.
Commit migrated from https://github.com/dotnet/coreclr/commit/
84add09f5aa6a6d179261016a016bf36aa0439b9
Jan Kotas [Sat, 10 Oct 2015 01:46:30 +0000 (18:46 -0700)]
Merge pull request dotnet/coreclr#1733 from kouvel/PerfFix
Add allocation fast path for arrays of value type elements outside Wi…
Commit migrated from https://github.com/dotnet/coreclr/commit/
66b24ad114f991cd882387f1a3bb280c7cd6877f
Steve Harter [Fri, 9 Oct 2015 21:11:25 +0000 (16:11 -0500)]
Merge pull request dotnet/coreclr#1723 from steveharter/FixDefaultLocale
Add support for obtaining default locale in Linux and fix issue with collation not being passed to ICU
Commit migrated from https://github.com/dotnet/coreclr/commit/
53dbf8f90879727308527f16bf1d4b5ef2931bb0
Jan Kotas [Fri, 9 Oct 2015 20:15:39 +0000 (13:15 -0700)]
Merge pull request dotnet/coreclr#1737 from dotnet-bot/from-tfs
Merge changes from TFS
Commit migrated from https://github.com/dotnet/coreclr/commit/
8d80e964182be49e67974c774090c3beac8ed1f5
Steve Harter [Thu, 8 Oct 2015 22:03:53 +0000 (17:03 -0500)]
Add support for obtaining default locale in Linux and fix issue with @collation= not being passed to ICU
Commit migrated from https://github.com/dotnet/coreclr/commit/
e3562c04b5e8a28b7d852ccbca627947b72a947d
Koundinya Veluri [Thu, 8 Oct 2015 21:40:14 +0000 (14:40 -0700)]
Add allocation fast path for arrays of value type elements outside Windows
- A microbenchmark involving byte array allocation was about 200% faster on Windows compared to Linux
- On Windows, using the portable version of the fast path is about 5% slower than using the asm version on the microbenchmark
- On Linux, using the portable fast path improves the microbenchmark perf by 160%
- With the fast path enabled on Linux, the microbenchmark on Windows (with asm fast path) is now about 17% faster than on Linux.
Commit migrated from https://github.com/dotnet/coreclr/commit/
e47ea2c24bab1938791d3c31e6a6510f5b254038
Bruce Forstall [Fri, 9 Oct 2015 15:38:34 +0000 (08:38 -0700)]
Fix contracts so scanRuntime@x86chk task succeeds
[tfs-changeset: 1535699]
Commit migrated from https://github.com/dotnet/coreclr/commit/
250782557977bec590ae0a0a83de01ba3b3d6ec3
Stephen Toub [Fri, 9 Oct 2015 12:21:00 +0000 (08:21 -0400)]
Merge pull request dotnet/coreclr#1719 from stephentoub/sb_direction
Add [Out] to a couple of StringBuilders in DllImports
Commit migrated from https://github.com/dotnet/coreclr/commit/
a717bb49a6e45f162367dcb303eb94eafb7491e5
Stephen Toub [Fri, 9 Oct 2015 12:18:53 +0000 (08:18 -0400)]
Merge pull request dotnet/coreclr#1722 from stephentoub/casing_branches
Improve String.ToLower/ToUpper throughput on Unix
Commit migrated from https://github.com/dotnet/coreclr/commit/
3a0f23d6308b396bf135ce586bfd23fc31f3a83d
Richard L Ford [Fri, 9 Oct 2015 05:16:58 +0000 (22:16 -0700)]
Merge pull request dotnet/coreclr#1721 from richardlford/fixruntimecontract
Fix incorrect runtime contract for FailedAssembly::Initialize
Commit migrated from https://github.com/dotnet/coreclr/commit/
b9920ebea8d8d77b65d0ca10ffdb840f0cd36f06
dotnet-bot [Fri, 9 Oct 2015 01:28:37 +0000 (18:28 -0700)]
Code changes for firing etw events when a pdb is dynamically loaded. This corresponds to the following user story: 1200238. As a profiler developer, I want to access PDB content for dynamic assemblies in the ETW event stream, so that I can show my users source for their callstack frames.
The main function in eventtrace.cpp, EmitCodeSymbols, divides the pdb stream into appropriate chunks and transmits them as events in order. There are helper functions which have been taken from the corresponding code in Desktop (Netfxdev1) to provide a dynamically loaded pdb through the profiler authored by Noah.
The test uses EventToSymbols, which is based on TraceEventParser to turn on the CodeSymbols and ModuleLoad keywords and listens for events that send out pdb chunks. It then concatenates the chunks to create the pdb on disk. The BigPdb.cs program loads a dll and pdb dynamically, which triggers the events by the runtime. Once the pdb is recreated, it is compared with the original to make sure there are no differences and if not, the test is declared as passed. The test also uses an EventSource event to transmit a unique cookie to make sure the pdb created corresponds to the process under test, in case there are mulitple instances of the test running.
[tfs-changeset: 1535373]
Commit migrated from https://github.com/dotnet/coreclr/commit/
39b1ec64c8a771c736dda297c1a4f601b052767d
Jan Kotas [Fri, 9 Oct 2015 00:07:02 +0000 (17:07 -0700)]
Merge pull request dotnet/coreclr#1726 from janvorli/fix-win-coreconsole
Fix Windows CoreConsole property list
Commit migrated from https://github.com/dotnet/coreclr/commit/
70a53424b553e1c6098c239e836b564f9468db52
Jan Vorlicek [Thu, 8 Oct 2015 23:13:14 +0000 (01:13 +0200)]
Fix Windows CoreConsole property list
The property list was broken - the keys list was missing a comma between two entries,
so they got merged into one string. And the values list was missing one value too.
Commit migrated from https://github.com/dotnet/coreclr/commit/
71327d39fd733ad736280606e381f6c058be0fbd
stephentoub [Thu, 8 Oct 2015 21:38:06 +0000 (17:38 -0400)]
Improve string.ToLower/ToUpper perf
Our Unix implementation of changing case is currently slower than our implementation on Windows. In our ChangeCase implementation in System.Globalization.Native that uses ICU, we have a loop that reads each code point, processes it, and writes out the result. That processing involves branching into four cases based on whether we're going to upper or to lower, and whether we're using Turkish or not. By manually hoisting the invariants and loop cloning in order to remove the branches from the inner loop, we can improve the performance of this routine by ~15-20%.
Commit migrated from https://github.com/dotnet/coreclr/commit/
c342e48672993c677897ebcf835b2c22397ec03c
Matt Mitchell [Thu, 8 Oct 2015 22:25:09 +0000 (15:25 -0700)]
Merge pull request dotnet/coreclr#1706 from mmitche/update-badges
Update badges to new job locations
Commit migrated from https://github.com/dotnet/coreclr/commit/
1dafc0bad41d6e393d97400f6d12eb8ce57e3632
Matt Mitchell [Thu, 8 Oct 2015 22:20:29 +0000 (15:20 -0700)]
Merge pull request dotnet/coreclr#1724 from mmitche/reenable-suse
Reenable OpenSuSE PRs
Commit migrated from https://github.com/dotnet/coreclr/commit/
db15cee62a1ddc6fe0ac708984a46da879f13d2c
Matt Mitchell [Thu, 8 Oct 2015 22:19:39 +0000 (15:19 -0700)]
Reenable OpenSuSE PRs
Commit migrated from https://github.com/dotnet/coreclr/commit/
2e5d14a827f4804086a25149e98204274cd20064
Jan Vorlicek [Thu, 8 Oct 2015 20:46:22 +0000 (22:46 +0200)]
Merge pull request dotnet/coreclr#1720 from janvorli/fix-osx-activation-and-exception-collision
Fix OSX hardware exception and activation collision
Commit migrated from https://github.com/dotnet/coreclr/commit/
a0b2e3347ac38961e7c626fea5970d5848facacd
Lakshmi Priya [Thu, 8 Oct 2015 20:32:02 +0000 (13:32 -0700)]
Merge pull request dotnet/coreclr#1689 from Priya91/palstring
Apply StackString for pal MAX_LONGPATH references.
Commit migrated from https://github.com/dotnet/coreclr/commit/
0633a257dd66f6b78bfac6f098c67f2ac11291e8
Richard L Ford [Thu, 8 Oct 2015 20:21:14 +0000 (13:21 -0700)]
Fix incorrect runtime contract for FailedAssembly::Initialize
FailedAssembly::Initialize may call CLRException::GetHR.
CLRException::GetHR is marked GC_TRIGGERS.
FailedAssembly::Initialize has to be marked GC_TRIGGERS
as well.
Commit migrated from https://github.com/dotnet/coreclr/commit/
36a36a7b4fcf050525351b0f758512fa2887b3ca
Jan Kotas [Thu, 8 Oct 2015 19:48:41 +0000 (12:48 -0700)]
Merge pull request dotnet/coreclr#1718 from stephentoub/stringcasing_allocations
Reduce allocations in string.ToLower/ToUpper on Unix
Commit migrated from https://github.com/dotnet/coreclr/commit/
0eb192ec5d67810bae92e265910d4a177d650a93
Jan Vorlicek [Thu, 8 Oct 2015 19:34:33 +0000 (21:34 +0200)]
Fix OSX hardware exception and activation collision
This change fixes a collision between hardware exception handling and activation
injection. Both require creating helper frame on stack and patching the target thread
context to inject invocation of a handler. When these two happened together,
they both tried to update the context and create helper frame on stack at the same
time, which lead to various issues including PAL_SEHException escaping from
the host application unhandled.
Commit migrated from https://github.com/dotnet/coreclr/commit/
278285afb2934d14ba12e68052b387fc29ef2158
Aditya Mandaleeka [Thu, 8 Oct 2015 19:20:23 +0000 (12:20 -0700)]
Merge pull request dotnet/coreclr#1683 from krixalis/master
Typos in qcall.h
Commit migrated from https://github.com/dotnet/coreclr/commit/
7582d1120bdab2bafcb5d5f05cdb9bea9deba40a
stephentoub [Thu, 8 Oct 2015 18:50:28 +0000 (14:50 -0400)]
Fix additional StringBuilder directions in corefx Unix code
Commit migrated from https://github.com/dotnet/coreclr/commit/
e25567bf1f2ecd0ecde6dcffc29c519fa941d3c7
Mike McLaughlin [Thu, 8 Oct 2015 18:35:30 +0000 (11:35 -0700)]
Merge pull request dotnet/coreclr#1711 from mikem8361/unhand
Fix unhandled exception debugger notification
Commit migrated from https://github.com/dotnet/coreclr/commit/
0c06df1d4c2b4759c63be7da83f79c1270ce0fa0
Lakshmi Priya Sekar [Wed, 7 Oct 2015 18:04:19 +0000 (11:04 -0700)]
Add Null check to lib_name before getting string length.
Commit migrated from https://github.com/dotnet/coreclr/commit/
4da174fb0d41eaa64455855a0473d0d942e73834
stephentoub [Thu, 8 Oct 2015 17:14:21 +0000 (13:14 -0400)]
Add [Out] to a couple of StringBuilders in DllImports
I audited all of the StringBuilders being used in P/Invokes in mscorlib. Almost all of them were correctly annotated as to their direction of needed marshaling. Two weren't.
Commit migrated from https://github.com/dotnet/coreclr/commit/
1d4061dd6455c5b41a8e9a012914115e29cc2008
stephentoub [Thu, 8 Oct 2015 15:51:14 +0000 (11:51 -0400)]
Reduce allocations in string.ToLower/ToUpper on Unix
On Windows, ToLower/ToUpper calls into InternalChangeCaseString in the runtime which does two optiizations:
- a 0-length check to just return an empty string if the source is empty
- allocates the string and writes the results into it directly
In our current Unix implementation, the ChangeCase implementation doesn't do either of these, no special-casing empty strings and first allocating a char[] into which the results are written and then constructing a string from that.
This commit brings the Unix implementation more in line with the Windows one, adding the 0-length check, and writing the results directly into the result string so as to avoid the unnecessary char[] allocation and copy.
Commit migrated from https://github.com/dotnet/coreclr/commit/
fbdbfa1241139848a38c66d2d3313529b30fc8b8
Jan Vorlicek [Thu, 8 Oct 2015 13:35:56 +0000 (15:35 +0200)]
Merge pull request dotnet/coreclr#1715 from janvorli/add-unhandled-exception-trap
Add high level unhandled exception trap
Commit migrated from https://github.com/dotnet/coreclr/commit/
3e82c94cafb35cef213665153ae59058b31dc368
Jan Vorlicek [Thu, 8 Oct 2015 12:41:56 +0000 (14:41 +0200)]
Add high level unhandled exception trap
After the recent exception handling changes, we were missing a high level
exception trap in the ExecuteAssembly. While it was fine for exceptions
comming from or through managed code, it was a problem for exceptions
happening in the ExecuteAssembly call chain before the managed code was
called.
So for example when ExecuteAssembly was called for assembly that didn't
have proper file access rights for the current user, the PAL_SEHException
leaked out of the host app (corerun) and the C++ runtime displayed
message like this:
libc++abi.dylib: terminating with uncaught exception of type PAL_SEHException
Abort trap: 6
The fix is to put back the unhandled managed exception trap that I've removed
with the recent exception handling changes, but to implement it in a slightly
different way - to contain exception holder.
This way, the call to InternalUnhandledExceptionFilter_Worker can be removed
from the UnwindManagedExceptionPass1, since this new trap would ensure it
is called for both the case when the one in UnwindManagedExceptionPass1 would
kick in and the case that this issue is fixing.
Commit migrated from https://github.com/dotnet/coreclr/commit/
253309471774373c3404818c7be2c5268597ff59
Stephen Toub [Thu, 8 Oct 2015 12:31:36 +0000 (08:31 -0400)]
Merge pull request dotnet/coreclr#1688 from kouvel/LoadFromNativeSearchPathsTest
Add unit test for loading native library from host-provided native se…
Commit migrated from https://github.com/dotnet/coreclr/commit/
c89c375cce7ee1e92c7b9e87f47de8e396b4cad9
Stephen Toub [Thu, 8 Oct 2015 02:37:47 +0000 (22:37 -0400)]
Merge pull request dotnet/coreclr#1702 from stephentoub/marshaling_allocs
Reduce garbage generated in StringBuilder marshaling
Commit migrated from https://github.com/dotnet/coreclr/commit/
8ee9ca8eacd9f9ba705707f23dfb3b381c9f2475
Brian Robbins [Wed, 7 Oct 2015 22:47:02 +0000 (15:47 -0700)]
Merge pull request dotnet/coreclr#1696 from brianrob/framepointers
Enable Frame Pointers for UNIX Builds
Commit migrated from https://github.com/dotnet/coreclr/commit/
b55cc2dfbd1486101d23d829f6e4e1313089ca35
Jan Kotas [Wed, 7 Oct 2015 22:42:37 +0000 (15:42 -0700)]
Merge pull request dotnet/coreclr#1693 from Kagamine/correct-word-spelling
Correct word spelling
Commit migrated from https://github.com/dotnet/coreclr/commit/
ee41080d445be450352370a8cf12a12c5b3aeaec
Jan Kotas [Wed, 7 Oct 2015 22:39:51 +0000 (15:39 -0700)]
Merge pull request dotnet/coreclr#1705 from kyulee1/protojit
Producing standalone Jit for testing
Commit migrated from https://github.com/dotnet/coreclr/commit/
9f2f56a174ad9bfbcdd4d5d02e35738d81fd723e
Mike McLaughlin [Wed, 7 Oct 2015 04:10:39 +0000 (21:10 -0700)]
Fix UnhandledException notifications. The IsDebuggerPresent check was left over from windows coreclr and prevented notifications from being sent.
Commit migrated from https://github.com/dotnet/coreclr/commit/
63f03435150a5deb5dcb94d3eec747e57851dd95
Mike McLaughlin [Wed, 7 Oct 2015 21:43:48 +0000 (14:43 -0700)]
Merge pull request dotnet/coreclr#1697 from mikem8361/osxbp
Fix managed breakpoints on OSX
Commit migrated from https://github.com/dotnet/coreclr/commit/
1a319210d5a3529e53713b60db023c9e123fc03a
stephentoub [Wed, 7 Oct 2015 21:43:00 +0000 (17:43 -0400)]
Address PR feedback
Remove addition of null terminator, which should already be added by ConvertToAnsi.
Commit migrated from https://github.com/dotnet/coreclr/commit/
d828af5feab1842cf8bcb533524f735f2f947767
Matt Mitchell [Wed, 7 Oct 2015 20:13:26 +0000 (13:13 -0700)]
Merge pull request dotnet/coreclr#1710 from mmitche/disable-centos-suse
Disable PRs on SuSE and CentOS until we get more nodes spun up
Commit migrated from https://github.com/dotnet/coreclr/commit/
12b348d224263a21d1c29d6075c3232d17d8d681
Matt Mitchell [Wed, 7 Oct 2015 20:11:20 +0000 (13:11 -0700)]
Disable PRs on SuSE and CentOS until we get more nodes spun up
Commit migrated from https://github.com/dotnet/coreclr/commit/
fe1da99deb2e2add5dfbc14729da3deb8ea9e102
Kyungwoo Lee [Mon, 5 Oct 2015 21:16:32 +0000 (14:16 -0700)]
Producing standalone Jit for testing
protojit.dll (Windows) / libprotojit.so (*nix) is produced in addition, which is a standalone Jit that are not attached to CoreCLR.
Note CoreCLR still embeds such Jit by default same as before to not disrupt the existing clients.
This (same) standalone Jit can be used for other testing purpose and also this can be specified as an altjit as well in CoreCLR.
Added to nuget dev package.
Commit migrated from https://github.com/dotnet/coreclr/commit/
465f56ef0c9c4b125d890e7ac0757e77d54e193c
Eric Erhardt [Wed, 7 Oct 2015 19:15:43 +0000 (14:15 -0500)]
Merge pull request dotnet/coreclr#1645 from eerhardt/Encoding
Implement Encodings on Linux
Commit migrated from https://github.com/dotnet/coreclr/commit/
565c81297f504128cccb7bc69b0a206edfb8e255
Mike McLaughlin [Wed, 7 Oct 2015 18:44:49 +0000 (11:44 -0700)]
Merge remote-tracking branch 'upstream/master' into osxbp
Commit migrated from https://github.com/dotnet/coreclr/commit/
7ab6730f5dc1c5724c9766c52fb743eee7979cfd
Josh Free [Wed, 7 Oct 2015 17:39:54 +0000 (10:39 -0700)]
Merge pull request dotnet/coreclr#1701 from janvorli/fix-osx-hardware-exceptions-2
Add missing file to the previous hardware exceptions fix
Commit migrated from https://github.com/dotnet/coreclr/commit/
b2d00c3774941c59352867b69489f6aadf93b6ac
stephentoub [Wed, 7 Oct 2015 16:48:53 +0000 (12:48 -0400)]
Reduce garbage generated in StringBuilder marshaling
When marshaling a StringBuilder In as Ansi (both done by default on Unix), we currently allocate enough native memory to hold the result. But then we use AnsiCharMarshaler.DoAnsiConversion to allocate a managed byte array and fill it with the converted results, and then we copy those results into the native memory. We can instead just call String's ConvertToAnsi directly, avoiding the extra managed array allocation and the extra memory copy.
Commit migrated from https://github.com/dotnet/coreclr/commit/
bf7a89cb0545b1c1f8d003770b7e8c71786f1ecb
Jan Vorlicek [Wed, 7 Oct 2015 16:45:43 +0000 (18:45 +0200)]
Add missing file to the previous hardware exceptions fix
Commit migrated from https://github.com/dotnet/coreclr/commit/
f4ba38110981f1ee72dad22ce6ffcb490ed70cc5
Matt Mitchell [Wed, 7 Oct 2015 15:41:51 +0000 (08:41 -0700)]
Merge pull request dotnet/coreclr#1692 from mmitche/add-netci
Initial commit of CI definition for inner loop
Commit migrated from https://github.com/dotnet/coreclr/commit/
1f21a82ee179d10882c22952c46617b6c9bc88b4
Matt Mitchell [Tue, 6 Oct 2015 22:56:08 +0000 (15:56 -0700)]
Initial commit of CI definition for inner loop
Commit migrated from https://github.com/dotnet/coreclr/commit/
706ff14711bebbd2fea3d3cccc567626fb0811ae
Eric Erhardt [Mon, 28 Sep 2015 22:41:01 +0000 (17:41 -0500)]
Implement Encodings on Linux
Our current Encodings implementation on Linux is stubbed out and
needs to be fully implemented for CoreClr.
Fix https://github.com/dotnet/corefx/issues/2774.
Commit migrated from https://github.com/dotnet/coreclr/commit/
64ae6f777b94330b8b61511c4a9912ab498971df
Jan Kotas [Wed, 7 Oct 2015 13:26:14 +0000 (06:26 -0700)]
Merge pull request dotnet/coreclr#1699 from janvorli/fix-osx-hardware-exceptions
Fix OSX hardware exception handling
Commit migrated from https://github.com/dotnet/coreclr/commit/
e6c4a4062a722373da88a3d4c61ad4e04f4ffc6c
Jan Vorlicek [Wed, 7 Oct 2015 12:21:36 +0000 (14:21 +0200)]
Fix OSX hardware exception handling
In my recent change that has added the runtime suspension for OSX,
I have also added a fix for the unwind info of the PAL_DispatchExceptionWrapper
function. It turns out that I've made a mistake in the offset in the set_cfa_register
and it has broken hardware exception handling on OSX since the unwinder was not
able to unwind correctly through the wrapper.
It also turns out that the same wrong offset in set_cfa_register is in the
ActivationHelperWrapper.S, but in that function, the unwinding still works correctly.
I've actually verified that with both the wrong and the correct offset, the unwinder
gets the same correct RSP / RBP at the time of the exception. So I believe linked ignored
the DWARF unwind info and used compact unwind info instead that it was able to
derive correctly on its own.
Also, the allocate_stack was incorrectly placed before the set_cfa_register, which again
didn't cause a problem due to the DWARF info being ignored. The issue with this one was
that it updates the CFA, but the CFA offset is relative to RBP at that point and
RBP didn't change.
As an additional fix, there was a problem in PAL_VirtualUnwind that @sergiy-k has
spotted. The return value of the recently added unw_is_signal_frame call was
overwriting the status code returned by the unw_step that we use on OSX to
detect walking out of stack.
Commit migrated from https://github.com/dotnet/coreclr/commit/
f1a03d87c7d43820bca94c0a5a7945e132f3b0e7
Koundinya Veluri [Tue, 6 Oct 2015 08:38:00 +0000 (01:38 -0700)]
Add unit test for loading native library from host-provided native search paths
Also used and added to existing macros for separator chars and removed the new members I had added to the Path class.
Related to dotnet/coreclr#1680
Commit migrated from https://github.com/dotnet/coreclr/commit/
dba31572605bc370b3b9eee403b3d2bb56cc4b56
Matt Ellis [Wed, 7 Oct 2015 06:47:20 +0000 (23:47 -0700)]
Merge pull request dotnet/coreclr#1694 from janvorli/fix-exception-handling-issue
Fix issue in the new exception handling
Commit migrated from https://github.com/dotnet/coreclr/commit/
b6ef71da841d460e2c77e1e75cfc8c1732dafd77
Mike McLaughlin [Tue, 6 Oct 2015 23:44:07 +0000 (16:44 -0700)]
Suppress debug exceptions when running under native (lldb/gdb) debugger.
Commit migrated from https://github.com/dotnet/coreclr/commit/
187ca0f9bce9e113f06beb34d5c5a45aa9c7c0fe
Mike McLaughlin [Tue, 8 Sep 2015 23:26:28 +0000 (16:26 -0700)]
Fix managed breakpoints on OSx.
Changed the OSx exception initialization to enable breakpoints/stepping.
Fix problem with faults and breakpoints in coreclr code not being forwarded to
native debugger (if being debugged).
Fix debugger shutdown where we release the transport too many times and assert in debug.
When building the register context for an exception don't save/restore the debug
registers because they cause an priviledged instruction fault.
Fixed the PAL's TRACE formatting by merging the printfcpp.cpp formatting into
silent_printf.cpp. "%p" wasn't being formatted correctly in a PAL TRACE statement.
The executable heap zone wasn't being set because the process heap
is used instead of creating a new heap.
Commit migrated from https://github.com/dotnet/coreclr/commit/
08f67f0acdb3935acc30b0a0f0ebee21513f2668
Brian Robbins [Tue, 6 Oct 2015 23:36:54 +0000 (16:36 -0700)]
Compile with frame pointers on UNIX builds.
Commit migrated from https://github.com/dotnet/coreclr/commit/
ebc35416c4b57a3415f4f1bbde385e159eb520ca
あまみや ゆうこ [Tue, 6 Oct 2015 23:00:15 +0000 (07:00 +0800)]
correct word spelling
Commit migrated from https://github.com/dotnet/coreclr/commit/
1f940efbe82d6ba5a9c7f8269f087d0709d77011
Jan Kotas [Tue, 6 Oct 2015 22:40:08 +0000 (15:40 -0700)]
Merge pull request dotnet/coreclr#1666 from jasonwilliams200OK/master
cmake: cleanup FreeBSD system include path
Commit migrated from https://github.com/dotnet/coreclr/commit/
287741559a8242acc398ab46b890dd5fbcefb3dd
Jan Kotas [Tue, 6 Oct 2015 22:39:44 +0000 (15:39 -0700)]
Merge pull request dotnet/coreclr#1687 from jasonwilliams200OK/__fastcall-Wmacro-redefined
code: Fix redefined macro warning
Commit migrated from https://github.com/dotnet/coreclr/commit/
6a1093d858037903625bc3581c56658bf83803e3
Lakshmi Priya Sekar [Mon, 5 Oct 2015 18:51:50 +0000 (11:51 -0700)]
Apply StackString for pal MAX_LONGPATH references.
Commit migrated from https://github.com/dotnet/coreclr/commit/
f724894027c9c72483262f17457dac44356cc719
Jan Vorlicek [Tue, 6 Oct 2015 21:46:49 +0000 (23:46 +0200)]
Fix issue in the new exception handling
There was a problem with scanned stack range stored in an exception tracker.
After unwinding a sequence of native frames during the 2nd pass of unwinding,
the scanned stack range was not cleared and referenced stack frames that
were unwound. In some rare cases related to throwing an exception from
a catch handler, this was causing the exception handling to consider stack
frames that accidentally had the same range of addresses as already visited,
which resulted in a wrong exception handler being executed.
Commit migrated from https://github.com/dotnet/coreclr/commit/
72cc25a8ef0c014cb9a899d51126cd00ed5823af
Peter Jas [Tue, 6 Oct 2015 18:13:59 +0000 (18:13 +0000)]
code: Fix redefined macro warning.
For example, in this log:
https://dotnet-ci.cloudapp.net/job/dotnet_coreclr_freebsd_debug_prtest/1082/consoleFull
search for '-Wmacro-redefined'. All 26 occurences are pointing to this line.
Commit migrated from https://github.com/dotnet/coreclr/commit/
6c152931f469c1336f5841d6d7f29fb20cd71308
Sergiy Kuryata [Tue, 6 Oct 2015 19:38:49 +0000 (12:38 -0700)]
Merge pull request dotnet/coreclr#1675 from wtgodbe/clrBugFixes
Fixed small bugs in decodemd.cpp, gcrecord.h, gc.cpp, Strike/util.cpp…
Commit migrated from https://github.com/dotnet/coreclr/commit/
ef9e505c66a82dc6928b8ddf9332a88e675d532d
William Godbe [Tue, 29 Sep 2015 21:06:58 +0000 (14:06 -0700)]
Fixed small bugs in decodemd.cpp, gcrecord.h, gc.cpp, Strike/util.cpp, and strike.cpp, as outlined in this article: viva64.com/en/b/0310/#ID0E5BJM
Commit migrated from https://github.com/dotnet/coreclr/commit/
e3942e300637c6e9a3d098476fafa11ab062642a
Marvin [Mon, 5 Oct 2015 22:05:40 +0000 (00:05 +0200)]
Typos in qcall.h
Apparently both "marshaling" and "marshalling" are valid, but for consistency's sake I opted for what the MSDN uses.
Commit migrated from https://github.com/dotnet/coreclr/commit/
311c9113e1209a886f85acb8b77021df2ea881e9
Matt Ellis [Mon, 5 Oct 2015 22:39:52 +0000 (15:39 -0700)]
Merge pull request dotnet/coreclr#1662 from steveharter/master
fix ICU compile error on CentOS
Commit migrated from https://github.com/dotnet/coreclr/commit/
bc146608854d1db9cdbcc0b08029a87754e12b49
Jan Vorlicek [Mon, 5 Oct 2015 21:21:54 +0000 (23:21 +0200)]
Merge pull request dotnet/coreclr#1677 from janvorli/refactor-exception-handling
Implement refactored Unix managed exception handling
Commit migrated from https://github.com/dotnet/coreclr/commit/
7889c40b6fb68ecd57a53894544f7a3ae111de0f
Koundinya Veluri [Mon, 5 Oct 2015 18:36:55 +0000 (11:36 -0700)]
Merge pull request dotnet/coreclr#1681 from kouvel/FixPathSeparators
Fix loading native libraries from native search paths provided by the…
Commit migrated from https://github.com/dotnet/coreclr/commit/
f7461fed79d952532146891de7c2746758fa9a88
Koundinya Veluri [Sat, 3 Oct 2015 18:45:17 +0000 (11:45 -0700)]
Fix loading native libraries from native search paths provided by the host
Fixed a couple of issues outside Windows regarding the directory separator char, path separator char, and absolute path determination.
Fixes dotnet/coreclr#1680
Commit migrated from https://github.com/dotnet/coreclr/commit/
ca6c4a9e7b6cb893ce1a79442f5048312d7f6357
Steve Harter [Thu, 1 Oct 2015 22:18:00 +0000 (17:18 -0500)]
Fix ICU compile error on CentOS by using DateFormatSymbols::NARROW insead of SHORT as SHORT is not defined in the version 50 of ICU being used by Centos. SHORT was added in ICU 51.
Commit migrated from https://github.com/dotnet/coreclr/commit/
4c11093793d420093e3a0eba51de31b804afc269
Jan Vorlicek [Mon, 5 Oct 2015 15:49:28 +0000 (17:49 +0200)]
Few fixes
This change fixes few issues that I've found:
- The frame popping in the Frame destructor is still needed, so I've put it back.
- There was a problem (I've found that it was there before this change) with the "ex"
variable passed to the StartUnwindingNativeFrames. The local was in frames that were
obsoleted by the StartUnwindingNativeFrames before the ThrowExceptionHelper was called.
And when runtime called malloc to allocate its internal exception data, the stack
of the malloc has overwritten part of the PAL_SEHException. It was not visible before
since that part was in the end of the ContextRecord. But now that I've put the
TargetFrameSp to the end of the PAL_SEHException, it was sometimes getting overwritten.
The fix is to create a copy of the "ex" out of stack.
Commit migrated from https://github.com/dotnet/coreclr/commit/
83c1a657ecac35030d4c1e0204daf12201535874
Jan Vorlicek [Fri, 2 Oct 2015 12:55:07 +0000 (14:55 +0200)]
Implement refactored Unix managed exception handling
This change refactors managed exception handling so that we perform full first
pass over both managed and native frames and only after we find target frame,
we perform the second pass.
This helps in debugging of unhandled managed exceptions since the debugger can
kick in at the end of the first pass when the stack frames are still intact and
it can show the context where the exception was fired.
Commit migrated from https://github.com/dotnet/coreclr/commit/
a723811c171a3dae340a08f24afedbfc238b7930
Matt Mitchell [Fri, 2 Oct 2015 23:17:49 +0000 (16:17 -0700)]
Merge pull request dotnet/coreclr#1672 from kouvel/IncludeTestsInBuildSh
On builds outside Windows, build native test components by default
Commit migrated from https://github.com/dotnet/coreclr/commit/
125fd8145cd2b57e6ad362910583c10b7f8a493d
Koundinya Veluri [Fri, 2 Oct 2015 18:05:54 +0000 (11:05 -0700)]
On builds outside Windows, build native test components by default
Commit migrated from https://github.com/dotnet/coreclr/commit/
ccfbffb17c5555b6c90e25d694777c0bd6a235fa