From 44fcbe719b41aae2b7f23e596b9bc4d9d5296dc1 Mon Sep 17 00:00:00 2001 From: Alex Crichton Date: Wed, 5 Sep 2018 15:37:39 -0700 Subject: [PATCH] [beta]: Remove `#[repr(transparent)]` from atomics Added in #52149 the discussion in #53514 is showing how we may not want to actually add this attribute to the atomic types. While we continue to debate #53514 this commit reverts the addition of the `transparent` attribute before it hits stable. --- src/libcore/sync/atomic.rs | 3 --- 1 file changed, 3 deletions(-) diff --git a/src/libcore/sync/atomic.rs b/src/libcore/sync/atomic.rs index 1e2b18bf9b038..e9d1fb8911504 100644 --- a/src/libcore/sync/atomic.rs +++ b/src/libcore/sync/atomic.rs @@ -124,7 +124,6 @@ pub fn spin_loop_hint() { /// [`bool`]: ../../../std/primitive.bool.html #[cfg(target_has_atomic = "8")] #[stable(feature = "rust1", since = "1.0.0")] -#[repr(transparent)] pub struct AtomicBool { v: UnsafeCell, } @@ -148,7 +147,6 @@ unsafe impl Sync for AtomicBool {} /// This type has the same in-memory representation as a `*mut T`. #[cfg(target_has_atomic = "ptr")] #[stable(feature = "rust1", since = "1.0.0")] -#[repr(transparent)] pub struct AtomicPtr { p: UnsafeCell<*mut T>, } @@ -978,7 +976,6 @@ macro_rules! atomic_int { /// /// [module-level documentation]: index.html #[$stable] - #[repr(transparent)] pub struct $atomic_type { v: UnsafeCell<$int_type>, }