[compiler-rt] [rtsan][NFC] Make Uninitialzed state explicit (PR #109856)

Chris Apple via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 24 13:08:11 PDT 2024


https://github.com/cjappl created https://github.com/llvm/llvm-project/pull/109856

Follow on to #109830 

There should be no functional change, as enums start at 0 anyway. This just makes the code more readable and prevents any future bugs.

>From fe8c0db71d17ddd3185032b766bdc663ded89000 Mon Sep 17 00:00:00 2001
From: Chris Apple <cja-private at pm.me>
Date: Tue, 24 Sep 2024 12:28:57 -0700
Subject: [PATCH] [rtsan][NFC] Make Uninitialzed state explicit, don't rely on
 0

---
 compiler-rt/lib/rtsan/rtsan.cpp | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/compiler-rt/lib/rtsan/rtsan.cpp b/compiler-rt/lib/rtsan/rtsan.cpp
index e6d2481b2c2a3d..84e4b8fae1e2fa 100644
--- a/compiler-rt/lib/rtsan/rtsan.cpp
+++ b/compiler-rt/lib/rtsan/rtsan.cpp
@@ -31,7 +31,8 @@ enum class InitializationState : u8 {
 } // namespace
 
 static StaticSpinMutex rtsan_inited_mutex;
-static atomic_uint8_t rtsan_initialized = {0};
+static atomic_uint8_t rtsan_initialized = {
+    static_cast<u8>(InitializationState::Uninitialized)};
 
 static void SetInitializationState(InitializationState state) {
   atomic_store(&rtsan_initialized, static_cast<u8>(state),



More information about the llvm-commits mailing list