[compiler-rt] [NFC] [scudo] Move static_assert to class it concerns (PR #84245)
via llvm-commits
llvm-commits at lists.llvm.org
Wed Mar 6 14:23:55 PST 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-compiler-rt-sanitizer
Author: Florian Mayer (fmayer)
<details>
<summary>Changes</summary>
---
Full diff: https://github.com/llvm/llvm-project/pull/84245.diff
1 Files Affected:
- (modified) compiler-rt/lib/scudo/standalone/combined.h (+5-4)
``````````diff
diff --git a/compiler-rt/lib/scudo/standalone/combined.h b/compiler-rt/lib/scudo/standalone/combined.h
index fa6077384d9826..5b25a7a6c2a916 100644
--- a/compiler-rt/lib/scudo/standalone/combined.h
+++ b/compiler-rt/lib/scudo/standalone/combined.h
@@ -1081,6 +1081,11 @@ class Allocator {
// An array of Size (at least one) elements of type Entry is immediately
// following to this struct.
};
+ static_assert(sizeof(AllocationRingBuffer) %
+ alignof(typename AllocationRingBuffer::Entry) ==
+ 0,
+ "invalid alignment");
+
// Pointer to memory mapped area starting with AllocationRingBuffer struct,
// and immediately followed by Size elements of type Entry.
atomic_uptr RingBufferAddress = {};
@@ -1596,10 +1601,6 @@ class Allocator {
atomic_store(&RingBufferAddress, reinterpret_cast<uptr>(RB),
memory_order_release);
- static_assert(sizeof(AllocationRingBuffer) %
- alignof(typename AllocationRingBuffer::Entry) ==
- 0,
- "invalid alignment");
}
void unmapRingBuffer() {
``````````
</details>
https://github.com/llvm/llvm-project/pull/84245
More information about the llvm-commits
mailing list