diff options
| author |  hlopko <hlopko@google.com> | 2017-08-07 15:54:03 +0200 |
|---|---|---|
| committer | Marcel Hlopko <hlopko@google.com> | 2017-08-08 13:01:34 +0200 |
| commit | df7cc48622a17907b7f84d0a8e3dc2470d1a8460 (patch) | |
| tree | 0acea1583ece577edd0b45073486d8e8ae221c89 /src/main/java/com/google/devtools/build | |
| parent | fe14baf924532d6077c877a95b103159bc91cd64 (diff) | |
Only add coverage flags when gcno file is expected
Change https://github.com/bazelbuild/bazel/commit/63dabb6cfd55febc14e221ec51b18120558bc23c refactored the coverage feature, but wrongly started
add coverage flags when the gcno file was not expected/read by bazel. This
can be harmful, since the size of inputs increases unnecessarily.
RELNOTES: None.
PiperOrigin-RevId: 164455431
Diffstat (limited to 'src/main/java/com/google/devtools/build')
| -rw-r--r-- | src/main/java/com/google/devtools/build/lib/rules/cpp/CppActionConfigs.java | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppActionConfigs.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/CppActionConfigs.java index 0922c08800..4e8171a5e7 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppActionConfigs.java +++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/CppActionConfigs.java @@ -779,7 +779,6 @@ public class CppActionConfigs { " flag_group {", " flag: '-fprofile-instr-generate'", " flag: '-fcoverage-mapping'", - " flag: '-g'", " }", " }", " flag_set {", @@ -809,9 +808,9 @@ public class CppActionConfigs { " action: 'objc-executable'", " action: 'objc++-executable'", " flag_group {", + " expand_if_all_available: 'gcov_gcno_file'", " flag: '-fprofile-arcs'", " flag: '-ftest-coverage'", - " flag: '-g'", " }", " }", " flag_set {", |