[PATCH] D14733: [Documentation] Add guidelines for grouping tests together

Davide Italiano via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 16 14:54:33 PST 2015


davide created this revision.
davide added reviewers: joker-eph, echristo.
davide added a subscriber: llvm-commits.

After our discussion on IRC. Please forgive my bad english =)

http://reviews.llvm.org/D14733

Files:
  TestingGuide.rst

Index: TestingGuide.rst
===================================================================
--- TestingGuide.rst
+++ TestingGuide.rst
@@ -240,6 +240,11 @@
 the :doc:`FileCheck tool <CommandGuide/FileCheck>`. *[The usage of grep in RUN
 lines is deprecated - please do not send or commit patches that use it.]*

+It is a good practice to put related tests into a single file rather than
+having a separate file per test. Check if there are files already covering
+your feature and consider adding your code there instead of creating a new
+file.
+
 Extra files
 -----------



-------------- next part --------------
A non-text attachment was scrubbed...
Name: D14733.40348.patch
Type: text/x-patch
Size: 574 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151116/7151f1c3/attachment.bin>


More information about the llvm-commits mailing list