<div dir="ltr"><br><div>Thank you for the comment. I updated the patch with support for the new CFI directive in MC/MCParser/AsmParser.cpp.</div><div><br></div><div>Please let me know if it is okay to commit.</div><div><br>
</div><div>Thanks,</div><div>Venkatraman</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Aug 29, 2013 at 6:11 PM, Bill Wendling <span dir="ltr"><<a href="mailto:wendling@apple.com" target="_blank">wendling@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Venkatraman,<br>
<br>
One comment, you should also add parsing support for the new CFI directive in lib/MC/MCParser/AsmParser.cpp.<br>
<br>
-bw<br>
<div><div class="h5"><br>
On Aug 28, 2013, at 1:34 PM, Venkatraman Govindaraju <<a href="mailto:venkatra@cs.wisc.edu">venkatra@cs.wisc.edu</a>> wrote:<br>
<br>
> Ping.<br>
><br>
><br>
> On Fri, Aug 9, 2013 at 10:57 PM, Venkatraman Govindaraju <<a href="mailto:venkatra@cs.wisc.edu">venkatra@cs.wisc.edu</a>> wrote:<br>
> Please review the attached patch.<br>
><br>
> This patch enables exception handling (using DwarfCFI) for SPARC.<br>
><br>
> This patch also modifies MC layer so that the SPARC backend can emit ".cfi_window_save" and ".cfi_register" directives, which are required for EH to work correctly in SPARC.<br>
><br>
> Thanks,<br>
> Venkatraman<br>
><br>
</div></div>> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br>
</blockquote></div><br></div>