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

[Petr Hosek via Phabricator via llvm-commits]

phosek accepted this revision.
phosek added a comment.

LGTM



================
Comment at: llvm/include/llvm/ProfileData/Coverage/CoverageMapping.h:430
+    // PosToID[].
+    auto it = PosToID.find(Condition);
+    if (it != PosToID.end())
----------------
Nit: This name doesn't follow LLVM naming conventions.


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

https://reviews.llvm.org/D138847