[PATCH] D138847: MC/DC in LLVM Source-Based Code Coverage: llvm-cov visualization

[Petr Hosek via Phabricator via llvm-commits]

phosek added inline comments.


================
Comment at: llvm/include/llvm/ProfileData/Coverage/CoverageMapping.h:249
 
+  using MCDC_Cond_ID = unsigned int;
+  typedef struct MCDCParams {
----------------
I think that `MCDCConditionID` would be a more idiomatic LLVM type name.


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

https://reviews.llvm.org/D138847