caio/clang-p2996
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[lld] Use std::tie to implement comparison operators (NFC) (#143726)

Browse Source 
std::tie facilitates lexicographical comparisons through std::tuple's
built-in operator< and operator>.
This commit is contained in:
Kazu Hirata
2025-06-11 12:50:19 -07:00
committed by GitHub
parent e266d6a5da
commit c1d21f4434
2 changed files with 5 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
lld/ELF/SyntheticSections.cpp
View File

@@ -1939,11 +1939,8 @@ bool AndroidPackedRelocationSection<ELFT>::updateAllocSize(Ctx &ctx) {
// For Rela, we also want to sort by r_addend when r_info is the same. This
// enables us to group by r_addend as well.
llvm::sort(nonRelatives, [](const Elf_Rela &a, const Elf_Rela &b) {
if (a.r_info != b.r_info)
return a.r_info < b.r_info;
if (a.r_addend != b.r_addend)
return a.r_addend < b.r_addend;
return a.r_offset < b.r_offset;
return std::tie(a.r_info, a.r_addend, a.r_offset) <
std::tie(b.r_info, b.r_addend, b.r_offset);
});
// Group relocations with the same r_info. Note that each group emits a group

8
lld/MachO/UnwindInfoSection.cpp
View File

@@ -535,11 +535,9 @@ void UnwindInfoSectionImpl::finalize() {
llvm::sort(commonEncodings,
[](const std::pair<compact_unwind_encoding_t, size_t> &a,
const std::pair<compact_unwind_encoding_t, size_t> &b) {
if (a.second == b.second)
// When frequencies match, secondarily sort on encoding
// to maintain parity with validate-unwind-info.py
return a.first > b.first;
return a.second > b.second;
// When frequencies match, secondarily sort on encoding
// to maintain parity with validate-unwind-info.py
return std::tie(a.second, a.first) > std::tie(b.second, b.first);
});
// Truncate the vector to 127 elements.