[PATCH] D15853: [PGO]: Simplify coverage data lowering code

David Li via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 18 16:58:05 PST 2016


davidxl added a comment.

I missed the review comments earlier.

The local var name is fixed. The getCoverageNamesVarName API name will be fixed later.

Regarding to your question:  the patch does not not change the behavior of the name handling. For all used functions, their names are handled in regular lowering procedure (getOrCreateRegionCounter) so their names will be skipped by lowerCoverageData actually.


http://reviews.llvm.org/D15853





More information about the cfe-commits mailing list