[llvm] [BOLT] Hash-based function matching (PR #96572)
shaw young via llvm-commits
llvm-commits at lists.llvm.org
Fri Jun 28 09:20:11 PDT 2024
================
@@ -2982,6 +2981,9 @@ void RewriteInstance::selectFunctionsToProcess() {
if (mustSkip(Function))
return false;
+ if (opts::MatchProfileWithFunctionHash)
+ return true;
+
----------------
shawbyoung wrote:
If this check is done in YAMLProfileReader::mayHaveProfileData(), it must also be done in DataAggregator::mayHaveProfileData(), so I think having the check in ShouldProcess makes sense from the perspective of code duplicity, as well as organizationally
https://github.com/llvm/llvm-project/pull/96572
More information about the llvm-commits
mailing list