[PATCH] D138846: MC/DC in LLVM Source-Based Code Coverage: LLVM back-end and compiler-rt

Jessica Paquette via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 17 20:27:50 PDT 2023


paquette accepted this revision.
paquette added a comment.
This revision is now accepted and ready to land.

I think a couple parts of this patch need clang-format before pushing, but otherwise LGTM.



================
Comment at: llvm/lib/ProfileData/InstrProfReader.cpp:725
+  // and not others. So if we record 0 bytes, just move on.
+  if (NumBitmapBytes == 0) {
+    return success();
----------------
I think that clang-format will remove the braces here.


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

https://reviews.llvm.org/D138846



More information about the llvm-commits mailing list