[PATCH] D83592: [Coverage] Add comment to skipped regions

Zequan Wu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 21 18:02:56 PDT 2020


zequanwu marked an inline comment as done.
zequanwu added inline comments.


================
Comment at: compiler-rt/test/profile/coverage_comments.cpp:9
+    int y = 0; /* comment */           // CHECK:  [[# @LINE]]| 1|  int y = 0; /* comment */
+    int z = 0; // comment              // CHECK:  [[# @LINE]]| 1|  int z = 0; // comment
+    // comment                         // CHECK:  [[# @LINE]]|  |  // comment
----------------
MaskRay wrote:
> I think you can just use `CHECK-NEXT:` for each line and remove duplicated code on the right side. See some recent `gcov-*` tests I added.
Thanks, I removed the duplicated code when fixed a coverage test failure.


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

https://reviews.llvm.org/D83592





More information about the llvm-commits mailing list