aeubanks added a comment. In D80692#2061219 <https://reviews.llvm.org/D80692#2061219>, @leonardchan wrote:
> Would be worthwhile to write a small test that asserts Sancov runs before the > other sanitizers under the new PM? > > Aside from this, LGTM. Asserting that sancov runs before other sanitizers seems like overspecification to me. Really all we want to know is that something like msan + sancov work together, and there are tests for that which we are fixing with this. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D80692/new/ https://reviews.llvm.org/D80692 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
