[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.


Repository:
  rC Clang

https://reviews.llvm.org/D41374





More information about the cfe-commits mailing list