[PATCH] D36209: [Bash-autocompletion] Add comment to test so that it is easier to fix

Yuka Takahashi via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 1 22:45:36 PDT 2017


yamaguchi created this revision.

clang/test/Driver/autocomplete.c is a test for --autocomplete, and this
test might break if people add/modify flags or HelpText. So I've add
comment for future developers so that they can fix this file according
to the change they had made.


https://reviews.llvm.org/D36209

Files:
  clang/test/Driver/autocomplete.c


Index: clang/test/Driver/autocomplete.c
===================================================================
--- clang/test/Driver/autocomplete.c
+++ clang/test/Driver/autocomplete.c
@@ -1,3 +1,7 @@
+// Test for --autocompletion flag, which is a API used for shell autocompletion.
+// If this test had broke due to adding/modifying flags or changing HelpText,
+// please modify this file according to the change you have made on flags used in this file.
+
 // RUN: %clang --autocomplete=-fsyn | FileCheck %s -check-prefix=FSYN
 // FSYN: -fsyntax-only
 // RUN: %clang --autocomplete=-std= | FileCheck %s -check-prefix=STD


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D36209.109274.patch
Type: text/x-patch
Size: 620 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170802/65c0fdd3/attachment.bin>


More information about the cfe-commits mailing list