[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