<div dir="ltr">I committed my patch, it seems to work OK when self-hosting clang for win64 and for building Chromium. I don't actually know if the unwind info is 100% accurate, though, so please try it out and file bugs if the unwinder gets off track in LLVM code.</div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Sep 3, 2014 at 4:57 PM, Reid Kleckner <span dir="ltr"><<a href="mailto:rnk@google.com" target="_blank">rnk@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 dir="ltr">I think this patch works, but it needs additional testing to be confident:<div><a href="http://reviews.llvm.org/D5181" target="_blank">http://reviews.llvm.org/D5181</a><br></div><div><br></div><div>You also need to change this code in Clang to get unwind info emitted:</div>

<div><br></div><div>bool Windows::IsUnwindTablesDefault() const {</div><div>  // FIXME: LLVM's lowering of Win64 data is broken right now.  MSVC's linker</div><div>  // says that our object files provide invalid .pdata contributions.  Until</div>

<div>  // that is fixed, don't ask for unwind tables.</div><div>  return false;</div><div>  //return getArch() == llvm::Triple::x86_64;</div><div>}</div><div><br></div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra">
<br><br><div class="gmail_quote">
On Wed, Sep 3, 2014 at 1:19 PM, Reid Kleckner <span dir="ltr"><<a href="mailto:rnk@google.com" target="_blank">rnk@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 dir="ltr">I intend to take care of this. It's been on my plate for some time, but not as high priority as 32-bit x86 ABI compatibility with MSVC.</div><div><div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Wed, Sep 3, 2014 at 12:44 PM,  <span dir="ltr"><<a href="mailto:hoganmeier@gmail.com" target="_blank">hoganmeier@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Some time ago the patch for Win64 exception handling was finally merged.<br>
There are still serious bugs though and Twobit attached a preliminary patch to that review.<br>
<a href="http://reviews.llvm.org/D4081#52" target="_blank">http://reviews.llvm.org/D4081#<u></u>52</a><br>
<br>
But nothing happened since then.<br>
<br>
I tried to fix it according to rnk's comment "the individual .pdata sections should be comdat associative with the individual .text sections" but don't know enough about the respective llvm internals.<br>



<br>
Could anybody finalize that patch and get this in?<br>
______________________________<u></u>_________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@cs.uiuc.edu" target="_blank">cfe-dev@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/cfe-dev</a><br>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div>