[compiler-rt] [rtsan] Make sure rtsan gets initialized on mac (PR #100188)
Chris Apple via llvm-commits
llvm-commits at lists.llvm.org
Sun Aug 11 14:00:15 PDT 2024
cjappl wrote:
Two very small updates since last review:
1. Matching llvm style "If you use a braced initializer list when initializing a variable, use an equals before the open curly brace:"
```cpp
- static atomic_uint8_t rtsan_initialized{0};
+ static atomic_uint8_t rtsan_initialized = {0};
```
2. Got rid of the now-unused `IsInitialized` in favor of the `__rtsan_is_initialized` method, this was cruft from an earlier version of this review.
These two changes may be reviewed in the latest two commits, if anyone wants to see them in isolation.
I also rebased on latest main. I will likely merge this early next week
https://github.com/llvm/llvm-project/pull/100188
More information about the llvm-commits
mailing list