[PATCH] D23160: [Coverage] Prevent creating a redundant counter if a nested body ends with a macro.

Vedant Kumar via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 4 11:44:12 PDT 2016


vsk accepted this revision.
vsk added a comment.
This revision is now accepted and ready to land.

Thanks for catching this! I couldn't really reduce your test case any further. This LGTM.

I guess it never makes sense to have two regions with the exact same start/end loc, and different counters. Do you think we should add assertions in llvm (either in llvm-cov, or in the coverage reader) which guard against this?


https://reviews.llvm.org/D23160





More information about the cfe-commits mailing list