[libcxx-commits] [PATCH] D116050: [libc++] Disable coverage with sanitize-coverage=0

[Louis Dionne via Phabricator via libcxx-commits]

ldionne requested changes to this revision.
ldionne added a comment.
This revision now requires changes to proceed.

I'm not super familiar with `-fsanitize-coverage`, but I don't understand why this change is correct. Why is it correct to go from excluding `edge,trace-cmp,indirect-calls,8bit-counters` from `-fsanitize-coverage` to excluding everything? Aren't we basically losing the ability to generate coverage information?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116050