[llvm-commits] new patches for compiling linux with integrated-as

[Eric Christopher]

On Fri, Nov 9, 2012 at 2:32 PM, Chandler Carruth <chandlerc at google.com>wrote:

> On Thu, Nov 8, 2012 at 6:05 AM, Roman Divacky <rdivacky at freebsd.org>
> wrote:
> >> - more .cfi* support: linux makes use of 3 CFI directives not yet
> (fully) supported
> >>   by llvm.
> >>
> >>   .cfi_startproc simple: i added the 'simple' keyword recognition but a
> review is needed.
> >>
> >>   .cfi_undefined: new directive, it's only recognized, someone else
> will have to provide
> >>   a proper implementation in MCStreamer::EmitCFIUndefined.
> >>
> >>   .cfi_register: new directive, i think MCStreamer::EmitCFIRegister
> does what it
> >>   should do, but review (and help with fixing it up :) is needed.
> >
> > I finished this I think, the patch attached.
>
> Maybe send the patch to a fresh email thread?
>
> Also, consider using Phabricator to simplify the review?
> http://llvm.org/docs/Phabricator.html has instructions.
>

I'd appreciate a new thread for sure. If you cc me (and maybe Rafael), one
of us should probably be able to look at it.

-eric
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20121109/60209bf6/attachment.html>