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

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


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-compiler-rt-sanitizer

Author: Chris Apple (cjappl)

<details>
<summary>Changes</summary>

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.

---
Full diff: https://github.com/llvm/llvm-project/pull/109856.diff


1 Files Affected:

- (modified) compiler-rt/lib/rtsan/rtsan.cpp (+2-1) 


``````````diff
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),

``````````

</details>


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


More information about the llvm-commits mailing list