[PATCH] D57982: [SanitizierCoverage] Avoid splitting critical edges when destination is a basic block containing unreachable

[Matt Morehouse via Phabricator via llvm-commits]

morehouse added a comment.

I don't know all the details of when we might have an `UnreachableInst`, but I think in general instrumenting blocks that end in unreachable is unhelpful for fuzzing, since we are about to crash anyway (and will therefore be saving the current input whether we "count" the new coverage or not).

Maybe there are cases where `UnreachablInst` is actually reachable and we should consider instrumenting those with counters, but that is beyond my LLVM knowledge.


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

https://reviews.llvm.org/D57982