[PATCH] D20997: [Coverage] Fix an assertion failure if the definition of an unused function spans multiple files.

Igor Kudrin via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 6 17:05:27 PDT 2016


ikudrin updated the summary for this revision.
ikudrin removed rL LLVM as the repository for this revision.
ikudrin updated this revision to Diff 59806.
ikudrin added a comment.

- Use `StartFileID` and `EndFileID` variables to eliminate redundant calls to `SM.getFileID()`.
- Add comment strings to asserts.


http://reviews.llvm.org/D20997

Files:
  lib/CodeGen/CoverageMappingGen.cpp
  test/CoverageMapping/Inputs/ends_a_scope_only
  test/CoverageMapping/Inputs/starts_a_scope_only
  test/CoverageMapping/unused_function.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D20997.59806.patch
Type: text/x-patch
Size: 4197 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160607/648b19d6/attachment.bin>


More information about the cfe-commits mailing list