]> granicus.if.org Git - llvm/commitdiff
[utils] coverage: Add help text about the --restrict flag (NFC)
authorVedant Kumar <vsk@apple.com>
Thu, 9 Feb 2017 19:37:18 +0000 (19:37 +0000)
committerVedant Kumar <vsk@apple.com>
Thu, 9 Feb 2017 19:37:18 +0000 (19:37 +0000)
Passing the --restrict flag to the coverage prep script before other
positional arguments is wrong, because it prevents the argparse module
from telling apart arguments to --restrict versus positional arguments.

Pointed out by Sean Callanan!

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@294616 91177308-0d34-0410-b5e6-96231b3b80d8

utils/prepare-code-coverage-artifact.py

index 726375e899cd741b6e4ce6e131b5802010d689b1..883cdd78049bec04f604a0f44b0f94190d13bd35 100644 (file)
@@ -6,6 +6,9 @@ from __future__ import print_function
 
 - Collate raw profiles into one indexed profile.
 - Generate html reports for the given binaries.
+
+Caution: The positional arguments to this script must be specified before any 
+optional arguments, such as --restrict.
 '''
 
 import argparse
@@ -84,7 +87,8 @@ if __name__ == '__main__':
                        help='Emit a unified report for all binaries')
     parser.add_argument('--restrict', metavar='R', type=str, nargs='*',
                        default=[],
-                       help='Restrict the reporting to the given source paths')
+                       help='Restrict the reporting to the given source paths'
+                   ' (must be specified after all other positional arguments)')
     args = parser.parse_args()
 
     if args.use_existing_profdata and args.only_merge: