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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits