<div dir="ltr">Thanks for taking care of this. I wasn't aware that we build compiler-rt separately. I'll resubmit the tool into llvm/tools/sancov.</div><br><div class="gmail_quote"><div dir="ltr">On Wed, Nov 11, 2015 at 7:58 AM Diego Novillo <<a href="mailto:dnovillo@google.com">dnovillo@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">I've reverted this commit in r252744 to restore our builds.</div><div dir="ltr"><div><br></div><div><br></div><div>Diego.</div></div><div class="gmail_extra"><div class="gmail_quote">On Wed, Nov 11, 2015 at 10:13 AM, Chandler Carruth via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br></div></div><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><span><div dir="ltr">On Tue, Nov 10, 2015 at 11:17 PM Alexey Samsonov via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Sorry for joining that late. I'm pretty sure compiler-rt is the wrong place for this tool, and we should move it under llvm/tools/ instead (together with llvm-symbolizer, llvm-cxxdump etc.):</div></div></blockquote><div><br></div></span><div>+1, please fix this ASAP, its breaking how we build and use compiler-rt. We may need to revert this temporarily to make progress.</div><span><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><br></div><div>* we don't include any LLVM headers in compiler-rt libraries (there will be some discussion to allow that for profile library, but there are different reasons).</div><div>actually, we even support standalone build of compiler-rt repository, and there is no guarantee that LLVM source code (or CMake rules definitions) will be available.</div><div>* sancov is a parser/transformer of specific binary format, not a compiler runtime (or resource file that should accompany it).</div><div>* sancov uses LLVM libs, so we want as much LLVM developers as possible to be able to build/test it. Currently it will never be compiled if</div><div>compiler-rt is missing, or sanitizers are not supported on host.</div><div>* LLVM test-suite is more suitable for writing lit tests for sancov: it should be tested on a host architecture like every other tool.</div></div></blockquote><div><br></div></span><div>+100</div></div></div>
<br></blockquote></div></div><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div></div></blockquote></div>