[llvm] 65abca4 - [MLGO] Fix InlineAdvisor and ModelUnderTrainingRunner after hasValue removal

Aiden Grossman via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 23 19:48:52 PST 2022


Author: Aiden Grossman
Date: 2022-11-24T03:48:34Z
New Revision: 65abca4611256794176b791a1380ad83ccf0741f

URL: https://github.com/llvm/llvm-project/commit/65abca4611256794176b791a1380ad83ccf0741f
DIFF: https://github.com/llvm/llvm-project/commit/65abca4611256794176b791a1380ad83ccf0741f.diff

LOG: [MLGO] Fix InlineAdvisor and ModelUnderTrainingRunner after hasValue removal

Recentlyin 4b6b248, llvm::Optional's hasValue method was removed as
described in
https://discourse.llvm.org/deprecating-llvm-optional-x-hasvalue-getvalue-getvalueor
This breaks InlineAdvisor and ModelUnderTrainingRunner. This patch fixes
them by changing the method to has_value, which hasValue was evaluating
to before.

Differential Revision: https://reviews.llvm.org/D138635

Added: 
    

Modified: 
    llvm/lib/Analysis/InlineAdvisor.cpp
    llvm/lib/Analysis/ModelUnderTrainingRunner.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Analysis/InlineAdvisor.cpp b/llvm/lib/Analysis/InlineAdvisor.cpp
index 3fafc3057a13..3642d316c0fb 100644
--- a/llvm/lib/Analysis/InlineAdvisor.cpp
+++ b/llvm/lib/Analysis/InlineAdvisor.cpp
@@ -217,7 +217,7 @@ bool InlineAdvisorAnalysis::Result::tryCreate(
     Advisor =
         llvm::getDevelopmentModeAdvisor(M, MAM, [&FAM, Params](CallBase &CB) {
           auto OIC = getDefaultInlineAdvice(CB, FAM, Params);
-          return OIC.hasValue();
+          return OIC.has_value();
         });
 #endif
     break;

diff  --git a/llvm/lib/Analysis/ModelUnderTrainingRunner.cpp b/llvm/lib/Analysis/ModelUnderTrainingRunner.cpp
index dc149f326271..2fe69cd16d58 100644
--- a/llvm/lib/Analysis/ModelUnderTrainingRunner.cpp
+++ b/llvm/lib/Analysis/ModelUnderTrainingRunner.cpp
@@ -40,7 +40,7 @@ ModelUnderTrainingRunner::ModelUnderTrainingRunner(
 
 void *ModelUnderTrainingRunner::evaluateUntyped() {
   LastEvaluationResult = Evaluator->evaluate();
-  if (!LastEvaluationResult.hasValue()) {
+  if (!LastEvaluationResult.has_value()) {
     Ctx.emitError("Error evaluating model.");
     return nullptr;
   }


        


More information about the llvm-commits mailing list