[llvm] [BOLT] Use std::tie to Implment operator< (NFC) (PR #139404)
via llvm-commits
llvm-commits at lists.llvm.org
Sat May 10 12:48:35 PDT 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-bolt
Author: Kazu Hirata (kazutakahirata)
<details>
<summary>Changes</summary>
---
Full diff: https://github.com/llvm/llvm-project/pull/139404.diff
3 Files Affected:
- (modified) bolt/include/bolt/Passes/FrameAnalysis.h (+2-3)
- (modified) bolt/include/bolt/Passes/PAuthGadgetScanner.h (+1-3)
- (modified) bolt/include/bolt/Profile/DataReader.h (+1-7)
``````````diff
diff --git a/bolt/include/bolt/Passes/FrameAnalysis.h b/bolt/include/bolt/Passes/FrameAnalysis.h
index 44b54d4ed45d4..d71c338bdcc37 100644
--- a/bolt/include/bolt/Passes/FrameAnalysis.h
+++ b/bolt/include/bolt/Passes/FrameAnalysis.h
@@ -10,6 +10,7 @@
#define BOLT_PASSES_FRAMEANALYSIS_H
#include "bolt/Passes/StackPointerTracking.h"
+#include <tuple>
namespace llvm {
namespace bolt {
@@ -53,9 +54,7 @@ struct ArgInStackAccess {
uint8_t Size;
bool operator<(const ArgInStackAccess &RHS) const {
- if (StackOffset != RHS.StackOffset)
- return StackOffset < RHS.StackOffset;
- return Size < RHS.Size;
+ return std::tie(StackOffset, Size) < std::tie(RHS.StackOffset, RHS.Size);
}
};
diff --git a/bolt/include/bolt/Passes/PAuthGadgetScanner.h b/bolt/include/bolt/Passes/PAuthGadgetScanner.h
index ee69337637da3..66251e195822c 100644
--- a/bolt/include/bolt/Passes/PAuthGadgetScanner.h
+++ b/bolt/include/bolt/Passes/PAuthGadgetScanner.h
@@ -43,9 +43,7 @@ struct MCInstInBBReference {
return BB == RHS.BB && BBIndex == RHS.BBIndex;
}
bool operator<(const MCInstInBBReference &RHS) const {
- if (BB != RHS.BB)
- return BB < RHS.BB;
- return BBIndex < RHS.BBIndex;
+ return std::tie(BB, BBIndex) < std::tie(RHS.BB, RHS.BBIndex);
}
operator MCInst &() const {
assert(BB != nullptr);
diff --git a/bolt/include/bolt/Profile/DataReader.h b/bolt/include/bolt/Profile/DataReader.h
index 314dcc9115586..4765eb28d53ca 100644
--- a/bolt/include/bolt/Profile/DataReader.h
+++ b/bolt/include/bolt/Profile/DataReader.h
@@ -90,13 +90,7 @@ struct BranchInfo {
}
bool operator<(const BranchInfo &RHS) const {
- if (From < RHS.From)
- return true;
-
- if (From == RHS.From)
- return (To < RHS.To);
-
- return false;
+ return std::tie(From, To) < std::tie(RHS.From, RHS.To);
}
/// Merges branch and misprediction counts of \p BI with those of this object.
``````````
</details>
https://github.com/llvm/llvm-project/pull/139404
More information about the llvm-commits
mailing list