<div dir="ltr">I'm happy to have this in the main LLVM repositiory.<div><br></div><div>The version in the test suite should likely stay there because the test suite should be buildable w/o LLVM itself -- it is largely a distinct thing. We re-use lit, but not much else from LLVM, and we wouldn't want to install the benchmark library the way we do lit.</div><div><br></div><div>One interesting point: we should have some way of running the in-tree benchmarks, likely with lit, much like we currently allow running unittests with lit. May be something you want to think about.</div></div><br><div class="gmail_quote"><div dir="ltr">On Sat, Jul 28, 2018 at 4:04 PM Dean Michael Berris via cfe-dev <<a href="mailto:cfe-dev@lists.llvm.org">cfe-dev@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><div dir="auto">I’m a huge fan of having more benchmarks, and support this proposal.</div></div><div><br><div class="gmail_quote"><div dir="ltr">On Sat, 28 Jul 2018 at 2:16 am, Kirill Bobyrev via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@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>As a part of upcoming new Clangd symbol index implementation, we would like to start support benchmarks of different Clangd pieces, such as index queries and code completion.</div><div><br></div><div>There are already two projects in the LLVM tree using google/benchmark library while keeping its source code in-tree: libcxx (libcxx/utils/google-benchmark) and test-suite (test-suite/MicroBenchmarks/libs/benchmark-1.3.0). Storing another copy of benchmark library sources in clang-tools-extra would be unreasonable. We already have google test library in LLVM tree (llvm/utils/unittest/googletest) and it is used across all other subprojects, which looks to be very similar to the benchmark library in terms of reusing it across the projects. I would like to know if putting benchmark library along with googletest would be the best option. At the same time, benchmark library could be updated to the newer version (1.4.1) in the process of pulling it to the main LLVM repository.</div><div><br></div><div>It would be great to get feedback on whether this proposal looks reasonable to the LLVM Community and having benchmark in the llvm/ repository would be the best solution to the described problem.</div><div><div><br></div><div>Kind regards,</div><div>Kirill Bobyrev</div></div></div>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div></div>-- <br><div dir="ltr" class="m_-5259507041924548492gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div>Dean</div></div></div>
_______________________________________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev</a><br>
</blockquote></div>