From 69abb15b81d610139f5939cf998d63899b2ffdf2 Mon Sep 17 00:00:00 2001 From: Craig Topper Date: Mon, 4 Mar 2019 04:26:31 +0000 Subject: [PATCH] [SubtargetFeatures] Add operator< for comparing SubtargetInfoKV objects. NFCI Use instead of passing a lambda to std::is_sorted. This is more consistent with SubtargetFeatureKV. llvm-svn: 355300 --- llvm/include/llvm/MC/SubtargetFeature.h | 5 +++++ llvm/lib/MC/MCSubtargetInfo.cpp | 5 +---- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/llvm/include/llvm/MC/SubtargetFeature.h b/llvm/include/llvm/MC/SubtargetFeature.h index 6dc4931..ee27827 100644 --- a/llvm/include/llvm/MC/SubtargetFeature.h +++ b/llvm/include/llvm/MC/SubtargetFeature.h @@ -100,6 +100,11 @@ struct SubtargetInfoKV { bool operator<(StringRef S) const { return StringRef(Key) < S; } + + /// Compare routine for std::is_sorted. + bool operator<(const SubtargetInfoKV &Other) const { + return StringRef(Key) < StringRef(Other.Key); + } }; //===----------------------------------------------------------------------===// diff --git a/llvm/lib/MC/MCSubtargetInfo.cpp b/llvm/lib/MC/MCSubtargetInfo.cpp index 540af9a..9e7ea59 100644 --- a/llvm/lib/MC/MCSubtargetInfo.cpp +++ b/llvm/lib/MC/MCSubtargetInfo.cpp @@ -87,10 +87,7 @@ const MCSchedModel &MCSubtargetInfo::getSchedModelForCPU(StringRef CPU) const { ArrayRef SchedModels(ProcSchedModels, ProcDesc.size()); - assert(std::is_sorted(SchedModels.begin(), SchedModels.end(), - [](const SubtargetInfoKV &LHS, const SubtargetInfoKV &RHS) { - return strcmp(LHS.Key, RHS.Key) < 0; - }) && + assert(std::is_sorted(SchedModels.begin(), SchedModels.end()) && "Processor machine model table is not sorted"); // Find entry -- 2.7.4