[PATCH] D41374: [Coverage] Fix use-after free in coverage emission
Vedant Kumar via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Dec 18 17:43:57 PST 2017
vsk accepted this revision.
vsk added a comment.
This revision is now accepted and ready to land.
Thanks, this lgtm as a stopgap.
As I mentioned offline, I think the goal should be to have non-modules and modules-enabled builds of a project produce identical coverage reports. I'll look into what exactly we're adding to the deferred decls map while iterating over it, and see if we're dropping any coverage mappings.
More information about the cfe-commits