diff options
| author | Calixte Denizet <cdenizet@mozilla.com> | 2018-09-20 08:53:06 +0000 | 
|---|---|---|
| committer | Calixte Denizet <cdenizet@mozilla.com> | 2018-09-20 08:53:06 +0000 | 
| commit | eb7f60201cb348223d3feec20638b3b084804a26 (patch) | |
| tree | 91714ce01f4b5ac14c50086733f91282dcf0609f /llvm/lib/Transforms/Instrumentation/GCOVProfiling.cpp | |
| parent | 226f3ef5a52db932e4acbcfdf9bb860a25ba6d59 (diff) | |
| download | bcm5719-llvm-eb7f60201cb348223d3feec20638b3b084804a26.tar.gz bcm5719-llvm-eb7f60201cb348223d3feec20638b3b084804a26.zip | |
[IR] Add a boolean field in DILocation to know if a line must covered or not
Summary:
Some lines have a hit counter where they should not have one.
For example, in C++, some cleanup is adding at the end of a scope represented by a '}'.
So such a line has a hit counter where a user expects to not have one.
The goal of the patch is to add this information in DILocation which is used to get the covered lines in GCOVProfiling.cpp.
A following patch in clang will add this information when generating IR (https://reviews.llvm.org/D49916).
Reviewers: marco-c, davidxl, vsk, javed.absar, rnk
Reviewed By: rnk
Subscribers: eraman, xur, danielcdh, aprantl, rnk, dblaikie, #debug-info, vsk, llvm-commits, sylvestre.ledru
Tags: #debug-info
Differential Revision: https://reviews.llvm.org/D49915
llvm-svn: 342631
Diffstat (limited to 'llvm/lib/Transforms/Instrumentation/GCOVProfiling.cpp')
| -rw-r--r-- | llvm/lib/Transforms/Instrumentation/GCOVProfiling.cpp | 3 | 
1 files changed, 2 insertions, 1 deletions
| diff --git a/llvm/lib/Transforms/Instrumentation/GCOVProfiling.cpp b/llvm/lib/Transforms/Instrumentation/GCOVProfiling.cpp index c50102f1f0f..b4da53fd974 100644 --- a/llvm/lib/Transforms/Instrumentation/GCOVProfiling.cpp +++ b/llvm/lib/Transforms/Instrumentation/GCOVProfiling.cpp @@ -593,7 +593,8 @@ void GCOVProfiler::emitProfileNotes() {              continue;            // Artificial lines such as calls to the global constructors. -          if (Loc.getLine() == 0) continue; +          if (Loc.getLine() == 0 || Loc.isImplicitCode()) +            continue;            if (Line == Loc.getLine()) continue;            Line = Loc.getLine(); | 

