[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