<br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Nov 9, 2012 at 2:32 PM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@google.com" target="_blank">chandlerc@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Thu, Nov 8, 2012 at 6:05 AM, Roman Divacky <<a href="mailto:rdivacky@freebsd.org">rdivacky@freebsd.org</a>> wrote:<br>

>> - more .cfi* support: linux makes use of 3 CFI directives not yet (fully) supported<br>
>>   by llvm.<br>
>><br>
>>   .cfi_startproc simple: i added the 'simple' keyword recognition but a review is needed.<br>
>><br>
>>   .cfi_undefined: new directive, it's only recognized, someone else will have to provide<br>
>>   a proper implementation in MCStreamer::EmitCFIUndefined.<br>
>><br>
>>   .cfi_register: new directive, i think MCStreamer::EmitCFIRegister does what it<br>
>>   should do, but review (and help with fixing it up :) is needed.<br>
><br>
> I finished this I think, the patch attached.<br>
<br>
</div>Maybe send the patch to a fresh email thread?<br>
<br>
Also, consider using Phabricator to simplify the review?<br>
<a href="http://llvm.org/docs/Phabricator.html" target="_blank">http://llvm.org/docs/Phabricator.html</a> has instructions.<br></blockquote><div><br></div><div>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.</div>
<div><br></div><div>-eric </div></div></div>