roc-lang/roc
1
1
Fork 0
mirror of https://github.com/roc-lang/roc.git synced 2024-11-10 18:08:55 +03:00
Code Issues Packages Projects Releases Wiki Activity

Atomic ordering for debug counter is stricter than nesscessary.

Browse Source
This commit is contained in:
Ross Smyth 2023-01-30 11:10:18 -05:00
parent 8afdc820a6
commit 4698608c4d
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
crates/compiler/types/src/types.rs
View File

@ -1786,7 +1786,9 @@ static mut TYPE_CLONE_COUNT: std::sync::atomic::AtomicUsize =
pub fn get_type_clone_count() -> usize {
if cfg!(debug_assertions) {
unsafe { TYPE_CLONE_COUNT.load(std::sync::atomic::Ordering::SeqCst) }
// A global counter just needs relaxed, and nothing relies upon this atomic for any
// happens-before relationships.
unsafe { TYPE_CLONE_COUNT.load(std::sync::atomic::Ordering::Relaxed) }
} else {
0
}
@ -1796,7 +1798,7 @@ impl Clone for Type {
fn clone(&self) -> Self {
#[cfg(debug_assertions)]
unsafe {
TYPE_CLONE_COUNT.fetch_add(1, std::sync::atomic::Ordering::SeqCst)
TYPE_CLONE_COUNT.fetch_add(1, std::sync::atomic::Ordering::Relaxed)
};
match self {