<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Aug 29, 2014 at 6:18 AM, Jon Roelofs <span dir="ltr"><<a href="mailto:jonathan@codesourcery.com" target="_blank">jonathan@codesourcery.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="">================<br>
Comment at: src/Unwind/Unwind-EHABI.cpp:223<br>
@@ +222,3 @@<br>
+    *len = (((data[1] >> 24) & 0xff) + 1) * 4;<br>
+    data++; // Skip the first word, which is the prel31 offset.<br>
+  } else {<br>
----------------<br>
</div><div class="">piman wrote:<br>
> Mmh, this makes assumptions about the personality routine doesn't it?<br>
> The generic model is supposed to be compatible with a variety of languages and I'm not sure we can assume the generic data is the same as what the C++ personality routine expects, can we?<br>
</div>In the generic model, the unwind opcodes come first, and then the LSDA comes after... this looks correct to me.<br></blockquote><div><br></div><div>Where in the EHABI spec is that defined? AFAICT all it says in section 6.2 is that there's data for the PR, but it doesn't say anything about the contents of that data.</div>
<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<a href="http://reviews.llvm.org/D5112" target="_blank">http://reviews.llvm.org/D5112</a><br>
<br>
<br>
</blockquote></div><br></div></div>