[libcxxabi][PATCH] unwinder

Nick Kledzik kledzik at apple.com
Mon Oct 7 14:43:41 PDT 2013


On Oct 6, 2013, at 1:03 PM, Howard Hinnant <hhinnant at apple.com> wrote:

> On Oct 3, 2013, at 3:42 PM, Nick Kledzik <kledzik at apple.com> wrote:
> 
>> 
>> On Oct 3, 2013, at 10:47 AM, Dmitri Gribenko <gribozavr at gmail.com> wrote:
>> 
>>> On Thu, Oct 3, 2013 at 10:36 AM, Nick Kledzik <kledzik at apple.com> wrote:
>>>> Years ago Apple wrote its own unwinder for MacOSX and iOS.  To make libcxxabi complete, Apple has decided the source code for its unwinder can be contributed to the open source LLVM libcxxabi project, with a dual licensed under LLVM and MIT license.
>>> 
>>> Great news!
>>> 
>>>> So, I’ve spent some time cleaning up the sources to make them conform with LLVM style
>>> 
>>> Not a review, just style comments:
>>> - please use three slashes for documentation comments that come before
>>> function declarations/definitions
>>> - some files contain tabs (for example, src/Unwind/config.h)
>>> - some files contain trailing whitespace
>>> - src/Unwind/UnwindRegistersRestore.s has some broken ASCII art
>> 
>> I’ve fixed those issues in this patch.
>> 
>> <libunwind2.patch>
> 
> Thanks!  Please commit.
Committed in r192136

-Nick





More information about the cfe-commits mailing list