<div dir="ltr">Would you mind sticking this patch on phabricator? It's pretty big and it'd be easier (at least for me) to review it.  It's located at <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=mQ4LZ2PUj9hpadE3cDHZnIdEwhEBrbAstXeMaFoB9tg&m=bs2KyKT5M_H_b4CqxmuxlIYNOP4Zn_kTyXtORAMdNBc&s=X2ttAt9W_A69gNBtKC0gaBB-9cegDHQwycj01JBv4jg&e=">http://reviews.llvm.org/</a><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Jun 13, 2015 at 9:01 AM, Jeff Muizelaar <span dir="ltr"><<a href="mailto:jmuizelaar@mozilla.com" target="_blank">jmuizelaar@mozilla.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Here’s a work in progress patch that adds support for FPO tables used for unwinding by x86-32 MSVC.<br>
<br>
The basic approach is to add a FPO_INSTRUCTION during X86FrameLowering along with some information in MachineModuleInfo. WinCodeViewLineTables then looks for this instruction and pulls out the relevant information and emits the FPO tables.<br>
<br>
Does this seem like a sane approach?<br>
<br>
Also, I’m using a generic instruction instead of an X86 one. Does this make sense because WinCodeViewLineTables can’t include the X86 Target headers?<br>
<br>
Thanks,<br>
<br>
-Jeff<br>
<br>
<br>_______________________________________________<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" rel="noreferrer" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div>