commit 29cd444602d2153326db31b78d12b2a0f6c49e70
author David Benjamin <davidben@google.com> Sat Aug 30 17:23:59 2025 -0400
committer Boringssl LUCI CQ <boringssl-scoped@luci-project-accounts.iam.gserviceaccount.com> Wed Sep 03 22:10:39 2025 -0700
tree c384666d296e544fcf988e7cecab7669a717f9e3
parent 4bc3b8427402f1d142a7f9aeb1a4e418775381b0

Remove stale static asserts about atomics

After the switch to C++, and
https://boringssl-review.googlesource.com/c/boringssl/+/79569, we no
longer depend on atomics being equivalently sized and aligned with their
underlying types.

Change-Id: I633c891a66539f7feaeab941a18172e8ac76dfb1
Reviewed-on: https://boringssl-review.googlesource.com/c/boringssl/+/81827
Reviewed-by: Adam Langley <agl@google.com>
Commit-Queue: David Benjamin <davidben@google.com>

crypto/internal.h

diff --git a/crypto/internal.h b/crypto/internal.h
index 82ba64d..c041c64 100644
--- a/crypto/internal.h
+++ b/crypto/internal.h
@@ -532,8 +532,6 @@
 
 using CRYPTO_atomic_u32 = std::atomic<uint32_t>;
 
-static_assert(sizeof(CRYPTO_atomic_u32) == sizeof(uint32_t));
-
 inline uint32_t CRYPTO_atomic_load_u32(const CRYPTO_atomic_u32 *val) {
   return val->load(std::memory_order_seq_cst);
 }
@@ -572,12 +570,6 @@
 
 #endif
 
-// See the comment in the |__cplusplus| section above.
-static_assert(sizeof(CRYPTO_atomic_u32) == sizeof(uint32_t),
-              "CRYPTO_atomic_u32 does not match uint32_t size");
-static_assert(alignof(CRYPTO_atomic_u32) == alignof(uint32_t),
-              "CRYPTO_atomic_u32 does not match uint32_t alignment");
-
 
 // Reference counting.