<div dir="ltr">Maybe a bug in Phabricator, but for some reason email notifications for this particular issue are no longer being sent out.  So I'm responding here since changes a new revision has been uploaded:<div><br>
</div><div>Most recent summary:</div><div>1) All major review issues addressed</div><div>2) rnk@ spoke to someone from Apple who said their biggest concern is whether it will have any impact on single-threaded compilation performance.</div>
<div>3) Ran tests on MacOSX by compiling sqlite3.c with / without the patch, and saw no noticeable impact on performance, which hopefully satisfies the concern raised in #2</div><div>4) Change was discussed on LLVMDev prior to initial upload to Phabricator, so some support has been built for this already.  If anyone feels like more is necessary before committing, please speak up.</div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jun 9, 2014 at 4:27 PM, Zachary Turner <span dir="ltr"><<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Address review comments.<br>
<div class="HOEnZb"><div class="h5"><br>
<a href="http://reviews.llvm.org/D4076" target="_blank">http://reviews.llvm.org/D4076</a><br>
<br>
Files:<br>
  include/llvm-c/Core.h<br>
  include/llvm/Support/ManagedStatic.h<br>
  include/llvm/Support/Threading.h<br>
  lib/IR/Core.cpp<br>
  lib/Support/ErrorHandling.cpp<br>
  lib/Support/ManagedStatic.cpp<br>
  lib/Support/Threading.cpp<br>
  lib/Support/Timer.cpp<br>
</div></div></blockquote></div><br></div>