[PATCH] D70571: [Coverage] Emit a gap region to cover switch bodies

Vedant Kumar via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 3 12:16:22 PST 2019


vsk marked an inline comment as done.
vsk added inline comments.


================
Comment at: clang/docs/SourceBasedCodeCoverage.rst:330
+last case ends). This gap region has a zero count: this causes "gap" areas in
+between case statements, which contain no executable code, to appear uncovered.
+
----------------
hans wrote:
> I thought the point is that it //doesn't// appear uncovered in coverage reports? I.e. it will have a zero count, but that's expected.
Yes, I'll make it clear that a zero count is expected here.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D70571/new/

https://reviews.llvm.org/D70571





More information about the cfe-commits mailing list