[scudo][standalone] Code tidying (NFC)
- we have clutter-reducing helpers for relaxed atomics that were barely
used, use them everywhere we can
- clang-format everything with a recent version
Differential Revision: https://reviews.llvm.org/D90649
GitOrigin-RevId: b3420adf5a8491b4c63b54c7750fbbb6368a0a23
diff --git a/memtag.h b/memtag.h
index 519c827..b9f8ccd 100644
--- a/memtag.h
+++ b/memtag.h
@@ -28,9 +28,7 @@
inline uptr untagPointer(uptr Ptr) { return Ptr & ((1ULL << 56) - 1); }
-inline uint8_t extractTag(uptr Ptr) {
- return (Ptr >> 56) & 0xf;
-}
+inline uint8_t extractTag(uptr Ptr) { return (Ptr >> 56) & 0xf; }
#else
@@ -82,7 +80,7 @@
class ScopedDisableMemoryTagChecks {
size_t PrevTCO;
- public:
+public:
ScopedDisableMemoryTagChecks() {
__asm__ __volatile__(".arch_extension mte; mrs %0, tco; msr tco, #1"
: "=r"(PrevTCO));
@@ -190,8 +188,8 @@
2:
)"
- : [ Cur ] "+&r"(RoundOldPtr), [ End ] "+&r"(NewPtr)
- : [ BlockEnd ] "r"(BlockEnd)
+ : [Cur] "+&r"(RoundOldPtr), [End] "+&r"(NewPtr)
+ : [BlockEnd] "r"(BlockEnd)
: "memory");
}
diff --git a/options.h b/options.h
index 3051e8a..2cffc4d 100644
--- a/options.h
+++ b/options.h
@@ -40,9 +40,7 @@
atomic_u32 Val;
public:
- Options load() const {
- return Options{atomic_load(&Val, memory_order_relaxed)};
- }
+ Options load() const { return Options{atomic_load_relaxed(&Val)}; }
void clear(OptionBit Opt) {
atomic_fetch_and(&Val, ~(1U << static_cast<u32>(Opt)),
@@ -54,7 +52,7 @@
}
void setFillContentsMode(FillContentsMode FillContents) {
- u32 Opts = atomic_load(&Val, memory_order_relaxed), NewOpts;
+ u32 Opts = atomic_load_relaxed(&Val), NewOpts;
do {
NewOpts = Opts;
NewOpts &= ~(3U << static_cast<u32>(OptionBit::FillContents0of2));
diff --git a/primary32.h b/primary32.h
index a159a58..d6c294b 100644
--- a/primary32.h
+++ b/primary32.h
@@ -190,7 +190,7 @@
const s32 Interval =
Max(Min(static_cast<s32>(Value), MaxReleaseToOsIntervalMs),
MinReleaseToOsIntervalMs);
- atomic_store(&ReleaseToOsIntervalMs, Interval, memory_order_relaxed);
+ atomic_store_relaxed(&ReleaseToOsIntervalMs, Interval);
return true;
}
// Not supported by the Primary, but not an error either.
@@ -462,8 +462,7 @@
}
if (!Force) {
- const s32 IntervalMs =
- atomic_load(&ReleaseToOsIntervalMs, memory_order_relaxed);
+ const s32 IntervalMs = atomic_load_relaxed(&ReleaseToOsIntervalMs);
if (IntervalMs < 0)
return 0;
if (Sci->ReleaseInfo.LastReleaseAtNs +
diff --git a/primary64.h b/primary64.h
index 1f7ac38..b6c6f32 100644
--- a/primary64.h
+++ b/primary64.h
@@ -191,7 +191,7 @@
const s32 Interval =
Max(Min(static_cast<s32>(Value), MaxReleaseToOsIntervalMs),
MinReleaseToOsIntervalMs);
- atomic_store(&ReleaseToOsIntervalMs, Interval, memory_order_relaxed);
+ atomic_store_relaxed(&ReleaseToOsIntervalMs, Interval);
return true;
}
// Not supported by the Primary, but not an error either.
@@ -470,8 +470,7 @@
}
if (!Force) {
- const s32 IntervalMs =
- atomic_load(&ReleaseToOsIntervalMs, memory_order_relaxed);
+ const s32 IntervalMs = atomic_load_relaxed(&ReleaseToOsIntervalMs);
if (IntervalMs < 0)
return 0;
if (Region->ReleaseInfo.LastReleaseAtNs +
diff --git a/secondary.h b/secondary.h
index da435fd..ca86d2d 100644
--- a/secondary.h
+++ b/secondary.h
@@ -94,7 +94,7 @@
bool EntryCached = false;
bool EmptyCache = false;
const u64 Time = getMonotonicTime();
- const u32 MaxCount = atomic_load(&MaxEntriesCount, memory_order_relaxed);
+ const u32 MaxCount = atomic_load_relaxed(&MaxEntriesCount);
{
ScopedLock L(Mutex);
if (EntriesCount >= MaxCount) {
@@ -121,15 +121,14 @@
s32 Interval;
if (EmptyCache)
empty();
- else if ((Interval = atomic_load(&ReleaseToOsIntervalMs,
- memory_order_relaxed)) >= 0)
+ else if ((Interval = atomic_load_relaxed(&ReleaseToOsIntervalMs)) >= 0)
releaseOlderThan(Time - static_cast<u64>(Interval) * 1000000);
return EntryCached;
}
bool retrieve(uptr Size, LargeBlock::Header **H) {
const uptr PageSize = getPageSizeCached();
- const u32 MaxCount = atomic_load(&MaxEntriesCount, memory_order_relaxed);
+ const u32 MaxCount = atomic_load_relaxed(&MaxEntriesCount);
ScopedLock L(Mutex);
if (EntriesCount == 0)
return false;
@@ -154,8 +153,8 @@
}
bool canCache(uptr Size) {
- return atomic_load(&MaxEntriesCount, memory_order_relaxed) != 0U &&
- Size <= atomic_load(&MaxEntrySize, memory_order_relaxed);
+ return atomic_load_relaxed(&MaxEntriesCount) != 0U &&
+ Size <= atomic_load_relaxed(&MaxEntrySize);
}
bool setOption(Option O, sptr Value) {
@@ -163,17 +162,16 @@
const s32 Interval =
Max(Min(static_cast<s32>(Value), MaxReleaseToOsIntervalMs),
MinReleaseToOsIntervalMs);
- atomic_store(&ReleaseToOsIntervalMs, Interval, memory_order_relaxed);
+ atomic_store_relaxed(&ReleaseToOsIntervalMs, Interval);
return true;
} else if (O == Option::MaxCacheEntriesCount) {
const u32 MaxCount = static_cast<u32>(Value);
if (MaxCount > EntriesArraySize)
return false;
- atomic_store(&MaxEntriesCount, MaxCount, memory_order_relaxed);
+ atomic_store_relaxed(&MaxEntriesCount, MaxCount);
return true;
} else if (O == Option::MaxCacheEntrySize) {
- atomic_store(&MaxEntrySize, static_cast<uptr>(Value),
- memory_order_relaxed);
+ atomic_store_relaxed(&MaxEntrySize, static_cast<uptr>(Value));
return true;
}
// Not supported by the Secondary Cache, but not an error either.
diff --git a/stack_depot.h b/stack_depot.h
index f2f4d95..7968f7e 100644
--- a/stack_depot.h
+++ b/stack_depot.h
@@ -20,7 +20,7 @@
static const u32 R = 24;
u32 H;
- public:
+public:
explicit MurMur2HashBuilder(u32 Init = 0) { H = Seed ^ Init; }
void add(u32 K) {
K *= M;