[PATCH] D42830: Add and link to a testing philosophy document in the developer documentation.

Eric Christopher via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 1 17:18:26 PST 2018


echristo created this revision.
echristo added reviewers: chandlerc, grosbach, dblaikie, rnk, hfinkel, lattner.
Herald added subscribers: mcrosier, sanjoy.

Here's an RFC with patch on documentation explaining the how and why of splitting up tests in the testsuite. Effectively: "Why we don’t emit assembly in the clang testsuite”. It largely elaborates on things already in the developer documentation, but provides a (hopefully) useful example and answers a few questions I've gotten a lot over the years. I wrote this a while back after having to explain for the Nth time why we split things up :)

I've added a few people that probably care about this sort of thing to the review line.

This patch:

- Adds a link to a testing philosophy document in the developer documentation.
- Creates a set of explanatory text in another file explaining our philosophy.


Repository:
  rL LLVM

https://reviews.llvm.org/D42830

Files:
  docs/DeveloperPolicy.rst
  docs/TestingPolicy.rst

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D42830.132515.patch
Type: text/x-patch
Size: 5716 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180202/4609bf9b/attachment.bin>


More information about the llvm-commits mailing list