================ @@ -2190,7 +2196,8 @@ void SampleProfileMatcher::findIRAnchors( } } -void SampleProfileMatcher::countMismatchedSamples(const FunctionSamples &FS) { +void SampleProfileMatcher::countMismatchedHashSamples( ---------------- WenleiHe wrote:
Why is the renaming? I can find the name `HashSamples` confusing... Including the original variable names (there seems to be quite some renaming going on in the patch that makes reviewing a bit uneasy...) https://github.com/llvm/llvm-project/pull/79090 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits