[PATCH] D18289: Attach profile summary information to module

Vedant Kumar via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 24 13:31:20 PDT 2016


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

This lgtm.

IMO we'd get the same coverage if the C file contained `void foo() {}`, but that's just a nit :). It'd be good to follow up with David about what he thinks is lacking in `general.proftext`, but that's for a later commit.


http://reviews.llvm.org/D18289





More information about the cfe-commits mailing list