[PATCH] D121457: Clamp negative coverage counters to zero.

Hiral via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 11 04:29:52 PST 2022


Hiralo created this revision.
Hiralo added a reviewer: vsk.
Hiralo added projects: clang, LLVM.
Herald added subscribers: dexonsmith, wenlei, dang, kbarton, nemanjai.
Herald added a project: All.
Hiralo requested review of this revision.
Herald added subscribers: llvm-commits, cfe-commits.

commit 60009ab7a6491095b99acd1445dfe0fc5418cb3f
Author: Hiral Oza <hiral.oza at netapp.com>
Date:   Fri Mar 11 10:19:51 2022 +0530

  Clamp negative coverage counters to zero.
  
  Because there is no locking on counter updates, race conditions may
  cause counters to become negative. Clamp them to zero when exporting
  LCOV reports.
  
  Optionally remove blank lines from coverage
  
  Added -omit-blank-lines option to add Skip regions to the coverage map
  for lines of source code that are blank.
  
  Author: Thom.Harp at netapp.com


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D121457

Files:
  clang/include/clang/Basic/CodeGenOptions.def
  clang/include/clang/Driver/Options.td
  clang/include/clang/Lex/PPCallbacks.h
  clang/include/clang/Lex/Preprocessor.h
  clang/lib/CodeGen/CodeGenPGO.cpp
  clang/lib/CodeGen/CoverageMappingGen.cpp
  clang/lib/CodeGen/CoverageMappingGen.h
  clang/lib/Lex/Preprocessor.cpp
  llvm/tools/llvm-cov/CoverageExporterLcov.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D121457.414636.patch
Type: text/x-patch
Size: 12729 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220311/2bf55bea/attachment.bin>


More information about the llvm-commits mailing list