<div class="gmail_quote">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><br>Thanks for the fast review!<br><br></div><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><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>