vsk added a comment.

Two minor missing items:

- Please include a binary test that verifies llvm-cov can prepare a report for 
a binary containing a Version6 coverage blob (for compatibility testing).
- Please also include a short blurb in docs/CoverageMappingFormat.rst 
explaining the format change.

Everything else looks fine.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95753

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to