<div dir="ltr"><div class="gmail_quote"><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">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><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><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><div>+100</div></div></div>