[PATCH] D51465: Revamp test-suite documentation

Renato Golin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 30 02:59:34 PDT 2018


rengolin added a comment.

AWESOME!

Very detailed, using the modern infrastructure and helpful even to those not using clang or wanting to run external benchmarks.

I haven't reviewed the whole text (non-English speaker), nor I ran the commands to validate the setup (I'm assuming you did, extensively!:), but this looks right on the money. Thank you!

Feel free to ping me again if no one takes any interest in this, but for the record, overall, LGTM.

Thanks,
--renato



================
Comment at: docs/CMake.rst:606
+Executing the tests
+===================
 
----------------
Do we want at least a hint here that there are more tests? A link to the new page with a short paragraph would do.


Repository:
  rL LLVM

https://reviews.llvm.org/D51465





More information about the llvm-commits mailing list