blob: 9e2c1a833795d014b9554d36a8ebbd7685c9d5b6 [file] [log] [blame]
Yuke Liaobb571bd62018-10-31 21:51:521#!/usr/bin/env python
2# Copyright 2018 The Chromium Authors. All rights reserved.
3# Use of this source code is governed by a BSD-style license that can be
4# found in the LICENSE file.
Sajjad Mirzabf9e66a2019-03-07 21:49:065"""Removes code coverage flags from invocations of the Clang C/C++ compiler.
Yuke Liaobb571bd62018-10-31 21:51:526
Sajjad Mirzabf9e66a2019-03-07 21:49:067If the GN arg `use_clang_coverage=true`, this script will be invoked by default.
8GN will add coverage instrumentation flags to almost all source files.
9
10This script is used to remove instrumentation flags from a subset of the source
11files. By default, it will not remove flags from any files. If the option
12--files-to-instrument is passed, this script will remove flags from all files
13except the ones listed in --files-to-instrument.
14
15This script also contains hard-coded exclusion lists of files to never
16instrument, indexed by target operating system. Files in these lists have their
17flags removed in both modes. The OS can be selected with --target-os.
Yuke Liaobb571bd62018-10-31 21:51:5218
19The path to the coverage instrumentation input file should be relative to the
20root build directory, and the file consists of multiple lines where each line
21represents a path to a source file, and the specified paths must be relative to
22the root build directory. e.g. ../../base/task/post_task.cc for build
23directory 'out/Release'.
24
25One caveat with this compiler wrapper is that it may introduce unexpected
26behaviors in incremental builds when the file path to the coverage
27instrumentation input file changes between consecutive runs, so callers of this
28script are strongly advised to always use the same path such as
29"${root_build_dir}/coverage_instrumentation_input.txt".
30
31It's worth noting on try job builders, if the contents of the instrumentation
32file changes so that a file doesn't need to be instrumented any longer, it will
33be recompiled automatically because if try job B runs after try job A, the files
34that were instrumented in A will be updated (i.e., reverted to the checked in
35version) in B, and so they'll be considered out of date by ninja and recompiled.
36
37Example usage:
38 clang_code_coverage_wrapper.py \\
39 --files-to-instrument=coverage_instrumentation_input.txt
40"""
41
Raul Tambre9e24293b2019-05-12 06:11:0742from __future__ import print_function
43
Yuke Liaobb571bd62018-10-31 21:51:5244import argparse
45import os
46import subprocess
47import sys
48
49# Flags used to enable coverage instrumentation.
Sajjad Mirzabf9e66a2019-03-07 21:49:0650# Flags should be listed in the same order that they are added in
51# build/config/coverage/BUILD.gn
Yuke Liao5eff7822019-02-28 03:56:2452_COVERAGE_FLAGS = [
Sajjad Mirza49c00e32019-03-01 22:46:5253 '-fprofile-instr-generate', '-fcoverage-mapping',
54 # Following experimental flags remove unused header functions from the
55 # coverage mapping data embedded in the test binaries, and the reduction
56 # of binary size enables building Chrome's large unit test targets on
57 # MacOS. Please refer to crbug.com/796290 for more details.
58 '-mllvm', '-limited-coverage-experimental=true'
Yuke Liao5eff7822019-02-28 03:56:2459]
Yuke Liaobb571bd62018-10-31 21:51:5260
Sajjad Mirza750bd0b2019-10-16 23:39:5161# Files that should not be built with coverage flags by default.
62_DEFAULT_COVERAGE_EXCLUSION_LIST = []
63
Sajjad Mirzabf9e66a2019-03-07 21:49:0664# Map of exclusion lists indexed by target OS.
65# If no target OS is defined, or one is defined that doesn't have a specific
Sajjad Mirza750bd0b2019-10-16 23:39:5166# entry, use _DEFAULT_COVERAGE_EXCLUSION_LIST.
Sajjad Mirzabf9e66a2019-03-07 21:49:0667_COVERAGE_EXCLUSION_LIST_MAP = {
Yun Liue643df752019-10-25 19:11:3368 'android': [
69 # This file caused webview native library failed on arm64.
70 '../../device/gamepad/dualshock4_controller.cc',
71 ],
Yuke Liao8098d702019-08-05 23:57:4172 'linux': [
73 # These files caused a static initializer to be generated, which
74 # shouldn't.
75 # TODO(crbug.com/990948): Remove when the bug is fixed.
Sajjad Mirza750bd0b2019-10-16 23:39:5176 '../../chrome/browser/media/router/providers/cast/cast_internal_message_util.cc', #pylint: disable=line-too-long
Yuke Liao8098d702019-08-05 23:57:4177 '../../chrome/common/media_router/providers/cast/cast_media_source.cc',
78 '../../components/cast_channel/cast_channel_enum.cc',
Sajjad Mirza750bd0b2019-10-16 23:39:5179 '../../components/cast_channel/cast_message_util.cc',
Yuke Liao8098d702019-08-05 23:57:4180 ],
Sajjad Mirzabf9e66a2019-03-07 21:49:0681 'chromeos': [
82 # These files caused clang to crash while compiling them. They are
83 # excluded pending an investigation into the underlying compiler bug.
84 '../../third_party/webrtc/p2p/base/p2p_transport_channel.cc',
85 '../../third_party/icu/source/common/uts46.cpp',
86 '../../third_party/icu/source/common/ucnvmbcs.cpp',
87 '../../base/android/android_image_reader_compat.cc',
Sajjad Mirza750bd0b2019-10-16 23:39:5188 ],
89 'win': [],
Sajjad Mirzabf9e66a2019-03-07 21:49:0690}
91
92
Sajjad Mirza750bd0b2019-10-16 23:39:5193
Sajjad Mirzabf9e66a2019-03-07 21:49:0694def _remove_flags_from_command(command):
95 # We need to remove the coverage flags for this file, but we only want to
96 # remove them if we see the exact sequence defined in _COVERAGE_FLAGS.
97 # That ensures that we only remove the flags added by GN when
98 # "use_clang_coverage" is true. Otherwise, we would remove flags set by
99 # other parts of the build system.
100 start_flag = _COVERAGE_FLAGS[0]
101 num_flags = len(_COVERAGE_FLAGS)
102 start_idx = 0
103 try:
104 while True:
105 idx = command.index(start_flag, start_idx)
106 start_idx = idx + 1
107 if command[idx:idx+num_flags] == _COVERAGE_FLAGS:
108 del command[idx:idx+num_flags]
109 break
110 except ValueError:
111 pass
Yuke Liaobb571bd62018-10-31 21:51:52112
113def main():
114 # TODO(crbug.com/898695): Make this wrapper work on Windows platform.
115 arg_parser = argparse.ArgumentParser()
116 arg_parser.usage = __doc__
117 arg_parser.add_argument(
118 '--files-to-instrument',
119 type=str,
Yuke Liaobb571bd62018-10-31 21:51:52120 help='Path to a file that contains a list of file names to instrument.')
Sajjad Mirzabf9e66a2019-03-07 21:49:06121 arg_parser.add_argument(
122 '--target-os',
123 required=False,
124 help='The OS to compile for.')
Yuke Liaobb571bd62018-10-31 21:51:52125 arg_parser.add_argument('args', nargs=argparse.REMAINDER)
126 parsed_args = arg_parser.parse_args()
127
Sajjad Mirzabf9e66a2019-03-07 21:49:06128 if (parsed_args.files_to_instrument and
129 not os.path.isfile(parsed_args.files_to_instrument)):
Yuke Liaobb571bd62018-10-31 21:51:52130 raise Exception('Path to the coverage instrumentation file: "%s" doesn\'t '
131 'exist.' % parsed_args.files_to_instrument)
132
133 compile_command = parsed_args.args
Sajjad Mirzabf9e66a2019-03-07 21:49:06134 if not any('clang' in s for s in compile_command):
135 return subprocess.call(compile_command)
136
Sajjad Mirza750bd0b2019-10-16 23:39:51137 target_os = parsed_args.target_os
138
Yuke Liaobb571bd62018-10-31 21:51:52139 try:
140 # The command is assumed to use Clang as the compiler, and the path to the
141 # source file is behind the -c argument, and the path to the source path is
142 # relative to the root build directory. For example:
143 # clang++ -fvisibility=hidden -c ../../base/files/file_path.cc -o \
144 # obj/base/base/file_path.o
Sajjad Mirza750bd0b2019-10-16 23:39:51145 # On Windows, clang-cl.exe uses /c instead of -c.
146 source_flag = '/c' if target_os == 'win' else '-c'
147 source_flag_index = compile_command.index(source_flag)
Yuke Liaobb571bd62018-10-31 21:51:52148 except ValueError:
Sajjad Mirza750bd0b2019-10-16 23:39:51149 print('%s argument is not found in the compile command.' % source_flag)
Yuke Liaobb571bd62018-10-31 21:51:52150 raise
151
Sajjad Mirza750bd0b2019-10-16 23:39:51152 if source_flag_index + 1 >= len(compile_command):
Yuke Liaobb571bd62018-10-31 21:51:52153 raise Exception('Source file to be compiled is missing from the command.')
154
Sajjad Mirzae677802c2019-12-18 21:51:59155 # On Windows, filesystem paths should use '\', but GN creates build commands
156 # that use '/'. We invoke os.path.normpath to ensure that the path uses the
157 # correct separator for the current platform (i.e. '\' on Windows and '/'
158 # otherwise).
159 compile_source_file = os.path.normpath(compile_command[source_flag_index + 1])
Sajjad Mirza750bd0b2019-10-16 23:39:51160 exclusion_list = _COVERAGE_EXCLUSION_LIST_MAP.get(
161 target_os, _DEFAULT_COVERAGE_EXCLUSION_LIST)
Sajjad Mirzabf9e66a2019-03-07 21:49:06162
163 if compile_source_file in exclusion_list:
164 _remove_flags_from_command(compile_command)
165 elif parsed_args.files_to_instrument:
166 with open(parsed_args.files_to_instrument) as f:
167 if compile_source_file not in f.read():
168 _remove_flags_from_command(compile_command)
Yuke Liaobb571bd62018-10-31 21:51:52169
170 return subprocess.call(compile_command)
171
Yuke Liaobb571bd62018-10-31 21:51:52172if __name__ == '__main__':
173 sys.exit(main())