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.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
The MLGO builders building dev configurations are currently broken due to this. I'm going to land this patch since it's small and I'm reasonably confident it is correct, just pushing the diff because I'm familiar with the workflow and we can rollback to here in case something does go awry.