Ping! Howard, I'd like you to look at this for style.<div><br></div><div>Nick<br><br><div class="gmail_quote">On 3 June 2011 01:48, Nick Lewycky <span dir="ltr"><<a href="mailto:nlewycky@google.com">nlewycky@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="gmail_quote"><div class="im">On 3 June 2011 01:43, Renato Golin <span dir="ltr"><<a href="mailto:rengolin@systemcall.org" target="_blank">rengolin@systemcall.org</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="border-left:1px solid rgb(204, 204, 204);margin:0pt 0pt 0pt 0.8ex;padding-left:1ex">

<div>On 3 June 2011 08:46, Nick Lewycky <<a href="mailto:nlewycky@google.com" target="_blank">nlewycky@google.com</a>> wrote:<br>
>> The attached patch implements the ARM EABI versions of<br>
>> __cxa_guard_{acquire,release,abort} while trying to share as much code as<br>
>> possible. It appears to me that it's unlikely that this will build correctly<br>
>> on __APPLE__ machines, but that's beyond my ability to deal with here. I<br>
>> have also not actually tested this on an ARM machine, but I did #define<br>
>> LIBCXXABI_ARMEABI and build the tests and those pass on x86-64.<br>
<br>
</div>Hi Nick,<br>
<br>
I also can't tell you about the Apple part (or the get_lock), but the<br>
ARM __cxa_* look correct.<br></blockquote></div><div><br>Thanks for the fast review!<br><br></div><div class="im"><blockquote class="gmail_quote" style="border-left:1px solid rgb(204, 204, 204);margin:0pt 0pt 0pt 0.8ex;padding-left:1ex">




Is there a test for this in Clang? Have you tried at least running them on OSX?<br></blockquote></div><div><br>There's a libcxxabi/test/test_guard.cpp, but it doesn't launch any threads. I have not even built it on OSX.<br>



<br>Nick<br> </div><blockquote class="gmail_quote" style="border-left:1px solid rgb(204, 204, 204);margin:0pt 0pt 0pt 0.8ex;padding-left:1ex">
<br>
cheers,<br>
<font color="#888888">--renato<br>
</font></blockquote></div><br>
</blockquote></div><br></div>