[PATCH] D144371: [LLVM-COV] Fix an issue: a statement after calling 'assert()' function is wrongly marked as 'not executed'

Ying Yi via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Mar 1 07:38:05 PST 2023


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


================
Comment at: clang/test/CoverageMapping/terminate-statements.cpp:335
+  ( true ? void (0) : abort() );  // CHECK: Gap,File 0, [[@LINE]]:33 -> [[@LINE+1]]:3 = #1
+  ( false ? void (0) : abort() ); // CHECK: Gap,File 0, [[@LINE]]:34 -> [[@LINE+1]]:3 = #2
+  return 0;
----------------
zequanwu wrote:
> For completeness, can you add following two tests:
> ```
> ( true ? abort() : void (0) );
> ( false ? abort() : void (0) );
> ```
Thanks, two tests have been added.



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

https://reviews.llvm.org/D144371



More information about the cfe-commits mailing list