<div dir="ltr">Looks good.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Apr 22, 2014 at 9:57 AM, Justin Bogner <span dir="ltr"><<a href="mailto:mail@justinbogner.com" target="_blank">mail@justinbogner.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">We previously treated ARM separately from the generic Itanium ABI for<br>
initializing guard variables.  This code duplication led to things like<br>
the ARM path missing the memory barrier for threadsafe handling, and a<br>
highly misleading comment about how we were (mis)using the generic ABI<br>
for ARM64 when really it went through the ARM codepath.<br>
<br>
This unifies the two code paths.  Functionally, this changes the ARM and<br>
ARM64 codepath to use one byte loads instead of 4 and 8, respectively,<br>
and adds the missing atomic acquire to these loads.  Other architectures<br>
are unchanged.<br>
<br>
Okay to commit?<br>
<br>
<br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div>