[libc-commits] [libc] [libc] Replace usage of GPU helpers with ones from 'gpuintrin.h' (PR #116454)

Shilei Tian via libc-commits libc-commits at lists.llvm.org
Fri Nov 15 20:14:30 PST 2024


================
@@ -67,18 +76,18 @@ template <bool Invert> struct Process {
   LIBC_INLINE ~Process() = default;
 
   uint32_t port_count = 0;
-  cpp::Atomic<uint32_t> *inbox = nullptr;
-  cpp::Atomic<uint32_t> *outbox = nullptr;
+  uint32_t *inbox = nullptr;
----------------
shiltian wrote:

Not sure which approach is better. With `std::atomic<>`, it’s clear that the variable is an atomic number. However, as a simple integer, its usage may not be safe if someone forgets to update it using atomic operations.

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


More information about the libc-commits mailing list