[PATCH] D66988: [NewPM][Sancov] Make Sancov a Module Pass instead of 2 Passes

2019-09-04 Thread Leonard Chan via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL370971: [NewPM][Sancov] Make Sancov a Module Pass instead of 2 Passes (authored by leonardchan, committed by ). Changed prior to commit: https://reviews.llvm.org/D66988?vs=218008&id=218781#toc Reposito

[PATCH] D66988: [NewPM][Sancov] Make Sancov a Module Pass instead of 2 Passes

2019-09-03 Thread Chandler Carruth via Phabricator via cfe-commits
chandlerc accepted this revision. chandlerc added a comment. This revision is now accepted and ready to land. LGTM This also makes sense to me why it would OOM. I suspect we're building a massive list, and at best we get lucky and they get de-duped later. At worst, this is actually fixing a ser

[PATCH] D66988: [NewPM][Sancov] Make Sancov a Module Pass instead of 2 Passes

2019-09-03 Thread Leonard Chan via Phabricator via cfe-commits
leonardchan added a comment. *ping* Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D66988/new/ https://reviews.llvm.org/D66988 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.o

[PATCH] D66988: [NewPM][Sancov] Make Sancov a Module Pass instead of 2 Passes

2019-08-30 Thread Leonard Chan via Phabricator via cfe-commits
leonardchan added a comment. *ping* This patch also seems to fix an issue with mismatched PC table and inline counters: ./out/Fuzz2/region_deserialize

[PATCH] D66988: [NewPM][Sancov] Make Sancov a Module Pass instead of 2 Passes

2019-08-29 Thread Leonard Chan via Phabricator via cfe-commits
leonardchan created this revision. leonardchan added reviewers: chandlerc, fedor.sergeev, philip.pfaffe. leonardchan added a project: Sanitizers. Herald added subscribers: cfe-commits, hiraditya. Herald added projects: clang, LLVM. This patch merges the sancov module and funciton passes into one m