<div dir="ltr">Looks good to me. Thanks for the fix.<div><br></div><div>David</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Nov 20, 2015 at 3:16 PM, Matthias Braun <span dir="ltr"><<a href="mailto:mbraun@apple.com" target="_blank">mbraun@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Xinliang,<br>
<br>
could you please do a post-commit review of r253735. I don't know much about how the compiler-rt build is supposed to work, but I assume new files have to be mentioned in <a href="http://clang_linux.mk" rel="noreferrer" target="_blank">clang_linux.mk</a> and <a href="http://clang_darwin.mk" rel="noreferrer" target="_blank">clang_darwin.mk</a> as well...<br>
<br>
(This is an attempt to fix our internal PGO bots which currently produce:<br>
Undefined symbols for architecture arm64:<br>
  "_llvmWriteProfData", referenced from:<br>
      ___llvm_profile_write_file in libclang_rt.profile_ios.a(InstrProfilingFile.o)<br>
)<br>
<span class="HOEnZb"><font color="#888888"><br>
- Matthias</font></span></blockquote></div><br></div>