<div dir="ltr"><div class="gmail_extra">Hey David,</div><div class="gmail_extra"><br></div><div class="gmail_extra">Thanks for the comments,</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jul 26, 2013 at 2:41 AM, David Chisnall <span dir="ltr"><<a href="mailto:David.Chisnall@cl.cam.ac.uk" target="_blank">David.Chisnall@cl.cam.ac.uk</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div id=":17i" style="overflow:hidden">Yes, that's the sort of thing.  c_thread_safety_attributes would probably be a bit more in keeping with the naming convention for __has_feature entries.<br>
</div></blockquote><div><br></div><div>Great! ;D</div><div>I attach the entire patch again with this reply.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div id=":17i" style="overflow:hidden">
On a mostly unrelated issue, we can do clang -E -dM to find out the list of predefined macros, but the only way of finding the valid __has_feature() and __has_extension() values that I know of is to read the source code.  It would be good if there was some command line option to output all of these...</div>
</blockquote></div><div class="gmail_extra"><br></div>I can hack on this over my spare time. Right now, we eagerly look forward to having "c_thread_safety_attributes". Is there anything else you think required for applying the patch?</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">Kind Regards,</div><div class="gmail_extra">Alex Wang,</div><div class="gmail_extra"><br></div></div>